@neocompose/cli 0.13.0 → 0.14.0

package/CHANGELOG.md

@@ -1,5 +1,42 @@
 # Changelog
 
+## [0.14.0] - 2026-07-28
+
+### Added
+
+- Stamp authored provenance on every row a materialized default produces.
+  Writing `new EyePart { Name = "RightEye" }` as a row in `EyesPart.Children`
+  copies `EyePart`'s default graph, and until now the copy had no link home:
+  a clip declared on `EyePart` addresses `EyePart`'s own class-default rows,
+  which no copy of them named, so it resolved on no nested instance at all.
+  Each copied row now carries the class-default row it came from, forwarding
+  that row's own stamp so a part three levels deep still points at the
+  original rather than an intermediate copy. Clips on nested class rows
+  resolve, at any depth, and the Unity exporter chains the stamp through a
+  placement unchanged. `neo pull` output is byte-identical either way.
+
+### Fixed
+
+- Push two sibling bare `new()` rows of a class whose default is row-backed.
+  Re-lowering a declared initializer adopted the `@id` written on it as the
+  copy's identity — which is the class-default row's id — so
+  `[new EyePart(), new EyePart()]` minted one id twice and failed with
+  `Value construction reuses identity`, and even a single such row was
+  rejected by the server as an existing value. That id is now the copy's
+  provenance instead of its identity.
+- Keep `genericBindings` and `bindingMembers` on an authored seed row across
+  the push transport. The schema commit route rebuilt each seed from a field
+  whitelist that omitted both, so an authored `NeoAnimationChildOverride` row
+  lost its concrete generic environment between the CLI and the server's own
+  re-lowering, and the push failed as a trusted-lowering mismatch.
+
+### Removed
+
+- The nested-clip-owner advisory added in 0.12.0, along with the `warnings`
+  array `neo push --json`, `neo status --json` and `neo diff --json` carried
+  it in. It described a defect that provenance now fixes, so leaving it would
+  warn on correct code.
+
 ## [0.13.0] - 2026-07-28
 
 ### Added

package/dist/neo.mjs

@@ -34559,6 +34559,7 @@ function cloneDefaultValueForMember(args) {
     );
   }
   const value = buildNewValue(args.projectId, body);
+  value.sourceValueId = sourceValue.sourceValueId ?? sourceValue.id;
   if (isMemberListBase(member) || isMemberDictionaryBase(member)) {
     stampGenericBindings(value, member, args);
   }
@@ -43823,7 +43824,8 @@ function staticValueSeedRow(value, loweredMemberId) {
           (entry) => typeof entry[1] === "string"
         )
       )
-    } : {}
+    } : {},
+    ...typeof value.sourceValueId === "string" ? { sourceValueId: value.sourceValueId } : {}
   };
 }
 function preserveReboundValue(context, currentValueId, nextValueId, binding) {
@@ -43868,7 +43870,7 @@ function lowerStaticSeed(context, member, sourceExpression, source, valueId) {
     ...localizedTexts.size === 0 ? {} : { localizedTexts: [...localizedTexts.values()] }
   };
 }
