@tscircuit/schematic-trace-solver 0.0.180 → 0.0.182

package/tests/repros/repro-mcp73831-charger-traces.test.ts

@@ -0,0 +1,16 @@
+import { expect, test } from "bun:test"
+import { SchematicTracePipelineSolver } from "lib/solvers/SchematicTracePipelineSolver/SchematicTracePipelineSolver"
+import "tests/fixtures/matcher"
+import inputProblem from "./assets/repro-mcp73831-charger-traces.input.json"
+
+// Captured from @tscircuit/core 0.0.1829 at commit b761dc8.
+// The source circuit contains only the MCP73831 charger, its programming and
+// LED resistors, VBAT capacitor, and LED. This preserves the long CHG_STAT path
+// needed to reproduce the VBAT label crossing it.
+test("repro VBAT label crossing the MCP73831 CHG_STAT trace", () => {
+  const solver = new SchematicTracePipelineSolver(inputProblem as any)
+
+  solver.solve()
+
+  expect(solver).toMatchSolverSnapshot(import.meta.path)
+})

package/tests/repros/repro-mouse-switch-ground-unnecessary-jog.test.ts

@@ -0,0 +1,15 @@
+import { expect, test } from "bun:test"
+import { SchematicTracePipelineSolver } from "lib/solvers/SchematicTracePipelineSolver/SchematicTracePipelineSolver"
+import "tests/fixtures/matcher"
+import inputProblem from "./assets/repro-mouse-switch-ground-unnecessary-jog.input.json"
+
+// Captured from @tscircuit/core 0.0.1829 at commit b761dc8.
+// The GND route should use one straight vertical trunk between its terminal
+// stubs instead of adding the unnecessary horizontal step near the midpoint.
+test("repro unnecessary jog in mouse switch ground trunk", () => {
+  const solver = new SchematicTracePipelineSolver(inputProblem as any)
+
+  solver.solve()
+
+  expect(solver).toMatchSolverSnapshot(import.meta.path)
+})

package/tests/repros/repro-rf1-unnecessary-dogleg.test.ts

@@ -0,0 +1,15 @@
+import { expect, test } from "bun:test"
+import { SchematicTracePipelineSolver } from "lib/solvers/SchematicTracePipelineSolver/SchematicTracePipelineSolver"
+import "tests/fixtures/matcher"
+import inputProblem from "./assets/repro-rf1-unnecessary-dogleg.input.json"
+
+// Captured from @tscircuit/core 0.0.1829 at commit b761dc8.
+// The nearly level RF1 endpoints should produce an effectively straight trace,
+// rather than a clearance-sized dogleg between the two inductors.
+test("repro RF1 unnecessary dogleg between aligned inductors", () => {
+  const solver = new SchematicTracePipelineSolver(inputProblem as any)
+
+  solver.solve()
+
+  expect(solver).toMatchSolverSnapshot(import.meta.path)
+})

package/tests/solvers/InlineNetLabelSolver/push-anchored-net-labels-away.test.ts

@@ -126,6 +126,68 @@ test("leaves a label without a movable connector in place", () => {
   expect(output.netLabelPlacements[0]).toEqual(label)
 })
 
+test("leaves a label in place when its new connector would cross another net", () => {
+  const inputProblem: InputProblem = {
+    chips: [
+      {
+        chipId: "U1",
+        center: { x: 0, y: 0 },
+        width: 2,
+        height: 1,
+        pins: [{ pinId: "U1.1", x: -0.5, y: 0.2, _facingDirection: "x-" }],
+      },
+    ],
+    directConnections: [],
+    netConnections: [],
+    availableNetLabelOrientations: {},
+  }
+  const label: NetLabelPlacement = {
+    globalConnNetId: "regular-net",
+    netId: "D_PLUS",
+    mspConnectionPairIds: [],
+    pinIds: ["U1.1"],
+    orientation: "x-",
+    anchorPoint: { x: -0.5, y: 0.2 },
+    center: { x: -0.6, y: 0.2 },
+    width: 0.2,
+    height: 0.2,
+  }
+  const crossingTrace: SolvedTracePath = {
+    mspPairId: "crossing-trace",
+    dcConnNetId: "crossing-net",
+    globalConnNetId: "crossing-net",
+    pins: [] as any,
+    tracePath: [
+      { x: -1.2, y: -0.2 },
+      { x: -1.2, y: 0.4 },
+    ],
+    mspConnectionPairIds: ["crossing-trace"],
+    pinIds: [],
+  }
+  const inlineLabel: InlineNetLabelPlacement = {
+    globalConnNetId: "inline-net",
+    netId: "SWCLK",
+    pinIds: ["U1.2"],
+    axis: "x",
+    anchorPoint: { x: -1.1, y: 0 },
+    center: { x: -1.1, y: 0.11 },
+    width: 1.2,
+    height: 0.12,
+    side: "y+",
+  }
+
+  const output = pushAnchoredNetLabelsAwayFromInlineLabels({
+    inputProblem,
+    traces: [crossingTrace],
+    netLabelPlacements: [label],
+    inlineNetLabelPlacements: [inlineLabel],
+  })
+
+  expect(output.movedLabelCount).toBe(0)
+  expect(output.netLabelPlacements[0]).toEqual(label)
+  expect(output.traces).toEqual([crossingTrace])
+})
+
 test("moves a contiguous label row together and shoves an anchored obstacle", () => {
   const inputProblem: InputProblem = {
     chips: [