@neocompose/cli 0.50.5 → 0.50.7

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [0.50.7] - 2026-09-14
+
+- Preserve virtual default fields when cloning classes or importing constructor arguments. Fix `neo pull` failing with `Missing key 'Body' on object` for CharacterBody instances that clone a sparse cosmetic config. Index expanded graphs once to keep descendant reads linear.
+
+## [0.50.6] - 2026-09-14
+
+- Match server preparation when adding world placements by loading their layer links and binding values without expanding unchanged placement siblings.
+
 ## [0.50.5] - 2026-09-14
 
 - Resolve push references, rewrite assigned identities, and cascade scripted member edits without rescanning the project for each changed record. Reuse animation replay indexes and constructor validation across roots. Preserve list membership indexes when an initializer only replaces entry content.

package/dist/neo.mjs

@@ -76470,7 +76470,13 @@ function virtualExpandedRecordForReceiver(receiver, ctx) {
   if (rowId === null) return null;
   const rows = ctx.vm.databaseVM?.virtualInstanceRowsForValue(rowId);
   if (rows === void 0 || rows.length === 0) return null;
-  const expanded = rows.find((row) => row.id === rowId);
+  ctx.__virtualRowsById ??= /* @__PURE__ */ new WeakMap();
+  let rowsById = ctx.__virtualRowsById.get(rows);
+  if (rowsById === void 0) {
+    rowsById = new Map(rows.map((row) => [row.id, row]));
+    ctx.__virtualRowsById.set(rows, rowsById);
+  }
+  const expanded = rowsById.get(rowId);
   if (expanded === void 0) return null;
   const record4 = expanded.value;
   if (typeof record4 !== "object" || record4 === null || Array.isArray(record4)) {
@@ -79022,7 +79028,7 @@ function evaluateInitializerInContext(init, member, ctx, createdValues, argument
   if (isMemberLookupBase(member)) {
     return encodeLookupInitializerResult(member, result.value, created, ctx);
   }
-  const rootRow = initializerRootRow(result.value, created);
+  let rootRow = initializerRootRow(result.value, created);
   for (const row of created) {
     if (row === rootRow) continue;
     createdValues.push(row);
@@ -79038,11 +79044,36 @@ function evaluateInitializerInContext(init, member, ctx, createdValues, argument
         existingValueRow
       };
     }
-    return { value: result.value, classId: existingRuntimeClassId };
+    if (existingValueRow !== null && existingRuntimeClassId !== null && isMemberClassBase(member)) {
+      const storage = constructorArgumentStorage(existingValueRow.id, ctx);
+      if (storage !== 3 /* Session */) {
+        const checkpoint = sessionCreationCheckpoint(ctx);
+        const value = cloneClassValueGraph(
+          result.value,
+          existingRuntimeClassId,
+          ctx
+        );
+        const importedRows = createdSessionValuesSince(ctx, checkpoint);
+        const imported = initializerRootRow(value, importedRows);
+        if (imported === null) {
+          throw new NSGetterRuntimeError(
+            `Initializer '${member.name}' imported a Class without a backing row.`
+          );
+        }
+        rootRow = imported;
+        result = { ...result, value };
+        for (const row of importedRows) {
+          if (row !== rootRow) createdValues.push(row);
+        }
+      }
+    }
+    if (rootRow === null) {
+      return { value: result.value, classId: existingRuntimeClassId };
+    }
   }
   deleteRuntimeSessionValue(ctx, rootRow.id);
   ctx.__executionState?.constructorGroups.delete(rootRow.id);
-  const constructorArgs = typeof result.value === "object" && result.value !== null ? ctx.__constructedArgumentsByValue.get(result.value) : void 0;
+  const constructorArgs = typeof result.value === "object" && result.value !== null ? ctx.__constructedArgumentsByValue.get(result.value) ?? rootRow.constructorArgs ?? void 0 : void 0;
   const provenance = pickInstanceProvenance({
     constructorArgs,
     instanceConstructorId: rootRow.instanceConstructorId,
@@ -80420,7 +80451,11 @@ function cloneConstructorArgumentGraph(args) {
       );
       const record4 = {};
       clone.value = record4;
-      for (const [schemaKey, childId] of Object.entries(source.value)) {
+      const sourceRecord = {
+        ...virtualExpandedRecordForReceiver(source.value, args.ctx),
+        ...source.value
+      };
+      for (const [schemaKey, childId] of Object.entries(sourceRecord)) {
         if (typeof childId !== "string") {
           throw new NSGetterRuntimeError(
             `Constructor import found a non-id Class field '${schemaKey}' on value '${source.id}'.`
@@ -80979,10 +81014,12 @@ function cloneClassValueGraph(receiver, expectedClassId, ctx) {
     registerRuntimeSessionValue(ctx, clone);
     clonedBySourceId.set(sourceRow.id, clone);
     if (typeof sourceRow.value === "object" && sourceRow.value !== null && !Array.isArray(sourceRow.value)) {
-      const sourceRecord = sourceRow.value;
-      const clonedRecord = { ...sourceRecord };
+      const clonedRecord = {
+        ...virtualExpandedRecordForReceiver(sourceRow.value, ctx),
+        ...sourceRow.value
+      };
       clone.value = clonedRecord;
-      for (const [key, childId] of Object.entries(sourceRecord)) {
+      for (const [key, childId] of Object.entries(clonedRecord)) {
         if (typeof childId !== "string") continue;
         const childMember = sourceTypeInfo?.type === 5 /* Dictionary */ ? null : ownedObjectChildMember(sourceRow, sourceMember, key, ctx);
         const childTypeInfo = sourceTypeInfo?.type === 5 /* Dictionary */ ? sourceTypeInfo.entryTypeInfo : void 0;
@@ -95528,6 +95565,16 @@ var init_project_version_relation_sweep_trigger = __esm({
 });
 
 // ../src/database/project-world-placement-bindings.ts
+function createdWorldPlacementIds(changes) {
+  if (changes.length === 0) return null;
+  const ids = /* @__PURE__ */ new Set();
+  for (const change of changes) {
+    if (change.recordKind !== "value" || change.operation !== "create" || change.deleted === true || !hasWorldPlacementSidecar(change.nextData))
+      return null;
+    ids.add(change.recordId);
+  }
+  return ids;
+}
 function hasWorldPlacementSidecar(value) {
   if (!isPlainRecord3(value) || !isPlainRecord3(value.value)) return false;
   const body = value.value;
@@ -96050,6 +96097,8 @@ function validateWorldContentSidecars(args) {
         valuesById
       });
       for (const placementId of placementIds) {
+        if (args.placementIds !== void 0 && !args.placementIds.has(placementId))
+          continue;
         const placement = valuesById.get(placementId);
         if (placement === void 0) {
           throw new Error(
@@ -96112,6 +96161,8 @@ function validateWorldContentSidecars(args) {
       placementList,
       containedIds
     )) {
+      if (args.placementIds !== void 0 && !args.placementIds.has(placementId))
+        continue;
       const placement = valuesById.get(placementId);
       if (placement === void 0) {
         throw new Error(
@@ -96133,6 +96184,8 @@ function validateWorldContentSidecars(args) {
     }
   }
   for (const value of args.values) {
+    if (args.placementIds !== void 0 && !args.placementIds.has(value.id))
+      continue;
     const record4 = valueObjectRecord(value);
     if (record4 === null) continue;
     if (optionalNonEmptyString(record4.assetClassId) !== null && !validatedPlacementIds.has(value.id) && !reachablePlacementIds.has(value.id)) {
@@ -103977,7 +104030,7 @@ function* prepareServerOwnedSchemaCommitPasses(args) {
     (change) => change.recordKind === "value"
   );
   const hasAdditionalContractImpact = (args.additionalImpactedIds?.size ?? 0) > 0 || (args.additionalChangedValueIds?.size ?? 0) > 0;
-  if (!hasSchemaChange && !hasMigrationChange && !hasDialogueSourceChange && !hasValueChange && !hasAdditionalContractImpact) {
+  if (!hasSchemaChange && !hasMigrationChange && !hasDialogueSourceChange && !hasValueChange && !hasAdditionalContractImpact && (args.authoredValueSeeds?.length ?? 0) === 0) {
     return authoredChanges;
   }
   args.passLifecycle?.("start", "neo-script-recompile-targets");
@@ -108765,6 +108818,7 @@ function assertStagedWorldContentLayerBindingsValid(current, projected, changes)
     classes,
     relations,
     values: projected.values,
+    placementIds: createdWorldPlacementIds(changes) ?? void 0,
     variants: worldPlacementVariantBindings({
       variants: projected.variants ?? [],
       variantFolders: projected.variantFolders ?? [],
@@ -112461,6 +112515,8 @@ function worldKindByClassId(document) {
   return result;
 }
 function commitPreparationWorldScopeKinds(args) {
+  if (createdWorldPlacementIds(args.changes) !== null)
+    return /* @__PURE__ */ new Set(["tileGrid", "tileLayerLink", "objectLayerLink"]);
   const allKinds = new Set(
     [...args.currentDocument.classes, ...args.postDocument.classes].flatMap(
       (schemaClass2) => typeof schemaClass2.system?.worldKind === "string" ? [schemaClass2.system.worldKind] : []
@@ -112612,7 +112668,10 @@ function planCommitPreparationInitialScope(args) {
     }
     if (change.recordKind !== "localized-text") {
       for (const valueId of collectStoredRecordIds(change.nextData))
-        rememberExact("value", valueId, true);
+        rememberExact("value", valueId, change.recordKind !== "value");
+      if (change.recordKind === "value")
+        for (const valueId of collectStoredDescendantRecordIds(change.nextData))
+          rememberExact("value", valueId, true);
     }
   }
   const currentTextsById = new Map(
@@ -113105,10 +113164,27 @@ function planCommitPreparationLoadedClosureScope(args) {
         });
       }
     }
+    if (args.placementCreationScope && record4.recordKind === "value" && record4.data !== null && typeof record4.data === "object") {
+      const body = Reflect.get(record4.data, "value");
+      if (body !== null && typeof body === "object") {
+        for (const key of [
+          "Children",
+          "Tiles",
+          "Objects",
+          "layerOverrideValueId"
+        ])
+          if (typeof Reflect.get(body, key) === "string")
+            rememberExact("value", Reflect.get(body, key));
+      }
+    }
     if (record4.includeDescendants !== false) {
       for (const valueId of collectStoredRecordIds(record4.data)) {
-        rememberDescendant("value", valueId);
+        if (record4.recordKind === "value") rememberExact("value", valueId);
+        else rememberDescendant("value", valueId);
       }
+      if (record4.recordKind === "value")
+        for (const valueId of collectStoredDescendantRecordIds(record4.data))
+          rememberDescendant("value", valueId);
     }
   }
   return {
@@ -113551,6 +113627,14 @@ function projectDocumentRecordsByKey(document) {
   add("value", document.values ?? []);
   return result;
 }
+function collectStoredDescendantRecordIds(value) {
+  if (value === null || typeof value !== "object" || Array.isArray(value))
+    return collectStoredRecordIds(value);
+  const descendants = [];
+  for (const [key, entry] of Object.entries(value))
+    if (key !== "containerId") descendants.push(entry);
+  return collectStoredRecordIds(descendants);
+}
 function collectStoredRecordIds(value) {
   const result = /* @__PURE__ */ new Set();
   const visit = (candidate) => {
@@ -114003,7 +114087,8 @@ function closeInMemoryLoadedScope(args) {
       records: closureRecords,
       rangeLimit: COMMIT_PREPARATION_SCOPE_RANGE_ROW_LIMIT,
       postingLimit: COMMIT_PREPARATION_SCOPE_POSTING_ROW_LIMIT,
-      animationSchema
+      animationSchema,
+      placementCreationScope: createdWorldPlacementIds(args.prepared) !== null
     });
     const answer = readCommitPreparationScopeInMemory({
       request,
@@ -114165,6 +114250,7 @@ var init_server_preparation_preflight = __esm({
     init_trusted_commit_verification();
     init_project_version_schema_commit();
     init_project_version_whole_graph_validation();
+    init_project_world_placement_bindings();
     init_projectRecordBuckets();
     init_compile_ns_property();
     init_valueHeadPlacementGraph();
@@ -133193,7 +133279,7 @@ var init_registry2 = __esm({
   "schema-contract/registry.mjs"() {
     "use strict";
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
-      cliVersion: "0.50.5",
+      cliVersion: "0.50.7",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {
@@ -140409,7 +140495,7 @@ There is no --keep-current: preserving current semantic state defines flatten.
 async function main() {
   const args = parseArgs(process.argv.slice(2));
   if (args.command === "--version") {
-    console.log("0.50.5");
+    console.log("0.50.7");
     return;
   }
   if (args.command === null || args.command === "help" || args.command === "--help" || args.command === "-h") {

package/package.json

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

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

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