You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Extract the Tools soft-fill well Show all chip into a shared ShowAllChip so Tools and Calculators cannot drift.
Place that chip on every Calculators hero and search surface: the shared home at /?mode=calculators, the catalogue at /calculators/search, CalculatorSearchHome, and the matching calculator search mockups.
Keep an empty /calculators/search visit on the browsable catalogue instead of bouncing back to the shared home, so Show all has a real destination.
Phone proof at http://localhost:4461/?mode=calculators (project id clinical-kb:c52ddf65534b): Show all is under the hero; click lands on /calculators/search with the 8-calculator catalogue and does not bounce home. The same chip stays on the catalogue and on ?q=depression&run=1.
npm run verify:pr-local — not run; the focused suite covers the changed routing and chip contracts.
npm run verify:ui — not run; phone journey was exercised locally instead of the full Chromium gate.
Risk: An empty /calculators/search now lists the calculator catalogue instead of redirecting home. Differentials keep the no-browse redirect. The chip is a navigation control only; scoring, citations, and answer generation are unchanged.
Rollback: Revert this commit. The previous empty-search redirect and Tools-only chip return.
Provider or production effects: None
RAG impact: no retrieval behaviour change — this only adds a catalogue navigation chip and lets /calculators/search browse the existing fixture list. No RAG ranking, retrieval, or answer-generation surfaces are touched.
Clinical Governance Preflight
Source-backed claims still require linked source verification before clinical use
No patient-identifiable document workflow was introduced or expanded without explicit governance approval
Service-role keys and private document access remain server-only
Demo/synthetic content remains clearly separated from real clinical sources
Source metadata, review status, and outdated/unknown-source behavior remain conservative
Deployment classification/TGA SaMD impact was checked when clinical decision-support behavior changed
The catalogue still presents the same source-cited scoring tools. Scores remain decision-support only and do not replace clinical judgement. Opening the full list from home does not change how a calculator scores, cites, or degrades.
Notes
The visual treatment matches the live Tools home chip: 36px tinted capsule, hairline icon well, 48px phone tap target (min-h-tap).
Rebuild the Tools launcher chip for a Calculators home with four interiors: the shipping transplant, a matched wordmark, a recommended icon-well capsule, and a symmetric count pair.
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Limit details: You’ve used the included review currently available. Your 101 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.
Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.
This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.
Keep the Tools-like phone home unchanged and vary only the Show all chip: outline, soft fill, and icon well.
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
cursorBot
changed the title
Add Calculators Show all chip mockup directionsAdd three Calculators Show all button phone mockupsAug 24, 2026
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Approved. Cursor Bugbot was not present after the first check poll, so that signal was skipped; remaining signals and the mockup-only scope do not require human review. No reviewers assigned.
Sent by Cursor Approval Agent: Pull Request Router and Approver
Keep the 48px tap target and implement a 36px soft capsule with a 28px icon well on Tools home. Add two tighter phone mockups beside the recommended style.
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
The reason will be displayed to describe this comment to others. Learn more.
Approved. Cursor Bugbot was not present after the first check poll, so that signal was skipped; remaining signals and the visual Show all chip plus mockup scope do not require human review. No reviewers assigned.
Sent by Cursor Approval Agent: Pull Request Router and Approver
Keep the 36px capsule and 48px tap, but tint the fill and add a hairline well so the hybrid of options 2 and 3 actually reads on the Tools home.
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
The reason will be displayed to describe this comment to others. Learn more.
Approved. Cursor Bugbot was not present after the first check poll, so that signal was skipped; remaining signals and the visual Show all chip plus mockup scope do not require human review. No reviewers assigned.
Sent by Cursor Approval Agent: Pull Request Router and Approver
Extract the soft-fill well chip so Tools and Calculators share one control, place it on the Calculators home and search surfaces, and keep an empty /calculators/search visit on the catalogue instead of bouncing home.
Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
The reason will be displayed to describe this comment to others. Learn more.
Approved. Cursor Bugbot was not present after the first check poll, so that signal was skipped; remaining signals and current review state do not require human review. No reviewers assigned.
Sent by Cursor Approval Agent: Pull Request Router and Approver
check:design-system-contract's arbitraryTracking ratchet flagged the new
Show all chip's tracking-[-0.01em]. -0.01em folds into --tracking-display
(-0.02em) per the documented letterspacing scale in globals.css, so switch
to the sanctioned tracking-[var(--tracking-display)] form.
The reason will be displayed to describe this comment to others. Learn more.
Approved. Cursor Bugbot was not present after the first check poll, so that signal was skipped; remaining signals and current review state do not require human review. No reviewers assigned.
Sent by Cursor Approval Agent: Pull Request Router and Approver
* Add branch review record for PR #2342 sweep
Records the Run-PR-style sweep check on PR #2342 (Improve Therapy
best-match visibility): review comments already fixed, branch synced
from main, CI green on completed checks.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1
* docs(ledger): record Run PR sweep of Dependabot PRs #2296, #2297, #2325, #2326
Immutable review records for the four-PR dependency sweep: CI fix on
#2325, clean main-syncs on #2296/#2297, and diagnosis-only on #2326
(Node 26 Docker bump incompatible with the engine-strict Node 24 pin).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1
* Add branch review record for PR #2339 sweep
Records the Run-PR-style sweep check on PR #2339 (therapy comparison
mobile design mockups): already fully green, only needed a main sync.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1
* Add branch review records for PR #2341, #2347 sweep
Records the Run-PR-style sweep checks on #2341 (dictionary filter
band, fixed via main sync, unrelated flake confirmed) and #2347
(browser test gate handoff, fixed stale generated file + doc-link
allowlist entries; owner closed the PR mid-sweep for unrelated reasons).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1
* Add branch review records for PR #2338, #2337, #2333 sweep
Records the Run-PR-style sweep checks: #2337 fixed a design-token
ratchet failure, #2333 fixed a tap-target size regression and
resolved a concurrent-push merge, #2338 was only a main sync. #2333
and #2338 still have an open PR-policy failure (missing Clinical
Governance Preflight section) left for the PR author to fill in.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C4RHy24AtgPobEQQwrj7u1
---------
Co-authored-by: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ShowAllChipso Tools and Calculators cannot drift./?mode=calculators, the catalogue at/calculators/search,CalculatorSearchHome, and the matching calculator search mockups./calculators/searchvisit on the browsable catalogue instead of bouncing back to the shared home, so Show all has a real destination.Verification
npx vitest run tests/calculators-mode.dom.test.tsx tests/consolidated-mode-home-redirect.test.ts tests/favourites-auth-gate.dom.test.tsx tests/calculators-show-all-mockups.dom.test.tsx tests/calculator-mockup-boundary.test.ts tests/shared-home-empty-state.dom.test.tsxTest Files 6 passed (6)/Tests 70 passed (70)http://localhost:4461/?mode=calculators(project idclinical-kb:c52ddf65534b): Show all is under the hero; click lands on/calculators/searchwith the 8-calculator catalogue and does not bounce home. The same chip stays on the catalogue and on?q=depression&run=1.npm run verify:pr-local— not run; the focused suite covers the changed routing and chip contracts.npm run verify:ui— not run; phone journey was exercised locally instead of the full Chromium gate.npm run verify:release— not requested.Calculators phone home with Show all chip
Calculators catalogue after Show all
calculators_phone_show_all_to_catalogue.mp4
Risk and rollout
/calculators/searchnow lists the calculator catalogue instead of redirecting home. Differentials keep the no-browse redirect. The chip is a navigation control only; scoring, citations, and answer generation are unchanged./calculators/searchbrowse the existing fixture list. No RAG ranking, retrieval, or answer-generation surfaces are touched.Clinical Governance Preflight
Clinical KB Database(sjrfecxgysukkwxsowpy)The catalogue still presents the same source-cited scoring tools. Scores remain decision-support only and do not replace clinical judgement. Opening the full list from home does not change how a calculator scores, cites, or degrades.
Notes
The visual treatment matches the live Tools home chip: 36px tinted capsule, hairline icon well, 48px phone tap target (
min-h-tap).To show artifacts inline, enable in settings.