AntFleet

Disagreement · be00b9c6-openai-1

Notification link hard-codes main branch; breaks when default branch is not main

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

GPT-5 finding

Notification link hard-codes main branch; breaks when default branch is not main

mediummaintainabilityhigh
  • skills/fork-skill-digest/SKILL.md:277
  • skills/fork-skill-digest/SKILL.md:280-281
Hard-coding main will 404 for repos that use a different default branch (e.g., master, trunk). This degrades notification UX and requires per-repo edits.

Recommendation

Use a branch-agnostic ref in the URL, e.g., /blob/HEAD/articles/fork-skill-digest-${today}.md, or detect the default branch via GITHUB_REF_NAME or gh api repos/$GITHUB_REPOSITORY and interpolate it.

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 →