@neocompose/cli 0.89.13 → 0.89.14

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 0.89.14
+
+### Patch Changes
+
+- f1b9697: Reduce local export and initializer-checking time by sharing dependency-tracked constructor frames, avoiding eager expansion of unrelated values, and skipping whole-project placement indexing for temporary rows. Track cached placement dependencies so relevant edits still invalidate retained work.
+- f89cf70: Fix animation validation rejecting an existing override member after its inherited method signature changes. Recompile the retained body against the new signature without changing the member ID.
+
+  Fix diagnostics and completion for `root.Assets` in nested `foreach` collection expressions, including `.spec.neo` files and parenthesized receivers.
+
 ## 0.89.13
 
 ### Patch Changes

package/dist/neo-runtime.mjs

@@ -19377,7 +19377,7 @@ var init_registry = __esm({
   "../packages/manifest/schema-contract/registry.mjs"() {
     "use strict";
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
-      cliVersion: "0.89.13",
+      cliVersion: "0.89.14",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {
@@ -36221,7 +36221,7 @@ function initializerEvaluatorCacheRevision(document) {
 function initializerReadDependenciesEqual(left, right) {
   if (left === right) return true;
   if (left === null || right === null) return false;
-  return left.hasUnattributableValueRead === right.hasUnattributableValueRead && left.hasValuePlacementRead === right.hasValuePlacementRead && setsEqual(left.valueIds, right.valueIds) && setsEqual(left.localizedTextIds, right.localizedTextIds) && setsEqual(left.variantIds, right.variantIds) && setsEqual(left.containerIds, right.containerIds) && setsEqual(left.staticMemberIds, right.staticMemberIds) && left.worldQueries.equals(right.worldQueries);
+  return left.hasUnattributableValueRead === right.hasUnattributableValueRead && left.hasValuePlacementRead === right.hasValuePlacementRead && left.hasUnnamedValuePlacementRead === right.hasUnnamedValuePlacementRead && setsEqual(left.placementIds, right.placementIds) && setsEqual(left.valueIds, right.valueIds) && setsEqual(left.localizedTextIds, right.localizedTextIds) && setsEqual(left.variantIds, right.variantIds) && setsEqual(left.containerIds, right.containerIds) && setsEqual(left.staticMemberIds, right.staticMemberIds) && left.worldQueries.equals(right.worldQueries);
 }
 var InitializerReadSet;
 var init_initializer_context = __esm({
@@ -36238,6 +36238,8 @@ var init_initializer_context = __esm({
       staticMemberIds = /* @__PURE__ */ new Set();
       worldQueries = new WorldGridQueryReadSet();
       hasValuePlacementRead = false;
+      hasUnnamedValuePlacementRead = false;
+      placementIds = /* @__PURE__ */ new Set();
       hasUnattributableValueRead = false;
       /**
        * The unattributable reads by scope, for a fork of the same document to
@@ -36261,8 +36263,10 @@ var init_initializer_context = __esm({
             return;
         }
       };
-      recordValuePlacementRead = () => {
+      recordValuePlacementRead = (valueId) => {
         this.hasValuePlacementRead = true;
+        if (valueId === void 0) this.hasUnnamedValuePlacementRead = true;
+        else this.placementIds.add(valueId);
       };
     };
   }
@@ -36886,6 +36890,7 @@ var init_getter_subscriptions = __esm({
             const reads = new InitializerReadSet();
             for (const key of [
               "valueIds",
+              "placementIds",
               "localizedTextIds",
               "variantIds",
               "containerIds",
@@ -36897,6 +36902,7 @@ var init_getter_subscriptions = __esm({
             for (const read of source.reads.worldQueries)
               reads.worldQueries.add(read);
             reads.hasValuePlacementRead = source.reads.hasValuePlacementRead;
+            reads.hasUnnamedValuePlacementRead = source.reads.hasUnnamedValuePlacementRead;
             reads.hasUnattributableValueRead = source.reads.hasUnattributableValueRead;
             cloned.setReads(copy, reads);
           }
@@ -58757,7 +58763,9 @@ function materializationReadTracking(recorder) {
       for (const read of readTracking.worldQueries) {
         recorder.recordWorldQueryRead?.(read);
       }
-      if (readTracking.hasValuePlacementRead) {
+      for (const id2 of readTracking.placementIds)
+        recorder.recordValuePlacementRead(id2);
+      if (readTracking.hasUnnamedValuePlacementRead) {
         recorder.recordValuePlacementRead();
       }
       for (const containerId of readTracking.containerEntryReadIds) {
@@ -62902,6 +62910,10 @@ function createHeadlessVirtualInstanceResolver(args) {
       if (valueId !== void 0) {
         for (const active of activeReads) active.placementIds.add(valueId);
         ownReads()?.placementIds.add(valueId);
+      } else {
+        for (const active of activeReads) active.global = true;
+        const own = ownReads();
+        if (own !== void 0) own.global = true;
       }
       args.readRecorder?.recordValuePlacementRead(valueId);
     },
@@ -63139,7 +63151,9 @@ function createHeadlessVirtualInstanceResolver(args) {
       for (const id2 of reads.variantIds) external.recordVariantRead(id2);
       for (const id2 of reads.staticMemberIds)
         external.recordStaticMemberRead(id2);
-      if (reads.placementIds.size > 0) external.recordValuePlacementRead();
+      for (const id2 of reads.placementIds)
+        external.recordValuePlacementRead(id2);
+      if (reads.global) external.recordGlobalRead("virtual-instance-expansion");
       for (const reason of reads.corpusIndexReads)
         external.recordGlobalRead(reason);
     }
@@ -63334,7 +63348,8 @@ function createHeadlessVirtualInstanceResolver(args) {
       share.containerChanged
     );
     if (shared === null) return null;
-    recorder.recordValuePlacementRead();
+    for (const id2 of shared.reads.placementIds)
+      recorder.recordValuePlacementRead(id2);
     for (const id2 of shared.reads.valueIds) recorder.recordValueRead(id2);
     for (const id2 of shared.reads.containerIds)
       recorder.recordContainerMembershipRead(id2);
@@ -63369,12 +63384,10 @@ function createHeadlessVirtualInstanceResolver(args) {
   const virtualInstanceRowsForValue = (receiverValueId) => {
     const adopted = adoptSharedExpansion(receiverValueId);
     if (adopted !== null) return adopted;
-    recorder.recordValuePlacementRead();
     recorder.recordValueRead(receiverValueId);
     const readPlacement = (valueId) => {
-      for (const active of activeReads) active.placementIds.add(valueId);
-      ownReads()?.placementIds.add(valueId);
-      return placements().get(valueId) ?? null;
+      recorder.recordValuePlacementRead(valueId);
+      return rawById.has(valueId) ? placements().get(valueId) ?? null : null;
     };
     let placement = readPlacement(receiverValueId);
     const roots = [];
@@ -63411,10 +63424,19 @@ function createHeadlessVirtualInstanceResolver(args) {
       observeRoot(known.instanceRootId);
       observeOwner(valueId, true);
     }
-    if (known?.instanceRootId === valueId) return known;
+    const readKnown = (graph2) => {
+      const result = resultByRootId.get(graph2.instanceRootId);
+      if (result !== void 0) {
+        settledRows(result);
+        forwardReads(result.reads);
+      }
+      observeOwner(valueId, true);
+      return graph2;
+    };
+    if (known?.instanceRootId === valueId) return readKnown(known);
     const stored = rawById.get(valueId);
     if (known !== void 0 && (stored === void 0 || !isStoredClassResolutionRoot(stored) || isMaterializedSpine(known.locationsById.get(valueId))))
-      return known;
+      return readKnown(known);
     virtualInstanceRowsForValue(valueId);
     if (frames.length === 0) return graphByValueId.get(valueId) ?? null;
     const graph = observeSettled(valueId);
@@ -63514,7 +63536,7 @@ function createHeadlessVirtualInstanceResolver(args) {
     readRecorder: recorder,
     recordReads: (target, read) => {
       let reads = callerReads.get(target);
-      if (reads === void 0) {
+      if (reads === void 0 || reads.valueIds !== target.valueIds || reads.containerIds !== target.containerIds || reads.localizedTextIds !== target.localizedTextIds || reads.variantIds !== target.variantIds || reads.staticMemberIds !== target.staticMemberIds) {
         reads = {
           valueIds: target.valueIds,
           containerIds: target.containerIds,
@@ -63538,6 +63560,9 @@ function createHeadlessVirtualInstanceResolver(args) {
         return read();
       } finally {
         activeReads.pop();
+        for (const id2 of reads.placementIds)
+          target.recordValuePlacementRead(id2);
+        reads.placementIds.clear();
         if (reads.global)
           target.recordUnattributableValueRead("virtual-instance-expansion");
         for (const reason of reads.corpusIndexReads)
@@ -63740,9 +63765,23 @@ function sharedHeadlessResolver(overlay2) {
     containerChanged: (id2) => overlay2.changedContainerIds.has(id2)
   };
 }
-function initializerStoredClassGraph(document, rootId, member) {
+function initializerStoredClassGraph(document, rootId, member, readTracking) {
   initializerEvaluatorLookups(document);
-  return headlessResolversByLookupKey.get(initializerBehaviorCacheKey(document))?.storedClassGraph(rootId, member) ?? null;
+  const resolver = headlessResolversByLookupKey.get(
+    initializerBehaviorCacheKey(document)
+  );
+  if (resolver === void 0) return null;
+  const read = () => resolver.storedClassGraph(rootId, member);
+  return readTracking === void 0 ? read() : resolver.recordReads(readTracking, read);
+}
+function initializerContainingClassGraph(document, valueId, readTracking) {
+  initializerEvaluatorLookups(document);
+  const resolver = headlessResolversByLookupKey.get(
+    initializerBehaviorCacheKey(document)
+  );
+  if (resolver === void 0) return null;
+  const read = () => resolver.virtualInstanceGraphForValue(valueId);
+  return readTracking === void 0 ? read() : resolver.recordReads(readTracking, read);
 }
 function initializerEvaluatorLookups(document, readTracking) {
   const trackedHeadless = readTracking !== void 0 && document.databaseVM === void 0;
@@ -63997,7 +64036,9 @@ function createInitializerMaterializationBatch(authoredDocument, pendingInitiali
     },
     // Resolver placement walks record every ancestor id. Upserts below also
     // invalidate old/new ownership targets, including newly attached rows.
-    recordValuePlacementRead: () => {
+    recordValuePlacementRead: (id2) => {
+      if (id2 !== void 0) recordRead(id2);
+      else for (const entry of activeReads) entry.global = true;
     }
   };
   const forget = (entry) => {
@@ -64021,10 +64062,11 @@ function createInitializerMaterializationBatch(authoredDocument, pendingInitiali
     entry.invalidated();
   };
   const watchReads = (reads, invalidated) => {
-    const global = reads.worldQueries.size > 0 || reads.hasValuePlacementRead || reads.corpusReadReasons.size > 0;
+    const global = reads.worldQueries.size > 0 || reads.hasUnnamedValuePlacementRead || reads.corpusReadReasons.size > 0;
     const entry = {
       reads: /* @__PURE__ */ new Set([
         ...reads.valueIds,
+        ...reads.placementIds,
         ...reads.containerIds,
         ...reads.containerEntryReadIds
       ]),
@@ -123439,9 +123481,18 @@ var init_effective_value_reader = __esm({
           if (position === void 0) break;
           const parentMember = this.memberForId(position.memberId);
           if (parentMember !== null && isClassSlotMember(parentMember)) {
-            const containingRows = this.resolveGraph(parentId, parentMember);
-            if (isMaterializedSpine(this.locations.get(valueId)))
-              return containingRows;
+            const canShareFrame = this.context.saveStaticBindings === void 0 && this.context.sessionStaticBindings === void 0 && parentMember.payload !== 1 /* Partial */ && isStoredClassResolutionRoot(this.rawRow(parentId));
+            const frame = canShareFrame ? initializerContainingClassGraph(
+              this.readDocument,
+              parentId,
+              this.context.readTracking
+            ) : null;
+            const location3 = frame?.locationsById.get(valueId);
+            if (frame === null || isMaterializedSpine(location3)) {
+              const containingRows = this.resolveGraph(parentId, parentMember);
+              if (isMaterializedSpine(this.locations.get(valueId)))
+                return containingRows;
+            }
             break;
           }
           parentId = position.parentValueId;
@@ -123456,7 +123507,12 @@ var init_effective_value_reader = __esm({
           materializeInitializerValue,
           materializeMemberDefaultValue: materializeMemberDefaultValue2,
           materializedIndex: this.materializedIndex,
-          resolveClassGraph: this.context.readTracking === void 0 && this.context.saveStaticBindings === void 0 && this.context.sessionStaticBindings === void 0 ? (root2, member) => initializerStoredClassGraph(this.readDocument, root2.id, member) : void 0,
+          resolveClassGraph: this.context.saveStaticBindings === void 0 && this.context.sessionStaticBindings === void 0 ? (root2, member) => initializerStoredClassGraph(
+            this.readDocument,
+            root2.id,
+            member,
+            this.context.readTracking
+          ) : void 0,
           onResolvedListenerDefaults: (ownerId, entry) => {
             if (entry === void 0) this.listenerDefaults.delete(ownerId);
             else this.listenerDefaults.set(ownerId, entry);
@@ -144949,7 +145005,7 @@ function createScheduledConstructionReplayComparer(args) {
         get worldQueries() {
           return created.reads.worldQueries;
         },
-        recordValuePlacementRead: () => created.reads.recordValuePlacementRead(),
+        recordValuePlacementRead: (valueId) => created.reads.recordValuePlacementRead(valueId),
         recordUnattributableValueRead: (reason) => created.reads.recordUnattributableValueRead(reason)
       };
       const reader = schemaReader.withIndexedSnapshot(document2).withRowOverrides(input.replayRows, { readTracking: tracking });
@@ -163615,10 +163671,10 @@ function buildValueEmitIndexesV4(records3, manifest, localizedTextDependencies,
       storedPlacements,
       rootPathsByValueId
     ),
-    readUnstoredSlots: () => unstoredSlots ??= unstoredSlotParents(
+    unstoredSlotParent: (id2) => (unstoredSlots ??= unstoredSlotParents(
       readDocument(),
-      readCompilationProject()
-    ),
+      readCompilationProject
+    ))(id2),
     collectionBindingValueIds: /* @__PURE__ */ new Set([
       ...manifest.members.flatMap(
         (member) => member.kind === "lookup" && typeof member.collectionValueId === "string" ? [member.collectionValueId] : []
@@ -174647,7 +174703,7 @@ function slotRootPath(context, targetId) {
       id2 = edge.parentValueId;
       continue;
     }
-    const slot = context.readUnstoredSlots().get(id2);
+    const slot = context.unstoredSlotParent(id2);
     if (slot === void 0) return null;
     segments.push(slot.schemaKey);
     id2 = slot.parentId;
@@ -174655,9 +174711,6 @@ function slotRootPath(context, targetId) {
   return null;
 }
 function unstoredSlotParents(document, project) {
-  const lookups = headlessVirtualInstanceLookups(document, {
-    initializerCompiler: documentInitializerCompiler(document, project)
-  });
   const classIndex = new Map(document.classes.map((c) => [c.id, c]));
   const memberIndex = new Map(document.members.map((m) => [m.id, m]));
   const slots = /* @__PURE__ */ new Map();
@@ -174676,24 +174729,36 @@ function unstoredSlotParents(document, project) {
       });
     }
   };
-  const visited = /* @__PURE__ */ new Set();
-  for (const row of document.values) {
-    addDerived(row);
-    if (typeof row.classId !== "string") continue;
-    const graph = lookups.virtualInstanceGraphForValue(row.id);
-    if (graph === null || visited.has(graph.instanceRootId)) continue;
-    visited.add(graph.instanceRootId);
-    for (const [id2, location3] of graph.locationsById) {
-      addDerived(location3.virtualRow);
-      if (location3.parentId == null) continue;
-      if (location3.parentSegment?.kind !== "class") continue;
-      slots.set(id2, {
-        parentId: location3.parentId,
-        schemaKey: location3.parentSegment.schemaKey
-      });
+  for (const row of document.values) addDerived(row);
+  function* expandSlots() {
+    const lookups = headlessVirtualInstanceLookups(document, {
+      initializerCompiler: documentInitializerCompiler(document, project())
+    });
+    const visited = /* @__PURE__ */ new Set();
+    for (const row of document.values) {
+      if (typeof row.classId !== "string") continue;
+      const graph = lookups.virtualInstanceGraphForValue(row.id);
+      if (graph === null || visited.has(graph.instanceRootId)) continue;
+      visited.add(graph.instanceRootId);
+      for (const [id2, location3] of graph.locationsById) {
+        addDerived(location3.virtualRow);
+        if (location3.parentId == null) continue;
+        if (location3.parentSegment?.kind !== "class") continue;
+        slots.set(id2, {
+          parentId: location3.parentId,
+          schemaKey: location3.parentSegment.schemaKey
+        });
+      }
+      yield;
     }
   }
-  return slots;
+  const expansion = expandSlots();
+  return (id2) => {
+    while (!slots.has(id2)) {
+      if (expansion.next().done) break;
+    }
+    return slots.get(id2);
+  };
 }
 function memberReferenceType(context, member) {
   if (typeof member.classId === "string") {
@@ -185652,12 +185717,15 @@ function hydrateProspectiveMemberOverride(data, members2) {
   return hydrated;
 }
 function validProspectiveMember(member) {
-  if (!isMember(member)) {
-    throw new Error(
-      `Prospective animation validation received an invalid member record ${recordLabel(member)}.`
-    );
-  }
-  return member;
+  if (isMember(member)) return member;
+  const authored = readWithoutClientDerivedBodies(
+    ProjectRecordKind.Member,
+    member
+  );
+  if (isMember(authored)) return authored;
+  throw new Error(
+    `Prospective animation validation received an invalid member record ${recordLabel(member)}.`
+  );
 }
 function closeInheritedCallableSignature(member, classes, members2) {
   if (typeof member.id !== "string") return member;
@@ -186081,6 +186149,7 @@ var sourceSchemaValidation, pulledAnimationValues, animationCompilationProject,
 var init_workspace_status_core = __esm({
   "src/project-source/workspace-status-core.ts"() {
     "use strict";
+    init_neo_script_derived_bodies();
     init_identity_rewriter();
     init_initializer_replay();
     init_project_version_whole_graph_validation();
@@ -194430,6 +194499,10 @@ function expressionTokensBefore(tokens, endExclusive) {
       start = name;
       continue;
     }
+    if (depth === 0 && token.kind === "keyword" && token.text === "in" && ![".", "?."].includes(tokens[start - 1]?.text ?? "")) {
+      const opener = receiverEnclosingOpeners(tokens)[start] ?? -1;
+      if (tokens[opener - 1]?.text === "foreach") break;
+    }
     if (depth === 0 && token.text === "when" && start < endExclusive - 1 && ![".", "?.", "!", "[", "<"].includes(tokens[start + 1]?.text ?? "") && followsSwitchPattern(tokens, start))
       break;
     if (depth === 0 && (token.kind === "punctuation" && [";", "{", "}", ",", ":", "?"].includes(token.text) || // Only `?.` and `!` continue a receiver; every other operator
@@ -243713,7 +243786,7 @@ var releaseWorkspaceLock = () => void 0;
 async function main() {
   const args = parseArgs(process.argv.slice(2));
   if (args.command === "--version") {
-    console.log("0.89.13");
+    console.log("0.89.14");
     return;
   }
   if (args.command === null || args.command === "help" || args.command === "--help" || args.command === "-h") {

package/dist/test-worker.mjs

@@ -22186,6 +22186,8 @@ var InitializerReadSet = class {
   staticMemberIds = /* @__PURE__ */ new Set();
   worldQueries = new WorldGridQueryReadSet();
   hasValuePlacementRead = false;
+  hasUnnamedValuePlacementRead = false;
+  placementIds = /* @__PURE__ */ new Set();
   hasUnattributableValueRead = false;
   /**
    * The unattributable reads by scope, for a fork of the same document to
@@ -22209,8 +22211,10 @@ var InitializerReadSet = class {
         return;
     }
   };
-  recordValuePlacementRead = () => {
+  recordValuePlacementRead = (valueId) => {
     this.hasValuePlacementRead = true;
+    if (valueId === void 0) this.hasUnnamedValuePlacementRead = true;
+    else this.placementIds.add(valueId);
   };
 };
 
@@ -22815,6 +22819,7 @@ var NeoScriptGetterSubscriptions = class _NeoScriptGetterSubscriptions {
         const reads = new InitializerReadSet();
         for (const key of [
           "valueIds",
+          "placementIds",
           "localizedTextIds",
           "variantIds",
           "containerIds",
@@ -22826,6 +22831,7 @@ var NeoScriptGetterSubscriptions = class _NeoScriptGetterSubscriptions {
         for (const read of source.reads.worldQueries)
           reads.worldQueries.add(read);
         reads.hasValuePlacementRead = source.reads.hasValuePlacementRead;
+        reads.hasUnnamedValuePlacementRead = source.reads.hasUnnamedValuePlacementRead;
         reads.hasUnattributableValueRead = source.reads.hasUnattributableValueRead;
         cloned.setReads(copy, reads);
       }
@@ -46965,7 +46971,9 @@ function materializationReadTracking(recorder) {
       for (const read of readTracking.worldQueries) {
         recorder.recordWorldQueryRead?.(read);
       }
-      if (readTracking.hasValuePlacementRead) {
+      for (const id of readTracking.placementIds)
+        recorder.recordValuePlacementRead(id);
+      if (readTracking.hasUnnamedValuePlacementRead) {
         recorder.recordValuePlacementRead();
       }
       for (const containerId of readTracking.containerEntryReadIds) {
@@ -50204,6 +50212,10 @@ function createHeadlessVirtualInstanceResolver(args) {
       if (valueId !== void 0) {
         for (const active of activeReads) active.placementIds.add(valueId);
         ownReads()?.placementIds.add(valueId);
+      } else {
+        for (const active of activeReads) active.global = true;
+        const own = ownReads();
+        if (own !== void 0) own.global = true;
       }
       args.readRecorder?.recordValuePlacementRead(valueId);
     },
@@ -50441,7 +50453,9 @@ function createHeadlessVirtualInstanceResolver(args) {
       for (const id of reads.variantIds) external.recordVariantRead(id);
       for (const id of reads.staticMemberIds)
         external.recordStaticMemberRead(id);
-      if (reads.placementIds.size > 0) external.recordValuePlacementRead();
+      for (const id of reads.placementIds)
+        external.recordValuePlacementRead(id);
+      if (reads.global) external.recordGlobalRead("virtual-instance-expansion");
       for (const reason of reads.corpusIndexReads)
         external.recordGlobalRead(reason);
     }
@@ -50636,7 +50650,8 @@ function createHeadlessVirtualInstanceResolver(args) {
       share.containerChanged
     );
     if (shared === null) return null;
-    recorder.recordValuePlacementRead();
+    for (const id of shared.reads.placementIds)
+      recorder.recordValuePlacementRead(id);
     for (const id of shared.reads.valueIds) recorder.recordValueRead(id);
     for (const id of shared.reads.containerIds)
       recorder.recordContainerMembershipRead(id);
@@ -50671,12 +50686,10 @@ function createHeadlessVirtualInstanceResolver(args) {
   const virtualInstanceRowsForValue = (receiverValueId) => {
     const adopted = adoptSharedExpansion(receiverValueId);
     if (adopted !== null) return adopted;
-    recorder.recordValuePlacementRead();
     recorder.recordValueRead(receiverValueId);
     const readPlacement = (valueId) => {
-      for (const active of activeReads) active.placementIds.add(valueId);
-      ownReads()?.placementIds.add(valueId);
-      return placements().get(valueId) ?? null;
+      recorder.recordValuePlacementRead(valueId);
+      return rawById.has(valueId) ? placements().get(valueId) ?? null : null;
     };
     let placement = readPlacement(receiverValueId);
     const roots = [];
@@ -50713,10 +50726,19 @@ function createHeadlessVirtualInstanceResolver(args) {
       observeRoot(known.instanceRootId);
       observeOwner(valueId, true);
     }
-    if (known?.instanceRootId === valueId) return known;
+    const readKnown = (graph2) => {
+      const result = resultByRootId.get(graph2.instanceRootId);
+      if (result !== void 0) {
+        settledRows(result);
+        forwardReads(result.reads);
+      }
+      observeOwner(valueId, true);
+      return graph2;
+    };
+    if (known?.instanceRootId === valueId) return readKnown(known);
     const stored = rawById.get(valueId);
     if (known !== void 0 && (stored === void 0 || !isStoredClassResolutionRoot(stored) || isMaterializedSpine(known.locationsById.get(valueId))))
-      return known;
+      return readKnown(known);
     virtualInstanceRowsForValue(valueId);
     if (frames.length === 0) return graphByValueId.get(valueId) ?? null;
     const graph = observeSettled(valueId);
@@ -50816,7 +50838,7 @@ function createHeadlessVirtualInstanceResolver(args) {
     readRecorder: recorder,
     recordReads: (target, read) => {
       let reads = callerReads.get(target);
-      if (reads === void 0) {
+      if (reads === void 0 || reads.valueIds !== target.valueIds || reads.containerIds !== target.containerIds || reads.localizedTextIds !== target.localizedTextIds || reads.variantIds !== target.variantIds || reads.staticMemberIds !== target.staticMemberIds) {
         reads = {
           valueIds: target.valueIds,
           containerIds: target.containerIds,
@@ -50840,6 +50862,9 @@ function createHeadlessVirtualInstanceResolver(args) {
         return read();
       } finally {
         activeReads.pop();
+        for (const id of reads.placementIds)
+          target.recordValuePlacementRead(id);
+        reads.placementIds.clear();
         if (reads.global)
           target.recordUnattributableValueRead("virtual-instance-expansion");
         for (const reason of reads.corpusIndexReads)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@neocompose/cli",
-  "version": "0.89.13",
+  "version": "0.89.14",
   "description": "Neo Compose native project-source CLI with bidirectional sync.",
   "license": "MIT",
   "type": "module",

package/skills/neocompose-cli/SKILL.md

@@ -9,7 +9,7 @@ description: >-
   `@neocompose/cli` or `node cli/bin/neo.mjs` in the neo-compose repository.
 ---
 
-<!-- reviewed-through-cli: 0.89.13 -->
+<!-- reviewed-through-cli: 0.89.14 -->
 
 # Neo Compose CLI
 

package/skills/neocompose-cli/references/cli-development.md

@@ -143,7 +143,7 @@ The marker near the top of `SKILL.md` must exactly match the package version,
 and this quoted copy is checked too:
 
 ```html
-<!-- reviewed-through-cli: 0.89.13 -->
+<!-- reviewed-through-cli: 0.89.14 -->
 ```
 
 Keep this skill on current behavior. Upgrade steps, rollout prerequisites,

package/skills/neocompose-cli/references/neoscript/testing-and-migrations.md

@@ -28,7 +28,8 @@ production source stops the run. It skips the extra source round trip that
 `neo push --dry-run` performs.
 
 - Specs live in colocated `*.spec.neo` files that never sync. Object
-  initializers work there as in bodies.
+  initializers work there as in bodies. The editor resolves project roots in
+  loop sources, including nested `foreach` loops over `root.Assets` collections.
 - Files run in parallel on up to four CPU workers. Set `test.maxWorkers` in
   `neo.json`, pass `--maxWorkers <n>`, or `--runInBand`. Hook and test order
   within a file is preserved; fixtures and mocks stay isolated.

package/skills/neocompose-cli/references/sync/unity-export.md

@@ -7,7 +7,9 @@ and generated C# under `<generatedTypesDirectory>/Generated/`.
 `Library/NeoCompose/export.json` is written last for Unity to ingest.
 
 - Only changed files are written. Codegen is skipped when schema inputs are
-  unchanged, but content is still validated. Repeated exports, undos, and
+  unchanged, but content is still validated. Initializer checks share settled
+  dependencies and resolve independent children without expanding every sibling.
+  Repeated exports, undos, and
   reverts reuse cached results after verifying output bytes.
 - The Unity project comes from `unityConfigPath`. Without it, an interactive
   export offers to link the one config asset naming this project; scripts get
@@ -22,7 +24,9 @@ and generated C# under `<generatedTypesDirectory>/Generated/`.
   owner or one of its constructed base types.
 
 Status and export stay clean when animation defaults use initializers or
-constructor arguments select collection rows through Lookups.
+constructor arguments select collection rows through Lookups. Changing an
+override method signature keeps its member ID; animation validation recompiles
+a retained body against the new inherited signature.
 
 ## Local exports