Skip to content

[codex] fix CLI validation and repo hygiene#8

Merged
Hanjo92 merged 2 commits into
mainfrom
codex/fix-repo-inspection-followups
Jun 5, 2026
Merged

[codex] fix CLI validation and repo hygiene#8
Hanjo92 merged 2 commits into
mainfrom
codex/fix-repo-inspection-followups

Conversation

@Hanjo92
Copy link
Copy Markdown
Owner

@Hanjo92 Hanjo92 commented Jun 5, 2026

Summary

  • Add regression coverage for missing CLI option values and invalid --repo-path inputs.
  • Fix CLI argument validation and repo path handling so failures are concise user-facing errors.
  • Add CI, GitHub templates, release/community docs, lockfile, and include docs/ in npm package contents.

Root Cause

  • The manual argument parser consumed the next token without checking whether it was another flag.
  • Repo context ingestion called readdirSync before validating that an explicit repo path was a directory.

Verification

  • npm ci
  • npm test
  • npm run smoke
  • npm pack --dry-run

Addresses #7.

@Hanjo92 Hanjo92 marked this pull request as ready for review June 5, 2026 08:16
@Hanjo92 Hanjo92 merged commit b47d7dc into main Jun 5, 2026
2 checks passed
@Hanjo92 Hanjo92 deleted the codex/fix-repo-inspection-followups branch June 5, 2026 08:17
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