@aelionsdk/render-ir 1.2.0-rc.5 → 1.2.0

package/README.md

@@ -6,10 +6,10 @@ evaluation for AelionSDK.
 ## Install
 
 ```bash
-npm install @aelionsdk/render-ir@next
+npm install @aelionsdk/render-ir
 ```
 
-`next` currently resolves to `1.2.0-rc.5`. Product applications should use
+`latest` currently resolves to `1.2.0`. Product applications should use
 `@aelionsdk/sdk`; direct use is for custom renderers, exporters and engine
 instrumentation.
 

package/dist/compiler.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"compiler.d.ts","sourceRoot":"","sources":["../src/compiler.ts"],"names":[],"mappings":"AAAA,OAAO,EAAsB,KAAK,aAAa,EAAmB,MAAM,2BAA2B,CAAC;AAGpG,OAAO,KAAK,EAaV,mBAAmB,EACnB,oBAAoB,EACrB,MAAM,YAAY,CAAC;AAscpB,qBAAa,yBAAyB;;IAIpC;;;;OAIG;IACI,IAAI,IAAI,yBAAyB;IAMxC,4EAA4E;IACrE,KAAK,IAAI,IAAI;IAOb,OAAO,CACZ,OAAO,EAAE,aAAa,EACtB,UAAU,EAAE,MAAM,EAClB,QAAQ,EAAE,MAAM,EAChB,uBAAuB,GACnB,oBAAoB,GACpB,mBAAmB,CAAC,OAAO,CAAC,CAAC,gBAAgB,CAAM,GACtD,mBAAmB;CAwOvB"}
+{"version":3,"file":"compiler.d.ts","sourceRoot":"","sources":["../src/compiler.ts"],"names":[],"mappings":"AAAA,OAAO,EAEL,KAAK,aAAa,EAGnB,MAAM,2BAA2B,CAAC;AAGnC,OAAO,KAAK,EAaV,mBAAmB,EACnB,oBAAoB,EACrB,MAAM,YAAY,CAAC;AAsepB,qBAAa,yBAAyB;;IAcpC;;;;OAIG;IACI,IAAI,IAAI,yBAAyB;IAcxC,4EAA4E;IACrE,KAAK,IAAI,IAAI;IAOb,OAAO,CACZ,OAAO,EAAE,aAAa,EACtB,UAAU,EAAE,MAAM,EAClB,QAAQ,EAAE,MAAM,EAChB,uBAAuB,GACnB,oBAAoB,GACpB,mBAAmB,CAAC,OAAO,CAAC,CAAC,gBAAgB,CAAM,GACtD,mBAAmB;CAoOvB"}

package/dist/compiler.js

