diff --git a/docs/branch-review-ledger.md b/docs/branch-review-ledger.md index 748dfc7c51..29f9bf0f39 100644 --- a/docs/branch-review-ledger.md +++ b/docs/branch-review-ledger.md @@ -1138,6 +1138,9 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie | 2026-07-28 | PR #1291 / `claude/issues-upload-limit-sync-123366` | `1607558188283d3497683f1067835d96f1031d3c` | CI babysit + merge conflict + Bugbot | FIXED. GitHub CONFLICTING/DIRTY was a real content conflict in `docs/outstanding-issues.md`: main had claimed `#084` for completed per-result grading evidence, colliding with this PR's upload-limit capture. Resolved by keeping main's ledger, renumbering the upload-limit recommendation to `#085`, and bumping `issues:next-id` to `086`. Synced again when main advanced with #1300. CodeRabbit date thread already resolved. Bugbot: zero `cursor[bot]` findings. Required CI green (PR required SUCCESS). | merge-tree CLEAN; prettier + docs:check-links PASS; hosted Change scope/Static/PR required SUCCESS; no provider-backed checks. | | 2026-07-28 | PR #1291 / `claude/issues-upload-limit-sync-123366` | `af140d11d5ca23dee0d8705d9933db967fc8c404` | Babysit closeout tip | Supersedes prior #1291 row at `16075581` after appending the conflict/Bugbot ledger record. Product delta vs main unchanged: `#085` upload-limit capture only. merge-tree CLEAN; awaiting exact-head required checks. | ledger append + check:branch-review-ledger PASS; prior tip hosted PR required SUCCESS. | | 2026-07-28 | PR #1302 / `claude/maturity-ledger-entry` | `64da2c1b34ae101590b8676af12ec6b49c14f0ad` | CI/conflict babysit + Codex threads + Bugbot | FIXED. Real content conflict with main: `#085` already claimed by upload-limit rec (#1291). Merged origin/main; renumbered maturity backlog to `#086`, bumped `issues:next-id` to `087`, added recommended-queue order 29 with go-ahead/RAG/provider stop rules. X7/M1 work orders arrived via main #1299. Codex P2 threads replied + resolved. Bugbot: zero cursor[bot] findings. CircleCI stub from main clears prior "no configuration" status error. | merge-tree CLEAN; prettier + docs:check-links + docs:check-scripts PASS; awaiting exact-head hosted CI; no provider-backed checks. | +| 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `30596680a1c0026ea83c8fab0da2cd303106e1af` | Merge conflict + CodeRabbit/Codex + Bugbot closeout | FIXED. Merged origin/main (was CONFLICTING). Removed undersized `.content-visibility-auto` (80px) in favor of shared `.content-auto` (auto 320px); dropped persistent broad `will-change: transform` so fixed composer ancestors stay viewport-anchored (main's `data-reserve-transitioning` padding-bottom scope retained); kept semantic z-* utilities + AccessibleTable `@container`/`flex flex-col`. Updated mode-menu source-contract test for `z-modal`. Resolved CodeRabbit + Codex threads. Local Bugbot: no P0/P1; skipped raising mockup/popover z above token scale (would undo audit). | `npm run test` 4114 passed; prior verify:cheap gates green before test pin fix; no provider-backed checks. | +| 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `f333ddc8` | CI Build flake: RAM guard | FIXED. Hosted Build failed on ubuntu-24.04 reporting 7.8 GiB because `guard-next-build.mjs` hard-exited under 10 GiB; successful sibling Build jobs skip that path on larger runners. Warn-only under CI/GITHUB_ACTIONS; keep local hard-fail. Prior tip: Unit/Static/Advisory UI green; review threads resolved. | Hosted Unit/Static/Advisory PASS on 6bafb32d; local full vitest 4114; no provider checks. | +| 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `0ab3597fecd1fd7af669c766bf4b66fa725c9936` | CI re-trigger after Docker RAM allow | FIXED app-image path: Dockerfile sets DOCKER_BUILD=1; guard warns under CI/GITHUB_ACTIONS/DOCKER_BUILD/ALLOW_LOW_MEMORY_NEXT_BUILD. Hosted Build/Unit/Static/Advisory/Production UI green on prior tip; app-image failed only on RAM hard-stop before Docker opt-in. | Awaiting exact-head CI after synchronize; no provider checks. | | 2026-07-28 | PR #1297 / `motion-audit-fixes-clean` | `cdde82e087ed3e7ad6be3662b1b96aa55462cae5` | CI babysit + Bugbot + main sync | FIXED. Required CI was already green on prior tip; branch was BEHIND (merge-tree CLEAN). Merged origin/main. Bugbot P2s fixed: wired shimmer-sweep into `.animate-skeleton-shimmer::after`, applied `.stagger-item` on AnswerSkeleton, knob uses `var(--ease-spring)`. ISSUE-02/05 reduced-motion halt retained. 0 review threads to resolve. | focused vitest 34/34; tsc --noEmit PASS; merge-tree CLEAN; no provider-backed checks. | | 2026-07-28 | PR #1297 / `motion-audit-fixes-clean` | `9154d6ef` | CI Build flake fix | FIXED. Hosted Build failed when a runner reported 7.8 GiB via `os.totalmem` and hit the local Docker RAM floor in `guard-next-build.mjs`. Gate now skips under `CI`/`GITHUB_ACTIONS` (local protection retained). Prior tip motion/Bugbot fixes unchanged. | CI=true guard exit 0; awaiting exact-head hosted Build; no provider checks. | | 2026-07-28 | PR #1297 / `motion-audit-fixes-clean` | tip after container RAM-gate fix | CI container Build flake | FIXED. `Container images / app-image` failed because Dockerfile `npm run build` hit the same local RAM floor (7.8 GiB, no CI env in the build stage). Guard now also skips for `CI=1` and `/.dockerenv`; Dockerfile sets `ENV CI=1`. | awaiting exact-head hosted container + PR required; no provider checks. | @@ -1158,10 +1161,13 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie | 2026-07-28 | PR #1307 / `claude/rag-019-fallback-guard` | `2e5edfc6c9fcc0bb75674b179607888bbf872b91` | PR policy body synced + template removed | Sync PR policy body SUCCESS applied Clinical Governance Preflight (7/7). Deleted temporary `PR_POLICY_BODY.md`. Awaiting Build/PR policy/PR required on this tip. | Hosted Sync SUCCESS; no provider checks. | | 2026-07-28 | PR #1294 / `execute-typography-fixes-clean-2` | `a0df13f45cffb769b852e55bc44b6891b7fd80e7` | Main conflict resolve + CodeRabbit | FIXED. Merged #1307; took main ALLOW_LOW_RAM_BUILD RAM-guard (dropped DOCKER_BUILD approach). Tightened answer-evidence heading contract to component-scoped bodies (rejects sibling h2). Codex P2 already resolved. | Vitest heading+guard 4/4; merge-tree CLEAN; no provider-backed checks. | | 2026-07-28 | PR #1309 / `claude/gates-skill` | `7dfe103bfa408052c9e899211b8373c7ccb708d3` | Conflict sync + Codex/CodeRabbit + Bugbot | FIXED. GitHub CONFLICTING/DIRTY was main-staleness only (`merge-tree` clean); merged `origin/main`. Codex P2: skill wrongly claimed `verify:ui` exits 0 under heavy-lock contention — corrected to 15m queue then exit 1 via `run-playwright.mjs`; mirrored in AGENTS.md. CodeRabbit: marked `${PIPESTATUS[0]}` as Bash-specific. Bugbot: zero `cursor[bot]` findings; confirmed same P2. No CI failures on prior tip. | `prettier --check` PASS; `docs:check-links` 1287 PASS; no provider-backed checks. | -| 2026-07-28 | PR #1297 / `motion-audit-fixes-clean` | `c5a145ddbcf66ebcb4c32acf75c6b9588e253b2a` | CI babysit + Bugbot + conflict closeout | READY. Merged origin/main; resolved real Dockerfile/guard-next-build conflicts by adopting main's evaluateNextBuildRamGuard + ALLOW_LOW_RAM_BUILD. Motion fixes retained: ISSUE-02/05, IMP-01 spring token, IMP-04 shimmer ::after wiring, IMP-02 AnswerSkeleton stagger. Bugbot re-review: no remaining P0/P1/P2. 0 review threads. Hosted PR required SUCCESS. | focused vitest earlier; guard-next-build Vitest 3/3; hosted Build/Static/Unit/Production UI/PR required SUCCESS; no provider checks. | -| 2026-07-28 | PR #1297 / `motion-audit-fixes-clean` | `68b3d1de343ef1164d389925a9f45d4dc1106de2` | Review-thread disposition | RESOLVED Codex P2 (shimmer already wired) + CodeRabbit reduced-motion shimmer kill (explicit `animation: none` on `::after`). Threads replied + resolved. | prior tip PR required SUCCESS; awaiting exact-head recheck; no provider checks. | +| 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `060d81355b97c3ba51232df8feee6132f1036b63` | Final closeout: conflicts + reviews + CI green | COMPLETED. Merged main (adopted main RAM-guard). CodeRabbit intrinsic-size + Codex will-change/transform threads resolved. Local Bugbot: no P0/P1. Hosted PR required SUCCESS (Build/Unit/Static/Advisory/Production UI). mergeable CLEAN. | Hosted CI run 30330463512 SUCCESS; local vitest 4114 + focused guard/nav/container 18; no provider checks. | | 2026-07-28 | PR #1294 / `execute-typography-fixes-clean-2` | `62ddd24dc8ad223cde67373ea35e18aee6065057` | CI green closeout | APPROVE. Hosted Production UI + PR required PASS on product tip `e5543dc6`. Codex/CodeRabbit threads resolved (0 open). Unique delta: diagnosis-detail S: locator + heading hierarchy contract. RAM-guard owned by main #1307. | Hosted Static/Unit/Safety/Advisory/Production UI/PR required PASS; Build/Container skipped (unchanged); Bugbot clean; no provider-backed checks. | | 2026-07-28 | PR #1294 / `execute-typography-fixes-clean-2` | `f64fa84a8010008917619c16aac79e4f172a70ff` | Ledger dedupe after main sync | Removed 2 exact duplicate #1307 rows introduced by merge=union during main sync (kept first copies). Hosted required checks green on prior product tip `e5543dc6`; this tip is ledger hygiene + docs-only main sync. | `check:branch-review-ledger` PASS after dedupe; no provider-backed checks. | +| 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `957a79e4` | Re-inspect: main sync + Bugbot | FIXED CONFLICTING/DIRTY: 1 commit behind main (#1294 typography) — ledger/tests auto-merged clean. Prior CodeRabbit/Codex threads remain resolved. Local Bugbot: no P0/P1. Residual only intentional z-token demotions for mockup/popover. | lint+typecheck PASS; focused vitest 19/19; awaiting exact-head hosted CI; no provider checks. | +| 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `129edaf39e76143527fee6a56356fc750d45bb75` | Re-inspect closeout: CI green + ledger dedupe | COMPLETED. Root cause of CONFLICTING was 1-commit main drift (#1294); synced clean. Deduped exact #1306 ledger row from merge=union. Prior review threads remain resolved. Bugbot re-run: no P0/P1. Hosted PR required SUCCESS on this tip. mergeable CLEAN. | lint/typecheck/format:check PASS; vitest 4122; local build PASS; hosted CI 30331626271 SUCCESS; no provider checks. | +| 2026-07-28 | PR #1297 / `motion-audit-fixes-clean` | `c5a145ddbcf66ebcb4c32acf75c6b9588e253b2a` | CI babysit + Bugbot + conflict closeout | READY. Merged origin/main; resolved real Dockerfile/guard-next-build conflicts by adopting main's evaluateNextBuildRamGuard + ALLOW_LOW_RAM_BUILD. Motion fixes retained: ISSUE-02/05, IMP-01 spring token, IMP-04 shimmer ::after wiring, IMP-02 AnswerSkeleton stagger. Bugbot re-review: no remaining P0/P1/P2. 0 review threads. Hosted PR required SUCCESS. | focused vitest earlier; guard-next-build Vitest 3/3; hosted Build/Static/Unit/Production UI/PR required SUCCESS; no provider checks. | +| 2026-07-28 | PR #1297 / `motion-audit-fixes-clean` | `68b3d1de343ef1164d389925a9f45d4dc1106de2` | Review-thread disposition | RESOLVED Codex P2 (shimmer already wired) + CodeRabbit reduced-motion shimmer kill (explicit `animation: none` on `::after`). Threads replied + resolved. | prior tip PR required SUCCESS; awaiting exact-head recheck; no provider checks. | | 2026-07-28 | PR #1297 / `motion-audit-fixes-clean` | tip-after-ledger-dedupe | Ledger hygiene | Dropped 1 exact-duplicate #1306 row introduced by ledger merge=union during main sync; first copy retained. | check:branch-review-ledger PASS; no provider checks. | | 2026-07-28 | PR #1297 / `motion-audit-fixes-clean` | `9997ac9944e7e67135570cd86cc056da5d84aa0a` | Ledger hygiene closeout | SUPERSEDES prior placeholder tip row. Dropped 1 exact-duplicate #1306 record from merge=union; motion/a11y product tip unchanged (`68b3d1de` + main sync). | check:branch-review-ledger PASS; awaiting exact-head hosted CI; no provider checks. | | 2026-07-28 | PR #1297 / `motion-audit-fixes-clean` | `8ef0c1b2d63451c51e8886e8ea076aad56498576` | CI babysit + Bugbot + review closeout | READY. Motion audit complete (ISSUE-02/05, IMP-01/02/04); reduced-motion shimmer kill; overlap gotoHome flake hardened; main synced; RAM-guard conflicts adopted main's ALLOW_LOW_RAM_BUILD. Codex + CodeRabbit threads resolved. Hosted PR required SUCCESS. | Hosted Build/Static/Unit/Advisory/Production UI/PR required SUCCESS; Bugbot clean; no provider checks. | @@ -1169,6 +1175,7 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie | 2026-07-28 | PR #1310 / claude/branch-review-ledger-fixes-42575f | 7c870c139211a419fe8b4dfacae3195a7a7caa2b | PR babysit: CI + Codex P2s + Bugbot-equivalent | Hosted PR required SUCCESS on 7c870c13. Fixed 3 Codex P2s (exact scope match, supersede mints distinct scope, verify full SHAs via git rev-parse) plus n/a-embedded hex and parenthetical ref-token false matches. 3 review threads replied+resolved. Mergeable; 0 behind main. Hosted Cursor Bugbot check not produced — bot-authored bugbot run/cursor review comments ignored; local Bugbot-style review done and defects fixed. | check:branch-review-ledger PASS; vitest repo-hygiene 25/25; lint; typecheck; full vitest 4133 pass; hosted Static/Unit/Build/Safety/PR-required SUCCESS. No provider-backed gates. | | 2026-07-28 | PR #1305 / execute-audit-remediation-fixes | 71ed5083616acff5b820069f2fd9835eab92ec88 | CI/conflict babysit + Bugbot + ledger merge residue | FIXED. Merged origin/main (#1310 ledger repair); converted 4 merge-residue heading records into table form (unique 2026-07-26 review kept as six-cell row). PR MERGEABLE + PR required PASS on this tip. Unresolved review threads 0. Bugbot-equivalent product-diff review: no P0/P1/P2; @cursor review requested. Trust gating + upload fail-closed + SettingsStateProvider + phone chrome viewport breakpoints retained. | check:branch-review-ledger PASS; focused vitest 41/41; hosted PR policy/Static/Build/Unit/Safety/Advisory/Production UI/PR required PASS; no provider-backed checks. | | 2026-07-28 | codex/universal-search-live-test-fix | 9f5994069cddb8a58308d9d5acb9403948a7f617 | live universal-search owner test handoff | APPROVE. Test-only fix aligns live owner coverage with the intentional federated and focused document timeout contract; no production behavior changed. | Node TypeScript syntax PASS; git diff --check PASS; full local gates blocked by an active exclusive repository lease; hosted required checks pending; no live provider tests run. | +| 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `067a17f8` | Resolve merge conflict with main | FIXED. GitHub CONFLICTING/DIRTY was 2 commits behind main (#1310 ledger repair + #1320 search test). `git merge origin/main` completed with ort auto-merge (ledger repaired by main); no content conflicts in product files. Unique CSS layout audit delta retained. | `check:branch-review-ledger` PASS; merge-tree clean vs origin/main; no provider checks. | | 2026-07-28 | PR #1304 / `fix-test-run-lock` | `7cc32c053c752bef19f3de408a1376428e54af74` | CI babysit: sync main | FIXED. GitHub CONFLICTING/DIRTY was staleness only (`git merge-tree` CLEAN; 7 behind). Merged origin/main. Required CI was already SUCCESS on prior tip `e6b826ed`; no product conflicts. Unique vs main remains knip.json (+ ledger). Bugbot/review threads: none unresolved. | merge-tree CLEAN; merge origin/main; no provider-backed checks. | | 2026-07-28 | PR #1310 / `claude/branch-review-ledger-fixes-42575f` (merged) | 422e43d86a69c88368454065c2b117f5982a43d6 | prlanded | LANDED as squash 422e43d86. Ledger repair + lookup/append tooling + hardened guard all present on main; guard PASS at 1107 records and repo-hygiene 25/25. Review improved the branch before merge and main is ahead of the authoring branch: findReviews now compares scope exactly (the original substring match would have let a branch-cleanup-deletion-pending row satisfy a branch-cleanup lookup and skip a branch that still needed cleanup), refTokens no longer false-hits on bare parenthetical prose, headMatches accepts an annotated 'sha (squash)' cell and rejects 'n/a - see ', and resolveHead now verifies full-length hex so a mistyped 40-char string cannot become an unmatchable HEAD. Authoring branch was deleted at merge; its unpushed local ledger-record commit was superseded by this row rather than pushed. | npm run check:branch-review-ledger PASS (1107 records) and vitest tests/repo-hygiene.test.ts 25/25 PASS, both run against origin/main after the merge. Pre-merge npm run verify:pr-local PASS on the merged tree (405 files / 4126 tests, build 3.7min). No provider-backed checks run. | | 2026-07-28 | PR #1305 / execute-audit-remediation-fixes | b101b69631edfe51bcbbc8f6c07e47157fe2c4e8 | CI green closeout after main re-sync | APPROVE for merge by human. Hosted PR required + Production UI PASS on tip after merging origin/main (#1320). MERGEABLE. Unresolved review threads 0. Bugbot-equivalent: no P0/P1/P2 on unique product delta; @cursor review requested. Product delta retained: clinical-notes trust gating answer wipe, SettingsStateProvider wiring, z-index ladder, OverlayProvider/card fixes, phone chrome viewport breakpoints. | Hosted PR policy/Static/Build/Unit/Safety/Advisory/Production UI/PR required PASS on b101b696; check:branch-review-ledger PASS; no provider-backed checks. | @@ -1176,7 +1183,10 @@ Records before 2026-07-28 were written by hand and had drifted: 146 lines carrie | 2026-07-28 | PR #1304 / `fix-test-run-lock` | `2abe39506f9f0ceaf8cf638bfa2b6dc37dc0ed2c` | CI babysit: resync main | FIXED. After green PR required on `f7d86c27`, main advanced by 1 commit (#1305); GitHub DIRTY again but `git merge-tree` CLEAN. Merged origin/main. No product conflicts; unique delta still knip.json (+ ledger). | merge-tree CLEAN; hosted PR required SUCCESS on prior tip; no provider-backed checks. | | 2026-07-28 | PR #1304 / `fix-test-run-lock` | `463e5c0adc77fe722e20376666f5991db3e288d9` | CI babysit closeout | MERGE-READY. Hosted PR required SUCCESS on exact tip; mergeable=MERGEABLE; 0 behind main; merge-tree CLEAN. Unique product delta: knip.json removes unused tailwindcss ignoreDependencies. Prior GitHub DIRTY labels during babysit were main-churn only. | Hosted CI run success on 463e5c0a; no provider-backed checks. | | 2026-07-28 | PR #1304 / fix-test-run-lock | 7cc32c053c752bef19f3de408a1376428e54af74 | CI babysit: sync main | SUPERSEDED (documenting stale-CI ledger error). Prior row for this HEAD incorrectly treated hosted required-CI SUCCESS on earlier tip e6b826ed9150f312c2e7a957f715019e73a7f0be as verification of this later merge commit 7cc32c05. No hosted required-CI result exists for this exact SHA. This ref has since advanced; the later row at 463e5c0adc77fe722e20376666f5991db3e288d9 recorded exact-tip hosted CI SUCCESS, so this commit's status is historical/superseded. | No hosted CI run on this exact HEAD; prior row reused results from e6b826ed; corrective ledger entry only. | +| 2026-07-28 | PR-1296 | 58a6fe241de122c81822a5f60064cc2c49b2f245 | PR #1296 full diff vs origin/main | FIXED P1 z-index ladder bypass; approve after exact-head CI | Removed semantic z-index utilities that lowered established overlay rungs and bypassed main lint; diff check pending; prior exact-head PR required and Production UI passed; refreshed exact-head CI pending | | 2026-07-28 | PR #1337 / claude/circleci-config-decision-f1efc6 | eb2f8f867de13ca64fbdb5e95242e7e5067be6dc | CI babysit: CircleCI verify + Codex P2s | FIXED. Root causes: (1) CircleCI missing PyMuPDF/pip so CI=1 pdf-extractor prerequisite failed; (2) naive docs-only md regex false-greened policy Markdown; (3) Bugbot P1: non-root npm install -g fails on cimg/node. Fixed by ci-change-scope docs_only halt, apt python3-pip + PyMuPDF==1.28.0, npm 11.x assert. Both Codex threads resolved. brace-expansion already on main via #1314. | format:check; contract Vitest 5/5; pdf-extractor 6/6 with CI=1+PyMuPDF; typecheck; lint; Bugbot re-review no remaining bugs; hosted CI/CircleCI re-running on eb2f8f86 | | 2026-07-28 | PR #1337 / claude/circleci-config-decision-f1efc6 | e9bcebb9ef7baddb2eed961bf40bbd89a358b41b | CI babysit: CircleCI verify green | FIXED. Root cause of CircleCI red: cimg/node:24.9 fails engine-strict npm ci because @babel/parser requires Node >=24.11. Switched to cimg/node:24.18. Also addressed Codex P2s (ci-change-scope docs-only; PyMuPDF venv+PYTHON_BIN). Hosted ci/circleci: verify SUCCESS on tip; GH PR required SUCCESS. Codex threads previously resolved. | circleci config validate; contract Vitest 6/6; hosted CircleCI verify SUCCESS (job 225); hosted PR required/Static/Unit SUCCESS | | 2026-07-28 | cursor/mode-secondary-navigation-dc4e | ea9f74062269cc820af5fdc9d3d1ce09e5cf0a4a | pr-1336-babysit | fixed P1 Suspense+DocumentViewer double-nav; P2 documents/search gate; CI babysit | verify:cheap,vitest-70+4189,tsc,eslint,build | | 2026-07-28 | cursor/mode-secondary-navigation-dc4e | e33997dbdd4a62689ff17ff1f536d9e830ae22ad | pr-1336-babysit | fixed Codex P1 horizontal scroll + P2 service section ids; merged main | vitest-62,eslint,tsc | +| 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `84d846d8d0d168ca2babcc6d699e0a88bb0379c0` | Inspect closeout: sync main + forced-colors scope | FIXED. GitHub CONFLICTING was unpushed main sync (local merge-tree CLEAN, 3 behind on remote tip). Pushed merge. Bugbot P2: removed broad forced-colors `!important` wipe on `.edge-glass-header`/`[aria-selected=true]`/`.surface-raised` (token remap retained; header Canvas fill already earlier). Mockup board `z-[2147483647]` → ladder `z-[100]`. Prior CodeRabbit/Codex threads remain resolved. | lint/typecheck/format:check PASS; vitest 4197; local build PASS; no provider checks. | +| 2026-07-28 | PR #1296 / `css-layout-audit-complete` | `f7c34fd7` | Sync main #1297 CSS/motion audit | FIXED. GitHub CONFLICTING after main landed #1297; `git merge origin/main` clean (globals.css + ledger auto-merge). Retained content-auto, scoped forced-colors token remap, mockup z-[100]. Prior tip hosted PR required SUCCESS. | merge-tree CLEAN; focused nav test PASS; no provider checks. | diff --git a/src/app/globals.css b/src/app/globals.css index bbcf0ca323..4acf352a94 100644 --- a/src/app/globals.css +++ b/src/app/globals.css @@ -123,6 +123,10 @@ /* Theme tokens */ :root { + /* Layout constraints */ + --content-max-width: 1440px; + --rail-width: 480px; + /* Clinical White: true-neutral graphite-to-ash ramp (de-blued). */ --neutral-0: #ffffff; --neutral-50: #f7f8fa; @@ -1809,7 +1813,7 @@ summary::-webkit-details-marker { beside the main column. */ .document-viewer-composer.floating-composer-edge { left: max(2rem, var(--safe-area-left)); - right: calc(max(0px, (100vw - 1440px) / 2) + 480px + 3.5rem + var(--safe-area-right)); + right: calc(max(0px, (100vw - var(--content-max-width)) / 2) + var(--rail-width) + 3.5rem + var(--safe-area-right)); } } @@ -2948,7 +2952,10 @@ html.theme-transitioning *:after { } } -/* Windows High Contrast / Forced Colors Compatibility (Improvement 2.3) */ +/* Windows High Contrast / Forced Colors Compatibility (Improvement 2.3). + Token remap only — glass-header Canvas fills live in the earlier + forced-colors block; do not blanket-reset [aria-selected="true"] or + invent unused .surface-raised selectors with !important. */ @media (forced-colors: active) { :root, .dark { diff --git a/src/components/AccessibleTable.tsx b/src/components/AccessibleTable.tsx index eee94613ff..324251b637 100644 --- a/src/components/AccessibleTable.tsx +++ b/src/components/AccessibleTable.tsx @@ -98,7 +98,7 @@ function AccessibleTableMarkup({ return (
diff --git a/src/components/DocumentManagementActions.tsx b/src/components/DocumentManagementActions.tsx index b7ed680672..5f4c5dfdf1 100644 --- a/src/components/DocumentManagementActions.tsx +++ b/src/components/DocumentManagementActions.tsx @@ -254,7 +254,7 @@ export function DocumentManagementActions({ disabled={pending} /> -

Original file: {document.file_name}

+

Original file: {document.file_name}