Skip to content

Enable TP1-style single-pin trace connections and eliminate order-dependent selectOne cache collisions#2009

Closed
AnasSarkiz wants to merge 3 commits intotscircuit:mainfrom
AnasSarkiz:main
Closed

Enable TP1-style single-pin trace connections and eliminate order-dependent selectOne cache collisions#2009
AnasSarkiz wants to merge 3 commits intotscircuit:mainfrom
AnasSarkiz:main

Conversation

@AnasSarkiz
Copy link
Member

This change improves trace selector reliability and usability by allowing single-pin components to be connected by component name (for example, TP1), while preserving strict behavior for multi-pin parts.

Impact

  • Single-pin components now resolve implicitly as pin1 in trace selectors, reducing selector verbosity and avoiding unnecessary connection errors.
  • Multi-pin components still require explicit pin selection, preventing ambiguous wiring.
  • selectOne caching is now type-scoped, removing order-dependent lookup behavior when typed and untyped queries are mixed.

Coverage

  • Added positive test for TP1 -> U1.pin2 implicit single-pin behavior.
  • Added negative test proving no implicit pin1 for multi-pin components (R1).
  • Added PCB/schematic snapshots to lock rendered output.

@AnasSarkiz AnasSarkiz requested a review from seveibar March 4, 2026 22:41
@vercel
Copy link

vercel bot commented Mar 4, 2026

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

Project Deployment Actions Updated (UTC)
tscircuit-core-benchmarks Ready Ready Preview, Comment Mar 4, 2026 11:37pm

Request Review

Copy link
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is crazy, anyway to make this more conventional?

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