@@ -1,7 +1,35 @@
-import { canonicalStringify } from '@aelionsdk/project-schema';
+import { canonicalStringify, } from '@aelionsdk/project-schema';
+/**
+ * Objects `deepFreezePlain` has already frozen, along with everything beneath
+ * them.
+ *
+ * An incremental compile reuses most of the previous IR verbatim, and those
+ * objects were deep-frozen by the compile that produced them. Re-walking them
+ * dominated incremental compile time -- roughly three quarters of a no-op
+ * compile at a thousand clips. Membership means the whole subtree is already
+ * frozen, so the walk can stop at the first reused node.
+ *
+ * Entries are held weakly, so a released IR is still collectable.
+ */
+const deepFrozen = new WeakSet();
+/** Item types the Render IR can compile, as a set so the check is one lookup. */
+const COMPILABLE_ITEM_TYPES = new Set([
+    'video',
+    'image',
+    'audio',
+    'text',
+    'caption',
+    'nested-sequence',
+    'generator',
+    'shape',
+    'material-content',
+    'adjustment',
+]);
 function deepFreezePlain(value, seen = new WeakSet()) {
     if (value === null || typeof value !== 'object')
         return value;
+    if (deepFrozen.has(value))
+        return value;
     if (seen.has(value))
         return value;
     const prototype = Object.getPrototypeOf(value);
@@ -10,6 +38,9 @@ function deepFreezePlain(value, seen = new WeakSet()) {
     seen.add(value);
     for (const entry of Object.values(value))
         deepFreezePlain(entry, seen);
+    // Recorded only after the whole subtree is frozen, so membership never
+    // promises more than has actually been done.
+    deepFrozen.add(value);
     return Object.freeze(value);
 }
 function object(value, context) {
@@ -416,6 +447,16 @@ function contentDuration(project, trackIds) {
 export class IncrementalRenderCompiler {
     #previous;
     #compiling = false;
+    /**
+     * Track fingerprints keyed by the source track object.
+     *
+     * A track fingerprint is `canonicalStringify(track)`, and a track carries every
+     * item id on it, so recomputing it for a thousand-clip track dominated the cost
+     * of an incremental compile even when nothing on that track changed. Commits
+     * share structure: editing one item leaves every untouched track at the same
+     * object identity, so identity is a sound key and an edited track simply misses.
+     */
+    #trackFingerprints = new WeakMap();
     /**
      * Creates an isolated compiler that reuses this compiler's immutable baseline.
      * Compiling on the fork cannot advance or corrupt the parent baseline; a host
@@ -426,6 +467,14 @@ export class IncrementalRenderCompiler {
         fork.#previous = this.#previous;
         return fork;
     }
+    #trackFingerprint(track) {
+        const cached = this.#trackFingerprints.get(track);
+        if (cached !== undefined)
+            return cached;
+        const fingerprint = canonicalStringify(track);
+        this.#trackFingerprints.set(track, fingerprint);
+        return fingerprint;
+    }
     /** Releases the incremental baseline retained for clip/transition reuse. */
     clear() {
         if (this.#compiling) {
@@ -465,36 +514,34 @@ export class IncrementalRenderCompiler {
                 const track = project.tracks[trackId];
                 if (track === undefined)
                     throw new RangeError(`Track ${trackId} does not exist`);
-                const clips = track.itemIds.flatMap(itemId => {
+                // A plain loop rather than `flatMap`: the reuse path runs once per item on
+                // every commit, and wrapping each result in a throwaway single-element array
+                // was the largest remaining cost of an incremental compile.
+                const clips = [];
+                for (const itemId of track.itemIds) {
                     const item = project.items[itemId];
                     if (item === undefined)
                         throw new RangeError(`Item ${itemId} does not exist`);
-                    if (item.type !== 'video' &&
-                        item.type !== 'image' &&
-                        item.type !== 'audio' &&
-                        item.type !== 'text' &&
-                        item.type !== 'caption' &&
-                        item.type !== 'nested-sequence' &&
-                        item.type !== 'generator' &&
-                        item.type !== 'shape' &&
-                        item.type !== 'material-content' &&
-                        item.type !== 'adjustment')
+                    if (!COMPILABLE_ITEM_TYPES.has(item.type)) {
                         throw new TypeError(`Render IR cannot compile item type ${item.type}`);
+                    }
                     const previous = previousClips.get(itemId);
                     if (canReuseByEntity &&
                         previous !== undefined &&
                         !previous.dependencyEntityIds.some(id => affectedEntityIds.has(id))) {
                         reusedClips += 1;
-                        return [previous];
+                        clips.push(previous);
+                        continue;
                     }
                     const candidate = compileClip(item, materials, project.assets);
                     if (previous?.fingerprint === candidate.fingerprint) {
                         reusedClips += 1;
-                        return [previous];
+                        clips.push(previous);
+                        continue;
                     }
                     compiledClips += 1;
-                    return [candidate];
-                });
+                    clips.push(candidate);
+                }
                 return {
                     id: track.id,
                     kind: track.kind,
@@ -506,7 +553,7 @@ export class IncrementalRenderCompiler {
                         : {}),
                     clips,
                     materialInstanceIds: [...track.materialInstanceIds],
-                    fingerprint: canonicalStringify(track),
+                    fingerprint: this.#trackFingerprint(track),
                 };
             });
             const transitions = sequence.transitionIds.map(id => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aelionsdk/render-ir",
-  "version": "1.2.0-rc.5",
+  "version": "1.2.0",
   "description": "Incremental render intermediate representation compiler for AelionSDK",
   "license": "MIT",
   "repository": {
@@ -34,9 +34,9 @@
     "provenance": true
   },
   "dependencies": {
-    "@aelionsdk/core": "1.2.0-rc.5",
-    "@aelionsdk/material-compiler": "1.2.0-rc.5",
-    "@aelionsdk/project-schema": "1.2.0-rc.5"
+    "@aelionsdk/core": "1.2.0",
+    "@aelionsdk/material-compiler": "1.2.0",
+    "@aelionsdk/project-schema": "1.2.0"
   },
   "scripts": {
     "build": "tsc -b",