AntFleet

Disagreement · a255f8b5-openai-0

Forks will publish a status page with wrong links/canonical URLs due to hardcoded upstream site/repo, contradicting “no extra config” claim

mismatch
repo 6f7fc663·PR #6·reviewed 1 week ago

Primary finding

Forks will publish a status page with wrong links/canonical URLs due to hardcoded upstream site/repo, contradicting “no extra config” claim

mediumdocs-gaphigh
  • docs/status.md:15
  • docs/_config.yml:4
  • docs/_config.yml:13
  • docs/_config.yml:51-52
The status page claims every fork gets a public dashboard with no extra config, but the site config hardcodes the original site URL and GitHub repo links. On forks or renames, canonical URLs and footer/social links will point to the upstream project, confusing users and undermining the claim of zero configuration. This is an actionable docs/config gap with concrete impact on published forks.

Recommendation

Make the Jekyll site fork-agnostic: - Use GitHub Pages metadata instead of hardcoded URLs: set url: "" and baseurl: "" and rely on the relative_url/absolute_url filters with site.github.url/baseurl, or explicitly set url: {{ site.github.url }} and baseurl: {{ site.github.baseurl }} via config defaults. - Replace hardcoded social_links user_url with {{ site.github.repository_url }} (or omit if the theme supports dynamic repo links). - Update the status page text to qualify that forks must enable GitHub Pages (and Actions) or remove the “no extra config” claim. These changes keep links and canonicals correct across forks and repo renames.

Counterpart finding

header_pages references pages that do not exist in this slice

lowdocs-gapmedium
  • docs/_config.yml:17-23
header_pages lists index.md, articles.md, activity.md, memory.md, skills.md alongside status.md, but only status.md is being added in this slice. If the other pages aren't present in docs/, minima's header silently omits them — not fatal, but the nav will look broken on forks/clean checkouts that the status page explicitly advertises ('Fork the repo and your copy inherits this page automatically'). Worth verifying these pages exist in the repo outside the diff.

Recommendation

Either ensure the referenced pages exist in docs/ as part of this PR or trim header_pages to only the pages that ship.

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.