@neocompose/cli 0.24.1 → 0.24.2

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## [0.24.2] - 2026-08-07
+
+### Fixed
+
+- Compile selectors on newly authored nested track rows with the lexical class
+  that declares the clip, including idless rows, explicit UUID rows, and
+  default rows copied into an instance graph.
+- Preserve same-pass default-row provenance and initializer-backed rows while
+  assembling authored seeds, instead of losing the source owner or replacing
+  an initializer with an undefined literal.
+- Accept compiled selector delegates in trusted static-value graph validation,
+  so server preparation agrees with P60 track construction.
+
+### Changed
+
+- Clarify in the bundled CLI skill that `this` in a track selector refers to
+  the class declaring the clip, not the nested track or a receiving instance.
+
 ## [0.24.1] - 2026-08-07
 
 ### Fixed

package/dist/neo.mjs

@@ -53791,11 +53791,12 @@ function compileNSInitializer(code, ctx) {
     );
   }
   try {
+    const { lexicalThisClass, ...compilerContext } = ctx;
     return compileStrict(
       `return
 ${code};`,
       createContext(
-        { ...ctx, thisClass: null },
+        { ...compilerContext, thisClass: lexicalThisClass ?? null },
         {
           scriptKind: "initializer",
           returnTypeInfo: ctx.returnTypeInfo,
@@ -53968,6 +53969,37 @@ var init_compiler_adapter = __esm({
 });
 
 // ../src/database/compile-ns-property.ts
+function initializerReferencedIdentifiers(source) {
+  let expression;
+  try {
+    expression = parseExpression(source);
+  } catch {
+    return /* @__PURE__ */ new Set();
+  }
+  const identifiers = /* @__PURE__ */ new Set();
+  const pending = [expression];
+  const visited = /* @__PURE__ */ new Set();
+  while (pending.length > 0) {
+    const value = pending.pop();
+    if (value === null || typeof value !== "object" || visited.has(value)) {
+      continue;
+    }
+    visited.add(value);
+    if (Reflect.get(value, "kind") === "ident") {
+      const name = Reflect.get(value, "name");
+      if (typeof name === "string") identifiers.add(name);
+    }
+    for (const child of Object.values(value)) pending.push(child);
+  }
+  return identifiers;
+}
+function initializerReferencesLexicalThis(source) {
+  return initializerReferencedIdentifiers(source).has("this");
+}
+function initializerReferencesAnyIdentifier(source, names) {
+  const identifiers = initializerReferencedIdentifiers(source);
+  return [...names].some((name) => identifiers.has(name));
+}
 function compileNSPropertyBodies(args) {
   if (args.member.kind !== 10 /* NSProperty */) return;
   if (isMemberNSPropertyContractBase(args.member)) return;
@@ -54221,6 +54253,7 @@ function compileValueRowInitializerBody(args) {
         constructors: args.constructors ?? [],
         returnTypeInfo,
         initializerName: args.member.name,
+        lexicalThisClass: args.lexicalThisClass ?? null,
         argumentTypes: initializerArgumentTypes(
           args.initializerOwnerClass,
           args.constructors ?? []
@@ -54775,6 +54808,7 @@ var init_compile_ns_property = __esm({
     init_lookup_declared_type();
     init_compiler_adapter();
     init_compiler_adapter();
+    init_src();
     NeoScriptBodyCompileError = class extends Error {
       memberId;
       memberName;
@@ -54797,6 +54831,41 @@ var init_compile_ns_property = __esm({
 });
 
 // ../src/database/value-row-owner-members.ts
+function valueIdsWithSourceChains(targetValueIds, valuesById) {
+  const result = new Set(targetValueIds);
+  for (const targetId of targetValueIds) {
+    const visited = /* @__PURE__ */ new Set();
+    let currentId = targetId;
+    while (currentId !== null && !visited.has(currentId)) {
+      visited.add(currentId);
+      result.add(currentId);
+      const sourceValueId = valuesById.get(currentId)?.sourceValueId;
+      currentId = typeof sourceValueId === "string" ? sourceValueId : null;
+    }
+  }
+  return result;
+}
+function initializerOwnerContext(document, valueId, rootOwners, valuesById) {
+  const visited = /* @__PURE__ */ new Set();
+  let currentId = valueId;
+  let lexicalRoot = rootOwners.get(valueId);
+  while (currentId !== null && !visited.has(currentId)) {
+    visited.add(currentId);
+    const sourceValueId = valuesById.get(currentId)?.sourceValueId;
+    if (typeof sourceValueId !== "string") break;
+    currentId = sourceValueId;
+    lexicalRoot = rootOwners.get(sourceValueId) ?? lexicalRoot;
+  }
+  if (lexicalRoot === void 0) {
+    return { ownerClass: null, lexicalThisClass: null };
+  }
+  const lexicalRootId = getField(lexicalRoot, "id");
+  const ownerClass = typeof lexicalRootId === "string" ? findSchemaPlacement(lexicalRootId, document.classes)?.ownerClass ?? null : null;
+  return {
+    ownerClass,
+    lexicalThisClass: lexicalRoot.isStatic === true ? null : ownerClass
+  };
+}
 function isRecordValue(value) {
   if (value === null) return false;
   if (typeof value !== "object") return false;
@@ -63335,7 +63404,11 @@ function evaluateMemberInitializer(args) {
     __constructedArgumentsByValue: /* @__PURE__ */ new WeakMap(),
     __createdStorageKeyDeclarations: args.storageKeyDeclarations ?? /* @__PURE__ */ new Map()
   };
-  const result = evaluateNSGetterWithEffects(compiled, ctx);
+  const result = evaluateNSGetterWithEffects(
+    compiled,
+    ctx,
+    args.argumentValues ?? []
+  );
   const created = result.createdSessionValues ?? [];
   if (isMemberLookupBase(args.member)) {
     return encodeLookupInitializerResult(
@@ -63390,7 +63463,8 @@ function evaluateInitializerMaterialization(args) {
     document: args.document,
     createdValues,
     storageKeyDeclarations,
-    ...args.storedConstructionReplay === true ? { storedConstructionReplay: true } : {}
+    ...args.storedConstructionReplay === true ? { storedConstructionReplay: true } : {},
+    ...args.argumentValues === void 0 ? {} : { argumentValues: args.argumentValues }
   });
   return { evaluated, createdValues, storageKeyDeclarations };
 }
@@ -63399,7 +63473,8 @@ function materializeInitializerValue(args) {
     init: args.row.init,
     member: args.member,
     document: args.document,
-    ...args.storedConstructionReplay === true ? { storedConstructionReplay: true } : {}
+    ...args.storedConstructionReplay === true ? { storedConstructionReplay: true } : {},
+    ...args.argumentValues === void 0 ? {} : { argumentValues: args.argumentValues }
   });
   const {
     init: _init,
@@ -63414,6 +63489,9 @@ function materializeInitializerValue(args) {
     ...evaluated.constructorArgs === void 0 ? {} : { constructorArgs: structuredClone(evaluated.constructorArgs) }
   };
   const allCreated = [root, ...createdValues];
+  for (const created of createdValues) {
+    if (created.classId === void 0) delete created.classId;
+  }
   stampCreatedValuesMapKey(allCreated, args.row.mapKey);
   storageKeyDeclarations.delete(root.id);
   applyDeclaredStorageKeyOverrides({
@@ -77011,6 +77089,12 @@ var init_project_version_static_value_writes = __esm({
           this.validateDialogueLookup(member, value, path);
           return;
         }
+        if (isMemberDelegateBase(member)) {
+          if (!isNSDelegateValue(value.value)) {
+            throw new Error(`${path} must store a valid NeoDelegate value.`);
+          }
+          return;
+        }
         if (isMemberClassBase(member)) {
           this.validateClass({ ...args, member, value });
           return;
@@ -80201,17 +80285,21 @@ function prepareServerOwnedValueInitializerBodies(args) {
     ...[...explicitRows.values()].map((row) => row.id),
     ...sweepRows.map((row) => row.id)
   ]);
+  const valuesById = new Map(
+    committedDocument.values.map((value) => [value.id, value])
+  );
+  const ownershipTargetIds = valueIdsWithSourceChains(targetIds, valuesById);
   const rootOwnerByValueId = /* @__PURE__ */ new Map();
   const ownerByValueId = resolveOwnerMembersForValues(
     committedDocument,
-    targetIds,
+    ownershipTargetIds,
     void 0,
     rootOwnerByValueId
   );
   const currentValueById = new Map(
     args.document.values.map((value) => [value.id, value])
   );
-  const compileOne = (document, row) => {
+  const compileOne = (document, row, documentValuesById) => {
     const member = ownerByValueId.get(row.id);
     if (member === void 0) {
       throw new Error(
@@ -80220,6 +80308,12 @@ function prepareServerOwnedValueInitializerBodies(args) {
     }
     const currentValue = currentValueById.get(row.id);
     const replaysStoredConstruction = isLiteralValueContent(currentValue) && (typeof currentValue.classId === "string" || currentValue.constructorArgs !== void 0);
+    const ownerContext = initializerOwnerContext(
+      document,
+      row.id,
+      rootOwnerByValueId,
+      documentValuesById
+    );
     compileValueRowInitializerBody({
       project: document.project,
       projectFiles: document.projectFiles,
@@ -80231,16 +80325,14 @@ function prepareServerOwnedValueInitializerBodies(args) {
       member,
       valueRow: row,
       valueId: row.id,
-      initializerOwnerClass: findSchemaPlacement(
-        rootOwnerByValueId.get(row.id)?.id ?? "",
-        document.classes
-      )?.ownerClass ?? null,
+      initializerOwnerClass: ownerContext.ownerClass,
+      lexicalThisClass: initializerReferencesLexicalThis(row.init.code) ? ownerContext.lexicalThisClass : null,
       ...replaysStoredConstruction ? { storedConstructionReplay: true } : {}
     });
   };
   for (const [index, row] of explicitRows) {
     const compiled = { ...row, init: { ...row.init } };
-    compileOne(committedDocument, compiled);
+    compileOne(committedDocument, compiled, valuesById);
     const change = args.prepared[index];
     if (change === void 0) continue;
     args.prepared[index] = { ...change, nextData: compiled };
@@ -80255,14 +80347,14 @@ function prepareServerOwnedValueInitializerBodies(args) {
   for (const row of sweepRows) {
     const compiled = { ...row, init: { ...row.init } };
     try {
-      compileOne(committedDocument, compiled);
+      compileOne(committedDocument, compiled, valuesById);
     } catch (postWriteError) {
       const current2 = currentById.get(row.id);
       if (current2 === void 0) throw postWriteError;
       const preWrite = initBackedValueRow(structuredClone(current2));
       if (preWrite === null) throw postWriteError;
       try {
-        compileOne(args.document, preWrite);
+        compileOne(args.document, preWrite, currentValueById);
       } catch (preWriteError) {
         if (sameCompileFailure(preWriteError, postWriteError)) continue;
       }
@@ -82833,9 +82925,16 @@ function replayAnimationDeclarationInitializersV4(records2, document, fallbackDo
   if (initializers.length === 0) return document;
   const rootKinds = /* @__PURE__ */ new Map();
   const rootOwners = /* @__PURE__ */ new Map();
+  const valuesById = new Map(
+    document.values.map((value) => [value.id, value])
+  );
+  const ownershipTargetIds = valueIdsWithSourceChains(
+    new Set(initializers.map(({ row }) => row.id)),
+    valuesById
+  );
   const owners = resolveOwnerMembersForValues(
     document,
-    new Set(initializers.map(({ row }) => row.id)),
+    ownershipTargetIds,
     void 0,
     rootOwners,
     rootKinds
@@ -82879,10 +82978,24 @@ function replayAnimationDeclarationInitializersV4(records2, document, fallbackDo
     if (!classBelongsToAnimationFamily(document.classes, declaredClassId)) {
       continue;
     }
-    const rootOwner = rootOwners.get(row.id);
-    const rootOwnerId = Reflect.get(rootOwner ?? {}, "id");
-    const initializerOwnerClass = typeof rootOwnerId !== "string" ? null : findSchemaPlacement(rootOwnerId, document.classes)?.ownerClass ?? null;
-    const evaluate = initializerOwnerClass?.requiredConstructorId === void 0 || initializerOwnerClass.requiredConstructorId === null;
+    const fallback = fallbackValues.get(row.id);
+    const ownerContext = initializerOwnerContext(
+      document,
+      row.id,
+      rootOwners,
+      valuesById
+    );
+    const initializerOwnerClass = ownerContext.ownerClass;
+    const requiredConstructor = initializerOwnerClass?.requiredConstructorId === void 0 || initializerOwnerClass.requiredConstructorId === null ? void 0 : (replayDocument.constructors ?? []).find(
+      (constructor2) => constructor2.id === initializerOwnerClass.requiredConstructorId
+    );
+    const requiredArgumentNames = new Set(
+      requiredConstructor?.argumentTypes.map((argument2) => argument2.name) ?? []
+    );
+    const evaluate = !initializerReferencesAnyIdentifier(
+      code,
+      requiredArgumentNames
+    );
     let replay;
     try {
       replay = replayStoredConstructionV4({
@@ -82894,9 +83007,15 @@ function replayAnimationDeclarationInitializersV4(records2, document, fallbackDo
         compileDocumentBodies: !documentBodiesCompiled,
         compilationProject,
         initializerOwnerClass,
-        // A parameterized declaration is a template. Its arguments exist only
-        // at concrete construction sites, so validate the authored body here
-        // without fabricating values for the class header parameters.
+        lexicalThisClass: ownerContext.lexicalThisClass,
+        ...requiredConstructor === void 0 || !evaluate ? {} : {
+          initializerArgumentValues: requiredConstructor.argumentTypes.map(
+            () => null
+          )
+        },
+        // A parameterized declaration is a template. Replay it only when this
+        // row does not read those parameters; null placeholders satisfy the
+        // unused compiler envelope without inventing authored values.
         evaluate
       });
       documentBodiesCompiled = true;
@@ -82907,7 +83026,6 @@ function replayAnimationDeclarationInitializersV4(records2, document, fallbackDo
       );
     }
     if (!evaluate) {
-      const fallback = fallbackValues.get(row.id);
       if (fallback !== void 0) values.set(row.id, fallback);
     }
     for (const [id2, value] of replay) {
@@ -83292,7 +83410,6 @@ var init_workspace_status_core = __esm({
     init_project_documents();
     init_animation_clips();
     init_classes();
-    init_inheritance();
     init_members();
     init_value_row_owner_members();
     init_project2();
@@ -83300,6 +83417,7 @@ var init_workspace_status_core = __esm({
     init_push_change_intent();
     init_initializer_replay();
     init_compiler_adapter();
+    init_compile_ns_property();
     init_materialized_construction_cache();
     init_project_version_whole_graph_validation();
   }
@@ -84164,6 +84282,7 @@ function replayStoredConstructionV4(args) {
     // Instance calls are self-contained. Declaration calls inherit the class
     // header parameters that are in lexical scope at their source site.
     initializerOwnerClass: args.initializerOwnerClass ?? null,
+    lexicalThisClass: initializerReferencesLexicalThis(args.code) ? args.lexicalThisClass ?? null : null,
     storedConstructionReplay: true,
     ...compilationProject === void 0 ? {} : { compilationProject }
   });
@@ -84185,7 +84304,8 @@ function replayStoredConstructionV4(args) {
         // explicit typed descriptor for evaluation that compiled the replay.
         member: compileMember,
         row: candidate,
-        storedConstructionReplay: true
+        storedConstructionReplay: true,
+        ...args.initializerArgumentValues === void 0 ? {} : { argumentValues: args.initializerArgumentValues }
       });
       break;
     } catch (error) {
@@ -84248,10 +84368,17 @@ function valueInitializerCompilationSites(document) {
   const cached = pulledValueInitializerCompilationSites.get(document);
   if (cached !== void 0) return cached;
   const rows = document.values.filter(isInitValueContent);
+  const valuesById = new Map(
+    document.values.map((value) => [value.id, value])
+  );
+  const ownershipTargetIds = valueIdsWithSourceChains(
+    new Set(rows.map((row) => row.id)),
+    valuesById
+  );
   const rootOwners = /* @__PURE__ */ new Map();
   const owners = resolveOwnerMembersForValues(
     document,
-    new Set(rows.map((row) => row.id)),
+    ownershipTargetIds,
     void 0,
     rootOwners
   );
@@ -84259,11 +84386,17 @@ function valueInitializerCompilationSites(document) {
   for (const row of rows) {
     const member = owners.get(row.id);
     if (member === void 0) continue;
-    const rootOwnerId = Reflect.get(rootOwners.get(row.id) ?? {}, "id");
+    const ownerContext = initializerOwnerContext(
+      document,
+      row.id,
+      rootOwners,
+      valuesById
+    );
     sites.set(row.id, {
       row,
       member,
-      ownerClass: typeof rootOwnerId !== "string" ? null : findSchemaPlacement(rootOwnerId, document.classes)?.ownerClass ?? null
+      ownerClass: ownerContext.ownerClass,
+      lexicalThisClass: ownerContext.lexicalThisClass
     });
   }
   pulledValueInitializerCompilationSites.set(document, sites);
@@ -84288,6 +84421,7 @@ function compilePulledValueInitializerBodyV4(document, valueId, compilationProje
     valueRow: site.row,
     valueId: String(Reflect.get(site.row, "id")),
     initializerOwnerClass: site.ownerClass,
+    lexicalThisClass: site.lexicalThisClass,
     storedConstructionReplay: true,
     ...compilationProject === void 0 ? {} : { compilationProject }
   });
@@ -84444,7 +84578,6 @@ var init_initializer_replay = __esm({
     init_compile_ns_property();
     init_init_backed_value_materialization();
     init_value_row_owner_members();
-    init_inheritance();
     init_project_document_read();
     init_localization2();
     init_server_preparation_preflight();
@@ -85888,11 +86021,9 @@ function staticValueSeedRow(value, loweredMemberId) {
   if (loweredMemberId === void 0) {
     throw new Error(`Authored value row ${id2} is missing memberId.`);
   }
-  return {
+  const fields = {
     id: id2,
     memberId: loweredMemberId,
-    value: value.value,
-    classId: stringOrNull(value.classId),
     ...typeof value.containerId === "string" ? { containerId: value.containerId } : {},
     ...isObjectRecord2(value.genericBindings) ? {
       genericBindings: Object.fromEntries(
@@ -85903,6 +86034,15 @@ function staticValueSeedRow(value, loweredMemberId) {
     } : {},
     ...typeof value.sourceValueId === "string" ? { sourceValueId: value.sourceValueId } : {}
   };
+  const init = isObjectRecord2(value.init) ? value.init : null;
+  if (init !== null && typeof init.code === "string") {
+    return { ...fields, init: { code: init.code } };
+  }
+  return {
+    ...fields,
+    value: value.value,
+    classId: stringOrNull(value.classId)
+  };
 }
 function preserveReboundValue(context, currentValueId, nextValueId, binding) {
   if (currentValueId === null || currentValueId === nextValueId) return;
@@ -86300,14 +86440,24 @@ 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;
+  const forwarded = materializationSourceValueData(
+    context,
+    rowId
+  )?.sourceValueId;
   return stringOrNull(forwarded) ?? rowId;
 }
+function materializationSourceValueData(context, valueId) {
+  if (context.pendingValues.has(valueId)) return valueData(context, valueId);
+  return context.reconstructed.get(`value:${valueId}`)?.fileFields ?? valueData(context, valueId);
+}
 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 };
+  return {
+    rowId,
+    body: materializationSourceValueData(context, rowId)?.value
+  };
 }
 function defaultChildRowId(body, key) {
   if (typeof key === "number") {
@@ -101396,7 +101546,7 @@ var init_registry2 = __esm({
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
       formatVersion: 3,
       contractVersion: "3.9",
-      cliVersion: "0.24.1",
+      cliVersion: "0.24.2",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@neocompose/cli",
-  "version": "0.24.1",
+  "version": "0.24.2",
   "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.24.1 -->
+<!-- reviewed-through-cli: 0.24.2 -->
 
 # Neo Compose CLI
 

package/skills/neocompose-cli/references/animation-and-world-authoring.md

@@ -178,6 +178,11 @@ row-ID coupling selectors were introduced to remove. For new track rows, omit
 `@id` and let the successful push assign it. Preserve IDs already present on
 pulled rows.
 
+A selector argument such as `this.SelectPants` is evaluated in the lexical
+scope of the class that declares the clip. Here `this` is the `LegPart`, not
+the nested track row or an outer object instance that later receives a copy of
+the declaration default.
+
 Each track has `StartFrame`, `Direction` (`.Forward` or `.Reverse`), and a
 crop window `OffsetStartIndex`/`OffsetEndIndex`. Crop before reversing, then
 schedule on the owning clip. Content beyond the owning clip truncates. Reject

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

@@ -82,7 +82,7 @@ wrappers.
 The marker near the top of `SKILL.md` must exactly match the package version:
 
 ```html
-<!-- reviewed-through-cli: 0.24.1 -->
+<!-- reviewed-through-cli: 0.24.2 -->
 ```
 
 The quoted version above is checked too, so this instruction cannot go stale