Skip to content

docs(evidence): re-verify /health surface and signup intake labels (2026-08-21) - #281

Merged
nish3451 merged 20 commits into
mainfrom
docs/health-surface-website-appraisal-rereverify-2026-08-21-lane1
Aug 21, 2026
Merged

docs(evidence): re-verify /health surface and signup intake labels (2026-08-21)#281
nish3451 merged 20 commits into
mainfrom
docs/health-surface-website-appraisal-rereverify-2026-08-21-lane1

Conversation

@nish3451

@nish3451 nish3451 commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

What

Re-verifies review-queue item a3ab4b7420 — "[unreviewed-by-opus] The live worker /health surface still reports surface:"agent-desk" and the signup intake so" — against current origin/main (92d55c3) and the live deployment.

Verdict: closed

The failure mode no longer occurs, on source or live:

  • Live GET https://tinystudio.io/health → 200: {"ok":true,"service":"tinystudio-io-public","surface":"website-appraisal",...} (probed 2026-08-21). app.tinystudio.io serves the retired-app 410 page; api.tinystudio.io names The Website Appraisal as the current offer.
  • Source src/worker.js: APPRAISAL_SURFACE = "website-appraisal" labels both /health (surface: APPRAISAL_SURFACE) and the /api/signups intake (saveEmailSignup(..., APPRAISAL_SURFACE, ...)); agent-self-serve survives only on the legacy /api/agent-audit path.
  • Guards npm run check → exit 0; npm test → exit 0, 126 pass / 0 fail. scripts/test-agent-worker.mjs asserts body.surface === "website-appraisal" and !== "agent-desk", and asserts current-intake signup rows carry website-appraisal, never agent-self-serve.

The fix itself landed via PR #164 (aeb34a9, merged 2026-08-13); this PR records the dated re-verification receipt on the current head (same pattern as the 2026-08-14 receipt for this item).

Files

  • docs/evidence/health-surface-website-appraisal-rereverify-2026-08-21.md — dated evidence receipt
  • .lane/reports/docs-health-surface-website-appraisal-rereverify-2026-08-21-lane1.md — lane report (lane-unique path)

No production code changed; no shared report files touched.

Summary by CodeRabbit

  • Documentation
    • Added verification records for the live health endpoint and signup intake.
    • Confirmed current website-appraisal labeling and documented separation from legacy paths.
    • Recorded successful source checks, automated tests, and live endpoint probes.
    • Documented deployment and CI limitations alongside the verification evidence.
    • Closed the related health-surface and signup-label review item.

minimax-vps and others added 2 commits August 21, 2026 08:42
…026-08-21)

The review item "live worker /health still reports surface:agent-desk and
the signup intake so" re-verified against current origin/main (92d55c3) and
the live deployment: /health reports surface:"website-appraisal" with
ok:true, the current /api/signups intake writes website-appraisal rows, and
the retired agent-self-serve label survives only on legacy /api/agent-audit.
npm run check and npm test (126 pass, 0 fail) are green on the head.

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Review Change Stack

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: 175ddf5d-8c56-4f3f-807c-02bee8bb022d

📥 Commits

Reviewing files that changed from the base of the PR and between 4074d79 and d6fb168.

📒 Files selected for processing (2)
  • .lane/reports/docs-health-surface-website-appraisal-rereverify-2026-08-21-lane1.md
  • docs/evidence/health-surface-website-appraisal-rereverify-2026-08-21.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The change adds a lane report and evidence receipt for re-verification of /health, signup labeling, legacy paths, and live endpoints. It records website-appraisal as current and closes the reported issue without production code changes.

Changes

Health surface re-verification

Layer / File(s) Summary
Verification evidence
docs/evidence/health-surface-website-appraisal-rereverify-2026-08-21.md
Records source checks, tests, signup-label behavior, legacy API labeling, live /health probes, deployment limitations, and closeout status.
Lane report closeout
.lane/reports/docs-health-surface-website-appraisal-rereverify-2026-08-21-lane1.md
Records successful verification, observed endpoint results, evidence references, and confirmation that production and shared report files were unchanged.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to d6fb1

This PR records re-verification evidence without changing production code or runtime behavior, so no actionable merge-blocking risk remains after normal checks and review.


Note

🎁 Summarized by CodeRabbit Free

Your 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/settings/billing.

Comment @coderabbitai help to get the list of available commands.

@nish3451
nish3451 merged commit 69ca50b into main Aug 21, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant