Skip to content

docs(lane1): re-verify homepage managed-service section absence is the honored snooze - #241

Merged
nish3451 merged 5 commits into
mainfrom
docs/lane1-homepage-managed-service-snooze-reverify-20260821
Aug 21, 2026
Merged

docs(lane1): re-verify homepage managed-service section absence is the honored snooze#241
nish3451 merged 5 commits into
mainfrom
docs/lane1-homepage-managed-service-snooze-reverify-20260821

Conversation

@nish3451

@nish3451 nish3451 commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Item

[unreviewed-by-opus] Live site is missing the entire <section id="managed-service"> block on the homepage while the merged homepage on main carries it

Disposition: intended, snooze honored — not a regression

The managed-service buyer path (PRs #10/#11) is snoozed-by-Nish (2026-08-08: do not build, publish, or deploy it without his explicit yes). The release lane strips the entire section via scripts/prepare-public-deploy-bundle.mjs (fail-closed in both directions), and both scripts/test-public-deploy-bundle.mjs and scripts/check-public-live-deploy.mjs assert its absence.

Fresh re-verification (2026-08-21)

Change

Single docs annotation in docs/measurement/public-conversion-signal.md recording the re-verification. No product change — the section returns only when Nish lifts the snooze.

Summary by CodeRabbit

  • Documentation
    • Added a re-verification confirming the homepage’s managed-service section is intentionally hidden while its snooze remains active.
    • Documented that the current deployment matches the approved source and filtering behavior.
    • Confirmed the behavior is intentional and not a regression.

minimax-vps and others added 2 commits August 21, 2026 09:31
…e honored snooze

The live homepage has no <section id="managed-service"> while main carries
one. This was dispositioned 2026-08-11 as intended (snooze honored), and
this run re-verifies it fresh: the live bundle is current (deploy-manifest
source_commit ccfbd4b, prepared 2026-08-21T03:59:56Z) and the fail-closed
snooze filter deliberately strips the buyer path. No product change.

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

Warning

Review limit reached

@nish3451, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 4 minutes

Limit details: You’ve used the included review currently available.

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: 5028f482-9684-400f-9c77-1aa28d6c24d9

📥 Commits

Reviewing files that changed from the base of the PR and between 86a7ef9 and a801dec.

📒 Files selected for processing (1)
  • .lane/reports/docs-lane1-homepage-managed-service-snooze-reverify-20260821.md
ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: 3ba085e1-2318-4385-9046-c713ce40eea9

📥 Commits

Reviewing files that changed from the base of the PR and between ccfbd4b and 86a7ef9.

📒 Files selected for processing (1)
  • docs/measurement/public-conversion-signal.md

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


📝 Walkthrough

Walkthrough

The documentation adds a 2026-08-21 re-verification of the managed-service section. It records that the section remains absent from production because the active snooze triggers fail-closed filtering.

Changes

Conversion Signal Documentation

Layer / File(s) Summary
Managed-service production verification
docs/measurement/public-conversion-signal.md
Adds source, live-site, and deployment-manifest evidence. Records that the section will return after the snooze is lifted.

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

Merge Risk: ⚪ Minimal · up to 86a7e

This PR records the verified homepage deployment behavior without changing product code or runtime behavior. No actionable merge-blocking risk remains beyond 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/login.

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

@nish3451
nish3451 merged commit d2d96a5 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