Skip to content

feat: add --output / -o flag to zeroshot run#25

Merged
mark-dingwall merged 2 commits into
mainfrom
feat/output-flag
Apr 9, 2026
Merged

feat: add --output / -o flag to zeroshot run#25
mark-dingwall merged 2 commits into
mainfrom
feat/output-flag

Conversation

@mark-dingwall

Copy link
Copy Markdown
Owner

Summary

  • Adds -o, --output <file> flag to zeroshot run to control the output filename for reports and documents
  • Propagates ZEROSHOT_OUTPUT_FILE env var through daemon mode to write-review-report.js and assemble-doc.js
  • Resolves relative paths to absolute before daemon spawn (daemon changes CWD to git root)
  • Auto-appends .md when no extension is provided
  • Also includes npm audit fix for non-breaking dependency vulnerabilities and refreshes .audit-ignore

Test plan

  • All 1566 existing tests pass
  • New unit tests for env var usage, path normalization, daemon env propagation
  • Lint passes (0 errors)
  • Manual: zeroshot run "test" --config docs-review-trace -o my-report → produces my-report.md
  • Manual: zeroshot run "test" --config docs-review-trace -o /tmp/review.md → produces /tmp/review.md
  • Manual: default behavior unchanged when -o omitted

🤖 Generated with Claude Code

mark-dingwall and others added 2 commits April 9, 2026 10:23
Propagates ZEROSHOT_OUTPUT_FILE env var through daemon mode to
write-review-report.js and assemble-doc.js. Resolves relative paths
to absolute before daemon spawn (daemon changes CWD to git root).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…vulns

Non-breaking fixes only. Remaining advisories (lodash via blessed-contrib,
serialize-javascript via mocha) require breaking changes and are tracked
in .audit-ignore.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mark-dingwall mark-dingwall merged commit e2f7a7d into main Apr 9, 2026
6 of 7 checks passed
@mark-dingwall mark-dingwall deleted the feat/output-flag branch April 9, 2026 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant