Add explicit debug boolean input to always enable debug logs - #41
Merged
Conversation
Add a new `debug` action input (default false) that always enables Oz
debug logging (`--debug`), independent of GitHub's step-debug signal.
Debug logging now activates when `debug` is true OR `core.isDebug()` is
true, preserving the existing GitHub-requested-debug behavior.
- action.yml: new `debug` boolean input (default false)
- src/index.ts: read core.getBooleanInput('debug'); pass --debug when
debug || core.isDebug()
- src/index.test.ts: cover debug true/false paths and that core.isDebug()
still forces --debug on; assert --debug appears exactly once when both
are on (4 new tests, 27 total)
- README.md: document the debug input in a new Debug Logging section
- dist/index.js: rebuilt bundle
- package-lock.json: reconcile @emnapi/wasi-threads 1.2.2 -> 1.2.3
(pre-existing drift from CVE-2026-13149 fix that broke `npm ci`;
required for the repo's documented checks to run)
Closes REMOTE-2331
Co-Authored-By: Warp <agent@warp.dev>
The lockfile was missing top-level @emnapi/core@1.11.3 and @emnapi/runtime@1.11.3 (optional peers of @napi-rs/wasm-runtime), so `npm ci` failed with EUSAGE on Node 24 and every required CI job died at the install step. This drift is pre-existing on main (introduced by the hand-edited lockfile in the CVE-2026-13149 fix) and also affected this branch. Regenerated the lockfile with `npm install --package-lock-only` using the repo's pinned toolchain (Node 24.4.0 per .node-version, npm 11.4.2): adds the two missing packages and reconciles stale `peer` markers. No dependency version changes beyond the additions. Verified on Node 24.4.0: npm ci, npm run format:check, npm run lint, npm test (27 passed), npm run build (dist and generated workflows unchanged). Co-Authored-By: Oz <oz-agent@warp.dev>
bnavetta
approved these changes
Jul 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds an explicit
debugboolean input to the Oz Agent GitHub Action so users can always enable Oz debug logging, independent of GitHub's step-debug signal.Previously, the action enabled
--debugonly when GitHub requested it (viacore.isDebug(), which reflects the runner's step-debug signal). This adds a newdebuginput (defaultfalse) and enables debug logging whendebugistrueORcore.isDebug()istrue— purely additive, preserving the existing GitHub-requested-debug behavior.action.yml: newdebugboolean input (defaultfalse)src/index.ts: readcore.getBooleanInput('debug'); pass--debugwhendebug || core.isDebug()src/index.test.ts: 4 new regression tests covering all combinations of thedebuginput andcore.isDebug()(see Verification)README.md: new "Debug Logging" section documenting the inputdist/index.js+dist/index.js.map: rebuilt bundle (this repo ships a compileddist/)package-lock.json: repair pre-existing lockfile drift (from the CVE-2026-13149 fix in fix: update brace-expansion to resolve CVE-2026-13149 #39) that brokenpm cionmainand on this branch — see Rework changes belowVerification
This is a headless GitHub Action change (no UI), verified in code per the repo's documented checks.
Regression tests (
src/index.test.ts, 4 new — 27 total passing):omits --debug when debug is false and GitHub step-debug is off— baseline behavior preservedadds --debug when the debug input is true even without GitHub step-debug— the new behavior (this test fails before the fix: oldif (core.isDebug())returned false → no--debug)still adds --debug from GitHub step-debug when the debug input is false— existing GitHub-requested-debug behavior preservedadds --debug exactly once when both debug input and GitHub step-debug are on— no duplicate flagRepo checks (all passing on Node 24.4.0, the version pinned in
.node-versionand used by CI):npm ci→ clean install, 333 packagesnpm test→ 27 passed (23 baseline + 4 new)npm run lint→ cleannpm run format:check→ cleannpm run build→dist/and generated workflows unchanged (check-dist clean)Rework changes
Rework cycle 1 — addressed the code-review finding that all four required CI jobs failed at
npm ci.package-lock.json):npm cifails on Node 24 — fixed. The lockfile was missing the top-level@emnapi/core@1.11.3and@emnapi/runtime@1.11.3entries (optional peers of@napi-rs/wasm-runtime), sonpm ciaborted withEUSAGE … Missing: @emnapi/core@1.11.3 from lock file. This drift is pre-existing onmain(introduced by the hand-edited lockfile in fix: update brace-expansion to resolve CVE-2026-13149 #39); the earlier partial patch on this branch fixed only@emnapi/wasi-threads. The lockfile was regenerated in full withnpm install --package-lock-onlyusing the repo's pinned toolchain (Node 24.4.0 / npm 11.4.2), which adds the two missing packages and reconciles stalepeermarkers. No dependency versions changed beyond those additions.npm ci,npm run format:check,npm run lint,npm test(27 passed),npm run build(no diff indist/,.github/workflows/, orconsumer-workflows/). All four required CI jobs are green on the PR.Originating thread: https://warpdev.slack.com/archives/C0BDQDW8V5E/p1784974804290319