@neocompose/cli 0.89.12 → 0.89.14

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);
       }
@@ -37246,6 +37252,13 @@ function applyConstructorFields(args) {
       let kept = 0;
       for (const row of createdValues) {
         if (row.id === keptId || row.containerId === keptId) {
+          if (row.containerId === keptId) {
+            if (ctx.__indexes !== void 0)
+              unindexEvaluatorRow(ctx.__indexes, row);
+            delete row.containerId;
+            if (ctx.__indexes !== void 0)
+              indexEvaluatorRow(ctx.__indexes, row);
+          }
           createdById.delete(row.id);
           continue;
         }
@@ -40459,6 +40472,9 @@ function materializeConstructorArgumentValue(args) {
       for (const entry of rows) sourceEntries.push(entry.value);
     }
     const ids = [];
+    const authoredEntries = args.ctx.__constructionEntryIds === void 0 ? void 0 : [];
+    if (authoredEntries !== void 0)
+      args.ctx.__constructionEntryIds?.set(row.id, authoredEntries);
     const references = recordedReferenceEntries(sourceEntries, args.ctx) ?? (args.collectionEntriesAreReferences === false ? 1 /* Values */ : collectionReferences(sourceEntries, args.ctx));
     for (const [index, rawEntry] of sourceEntries.entries()) {
       if (args.collectionOwnerId !== void 0 && args.collectionEntriesAreReferences !== false && isReferenceEntry(references, index) && typeof rawEntry === "string" && evalValueById(
@@ -40468,6 +40484,7 @@ function materializeConstructorArgumentValue(args) {
         args.ctx.__valueOverlay
       ) !== null) {
         ids.push(rawEntry);
+        authoredEntries?.push(rawEntry);
         continue;
       }
       const entryValue = constructorCollectionEntryValue(
@@ -40489,6 +40506,7 @@ function materializeConstructorArgumentValue(args) {
         },
         destinationName: `${args.member.name}[${index}]`
       });
+      authoredEntries?.push(child.id);
       if (listKindOf(args.member) === 1 /* Unordered */) {
         child.containerId = row.id;
       } else {
@@ -43306,7 +43324,16 @@ var MaterializedValueGraphContext = class _MaterializedValueGraphContext {
   ownershipFacts() {
     if (this.ownership !== void 0) return this.ownership;
     const targetValueIds = new Set(this.valuesById.keys());
-    const owners = resolveOwnerMembersForValues(this.document, targetValueIds);
+    const owners = resolveOwnerMembersForValues(
+      this.document,
+      targetValueIds,
+      [...this.contextualRootMembersByValueId ?? []].map(
+        ([valueId, member]) => ({
+          valueId,
+          member: resolveMember(member, this.document.members)
+        })
+      )
+    );
     this.ownership = {
       owners,
       independentlyOwnedValueIds: new Set(owners.keys())
@@ -46238,6 +46265,7 @@ function evaluateInitializerMaterialization(args) {
   const createdValues = [];
   const storageKeyDeclarations = /* @__PURE__ */ new Map();
   const evaluated = args.evaluateInitializer({
+    constructionEntryIds: args.constructionEntryIds,
     randomSource: args.randomSource,
     genericEnv: args.genericEnv,
     init: args.init,
@@ -46317,7 +46345,9 @@ function finalizeMaterializedRoot(args) {
   });
 }
 function materializeInitializerValue(args) {
+  const constructionEntryIds = args.captureConstructionEntries ? /* @__PURE__ */ new Map() : void 0;
   const { evaluated, createdValues, storageKeyDeclarations } = evaluateInitializerMaterialization({
+    constructionEntryIds,
     randomSource: args.randomSource,
     evaluateInitializer: args.evaluateInitializer,
     init: args.row.init,
@@ -46369,6 +46399,7 @@ function materializeInitializerValue(args) {
   return {
     root,
     createdValues,
+    constructionEntryIds,
     snapshot: evaluated.snapshot,
     pinnedRootSchemaKeys: evaluated.pinnedRootSchemaKeys ?? /* @__PURE__ */ new Set(),
     pinnedSchemaKeysByValueId: evaluated.pinnedSchemaKeysByValueId ?? /* @__PURE__ */ new Map()
@@ -46838,7 +46869,7 @@ function buildVirtualInstanceMaterializedIndex(args) {
   return {
     rows,
     rowsById,
-    graph: new MaterializedValueGraphContext(
+    graph: args.materializedGraph ?? new MaterializedValueGraphContext(
       { ...args.document, values: rows },
       rowsById
     ),
@@ -46940,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) {
@@ -47314,7 +47347,9 @@ function resolveVirtualInstanceGraph(args) {
       if (parent !== void 0 && isMemberEntryCollectionBase(parent.member) && listKindOf(parent.member) === 1 /* Unordered */)
         effective.containerId = parentId;
     }
-    if (isClassSlotMember(indexed.member) && effective.value !== null) {
+    if (args.opaquePaths?.has(pathKey)) {
+      effective.value = materialized?.value ?? indexed.expandedRow.value;
+    } else if (isClassSlotMember(indexed.member) && effective.value !== null) {
       const storedRecord = stringRecord(materialized?.value);
       const classId = rowClassId(indexed.member, effective);
       const constructedChildren = materialized === null ? /* @__PURE__ */ new Map() : settledAggregateChildren(materialized, classId, pathKey);
@@ -48648,6 +48683,7 @@ function indexExpansion(args) {
     discoveredNodes?.push(node);
     nodesByPath.set(pathKey, node);
     appendChildPath(parentPath, pathKey);
+    if (args.opaquePaths?.has(pathKey)) return;
     if (isClassTypedMember(member)) {
       const record4 = stringRecord(row.value);
       const classId = row.classId ?? member.classId;
@@ -50176,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);
     },
@@ -50413,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);
     }
@@ -50608,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);
@@ -50643,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 = [];
@@ -50685,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);
@@ -50788,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,
@@ -50812,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)
@@ -51152,6 +51205,7 @@ function evaluateMemberInitializer(args) {
   );
   const compiled = compileMemberInitializer(args, databaseVM);
   const ctx = {
+    __constructionEntryIds: args.constructionEntryIds,
     randomSource: args.randomSource,
     randomIdentity,
     ...args.sessionValues === void 0 ? {} : { __runtimeSessionValues: args.sessionValues },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@neocompose/cli",
-  "version": "0.89.12",
+  "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.12 -->
+<!-- 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.12 -->
+<!-- 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
 

package/skills/neocompose-cli/references/values/identities.md

@@ -63,6 +63,13 @@ Quest HardAsStone = new Quest() { Name = "Hard as stone" };
 form above. Never put an ID in both places. A local keeps its concrete type
 when a base-typed member consumes it.
 
+Before its first push, a named aggregate in an owning graph reserves its
+identity by its terminal local declaration. Inserting or reordering other list
+entries keeps that value and its descendants stable across local exports.
+Aliases share the terminal declaration's identity. Scalar locals remain reusable
+values, so each owning occurrence gets its own identity. Keep the assigned
+`@id` when renaming or moving a declaration after push.
+
 ## Locals as constructor arguments
 
 In constructor arguments and executable operands, a class local names its one