AntFleet

Disagreement · 8dbec532-anthropic-3

Conflicting auto-detect order between SKILL.md text and env var docs

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

Opus finding

Conflicting auto-detect order between SKILL.md text and env var docs

lowdocs-gapmedium
  • opensea/opensea-wallet/SKILL.md:96
  • opensea/opensea-wallet/SKILL.md:26-30
SKILL.md lists Privy as 'default provider' and gives an auto-detect order that places Fireblocks ahead of Turnkey, but the supported-providers table earlier in the same file lists Privy/Turnkey/Fireblocks/Bankr/Private Key in a different order. There is no canonical source-of-truth file in the PR establishing detection order, so a user inferring behavior from the table would expect Turnkey before Fireblocks. If the implementation diverges from this prose ordering, agents will silently pick up the 'wrong' provider when multiple credentials are present (a common operator mistake). This is not necessarily wrong, but the doc itself does not internally agree.

Recommendation

State the detection order once, near the providers table, and either reference the implementation file or pin a test that asserts the order. Ensure both the auto-detect prose and the references/wallet-setup.md are consistent.

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 →