Skip to content

Fix boolean CLI flags swallowing commands and search terms - #49

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
anubhabx:codex/fix-cli-boolean-arguments
Sep 9, 2026
Merged

Fix boolean CLI flags swallowing commands and search terms#49
ralyodio merged 1 commit into
profullstack:masterfrom
anubhabx:codex/fix-cli-boolean-arguments

Conversation

@anubhabx

@anubhabx anubhabx commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Title: Fix boolean CLI flags swallowing commands and search terms

agenticjobs search --remote rust --agents developer currently loses both search words, and agenticjobs --json search rust parses rust as the command. The parser treats every option as potentially taking a string value.

Recognize the CLI's boolean options so they leave subsequent positional arguments intact. Preserve explicit boolean values (--remote false, --remote=false), string-valued options, repeated application answers, and --.

Validation:

  • Added four regression tests; the two bug reproductions failed before the fix, and all four pass afterward.
  • Type checking passes.
  • 120 tests pass when excluding api.test.ts and sw.test.ts.
  • The complete test run is not green in this Windows environment: the API suite reports no app, and the service-worker asset hash differs. Neither suite is modified by this PR.

Prepared with Codex assistance for Anubhab. This is a contribution toward the bug-fix reward advertised at https://ugig.net/gigs/8b2a21db-ce15-433f-a73d-68f2e3349e87 . Please confirm whether this PR qualifies for the listed $0.25 reward and the supported payout route. No reward is being represented as approved or paid.

Formal uGig application: 43ba865e-5a85-434a-b709-ddce36d95bb2 (pending at the listed $0.25 rate).

@ralyodio
ralyodio merged commit e028bb3 into profullstack:master Sep 9, 2026
4 checks passed
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.

2 participants