docs(evidence): re-verify the Website Appraisal product contract on current main (2026-08-12) - #148
Conversation
…urrent main (2026-08-12)
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.
|
Warning Review limit reached
Next review available in: 2 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (1)
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 |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5418b08838
ℹ️ 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".
| Commits between `872fd23` and `ad9cee3` (2026-08-12): three AI-search evidence | ||
| commits (`8606b0c`, `62eec0a`, `ed62202`), a docs/evidence ship-verify commit | ||
| (`43c9e14`), and merges of PR #141 and #43. None touch `README.md`, `MEMORY.md`, | ||
| `package.json`, the specs, or `scripts/test-product-contract.mjs` — the |
There was a problem hiding this comment.
Correct the incomplete commit-range inventory
The claimed range is incomplete: git log 872fd23..ad9cee3 contains 20 commits, including product/worker, CI, dependency, and public-page changes, rather than only the six commits listed here. The statement that none touched package.json is also false because 37ddaed updates its Wrangler dependency. Since this section is the receipt's justification that the previously verified surfaces did not change, omitting those commits makes the recorded provenance inaccurate; either enumerate the full range and its affected files or narrow the stated baseline.
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.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: dfd3a7a882
ℹ️ 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".
| wired into `npm test`, and the full suite (92/92) plus live surfaces verify the | ||
| current truth on this head. Nothing further to change; the item can be ticked. |
There was a problem hiding this comment.
Keep the receipt open until
/audit is current
The commit's parent already records in docs/evidence/ai-search-evidence-lag-2026-08-12.md:63-83 that a same-day live fetch of /audit still embeds the retired-Agent-Desk q5 truth and differs from source, with deployment blocked. Because the current plan includes /audit among the owned current public surfaces, this conclusion cannot say the live surfaces verify current truth or that the item can be ticked; qualify the receipt as repository-only or re-verify /audit after the corrected artifact is deployed.
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: 34dacff7a5
ℹ️ 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".
| repo truth"), the abandoned round-1 candidate lane was closed out by PR #72 | ||
| (`872fd23`), and a closeout receipt re-verified the item on 2026-08-11. This | ||
| receipt re-verifies every acceptance criterion against the current head | ||
| (`ad9cee3`, 2026-08-12) and live, because the item is still tagged |
There was a problem hiding this comment.
Re-run the receipt against its actual parent
This receipt does not verify the commit it lands on: ed46ff9 has first parent 0e357e0, while this line calls the older ad9cee3 revision the current head; fe4d2ae, 883e106, and 0e357e0 intervene. Consequently, the later claims that checks were freshly run “on this head” do not establish results for the actual current tree. Re-run and pin the verification to 0e357e0 (or explicitly qualify ad9cee3 as a historical baseline).
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.
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: 774a6165fa
ℹ️ 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".
| `npm run test:worker` — 55/55 (incl. legacy `/api/agent-audit` mechanics); | ||
| `npm run test:ui` — 16/16. | ||
| - Full `npm test` chain — 92/92 pass, exit 0. |
There was a problem hiding this comment.
Refresh totals for the new rate-limit tests
On this commit's actual tree, npm run test:worker reports 59 tests and the full chain totals 96, rather than the recorded 55 and 92. The fresh evidence beyond the earlier stale-head comment is that this version now lands directly on 66aa81e, which adds four signup rate-limit tests; leaving the historical counts here makes the claimed fresh verification inaccurate even though the suite passes.
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.
What
The tracker item "[unreviewed-by-grok] Replace the retired Agent Desk as the repository's active product contract — README, MEMORY" was already fixed and merged as PR #58 (
11864a7) and its abandoned round-1 lane closed out by PR #72 (872fd23). This PR ships a fresh 2026-08-12 re-verification receipt on the current head (ad9cee3), since the item is still tagged unreviewed-by-grok and main moved after the 2026-08-11 receipt.Why no code change
All four acceptance criteria are already met on current main — re-verified deterministically today:
/api/agent-auditmechanics.scripts/test-product-contract.mjs(innpm test) rejects any return to active Agent Desk framing.npm run check✓, headings 6/6, sitemap 7/7, worker 55/55, ui 16/16, contract 8/8 — 92/92 total, exit 0.Live (2026-08-12): homepage titles "TinyStudio — The Website Appraisal";
/agent-desktitles "TinyStudio — the retired Agent Desk"; llms.txt presents the Agent Desk as "Legacy … is demoted and is not the product TinyStudio sells".Re-implementing already-merged docs/guards would be churn (the stale
candidate/product-contract-*branches show exactly that trap); the durable deliverable is the re-verification receipt.