Cover calculator Show all recovery and record audit acceptance

data/repo-awareness-snapshot.json

@@ -6282,6 +6282,14 @@
         "outcome": "PASS: closure evidence is specific and internally consistent; removed incidental ledger-row reordering",
         "checks": "outstanding-issues guard; Prettier; diff review; zero unresolved threads"
       },
+      {
+        "date": "2026-10-17",
+        "ref": "HEAD",
+        "head": "0537cc935b86824db40ca0c117894b67638e2d95",
+        "scope": "PR 3628 review and fixes",
+        "outcome": "Found atomic worker-scope, comparator-unit, telemetry-budget, and document-source retention defects; fixes prepared with regression coverage.",
+        "checks": "Independent standards and specification inspection of the pinned PR diff; follow-up working-tree targeted proof 295/295 tests and 10 local SQL scenarios; hosted CI pending."
+      },
       {
         "date": "2026-08-06",
         "ref": "claude/pr-handoff-stop-hook (PR #1649)",
@@ -9146,6 +9154,22 @@
         "outcome": "Pending deletion: content already on origin/main or superseded old-lineage snapshot / throwaway CI-retry branch; branch-only-file sweep found no novel work. Remote deletion DENIED (HTTP 403, session lacks ref-delete permission); branch still exists at this HEAD — operator must complete deletion. Scope is deletion-pending so future cleanup passes re-evaluate rather than skip.",
         "checks": "Provider-backed GitHub PR inventory via MCP (400 PRs #264-#674, all merged=false); local read-only git: rewritten-main-aware cherry-pick + two-dot tree + branch-only-file novelty sweep vs origin/main e75fad90; adversarial re-check. Remote `git push --delete` attempted (provider-backed write) and DENIED with HTTP 403. No OpenAI/Supabase/live-eval checks."
       },
+      {
+        "date": "2026-10-18",
+        "ref": "codex/design-audit-acceptance",
+        "head": "2900a6363462efb6eaab614280c7a40b89eb28d8",
+        "scope": "Design audit acceptance follow-up",
+        "outcome": "Independent scoped adversarial review found no confirmed defect in the stateful calculator regression or acceptance handoff. Dedicated same-URL recovery coverage is added; production source is unchanged. Railway health verified, direct HTTP blocked by network proxy, six device/clinical acceptance checks remain explicit.",
+        "checks": "Owning calculator suite26passed3.62s; negative-control removal of clearQuery yields1failed25passed with PHQ-9 retained, then restored26pass. Independent static review verifies modified-click preservation and URL cleanup. Merged545treeequalsreviewed58e302. No browser/fullsuite/release/clinicaleval claimed for newtest-onlydiff."
+      },
+      {
+        "date": "2026-10-18",
+        "ref": "codex/design-audit-acceptance",
+        "head": "2900a6363462efb6eaab614280c7a40b89eb28d8",
+        "scope": "Design audit acceptance follow-up (supersedes 2026-10-18)",
+        "outcome": "Wording correction only (CodeRabbit). Railway deployment status was verified; public HTTP health remains unverified because direct HTTP was blocked by the network proxy. Otherwise unchanged: independent scoped adversarial review found no confirmed defect in the stateful calculator regression or acceptance handoff; dedicated same-URL recovery coverage is added; production source is unchanged; six device/clinical acceptance checks remain explicit.",
+        "checks": "No new checks: restates the superseded record's evidence (owning calculator suite 26 passed; negative-control removal of clearQuery 1 failed 25 passed, restored 26 passed; static review of modified-click preservation and URL cleanup). No public HTTP health, browser, full-suite, release or clinical eval claimed."
+      },
       {
         "date": "2026-07-28",
         "ref": "claude/navigation-pane-mockups-0600af",
@@ -12642,6 +12666,14 @@
         "outcome": "P2 fixed: native install prompt now avoids hero reflow, composer overlap, and prompt-induced CLS",
         "checks": "pwa DOM 10/10; PWA Chromium 5/5; format; exact-head merge-tree"
       },