-function lowerSeedRow(context, member, sourceExpression, source, valueId, path, rows, localizedTexts, containerId, inheritedEnvironment) {
+function lowerSeedRow(context, member, sourceExpression, source, valueId, path, rows, localizedTexts, containerId, inheritedEnvironment, materialization) {
   const { expression } = annotatedValue(sourceExpression);
   const resolvedMember = resolveGenericSchemaMember(
     context,
@@ -43941,11 +43943,16 @@ function lowerSeedRow(context, member, sourceExpression, source, valueId, path,
       if (rows.has(childValueId)) {
         throw new Error(`Value construction reuses identity ${childValueId}.`);
       }
+      const projectionStamp = materializedRowStamp(
+        context,
+        materializedChild(context, materialization, schemaKey)
+      );
       rows.set(childValueId, {
         id: childValueId,
         memberId: projectedMember.id,
         value: [target.id],
-        classId: null
+        classId: null,
+        ...projectionStamp === null ? {} : { sourceValueId: projectionStamp }
       });
       body[schemaKey] = childValueId;
     }
@@ -43972,7 +43979,8 @@ function lowerSeedRow(context, member, sourceExpression, source, valueId, path,
         rows,
         localizedTexts,
         void 0,
-        environment
+        environment,
+        materializedChild(context, materialization, assignment.name)
       );
     }
     materializeRequiredDefaults(
@@ -44005,7 +44013,8 @@ function lowerSeedRow(context, member, sourceExpression, source, valueId, path,
           rows,
           localizedTexts,
           resolvedMember.listKind === "unordered" ? valueId : void 0,
-          inheritedEnvironment
+          inheritedEnvironment,
+          materializedChild(context, materialization, index)
         )
       );
       value = resolvedMember.listKind === "unordered" ? [] : ids;
@@ -44029,7 +44038,8 @@ function lowerSeedRow(context, member, sourceExpression, source, valueId, path,
         rows,
         localizedTexts,
         void 0,
-        inheritedEnvironment
+        inheritedEnvironment,
+        materializedChild(context, materialization, entry.key.value)
       );
     }
     value = body;
@@ -44066,13 +44076,15 @@ function lowerSeedRow(context, member, sourceExpression, source, valueId, path,
     value = lowered.value;
     classId = typeof lowered.classId === "string" ? lowered.classId : null;
   }
+  const sourceValueId = materializedRowStamp(context, materialization);
   const row = {
     id: valueId,
     memberId: member.id,
     value,
     classId,
     ...containerId === void 0 ? {} : { containerId },
-    ...genericBindings === void 0 ? {} : { genericBindings }
+    ...genericBindings === void 0 ? {} : { genericBindings },
+    ...sourceValueId === null ? {} : { sourceValueId }
   };
   if (rows.has(valueId)) {
     throw new Error(`Value construction reuses identity ${valueId}.`);
@@ -44101,10 +44113,43 @@ function materializeRequiredDefaults(context, member, effectiveClass, body, sour
       rows,
       localizedTexts,
       void 0,
-      environment
+      environment,
+      // The row filling this key comes from the member's inline declaration
+      // default, which is not itself a row, so it carries no stamp. Its
+      // children mirror that default's own rows and do.
+      { rowId: null, body: declaredDefaultBody(context, childMember) }
     );
   }
 }
