@neocompose/cli 0.56.0 → 0.56.1

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## [0.56.1] - 2026-09-23
+
+- Changing a member that a variant's `initialize` or `apply` closure uses, such as its storage, recompiles the closure. The closure used to keep the code compiled against the old member, so after a member changed from `.Session` to `.Writable`, a variant that writes it failed in the game with "Member '…' is not Session-owned." To repair closures an earlier push left stale, add `--force-recompile` to your next push.
+
 ## [0.56.0] - 2026-09-23
 
 - `@storage(allowed: .Writable)` declares a member that is always writable at runtime and persists wherever its owner persists: Save under a Save parent, Session otherwise. NeoScript can write it through any receiver, including one read from Assets, and animation clips can target it. `neo pull` writes it back in the same form. Earlier CLIs fail on a project that uses it, so upgrade before pulling.

package/dist/neo.mjs

@@ -95076,6 +95076,121 @@ var init_member_access_modifier_validation = __esm({
   }
 });
 
+// ../src/database/project-version-write-changes.ts
+function applyProjectVersionWriteChanges(document, changes) {
+  return projectVersionDocumentWithChanges(document, changes, false);
+}
+function projectVersionDocumentWithChanges(document, changes, reuseUnchangedBuckets) {
+  const projectRows = (rows, kind) => {
+    const relevant = changes.filter((change) => change.recordKind === kind);
+    if (reuseUnchangedBuckets && relevant.length === 0) return rows;
+    return applyArrayChanges(rows, relevant);
+  };
+  const hasGranularDialogueRecords = document.dialogueRecords !== void 0 || changes.some((change) => change.recordKind === "dialogue");
+  const hasGranularDialogueNodes = document.dialogueNodes !== void 0 || changes.some((change) => change.recordKind === "dialogue-node");
+  const dialogueRecords = hasGranularDialogueRecords ? projectRows(document.dialogueRecords ?? [], "dialogue") : void 0;
+  const dialogueNodes = hasGranularDialogueNodes ? projectRows(document.dialogueNodes ?? [], "dialogue-node") : void 0;
+  const projected = {
+    localizedTexts: projectRows(
+      document.localizedTexts ?? [],
+      "localized-text"
+    ),
+    localizationConfig: applySingletonChange(
+      document.localizationConfig,
+      changes,
+      "localization-config"
+    ),
+    project: applySingletonChange(document.project, changes, "project"),
+    members: projectRows(document.members, "member"),
+    classes: projectRows(document.classes, "class"),
+    // P43 §6. Declared constructors participate in the post-transaction
+    // document so the whole-graph validator and every body compile see the
+    // same constructor surface the transaction is about to persist.
+    constructors: projectRows(document.constructors ?? [], "constructor"),
+    // P67 §2.1. Same reason as constructors: without these buckets the
+    // whole-graph validator, the variant ownership roots, and every body
+    // compile would grade the transaction against the pre-transaction
+    // variant set — so a newly pushed variant's closures would compile
+    // against a document that does not contain it.
+    variants: projectRows(document.variants ?? [], "variant"),
+    variantFolders: projectRows(
+      document.variantFolders ?? [],
+      "variant-folder"
+    ),
+    values: projectRows(document.values ?? [], "value"),
+    // Project-file staging finalizes pending uploads before schema
+    // preparation. Defaults and authored instance graphs may reference a file
+    // created by the same source transaction, so every construction pass must
+    // see that finalized row in its post-write document.
+    projectFiles: projectRows(document.projectFiles ?? [], "project-file"),
+    // Import-setting templates are part of the same file graph. A new file
+    // may select a new template in one source transaction, and default-value
+    // materialization validates that relationship immediately against this
+    // post-write document.
+    textureTemplates: projectRows(
+      document.textureTemplates ?? [],
+      "unity-texture-template"
+    ),
+    audioClipTemplates: projectRows(
+      document.audioClipTemplates ?? [],
+      "unity-audio-clip-template"
+    ),
+    internalRecordRelations: projectRows(
+      document.internalRecordRelations ?? [],
+      "internal-record-relation"
+    ),
+    enums: projectRows(document.enums, "enum"),
+    interfaces: projectRows(document.interfaces, "interface"),
+    dialogues: dialogueRecords === void 0 || dialogueNodes === void 0 || dialogueRecords === document.dialogueRecords && dialogueNodes === document.dialogueNodes ? document.dialogues : materializeDialogues(
+      dialogueRecords,
+      dialogueNodes
+    ),
+    dialogueGroups: projectRows(
+      document.dialogueGroups ?? [],
+      "dialogue-group"
+    ),
+    migrations: projectRows(document.migrations ?? [], "migration")
+  };
+  return {
+    ...document,
+    ...projected,
+    ...dialogueRecords === void 0 ? {} : { dialogueRecords },
+    ...dialogueNodes === void 0 ? {} : { dialogueNodes }
+    // The document spread above carries every bucket the caller's view declares
+    // beyond the projected ones, so the result is the same view; TypeScript
+    // cannot prove that of an object literal built over a type parameter.
+  };
+}
+function applySingletonChange(current, changes, recordKind) {
+  const change = changes.find(
+    (candidate) => candidate.recordKind === recordKind
+  );
+  if (change === void 0) return current;
+  if (change.operation === "delete" || change.nextData === void 0) {
+    throw new Error(`Schema commit cannot delete the ${recordKind} record.`);
+  }
+  return change.nextData;
+}
+function applyArrayChanges(current, changes) {
+  const byId = new Map(current.map((record6) => [record6.id, record6]));
+  for (const change of changes) {
+    if (change.operation === "delete") {
+      byId.delete(change.recordId);
+      continue;
+    }
+    if (change.nextData !== void 0) {
+      byId.set(change.recordId, change.nextData);
+    }
+  }
+  return [...byId.values()];
+}
+var init_project_version_write_changes = __esm({
+  "../src/database/project-version-write-changes.ts"() {
+    "use strict";
+    init_projectDocumentDialogueMaterialization();
+  }
+});
+
 // ../src/database/member-localization-normalization.ts
 function planMemberLocalizationNormalization(input, now = 0) {
   const before = expandPackedDocumentValues(input);
@@ -95366,7 +95481,7 @@ var init_member_localization_normalization = __esm({
     init_constructor_argument_ownership();
     init_variant_value_graph();
     init_packed_value_write_fold();
-    init_project_version_schema_commit();
+    init_project_version_write_changes();
     init_packed_value_encoding();
     init_inheritance();
     init_members();
@@ -100590,16 +100705,38 @@ function prepareServerOwnedValueInitializerBodies(args) {
   }
 }
 function collectDelegateSweepRows(args) {
-  if (args.targetIds.size === 0) return [];
+  const closureRows = [];
+  const inlineRows = [];
+  if (args.targetIds.size === 0) return { closureRows, inlineRows };
   const preparedValueIds = indexChangesByRecordId(args.prepared, "value");
-  const sweepRows = [];
   for (const value of args.postDocument.values) {
     if (!args.targetIds.has(value.id)) continue;
     if (preparedValueIds.has(value.id)) continue;
     const row = delegateValueRow(value);
-    if (row !== null) sweepRows.push(row);
+    if (row !== null) {
+      closureRows.push(row);
+      continue;
+    }
+    const inlineRow = inlineDelegateSweepRow(value);
+    if (inlineRow !== null) inlineRows.push(inlineRow);
+  }
+  return { closureRows, inlineRows };
+}
+function inlineDelegateSweepRow(value) {
+  if (!isMemberValue(value) || !isLiteralValueContent(value)) return null;
+  const constructorArgs = asRecord4(
+    value.constructorArgs
+  );
+  if (constructorArgs === null) return null;
+  let sourceArgs = null;
+  for (const [key, argument2] of Object.entries(constructorArgs)) {
+    const closure = asRecord4(argument2);
+    if (typeof closure?.code !== "string") continue;
+    if (!("action" in closure)) continue;
+    sourceArgs ??= { ...constructorArgs };
+    sourceArgs[key] = { code: closure.code };
   }
-  return sweepRows;
+  return sourceArgs === null ? null : { ...value, constructorArgs: sourceArgs };
 }
 function delegateValueRow(value) {
   if (!isMemberValue(value) || !isLiteralValueContent(value)) return null;
@@ -100746,8 +100883,8 @@ function prepareServerOwnedDelegateValueBodies(args) {
     const withInlineArgs = inlineDelegateArgumentRow(change.nextData);
     if (withInlineArgs !== null) inlineArgumentRows.set(index, withInlineArgs);
   }
-  const sweepRows = collectDelegateSweepRows(args);
-  if (explicitRows.size === 0 && sweepRows.length === 0 && inlineArgumentRows.size === 0) {
+  const { closureRows: sweepRows, inlineRows: inlineSweepRows } = collectDelegateSweepRows(args);
+  if (explicitRows.size === 0 && sweepRows.length === 0 && inlineArgumentRows.size === 0 && inlineSweepRows.length === 0) {
     return;
   }
   const committedDocument = applyProjectVersionWriteChanges(
@@ -100756,7 +100893,7 @@ function prepareServerOwnedDelegateValueBodies(args) {
   );
   const targetIds = valueIdsFromRows(
     [...explicitRows.values(), ...inlineArgumentRows.values()],
-    sweepRows
+    [...sweepRows, ...inlineSweepRows]
   );
   const rootOwnerByValueId = /* @__PURE__ */ new Map();
   const delegateScope = withVariantOwnershipRoots(
@@ -100831,7 +100968,7 @@ function prepareServerOwnedDelegateValueBodies(args) {
       };
     }
   }
-  if (sweepRows.length === 0) return;
+  if (sweepRows.length === 0 && inlineSweepRows.length === 0) return;
   const currentById = new Map(
     args.document.values.map((value) => [value.id, value])
   );
@@ -100841,40 +100978,65 @@ function prepareServerOwnedDelegateValueBodies(args) {
     prepared: args.prepared,
     contentHashHeads: args.contentHashHeads
   });
+  const refresh = (id2, compiled) => {
+    const current = currentById.get(id2);
+    if (current === void 0 || canonicallyEqual2(current, compiled)) return;
+    const contentHash = baseHashFor(id2);
+    if (contentHash === void 0) {
+      throw new Error(
+        `Cannot safely refresh the compiled NeoDelegate closure for value "${id2}": its current content hash is unavailable.`
+      );
+    }
+    args.prepared.push({
+      recordKind: "value",
+      recordId: id2,
+      operation: "update",
+      nextData: compiled,
+      intent: createProjectVersionIntent("value.update", {
+        source: "server-neoscript-recompile"
+      }),
+      ...contentHash === null ? {} : { expectedBaseContentHash: contentHash }
+    });
+  };
   for (const row of sweepRows) {
     const compiled = { ...row, value: { ...row.value } };
     try {
       compileOne(delegateScope.document, compiled);
     } catch (postWriteError) {
-      const current2 = delegateValueRow(
+      const current = delegateValueRow(
         structuredClone(currentById.get(row.id))
       );
-      if (current2 === null) throw postWriteError;
+      if (current === null) throw postWriteError;
       try {
-        compileOne(args.document, current2);
+        compileOne(args.document, current);
       } catch (preWriteError) {
         if (sameCompileFailure(preWriteError, postWriteError)) continue;
       }
       throw postWriteError;
     }
-    const current = currentById.get(row.id);
-    if (current === void 0 || canonicallyEqual2(current, compiled)) continue;
-    const contentHash = baseHashFor(row.id);
-    if (contentHash === void 0) {
-      throw new Error(
-        `Cannot safely refresh the compiled NeoDelegate closure for value "${row.id}": its current content hash is unavailable.`
-      );
+    refresh(row.id, compiled);
+  }
+  const compileInlineOne = (document, row) => compileInlineDelegateConstructorArguments({
+    document,
+    row,
+    ownerMember: ownerByValueId.get(row.id),
+    rootOwner: rootOwnerByValueId.get(row.id) ?? null
+  });
+  for (const row of inlineSweepRows) {
+    let compiledArgs;
+    try {
+      compiledArgs = compileInlineOne(delegateScope.document, row);
+    } catch (postWriteError) {
+      const current = inlineDelegateSweepRow(currentById.get(row.id));
+      if (current === null) throw postWriteError;
+      try {
+        compileInlineOne(args.document, current);
+      } catch (preWriteError) {
+        if (sameCompileFailure(preWriteError, postWriteError)) continue;
+      }
+      throw postWriteError;
     }
-    args.prepared.push({
-      recordKind: "value",
-      recordId: row.id,
-      operation: "update",
-      nextData: compiled,
-      intent: createProjectVersionIntent("value.update", {
-        source: "server-neoscript-recompile"
-      }),
-      ...contentHash === null ? {} : { expectedBaseContentHash: contentHash }
-    });
+    refresh(row.id, { ...row, constructorArgs: compiledArgs });
   }
 }
 function prepareServerOwnedConstructorBodies(args) {
@@ -101129,113 +101291,6 @@ function assertUniqueChanges(changes) {
     }
   }
 }
-function applyProjectVersionWriteChanges(document, changes) {
-  return projectVersionDocumentWithChanges(document, changes, false);
-}
-function projectVersionDocumentWithChanges(document, changes, reuseUnchangedBuckets) {
-  const projectRows = (rows, kind) => {
-    const relevant = changes.filter((change) => change.recordKind === kind);
-    if (reuseUnchangedBuckets && relevant.length === 0) return rows;
-    return applyArrayChanges(rows, relevant);
-  };
-  const hasGranularDialogueRecords = document.dialogueRecords !== void 0 || changes.some((change) => change.recordKind === "dialogue");
-  const hasGranularDialogueNodes = document.dialogueNodes !== void 0 || changes.some((change) => change.recordKind === "dialogue-node");
-  const dialogueRecords = hasGranularDialogueRecords ? projectRows(document.dialogueRecords ?? [], "dialogue") : void 0;
-  const dialogueNodes = hasGranularDialogueNodes ? projectRows(document.dialogueNodes ?? [], "dialogue-node") : void 0;
-  const projected = {
-    localizedTexts: projectRows(
-      document.localizedTexts ?? [],
-      "localized-text"
-    ),
-    localizationConfig: applySingletonChange(
-      document.localizationConfig,
-      changes,
-      "localization-config"
-    ),
-    project: applySingletonChange(document.project, changes, "project"),
-    members: projectRows(document.members, "member"),
-    classes: projectRows(document.classes, "class"),
-    // P43 §6. Declared constructors participate in the post-transaction
-    // document so the whole-graph validator and every body compile see the
-    // same constructor surface the transaction is about to persist.
-    constructors: projectRows(document.constructors ?? [], "constructor"),
-    // P67 §2.1. Same reason as constructors: without these buckets the
-    // whole-graph validator, the variant ownership roots, and every body
-    // compile would grade the transaction against the pre-transaction
-    // variant set — so a newly pushed variant's closures would compile
-    // against a document that does not contain it.
-    variants: projectRows(document.variants ?? [], "variant"),
-    variantFolders: projectRows(
-      document.variantFolders ?? [],
-      "variant-folder"
-    ),
-    values: projectRows(document.values ?? [], "value"),
-    // Project-file staging finalizes pending uploads before schema
-    // preparation. Defaults and authored instance graphs may reference a file
-    // created by the same source transaction, so every construction pass must
-    // see that finalized row in its post-write document.
-    projectFiles: projectRows(document.projectFiles ?? [], "project-file"),
-    // Import-setting templates are part of the same file graph. A new file
-    // may select a new template in one source transaction, and default-value
-    // materialization validates that relationship immediately against this
-    // post-write document.
-    textureTemplates: projectRows(
-      document.textureTemplates ?? [],
-      "unity-texture-template"
-    ),
-    audioClipTemplates: projectRows(
-      document.audioClipTemplates ?? [],
-      "unity-audio-clip-template"
-    ),
-    internalRecordRelations: projectRows(
-      document.internalRecordRelations ?? [],
-      "internal-record-relation"
-    ),
-    enums: projectRows(document.enums, "enum"),
-    interfaces: projectRows(document.interfaces, "interface"),
-    dialogues: dialogueRecords === void 0 || dialogueNodes === void 0 || dialogueRecords === document.dialogueRecords && dialogueNodes === document.dialogueNodes ? document.dialogues : materializeDialogues(
-      dialogueRecords,
-      dialogueNodes
-    ),
-    dialogueGroups: projectRows(
-      document.dialogueGroups ?? [],
-      "dialogue-group"
-    ),
-    migrations: projectRows(document.migrations ?? [], "migration")
-  };
-  return {
-    ...document,
-    ...projected,
-    ...dialogueRecords === void 0 ? {} : { dialogueRecords },
-    ...dialogueNodes === void 0 ? {} : { dialogueNodes }
-    // The document spread above carries every bucket the caller's view declares
-    // beyond the projected ones, so the result is the same view; TypeScript
-    // cannot prove that of an object literal built over a type parameter.
-  };
-}
-function applySingletonChange(current, changes, recordKind) {
-  const change = changes.find(
-    (candidate) => candidate.recordKind === recordKind
-  );
-  if (change === void 0) return current;
-  if (change.operation === "delete" || change.nextData === void 0) {
-    throw new Error(`Schema commit cannot delete the ${recordKind} record.`);
-  }
-  return change.nextData;
-}
-function applyArrayChanges(current, changes) {
-  const byId = new Map(current.map((record6) => [record6.id, record6]));
-  for (const change of changes) {
-    if (change.operation === "delete") {
-      byId.delete(change.recordId);
-      continue;
-    }
-    if (change.nextData !== void 0) {
-      byId.set(change.recordId, change.nextData);
-    }
-  }
-  return [...byId.values()];
-}
 function asRecord4(value) {
   if (typeof value !== "object" || value === null || Array.isArray(value)) {
     return null;
@@ -101430,7 +101485,6 @@ var init_project_version_schema_commit = __esm({
     init_evaluateInitializer();
     init_virtual_instance_graph();
     init_virtual_instance_values();
-    init_projectDocumentDialogueMaterialization();
     init_valueHeadPlacementGraph();
     init_canonical_json();
     init_compile_ns_property();
@@ -101455,6 +101509,7 @@ var init_project_version_schema_commit = __esm({
     init_project_version_relation_sweep_trigger();
     init_project_version_static_value_writes();
     init_project_version_transaction_planning();
+    init_project_version_write_changes();
     init_value_row_owner_members();
     init_variant_value_graph();
     SERVER_COMPILATION_PROJECT_CACHE_LIMIT = 4;
@@ -107537,7 +107592,7 @@ var init_project_version_whole_graph_validation = __esm({
     init_project_root_members();
     init_project_schema_identifier_validation();
     init_packed_value_write_fold();
-    init_project_version_schema_commit();
+    init_project_version_write_changes();
     init_project_version_system_protection_writes();
     init_reserved_record_id_validation();
     init_members();
@@ -139485,7 +139540,7 @@ var init_registry2 = __esm({
   "schema-contract/registry.mjs"() {
     "use strict";
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
-      cliVersion: "0.56.0",
+      cliVersion: "0.56.1",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {
@@ -147306,7 +147361,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.56.0");
+    console.log("0.56.1");
     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.56.0",
+  "version": "0.56.1",
   "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.56.0 -->
+<!-- reviewed-through-cli: 0.56.1 -->
 
 # Neo Compose CLI
 

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

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