fix(therapy-compass): redesign Pathways mobile picker and step stack

data/repo-awareness-snapshot.json

@@ -1,8 +1,8 @@
 {
  "version": "repo-awareness-snapshot-v1",
  "captured_revision": {
- "sha": "1d9e1234e1c6c3d642efed81ebe65a6056d5d120",
- "committed_at": "2026-08-27T11:37:53+00:00"
+ "sha": "2cfbe5647dc919f367486a4d4c2102524329b7c6",
+ "committed_at": "2026-08-27T11:41:19+00:00"
  },
  "routes": {
  "modes": [
@@ -3959,6 +3959,38 @@
  "outcome": "Re-synced with origin/main after PR #2416 landed (mechanical conflict only in data/repo-awareness-snapshot.json, regenerated). Also found and fixed a real regression from the earlier sweep's conflict resolution: DsmCompareChrome's onCommit had dropped ids.filter(Boolean) that origin/main carries, producing ids=,bipolar-ii-disorder after removing the first of two selections instead of ids=bipolar-ii-disorder -- exactly the failure tests/ui-route-coverage.spec.ts caught in CI. Restored the filter to match main. Added the required Clinical Governance Preflight section to the PR body since the regenerated snapshot touches data/. No review threads outstanding.",
  "checks": "npx vitest run tests/compare-catalog.test.ts tests/compare-ids-chrome.dom.test.tsx tests/compare-slot-strip.dom.test.tsx tests/therapy-compare-phone-layout.dom.test.tsx tests/therapy-compare-tray.dom.test.tsx tests/phone-dock-addon-contract.test.ts -- 57 passed; no provider-backed checks run"
  },
+ {
+ "date": "2026-08-27",
+ "ref": "codex/therapy-pathways-redesign (PR #2413)",
+ "head": "05e00976ab04796d7f44593b692f46cb8919943e",
+ "scope": "Run PR follow-up: second merge-drift + PR-policy fix",
+ "outcome": "Re-synced with origin/main after PR #2416 landed (mechanical conflict only in data/repo-awareness-snapshot.json, regenerated); added the required Clinical Governance Preflight section to the PR body since the regenerated snapshot touches data/, a path scripts/pr-policy.mjs treats as clinical-risk. No review threads outstanding.",
+ "checks": "npx vitest run tests/therapy-pathways-mobile.dom.test.tsx tests/therapy-compass-responsive-contract.test.ts tests/playwright-pr-shards.test.ts -- 25 passed; no provider-backed checks run"
+ },
+ {
+ "date": "2026-08-27",
+ "ref": "codex/therapy-pathways-redesign (PR #2413)",
+ "head": "be2d9b0291965258e5933006dc9b914961bb0dc3",
+ "scope": "Run PR sweep: CI fix + threads + drift",
+ "outcome": "CI was already green pre-sweep (PR required: success on d3f39c4); branch was already current with main (0 behind), so no drift merge needed. Fixed 3 unresolved CodeRabbit review threads: (1) pathway/step 'linked steps' counts used total steps.length instead of counting only steps with a matched therapySlug — added pathwayLinkedStepCount() and wired it into pathway-review-label.ts, pathway-picker-sheet.tsx (x2), pathway-step-stack.tsx; (2) ui-therapy-pathways.spec.ts readCautionGeometry substituted +/-Infinity for a missing caution/dock element, masking a broken overlap assertion — now throws if either is absent; (3) the 'Change pathway' Playwright test only re-clicked the already-active Anxiety pathway row, which could pass with a broken selectPathway — now selects Mood pathway, asserts the URL/heading changed, then switches back to Anxiety before the anxiety-scoped scroll assertions. All 3 threads replied with commit SHA and resolved.",
+ "checks": "npx vitest run tests/therapy-pathways-mobile.dom.test.tsx tests/therapy-compass-responsive-contract.test.ts tests/playwright-pr-shards.test.ts tests/therapy-compass-pathways.test.ts -- 34 passed; npm run lint -- exit 0 (gate-receipts pass); npm run typecheck -- exit 0, run foreground twice, gate-receipts recorded pass for typecheck:internal (5613 input files); npm run format -- applied (1 file reformatted, amended into commit); no provider-backed checks run"
+ },
+ {
+ "date": "2026-08-27",
+ "ref": "codex/therapy-pathways-redesign (PR #2413)",
+ "head": "d2bbc44f084707d8844485c67c8cd76d57e04b39",
+ "scope": "Run PR sweep: CI fix + threads + drift",
+ "outcome": "Before: mergeable_state dirty (conflict vs main), all real CI green on prior head (19fca0d), 0 unresolved review threads (4 PR comments were all automated bot usage-limit/status noise, no actionable findings). After: merged origin/main (b183dc65) into branch; only conflicts were generated files docs/design-system/COMPONENTS.md and docs/design-system/adoption-manifest.json, resolved by regenerating via 'node scripts/generate-design-system-adoption.mjs --write' (no manual edits); PR mergeability check now succeeds. No review threads needed action (none existed). No CI fix needed beyond the merge itself.",
+ "checks": "npx vitest run tests/therapy-pathways-mobile.dom.test.tsx tests/therapy-compass-responsive-contract.test.ts tests/playwright-pr-shards.test.ts -- 25 passed; npm run check:design-system-adoption -- design system adoption checked: 55 components, 76 roots; npm run check:playwright-pr-shards -- PR UI shard parity OK: 31 production specs across 3 groups; pre-commit hook ran docs:check-inventory and docs:check-index -- both current. No provider-backed checks run."
+ },
+ {
+ "date": "2026-08-27",
+ "ref": "codex/therapy-pathways-redesign (PR #2413)",
+ "head": "f8537abbef36b5e9f72eb1c745f565253a276153",
+ "scope": "Run PR sweep: CI fix + threads + drift",
+ "outcome": "Before: PR required failing (Static PR checks red on check:repo-awareness-snapshot — committed data/repo-awareness-snapshot.json review_state was stale after prior sweep's ledger/docs commits); 0 unresolved review threads; branch already contained origin/main (no drift). After: ran npm run snapshot:repo-awareness to regenerate the snapshot, verified npm run check:repo-awareness-snapshot and npm run format:changed pass, committed and pushed f8537ab. All other required jobs (Build, Unit coverage, Production UI critical, Safety and config checks, Production UI shards 1-3, Lighthouse budget) were already green before this fix and untouched by it. No review threads existed to address. No main-sync needed (branch already up to date).",
+ "checks": "npm run check:repo-awareness-snapshot (PASS, reproduces prior CI failure then passes after regen); npm run format:changed (PASS); npm run snapshot:repo-awareness (regenerated data/repo-awareness-snapshot.json); no provider-backed checks run"
+ },
  {
  "date": "2026-08-27",
  "ref": "codex/tooling-hooks-ci-invariants",
@@ -25001,8 +25033,8 @@
  }
  ],
  "counts": {
- "records": 2639,
- "refs": 1610
+ "records": 2643,
+ "refs": 1611
  }
  }
 }