+function materializedRowStamp(context, materialization) {
+  if (materialization === void 0) return null;
+  const rowId = materialization.rowId;
+  if (rowId === null) return null;
+  const forwarded = valueData(context, rowId)?.sourceValueId;
+  return stringOrNull(forwarded) ?? rowId;
+}
+function materializedChild(context, materialization, key) {
+  if (materialization === void 0) return void 0;
+  const rowId = defaultChildRowId(materialization.body, key);
+  if (rowId === null) return { rowId: null, body: void 0 };
+  return { rowId, body: valueData(context, rowId)?.value };
+}
+function defaultChildRowId(body, key) {
+  if (typeof key === "number") {
+    if (!Array.isArray(body)) return null;
+    return stringOrNull(body[key]);
+  }
+  if (!isObjectRecord2(body)) return null;
+  return stringOrNull(body[key]);
+}
+function declaredDefaultBody(context, member) {
+  const declared = member.defaultValue;
+  if (declared !== null && !("serverValueId" in declared)) {
+    return declared.value;
+  }
+  const base = stateData(context, "member", member.id)?.defaultValue;
+  return isObjectRecord2(base) ? base.value : void 0;
+}
 function storedSchemaEntries(context, schemaClass2) {
   const entries = /* @__PURE__ */ new Map();
   const visited = /* @__PURE__ */ new Set();
@@ -44126,10 +44171,11 @@ function ownsValueRow(member) {
   if (member.isReadOnly) return false;
   return member.kind !== "computed" && member.kind !== "function" && member.kind !== "scriptFunction";
 }
-function lowerSeedChild(context, member, expression, source, path, rows, localizedTexts, containerId, environment) {
+function lowerSeedChild(context, member, expression, source, path, rows, localizedTexts, containerId, environment, materialization) {
   const symbolId = sourceValueSymbol(context, expression);
   if (symbolId !== null) return symbolId;
-  const valueId = annotatedValue(expression).id ?? pendingNestedValueId(source, path);
+  const annotatedId = materialization === void 0 ? annotatedValue(expression).id : null;
+  const valueId = annotatedId ?? pendingNestedValueId(source, path);
   lowerSeedRow(
     context,
     member,
@@ -44140,7 +44186,8 @@ function lowerSeedChild(context, member, expression, source, path, rows, localiz
     rows,
     localizedTexts,
     containerId,
-    environment
+    environment,
+    materialization
   );
   return valueId;
 }
@@ -45231,7 +45278,8 @@ function valueData(context, id2) {
       value: pending.value,
       classId: pending.classId,
       ...pending.containerId === void 0 ? {} : { containerId: pending.containerId },
-      ...pending.genericBindings === void 0 ? {} : { genericBindings: pending.genericBindings }
+      ...pending.genericBindings === void 0 ? {} : { genericBindings: pending.genericBindings },
+      ...pending.sourceValueId === void 0 ? {} : { sourceValueId: pending.sourceValueId }
     };
   }
   return stateData(context, "value", id2);
@@ -46140,6 +46188,7 @@ function externallyReferencedValueIds(records2) {
     if (isObjectRecord2(value)) {
       for (const [childKey, child] of Object.entries(value)) {
         if (childKey === "id") continue;
+        if (childKey === "sourceValueId") continue;
         visit(child);
       }
     }
@@ -48716,24 +48765,6 @@ function assertAnimationClipDocumentValid(document) {
     context.validateClipMember(member);
   }
 }
-function collectAnimationClipNestedOwnerWarnings(document) {
-  try {
-    return new AnimationValidationContext(
-      document
-    ).collectNestedOwnerWarnings();
-  } catch {
-    return [];
-  }
-}
-function compareNestedChildrenRowUseSites(left, right) {
-  if (left.ownerClassName !== right.ownerClassName) {
-    return left.ownerClassName < right.ownerClassName ? -1 : 1;
-  }
-  if (left.rowValueId !== right.rowValueId) {
-    return left.rowValueId < right.rowValueId ? -1 : 1;
-  }
-  return 0;
-}
 function isRecord3(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
@@ -49612,188 +49643,6 @@ var init_animation_clips = __esm({
         }
         return false;
       }
-      // ---------------------------------------------------------------------------
-      // P41 §2.3 SCAFFOLDING — DELETE EVERYTHING BELOW THIS BANNER WHEN P44 LANDS,
-      // together with `collectAnimationClipNestedOwnerWarnings` and its tests. See
-      // that function's doc block for why the check exists and why P44 retires it.
-      // ---------------------------------------------------------------------------
-      collectNestedOwnerWarnings() {
-        const useSiteByClassId = this.nestedChildrenRowUseSites();
-        if (useSiteByClassId.size === 0) return [];
-        const warnings = [];
-        for (const member of this.document.members) {
-          if (!isMemberClass(member)) continue;
-          if (!this.classHasWorldKind(member.classId, "animationClip")) continue;
-          const ownerClassId2 = this.optionalClassBinding(
-            member,
-            WORLD_ANIMATION_CLIP_TARGET_PARAM_ID
-          );
-          if (ownerClassId2 === null) continue;
-          const useSite = useSiteByClassId.get(ownerClassId2);
-          if (useSite === void 0) continue;
-          if (!this.clipDeclaresChildReferences(member)) continue;
-          const ownerClassName = this.classById.get(ownerClassId2)?.name ?? ownerClassId2;
-          const nestedRowClassName = this.classById.get(useSite.rowClassId)?.name ?? useSite.rowClassId;
-          const nesting = useSite.rowClassId === ownerClassId2 ? "which is also used as a nested Children row" : `whose subclass "${nestedRowClassName}" is used as a nested Children row`;
-          warnings.push({
-            clipName: member.name,
-            ownerClassName,
-            nestedOwnerClassName: useSite.ownerClassName,
-            nestedRowClassName,
-            nestedRowValueId: useSite.rowValueId,
-            message: `Animation clip "${member.name}" overrides or tracks children of "${ownerClassName}", ${nesting} (value "${useSite.rowValueId}" in class "${useSite.ownerClassName}"). A nested instance carries re-lowered copies of those rows, so the clip resolves none of them there. Give nested Children rows authored provenance (P44), or declare the child rows through a constructor (P43).`
-          });
-        }
-        return warnings;
-      }
-      /**
-       * Every class an authored `Children` list instantiates as a row, keyed by
-       * that class and each of its ancestors — a nested row of a subclass inherits
-       * the base class's clips, so the base is affected too.
-       *
-       * Both authoring shapes count. A class-default `Children` is one; a
-       * `Children` list authored on a row inside one is the other, and it is the
-       * shape P41 §3 itself writes — `CharacterBody.Children` holds a `HeadPart`
-       * row whose own `Children` holds the hat. Scanning only class defaults would
-       * miss every slot below the first level.
-       *
-       * A layer's directly placed objects hang off `NeoObjectLayerLink.Objects`, a
-       * different member, so a directly placed class is never collected here.
-       */
-      nestedChildrenRowUseSites() {
-        const declaringClasses = /* @__PURE__ */ new Map();
-        for (const schemaClass2 of this.document.classes) {
-          for (const [schemaKey, memberId] of Object.entries(schemaClass2.schema)) {
-            if (typeof memberId !== "string") continue;
-            if (declaringClasses.has(memberId)) continue;
-            declaringClasses.set(memberId, { schemaClass: schemaClass2, schemaKey });
-          }
-        }
-        const sites = [];
-        const pending = [];
-        for (const member of this.document.members) {
-          if (!this.memberDescendsFrom(member.id, WORLD_OBJECT_CHILDREN_MEMBER_ID)) {
-            continue;
-          }
-          const declaration = declaringClasses.get(member.id);
-          if (declaration === void 0) continue;
-          const node = this.resolveDefinitionChild(
-            null,
-            declaration.schemaKey,
-            member
-          );
-          pending.push(
-            ...this.childrenRowUseSites(node, member, declaration.schemaClass.name)
-          );
-        }
-        const visited = /* @__PURE__ */ new Set();
-        for (let index = 0; index < pending.length; index += 1) {
-          const site = pending[index];
-          if (site === void 0) continue;
-          if (visited.has(site.rowValueId)) continue;
-          visited.add(site.rowValueId);
-          sites.push(site);
-          const row = this.valueById.get(site.rowValueId);
-          if (row === void 0) continue;
-          const field = this.field(
-            site.rowClassId,
-            WORLD_OBJECT_CHILDREN_MEMBER_ID
-          );
-          if (field === null) continue;
-          const rowClassName = this.classById.get(site.rowClassId)?.name ?? site.rowClassId;
-          pending.push(
-            ...this.childrenRowUseSites(
-              { ...row, value: this.listFieldIds(row, field) },
-              field.member,
-              rowClassName
-            )
-          );
-        }
-        sites.sort(compareNestedChildrenRowUseSites);
-        const useSiteByClassId = /* @__PURE__ */ new Map();
-        for (const site of sites) {
-          if (useSiteByClassId.has(site.rowClassId)) continue;
-          useSiteByClassId.set(site.rowClassId, site);
-        }
-        for (const site of sites) {
-          for (const classId of this.classAndAncestorIds(site.rowClassId)) {
-            if (useSiteByClassId.has(classId)) continue;
-            useSiteByClassId.set(classId, site);
-          }
-        }
-        return useSiteByClassId;
-      }
-      /** The rows an authored `Children` list holds, as use sites of `ownerClassName`. */
-      childrenRowUseSites(node, listMember, ownerClassName) {
-        if (!Array.isArray(node?.value)) return [];
-        const entryClassId = this.listEntryClassId(listMember);
-        const sites = [];
-        for (const rowValueId of node.value) {
-          if (typeof rowValueId !== "string") continue;
-          const row = this.valueById.get(rowValueId);
-          const rowClassId = typeof row?.classId === "string" ? row.classId : entryClassId;
-          if (rowClassId === void 0) continue;
-          sites.push({ ownerClassName, rowValueId, rowClassId });
-        }
-        return sites;
-      }
-      /** True when the clip names at least one child through a track or override. */
-      clipDeclaresChildReferences(clipMember) {
-        const clipNode = this.optionalMemberRootNode(clipMember);
-        if (clipNode === null) return false;
-        const tracks = this.optionalListRows(
-          clipNode,
-          clipMember.classId,
-          WORLD_ANIMATION_CLIP_TRACKS_MEMBER_ID
-        );
-        if (tracks !== null && tracks.length > 0) return true;
-        const frames = this.optionalListRows(
-          clipNode,
-          clipMember.classId,
-          WORLD_ANIMATION_CLIP_FRAMES_MEMBER_ID
-        ) ?? [];
-        for (const frame of frames) {
-          if (typeof frame.classId !== "string") continue;
-          const childOverrides = this.optionalListRows(
-            frame,
-            frame.classId,
-            WORLD_ANIMATION_FRAME_CHILD_OVERRIDES_MEMBER_ID
-          );
-          if (childOverrides !== null && childOverrides.length > 0) return true;
-        }
-        return false;
-      }
-      /** {@link requireListField} without the throws, for diagnostics-only readers. */
-      optionalListRows(parent, classId, memberId) {
-        const field = this.field(classId, memberId);
-        if (field === null) return null;
-        const ids = this.listFieldIds(parent, field);
-        if (!Array.isArray(ids)) return null;
-        const inferredClassId = this.listEntryClassId(field.member);
-        const rows = [];
-        for (const id2 of ids) {
-          if (typeof id2 !== "string") return null;
-          const entry = this.valueById.get(id2);
-          if (entry === void 0) return null;
-          rows.push(
-            typeof entry.classId === "string" || inferredClassId === void 0 ? entry : { ...entry, classId: inferredClassId }
-          );
-        }
-        return rows;
-      }
-      /** The class itself followed by every ancestor it extends. */
-      classAndAncestorIds(classId) {
-        const ids = [];
-        const visited = /* @__PURE__ */ new Set();
-        let current = this.classById.get(classId);
-        if (current === void 0) return [classId];
-        while (current !== void 0 && !visited.has(current.id)) {
-          visited.add(current.id);
-          ids.push(current.id);
-          current = current.extendsClassId ? this.classById.get(current.extendsClassId) : void 0;
-        }
-        return ids;
-      }
     };
   }
 });
