issues: capture design-system Track A2-A6 and B1-B4, plus the tap-token blocker from #218 - #1681
Conversation
|
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: 6 minutes 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 (1)
Comment |
Review-and-fix summary — PR #1681Tip: Fixed vs dispositioned
Required CI (this tip)
Decisive local proofAlso: Residual risks
Merge left to you. No push, no force-push, no auto-merge arm from this pass. |
Summary
docs/design-system/HANDOVER-2026-08-07.md§4 into the durable ledger, so the plan survives the session that read it. Append-only: ten new rows, one existing row updated, nothing removed or reordered.#261A2 — retire--shadow-focusfrom the search composer (visible focus-state change;docs/search-chrome-behaviour.mdfirst).#262A3 — finish the token debt:--shadow-tight's 155 consumers onto the--eNladder, a step-selection lint for the eight non-standard type steps, and extending the contract ratchet to raw padding / radius / line-height. Records explicitly thatcheck:type-scalealready ships the arbitrary-value half, so no lint should duplicate it.#263A4 — the 13 openCOMPONENTS.md§0.4 defect rows, with the roughly five stale ones excluded.#264A5 — gate the six ungated prohibitions. Flagged as the highest-value Track A item: a prohibition with no gate row is a suggestion.#265A6 — gates 2, 4, 7 and 8 from partial to blocking.#266B1 — adopt the 24 unadopted components demand-driven, never as a race to 53/53.#267B2 —AnswerFooterandDoseLineneed a provenance/dose payload the answer surface does not yet produce; backend-shaped, do not stub.#268B3 — the 19 genuine bare-dash sites ontoMissingValue, with the roughly 5 calculatorderived.started ? score : "—"sites recorded as permanently excluded.#269B4 — the per-component visual state matrix, recorded as blocked on#118and unbuildable on Windows becausesnapshotPathTemplatecarries{platform}.#270— new, from PR feat(design-system): give cn() tailwind-merge (#218) #1678: 22 call sites pairmin-h-tap/h-tapwith a numeric height that is already dead, which is why--spacing-tapis deliberately absent from the tailwind-merge config. Carries the exact next action and the stop rule (do not instead lower any target tomin-h-11).#155updated with new evidence rather than a duplicate row: on 2026-08-07 a concurrent session in the same worktree committed between this session'sgit addandgit commit, sweeping the staged index into a foreign commit and re-pointing the in-use branch.git commitreported "nothing to commit, working tree clean"; onlygit log -1revealed it. Adds the check that catches it.Verification
npm run check:outstanding-issues— exit 0. "Outstanding-issues guard passed: 268 rows (127 open, 141 archived), unique ids, next-id=271 above the highest, no merge driver, no ids deleted from base 4600268." Self-tests for both the guard and the writer also passed.npm run issues:add/npm run issues:update, never hand-edited, so ids come from the marker and the table shape is the writer's.ISSUES_LIST_UPDATED path=C:\Users\joshs\OneDrive\ISSUES-LIST.html.git merge-tree --write-tree origin/main HEAD— clean.Verification not run: the broader gates. This is a documentation-only, append-only change to a single Prettier-excluded file that no source path imports;
check:outstanding-issuesis the gate that governs it and it runs inverify:cheapandstatic-pr.Risk and rollout
next-idmarker, or a reappearing merge driver.Notes
docs/outstanding-issues.mdhas no merge driver by design (#133), so if this conflicts, rebuild fromorigin/mainand re-apply only these rows — never take one side wholesale.🤖 Generated with Claude Code