docs/branch-review-records/093e552fb22a9e87c899dd093a3c133dd13a262ad6b2fc4fbb5b5a6deee67d93.record.md

@@ -0,0 +1 @@
+| 2026-08-27 | codex/therapy-pathways-redesign (PR #2413) | 05e00976ab04796d7f44593b692f46cb8919943e | Run PR follow-up: second merge-drift + PR-policy fix | Re-synced with origin/main after PR #2416 landed (mechanical conflict only in data/repo-awareness-snapshot.json, regenerated); added the required Clinical Governance Preflight section to the PR body since the regenerated snapshot touches data/, a path scripts/pr-policy.mjs treats as clinical-risk. No review threads outstanding. | npx vitest run tests/therapy-pathways-mobile.dom.test.tsx tests/therapy-compass-responsive-contract.test.ts tests/playwright-pr-shards.test.ts -- 25 passed; no provider-backed checks run |

docs/branch-review-records/7895a1814c28b2fe53fd6877a91f0bc86535eee70ed77c22032f6989646a2a1d.record.md

@@ -0,0 +1 @@
+| 2026-08-27 | codex/therapy-pathways-redesign (PR #2413) | f8537abbef36b5e9f72eb1c745f565253a276153 | Run PR sweep: CI fix + threads + drift | Before: PR required failing (Static PR checks red on check:repo-awareness-snapshot — committed data/repo-awareness-snapshot.json review_state was stale after prior sweep's ledger/docs commits); 0 unresolved review threads; branch already contained origin/main (no drift). After: ran npm run snapshot:repo-awareness to regenerate the snapshot, verified npm run check:repo-awareness-snapshot and npm run format:changed pass, committed and pushed f8537ab. All other required jobs (Build, Unit coverage, Production UI critical, Safety and config checks, Production UI shards 1-3, Lighthouse budget) were already green before this fix and untouched by it. No review threads existed to address. No main-sync needed (branch already up to date). | npm run check:repo-awareness-snapshot (PASS, reproduces prior CI failure then passes after regen); npm run format:changed (PASS); npm run snapshot:repo-awareness (regenerated data/repo-awareness-snapshot.json); no provider-backed checks run |

docs/branch-review-records/ad2f2cc55ed454d781f0501ef25a67e6f7bbaf869908014dfceacbcd2b8838d4.record.md

@@ -0,0 +1 @@
+| 2026-08-27 | codex/therapy-pathways-redesign (PR #2413) | be2d9b0291965258e5933006dc9b914961bb0dc3 | Run PR sweep: CI fix + threads + drift | CI was already green pre-sweep (PR required: success on d3f39c4); branch was already current with main (0 behind), so no drift merge needed. Fixed 3 unresolved CodeRabbit review threads: (1) pathway/step 'linked steps' counts used total steps.length instead of counting only steps with a matched therapySlug — added pathwayLinkedStepCount() and wired it into pathway-review-label.ts, pathway-picker-sheet.tsx (x2), pathway-step-stack.tsx; (2) ui-therapy-pathways.spec.ts readCautionGeometry substituted +/-Infinity for a missing caution/dock element, masking a broken overlap assertion — now throws if either is absent; (3) the 'Change pathway' Playwright test only re-clicked the already-active Anxiety pathway row, which could pass with a broken selectPathway — now selects Mood pathway, asserts the URL/heading changed, then switches back to Anxiety before the anxiety-scoped scroll assertions. All 3 threads replied with commit SHA and resolved. | npx vitest run tests/therapy-pathways-mobile.dom.test.tsx tests/therapy-compass-responsive-contract.test.ts tests/playwright-pr-shards.test.ts tests/therapy-compass-pathways.test.ts -- 34 passed; npm run lint -- exit 0 (gate-receipts pass); npm run typecheck -- exit 0, run foreground twice, gate-receipts recorded pass for typecheck:internal (5613 input files); npm run format -- applied (1 file reformatted, amended into commit); no provider-backed checks run |

docs/branch-review-records/d99361a8c9a53049ee45227f0230020721e9aebb18ee7548804835c510538765.record.md

@@ -0,0 +1 @@
+| 2026-08-27 | codex/therapy-pathways-redesign (PR #2413) | d2bbc44f084707d8844485c67c8cd76d57e04b39 | Run PR sweep: CI fix + threads + drift | Before: mergeable_state dirty (conflict vs main), all real CI green on prior head (19fca0d), 0 unresolved review threads (4 PR comments were all automated bot usage-limit/status noise, no actionable findings). After: merged origin/main (b183dc65) into branch; only conflicts were generated files docs/design-system/COMPONENTS.md and docs/design-system/adoption-manifest.json, resolved by regenerating via 'node scripts/generate-design-system-adoption.mjs --write' (no manual edits); PR mergeability check now succeeds. No review threads needed action (none existed). No CI fix needed beyond the merge itself. | npx vitest run tests/therapy-pathways-mobile.dom.test.tsx tests/therapy-compass-responsive-contract.test.ts tests/playwright-pr-shards.test.ts -- 25 passed; npm run check:design-system-adoption -- design system adoption checked: 55 components, 76 roots; npm run check:playwright-pr-shards -- PR UI shard parity OK: 31 production specs across 3 groups; pre-commit hook ran docs:check-inventory and docs:check-index -- both current. No provider-backed checks run. |

docs/design-system/COMPONENTS.md

@@ -976,7 +976,7 @@ This generated snapshot is a local source-derived inventory. It does not assert
 | `AnswerFooter` | answer | yes | yes | inherited-global-root | yes | no | 1 |
 | `AsyncButton` | controls | yes | yes | inherited-global-root | yes | no | 4 |
 | `Breadcrumb` | layout | yes | yes | inherited-global-root | yes | no | 1 |
-| `Button` | controls | yes | yes | inherited-global-root | yes | no | 17 |
+| `Button` | controls | yes | yes | inherited-global-root | yes | no | 19 |
 | `Checkbox` | controls | yes | yes | no | yes | no | 0 |
 | `Chip` | controls | yes | yes | inherited-global-root | yes | no | 5 |
 | `ChoiceChip` | controls | yes | yes | inherited-global-root | yes | no | 4 |
@@ -1012,7 +1012,7 @@ This generated snapshot is a local source-derived inventory. It does not assert
 | `SearchField` | controls | yes | yes | no | yes | no | 0 |
 | `SegmentedControl` | controls | yes | yes | inherited-global-root | yes | no | 8 |
 | `Select` | controls | yes | yes | inherited-global-root | yes | no | 2 |
-| `Sheet` | layout | yes | yes | inherited-global-root | yes | no | 31 |
+| `Sheet` | layout | yes | yes | inherited-global-root | yes | no | 32 |
 | `Skeleton` | feedback | yes | yes | inherited-global-root | yes | no | 6 |
 | `SourceDesignationBadge` | source | yes | yes | inherited-global-root | yes | no | 4 |
 | `SourceProvenance` | source | yes | yes | inherited-global-root | yes | no | 1 |

docs/design-system/adoption-manifest.json

@@ -281,6 +281,8 @@
  "src/components/AccessibleTable.tsx",
  "src/components/clinical-dashboard/signed-image.tsx",
  "src/components/privacy-quiet-signal-page.tsx",
+ "src/components/therapy-compass/pathway-picker-sheet.tsx",
+ "src/components/therapy-compass/pathway-step-stack.tsx",
  "src/components/therapy-compass/recommend-scenario-control.tsx",
  "src/components/therapy-compass/record/compare-action.tsx",
  "src/components/therapy-compass/screens/brief-screen.tsx",
@@ -300,6 +302,8 @@
  "src/components/AccessibleTable.tsx",
  "src/components/clinical-dashboard/signed-image.tsx",
  "src/components/privacy-quiet-signal-page.tsx",
+ "src/components/therapy-compass/pathway-picker-sheet.tsx",
+ "src/components/therapy-compass/pathway-step-stack.tsx",
  "src/components/therapy-compass/recommend-scenario-control.tsx",
  "src/components/therapy-compass/record/compare-action.tsx",
  "src/components/therapy-compass/screens/brief-screen.tsx",
@@ -1663,6 +1667,7 @@
  "src/components/forms/form-priority-facts-section.tsx",
  "src/components/in-page-nav/in-page-nav-header.tsx",
  "src/components/mode-nav/mode-nav.tsx",
+ "src/components/therapy-compass/pathway-picker-sheet.tsx",
  "src/components/therapy-compass/recommend-scenario-control.tsx",
  "src/components/therapy-compass/record/key-facts.tsx",
  "src/components/therapy-compass/therapy-compare-tray.tsx",
@@ -1697,6 +1702,7 @@
  "src/components/forms/form-priority-facts-section.tsx",
  "src/components/in-page-nav/in-page-nav-header.tsx",
  "src/components/mode-nav/mode-nav.tsx",
+ "src/components/therapy-compass/pathway-picker-sheet.tsx",
  "src/components/therapy-compass/recommend-scenario-control.tsx",
  "src/components/therapy-compass/record/key-facts.tsx",
  "src/components/therapy-compass/therapy-compare-tray.tsx",

playwright.config.ts

@@ -23,15 +23,15 @@ const chromiumExecutablePath = process.env.PLAYWRIGHT_CHROMIUM_EXECUTABLE_PATH;
 // `tests/playwright-project-isolation.test.ts` asserts every such file on disk is
 // matched here.
 const productionSpecPattern =
- /.*(?:answer-progress-ui-smoke|dsm-ui-smoke|ui-(smoke|stress|accessibility|caring-contacts-workspace|clinical-ask|dictionary|document-canvas|tools|overlap|universal-search|specifiers|formulation(?:-result-cards)?|forms-section-nav|chrome-scroll|therapy-nav-scroll|mode-nav-density|phone-motion|phone-scroll(?:-[a-z0-9-]+)?|pwa|route-coverage|style-contract|visual-artifacts|hydration))\.spec\.ts/;
+ /.*(?:answer-progress-ui-smoke|dsm-ui-smoke|ui-(smoke|stress|accessibility|caring-contacts-workspace|clinical-ask|dictionary|document-canvas|tools|overlap|universal-search|specifiers|formulation(?:-result-cards)?|forms-section-nav|chrome-scroll|therapy-nav-scroll|therapy-pathways|mode-nav-density|phone-motion|phone-scroll(?:-[a-z0-9-]+)?|pwa|route-coverage|style-contract|visual-artifacts|hydration))\.spec\.ts/;
 const mockupSpecPattern =
  /.*ui-(answer-chat-perfected-mockup|care-plan-mockup|caring-contact-mockup|document-image-status-mockup|document-top-navigation-mockup|sidebar-live-mockup|therapy-navigation-mockup|tools|tools-collapse|tools-search-mode-mockup|tools-task-directory|ward-management|ward-coordinator|ward-roles|ward-discharges)\.spec\.ts/;
 const mockupTag = /@mockup/;
 
 export default defineConfig({
  testDir: "./tests",
  testMatch:
- /.*(?:answer-progress-ui-smoke|dsm-ui-smoke|ui-(smoke|stress|accessibility|answer-chat-perfected-mockup|care-plan-mockup|caring-contact-mockup|caring-contacts-workspace|clinical-ask|dictionary|document-canvas|document-image-status-mockup|document-top-navigation-mockup|sidebar-live-mockup|therapy-navigation-mockup|tools|tools-collapse|tools-search-mode-mockup|tools-task-directory|ward-(?:management|coordinator|roles|discharges)|overlap|universal-search|specifiers|formulation(?:-result-cards)?|forms-section-nav|chrome-scroll|therapy-nav-scroll|mode-nav-density|phone-motion|phone-scroll(?:-[a-z0-9-]+)?|pwa|route-coverage|style-contract|visual-artifacts|hydration))\.spec\.ts/,
+ /.*(?:answer-progress-ui-smoke|dsm-ui-smoke|ui-(smoke|stress|accessibility|answer-chat-perfected-mockup|care-plan-mockup|caring-contact-mockup|caring-contacts-workspace|clinical-ask|dictionary|document-canvas|document-image-status-mockup|document-top-navigation-mockup|sidebar-live-mockup|therapy-navigation-mockup|tools|tools-collapse|tools-search-mode-mockup|tools-task-directory|ward-(?:management|coordinator|roles|discharges)|overlap|universal-search|specifiers|formulation(?:-result-cards)?|forms-section-nav|chrome-scroll|therapy-nav-scroll|therapy-pathways|mode-nav-density|phone-motion|phone-scroll(?:-[a-z0-9-]+)?|pwa|route-coverage|style-contract|visual-artifacts|hydration))\.spec\.ts/,
  timeout: 60_000,
  retries: 0,
  // Fail the run if a stray `test.only` is committed: otherwise it silently