@@ -50072,7 +49921,6 @@ function compareWorkspacePaths(left, right) {
 function computeWorkspaceStatus(workspace, options = {}) {
   const conflictedFiles = [];
   const parseErrors = [];
-  const warnings = [];
   const migrationSources = [];
   const projectSources = [];
   const sourceEntries = options.virtualSourceFiles === void 0 ? listProjectSourceFilesV4(workspace.root).map((filePath) => ({
@@ -50102,7 +49950,6 @@ function computeWorkspaceStatus(workspace, options = {}) {
       changes: [],
       conflictedFiles,
       parseErrors,
-      warnings,
       reconstructed: /* @__PURE__ */ new Map(),
       staticValueSeeds: /* @__PURE__ */ new Map(),
       binaryChanges: [],
@@ -50151,7 +49998,6 @@ function computeWorkspaceStatus(workspace, options = {}) {
         changes: [],
         conflictedFiles,
         parseErrors,
-        warnings,
         reconstructed: /* @__PURE__ */ new Map(),
         staticValueSeeds,
         binaryChanges: [],
@@ -50176,7 +50022,6 @@ function computeWorkspaceStatus(workspace, options = {}) {
         changes: [],
         conflictedFiles,
         parseErrors,
-        warnings,
         reconstructed: /* @__PURE__ */ new Map(),
         staticValueSeeds,
         binaryChanges: [],
@@ -50224,7 +50069,6 @@ function computeWorkspaceStatus(workspace, options = {}) {
       changes: [],
       conflictedFiles,
       parseErrors,
-      warnings,
       reconstructed: /* @__PURE__ */ new Map(),
       staticValueSeeds: /* @__PURE__ */ new Map(),
       binaryChanges: [],
@@ -50441,16 +50285,15 @@ function computeWorkspaceStatus(workspace, options = {}) {
   )) {
     parseErrors.push(new SchemaSourceError(message, "<project-files>", 1, 1));
   }
-  let validatedAnimationRecords = null;
   try {
-    const animationRecords = prospectiveAnimationRecords(
-      workspace.state.records,
-      reconstructed3,
-      changes,
-      staticValueSeeds
+    validateProspectiveAnimationRecordsV4(
+      prospectiveAnimationRecords(
+        workspace.state.records,
+        reconstructed3,
+        changes,
+        staticValueSeeds
+      )
     );
-    validateProspectiveAnimationRecordsV4(animationRecords);
-    validatedAnimationRecords = animationRecords;
   } catch (error) {
     parseErrors.push(
       new SchemaSourceError(
@@ -50461,11 +50304,6 @@ function computeWorkspaceStatus(workspace, options = {}) {
       )
     );
   }
-  if (validatedAnimationRecords !== null) {
-    warnings.push(
-      ...collectProspectiveAnimationWarningsV4(validatedAnimationRecords)
-    );
-  }
   for (const binary of binaryChanges) {
     if (binary.action !== "missing-local") continue;
     parseErrors.push(
@@ -50481,7 +50319,6 @@ function computeWorkspaceStatus(workspace, options = {}) {
     changes,
     conflictedFiles,
     parseErrors,
-    warnings,
     reconstructed: reconstructed3,
     staticValueSeeds,
     binaryChanges,
@@ -50493,17 +50330,6 @@ function validateProspectiveAnimationRecordsV4(records2) {
   if (document === null) return;
   assertAnimationClipDocumentValid(document);
 }
-function collectProspectiveAnimationWarningsV4(records2) {
-  try {
-    const document = prospectiveAnimationDocumentV4(records2);
-    if (document === null) return [];
-    return collectAnimationClipNestedOwnerWarnings(document).map(
-      (warning) => warning.message
-    );
-  } catch {
-    return [];
-  }
-}
 function prospectiveAnimationDocumentV4(records2) {
   const candidates = [...records2];
   const hasAnimationSchema = candidates.some(
@@ -69253,9 +69079,6 @@ ${blockingErrors.map((error) => `  ${error.message}`).join("\n")}`
     changes: status.changes,
     binaryChanges: status.binaryChanges ?? []
   });
-  if (options.json !== true) {
-    for (const message of status.warnings) warn(message);
-  }
   if (status.changes.length === 0) {
     if (options.json === true) {
       console.log(
@@ -69264,8 +69087,7 @@ ${blockingErrors.map((error) => `  ${error.message}`).join("\n")}`
           dryRun: options.dryRun,
           changeCount: 0,
           staticValueSeedCount: 0,
-          changes: [],
-          warnings: status.warnings
+          changes: []
         })
       );
     } else {
@@ -69327,8 +69149,7 @@ ${blockingErrors.map((error) => `  ${error.message}`).join("\n")}`
           kind: change.kind,
           recordKind: change.recordKind,
           recordId: change.recordId
-        })),
-        warnings: status.warnings
+        }))
       })
     );
   } else {
@@ -71532,7 +71353,6 @@ function projectStatusJsonV4(status, options) {
       blocking: isBlockingSchemaSourceError(error),
       message: error.message
     })),
-    warnings: status.warnings,
     records: status.changes.map(
       (change) => recordChangeJsonV4(change, status, options)
     ),
@@ -72211,9 +72031,6 @@ async function main() {
       for (const error of status.parseErrors) {
         console.log(`${sym.warn} ${error.message}`);
       }
-      for (const message of status.warnings) {
-        console.log(`${sym.warn} ${message}`);
-      }
       for (const group of groupProjectStatusChangesV4(status)) {
         console.log(color.bold(group.source));
         for (const change of group.changes) {

package/package.json

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