@tscircuit/core 0.0.1105 → 0.0.1107

package/dist/index.d.ts

@@ -512,12 +512,13 @@ declare abstract class PrimitiveComponent<ZodProps extends ZodType = any> extend
     protected _validatePcbCoordinateReferences(params: {
         rawValue: unknown;
         axis: "pcbX" | "pcbY";
-        propertyNameForError?: string;
+        propertyName?: string;
     }): void;
     protected _resolvePcbCoordinate(rawValue: unknown, axis: "pcbX" | "pcbY", options?: {
         allowBoardVariables?: boolean;
         allowComponentVariables?: boolean;
         componentVariables?: Record<string, number>;
+        propertyName?: string;
     }): number;
     private _shouldAllowBoardVariablesByDefault;
     private _isInsideFootprint;
@@ -527,10 +528,13 @@ declare abstract class PrimitiveComponent<ZodProps extends ZodType = any> extend
      * Position can be specified via pcbX/pcbY or edge-based props.
      */
     _hasUserDefinedPcbPosition(): boolean;
-    resolvePcbCoordinate(rawValue: unknown, axis: "pcbX" | "pcbY", options?: {
+    resolvePcbCoordinate(params: {
+        rawValue: unknown;
+        axis: "pcbX" | "pcbY";
         allowBoardVariables?: boolean;
         allowComponentVariables?: boolean;
         componentVariables?: Record<string, number>;
+        propertyName?: string;
     }): number;
     /**
      * Computes a transformation matrix from the props of this component for PCB

package/dist/index.js

@@ -1271,16 +1271,16 @@ var PrimitiveComponent2 = class extends Renderable {
     this._validatePcbCoordinateReferences({
       rawValue: rawPcbX,
       axis: "pcbX",
-      propertyNameForError: "pcbX"
+      propertyName: "pcbX"
     });
     this._validatePcbCoordinateReferences({
       rawValue: rawPcbY,
       axis: "pcbY",
-      propertyNameForError: "pcbY"
+      propertyName: "pcbY"
     });
   }
   _validatePcbCoordinateReferences(params) {
-    const { rawValue, axis, propertyNameForError = axis } = params;
+    const { rawValue, axis, propertyName = axis } = params;
     if (typeof rawValue !== "string") return;
     const isNormalComponent = this._isNormalComponent === true;
     const allowComponentVariables = !isNormalComponent && !this._isInsideFootprint();
@@ -1289,8 +1289,8 @@ var PrimitiveComponent2 = class extends Renderable {
       calcIdentifiers = extractCalcIdentifiers(rawValue);
     } catch {
       this._reportInvalidComponentPropertyError(
-        propertyNameForError,
-        `Invalid ${propertyNameForError} value for ${this.componentName}: Invalid calc() expression. expression="${rawValue}"`
+        propertyName,
+        `Invalid ${propertyName} value for ${this.componentName}: Invalid calc() expression. expression="${rawValue}"`
       );
       return;
     }
@@ -1299,14 +1299,18 @@ var PrimitiveComponent2 = class extends Renderable {
     );
     if (includesComponentVariable && !allowComponentVariables) {
       this._reportInvalidComponentPropertyError(
-        propertyNameForError,
-        `Invalid ${propertyNameForError} value for ${this.componentName}: component-relative calc references are not supported for footprint elements (${this.componentName}); ${propertyNameForError} will be ignored. expression="${rawValue}"`
+        propertyName,
+        `Invalid ${propertyName} value for ${this.componentName}: component-relative calc references are not supported for footprint elements (${this.componentName}); ${propertyName} will be ignored. expression="${rawValue}"`
       );
     }
   }
   _resolvePcbCoordinate(rawValue, axis, options = {}) {
     if (rawValue == null) return 0;
-    if (typeof rawValue === "number") return rawValue;
+    const propertyName = options.propertyName ?? axis;
+    if (typeof rawValue === "number") {
+      if (Number.isFinite(rawValue)) return rawValue;
+      return 0;
+    }
     if (typeof rawValue !== "string") {
       throw new Error(
         `Invalid ${axis} value for ${this.componentName}: ${String(rawValue)}`
@@ -1321,14 +1325,18 @@ var PrimitiveComponent2 = class extends Renderable {
       const board = this._getBoard();
       const boardVariables = board?._getBoardCalcVariables() ?? {};
       if (includesBoardVariable && !board) {
-        throw new Error(
-          `Cannot resolve ${axis} for ${this.componentName}: no board found for board.* variables`
+        this._reportInvalidComponentPropertyError(
+          propertyName,
+          `Invalid ${propertyName} value for ${this.componentName}: no board found for board.* variables. expression="${rawValue}"`
         );
+        return 0;
       }
       if (includesBoardVariable && board && Object.keys(boardVariables).length === 0) {
-        throw new Error(
-          "Cannot do calculations based on board size when the board is auto-sized"
+        this._reportInvalidComponentPropertyError(
+          propertyName,
+          `Invalid ${propertyName} value for ${this.componentName}: Cannot do calculations based on board size when the board is auto-sized. expression="${rawValue}"`
         );
+        return 0;
       }
       Object.assign(knownVariables, boardVariables);
     }
@@ -1359,9 +1367,11 @@ var PrimitiveComponent2 = class extends Renderable {
       return evaluateCalcString(rawValue, { knownVariables });
     } catch (error) {
       const message = error instanceof Error ? error.message : String(error);
-      throw new Error(
-        `Invalid ${axis} value for ${this.componentName}: ${message}`
+      this._reportInvalidComponentPropertyError(
+        propertyName,
+        `Invalid ${propertyName} value for ${this.componentName}: ${message}. expression="${rawValue}"`
       );
+      return 0;
     }
   }
   _shouldAllowBoardVariablesByDefault() {
@@ -1401,8 +1411,21 @@ var PrimitiveComponent2 = class extends Renderable {
     const props = this._parsedProps;
     return props.pcbX !== void 0 || props.pcbY !== void 0 || props.pcbLeftEdgeX !== void 0 || props.pcbRightEdgeX !== void 0 || props.pcbTopEdgeY !== void 0 || props.pcbBottomEdgeY !== void 0;
   }
-  resolvePcbCoordinate(rawValue, axis, options = {}) {
-    return this._resolvePcbCoordinate(rawValue, axis, options);
+  resolvePcbCoordinate(params) {
+    const {
+      rawValue,
+      axis,
+      allowBoardVariables,
+      allowComponentVariables,
+      componentVariables,
+      propertyName
+    } = params;
+    return this._resolvePcbCoordinate(rawValue, axis, {
+      allowBoardVariables,
+      allowComponentVariables,
+      componentVariables,
+      propertyName
+    });
   }
   /**
    * Computes a transformation matrix from the props of this component for PCB
@@ -7001,14 +7024,14 @@ var SilkscreenText = class extends PrimitiveComponent2 {
         isFlipped ? flipY2() : identity4()
       ),
       {
-        x: this.resolvePcbCoordinate(
-          resolvedPcbSxPcbX ?? props.pcbX ?? 0,
-          "pcbX"
-        ),
-        y: this.resolvePcbCoordinate(
-          resolvedPcbSxPcbY ?? props.pcbY ?? 0,
-          "pcbY"
-        )
+        x: this.resolvePcbCoordinate({
+          rawValue: resolvedPcbSxPcbX ?? props.pcbX ?? 0,
+          axis: "pcbX"
+        }),
+        y: this.resolvePcbCoordinate({
+          rawValue: resolvedPcbSxPcbY ?? props.pcbY ?? 0,
+          axis: "pcbY"
+        })
       }
     ) : this._getGlobalPcbPositionBeforeLayout();
     const uniformPadding = props.knockoutPadding ?? 0;
@@ -11210,28 +11233,28 @@ var NormalComponent3 = class extends PrimitiveComponent2 {
       this._validatePcbCoordinateReferences({
         rawValue: pcbLeftEdgeX,
         axis: "pcbX",
-        propertyNameForError: "pcbLeftEdgeX"
+        propertyName: "pcbLeftEdgeX"
       });
     }
     if (pcbRightEdgeX !== void 0) {
       this._validatePcbCoordinateReferences({
         rawValue: pcbRightEdgeX,
         axis: "pcbX",
-        propertyNameForError: "pcbRightEdgeX"
+        propertyName: "pcbRightEdgeX"
       });
     }
     if (pcbTopEdgeY !== void 0) {
       this._validatePcbCoordinateReferences({
         rawValue: pcbTopEdgeY,
         axis: "pcbY",
-        propertyNameForError: "pcbTopEdgeY"
+        propertyName: "pcbTopEdgeY"
       });
     }
     if (pcbBottomEdgeY !== void 0) {
       this._validatePcbCoordinateReferences({
         rawValue: pcbBottomEdgeY,
         axis: "pcbY",
-        propertyNameForError: "pcbBottomEdgeY"
+        propertyName: "pcbBottomEdgeY"
       });
     }
   }
@@ -11263,8 +11286,16 @@ var NormalComponent3 = class extends PrimitiveComponent2 {
     const parentGroup = this.getGroup();
     const positionedRelativeToGroupId = parentGroup?.pcb_group_id ?? void 0;
     const positionedRelativeToBoardId = positionedRelativeToGroupId ? void 0 : this._getBoard()?.pcb_board_id ?? void 0;
-    const resolvedPcbX = this._resolvedPcbCalcOffsetX ?? (props.pcbX !== void 0 ? this._resolvePcbCoordinate(props.pcbX, "pcbX") : pcbLeftEdgeX !== void 0 ? this._resolvePcbCoordinate(pcbLeftEdgeX, "pcbX") + componentWidth / 2 : pcbRightEdgeX !== void 0 ? this._resolvePcbCoordinate(pcbRightEdgeX, "pcbX") - componentWidth / 2 : void 0);
-    const resolvedPcbY = this._resolvedPcbCalcOffsetY ?? (props.pcbY !== void 0 ? this._resolvePcbCoordinate(props.pcbY, "pcbY") : pcbTopEdgeY !== void 0 ? this._resolvePcbCoordinate(pcbTopEdgeY, "pcbY") - componentHeight / 2 : pcbBottomEdgeY !== void 0 ? this._resolvePcbCoordinate(pcbBottomEdgeY, "pcbY") + componentHeight / 2 : void 0);
+    const resolvedPcbX = this._resolvedPcbCalcOffsetX ?? (props.pcbX !== void 0 ? this._resolvePcbCoordinate(props.pcbX, "pcbX") : pcbLeftEdgeX !== void 0 ? this._resolvePcbCoordinate(pcbLeftEdgeX, "pcbX", {
+      propertyName: "pcbLeftEdgeX"
+    }) + componentWidth / 2 : pcbRightEdgeX !== void 0 ? this._resolvePcbCoordinate(pcbRightEdgeX, "pcbX", {
+      propertyName: "pcbRightEdgeX"
+    }) - componentWidth / 2 : void 0);
+    const resolvedPcbY = this._resolvedPcbCalcOffsetY ?? (props.pcbY !== void 0 ? this._resolvePcbCoordinate(props.pcbY, "pcbY") : pcbTopEdgeY !== void 0 ? this._resolvePcbCoordinate(pcbTopEdgeY, "pcbY", {
+      propertyName: "pcbTopEdgeY"
+    }) - componentHeight / 2 : pcbBottomEdgeY !== void 0 ? this._resolvePcbCoordinate(pcbBottomEdgeY, "pcbY", {
+      propertyName: "pcbBottomEdgeY"
+    }) + componentHeight / 2 : void 0);
     db.pcb_component.update(this.pcb_component_id, {
       position_mode: "relative_to_group_anchor",
       positioned_relative_to_pcb_group_id: positionedRelativeToGroupId,
@@ -13092,65 +13123,65 @@ function Group_doInitialPcbCalcPlacementResolution(group) {
       y: pcbComponent.center.y
     };
     if (rawPcbX !== void 0) {
-      const resolvedPcbX = component.resolvePcbCoordinate(rawPcbX, "pcbX", {
+      const resolvedPcbX = component.resolvePcbCoordinate({
+        rawValue: rawPcbX,
+        axis: "pcbX",
         allowComponentVariables: true,
         componentVariables: componentVars
       });
       component._resolvedPcbCalcOffsetX = resolvedPcbX;
       nextCenter.x = resolvedPcbX;
     } else if (rawPcbLeftEdgeX !== void 0) {
-      const resolvedPcbLeftEdgeX = component.resolvePcbCoordinate(
-        rawPcbLeftEdgeX,
-        "pcbX",
-        {
-          allowComponentVariables: true,
-          componentVariables: componentVars
-        }
-      );
+      const resolvedPcbLeftEdgeX = component.resolvePcbCoordinate({
+        rawValue: rawPcbLeftEdgeX,
+        axis: "pcbX",
+        allowComponentVariables: true,
+        componentVariables: componentVars,
+        propertyName: "pcbLeftEdgeX"
+      });
       const resolvedPcbX = resolvedPcbLeftEdgeX + componentWidth / 2;
       component._resolvedPcbCalcOffsetX = resolvedPcbX;
       nextCenter.x = resolvedPcbX;
     } else if (rawPcbRightEdgeX !== void 0) {
-      const resolvedPcbRightEdgeX = component.resolvePcbCoordinate(
-        rawPcbRightEdgeX,
-        "pcbX",
-        {
-          allowComponentVariables: true,
-          componentVariables: componentVars
-        }
-      );
+      const resolvedPcbRightEdgeX = component.resolvePcbCoordinate({
+        rawValue: rawPcbRightEdgeX,
+        axis: "pcbX",
+        allowComponentVariables: true,
+        componentVariables: componentVars,
+        propertyName: "pcbRightEdgeX"
+      });
       const resolvedPcbX = resolvedPcbRightEdgeX - componentWidth / 2;
       component._resolvedPcbCalcOffsetX = resolvedPcbX;
       nextCenter.x = resolvedPcbX;
     }
     if (rawPcbY !== void 0) {
-      const resolvedPcbY = component.resolvePcbCoordinate(rawPcbY, "pcbY", {
+      const resolvedPcbY = component.resolvePcbCoordinate({
+        rawValue: rawPcbY,
+        axis: "pcbY",
         allowComponentVariables: true,
         componentVariables: componentVars
       });
       component._resolvedPcbCalcOffsetY = resolvedPcbY;
       nextCenter.y = resolvedPcbY;
     } else if (rawPcbTopEdgeY !== void 0) {
-      const resolvedPcbTopEdgeY = component.resolvePcbCoordinate(
-        rawPcbTopEdgeY,
-        "pcbY",
-        {
-          allowComponentVariables: true,
-          componentVariables: componentVars
-        }
-      );
+      const resolvedPcbTopEdgeY = component.resolvePcbCoordinate({
+        rawValue: rawPcbTopEdgeY,
+        axis: "pcbY",
+        allowComponentVariables: true,
+        componentVariables: componentVars,
+        propertyName: "pcbTopEdgeY"
+      });
       const resolvedPcbY = resolvedPcbTopEdgeY - componentHeight / 2;
       component._resolvedPcbCalcOffsetY = resolvedPcbY;
       nextCenter.y = resolvedPcbY;
     } else if (rawPcbBottomEdgeY !== void 0) {
-      const resolvedPcbBottomEdgeY = component.resolvePcbCoordinate(
-        rawPcbBottomEdgeY,
-        "pcbY",
-        {
-          allowComponentVariables: true,
-          componentVariables: componentVars
-        }
-      );
+      const resolvedPcbBottomEdgeY = component.resolvePcbCoordinate({
+        rawValue: rawPcbBottomEdgeY,
+        axis: "pcbY",
+        allowComponentVariables: true,
+        componentVariables: componentVars,
+        propertyName: "pcbBottomEdgeY"
+      });
       const resolvedPcbY = resolvedPcbBottomEdgeY + componentHeight / 2;
       component._resolvedPcbCalcOffsetY = resolvedPcbY;
       nextCenter.y = resolvedPcbY;
@@ -18359,7 +18390,7 @@ import { identity as identity5 } from "transformation-matrix";
 var package_default = {
   name: "@tscircuit/core",
   type: "module",
-  version: "0.0.1104",
+  version: "0.0.1106",
   types: "dist/index.d.ts",
   main: "dist/index.js",
   module: "dist/index.js",
@@ -18391,7 +18422,7 @@ var package_default = {
     "@biomejs/biome": "^1.8.3",
     "@resvg/resvg-js": "^2.6.2",
     "@tscircuit/alphabet": "0.0.18",
-    "@tscircuit/checks": "0.0.108",
+    "@tscircuit/checks": "0.0.110",
     "@tscircuit/capacity-autorouter": "^0.0.326",
     "@tscircuit/circuit-json-util": "^0.0.82",
     "@tscircuit/common": "^0.0.20",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tscircuit/core",
   "type": "module",
-  "version": "0.0.1105",
+  "version": "0.0.1107",
   "types": "dist/index.d.ts",
   "main": "dist/index.js",
   "module": "dist/index.js",
@@ -33,7 +33,7 @@
     "@biomejs/biome": "^1.8.3",
     "@resvg/resvg-js": "^2.6.2",
     "@tscircuit/alphabet": "0.0.18",
-    "@tscircuit/checks": "0.0.108",
+    "@tscircuit/checks": "0.0.110",
     "@tscircuit/capacity-autorouter": "^0.0.326",
     "@tscircuit/circuit-json-util": "^0.0.82",
     "@tscircuit/common": "^0.0.20",