Skip to content

Show recommended TI parts with an expandable MCP JSON response - #234

Merged
seveibar merged 1 commit into
mainfrom
codex/ti-picker-recommendations-mcp-json
Sep 3, 2026
Merged

Show recommended TI parts with an expandable MCP JSON response#234
seveibar merged 1 commit into
mainfrom
codex/ti-picker-recommendations-mcp-json

Conversation

@imrishabh18

Copy link
Copy Markdown
Member

Summary

  • Keep Recommended badges on matching local subcircuits and show distinct wider-portfolio recommendations alongside the selectable parts.
  • Add a collapsed-by-default MCP response row that expands the complete, formatted tool-response JSON without changing the selected part.
  • Fix the live proxy request by removing the unsupported format=details parameter and parsing conversation.response product findings. Preserve legacy recommendation-array support, deduplicate repeated findings, and retain the full MCP payload for inspection.
  • Cache both recommendations and their JSON, show HTTP failures without blocking local part selection, and add regression coverage for parsing, badge matching, caching, and retries.

Verification

  • cd system-block-ui && bun test — 77 tests passed.
  • cd system-block-ui && bun run build — passed (existing large-chunk warning).
  • Biome formatting checks and git diff --check — passed.
  • Browser checks with the live TI proxy: recommended badges, description expansion, JSON expand/collapse by click and keyboard, preserved current-part selection, and Escape to close the picker.
  • Confirmed MCP error payloads remain inspectable when the upstream service returns no product findings.

This PR preserves both recommendation badges and MCP-response inspection; it does not modify the separate MCP-only proposal in #231.

@vercel

vercel Bot commented Sep 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
ti-system-block-ui Ready Ready Preview Sep 3, 2026 4:21pm UTC

Request Review

@imrishabh18
imrishabh18 requested a review from seveibar September 3, 2026 16:27
@seveibar
seveibar merged commit 720b574 into main Sep 3, 2026
4 of 5 checks passed
@seveibar
seveibar deleted the codex/ti-picker-recommendations-mcp-json branch September 3, 2026 17:50
@tscircuitbot

Copy link
Copy Markdown

Thank you for your contribution! 🎉

PR Rating: ⭐⭐⭐
Impact: Major

Track your contributions and see the leaderboard at: tscircuit Contribution Tracker


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.

3 participants