Skip to content

feat(mockups): recommended settings feature mockups - #1657

Closed
BigSimmo wants to merge 21 commits into
mainfrom
cursor/settings-features-mockups-97ac
Closed

feat(mockups): recommended settings feature mockups#1657
BigSimmo wants to merge 21 commits into
mainfrom
cursor/settings-features-mockups-97ac

Conversation

@BigSimmo

@BigSimmo BigSimmo commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Design-scratch mockups for the 32 recommended new settings options (not already in product Settings), organised by the recommended build order.
  • Interactive phone + desktop settings surfaces per group, plus a full gallery overview listing every option and what it adds.
  • Routes (404 in production; mockups-only):
    • /mockups/settings-recommended-features
    • /mockups/settings-recommended-features/gallery
    • /mockups/settings-recommended-features/{group} for each of: source-evidence, answer-reading, clinical-workflow, privacy-trust, productivity, accessibility-device

Screenshots

Source & evidence desktop
Settings gallery desktop
Source & evidence phone
Clinical workflow desktop
Privacy & trust desktop

Verification

  • npm run ensure — app at http://localhost:4461
  • All mockup routes return HTTP 200
  • Playwright screenshots captured for desktop + phone across hub, gallery, and all 6 groups
  • Verification not run beyond mockup smoke: design-scratch only; no product behaviour change

Risk and rollout

  • Risk: Low — mockup routes under src/app/mockups/** only
  • Rollback: revert commit / close PR
  • Provider or production effects: None

Notes

  • Does not wire clinical defaults or change product Settings behaviour.
  • Existing inert Settings rows (jurisdiction, population, answer style, notifications) are intentionally excluded — these mockups cover new options only.

To show artifacts inline, enable in settings.

Open in Web Open in Cursor 

Design-scratch phone and desktop mockups for the 32 recommended new
settings options across six groups, plus a full gallery overview.

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e8618f4b-4c01-4883-a4cd-bffbf03729a0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@supabase

supabase Bot commented Aug 6, 2026

Copy link
Copy Markdown

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 ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

cursoragent and others added 2 commits August 6, 2026 17:23
Hardcode group ids/metadata in the server page so dynamic mockup
routes do not import SETTINGS_FEATURE_GROUPS from a use-client module.

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
Unblocks static-pr and site-map unit coverage on this branch.

Co-authored-by: BigSimmo <BigSimmo@users.noreply.github.com>
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

CI triage

CI failed on this PR. Automated classification of the 2 failed job(s):

  • Production UI (2)needs investigation: inspect the failing step and uploaded diagnostics; rerun only after classifying the cause.
  • PR requiredneeds investigation: inspect the failing step and uploaded diagnostics; rerun only after classifying the cause.

Compared with main CI run #8674 (cancelled).

Classification is evidence routing, not permission to ignore a failure. Exact quarantined Playwright identities remain governed by the flake ledger.

@BigSimmo

BigSimmo commented Aug 7, 2026

Copy link
Copy Markdown
Owner Author

Autopilot + Bugbot summary (PR #1657)

Final SHA: 8a44d419b3d0fb190e15c4522ddfd32b0bdc5c95 (was d64288ce0f2f111b1838ded31a23d0020b38da8b)

Snapshot / actions

Area Result
Conflicts None — merge-tree clean vs origin/main
Behind One late sync pushed (merge(main): sync PR #1657). Main advanced 1 commit again after push (behind≈1, still MERGEABLE / merge-tree clean). No second sync this pass.
Review threads 0 unresolved (including Bugbot)
Required CI @ prior tip Green (PR required, Production UI 1/2/3, Static PR, Build, Unit coverage, …)
Bugbot Skipped — subagent usage limit
Manual delta review Mockup-only (src/app/mockups/settings-recommended-features/** + companion component + site-map). No P0/P1. Buttons wired; group metadata duplicated server-side intentionally to avoid client import.
Code fixes None
Local gates verify:cheap started but stuck/contended on check:design-sync-contract (other worktree heavy runs). Hosted CI remains the proof for the prior tip; new tip will re-run CI.
Ledger Local-only no-change rows at review scope (not pushed — no fix commits).

Blockers

  1. Draft — remains draft (BLOCKED/draft policy).
  2. CI re-run on 8a44d419 after sync (expected).
  3. Possibly 1 commit behind main after post-push main movement — merge-tree still clean; sync again only if merge is blocked.
  4. Bugbot not run (usage limit) — manual review only.

No merge performed.

@cursor

cursor Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Closing as low-yield design-scratch: recommended settings mockup routes only; product Settings unchanged. Branch left in place for recovery — not deleted.

@BigSimmo BigSimmo closed this Aug 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants