Skip to content

Coalesce same-net trace segments#357

Open
vinaykrsinghal-stage wants to merge 1 commit into
tscircuit:mainfrom
vinaykrsinghal-stage:codex/same-net-trace-coalesce
Open

Coalesce same-net trace segments#357
vinaykrsinghal-stage wants to merge 1 commit into
tscircuit:mainfrom
vinaykrsinghal-stage:codex/same-net-trace-coalesce

Conversation

@vinaykrsinghal-stage
Copy link
Copy Markdown

@vinaykrsinghal-stage vinaykrsinghal-stage commented May 16, 2026

Summary

  • Adds a post-cleanup same-net trace coalescing pass that trims redundant endpoint overlap already covered by another same-net trace and snaps close overlapping internal same-net segments onto a shared run
  • Rejects coalescing candidates that would introduce a different-net trace crossing
  • Updates affected example snapshots, including example29, where the pass removes repeated same-net stubs

/claim #29
/claim #34

Validation

  • bun test
  • bun run format:check
  • bun run build
  • git diff --check

AI-assisted with OpenAI Codex; I reviewed the diff, visual snapshot changes, and local verification before submitting.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 16, 2026

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

Project Deployment Actions Updated (UTC)
schematic-trace-solver Ready Ready Preview, Comment May 16, 2026 0:32am

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant