docs(evidence): re-verify broken external links finding 78fcaed682fa against current main and live - #125
Conversation
…against current main and live
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe evidence document now records current-head checks, passing tests, live ChangesExternal link re-verification
Estimated code review effort: 1 (Trivial) | ~3 minutes Mergeability Score: ⚪ Minimal · up to This localized documentation-only change is merge-ready after normal checks and review; no actionable merge-blocking risk remains. Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d2c3369c40
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| `git log -p 8b42e0a..origin/main -- public/audit.html evidence-fixtures/ai-search/` | ||
| is empty, and the "External citation links (dogfood 78fcaed682fa)" guard in |
There was a problem hiding this comment.
Running this query with the stated head 18128e8 is not empty: it returns 2ae7504, d4a2c30, and 9302611, which changed evidence-fixtures/ai-search/controlled-questions.json and/or public/audit.html. This makes the receipt's reproducible history evidence false even if none of those diffs changed an external citation URL; narrow the command to the citation data or describe the returned non-citation changes accurately.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cf53c6e9d4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| errors, no page errors, and all 15 rendered external anchor links resolved | ||
| HTTP 200 with redirects followed (the same 12 unique URLs as the closeout: |
There was a problem hiding this comment.
When this receipt is applied to its parent ad9cee3, the embedded evidence bundle already contains the merged 2026-08-09 rerun from 8606b0c, which renders 24 external anchors representing 22 unique URLs—not the 15/12 measured here. Consequently, newly introduced links such as SoundBetter, Peerspace, LinkedIn, and GitHub remain unprobed, so this table cannot support the concluding claim that all links on current main/live resolve. Fresh evidence beyond the earlier history-query comment is that the reviewed commit's own parent contains this newer bundle; rerun the live probe against that bundle and update the receipt before declaring the finding re-closed.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
There was a problem hiding this comment.
nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
There was a problem hiding this comment.
nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
There was a problem hiding this comment.
nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
There was a problem hiding this comment.
nish3451 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4cb3f4ad27
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| 1. Source checks on this head: `npm run check` passes ("TinyStudio.io checks | ||
| passed") — the guard still refuses any AI-search source URL on the App | ||
| Store family of hosts (`apps.apple.com`, `itunes.apple.com`) that lacks an | ||
| app id, and still refuses drift between the embedded bundle on the audit | ||
| page and the fixture — and the full `npm test` suite passes (check, | ||
| headings 6/6, sitemap, worker, ui, contract; 92 subtests, 0 failures). |
There was a problem hiding this comment.
The reviewed commit is based on 66aa81e, not 18128e8; that newer parent includes #161's four additional worker subtests. Running npm test on the reviewed tree reports 96 tests (6 heading + 7 sitemap + 59 worker + 16 UI + 8 contract), so the recorded 92-test result came from the older head and does not substantiate the claim that the full suite passed on "this head." Re-run the suite against the actual parent and record its head and count.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2f256504f1
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| https://tinystudio.io/audit` → HTTP 200 with the CSP header, no console | ||
| errors, no page errors, and all 15 rendered external anchor links resolved | ||
| HTTP 200 with redirects followed (the same 12 unique URLs as the closeout: | ||
| fiberygoodness.com ×3, tagvenue.com ×2, apps.apple.com id-carrying form, |
There was a problem hiding this comment.
For the 15-row table recorded here, Fibery appears in rows 1, 4, 11, and 15, so fiberygoodness.com ×3 is false and the listed multiplicities account for only 14 anchors. This leaves the claimed 15-link measurement internally inconsistent even before considering the newer evidence bundle; record ×4 or correct the table so the totals are reproducible.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
1 participant
What
Re-verifies dogfood finding 78fcaed682fa ("Broken external links on /audit.html", audit run 20260808T074205Z-msk2fl3n) against current main and the live site, following the repo's evidence convention (same pattern as the canonical-URL re-verify in #114).
Why
The finding's fix (PR #33, id-carrying App Store citation + offline CI guard in
scripts/check-site.mjs) and closeout evidence (PR #79) are already on main. Seven commits touched the public surface since the closeout measurement at 8b42e0a — none touchedpublic/audit.htmlorevidence-fixtures/ai-search/. This re-verification proves the finding is still closed today.Verification (2026-08-12)
npm run checkpasses — the "External citation links (dogfood 78fcaed682fa)" guard still rejects any App Store family source URL without an app id and refuses embedded-bundle drift.npm testpasses — 92 subtests, 0 failures.https://tinystudio.io/audit→ 200 with CSP header, no console/page errors; all 15 rendered external anchor links resolve HTTP 200 (12 unique URLs, unchanged from the closeout).https://apps.apple.com/app/tinystudio→ still 404, still absent from the served bundle and the fixture.https://tinystudio.io/audit.html→ 307 →/audit.Outcome
Nothing further to change on the finding's page — the item is satisfied and remains closed.
Summary by CodeRabbit