@tscircuit/checks 0.0.176 → 0.0.178

package/dist/index.js

@@ -3383,6 +3383,19 @@ var getPcbTraceLength = (pcbTrace) => {
   }
   return traceLength;
 };
+var getReferencedPcbPortIds = (pcbTrace) => {
+  const pcbPortIds = /* @__PURE__ */ new Set();
+  for (const routePoint of pcbTrace.route) {
+    if (routePoint.route_type !== "wire") continue;
+    if (routePoint.start_pcb_port_id) {
+      pcbPortIds.add(routePoint.start_pcb_port_id);
+    }
+    if (routePoint.end_pcb_port_id) {
+      pcbPortIds.add(routePoint.end_pcb_port_id);
+    }
+  }
+  return pcbPortIds;
+};
 var checkPcbTraceLengths = (circuitJson) => {
   const sourceTraces = circuitJson.filter(
     (element) => element.type === "source_trace"
@@ -3390,12 +3403,53 @@ var checkPcbTraceLengths = (circuitJson) => {
   const pcbTraces = circuitJson.filter(
     (element) => element.type === "pcb_trace"
   );
+  const pcbPorts = circuitJson.filter(
+    (element) => element.type === "pcb_port"
+  );
   const sourceTracesById = new Map(
     sourceTraces.map((sourceTrace) => [
       sourceTrace.source_trace_id,
       sourceTrace
     ])
   );
+  const pcbPortIdsBySourcePortId = /* @__PURE__ */ new Map();
+  for (const pcbPort of pcbPorts) {
+    if (!pcbPort.source_port_id) continue;
+    const pcbPortIds = pcbPortIdsBySourcePortId.get(pcbPort.source_port_id) ?? /* @__PURE__ */ new Set();
+    pcbPortIds.add(pcbPort.pcb_port_id);
+    pcbPortIdsBySourcePortId.set(pcbPort.source_port_id, pcbPortIds);
+  }
+  const pcbTracesBySourceTraceId = /* @__PURE__ */ new Map();
+  for (const pcbTrace of pcbTraces) {
+    if (!pcbTrace.source_trace_id) continue;
+    const matchingPcbTraces = pcbTracesBySourceTraceId.get(pcbTrace.source_trace_id) ?? [];
+    matchingPcbTraces.push(pcbTrace);
+    pcbTracesBySourceTraceId.set(pcbTrace.source_trace_id, matchingPcbTraces);
+  }
+  const exactEndpointPcbTraceIdsBySourceTraceId = /* @__PURE__ */ new Map();
+  for (const sourceTrace of sourceTraces) {
+    if (sourceTrace.connected_source_port_ids.length !== 2) continue;
+    const endpointPcbPortIds = sourceTrace.connected_source_port_ids.map(
+      (sourcePortId) => pcbPortIdsBySourcePortId.get(sourcePortId)
+    );
+    if (endpointPcbPortIds.some((pcbPortIds) => !pcbPortIds?.size)) continue;
+    const exactEndpointPcbTraceIds = new Set(
+      (pcbTracesBySourceTraceId.get(sourceTrace.source_trace_id) ?? []).filter((pcbTrace) => {
+        const referencedPcbPortIds = getReferencedPcbPortIds(pcbTrace);
+        return endpointPcbPortIds.every(
+          (pcbPortIds) => [...pcbPortIds].some(
+            (pcbPortId) => referencedPcbPortIds.has(pcbPortId)
+          )
+        );
+      }).map((pcbTrace) => pcbTrace.pcb_trace_id)
+    );
+    if (exactEndpointPcbTraceIds.size > 0) {
+      exactEndpointPcbTraceIdsBySourceTraceId.set(
+        sourceTrace.source_trace_id,
+        exactEndpointPcbTraceIds
+      );
+    }
+  }
   const warnings = [];
   for (const pcbTrace of pcbTraces) {
     if (!pcbTrace.source_trace_id) continue;
@@ -3403,6 +3457,10 @@ var checkPcbTraceLengths = (circuitJson) => {
     if (!sourceTrace) continue;
     const maximumTraceLength = sourceTrace.max_length;
     if (typeof maximumTraceLength !== "number") continue;
+    const exactEndpointPcbTraceIds = exactEndpointPcbTraceIdsBySourceTraceId.get(sourceTrace.source_trace_id);
+    if (exactEndpointPcbTraceIds && !exactEndpointPcbTraceIds.has(pcbTrace.pcb_trace_id)) {
+      continue;
+    }
     const actualTraceLength = getPcbTraceLength(pcbTrace);
     if (actualTraceLength <= maximumTraceLength) continue;
     warnings.push({
@@ -3920,6 +3978,11 @@ function checkAllPinsInComponentAreUnderspecified(circuitJson) {
 
 // lib/check-no-power-pin-defined.ts
 import { cju as cju8 } from "@tscircuit/circuit-json-util";
+
+// lib/util/should-check-chip-power-ground-pins.ts
+var shouldCheckChipPowerGroundPins = (component, ports) => component.ftype === "simple_chip" && ports.filter((port) => port.do_not_connect !== true).length >= 2;
+
+// lib/check-no-power-pin-defined.ts
 function checkNoPowerPinDefined(circuitJson) {
   const warnings = [];
   const db = cju8(circuitJson);
@@ -3933,9 +3996,8 @@ function checkNoPowerPinDefined(circuitJson) {
     portsByComponent.set(port.source_component_id, existing);
   }
   for (const component of sourceComponents) {
-    if (component.ftype !== "simple_chip") continue;
     const componentPorts = portsByComponent.get(component.source_component_id) ?? [];
-    if (componentPorts.length === 0) continue;
+    if (!shouldCheckChipPowerGroundPins(component, componentPorts)) continue;
     const hasRequiredPowerPin = componentPorts.some(
       (port) => port.requires_power === true
     );
@@ -3968,9 +4030,8 @@ function checkNoGroundPinDefined(circuitJson) {
     portsByComponent.set(port.source_component_id, existing);
   }
   for (const component of sourceComponents) {
-    if (component.ftype !== "simple_chip") continue;
     const componentPorts = portsByComponent.get(component.source_component_id) ?? [];
-    if (componentPorts.length === 0) continue;
+    if (!shouldCheckChipPowerGroundPins(component, componentPorts)) continue;
     const hasRequiredGroundPin = componentPorts.some(
       (port) => port.requires_ground === true
     );