@neocompose/cli 0.10.1 → 0.10.2

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## [0.10.2] - 2026-07-25
+
+### Fixed
+
+- Read `required` and an override's `extendsMemberId` from source instead of
+  carrying the pulled value forward. A member whose stored `required`
+  contradicted its own return type, and an override pointing past the member
+  it directly overrides, are corrected on the next push.
+- Persist a class's `schemaKeyOrder` only where the declaration order differs
+  from the one its schema map already implies. An equivalent order is stored
+  as absent rather than spelled out, and the answer no longer depends on
+  having pulled the class.
+- Restore the server-owned envelope on reconstructed records before
+  prospective animation validation, so a workspace with animation schema no
+  longer reports every class as malformed. Validation failures now name the
+  offending record.
+
 ## [0.10.1] - 2026-07-25
 
 ### Changed

package/dist/neo.mjs

@@ -28915,11 +28915,7 @@ function lowerClass(context, declaration) {
       selectionSpan: span(parameter3.identity)
     };
   });
-  const baseEffectiveOrder = base?.schemaKeyOrder ?? (base ? Object.keys(base.schema) : []);
-  const baseOwnOrder = baseEffectiveOrder.filter(
-    (name) => Object.hasOwn(base?.schema ?? {}, name)
-  );
-  const persistedSchemaOrder = base && JSON.stringify(baseOwnOrder) === JSON.stringify(schemaKeyOrder) ? base.schemaKeyOrder : schemaKeyOrder;
+  const persistedSchemaOrder = isImpliedSchemaKeyOrder(schemaKeyOrder) ? null : schemaKeyOrder;
   return {
     id: id2,
     source: sourceIdentity2(declaration, "class", declaration.name),
@@ -29120,7 +29116,7 @@ function projectSchemaJsonValue(value) {
 }
 function memberCommon(context, ownerClass, declaration, id2, owner, base) {
   const storage = annotation(declaration.annotations, "storage");
-  const inheritedId = declaration.modifiers.includes("override") || declaration.modifiers.includes("sealed") ? base?.overrideOf ?? inheritedMemberId(context, ownerClass, declaration.name) : null;
+  const inheritedId = declaration.modifiers.includes("override") || declaration.modifiers.includes("sealed") ? inheritedMemberId(context, ownerClass, declaration.name) : null;
   return {
     id: id2,
     source: sourceIdentity2(declaration, "member", declaration.name),
@@ -29135,7 +29131,10 @@ function memberCommon(context, ownerClass, declaration, id2, owner, base) {
     accessModifier: owner.kind === "classMember" ? effectiveSourceAccessModifier(declaration.modifiers, "class") : "public",
     modifier: memberModifier2(declaration.modifiers, base?.modifier),
     locked: hasAnnotation(declaration.annotations, "locked"),
-    required: declaration.kind === "field" ? !declaration.type.nullable : base?.required ?? false,
+    // Every declaration kind carries a type, and its nullability is the whole
+    // of what `required` means. Properties and getters used to fall back to
+    // the pulled value, which left a fresh workspace calling them optional.
+    required: !declaration.type.nullable,
     defaultValue: null,
     overrideOf: inheritedId,
     system: systemMetadata(declaration.annotations),
@@ -30161,6 +30160,9 @@ function referenceId(raw) {
     return null;
   }
 }
+function isImpliedSchemaKeyOrder(order) {
+  return order.every((key, index) => index === 0 || order[index - 1] <= key);
+}
 function memberModifier2(modifiers, baseModifier) {
   if (modifiers.includes("override")) {
     if (modifiers.includes("abstract")) return "abstractOverride";
@@ -48915,28 +48917,28 @@ function validateProspectiveAnimationRecordsV4(records2) {
     if (record3.recordKind === "project") {
       if (!isProject(record3.data)) {
         throw new Error(
-          "Prospective animation validation received an invalid project record."
+          `Prospective animation validation received an invalid project record ${recordLabel(record3.data)}.`
         );
       }
       project = record3.data;
     } else if (record3.recordKind === "class") {
       if (!isNeoSchemaClass(record3.data)) {
         throw new Error(
-          "Prospective animation validation received an invalid class record."
+          `Prospective animation validation received an invalid class record ${recordLabel(record3.data)}.`
         );
       }
       classes.push(record3.data);
     } else if (record3.recordKind === "member") {
       if (!isMember(record3.data)) {
         throw new Error(
-          "Prospective animation validation received an invalid member record."
+          `Prospective animation validation received an invalid member record ${recordLabel(record3.data)}.`
         );
       }
       members.push(record3.data);
     } else if (record3.recordKind === "value") {
       if (!isMemberValue(record3.data)) {
         throw new Error(
-          "Prospective animation validation received an invalid value record."
+          `Prospective animation validation received an invalid value record ${recordLabel(record3.data)}.`
         );
       }
       values.push(record3.data);
@@ -48945,6 +48947,18 @@ function validateProspectiveAnimationRecordsV4(records2) {
   if (project === void 0) return;
   assertAnimationClipDocumentValid({ project, classes, members, values });
 }
+function serverEnvelope(stored) {
+  if (!isObjectRecord2(stored)) return {};
+  const envelope = {};
+  for (const field of ["projectId", "createdAt", "updatedAt"]) {
+    if (stored[field] !== void 0) envelope[field] = stored[field];
+  }
+  return envelope;
+}
+function recordLabel(data) {
+  const id2 = typeof data.id === "string" ? data.id : "<no id>";
+  return typeof data.name === "string" ? `${id2} ("${data.name}")` : id2;
+}
 function prospectiveAnimationRecords(base, reconstructed3, changes, seeds) {
   const prospective = /* @__PURE__ */ new Map();
   for (const [key, state] of Object.entries(base)) {
@@ -48957,16 +48971,25 @@ function prospectiveAnimationRecords(base, reconstructed3, changes, seeds) {
     const key = recordStateKey(change.recordKind, change.recordId);
     if (change.kind === "delete") prospective.delete(key);
   }
-  for (const [key, record3] of reconstructed3) {
-    prospective.set(key, {
-      recordKind: record3.recordKind,
-      data: record3.fullData
-    });
-  }
-  const projectId = [...prospective.values()].find(
+  const projectRecord = [...prospective.values()].find(
     (record3) => record3.recordKind === "project"
-  )?.data.id;
+  ) ?? [...reconstructed3.values()].filter((record3) => record3.recordKind === "project").map((record3) => ({
+    recordKind: record3.recordKind,
+    data: record3.fullData
+  }))[0];
+  const projectId = projectRecord?.data.id;
   const seedEnvelope = typeof projectId === "string" ? { projectId, createdAt: 0, updatedAt: 0 } : {};
+  for (const [key, record3] of reconstructed3) {
+    const envelope = {
+      ...seedEnvelope,
+      ...serverEnvelope(prospective.get(key)?.data)
+    };
+    const data = { ...record3.fullData };
+    for (const [field, value] of Object.entries(envelope)) {
+      if (data[field] === void 0) data[field] = value;
+    }
+    prospective.set(key, { recordKind: record3.recordKind, data });
+  }
   for (const [memberId, seed] of seeds) {
     for (const bindingMember of seed.bindingMembers ?? []) {
       prospective.set(recordStateKey("member", bindingMember.id), {

package/package.json

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