@neocompose/cli 0.67.0 → 0.67.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [0.67.1] - 2026-10-04
+
+- Pull adopts records the server only recompiled into `.neo/state.json` before it reads local sources, even when the same pull also merges authored changes. Previously one authored change anywhere in the pull sent every recompiled record through the merge, which read the stale baseline and failed.
+- Status, push, and pull refuse a baseline holding a compiled body this CLI cannot read, name the record, and say to run `neo pull`, or `neo pull --force` after backing up local edits when pull cannot refresh it. Previously they failed with "init.compiled carries compiler revision 16" and suggested `npm run compiler-revision:mint`, which cannot fix a local file.
+- A baseline pulled before 0.67.0 also holds 64-bit floats, which only the server's copy rounded. Plain `neo pull` cannot refresh it. Commit or back up local edits, run `neo pull --force` once, then restore them.
+
 ## [0.67.0] - 2026-10-04
 
 - `float` is a 32-bit float everywhere, as in C#. Float literals, defaults, constructor arguments, and stored values round to the nearest 32-bit float, and pull prints the shortest decimal that reads back as that float, so `0.7` stays `0.7`. Previously push stored the 64-bit double, and Unity read a value that its own `float` could not hold.

package/dist/neo.mjs

@@ -30114,7 +30114,7 @@ var init_registry = __esm({
   "../packages/manifest/schema-contract/registry.mjs"() {
     "use strict";
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
-      cliVersion: "0.67.0",
+      cliVersion: "0.67.1",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {
@@ -40491,6 +40491,31 @@ function collectNeoScriptCompiledBodyStamps(record6) {
   visit(record6, "");
   return stamps;
 }
+function firstRejectedNeoScriptCompiledBody(node, rollout = ACTIVE_NEOSCRIPT_COMPILER_REVISION_ROLLOUT) {
+  if (Array.isArray(node)) {
+    for (let index = 0; index < node.length; index += 1) {
+      const found = firstRejectedNeoScriptCompiledBody(node[index], rollout);
+      if (found !== null) {
+        return { ...found, path: prefixPath(`[${String(index)}]`, found.path) };
+      }
+    }
+    return null;
+  }
+  if (typeof node !== "object" || node === null) return null;
+  const object2 = node;
+  if (isCompiledBodyShape(object2) && !isAcceptedNeoScriptCompilerRevision(object2.compilerRevision, rollout)) {
+    return { path: "", stamp: object2.compilerRevision };
+  }
+  for (const key of Object.keys(object2)) {
+    const found = firstRejectedNeoScriptCompiledBody(object2[key], rollout);
+    if (found === null) continue;
+    return { ...found, path: prefixPath(key, found.path) };
+  }
+  return null;
+}
+function prefixPath(head, rest) {
+  return rest === "" || rest.startsWith("[") ? head + rest : `${head}.${rest}`;
+}
 function isCompiledBodyShape(object2) {
   if (!Array.isArray(object2.parameters)) return false;
   if (!Array.isArray(object2.instructions)) return false;
@@ -112239,6 +112264,62 @@ var init_schema_source_validation = __esm({
   }
 });
 
+// src/project-sync/compiler-metadata.ts
+function compilerOnlyChanges(previous, document, changedKeys) {
+  const keys = changedKeys ?? /* @__PURE__ */ new Set([...Object.keys(previous), ...document.records.keys()]);
+  const refreshed = /* @__PURE__ */ new Set();
+  let complete2 = true;
+  for (const key of keys) {
+    const before = previous[key];
+    const after = document.records.get(key);
+    if (before !== void 0 && after !== void 0 && !after.deleted && before.contentHash === after.contentHash)
+      continue;
+    if (before === void 0 || after === void 0 || after.deleted || before.recordId !== after.recordId || before.recordKind !== after.recordKind || !isProjectRecordKind(after.recordKind) || !isObjectRecord2(before.data) || !isObjectRecord2(after.data) || !canonicallyEqual(
+      authoredCompilerInput(after.recordKind, before.data),
+      authoredCompilerInput(after.recordKind, after.data)
+    )) {
+      complete2 = false;
+      continue;
+    }
+    refreshed.add(key);
+  }
+  return { keys: refreshed, complete: complete2 };
+}
+function authoredCompilerInput(kind, data) {
+  const result = stripClientDerivedBodies(kind, data);
+  stampUILogicBodies(kind, result);
+  if (isObjectRecord2(result)) delete result.updatedAt;
+  return result;
+}
+function assertCurrentBaseline(records2) {
+  for (const [key, record6] of Object.entries(records2)) {
+    if (!isProjectRecordKind(record6.recordKind)) continue;
+    const rejected = firstRejectedNeoScriptCompiledBody(record6.data);
+    if (rejected === null) continue;
+    try {
+      validateProjectRecordByKind(record6.recordKind, record6.data, {
+        members: "authored-or-compiled"
+      });
+      continue;
+    } catch {
+    }
+    const carried = rejected.stamp === void 0 ? "has no compiler revision stamp" : `carries compiler revision ${JSON.stringify(rejected.stamp)}`;
+    throw new Error(
+      `The pull baseline in .neo/state.json predates this CLI: ${key} ${rejected.path} ${carried}. Run \`neo pull\` to refresh it. If pull reports this again, commit or back up your local edits, run \`neo pull --force\`, then restore them.`
+    );
+  }
+}
+var init_compiler_metadata = __esm({
+  "src/project-sync/compiler-metadata.ts"() {
+    "use strict";
+    init_project2();
+    init_neo_script_derived_bodies();
+    init_neoscript_compiler_revision();
+    init_project_record_validation();
+    init_value_comparison();
+  }
+});
+
 // src/project-sync/value-record-comparison.ts
 function valueRecordComparisonBody(value) {
   if (!isObjectRecord2(value)) return value;
@@ -118422,6 +118503,7 @@ function materializeWorkspaceStatusInitializers(workspace, status, forceRecompil
   });
 }
 function computeWorkspaceStatus(workspace, options) {
+  assertCurrentBaseline(workspace.state.records);
   let phaseStarted = performance.now();
   options.onPhase?.("Reading project schema\u2026");
   const reportPhase = (phase, nextLabel) => {
@@ -120396,6 +120478,7 @@ var init_workspace_status_core = __esm({
     init_key_reference_spelling();
     init_workspace();
     init_source_diagnostics();
+    init_compiler_metadata();
     init_migration_files();
     init_value_comparison();
     init_projection();
@@ -137892,40 +137975,6 @@ var init_static_value_seed_records = __esm({
   }
 });
 
-// src/project-sync/compiler-metadata.ts
-function compilerMetadataChangedKeys(previous, document, changedKeys) {
-  const keys = changedKeys ?? /* @__PURE__ */ new Set([...Object.keys(previous), ...document.records.keys()]);
-  const refreshed = /* @__PURE__ */ new Set();
-  for (const key of keys) {
-    const before = previous[key];
-    const after = document.records.get(key);
-    if (before === void 0 || after === void 0 || after.deleted || before.recordId !== after.recordId || before.recordKind !== after.recordKind || !isProjectRecordKind(after.recordKind))
-      return null;
-    if (before.contentHash === after.contentHash) continue;
-    if (!isObjectRecord2(before.data) || !isObjectRecord2(after.data) || !canonicallyEqual(
-      authoredCompilerInput(after.recordKind, before.data),
-      authoredCompilerInput(after.recordKind, after.data)
-    ))
-      return null;
-    refreshed.add(key);
-  }
-  return refreshed.size === 0 ? null : refreshed;
-}
-function authoredCompilerInput(kind, data) {
-  const result = stripClientDerivedBodies(kind, data);
-  stampUILogicBodies(kind, result);
-  if (isObjectRecord2(result)) delete result.updatedAt;
-  return result;
-}
-var init_compiler_metadata = __esm({
-  "src/project-sync/compiler-metadata.ts"() {
-    "use strict";
-    init_project2();
-    init_neo_script_derived_bodies();
-    init_value_comparison();
-  }
-});
-
 // src/commands/pull.ts
 var pull_exports = {};
 __export(pull_exports, {
@@ -138014,20 +138063,23 @@ async function runNormalPull(workspace, options, progress) {
     document = await fetchProjectDocument(workspace);
     nextCursor = document.revision === void 0 ? null : cursorFromRevision(document.revision);
   }
-  const compilerRefreshKeys = !destructive && hasBaseline && !hadConflicts && !options.regenerateSourceNames ? compilerMetadataChangedKeys(
-    workspace.state.records,
-    document,
-    changedRecordKeys
-  ) : null;
-  if (compilerRefreshKeys !== null) {
-    finishRecordOnlyDeltaPull({
-      workspace,
+  if (!destructive && hasBaseline && !hadConflicts) {
+    const compilerOnly = compilerOnlyChanges(
+      workspace.state.records,
       document,
-      changedRecordKeys: compilerRefreshKeys,
-      nextCursor,
-      progress
-    });
-    return;
+      changedRecordKeys
+    );
+    if (compilerOnly.complete && compilerOnly.keys.size > 0 && !options.regenerateSourceNames) {
+      finishRecordOnlyDeltaPull({
+        workspace,
+        document,
+        changedRecordKeys: compilerOnly.keys,
+        nextCursor,
+        progress
+      });
+      return;
+    }
+    adoptServerRecords(workspace, document, compilerOnly.keys);
   }
   let localByKey = /* @__PURE__ */ new Map();
   let localStatus = null;
@@ -138616,9 +138668,25 @@ function deltaRequiresSourceProjectionV4(args) {
   return false;
 }
 function finishRecordOnlyDeltaPull(args) {
-  const records2 = { ...args.workspace.state.records };
-  for (const key of args.changedRecordKeys) {
-    const server = args.document.records.get(key);
+  adoptServerRecords(args.workspace, args.document, args.changedRecordKeys);
+  if (args.nextCursor !== null) {
+    args.workspace.state.documentRevisionCursor = mutableCursor(
+      args.nextCursor
+    );
+  }
+  args.workspace.state.headTransactionHash = pulledHeadTransactionHash(
+    args.document
+  );
+  writeWorkspaceState(args.workspace.root, args.workspace.state);
+  args.progress.succeed(
+    `Pulled ${args.changedRecordKeys.size} changed record(s) \u2014 0 file(s) updated \u2014 0 removed.`
+  );
+}
+function adoptServerRecords(workspace, document, keys) {
+  if (keys.size === 0) return;
+  const records2 = { ...workspace.state.records };
+  for (const key of keys) {
+    const server = document.records.get(key);
     if (server === void 0) {
       delete records2[key];
       continue;
@@ -138633,19 +138701,7 @@ function finishRecordOnlyDeltaPull(args) {
       ...previous?.projectBinary === void 0 ? {} : { projectBinary: previous.projectBinary }
     };
   }
-  args.workspace.state.records = records2;
-  if (args.nextCursor !== null) {
-    args.workspace.state.documentRevisionCursor = mutableCursor(
-      args.nextCursor
-    );
-  }
-  args.workspace.state.headTransactionHash = pulledHeadTransactionHash(
-    args.document
-  );
-  writeWorkspaceState(args.workspace.root, args.workspace.state);
-  args.progress.succeed(
-    `Pulled ${args.changedRecordKeys.size} changed record(s) \u2014 0 file(s) updated \u2014 0 removed.`
-  );
+  workspace.state.records = records2;
 }
 function pulledHeadTransactionHash(document) {
   return document.revision?.headTransactionHash ?? null;
@@ -156777,7 +156833,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.67.0");
+    console.log("0.67.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.67.0",
+  "version": "0.67.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.67.0 -->
+<!-- reviewed-through-cli: 0.67.1 -->
 
 # Neo Compose CLI
 

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

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

package/skills/neocompose-cli/references/commands-and-sync.md

@@ -40,12 +40,14 @@ without compiling it; use `neo status` to inspect local edits and diagnostics.
 An undo/reset metadata change invalidates the cursor and causes a safe
 full-snapshot fallback.
 
-After a compiler upgrade, run `neo pull` with `.neo/state.json` intact. When
-remote changes affect only compiler-owned bodies or timestamps, pull refreshes
-the cached records before local compilation and leaves authored files and
-managed binaries untouched. It handles both changed-record pulls and full
-snapshot refreshes. Added or deleted records, authored changes, and unresolved
-conflicts still require the normal merge path.
+After a compiler upgrade, run `neo pull` with `.neo/state.json` intact. Pull
+refreshes every cached record whose remote change touches only compiler-owned
+bodies or timestamps before local compilation, even when the same pull merges
+other changes. When nothing else changed, it leaves authored files and managed
+binaries untouched. Added or deleted records, authored changes, and unresolved
+conflicts take the normal merge path. Status, push, and pull refuse a baseline
+holding a compiled body this CLI rejects and name the record. If `neo pull`
+cannot refresh it, back up local edits and run `neo pull --force`.
 
 If a push commits but source rewriting fails, `neo pull` resumes its saved
 result before checking the remote head. The recovery journal in `.neo/state.json`