@tscircuit/core 0.0.1965 → 0.0.1967

package/dist/index.d.ts

@@ -358,6 +358,7 @@ interface DebugLogOutputEvent {
 }
 
 interface IIsolatedCircuit {
+    _hasRenderLifecycleListeners?: boolean;
     hasEventListener?(event: RootCircuitEventName): boolean;
     emit(event: RootCircuitEventName, ...args: any[]): void;
     on(event: RootCircuitEventName, listener: (...args: any[]) => void): void;
@@ -395,7 +396,10 @@ type IRenderable = RenderPhaseFunctions & {
     runRenderCycle(): void;
 };
 declare abstract class Renderable implements IRenderable {
-    renderPhaseStates: RenderPhaseStates;
+    private _phaseStatesByName?;
+    private readonly _phaseStatesByIndex;
+    get renderPhaseStates(): RenderPhaseStates;
+    set renderPhaseStates(states: RenderPhaseStates);
     shouldBeRemoved: boolean;
     children: IRenderable[];
     /** PCB-only SMTPads, PlatedHoles, Holes, Silkscreen elements etc. */
@@ -595,6 +599,7 @@ declare class IsolatedCircuit {
     selectOne(selector: string, opts?: {
         type?: "component" | "port";
     }): PrimitiveComponent | null;
+    _hasRenderLifecycleListeners: boolean;
     _eventListeners: Record<RootCircuitEventName, Array<(...args: any[]) => void>>;
     emit(event: RootCircuitEventName, ...args: any[]): void;
     hasEventListener(event: RootCircuitEventName): boolean;

package/dist/index.js

@@ -389,6 +389,17 @@ var mergeRoutes = (routes) => {
 // lib/components/primitive-components/Net.ts
 import { netProps } from "@tscircuit/props";
 
+// lib/components/base-components/PrimitiveComponent/get-construction-props-schema.ts
+var constructionSchemas = /* @__PURE__ */ new WeakMap();
+var getConstructionPropsSchema = (schema) => {
+  let constructionSchema = constructionSchemas.get(schema);
+  if (!constructionSchema) {
+    constructionSchema = "partial" in schema ? schema.partial({ name: true }) : schema;
+    constructionSchemas.set(schema, constructionSchema);
+  }
+  return constructionSchema;
+};
+
 // lib/components/base-components/PrimitiveComponent/select-net-by-literal-selector.ts
 var netsBySelectorByNets = /* @__PURE__ */ new WeakMap();
 var selectNetByLiteralSelector = (nets, selector) => {
@@ -611,7 +622,21 @@ var asyncPhaseDependencies = {
 var globalRenderCounter = 0;
 var globalAsyncEffectCounter = 0;
 var Renderable = class _Renderable {
-  renderPhaseStates;
+  _phaseStatesByName;
+  _phaseStatesByIndex;
+  // Build the named map once, sharing the state objects used by the render loop.
+  // Once exposed, use the map so callers can also replace individual entries.
+  get renderPhaseStates() {
+    return this._phaseStatesByName ??= Object.fromEntries(
+      orderedRenderPhases.map((phase, index) => [
+        phase,
+        this._phaseStatesByIndex[index]
+      ])
+    );
+  }
+  set renderPhaseStates(states) {
+    this._phaseStatesByName = states;
+  }
   shouldBeRemoved = false;
   children;
   /** PCB-only SMTPads, PlatedHoles, Holes, Silkscreen elements etc. */
@@ -626,19 +651,19 @@ var Renderable = class _Renderable {
   constructor(props) {
     this._renderId = `${globalRenderCounter++}`;
     this.children = [];
-    this.renderPhaseStates = {};
-    for (const phase of orderedRenderPhases) {
-      this.renderPhaseStates[phase] = {
+    this._phaseStatesByIndex = Array.from(
+      { length: orderedRenderPhases.length },
+      () => ({
         initialized: false,
         dirty: false
-      };
-    }
+      })
+    );
   }
   _markDirty(phase) {
-    this.renderPhaseStates[phase].dirty = true;
     const phaseIndex = renderPhaseIndexMap.get(phase);
-    for (let i = phaseIndex + 1; i < orderedRenderPhases.length; i++) {
-      this.renderPhaseStates[orderedRenderPhases[i]].dirty = true;
+    for (let i = phaseIndex; i < orderedRenderPhases.length; i++) {
+      const state = this._phaseStatesByName?.[orderedRenderPhases[i]] ?? this._phaseStatesByIndex[i];
+      state.dirty = true;
     }
     if (this.parent?._markDirty) {
       this.parent._markDirty(phase);
@@ -694,8 +719,9 @@ ${error.stack}`
     });
   }
   _emitRenderLifecycleEvent(phase, startOrEnd) {
-    const granular_event_type = `renderable:renderLifecycle:${phase}:${startOrEnd}`;
     const root = this._getRootCircuit();
+    if (root?._hasRenderLifecycleListeners === false && !debug.enabled) return;
+    const granular_event_type = `renderable:renderLifecycle:${phase}:${startOrEnd}`;
     const hasListeners = root && (!root.hasEventListener || root.hasEventListener(granular_event_type) || root.hasEventListener("renderable:renderLifecycle:anyEvent"));
     if (!debug.enabled && !hasListeners) return;
     const componentDisplayName = this.getString();
@@ -775,9 +801,9 @@ ${error.stack}`
    */
   runRenderPhase(phase) {
     this._currentRenderPhase = phase;
-    const phaseState = this.renderPhaseStates[phase];
-    const isInitialized = phaseState.initialized;
-    const isDirty = phaseState.dirty;
+    const phaseIndex = renderPhaseIndexMap.get(phase);
+    const phaseState = this._phaseStatesByName?.[phase] ?? this._phaseStatesByIndex[phaseIndex];
+    const { initialized: isInitialized, dirty: isDirty } = phaseState;
     if (!isInitialized && this.shouldBeRemoved) return;
     if (this.shouldBeRemoved && isInitialized) {
       this._emitRenderLifecycleEvent(phase, "start");
@@ -787,10 +813,12 @@ ${error.stack}`
       this._emitRenderLifecycleEvent(phase, "end");
       return;
     }
-    const prevPhaseIndex = renderPhaseIndexMap.get(phase) - 1;
-    if (prevPhaseIndex >= 0) {
+    const prevPhaseIndex = phaseIndex - 1;
+    if (this._asyncEffects.length > 0 && prevPhaseIndex >= 0) {
       const prevPhase = orderedRenderPhases[prevPhaseIndex];
-      const hasIncompleteEffects = this._asyncEffects.filter((e) => e.phase === prevPhase).some((e) => !e.complete);
+      const hasIncompleteEffects = this._asyncEffects.some(
+        (e) => e.phase === prevPhase && !e.complete
+      );
       if (hasIncompleteEffects) return;
     }
     const deps = asyncPhaseDependencies[phase] || [];
@@ -818,7 +846,7 @@ ${error.stack}`
   }
   runRenderPhaseForChildren(phase) {
     for (const child of this.children) {
-      if ("_isIsolatedSubcircuit" in this && this._isIsolatedSubcircuit && "doInitialRenderIsolatedSubcircuits" in this && phase === "RenderIsolatedSubcircuits") {
+      if (phase === "RenderIsolatedSubcircuits" && "_isIsolatedSubcircuit" in this && this._isIsolatedSubcircuit && "doInitialRenderIsolatedSubcircuits" in this) {
         continue;
       }
       child.runRenderPhaseForChildren(phase);
@@ -1575,9 +1603,7 @@ var PrimitiveComponent2 = class extends Renderable {
     this.childrenPendingRemoval = [];
     this.props = props ?? {};
     this.externallyAddedAliases = [];
-    const zodProps = "partial" in this.config.zodProps ? this.config.zodProps.partial({
-      name: true
-    }) : this.config.zodProps;
+    const zodProps = getConstructionPropsSchema(this.config.zodProps);
     const parsePropsResult = zodProps.safeParse(props ?? {});
     if (parsePropsResult.success) {
       this._parsedProps = parsePropsResult.data;
@@ -63774,7 +63800,7 @@ function Group_getRoutingPhasePlans(group) {
 var package_default = {
   name: "@tscircuit/core",
   type: "module",
-  version: "0.0.1964",
+  version: "0.0.1966",
   types: "dist/index.d.ts",
   main: "dist/index.js",
   module: "dist/index.js",
@@ -68640,6 +68666,7 @@ var IsolatedCircuit = class {
     this._guessRootComponent();
     return this.firstChild?.selectOne(selector, opts) ?? null;
   }
+  _hasRenderLifecycleListeners = false;
   _eventListeners = {};
   emit(event, ...args) {
     if (event === "asyncEffect:start") {
@@ -68660,12 +68687,22 @@ var IsolatedCircuit = class {
       this._eventListeners[event] = [];
     }
     this._eventListeners[event].push(listener);
+    if (event.startsWith("renderable:renderLifecycle:")) {
+      this._hasRenderLifecycleListeners = true;
+    }
   }
   removeListener(event, listener) {
     if (!this._eventListeners[event]) return;
     this._eventListeners[event] = this._eventListeners[event].filter(
       (l) => l !== listener
     );
+    if (event.startsWith("renderable:renderLifecycle:")) {
+      this._hasRenderLifecycleListeners = Object.entries(
+        this._eventListeners
+      ).some(
+        ([eventName, listeners]) => eventName.startsWith("renderable:renderLifecycle:") && listeners.length > 0
+      );
+    }
   }
   enableDebug(debug13) {
     if (typeof debug13 === "string") {
@@ -69444,6 +69481,7 @@ var findFloatingCopper = (circuitJson, generatedStitchingViaIds = /* @__PURE__ *
 var Board_doInitialPcbCopperPourCleanup = (board) => {
   if (!board.root || board.root.pcbDisabled) return;
   const { db } = board.root;
+  if (db.pcb_copper_pour.list().length === 0) return;
   const circuitJson = db.subtree({ subcircuit_id: board.subcircuit_id }).toArray();
   if (db.pcb_autorouting_error.list().length > 0) return;
   if (!circuitJson.some((element) => element.type === "pcb_copper_pour")) return;
@@ -70087,6 +70125,10 @@ var Board = class extends Group5 {
     const pcbDisabled = this.root?.pcbDisabled;
     const schematicDisabled = this.root?.schematicDisabled;
     const drcChecksDisabled = this.root?.platform?.drcChecksDisabled ?? this.getInheritedProperty("drcChecksDisabled");
+    if (drcChecksDisabled) {
+      this._drcChecksComplete = true;
+      return;
+    }
     const netlistDrcChecksDisabled = this.root?.platform?.netlistDrcChecksDisabled ?? this.getInheritedProperty("netlistDrcChecksDisabled");
     const pinSpecificationDrcChecksDisabled = this.getInheritedProperty(
       "pinSpecificationDrcChecksDisabled"

package/package.json

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