Skip to content

fix: CodeRabbit auto-fixes for PR #1415 #1425

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
coderabbitai wants to merge 1 commit into from
Closed

fix: CodeRabbit auto-fixes for PR #1415 #1425

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs/branch-review-ledger.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1322,7 +1322,7 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie
| 2026-07-30 | claude/test-coverage-analysis-2vcd8a | 4f498b66a56b2a7eddde6c841a79621f23b59cc7 | PR #1398 babysit | BLOCKER CLEARED: CONFLICTING due to docs/outstanding-issues.md vs main (#115 band-adoption follow-up). Kept main #115 + next-id=116; preserved PR #109 single-branch/refspec update. Prior tip had no GitHub CI suite (only PR Policy/CircleCI) — push retriggered full CI. 0 review threads; 0 Bugbot findings. | verify:cheap PASS (432 files, 4467 passed \| 4 skipped); repo-hygiene 38/38; sweep:branch-ledger --no-fetch exit 0; format:changed PASS; Bugbot none; hosted CI re-triggered on tip |
| 2026-07-30 | PR #1394 / `claude/top-search-design-mockups-w53znc` | `0d47141fc030684299dcb265e3d853c93b9e2a91` | CI/review closeout — merged | MERGED as squash `0d47141f`. Prior tip `4a001efa` had required CI green after prettier fix `61314887` (Static PR/CircleCI red on `#096` padding) and main sync. Layout/`/tools` false-positive fixed; `#115` deferred; review threads resolved. Post-merge ledger-only follow-up. | hosted Static/Unit/PR-required/CircleCI pass on pre-merge tip; vitest adoption 6/6; typecheck; Bugbot no open P0/P1; merge-tree clean |
| 2026-07-30 | cursor/pr-1394-ledger-closeout-c2bf | f734dc4d4c8b19d5fec43bbd388c2a421e47668a | PR #1399 babysit / CI+Bugbot closeout | MERGE-READY. No failing CI, no unresolved review threads, merge-tree clean vs origin/main, Bugbot no bugs. Docs-only ledger append for merged #1394; no code fix required. | hosted PR required SUCCESS; Static PR SUCCESS (lint/typecheck/format/ledger); CircleCI verify SUCCESS; local check:branch-review-ledger PASS; prettier PASS; lint PASS; Bugbot pr-bugbot no findings |
| 2026-07-30 | claude/design-computed-style-proof | 228ffc8583dc0579cfba033bcbe0e7ef18be0317 | #094 computed-style proof: unlayered-cascade check in check-design-system-contract.mjs plus ui-smoke computed-style assertions for the accent rail, forced-colors thickness and 44px tap targets; records ledger #116 worktree-deregistration git redirection and #117 issues:next-id collisions | PR #1415 opened; auto-merge off | verify:pr-local 432/432 files 4460 tests; new Playwright case 1 passed; both gates negative-tested red on the injected @layer regression then reverted clean |
| 2026-07-30 | claude/design-computed-style-proof | 228ffc8583dc0579cfba033bcbe0e7ef18be0317 | #094 computed-style proof: unlayered-cascade check in check-design-system-contract.mjs plus ui-smoke computed-style assertions for the accent rail, forced-colors thickness and 44px tap targets; records ledger #123 worktree-deregistration git redirection and #124 issues:next-id collisions | PR #1415 opened; auto-merge off | verify:pr-local 432/432 files 4460 tests; new Playwright case 1 passed; both gates negative-tested red on the injected @layer regression then reverted clean |
| 2026-07-30 | PR #1396 / claude/latency-findings-impl-s8g01v | 6f75bba54684c104a9bd70b36c401f04ca4c57b5 | Babysit: sync main after Claude pre-paint fix | Synced origin/main (ledger-only #1399). MERGEABLE; merge-tree clean. Claude tip added pre-hydration overlay reserve fix. No unresolved threads. Bugbot still empty on prior tips. Contract 28/28. | header-scroll-hide-contract 28/28; merge-tree clean; prior verify:cheap/typecheck/lint retained |
| 2026-07-30 | claude/test-coverage-analysis-2vcd8a | d5842e62238237ff5c47da0b32ef8d9f12819714 | PR #1398 babysit | COMPLETE for tip: cleared main conflict; fixed Codex P2 (reject refs/*→origin/* nesting); prior Codex P2 (destination check) already fixed in de594186 and resolved; 0 unresolved threads; merge-tree clean. Hosted CI re-running. | repo-hygiene 40/40; verify:cheap earlier PASS on pre-tip; format:changed PASS; Bugbot none; Codex P2 resolved |
| 2026-07-30 | PR #1396 / claude/latency-findings-impl-s8g01v | 9d03b84f1a32a056f74727b4e6bdd5558c346bf0 | Babysit: resolve outstanding-issues after #1402/#1398 | FIXED CONFLICTING: took main open-table (widened cols + #109 refspec) and kept #116/#117 phone-chrome gaps (next-id=118). MERGEABLE expected. Codex P1s already fixed on tip and threads resolved. No Bugbot findings. | merge-tree clean; contract 28/28 earlier; verify:cheap on prior tip |
Expand Down
Loading