Add @appdmg/cli package#13
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fc8e9088ee
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| return 0 | ||
| } | ||
|
|
||
| const shouldWrite = !options.quiet || options.verbose |
There was a problem hiding this comment.
Separate verbose errors from quiet progress/success output
--verbose is documented as controlling error detail, while --quiet is documented to suppress progress/success/error output; however, shouldWrite is computed as !options.quiet || options.verbose and reused for progress and success paths, so --quiet --verbose on a successful run still emits progress and completion text. This breaks the stated CLI behavior for users who want quiet successful runs but verbose failure diagnostics.
Useful? React with 👍 / 👎.
Summary
Adds the first-stage
@appdmg/clipackage with the newappdmg-cliexecutable.packages/clias a separate Node.js 24 CommonJS packageappdmg-clibin, with noappdmgcompatibility alias<json-path> <dmg-path>node:util.parseArgs--help,--version,--quiet, and--verbose@appdmg/appdmgwith exact version1.0.0The CLI package lock pins the merged GitHub tarballs for
@appdmg/appdmgand helper packages until the scoped npm release-chain task publishes them.Verification
Library package:
npm cinpm testnpm audit --audit-level=moderatenpm ls --omit=dev --allnpm pack --dry-runnode --check ...for library sources/testsCLI package:
npm cinpm testnpm audit --audit-level=moderatenpm ls --omit=dev --allnpm pack --dry-runnode --check bin/appdmg-cli.jsnode --check lib/args.jsnode --check lib/main.jsnode --check lib/render-progress.jsnode --check test/args.test.jsnode --check test/main.test.jsCloses #7