@tscircuit/schematic-trace-solver 0.0.152 → 0.0.154

package/dist/index.d.ts

@@ -108,11 +108,11 @@ interface InputDirectConnection {
     netLabelText?: string;
     netLabelWidth?: number;
     /**
-     * Width of the conventional anchored label to use only when an inline label
-     * cannot be placed. Unlike `netLabelWidth`, this does not affect whether or
-     * how the point-to-point connection is routed.
+     * Width of the conventional label anchored to a trace endpoint. Unlike
+     * `netLabelWidth`, this does not affect whether or how the point-to-point
+     * connection is routed.
      */
-    fallbackNetLabelWidth?: number;
+    anchoredNetLabelWidth?: number;
     /**
      * When true, this point-to-point connection may be labeled with an "inline
      * net label": the net name is drawn parallel to (and offset from) the routed
@@ -149,7 +149,7 @@ interface InputNetConnection {
      */
     netLabelText?: string;
     netLabelWidth?: number;
-    fallbackNetLabelWidth?: number;
+    anchoredNetLabelWidth?: number;
     netLabelHeight?: number;
     /**
      * When true, a named one- or two-pin net may use inline labels. A single-pin

package/dist/index.js

@@ -679,21 +679,20 @@ import "graphics-debug";
 import { calculateElbow as calculateElbow2 } from "calculate-elbow";
 
 // lib/utils/getNetLabelWidthForConnection.ts
-var getConfiguredWidth = (connections, includeFallbackNetLabelWidth) => {
-  const explicitWidth = connections.find(
+var getConfiguredWidth = (connections, isPortOnlyLabel2) => {
+  const width = connections.find(
     (connection) => connection?.netLabelWidth !== void 0
   )?.netLabelWidth;
-  if (explicitWidth !== void 0) return explicitWidth;
-  if (!includeFallbackNetLabelWidth) return void 0;
+  if (width !== void 0 || !isPortOnlyLabel2) return width;
   return connections.find(
-    (connection) => connection?.fallbackNetLabelWidth !== void 0
-  )?.fallbackNetLabelWidth;
+    (connection) => connection?.anchoredNetLabelWidth !== void 0
+  )?.anchoredNetLabelWidth;
 };
 var getNetLabelWidthForConnection = ({
   inputProblem,
   netId,
   pinIds,
-  includeFallbackNetLabelWidth = true
+  isPortOnlyLabel: isPortOnlyLabel2
 }) => {
   if (netId) {
     const widthByNetId = getConfiguredWidth(
@@ -705,7 +704,7 @@ var getNetLabelWidthForConnection = ({
           (connection) => connection.netId === netId
         )
       ],
-      includeFallbackNetLabelWidth
+      isPortOnlyLabel2
     );
     if (widthByNetId !== void 0) return widthByNetId;
   }
@@ -718,7 +717,7 @@ var getNetLabelWidthForConnection = ({
         (connection) => connection.pinIds.some((pinId) => pinIds.includes(pinId))
       )
     ],
-    includeFallbackNetLabelWidth
+    isPortOnlyLabel2
   );
 };
 
@@ -1329,7 +1328,7 @@ var SchematicTraceSingleLineSolver2 = class extends BaseSolver {
       inputProblem: this.inputProblem,
       netId,
       pinIds: this.pins.map((pin) => pin.pinId),
-      includeFallbackNetLabelWidth: false
+      isPortOnlyLabel: false
     });
     const netLabelHeight = this.getNetLabelHeightForConnectionPair(netId);
     const padding = {
@@ -3270,7 +3269,8 @@ var NetLabelPlacementSolver = class extends BaseSolver {
     return getNetLabelWidthForConnection({
       inputProblem: this.inputProblem,
       netId: group.netId,
-      pinIds
+      pinIds,
+      isPortOnlyLabel: group.portOnlyPinId !== void 0
     });
   }
   getNetLabelHeightForGroup(group) {
@@ -8620,7 +8620,8 @@ var AvailableNetOrientationSolver = class extends BaseSolver {
     const blockedLabelWidth = getNetLabelWidthForConnection({
       inputProblem: this.inputProblem,
       netId: blockedLabel.netId,
-      pinIds: blockedLabel.pinIds
+      pinIds: blockedLabel.pinIds,
+      isPortOnlyLabel: this.isPortOnlyLabel(blockedLabel)
     });
     let columnLabelIndex;
     if (blockedLabelWidth !== void 0) {
@@ -8630,7 +8631,8 @@ var AvailableNetOrientationSolver = class extends BaseSolver {
         return getNetLabelWidthForConnection({
           inputProblem: this.inputProblem,
           netId: label.netId,
-          pinIds: label.pinIds
+          pinIds: label.pinIds,
+          isPortOnlyLabel: this.isPortOnlyLabel(label)
         }) === blockedLabelWidth;
       });
     }
@@ -8855,7 +8857,8 @@ var AvailableNetOrientationSolver = class extends BaseSolver {
       netLabelWidth: getNetLabelWidthForConnection({
         inputProblem: this.inputProblem,
         netId: label.netId,
-        pinIds: label.pinIds
+        pinIds: label.pinIds,
+        isPortOnlyLabel: this.isPortOnlyLabel(label)
       }),
       netLabelHeight: this.getNetLabelHeight(label)
     });
@@ -9293,7 +9296,8 @@ var AvailableNetOrientationSolver = class extends BaseSolver {
       netLabelWidth: getNetLabelWidthForConnection({
         inputProblem: this.inputProblem,
         netId: label.netId,
-        pinIds: label.pinIds
+        pinIds: label.pinIds,
+        isPortOnlyLabel: this.isPortOnlyLabel(label)
       }),
       netLabelHeight: this.getNetLabelHeight(label)
     });
@@ -9315,7 +9319,8 @@ var AvailableNetOrientationSolver = class extends BaseSolver {
       netLabelWidth: getNetLabelWidthForConnection({
         inputProblem: this.inputProblem,
         netId: label.netId,
-        pinIds: label.pinIds
+        pinIds: label.pinIds,
+        isPortOnlyLabel: this.isPortOnlyLabel(label)
       }),
       netLabelHeight: this.getNetLabelHeight(label)
     });
@@ -9374,7 +9379,8 @@ var AvailableNetOrientationSolver = class extends BaseSolver {
       netLabelWidth: getNetLabelWidthForConnection({
         inputProblem: this.inputProblem,
         netId: label.netId,
-        pinIds: label.pinIds
+        pinIds: label.pinIds,
+        isPortOnlyLabel: this.isPortOnlyLabel(label)
       }),
       netLabelHeight: this.getNetLabelHeight(label)
     });
@@ -9429,7 +9435,8 @@ var AvailableNetOrientationSolver = class extends BaseSolver {
       netLabelWidth: getNetLabelWidthForConnection({
         inputProblem: this.inputProblem,
         netId: label.netId,
-        pinIds: label.pinIds
+        pinIds: label.pinIds,
+        isPortOnlyLabel: this.isPortOnlyLabel(label)
       }),
       netLabelHeight: this.getNetLabelHeight(label)
     });
@@ -10643,7 +10650,8 @@ var getNetLabelWidth = (inputProblem, label) => {
   const configuredWidth = getNetLabelWidthForConnection({
     inputProblem,
     netId: label.netId,
-    pinIds: label.pinIds
+    pinIds: label.pinIds,
+    isPortOnlyLabel: label.mspConnectionPairIds.length === 0
   });
   if (configuredWidth !== void 0) return configuredWidth;
   if (label.orientation === "y+" || label.orientation === "y-") {
@@ -12131,6 +12139,7 @@ var pathEntersAnyNetLabel = ({
 // lib/solvers/SameNetJunctionAlignmentSolver/alignSameNetJunctions.ts
 var MAX_ALIGNED_LOAD_PIN_OFFSET = 0.2;
 var MAX_SAME_NET_LABEL_BOUNDARY_RAIL_OFFSET = 0.2;
+var MAX_SHARED_PIN_RAIL_OFFSET = 0.05;
 var getSharedPin = ({
   donorTrace,
   branchTrace
@@ -12170,6 +12179,28 @@ var railIsOnFacingSide = ({
   if (pin._facingDirection === "y+") return railY > pin.y;
   return false;
 };
+var railIsOnSharedPinFacingSide = ({
+  railY,
+  pin
+}) => {
+  if (pin._facingDirection === "y+") return railY > pin.y;
+  if (pin._facingDirection === "y-") return railY < pin.y;
+  return false;
+};
+var getHorizontalPinApproachPoint = ({
+  branchTrace,
+  sharedPin,
+  otherPin
+}) => {
+  if (otherPin._facingDirection !== "x-" && otherPin._facingDirection !== "x+") {
+    return null;
+  }
+  const sharedToOtherPath = branchTrace.pins[0].pinId === sharedPin.pinId ? branchTrace.tracePath : [...branchTrace.tracePath].reverse();
+  const approachPoint = sharedToOtherPath.at(-2);
+  if (!approachPoint || !nearlyEqual(approachPoint.y, otherPin.y)) return null;
+  const approachesFromFacingSide = otherPin._facingDirection === "x-" ? approachPoint.x < otherPin.x : approachPoint.x > otherPin.x;
+  return approachesFromFacingSide ? approachPoint : null;
+};
 var getAttachedLabelIndexes = (trace, netLabelPlacements) => netLabelPlacements.flatMap((label, index) => {
   const labelOwnsTrace = label.mspConnectionPairIds.includes(trace.mspPairId) || label.mspConnectionPairIds.length === 0 && label.globalConnNetId === trace.globalConnNetId;
   return labelOwnsTrace && tracePathContainsPoint(trace.tracePath, label.anchorPoint) ? [index] : [];
@@ -12218,19 +12249,45 @@ var getAlignedBranchPath = ({
   if (branchRail && nearlyEqual(branchRail.start.y, donorRail.start.y)) {
     return null;
   }
-  if (!railIsOnFacingSide({ railY: donorRail.start.y, pin: otherPin })) {
+  const railFacesOtherPin = railIsOnFacingSide({
+    railY: donorRail.start.y,
+    pin: otherPin
+  });
+  const railFacesSharedPin = railIsOnSharedPinFacingSide({
+    railY: donorRail.start.y,
+    pin: sharedPin
+  }) && Math.abs(donorRail.start.y - sharedPin.y) <= MAX_SHARED_PIN_RAIL_OFFSET;
+  if (!railFacesOtherPin && !railFacesSharedPin) {
     return null;
   }
   const junction = getJunctionPoint({ segment: donorRail, sharedPin });
   const extendsDonorRail = donorOtherPin.x < junction.x && otherPin.x > junction.x || donorOtherPin.x > junction.x && otherPin.x < junction.x;
   if (!extendsDonorRail) return null;
-  const sharedToOther = simplifyPath([
-    { x: sharedPin.x, y: sharedPin.y },
-    { x: junction.x, y: sharedPin.y },
-    { x: junction.x, y: junction.y },
-    { x: otherPin.x, y: junction.y },
-    { x: otherPin.x, y: otherPin.y }
-  ]);
+  let sharedToOther;
+  if (!railFacesOtherPin && railFacesSharedPin) {
+    const approachPoint = getHorizontalPinApproachPoint({
+      branchTrace,
+      sharedPin,
+      otherPin
+    });
+    if (!approachPoint) return null;
+    sharedToOther = simplifyPath([
+      { x: sharedPin.x, y: sharedPin.y },
+      { x: junction.x, y: sharedPin.y },
+      { x: junction.x, y: junction.y },
+      { x: approachPoint.x, y: junction.y },
+      { x: approachPoint.x, y: otherPin.y },
+      { x: otherPin.x, y: otherPin.y }
+    ]);
+  } else {
+    sharedToOther = simplifyPath([
+      { x: sharedPin.x, y: sharedPin.y },
+      { x: junction.x, y: sharedPin.y },
+      { x: junction.x, y: junction.y },
+      { x: otherPin.x, y: junction.y },
+      { x: otherPin.x, y: otherPin.y }
+    ]);
+  }
   if (branchTrace.pins[0].pinId === sharedPin.pinId) return sharedToOther;
   return [...sharedToOther].reverse();
 };

package/lib/solvers/AvailableNetOrientationSolver/AvailableNetOrientationSolver.ts

@@ -185,6 +185,7 @@ export class AvailableNetOrientationSolver extends BaseSolver {
       inputProblem: this.inputProblem,
       netId: blockedLabel.netId,
       pinIds: blockedLabel.pinIds,
+      isPortOnlyLabel: this.isPortOnlyLabel(blockedLabel),
     })
     let columnLabelIndex: number | undefined
     if (blockedLabelWidth !== undefined) {
@@ -196,6 +197,7 @@ export class AvailableNetOrientationSolver extends BaseSolver {
             inputProblem: this.inputProblem,
             netId: label.netId,
             pinIds: label.pinIds,
+            isPortOnlyLabel: this.isPortOnlyLabel(label),
           }) === blockedLabelWidth
         )
       })
@@ -496,6 +498,7 @@ export class AvailableNetOrientationSolver extends BaseSolver {
         inputProblem: this.inputProblem,
         netId: label.netId,
         pinIds: label.pinIds,
+        isPortOnlyLabel: this.isPortOnlyLabel(label),
       }),
       netLabelHeight: this.getNetLabelHeight(label),
     })
@@ -1113,6 +1116,7 @@ export class AvailableNetOrientationSolver extends BaseSolver {
         inputProblem: this.inputProblem,
         netId: label.netId,
         pinIds: label.pinIds,
+        isPortOnlyLabel: this.isPortOnlyLabel(label),
       }),
       netLabelHeight: this.getNetLabelHeight(label),
     })
@@ -1141,6 +1145,7 @@ export class AvailableNetOrientationSolver extends BaseSolver {
         inputProblem: this.inputProblem,
         netId: label.netId,
         pinIds: label.pinIds,
+        isPortOnlyLabel: this.isPortOnlyLabel(label),
       }),
       netLabelHeight: this.getNetLabelHeight(label),
     })
@@ -1224,6 +1229,7 @@ export class AvailableNetOrientationSolver extends BaseSolver {
         inputProblem: this.inputProblem,
         netId: label.netId,
         pinIds: label.pinIds,
+        isPortOnlyLabel: this.isPortOnlyLabel(label),
       }),
       netLabelHeight: this.getNetLabelHeight(label),
     })
@@ -1309,6 +1315,7 @@ export class AvailableNetOrientationSolver extends BaseSolver {
         inputProblem: this.inputProblem,
         netId: label.netId,
         pinIds: label.pinIds,
+        isPortOnlyLabel: this.isPortOnlyLabel(label),
       }),
       netLabelHeight: this.getNetLabelHeight(label),
     })

package/lib/solvers/NetLabelPlacementSolver/NetLabelPlacementSolver.ts

@@ -286,6 +286,7 @@ export class NetLabelPlacementSolver extends BaseSolver {
       inputProblem: this.inputProblem,
       netId: group.netId,
       pinIds,
+      isPortOnlyLabel: group.portOnlyPinId !== undefined,
     })
   }
 

package/lib/solvers/SameNetJunctionAlignmentSolver/alignSameNetJunctions.ts

@@ -34,6 +34,9 @@ interface HorizontalSegment {
 const MAX_ALIGNED_LOAD_PIN_OFFSET = 0.2
 // Limit label-boundary alignment to small corrections that cannot create spikes.
 const MAX_SAME_NET_LABEL_BOUNDARY_RAIL_OFFSET = 0.2
+// A shared vertical pin can anchor an existing rail when the rail is only a
+// symbol-stem correction away from the pin itself.
+const MAX_SHARED_PIN_RAIL_OFFSET = 0.05
 
 const getSharedPin = ({
   donorTrace,
@@ -96,6 +99,48 @@ const railIsOnFacingSide = ({
   return false
 }
 
+const railIsOnSharedPinFacingSide = ({
+  railY,
+  pin,
+}: {
+  railY: number
+  pin: InputPin
+}) => {
+  if (pin._facingDirection === "y+") return railY > pin.y
+  if (pin._facingDirection === "y-") return railY < pin.y
+  return false
+}
+
+const getHorizontalPinApproachPoint = ({
+  branchTrace,
+  sharedPin,
+  otherPin,
+}: {
+  branchTrace: SolvedTracePath
+  sharedPin: InputPin
+  otherPin: InputPin
+}): Point | null => {
+  if (
+    otherPin._facingDirection !== "x-" &&
+    otherPin._facingDirection !== "x+"
+  ) {
+    return null
+  }
+
+  const sharedToOtherPath =
+    branchTrace.pins[0].pinId === sharedPin.pinId
+      ? branchTrace.tracePath
+      : [...branchTrace.tracePath].reverse()
+  const approachPoint = sharedToOtherPath.at(-2)
+  if (!approachPoint || !nearlyEqual(approachPoint.y, otherPin.y)) return null
+
+  const approachesFromFacingSide =
+    otherPin._facingDirection === "x-"
+      ? approachPoint.x < otherPin.x
+      : approachPoint.x > otherPin.x
+  return approachesFromFacingSide ? approachPoint : null
+}
+
 const getAttachedLabelIndexes = (
   trace: SolvedTracePath,
   netLabelPlacements: NetLabelPlacement[],
@@ -165,7 +210,17 @@ const getAlignedBranchPath = ({
   if (branchRail && nearlyEqual(branchRail.start.y, donorRail.start.y)) {
     return null
   }
-  if (!railIsOnFacingSide({ railY: donorRail.start.y, pin: otherPin })) {
+  const railFacesOtherPin = railIsOnFacingSide({
+    railY: donorRail.start.y,
+    pin: otherPin,
+  })
+  const railFacesSharedPin =
+    railIsOnSharedPinFacingSide({
+      railY: donorRail.start.y,
+      pin: sharedPin,
+    }) &&
+    Math.abs(donorRail.start.y - sharedPin.y) <= MAX_SHARED_PIN_RAIL_OFFSET
+  if (!railFacesOtherPin && !railFacesSharedPin) {
     return null
   }
 
@@ -175,13 +230,31 @@ const getAlignedBranchPath = ({
     (donorOtherPin.x > junction.x && otherPin.x < junction.x)
   if (!extendsDonorRail) return null
 
-  const sharedToOther = simplifyPath([
-    { x: sharedPin.x, y: sharedPin.y },
-    { x: junction.x, y: sharedPin.y },
-    { x: junction.x, y: junction.y },
-    { x: otherPin.x, y: junction.y },
-    { x: otherPin.x, y: otherPin.y },
-  ])
+  let sharedToOther: Point[]
+  if (!railFacesOtherPin && railFacesSharedPin) {
+    const approachPoint = getHorizontalPinApproachPoint({
+      branchTrace,
+      sharedPin,
+      otherPin,
+    })
+    if (!approachPoint) return null
+    sharedToOther = simplifyPath([
+      { x: sharedPin.x, y: sharedPin.y },
+      { x: junction.x, y: sharedPin.y },
+      { x: junction.x, y: junction.y },
+      { x: approachPoint.x, y: junction.y },
+      { x: approachPoint.x, y: otherPin.y },
+      { x: otherPin.x, y: otherPin.y },
+    ])
+  } else {
+    sharedToOther = simplifyPath([
+      { x: sharedPin.x, y: sharedPin.y },
+      { x: junction.x, y: sharedPin.y },
+      { x: junction.x, y: junction.y },
+      { x: otherPin.x, y: junction.y },
+      { x: otherPin.x, y: otherPin.y },
+    ])
+  }
 
   if (branchTrace.pins[0].pinId === sharedPin.pinId) return sharedToOther
   return [...sharedToOther].reverse()

package/lib/solvers/SchematicTraceLinesSolver/SchematicTraceSingleLineSolver2/SchematicTraceSingleLineSolver2.ts

@@ -206,7 +206,7 @@ export class SchematicTraceSingleLineSolver2 extends BaseSolver {
       inputProblem: this.inputProblem,
       netId,
       pinIds: this.pins.map((pin) => pin.pinId),
-      includeFallbackNetLabelWidth: false,
+      isPortOnlyLabel: false,
     })
     const netLabelHeight = this.getNetLabelHeightForConnectionPair(netId)
     const padding: Required<RectPadding> = {

package/lib/solvers/TraceAnchoredNetLabelOverlapSolver/candidates.ts

@@ -315,6 +315,7 @@ const getNetLabelWidth = (
     inputProblem,
     netId: label.netId,
     pinIds: label.pinIds,
+    isPortOnlyLabel: label.mspConnectionPairIds.length === 0,
   })
   if (configuredWidth !== undefined) return configuredWidth
 

package/lib/types/InputProblem.ts

@@ -60,11 +60,11 @@ export interface InputDirectConnection {
   netLabelWidth?: number
 
   /**
-   * Width of the conventional anchored label to use only when an inline label
-   * cannot be placed. Unlike `netLabelWidth`, this does not affect whether or
-   * how the point-to-point connection is routed.
+   * Width of the conventional label anchored to a trace endpoint. Unlike
+   * `netLabelWidth`, this does not affect whether or how the point-to-point
+   * connection is routed.
    */
-  fallbackNetLabelWidth?: number
+  anchoredNetLabelWidth?: number
 
   /**
    * When true, this point-to-point connection may be labeled with an "inline
@@ -106,7 +106,7 @@ export interface InputNetConnection {
    */
   netLabelText?: string
   netLabelWidth?: number
-  fallbackNetLabelWidth?: number
+  anchoredNetLabelWidth?: number
   netLabelHeight?: number
 
   /**

package/lib/utils/getNetLabelWidthForConnection.ts

@@ -10,29 +10,28 @@ type ConnectionWithLabelWidth = InputDirectConnection | InputNetConnection
 
 const getConfiguredWidth = (
   connections: Array<ConnectionWithLabelWidth | undefined>,
-  includeFallbackNetLabelWidth: boolean,
+  isPortOnlyLabel: boolean,
 ) => {
-  const explicitWidth = connections.find(
+  const width = connections.find(
     (connection) => connection?.netLabelWidth !== undefined,
   )?.netLabelWidth
-  if (explicitWidth !== undefined) return explicitWidth
-  if (!includeFallbackNetLabelWidth) return undefined
+  if (width !== undefined || !isPortOnlyLabel) return width
 
   return connections.find(
-    (connection) => connection?.fallbackNetLabelWidth !== undefined,
-  )?.fallbackNetLabelWidth
+    (connection) => connection?.anchoredNetLabelWidth !== undefined,
+  )?.anchoredNetLabelWidth
 }
 
 export const getNetLabelWidthForConnection = ({
   inputProblem,
   netId,
   pinIds,
-  includeFallbackNetLabelWidth = true,
+  isPortOnlyLabel,
 }: {
   inputProblem: InputProblem
   netId?: NetId
   pinIds: readonly PinId[]
-  includeFallbackNetLabelWidth?: boolean
+  isPortOnlyLabel: boolean
 }): number | undefined => {
   if (netId) {
     const widthByNetId = getConfiguredWidth(
@@ -44,7 +43,7 @@ export const getNetLabelWidthForConnection = ({
           (connection) => connection.netId === netId,
         ),
       ],
-      includeFallbackNetLabelWidth,
+      isPortOnlyLabel,
     )
     if (widthByNetId !== undefined) return widthByNetId
   }
@@ -58,6 +57,6 @@ export const getNetLabelWidthForConnection = ({
         connection.pinIds.some((pinId) => pinIds.includes(pinId)),
       ),
     ],
-    includeFallbackNetLabelWidth,
+    isPortOnlyLabel,
   )
 }

package/package.json

@@ -5,7 +5,7 @@
     "url": "https://github.com/tscircuit/schematic-trace-solver.git"
   },
   "main": "dist/index.js",
-  "version": "0.0.152",
+  "version": "0.0.154",
   "type": "module",
   "scripts": {
     "start": "cosmos",

package/site/bug-reports/bug-report-20260825T103621Z.page.tsx

@@ -0,0 +1,4 @@
+import { PipelineDebugger } from "site/components/PipelineDebugger"
+import inputProblem from "../../tests/bug-reports/bug-report-20260825T103621Z/bug-report-20260825T103621Z.json"
+
+export default () => <PipelineDebugger inputProblem={inputProblem as any} />