Skip to content

Add Tools search redesign mockup gallery (/mockups/tools-search) - #1653

Closed
BigSimmo wants to merge 20 commits into
mainfrom
cursor/tools-search-mockups-72e1
Closed

Add Tools search redesign mockup gallery (/mockups/tools-search)#1653
BigSimmo wants to merge 20 commits into
mainfrom
cursor/tools-search-mockups-72e1

Conversation

@BigSimmo

@BigSimmo BigSimmo commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds a design-scratch gallery at /mockups/tools-search for the Preserve active indexing-v3 processing locks during strict-gate seeding #162 Tools search results redesign.
  • Shows directions A Compact Results Instrument (product pick), B Dense Launcher List, and C Split Command Workspace with desktop/phone comps from public/mockups/mode-page-redesign-2026-07/tools-search/.
  • Direction A also offers the combined desktop+phone perfected comp.

Why

The IDE canvas artifact from the design pass does not open reliably in this Cloud Agent session. This route makes the same comps browsable in the running app.

Test plan

  • npm run ensure → open /mockups/tools-search
  • Switch A/B/C and Desktop/Phone; confirm images load
  • On A, open “A · Desktop + phone” combined view

Notes

  • Mockup route only — no production /tools behaviour change.
  • RAG impact: no retrieval behaviour change — mockup gallery only.
Open in Web Open in Cursor 

Expose directions A/B/C (Compact Results, Dense List, Split Command)
from the #162 comps at /mockups/tools-search so the designs can be
browsed in the running app when the IDE canvas cannot open.

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: afb5ddb4-8fc9-4d7c-a85c-e91d858f4a38

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

@BigSimmo

BigSimmo commented Aug 7, 2026

Copy link
Copy Markdown
Owner Author

Autopilot + Bugbot — PR #1653

Final SHA: 24e5edaa4d71308a296ea55eac16c645904e8d36 (was a7fbc26a917b347d90c6bab1e6c1b2ede6422263)

Snapshot / actions

  • Confirmed open draft; product delta remains 3 files (/mockups/tools-search gallery + site-map).
  • Merge-tree vs origin/main: clean before and after sync.
  • Unresolved review threads: 0 (no Bugbot/CodeRabbit inline findings; CodeRabbit skipped draft).
  • Prior tip CI: required checks green (PR required pass), including Production UI shards.
  • Behind-but-clean at a7fbc26: one late merge(main) → pushed 24e5eda.
  • High-confidence delta review: no P0/P1. Mockup assets present; production mockups layout still notFound when disabled; buttons wired. Bugbot subagent unavailable (usage limit) — manual review substituted.
  • Ledger: local-only clean-local-no-change at 24e5eda (not pushed as ledger-only tip).

Checks

  • Prettier check on changed mockup files: pass
  • sitemap:check: started clean under partial verify:cheap (static gates through icon-scale passed)
  • Full verify:cheap / heavy lint: blocked by cross-worktree lock contention (DATABASE_HEAVY_RUN_ADMISSION_BUSY); not green locally this pass
  • verify:pr-local --dry-run: executable/mockup scope selected

Blockers remaining

  1. Draft — merge state stays blocked until marked ready
  2. BEHIND againmain advanced ~14 commits after the one allowed sync; merge-tree still clean; needs a later sync (not looped this pass)
  3. CI on 24e5eda will re-run after the sync push

No merge performed.

@cursor

cursor Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Closing as low-yield design-scratch: Tools search gallery only; superseded by #1654’s perfected Tools frames. Production adoption remains tracked as /issues #162. 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