AntFleet

Disagreement · d121d4bf-openai-0

Script docstrings and usage mention .sh instead of .py, causing user invocation confusion

mismatch
repo a16d2030·PR #4·reviewed 1 week ago

Primary finding

Script docstrings and usage mention .sh instead of .py, causing user invocation confusion

mediumdocs-gaphigh
  • symbiosis/scripts/symbiosis-quote.py:2-6
  • symbiosis/scripts/symbiosis-swap.py:2-7
Both scripts are Python (.py) but their embedded help strings and examples instruct users to run .sh files. This is likely to cause immediate failures and confusion when following the on-script instructions.

Recommendation

Update the docstrings in both scripts to reference .py filenames (symbiosis-quote.py and symbiosis-swap.py) in Usage and Example lines.

Counterpart finding

Docstring usage strings reference non-existent .sh wrappers

lowdocs-gaphigh
  • symbiosis/scripts/symbiosis-quote.py:4-6
  • symbiosis/scripts/symbiosis-swap.py:4-7
  • symbiosis/SKILL.md:36-46
The Python files are named `.py` and SKILL.md correctly invokes them as `.py`, but the `__doc__` printed when arg-count validation fails tells the user to run `./symbiosis-quote.sh` / `./symbiosis-swap.sh`. There are no `.sh` wrappers in the slice. Users who misuse the script will be told to run a non-existent file. This is a misleading message produced exactly at the error path, which is the worst place for it.

Recommendation

Update docstrings to say `symbiosis-quote.py` / `symbiosis-swap.py`.

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.

← back to all disagreementsview public receiptssee unanimous findings + anatomies →
Tweet ↗