@tscircuit/core 0.0.1966 → 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

@@ -622,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. */
@@ -637,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);
@@ -705,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();
@@ -786,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");
@@ -798,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] || [];
@@ -829,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);
@@ -63783,7 +63800,7 @@ function Group_getRoutingPhasePlans(group) {
 var package_default = {
   name: "@tscircuit/core",
   type: "module",
-  version: "0.0.1965",
+  version: "0.0.1966",
   types: "dist/index.d.ts",
   main: "dist/index.js",
   module: "dist/index.js",
@@ -68649,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") {
@@ -68669,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") {

package/package.json

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