@tscircuit/core 0.0.50 → 0.0.51

package/dist/index.d.ts

@@ -78,7 +78,7 @@ declare abstract class Renderable implements IRenderable {
      */
     runRenderPhase(phase: RenderPhase): void;
     runRenderPhaseForChildren(phase: RenderPhase): void;
-    renderError(message: string | PCBTraceError | PCBPlacementError): void;
+    renderError(message: string | Omit<PCBTraceError, "pcb_error_id"> | Omit<PCBPlacementError, "pcb_error_id">): void;
 }
 
 interface BaseComponentConfig {
@@ -229,6 +229,7 @@ declare abstract class PrimitiveComponent<ZodProps extends ZodType = any> extend
     }): PrimitiveComponent | null;
     getAvailablePcbLayers(): string[];
     getDescendants(): PrimitiveComponent[];
+    renderError(message: Parameters<typeof Renderable.prototype.renderError>[0]): void;
     getString(): string;
     get [Symbol.toStringTag](): string;
 }

package/dist/index.js

@@ -125,6 +125,7 @@ var Renderable = class {
     if (typeof message === "string") {
       throw new Error(message);
     }
+    throw new Error(JSON.stringify(message, null, 2));
   }
 };
 
@@ -487,6 +488,15 @@ var PrimitiveComponent = class extends Renderable {
     }
     return descendants;
   }
+  // TODO we shouldn't need to override this, errors can be rendered and handled
+  // by the Renderable class, however, the Renderable class currently doesn't
+  // have access to the database or cleanup
+  renderError(message) {
+    if (typeof message === "string") {
+      return super.renderError(message);
+    }
+    this.root?.db.pcb_error.insert(message);
+  }
   getString() {
     const { lowercaseComponentName: cname, _parsedProps: props, parent } = this;
     if (parent?.props?.name && props?.name) {
@@ -2495,9 +2505,16 @@ searched component ${targetComponent.getString()}, which has ports: ${targetComp
       });
       const traces = ijump.solveAndMapToTraces();
       if (traces.length === 0) {
-        this.renderError(
-          `Could not find a route between ${JSON.stringify(a)} and ${JSON.stringify(b)} for ${this}`
-        );
+        this.renderError({
+          type: "pcb_error",
+          error_type: "pcb_trace_error",
+          message: `Could not find a route for ${this}`,
+          source_trace_id: this.source_trace_id,
+          center: { x: (a.x + b.x) / 2, y: (a.y + b.y) / 2 },
+          pcb_port_ids: ports.map((p) => p.pcb_port_id),
+          pcb_trace_id: this.pcb_trace_id,
+          pcb_component_ids: ports.map((p) => p.pcb_component_id)
+        });
         return;
       }
       const [trace] = traces;

package/lib/components/base-components/PrimitiveComponent.ts

@@ -442,6 +442,17 @@ export abstract class PrimitiveComponent<
     return descendants
   }
 
+  // TODO we shouldn't need to override this, errors can be rendered and handled
+  // by the Renderable class, however, the Renderable class currently doesn't
+  // have access to the database or cleanup
+  renderError(message: Parameters<typeof Renderable.prototype.renderError>[0]) {
+    if (typeof message === "string") {
+      return super.renderError(message)
+    }
+    // TODO this needs to be cleaned up at some point!
+    this.root?.db.pcb_error.insert(message)
+  }
+
   getString(): string {
     const { lowercaseComponentName: cname, _parsedProps: props, parent } = this
     if (parent?.props?.name && props?.name) {

package/lib/components/base-components/Renderable.ts

@@ -110,11 +110,17 @@ export abstract class Renderable implements IRenderable {
     }
   }
 
-  renderError(message: string | PCBTraceError | PCBPlacementError) {
+  renderError(
+    message:
+      | string
+      | Omit<PCBTraceError, "pcb_error_id">
+      | Omit<PCBPlacementError, "pcb_error_id">,
+  ) {
     // TODO add to render phase error list and try to add position or
     // relationships etc.
     if (typeof message === "string") {
       throw new Error(message)
     }
+    throw new Error(JSON.stringify(message, null, 2))
   }
 }

package/lib/components/primitive-components/Trace.ts

@@ -439,9 +439,16 @@ export class Trace extends PrimitiveComponent<typeof traceProps> {
       })
       const traces = ijump.solveAndMapToTraces()
       if (traces.length === 0) {
-        this.renderError(
-          `Could not find a route between ${JSON.stringify(a)} and ${JSON.stringify(b)} for ${this}`,
-        )
+        this.renderError({
+          type: "pcb_error",
+          error_type: "pcb_trace_error",
+          message: `Could not find a route for ${this}`,
+          source_trace_id: this.source_trace_id!,
+          center: { x: (a.x + b.x) / 2, y: (a.y + b.y) / 2 },
+          pcb_port_ids: ports.map((p) => p.pcb_port_id!),
+          pcb_trace_id: this.pcb_trace_id!,
+          pcb_component_ids: ports.map((p) => p.pcb_component_id!),
+        })
         return
       }
       // TODO ijump returns multiple traces for some reason

package/package.json

@@ -2,7 +2,7 @@
   "name": "@tscircuit/core",
   "module": "index.ts",
   "type": "module",
-  "version": "0.0.50",
+  "version": "0.0.51",
   "types": "dist/index.d.ts",
   "main": "dist/index.js",
   "files": [