@tscircuit/core 0.0.1140 → 0.0.1141

package/dist/index.js

@@ -3885,6 +3885,64 @@ var getViaDiameterDefaultsWithOverrides = (overrides, pcbStyle) => {
   };
 };
 
+// lib/components/primitive-components/Port/areAllPcbPrimitivesOverlapping.ts
+var areAllPcbPrimitivesOverlapping = (pcbPrimitives) => {
+  if (pcbPrimitives.length <= 1) return true;
+  const bounds = pcbPrimitives.map((p) => {
+    const circuitBounds = p._getPcbCircuitJsonBounds();
+    return {
+      left: circuitBounds.bounds.left,
+      right: circuitBounds.bounds.right,
+      top: circuitBounds.bounds.top,
+      bottom: circuitBounds.bounds.bottom
+    };
+  });
+  const overlaps = Array(bounds.length).fill(false).map(() => Array(bounds.length).fill(false));
+  for (let i = 0; i < bounds.length; i++) {
+    for (let j = i + 1; j < bounds.length; j++) {
+      const a = bounds[i];
+      const b = bounds[j];
+      overlaps[i][j] = overlaps[j][i] = !(a.right < b.left || a.left > b.right || a.bottom > b.top || a.top < b.bottom);
+    }
+  }
+  const visited = /* @__PURE__ */ new Set();
+  const dfs = (node) => {
+    visited.add(node);
+    for (let i = 0; i < bounds.length; i++) {
+      if (overlaps[node][i] && !visited.has(i)) {
+        dfs(i);
+      }
+    }
+  };
+  dfs(0);
+  return visited.size === bounds.length;
+};
+
+// lib/components/primitive-components/Trace/getPcbSelectorErrorForTracePort.ts
+function formatPcbPrimitiveForError(component) {
+  const portHints = component._parsedProps?.portHints ?? [];
+  const aliases = portHints.filter(Boolean);
+  if (aliases.length > 0) {
+    return `<${component.lowercaseComponentName}(${aliases.map((alias) => `.${alias}`).join(", ")}) />`;
+  }
+  return `<${component.lowercaseComponentName} />`;
+}
+function getPcbSelectorErrorForTracePort(selector, port) {
+  if (port.pcb_port_id) return null;
+  const pcbMatches = port.matchedComponents.filter((c) => c.isPcbPrimitive);
+  const parentName = port.getParentNormalComponent()?.props.name ?? port.parent?.props.name ?? "unknown";
+  const rawPinSelectors = Array.from(
+    new Set(
+      port.getNameAndAliases().filter((alias) => /^pin\d+$/.test(alias)).map((alias) => `${parentName}.${alias}`)
+    )
+  );
+  if (pcbMatches.length > 1 && !areAllPcbPrimitivesOverlapping(pcbMatches)) {
+    const suggestion = rawPinSelectors.length > 0 ? ` Use a raw pin selector like ${rawPinSelectors.map((s) => `"${s}"`).join(" or ")}.` : "";
+    return `Trace selector "${selector}" resolved to "${parentName}.${port.props.name}", but that target maps to multiple non-overlapping PCB pads: ${pcbMatches.map((c) => formatPcbPrimitiveForError(c)).join(", ")}.${suggestion}`;
+  }
+  return null;
+}
+
 // lib/components/primitive-components/Trace/Trace_doInitialPcbTraceRender.ts
 var portToObjective = (port) => {
   const portPosition = port._getGlobalPcbPositionAfterLayout();
@@ -3928,10 +3986,12 @@ function Trace_doInitialPcbTraceRender(trace) {
   }
   let allPortsFound;
   let ports;
+  let portsWithSelectors;
   try {
     const connectedPorts = trace._findConnectedPorts();
     allPortsFound = connectedPorts.allPortsFound;
     ports = connectedPorts.ports ?? [];
+    portsWithSelectors = connectedPorts.portsWithSelectors ?? [];
   } catch (error) {
     if (error instanceof TraceConnectionError) {
       db.source_trace_not_connected_error.insert({
@@ -3945,6 +4005,20 @@ function Trace_doInitialPcbTraceRender(trace) {
   }
   const portsConnectedOnPcbViaNet = [];
   if (!allPortsFound) return;
+  const pcbSelectorError = portsWithSelectors.map(
+    ({ selector, port }) => getPcbSelectorErrorForTracePort(selector, port)
+  ).find(Boolean);
+  if (pcbSelectorError) {
+    db.pcb_trace_error.insert({
+      error_type: "pcb_trace_error",
+      source_trace_id: trace.source_trace_id,
+      message: pcbSelectorError,
+      pcb_trace_id: trace.pcb_trace_id,
+      pcb_component_ids: [],
+      pcb_port_ids: ports.map((p) => p.pcb_port_id).filter(Boolean)
+    });
+    return;
+  }
   const portsWithoutMatchedPcbPrimitive = [];
   for (const port of ports) {
     if (!port._hasMatchedPcbPrimitive()) {
@@ -4368,6 +4442,20 @@ function Trace_doInitialPcbManualTraceRender(trace) {
     throw error;
   }
   if (!allPortsFound) return;
+  const pcbSelectorError = portsWithSelectors.map(
+    ({ selector, port }) => getPcbSelectorErrorForTracePort(selector, port)
+  ).find(Boolean);
+  if (pcbSelectorError) {
+    db.pcb_trace_error.insert({
+      error_type: "pcb_trace_error",
+      source_trace_id: trace.source_trace_id,
+      message: pcbSelectorError,
+      pcb_trace_id: trace.pcb_trace_id,
+      pcb_component_ids: [],
+      pcb_port_ids: ports.map((p) => p.pcb_port_id).filter(Boolean)
+    });
+    return;
+  }
   const portsWithoutMatchedPcbPrimitive = [];
   for (const port of ports) {
     if (!port._hasMatchedPcbPrimitive()) {
@@ -4491,6 +4579,18 @@ function Trace_doInitialPcbManualTraceRender(trace) {
         });
         continue;
       }
+      const pcbTargetError = getPcbSelectorErrorForTracePort(pt, resolvedPort);
+      if (pcbTargetError) {
+        db.pcb_trace_error.insert({
+          error_type: "pcb_trace_error",
+          source_trace_id: trace.source_trace_id,
+          message: pcbTargetError,
+          pcb_trace_id: trace.pcb_trace_id,
+          pcb_component_ids: [],
+          pcb_port_ids: []
+        });
+        return;
+      }
       const portPos = resolvedPort._getGlobalPcbPositionAfterLayout();
       coordinates = { x: portPos.x, y: portPos.y };
       isGlobalPosition = true;
@@ -5262,14 +5362,27 @@ var createInstanceFromReactElement = (reactElm) => {
 import "circuit-json";
 import "zod";
 
+// lib/utils/schematic/getPinNumberFromPinLabelsKey.ts
+var PIN_LABELS_KEY_RE = /^(?:pin)?(\d+)$/;
+var getPinNumberFromPinLabelsKey = (pinKey) => {
+  const match = pinKey.match(PIN_LABELS_KEY_RE);
+  if (!match) return null;
+  return Number.parseInt(match[1], 10);
+};
+
 // lib/utils/schematic/parsePinNumberFromLabelsOrThrow.ts
 var parsePinNumberFromLabelsOrThrow = (pinNumberOrLabel, pinLabels) => {
   if (typeof pinNumberOrLabel === "number") {
     return pinNumberOrLabel;
   }
+  const directPinNumber = getPinNumberFromPinLabelsKey(pinNumberOrLabel);
+  if (directPinNumber !== null) {
+    return directPinNumber;
+  }
   if (pinNumberOrLabel.startsWith("pin")) {
-    const pinNumber = Number(pinNumberOrLabel.slice(3));
-    return pinNumber;
+    throw new Error(
+      `Invalid pinLabels key "${pinNumberOrLabel}". Expected "pin\${number}" (e.g. pin1, pin2).`
+    );
   }
   if (!pinLabels) {
     throw new Error(
@@ -5279,7 +5392,13 @@ var parsePinNumberFromLabelsOrThrow = (pinNumberOrLabel, pinLabels) => {
   for (const pinNumberKey in pinLabels) {
     const aliases = Array.isArray(pinLabels[pinNumberKey]) ? pinLabels[pinNumberKey] : [pinLabels[pinNumberKey]];
     if (aliases.includes(pinNumberOrLabel)) {
-      return Number(pinNumberKey.replace("pin", ""));
+      const pinNumber = getPinNumberFromPinLabelsKey(pinNumberKey);
+      if (pinNumber === null) {
+        throw new Error(
+          `Invalid pinLabels key "${pinNumberKey}". Expected "pin\${number}" (e.g. pin1, pin2).`
+        );
+      }
+      return pinNumber;
     }
   }
   throw new Error(
@@ -8360,39 +8479,6 @@ import "schematic-symbols";
 import { applyToPoint as applyToPoint16, compose as compose4, translate as translate3 } from "transformation-matrix";
 import { z as z7 } from "zod";
 
-// lib/components/primitive-components/Port/areAllPcbPrimitivesOverlapping.ts
-var areAllPcbPrimitivesOverlapping = (pcbPrimitives) => {
-  if (pcbPrimitives.length <= 1) return true;
-  const bounds = pcbPrimitives.map((p) => {
-    const circuitBounds = p._getPcbCircuitJsonBounds();
-    return {
-      left: circuitBounds.bounds.left,
-      right: circuitBounds.bounds.right,
-      top: circuitBounds.bounds.top,
-      bottom: circuitBounds.bounds.bottom
-    };
-  });
-  const overlaps = Array(bounds.length).fill(false).map(() => Array(bounds.length).fill(false));
-  for (let i = 0; i < bounds.length; i++) {
-    for (let j = i + 1; j < bounds.length; j++) {
-      const a = bounds[i];
-      const b = bounds[j];
-      overlaps[i][j] = overlaps[j][i] = !(a.right < b.left || a.left > b.right || a.bottom > b.top || a.top < b.bottom);
-    }
-  }
-  const visited = /* @__PURE__ */ new Set();
-  const dfs = (node) => {
-    visited.add(node);
-    for (let i = 0; i < bounds.length; i++) {
-      if (overlaps[node][i] && !visited.has(i)) {
-        dfs(i);
-      }
-    }
-  };
-  dfs(0);
-  return visited.size === bounds.length;
-};
-
 // lib/components/primitive-components/Port/getCenterOfPcbPrimitives.ts
 var getCenterOfPcbPrimitives = (pcbPrimitives) => {
   if (pcbPrimitives.length === 0) {
@@ -10384,6 +10470,21 @@ var NormalComponent3 = class extends PrimitiveComponent2 {
       invalidPinLabelsMessages = messages;
     }
     super(filteredProps);
+    if (filteredProps.pinLabels && !Array.isArray(filteredProps.pinLabels)) {
+      const invalidPinKey = Object.keys(filteredProps.pinLabels).find(
+        (pinKey) => getPinNumberFromPinLabelsKey(pinKey) === null
+      );
+      if (invalidPinKey) {
+        throw new InvalidProps(this.lowercaseComponentName, this.props, {
+          _errors: [],
+          pinLabels: {
+            _errors: [
+              `Invalid pinLabels key "${invalidPinKey}". Expected "pin\${number}" (e.g. pin1, pin2).`
+            ]
+          }
+        });
+      }
+    }
     this._invalidPinLabelMessages = invalidPinLabelsMessages;
     this._addChildrenFromStringFootprint();
     this.initPorts();
@@ -10476,21 +10577,26 @@ var NormalComponent3 = class extends PrimitiveComponent2 {
     }
     const pinLabels = this._parsedProps.pinLabels;
     if (pinLabels) {
-      for (let [pinNumber, label] of Object.entries(pinLabels)) {
-        pinNumber = pinNumber.replace("pin", "");
+      for (const [pinKey, label] of Object.entries(pinLabels)) {
+        const pinNumber = getPinNumberFromPinLabelsKey(pinKey);
+        if (pinNumber === null) {
+          throw new Error(
+            `Invalid pinLabels key "${pinKey}". Expected "pin\${number}" (e.g. pin1, pin2).`
+          );
+        }
         let existingPort = portsToCreate.find(
-          (p) => p._parsedProps.pinNumber === Number(pinNumber)
+          (p) => p._parsedProps.pinNumber === pinNumber
         );
         const primaryLabel = Array.isArray(label) ? label[0] : label;
         const otherLabels = Array.isArray(label) ? label.slice(1) : [];
         if (!existingPort) {
           existingPort = new Port(
             {
-              pinNumber: parseInt(pinNumber),
+              pinNumber,
               name: primaryLabel,
               aliases: [
                 ...otherLabels,
-                ...opts.additionalAliases?.[`pin${parseInt(pinNumber)}`] ?? []
+                ...opts.additionalAliases?.[`pin${pinNumber}`] ?? []
               ]
             },
             {
@@ -11274,7 +11380,7 @@ var NormalComponent3 = class extends PrimitiveComponent2 {
       if (Array.isArray(pinLabels)) {
         return pinLabels.length;
       }
-      const pinNumbers = Object.keys(pinLabels).map((k) => k.startsWith("pin") ? parseInt(k.slice(3)) : parseInt(k)).filter((n) => !Number.isNaN(n));
+      const pinNumbers = Object.keys(pinLabels).map((k) => getPinNumberFromPinLabelsKey(k)).filter((n) => n !== null && !Number.isNaN(n));
       if (pinNumbers.length > 0) {
         return Math.max(...pinNumbers);
       }
@@ -18849,7 +18955,7 @@ import { identity as identity5 } from "transformation-matrix";
 var package_default = {
   name: "@tscircuit/core",
   type: "module",
-  version: "0.0.1139",
+  version: "0.0.1140",
   types: "dist/index.d.ts",
   main: "dist/index.js",
   module: "dist/index.js",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tscircuit/core",
   "type": "module",
-  "version": "0.0.1140",
+  "version": "0.0.1141",
   "types": "dist/index.d.ts",
   "main": "dist/index.js",
   "module": "dist/index.js",