docs(lane1): reverify google-ai-answers brand-disambiguation item — source fixed in PR #29, live blocked on missing Cloudflare token - #184
Conversation
…ource fixed in PR #29, live blocked on missing Cloudflare token (#182) The Google AI answers disambiguation item is already fixed on main: commit 4202d54 (PR #29) put the "independent product company at tinystudio.in, not affiliated with other apps or studios using the name" statement in every public page's Organization JSON-LD, in the homepage visible copy, in llms.txt, and in its generator template, and wired scripts/test-public-brand-disambiguation.mjs into npm test and npm run ci. Re-verified against fresh origin/main: the guard script passes (11 checks, 0 failures) and the source homepage carries the identity statement. Live tinystudio.in still serves a stale bundle with no JSON-LD and no disambiguation copy, and check-public-live-deploy.mjs fails 7/15 checks; the deploy lane is fail-closed on the missing CLOUDFLARE_API_TOKEN secret (NEEDS-NISH). No source change is possible or needed; this report documents the verification. Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
|
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.
Your trial has ended. Reactivate Greptile to resume code reviews.
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdded a verification report for the merged Tiny Studio brand-disambiguation fix. The report documents source and live evidence, blocked Cloudflare deployments, affected files, and redeployment steps. No source files changed. ChangesBrand-disambiguation verification
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This PR adds a verification report without changing application behavior or deployment configuration; no actionable merge-blocking risk remains beyond normal checks and review. 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 |
…ation-reverify-20260815
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
…ation-reverify-20260815
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
Lane 1 reverify: Google AI answers brand-disambiguation item
Verdict: source already fixed on main (PR #29); live blocked on missing Cloudflare token (NEEDS-NISH).
Source state — fixed and test-pinned
4202d54(PR fix(public): disambiguate Tiny Studio from unrelated tiny studio brands #29) is an ancestor of freshorigin/main.public/llms.txt, and thellms.txtgenerator template.scripts/test-public-brand-disambiguation.mjsis wired intonpm testandnpm run ci.Live state — stale bundle, deploy-pipeline gap
tinystudio.inserves a stale build with no JSON-LD at all and no disambiguation copy; livellms.txtalso lacks it.node scripts/check-public-live-deploy.mjs→ 15 checks, 7 failures (all four live proofs fail, not just fix(public): disambiguate Tiny Studio from unrelated tiny studio brands #29's)..github/workflows/deploy-public-site.ymlis fail-closed onCLOUDFLARE_API_TOKEN, which is still missing from repo secrets (onlyCLOUDFLARE_ACCOUNT_IDpresent). All recent deploy runs on main fail.No source change possible or needed
Re-implementing a merged, test-pinned fix would duplicate shipped work. Unblock = Nish provisions
CLOUDFLARE_API_TOKEN; the next deploy carries ~17 merged public fixes live.Summary by CodeRabbit