AntFleet

Disagreement · 9e289239-openai-3

Misleading comment: Next.js App Router searchParams is not a Promise

solo GPT-5
repo e24ef98c·PR #10·reviewed 1 week ago

GPT-5 finding

Misleading comment: Next.js App Router searchParams is not a Promise

lowdocs-gaphigh
  • apps/web/app/receipts/page.tsx
In Next.js App Router, searchParams is a plain object, not a Promise. Awaiting a non-Promise value is a no-op in JS and will work, but the comment is inaccurate and the Promise typing could propagate confusion or copy/paste into other components.

Recommendation

Correct the comment and type to `searchParams: SearchParams` and drop the unnecessary await. This avoids misleading future edits and reduces cognitive load.

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 →
← back to all disagreementsview public receiptssee unanimous findings + anatomies →
Tweet ↗