fix(public): clear the stale Agent Desk claims from retired surfaces and the homepage - #62
fix(public): clear the stale Agent Desk claims from retired surfaces and the homepage#62nish3451 wants to merge 15 commits into
Conversation
…and the homepage The retired app/api surfaces claimed tinystudio.io 'runs the self-serve Agent Desk from the main domain' and /health reported surface 'agent-desk' — but the Agent Desk is retired and demoted (offer.md: 'is not the current offer'), and the current offer is the Website Appraisal. Two public endpoints and the health check were publishing product truth the rest of the site contradicts. The homepage also shipped two developer REPLACE comments inside the 'Who signs it' section, including an empty sig-note paragraph meant to carry a credential line. - retired app page and retired API message now name the Website Appraisal; retired framing and 410s unchanged - /health surface: website-appraisal - homepage REPLACE comments and empty sig-note paragraph removed, dead .sig-note:empty CSS rule removed with them - check-site.mjs: served marketing pages must not ship REPLACE comments - test-agent-worker.mjs: retired surfaces and /health pinned to the current offer, never the Agent Desk - docs/evidence/retired-surface-copy-2026-08-10.md: observed/fix/verify
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: 1 second 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: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe change retires Agent Desk messaging across app, API, and health responses. It removes homepage placeholders, adds site validation for replacement comments, updates test fixtures and endpoint coverage, and records deployment verification steps. ChangesRetired surface correction
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 71a32b0fe1
ℹ️ 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".
| ok, | ||
| service: "tinystudio-io-public", | ||
| surface: "agent-desk", | ||
| surface: "website-appraisal", |
There was a problem hiding this comment.
Align the health label with the checks it performs
When the legacy Agent Desk AI binding or either agent_* table is unavailable while the Website Appraisal homepage and signup flow remain functional, this endpoint now identifies the affected surface as website-appraisal and returns 503 because ok is still gated on those Agent Desk dependencies. Monitoring keyed by surface will therefore report the current appraisal as unhealthy for an unrelated legacy failure; either retain an Agent Desk-specific label or revise the checks to cover the Website Appraisal's actual assets and signup dependencies.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/evidence/retired-surface-copy-2026-08-10.md`:
- Around line 35-40: Update the “Live re-check after deploy” verification
commands to assert HTTP status codes as well as checking response bodies:
require 410 responses from the retired app and API endpoints, and require a 200
response from the /health endpoint while preserving the existing surface check.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: dec2d788-fbcb-451a-b393-6979c13ca5c0
📒 Files selected for processing (6)
docs/evidence/retired-surface-copy-2026-08-10.mdpublic/index.csspublic/index.htmlscripts/check-site.mjsscripts/test-agent-worker.mjssrc/worker.js
💤 Files with no reviewable changes (2)
- public/index.html
- public/index.css
| Live re-check after deploy: | ||
|
|
||
| ```bash | ||
| curl -s https://app.tinystudio.io/ | grep -c "Agent Desk" # 0 | ||
| curl -s https://api.tinystudio.io/ | grep -c "Agent Desk" # 0 | ||
| curl -s https://tinystudio.io/health # "surface":"website-appraisal" |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Assert HTTP status codes in the live re-check.
The grep commands only inspect response bodies. A 404, 500, or proxy error page without Agent Desk can produce the same 0 result. Require 410 for the retired app and API, and 200 for /health.
Proposed verification update
- curl -s https://app.tinystudio.io/ | grep -c "Agent Desk" # 0
- curl -s https://api.tinystudio.io/ | grep -c "Agent Desk" # 0
- curl -s https://tinystudio.io/health # "surface":"website-appraisal"
+ app_status="$(curl -sS -o /tmp/tinystudio-app.html -w '%{http_code}' https://app.tinystudio.io/)"
+ test "$app_status" = "410" && ! grep -Fiq "Agent Desk" /tmp/tinystudio-app.html
+ api_status="$(curl -sS -o /tmp/tinystudio-api.json -w '%{http_code}' https://api.tinystudio.io/)"
+ test "$api_status" = "410" && ! grep -Fiq "Agent Desk" /tmp/tinystudio-api.json
+ health_status="$(curl -sS -o /tmp/tinystudio-health.json -w '%{http_code}' https://tinystudio.io/health)"
+ test "$health_status" = "200" && grep -Eq '"surface"[[:space:]]*:[[:space:]]*"website-appraisal"' /tmp/tinystudio-health.json📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Live re-check after deploy: | |
| ```bash | |
| curl -s https://app.tinystudio.io/ | grep -c "Agent Desk" # 0 | |
| curl -s https://api.tinystudio.io/ | grep -c "Agent Desk" # 0 | |
| curl -s https://tinystudio.io/health # "surface":"website-appraisal" | |
| Live re-check after deploy: | |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/evidence/retired-surface-copy-2026-08-10.md` around lines 35 - 40,
Update the “Live re-check after deploy” verification commands to assert HTTP
status codes as well as checking response bodies: require 410 responses from the
retired app and API endpoints, and require a 200 response from the /health
endpoint while preserving the existing surface check.
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.
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.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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.
|
Closing: retired-surface copy landed (homepage cleanups). |
What
The retired surfaces still claimed tinystudio.io "runs the self-serve Agent Desk from the main domain":
https://app.tinystudio.io/(410 HTML),https://api.tinystudio.io/(410 JSON), and/health("surface":"agent-desk"). The Agent Desk is retired — its own page frames as "the retired Agent Desk", and offer.md says it "is demoted and is not the current offer". The current offer is the Website Appraisal. Two public endpoints and the health check were publishing product truth the rest of the site contradicts.Separately, the live homepage source shipped two developer
<!-- REPLACE: -->comments inside the "Who signs it" section — the site's strongest credibility claim — including an empty<p class="sig-note">meant to carry "one true credential line" (hidden by CSS, but visible in the money page's source to any viewer).Changes
src/worker.js— retired app page and retired API message now name the current offer: "TinyStudio.io now runs the Website Appraisal — the free leak audit of high-ticket service homepages." Retired framing and 410 statuses unchanged./healthreports"surface":"website-appraisal".public/index.html— the two REPLACE comments and the emptysig-noteparagraph removed; no credential invented, placeholders only.public/index.css— dead.sig-note:emptyrule removed.scripts/check-site.mjs— every served marketing page (home, audit, agents, pricing, specimen, brief-requested) is rejected if it ships a<!-- REPLACE:comment.scripts/test-agent-worker.mjs— retired app surface (410 HTML), retired api surface (410 JSON), and/healthasserted to name the Website Appraisal and never "Agent Desk"; healthsurfacepinned.docs/evidence/retired-surface-copy-2026-08-10.md— observed live responses, fix, guard, verify.Verification
npm testgreen: check 6/6, headings 7/7, sitemap 56/56, ui 16/16.npx wrangler deploy --dry-runpasses.Verify after deploy
Summary by CodeRabbit
Updates
Quality Improvements