Skip to content

fix(ingest): accept --flag=value equals form in the CLI parser#37

Merged
mcp-tool-shop merged 1 commit into
mainfrom
fix/ingest-equals-form-args
Jun 2, 2026
Merged

fix(ingest): accept --flag=value equals form in the CLI parser#37
mcp-tool-shop merged 1 commit into
mainfrom
fix/ingest-equals-form-args

Conversation

@mcp-tool-shop
Copy link
Copy Markdown
Contributor

Self-audit follow-up. The ingest CLI parsed only --flag value; dogfood-swarm's commands/persist.js invokes it as --provenance=stub --file=... (equals form), so --provenance read as missing → "flag is required" → swarm persist --ingest silently failed on every platform.

Fix: parse --flag=value alongside --flag value for --provenance / --file / --payload. New equals-form-args.test.js pins both forms + a control. Ingest suite 120 → 123, npm run verify green.

🤖 Generated with Claude Code

The ingest CLI parsed only the space form (`--provenance value`). But
dogfood-swarm's commands/persist.js invokes it via execSync as
`node run.js --provenance=stub --file=...` (equals form), which fell through
to positionalArgs — `provenanceMode` stayed null and the CLI died with
"--provenance flag is required", silently breaking `swarm persist --ingest`
on every platform.

Parse `--flag=value` alongside `--flag value` for --provenance / --file /
--payload. Surfaced by the persist exit-code test in
dogfood-swarm/meta-amendB-operator-output.test.js during the self-audit;
that test stayed green (the malformed run still exits non-zero), but the
ingest never actually ran. New equals-form-args.test.js pins both forms +
a control that proves the missing-flag error still fires when omitted.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mcp-tool-shop mcp-tool-shop merged commit 7bd24d6 into main Jun 2, 2026
4 checks passed
@mcp-tool-shop mcp-tool-shop deleted the fix/ingest-equals-form-args branch June 2, 2026 21:29
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