feat: make the anonymous one-page check a clickable CTA on the methodology page - #90
Conversation
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: 35 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: Pro Plus Run ID: 📒 Files selected for processing (3)
Comment |
…n — /check free-entry surface live-stale, window must not start (#134) `npm run audit:live-promise` fails on 9 of 16 surfaces against the deployed site (pre-#85/#88/#90/#100/#107 copy: missing no-storage disclosure on /check, missing footer terms/privacy links on /demo /methodology /packages, missing /check cross-links on /support /terms /privacy, 422-vs-400 on non-http scheme, www static asset 200-vs-301). Root cause is the fleet release deploying a stale Worker + assets bundle on 2026-08-13 22:40 (its log says "No updated asset files to upload" while recording index-Dd3Lei8e.js as the marker; live serves the older index-DX7O9nYF.js). Repo source is green (npm run check + offline spot-check lock pass); this is deploy machinery, not copy drift. The experiment log now carries the dated precondition note so the founder does not start the seven-day window against a broken /check, plus the resume condition (re-verify after a release that actually swaps the live Worker). Co-authored-by: minimax-vps-lane1 <minimax-vps@nish.fleet> Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
What
The
/methodologypage listed the anonymous one-page check as plain text (at https://seofixkit.com/check:). This turns it into a real clickable CTA:/checkURL in the "What is live today" list item is now an<a href>link.Check one page nowCTA button below the list, styled with the same.ctatreatment the demo page uses (added the missing.ctarule topublicProductPageHtml's CSS).methodologyHtmlfromworker/routes/pages.js.Tests
npm run test:public-pages— 7 pass (added assertions that the URL is a real link and the CTA exists)node --test scripts/live-promise-spot-check.test.mjs— 14 pass (added a live spot-check expectation forhref=.../checkon /methodology)node --test worker/index.test.mjs— 10 pass