Cancel the mistaken CLS-regression outstanding-issues request - #2078
Conversation
Follow-up investigation on origin/main (proper CHROME_PATH, matching Chromium build family) shows every measured Lighthouse route/strategy matches its committed baseline exactly, including mobile-root CLS at 0.013/0.013. There is no code-level CLS regression -- the repeated CI failures that prompted docs/outstanding-issues-inbox/d92786de-....json were environmental/CI-infra noise, not a product defect.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 23 minutes Limit details: You’ve used all 1 included review currently available under your plan. You completed 97 included PR reviews in the past 7 days; at that activity level, included reviews refill at 1 review per hour. Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab. 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: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
Comment |
Summary
docs/outstanding-issues-inbox/d92786de-de31-4118-84cc-0a9098e7f2e0.json), which claimed a real mobile CLS regression on/.Context
While investigating why PR #2050's CI kept failing
Lighthouse budget, I initially concluded (in #2059) that a real, code-level mobile CLS regression had reappeared on/(measuring 0.236 against a 0.013 baseline across multiple CI runs on the correctly pinned browser).Follow-up investigation reverses that conclusion. Running the same Lighthouse harness locally against
origin/main(CHROME_PATH=/opt/pw-browsers/chromium npm run verify:lighthouse -- --keep, after correcting an earlier wrongCHROME_PATHthat made every measurement silently fail with exit code 0 — a trap worth noting for future runs) produced this table:Every route/strategy — including
mobile-rootCLS — matches its committed baseline exactly.mainhas no code-level CLS regression. The repeated 0.236 CI failures were environmental/CI-infra noise (most plausibly resource contention from the unusually high concurrent CI load PR #2050 generated while its branch was auto-resyncing many times in a short window), not a product defect.This PR queues a
cancelrequest against the originaladdrequest perscripts/ledger-inbox.mjs(node scripts/ledger-inbox.mjs cancel <uuid> --reason "..."), so the false finding does not get reconciled into the canonicaldocs/outstanding-issues.mdledger.Verification
node scripts/ledger-inbox.mjs cancel ...— the script's own validation accepted the request and reported it merge-safe.docs/outstanding-issues-inbox/. No source/build/test surface is touched.Risk and rollout
Not applicable — append-only tracking document correcting a prior tracking document; no code/behavior change.
Notes
Generated by Claude Code