AntFleet

Disagreement · cf4e74ac-anthropic-6

cmd_list 'time_left' suppresses zero-second windows and rounds bullPct lossily

solo Opus
repo a16d2030·PR #1·reviewed 1 week ago

Opus finding

cmd_list 'time_left' suppresses zero-second windows and rounds bullPct lossily

lowbugmedium
  • agenticbets/scripts/agenticbets.py:122-128
  • agenticbets/scripts/agenticbets.py:120-121
(1) `secondsToLock and >0` treats 0 as falsy — at exactly t=0 the display shows '-', conflating 'just locked' with 'not open'. The API doc says null means not-open, so the existence check should be `is not None`. (2) `bear = 100 - round(bullPct)` ignores the actual bearPct from the API and can show e.g. UP 50%/DOWN 50% when the real split is 49.4/50.6 due to rounding. For a strategy tool this is misleading; use the real values.

Recommendation

Use `m['secondsToLock'] is not None and m['secondsToLock'] >= 0`. Use real `bearPct` from the API instead of `100 - round(bullPct)`.

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.

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