AntFleet

Disagreement · e488cbca-openai-1

--json mode emits non-JSON headers and summary lines, breaking machine parsing

solo GPT-5
repo 6f7fc663·PR #29·reviewed 1 week ago

GPT-5 finding

--json mode emits non-JSON headers and summary lines, breaking machine parsing

mediumapi-contracthigh
  • skills/skill-security-scan/scan.sh:293-296
  • skills/skill-security-scan/scan.sh:302-311
Even when JSON_OUTPUT is true, the script prints human-readable headers and summaries before the JSON. Tools expecting clean JSON from stdout will fail to parse.

Recommendation

When JSON_OUTPUT is true, suppress all non-JSON output and write only the JSON array to stdout. Optionally add a --quiet flag for text mode. Consider writing human-readable output to stderr if needed.

Other reviewer

The other reviewer flagged nothing in this file/line range.

Why this didn't post

This finding didn't meet AntFleet's unanimous agreement threshold. Both frontier models review every PR independently; only findings they both flag with the same severity and category are posted to the PR. This one fell through.

read the methodology →

From the same review

These findings passed the unanimous gate on the same PR review. The disagreement above was filtered out; the findings below were posted.