@tscircuit/checks 0.0.175 → 0.0.177

package/dist/index.js

@@ -2690,14 +2690,82 @@ function checkSourceTracesHavePcbTraces(circuitJson) {
 }
 
 // lib/check-traces-are-contiguous/check-traces-are-contiguous.ts
+import { pointToSegmentDistance as pointToSegmentDistance2 } from "@tscircuit/math-utils";
 import {
   getReadableNameForPcbPort as getReadableNameForPcbPort2,
   getReadableNameForPcbTrace
 } from "@tscircuit/circuit-json-util";
-import { PcbConnectivityMap as PcbConnectivityMap2 } from "circuit-json-to-connectivity-map";
+import {
+  getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson7,
+  PcbConnectivityMap as PcbConnectivityMap2
+} from "circuit-json-to-connectivity-map";
+var ENDPOINT_CONTACT_EPSILON = 1e-9;
+var TRACE_SEGMENT_GEOMETRY_EPSILON = 1e-9;
 function routePointTouchesPad(point, pad) {
   return point.route_type === "wire" && getLayersOfPcbElement(pad).includes(point.layer) && isPointInPad(point, pad);
 }
+function getTraceWireSegmentsByNetAndLayer(pcbTraces, fullConnectivityMap) {
+  const segmentsByNetAndLayer = /* @__PURE__ */ new Map();
+  for (const trace of pcbTraces) {
+    if (trace.route_thickness_mode === "interpolated") continue;
+    const netId = fullConnectivityMap.getNetConnectedToId(trace.pcb_trace_id);
+    if (!netId) continue;
+    for (let i = 0; i < trace.route.length - 1; i++) {
+      const start = trace.route[i];
+      const end = trace.route[i + 1];
+      if (start.route_type !== "wire" || end.route_type !== "wire") continue;
+      if (start.layer !== end.layer) continue;
+      if (Math.hypot(start.x - end.x, start.y - end.y) <= TRACE_SEGMENT_GEOMETRY_EPSILON) {
+        continue;
+      }
+      const segmentsByLayer = segmentsByNetAndLayer.get(netId) ?? /* @__PURE__ */ new Map();
+      const segments = segmentsByLayer.get(start.layer) ?? [];
+      segments.push({ trace, start, end });
+      segmentsByLayer.set(start.layer, segments);
+      segmentsByNetAndLayer.set(netId, segmentsByLayer);
+    }
+  }
+  return segmentsByNetAndLayer;
+}
+function getEndpointTraceCopperWidth(trace, endpoint) {
+  if (trace.route_thickness_mode === "interpolated") return void 0;
+  let segmentStartIndex = endpoint === "start" ? 0 : trace.route.length - 2;
+  const indexStep = endpoint === "start" ? 1 : -1;
+  while (segmentStartIndex >= 0 && segmentStartIndex < trace.route.length - 1) {
+    const segmentStart = trace.route[segmentStartIndex];
+    const segmentEnd = trace.route[segmentStartIndex + 1];
+    if (segmentStart?.route_type !== "wire" || segmentEnd?.route_type !== "wire" || segmentStart.layer !== segmentEnd.layer) {
+      return void 0;
+    }
+    if (Math.hypot(segmentStart.x - segmentEnd.x, segmentStart.y - segmentEnd.y) > TRACE_SEGMENT_GEOMETRY_EPSILON) {
+      return segmentStart.width;
+    }
+    segmentStartIndex += indexStep;
+  }
+  return void 0;
+}
+function routePointTouchesLogicallyConnectedTraceCopper({
+  point,
+  endpointTraceCopperWidth,
+  ownerTrace,
+  traceWireSegmentsByNetAndLayer,
+  fullConnectivityMap
+}) {
+  if (point.route_type !== "wire") return false;
+  const ownerNetId = fullConnectivityMap.getNetConnectedToId(
+    ownerTrace.pcb_trace_id
+  );
+  if (!ownerNetId) return false;
+  const candidateSegments = traceWireSegmentsByNetAndLayer.get(ownerNetId)?.get(point.layer) ?? [];
+  for (const segment of candidateSegments) {
+    if (segment.trace.pcb_trace_id === ownerTrace.pcb_trace_id) continue;
+    const maximumContactDistance = endpointTraceCopperWidth / 2 + segment.start.width / 2 + ENDPOINT_CONTACT_EPSILON;
+    if (pointToSegmentDistance2(point, segment.start, segment.end) <= maximumContactDistance) {
+      return true;
+    }
+  }
+  return false;
+}
 function getRoutePointCenter(point) {
   if (point.route_type === "through_pad") {
     return {
@@ -2783,6 +2851,19 @@ function checkTracesAreContiguous(circuitJson) {
   );
   const padMap = /* @__PURE__ */ new Map();
   const pcbConnectivityMap = new PcbConnectivityMap2(circuitJson);
+  let fullConnectivityMap;
+  let traceWireSegmentsByNetAndLayer;
+  const getFullConnectivityMap = () => {
+    fullConnectivityMap ??= getFullConnectivityMapFromCircuitJson7(circuitJson);
+    return fullConnectivityMap;
+  };
+  const getTraceWireSegmentIndex = () => {
+    traceWireSegmentsByNetAndLayer ??= getTraceWireSegmentsByNetAndLayer(
+      pcbTraces,
+      getFullConnectivityMap()
+    );
+    return traceWireSegmentsByNetAndLayer;
+  };
   const checkedSourceTraceIds = /* @__PURE__ */ new Set();
   for (const pad of pcbSmtPads) {
     if (pad.pcb_port_id) {
@@ -2948,7 +3029,6 @@ function checkTracesAreContiguous(circuitJson) {
     if (expectedPorts.length === 0) {
       let firstConnectsToAnyPad = false;
       let lastConnectsToAnyPad = false;
-      const endpointsAreSame = firstPoint.route_type === "wire" && lastPoint.route_type === "wire" && Math.abs(firstPoint.x - lastPoint.x) < 0.01 && Math.abs(firstPoint.y - lastPoint.y) < 0.01;
       for (const pads of padMap.values()) {
         if (pads.some((pad) => routePointTouchesPad(firstPoint, pad))) {
           firstConnectsToAnyPad = true;
@@ -2957,7 +3037,26 @@ function checkTracesAreContiguous(circuitJson) {
           lastConnectsToAnyPad = true;
         }
       }
-      if (!firstConnectsToAnyPad && firstPoint.route_type === "wire") {
+      const firstEndpointTraceCopperWidth = !firstConnectsToAnyPad ? getEndpointTraceCopperWidth(trace, "start") : void 0;
+      const lastEndpointTraceCopperWidth = !lastConnectsToAnyPad ? getEndpointTraceCopperWidth(trace, "end") : void 0;
+      const firstConnectsToLogicallyConnectedTraceCopper = firstEndpointTraceCopperWidth !== void 0 && routePointTouchesLogicallyConnectedTraceCopper({
+        point: firstPoint,
+        endpointTraceCopperWidth: firstEndpointTraceCopperWidth,
+        ownerTrace: trace,
+        traceWireSegmentsByNetAndLayer: getTraceWireSegmentIndex(),
+        fullConnectivityMap: getFullConnectivityMap()
+      });
+      const lastConnectsToLogicallyConnectedTraceCopper = lastEndpointTraceCopperWidth !== void 0 && routePointTouchesLogicallyConnectedTraceCopper({
+        point: lastPoint,
+        endpointTraceCopperWidth: lastEndpointTraceCopperWidth,
+        ownerTrace: trace,
+        traceWireSegmentsByNetAndLayer: getTraceWireSegmentIndex(),
+        fullConnectivityMap: getFullConnectivityMap()
+      });
+      const firstIsConnected = firstConnectsToAnyPad || firstConnectsToLogicallyConnectedTraceCopper;
+      const lastIsConnected = lastConnectsToAnyPad || lastConnectsToLogicallyConnectedTraceCopper;
+      const endpointsAreSame = firstPoint.route_type === "wire" && lastPoint.route_type === "wire" && firstPoint.layer === lastPoint.layer && Math.hypot(firstPoint.x - lastPoint.x, firstPoint.y - lastPoint.y) <= ENDPOINT_CONTACT_EPSILON;
+      if (!firstIsConnected && firstPoint.route_type === "wire") {
         errors.push({
           type: "pcb_trace_error",
           message: `Trace [${traceName}] has disconnected endpoint at (${firstPoint.x.toFixed(2)}, ${firstPoint.y.toFixed(2)})`,
@@ -2970,7 +3069,7 @@ function checkTracesAreContiguous(circuitJson) {
           pcb_port_ids: []
         });
       }
-      if (!lastConnectsToAnyPad && lastPoint.route_type === "wire" && !(endpointsAreSame && !firstConnectsToAnyPad)) {
+      if (!lastIsConnected && lastPoint.route_type === "wire" && !(endpointsAreSame && !firstIsConnected)) {
         errors.push({
           type: "pcb_trace_error",
           message: `Trace [${traceName}] has disconnected endpoint at (${lastPoint.x.toFixed(2)}, ${lastPoint.y.toFixed(2)})`,
@@ -3074,7 +3173,7 @@ import {
   getPrimaryId as getPrimaryId3
 } from "@tscircuit/circuit-json-util";
 import { doBoundsOverlap as doBoundsOverlap3 } from "@tscircuit/math-utils";
-import { getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson7 } from "circuit-json-to-connectivity-map";
+import { getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson8 } from "circuit-json-to-connectivity-map";
 
 // lib/check-pcb-components-overlap/doPcbElementsOverlap.ts
 import { getBoundsOfPcbElements as getBoundsOfPcbElements4 } from "@tscircuit/circuit-json-util";
@@ -3113,7 +3212,7 @@ var formatOverlapElementDescription = (circuitJson, element) => {
 };
 function checkPcbComponentOverlap(circuitJson) {
   const errors = [];
-  const connMap = getFullConnectivityMapFromCircuitJson7(circuitJson);
+  const connMap = getFullConnectivityMapFromCircuitJson8(circuitJson);
   const smtPads = cju6(circuitJson).pcb_smtpad.list();
   const platedHoles = cju6(circuitJson).pcb_plated_hole.list();
   const holes = cju6(circuitJson).pcb_hole.list();
@@ -3377,7 +3476,7 @@ import {
 } from "@tscircuit/circuit-json-util";
 import { formatMm } from "format-si-unit";
 import {
-  getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson8
+  getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson9
 } from "circuit-json-to-connectivity-map";
 function checkPadPadClearance(circuitJson, {
   connMap,
@@ -3387,7 +3486,7 @@ function checkPadPadClearance(circuitJson, {
   if (pads.length < 2) return [];
   const board = getPcbBoard(circuitJson);
   minClearance ??= getBoardDrcValue(board, "min_pad_edge_to_pad_edge_clearance") ?? jlcMinTolerances.min_pad_edge_to_pad_edge_clearance;
-  connMap ??= getFullConnectivityMapFromCircuitJson8(circuitJson);
+  connMap ??= getFullConnectivityMapFromCircuitJson9(circuitJson);
   const spatialIndex = new SpatialObjectIndex({
     objects: pads,
     getBounds: getPadBounds,
@@ -3442,7 +3541,7 @@ import {
 } from "@tscircuit/circuit-json-util";
 import { formatMm as formatMm2 } from "format-si-unit";
 import {
-  getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson9
+  getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson10
 } from "circuit-json-to-connectivity-map";
 function checkPadTraceClearance(circuitJson, {
   connMap,
@@ -3453,7 +3552,7 @@ function checkPadTraceClearance(circuitJson, {
   if (pads.length === 0 || segments.length === 0) return [];
   const board = getPcbBoard(circuitJson);
   minClearance ??= getBoardDrcValue(board, "min_trace_to_pad_edge_clearance") ?? jlcMinTolerances.min_trace_to_pad_edge_clearance;
-  connMap ??= getFullConnectivityMapFromCircuitJson9(circuitJson);
+  connMap ??= getFullConnectivityMapFromCircuitJson10(circuitJson);
   const spatialIndex = new SpatialObjectIndex({
     objects: pads,
     getBounds: getPadBounds,
@@ -3503,7 +3602,7 @@ function checkPadTraceClearance(circuitJson, {
 import { getReadableNameForElement as getReadableNameForElement8 } from "@tscircuit/circuit-json-util";
 import { formatMm as formatMm3 } from "format-si-unit";
 import {
-  getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson10
+  getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson11
 } from "circuit-json-to-connectivity-map";
 function checkViaTraceClearance(circuitJson, {
   connMap,
@@ -3514,7 +3613,7 @@ function checkViaTraceClearance(circuitJson, {
   if (vias.length === 0 || segments.length === 0) return [];
   const board = getPcbBoard(circuitJson);
   minClearance ??= getBoardDrcValue(board, "min_trace_to_pad_edge_clearance") ?? jlcMinTolerances.min_trace_to_pad_edge_clearance;
-  connMap ??= getFullConnectivityMapFromCircuitJson10(circuitJson);
+  connMap ??= getFullConnectivityMapFromCircuitJson11(circuitJson);
   const errors = /* @__PURE__ */ new Map();
   const overlappingPairIds = /* @__PURE__ */ new Set();
   for (const via of vias) {
@@ -3558,7 +3657,7 @@ import {
 } from "@tscircuit/circuit-json-util";
 import { formatMm as formatMm4 } from "format-si-unit";
 import {
-  getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson11
+  getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson12
 } from "circuit-json-to-connectivity-map";
 function checkViaPadClearance(circuitJson, {
   connMap,
@@ -3571,7 +3670,7 @@ function checkViaPadClearance(circuitJson, {
   if (vias.length === 0 || pads.length === 0) return [];
   const board = getPcbBoard(circuitJson);
   const requiredClearance = minClearance ?? getBoardDrcValue(board, "min_pad_edge_to_pad_edge_clearance") ?? jlcMinTolerances.min_pad_edge_to_pad_edge_clearance;
-  connMap ??= getFullConnectivityMapFromCircuitJson11(circuitJson);
+  connMap ??= getFullConnectivityMapFromCircuitJson12(circuitJson);
   const padIndex = new SpatialObjectIndex({
     objects: pads,
     getBounds: getPadBounds,
@@ -3615,7 +3714,7 @@ function checkViaPadClearance(circuitJson, {
 
 // lib/check-vias-in-pads.ts
 import { getPrimaryId as getPrimaryId7 } from "@tscircuit/circuit-json-util";
-import { getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson12 } from "circuit-json-to-connectivity-map";
+import { getFullConnectivityMapFromCircuitJson as getFullConnectivityMapFromCircuitJson13 } from "circuit-json-to-connectivity-map";
 function checkViasInPads(circuitJson) {
   const board = getPcbBoard(circuitJson);
   if (board && "is_via_in_pad_allowed" in board && board.is_via_in_pad_allowed === true) {
@@ -3626,7 +3725,7 @@ function checkViasInPads(circuitJson) {
   );
   const pads = getPads(circuitJson);
   if (vias.length === 0 || pads.length === 0) return [];
-  const connMap = getFullConnectivityMapFromCircuitJson12(circuitJson);
+  const connMap = getFullConnectivityMapFromCircuitJson13(circuitJson);
   const padOrdinals = new Map(
     pads.map((pad, index) => [getPrimaryId7(pad), index])
   );
@@ -3821,6 +3920,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);
@@ -3834,9 +3938,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
     );
@@ -3869,9 +3972,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
     );