Skip to content

Fix Circuit JSON inflation fidelity for boards and subcircuits - #4165

Open
ShiboSoftwareDev wants to merge 4 commits into
mainfrom
fix/inflate-testpoint-pinheader
Open

ShiboSoftwareDev wants to merge 4 commits into
mainfrom
fix/inflate-testpoint-pinheader

Conversation

@ShiboSoftwareDev

@ShiboSoftwareDev ShiboSoftwareDev commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • inflate source nets, test points, pin headers, schematic sheets, and schematic proxy components
  • preserve imported board geometry/appearance, copper pours, footprint rotations, silkscreen properties, named traces, and PCB route endpoint references
  • keep imported routed boards stable by avoiding a second DRC/autorouting pass while validating newly generated routes normally
  • support schematic-only and do-not-place chips without producing false footprint errors
  • keep routing/manufacturing limits controlled by explicit destination board props so importing geometry does not silently tighten an autorouter phase

Verification

  • all 26 subcircuit Circuit JSON tests pass
  • targeted panel/cache/imported-reroute snapshot regressions pass
  • USB-C named-net and AM62L dense-routing regressions pass locally
  • Arduino Uno reroute, layer persistence, and PCB trace inflation regressions pass
  • TypeScript, Biome, package build, smoke test, and CI shards 1-9 pass
  • verified against four TI EVM subtraction snapshots in the downstream package

CI baseline note

CI shard 10 reports the same USB-C and AM62L router assertions on the current main branch (main run); the PR introduces no additional shard-10 failure.

Downstream

@vercel

vercel Bot commented Sep 26, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
tscircuit-core-benchmarks Ready Ready Preview Sep 26, 2026 11:53pm UTC

Request Review

This branch has not been deployed

No deployments
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.

1 participant