Skip to content

Merge close same-net trace lines#551

Open
Monster5860 wants to merge 1 commit into
tscircuit:mainfrom
Monster5860:codex/issue-34-merge-close-same-net-lines
Open

Merge close same-net trace lines#551
Monster5860 wants to merge 1 commit into
tscircuit:mainfrom
Monster5860:codex/issue-34-merge-close-same-net-lines

Conversation

@Monster5860

@Monster5860 Monster5860 commented Jun 17, 2026

Copy link
Copy Markdown

Closes #34.

/claim #34

Summary

  • add a cleanup pass that snaps nearby overlapping same-net horizontal or vertical interior trace segments onto existing anchor coordinates
  • preserve endpoint trace segments and trace identities while updating branch path geometry
  • simplify paths after snapping and cover horizontal, vertical, and different-net cases
  • update affected example SVG snapshots for the expected visual line alignment changes

Tests

  • bun run format:check
  • bunx tsc --noEmit
  • bun test -> 73 pass, 4 skip, 0 fail

@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
schematic-trace-solver Ready Ready Preview, Comment Jun 17, 2026 4:10am

Request Review

@Monster5860

Copy link
Copy Markdown
Author

Local validation is now complete on the latest commit:\n\n- bun run format:check\n- bunx tsc --noEmit\n- bun test -> 73 pass, 4 skip, 0 fail\n\nThe fork PR workflows still show action_required, so they appear to be waiting for maintainer approval before running on GitHub Actions.

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.

Merge same-net trace lines that are close together (make at the same Y or same X)

2 participants