+      {
+        "date": "2026-10-18",
+        "ref": "main",
+        "head": "54559adabe5b51396c2d1c0eb395f2cf79b43fbf",
+        "scope": "PR 3632 landed-content verification",
+        "outcome": "PR was externally squash-merged during the review at 2026-10-18T05:13:46Z. Landed commit tree exactly equals reviewed head58e30257a45c0e19107b734f5ce2416eb9e6c8ad. No missing work detected. This agent did not merge, arm auto-merge, push or deploy. Manual acceptance and deployment health remain unverified.",
+        "checks": "Fresh origin/main54559adabe5b51396c2d1c0eb395f2cf79b43fbf has one parent; git diff --stat against reviewed head is empty; both tree hashes9e7fab3150b0448392438152c96e459d81a06c9d. Reviewed head required CI39098909136 and PR required117593597228 succeeded. No production-health or provider clinical eval invoked."
+      },
       {
         "date": "2026-07-26",
         "ref": "PR #1246 / `codex/standardize-header-and-footer-behavior`",
@@ -13066,6 +13098,14 @@
         "outcome": "safe remote delete: signed-URL cache invalidation landed more strongly in merged PR #1374 with pre-publish identity clearing and extra tests; archived batch18",
         "checks": "current auth source/test inspection; origin/main pickaxe history; redundant cherry-pick proof; bundle verify"
       },
+      {
+        "date": "2026-10-18",
+        "ref": "codex/design-audit-resolution",
+        "head": "58e30257a45c0e19107b734f5ce2416eb9e6c8ad",
+        "scope": "CI failure and merge-safety review",
+        "outcome": "No confirmed code/CI blocker. Historical failures repaired in published commits; independent scoped review found no defect. All five review threads resolved. Technically ready for owner merge; manual Safari/PWA/keyboard/AT and populated clinical-state acceptance remains unverified. No merge or push performed.",
+        "checks": "Hosted CI 39098909136 at exact head: PR required 117593597228 success; both coverage partitions and aggregate, all production UI shards and critical, builds, static, safety and Lighthouse success; Secret Scan and SAST success. Local audit:final-merge dry-run and git diff --check passed. No new local test suite or provider-backed clinical eval run; reused prior versioned regression evidence."
+      },
       {
         "date": "2026-07-22",
         "ref": "PR #1075 / `codex/reconcile-route-reachability-ast`",
@@ -14010,6 +14050,14 @@
         "outcome": "before: CONFLICTING + PR policy FAIL + unresolved Codex P1 (conflict markers in answer/route.ts). after: conflict markers removed and pushed (80212dd91, 632e84c94); merge origin/main aborted (non-trivial conflicts: privacy/page.tsx, answer-render-policy.ts, source-authority-metadata.ts, upload/route.ts, supabase/drift-manifest.json, settings-dialog, validation/answer-request, plus UI/docs/tests); PR policy still FAIL (Clinical Governance Preflight missing — body edit forbidden this sweep); thread reply/resolve needs parent (comment 3644028277 / thread PRRT_kwDOSh5Fis6Tfkev) — ManagePullRequest/GitHub write MCP unavailable",
         "checks": "typecheck:internal pass; vitest clinical-dashboard-merge-artifacts + visual-evidence-tabs pass (9); no provider-backed checks run"
       },
+      {
+        "date": "2026-10-18",
+        "ref": "codex/design-audit-acceptance",
+        "head": "633a8b7a509b6c0863059f09f849a2ec1a58e8ba",
+        "scope": "Automated acceptance follow-up review",
+        "outcome": "Two confirmed automated-edit issues: a missing usePathname mock prevents the new header fixture from rendering, and an already published immutable review was rewritten. Restore the scoped page/provider harness and original record bytes. The test name explicitly limits its contract; no production shell or physical-device coverage is claimed. Corrections verified locally; publication pending.",
+        "checks": "Independent source review and local reproduction: 1 failed, 25 passed at bot head, missing next/navigation usePathname export. Corrected owning suite: 26 passed. Original regression negative control: removing clearQuery left PHQ-9 in the input. Original immutable record restored byte-for-byte against ed3dc82abd26be523d0d643dc7bfd1545587162e. Production source unchanged."
+      },
       {
         "date": "2026-07-13",
         "ref": "claude/scroll-icon-design-8b7675",

docs/branch-review-records/0f870b9af5a6c1a2295cb7d9462fa0126769cbf78d566bdff78f8e468bce23d0.record.md

@@ -0,0 +1 @@
+| 2026-10-18 | codex/design-audit-acceptance | 2900a6363462efb6eaab614280c7a40b89eb28d8 | Design audit acceptance follow-up (supersedes 2026-10-18) | Wording correction only (CodeRabbit). Railway deployment status was verified; public HTTP health remains unverified because direct HTTP was blocked by the network proxy. Otherwise unchanged: independent scoped adversarial review found no confirmed defect in the stateful calculator regression or acceptance handoff; dedicated same-URL recovery coverage is added; production source is unchanged; six device/clinical acceptance checks remain explicit. | No new checks: restates the superseded record's evidence (owning calculator suite 26 passed; negative-control removal of clearQuery 1 failed 25 passed, restored 26 passed; static review of modified-click preservation and URL cleanup). No public HTTP health, browser, full-suite, release or clinical eval claimed. |

docs/branch-review-records/2c05f34aa08168b0caac1ae79287955541f16c3f91d5a00f2df4c34e0b3223ff.record.md

@@ -0,0 +1 @@
+| 2026-10-18 | main | 54559adabe5b51396c2d1c0eb395f2cf79b43fbf | PR 3632 landed-content verification | PR was externally squash-merged during the review at 2026-10-18T05:13:46Z. Landed commit tree exactly equals reviewed head58e30257a45c0e19107b734f5ce2416eb9e6c8ad. No missing work detected. This agent did not merge, arm auto-merge, push or deploy. Manual acceptance and deployment health remain unverified. | Fresh origin/main54559adabe5b51396c2d1c0eb395f2cf79b43fbf has one parent; git diff --stat against reviewed head is empty; both tree hashes9e7fab3150b0448392438152c96e459d81a06c9d. Reviewed head required CI39098909136 and PR required117593597228 succeeded. No production-health or provider clinical eval invoked. |

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

@@ -0,0 +1 @@
+| 2026-10-18 | codex/design-audit-resolution | 58e30257a45c0e19107b734f5ce2416eb9e6c8ad | CI failure and merge-safety review | No confirmed code/CI blocker. Historical failures repaired in published commits; independent scoped review found no defect. All five review threads resolved. Technically ready for owner merge; manual Safari/PWA/keyboard/AT and populated clinical-state acceptance remains unverified. No merge or push performed. | Hosted CI 39098909136 at exact head: PR required 117593597228 success; both coverage partitions and aggregate, all production UI shards and critical, builds, static, safety and Lighthouse success; Secret Scan and SAST success. Local audit:final-merge dry-run and git diff --check passed. No new local test suite or provider-backed clinical eval run; reused prior versioned regression evidence. |

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

@@ -0,0 +1 @@
+| 2026-10-18 | codex/design-audit-acceptance | 2900a6363462efb6eaab614280c7a40b89eb28d8 | Design audit acceptance follow-up | Independent scoped adversarial review found no confirmed defect in the stateful calculator regression or acceptance handoff. Dedicated same-URL recovery coverage is added; production source is unchanged. Railway health verified, direct HTTP blocked by network proxy, six device/clinical acceptance checks remain explicit. | Owning calculator suite26passed3.62s; negative-control removal of clearQuery yields1failed25passed with PHQ-9 retained, then restored26pass. Independent static review verifies modified-click preservation and URL cleanup. Merged545treeequalsreviewed58e302. No browser/fullsuite/release/clinicaleval claimed for newtest-onlydiff. |

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

@@ -0,0 +1 @@
+| 2026-10-18 | codex/design-audit-acceptance | 633a8b7a509b6c0863059f09f849a2ec1a58e8ba | Automated acceptance follow-up review | Two confirmed automated-edit issues: a missing usePathname mock prevents the new header fixture from rendering, and an already published immutable review was rewritten. Restore the scoped page/provider harness and original record bytes. The test name explicitly limits its contract; no production shell or physical-device coverage is claimed. Corrections verified locally; publication pending. | Independent source review and local reproduction: 1 failed, 25 passed at bot head, missing next/navigation usePathname export. Corrected owning suite: 26 passed. Original regression negative control: removing clearQuery left PHQ-9 in the input. Original immutable record restored byte-for-byte against ed3dc82abd26be523d0d643dc7bfd1545587162e. Production source unchanged. |

docs/design-system/audit-resolution-2026-10-17.md

@@ -124,3 +124,22 @@ The combined production source signature relative to base `07d4ff255` is `13ce8a
 After those browser checks completed, the automated Services edit received its required Prettier cleanup. An esbuild comparison confirmed it emits identical minified JavaScript before and after formatting. The resulting source signature is `729c7ec49cf24f7f2291e903e6865c91ecf0af4441884326e2130e7f81e64398`; the only source difference from the verified build is that formatting cleanup. Latest scoped lint passed, diff integrity reports **47 changed test files, 741 → 797 cases / 27 self-tests passed**, and repository-awareness freshness reports **320 pages, 638 documents, 2803 reviews**.
 
 The CI run on automated head `6c91636b2` reproduced the three now-corrected unit failures, the bot's Services formatting failure, and a Lighthouse desktop Documents-search LCP breach (1247ms against a 961ms baseline; two of three samples exceeded tolerance). The LCP element is an unchanged static paragraph; the cause is not established by that observation. The budget remains unchanged, and the follow-up head's hosted verdict is pending.
+
+## Acceptance follow-up — 18 October 2026
+
+PR [#3632](https://github.com/BigSimmo/PsychSift/pull/3632) merged as `54559adabe5b51396c2d1c0eb395f2cf79b43fbf`. Its tree exactly matches reviewed head `58e30257a45c0e19107b734f5ce2416eb9e6c8ad`. That head passed [CI run 39098909136](https://github.com/BigSimmo/PsychSift/actions/runs/39098909136), including the authoritative `PR required` aggregate, coverage, all production Chromium shards, critical UI, builds, static checks, safety/config and Lighthouse. Secret Scan and SAST passed. This supersedes the pending publication-time statuses above; the release browser matrix was skipped.
+
+A dedicated stateful DOM regression now types a shared calculator query at `/calculators/search`, verifies that Show all targets the same URL, and checks that a plain click clears the composer and restores every calculator. Ctrl, Cmd, Shift and Alt clicks preserve the current query/results. The provider harness exercises the real calculator page with controlled shared state; it does not stand in for a browser test of the entire search shell. Removing the existing `clearQuery` call made this regression fail with `PHQ-9` still in the input; restoring it passed the owning 26-case suite. Production code is unchanged by this follow-up.
+
+Read-only Railway verification observed both production services on the audited squash commit: app deployment `766807f0-6d3b-4433-8515-9333252498fa` settled successfully at 05:46:41 UTC and worker deployment `3a40f0ee-dc5b-4f55-a51f-d9e2bb1a7d3c` at 05:36:14 UTC. Both had one running replica, zero crashed replicas, no warnings/critical notifications, no failed deployments in the preceding eight hours and no pending work. Later `62558e5c` deployments were skipped, so that revision was not live at observation. A direct anonymous `/api/health` request was blocked by the execution environment's network proxy (403); custom-domain HTTP health remains unverified. No deployment, configuration or database change was performed.
+
+### Remaining device and clinical acceptance
+
+Record the tested revision, device/OS/browser, display settings, observed result and evidence for each check. These are unobserved acceptance items, not confirmed defects; the existing clinician hold remains in force.
+
+- [ ] Physical iPhone Safari: navigate Modes, search Services, shortlist/compare, open and dismiss a sheet, scroll to end-of-results actions, and rotate. Confirm reachable 48px targets, no clipped controls and restored focus/context.
+- [ ] Actual installed PWA: install and launch from the device home screen; repeat catalogue/search/sheet journeys. Check standalone safe-area spacing and that the dock does not hide page actions. Browser emulation does not satisfy this item.
+- [ ] Real virtual keyboard: focus and type in the shared composer, open/clear filters and dismiss/reopen the keyboard. Confirm that focused fields and results/actions remain reachable without overlapping chrome.
+- [ ] Assistive technology and keyboard: verify meaningful Modes names, Tab order, sheet focus containment/Escape/return, single error announcements, and Therapy zero/nonzero/interrupted-result announcements with the scenario sheet open. Record VoiceOver or TalkBack observations; DOM live-region assertions alone do not establish speech delivery.
+- [ ] Zoom/reflow and contrast: inspect 200% text enlargement and 400% zoom/320 CSS-pixel reflow, light/dark themes, visible focus, disabled controls and errors. Measure applicable WCAG text and non-text contrast; do not infer compliance from token names.
+- [ ] Remaining populated clinical/provider states: an authorized clinician checks the relevant real information-consuming flows against verified sources, with source status/disclosures preserved. Synthetic fixtures do not close this item; the bedside calculator remains hidden until separately verified.

tests/calculators-mode.dom.test.tsx

@@ -2,6 +2,7 @@
 
 import { fireEvent, render, screen, within } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
+import { useState } from "react";
 import { describe, expect, it, vi } from "vitest";
 
 const navigation = vi.hoisted(() => ({
@@ -428,6 +429,53 @@ describe("calculator results surface", () => {
     );
   });
 
+  it("clears the page's shared-query provider on same-URL Show all, preserving modified clicks", async () => {
+    const user = userEvent.setup();
+    const previousUrl = window.location.href;
+    window.history.replaceState(null, "", "/calculators/search");
+
+    // Keep the provider state live: a callback spy alone would not prove that
+    // clearing the shared composer also restores the visible catalogue.
+    function CalculatorPageProviderHarness() {
+      const [query, setQuery] = useState("");
+      return (
+        <SearchCommandProvider value={{ query, modeId: "calculators", clearQuery: () => setQuery("") }}>
+          <input
+            aria-label="Shared calculator query"
+            value={query}
+            onChange={(event) => setQuery(event.target.value)}
+          />
+          <CalculatorsSearchPage />
+        </SearchCommandProvider>
+      );
+    }
+
+    try {
+      render(<CalculatorPageProviderHarness />);
+      const input = screen.getByRole("textbox", { name: "Shared calculator query" });
+      await user.type(input, "PHQ-9");
+      const results = screen.getByTestId("calculator-result-list");
+      expect(within(results).getAllByRole("listitem")).toHaveLength(1);
+      expect(within(results).getByRole("button", { name: /^Open PHQ-9/ })).toBeVisible();
+      const showAll = screen.getByRole("link", { name: "Show all calculators" });
+      expect(showAll).toHaveAttribute("href", window.location.pathname);
+
+      for (const modifier of [{ ctrlKey: true }, { metaKey: true }, { shiftKey: true }, { altKey: true }]) {
+        fireEvent.click(showAll, modifier);
+        expect(input).toHaveValue("PHQ-9");
+        expect(within(results).getAllByRole("listitem")).toHaveLength(1);
+      }
+
+      await user.click(showAll);
+      expect(input).toHaveValue("");
+      expect(within(results).getAllByRole("listitem")).toHaveLength(calculators.length);
+      expect(screen.queryByTestId("calculators-show-all")).toBeNull();
+      expect(window.location.pathname + window.location.search).toBe("/calculators/search");
+    } finally {
+      window.history.replaceState(null, "", previousUrl);
+    }
+  });
+
   it("distinguishes filtered and query-only empty recovery", async () => {
     const user = userEvent.setup();
     const { rerender } = render(