Skip to content

[codex] Centralize CLI argument parsing - #28

Draft
rushillshah wants to merge 1 commit into
mainfrom
codex/shared-cli-args
Draft

[codex] Centralize CLI argument parsing#28
rushillshah wants to merge 1 commit into
mainfrom
codex/shared-cli-args

Conversation

@rushillshah

Copy link
Copy Markdown
Owner

Scope

  • move duplicated CLI option parsing into lib/args.js
  • use the shared parser in the main CLI, terminal UI, and graph review entrypoints
  • preserve repeated options consistently and add focused parser coverage

Partially addresses #11.

Validation

Risk

Low. The shared parser preserves the main CLI behavior and makes repeated options consistent in the UI and graph review entrypoints.

@rushillshah

Copy link
Copy Markdown
Owner Author

Maintenance blocker (2026-07-19): focused parser tests, npm run check, CLI smokes, and npm pack --dry-run pass. Both CI jobs fail only on the same four date-sensitive baseline tests tracked by #23 and fixed in green draft PR #25 (67 passed / 4 failed here). Next action: review and merge #25, then rerun this PR CI; keep #28 draft until checks pass and review is complete.

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