@neocompose/cli 0.82.1 → 0.82.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.82.2
+
+### Patch Changes
+
+- 9e3a8d3: Fix post-push exports of class defaults that spell a field's declared default, such as animation FPS. Keep those fields sparse on subsequent analysis and preserve existing child-row ownership when adding a real default override.
+
 ## 0.82.1
 
 ### Patch Changes

package/dist/neo.mjs

@@ -55955,7 +55955,7 @@ var init_registry = __esm({
   "../packages/manifest/schema-contract/registry.mjs"() {
     "use strict";
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
-      cliVersion: "0.82.1",
+      cliVersion: "0.82.2",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {
@@ -121102,6 +121102,7 @@ function* prepareServerOwnedSchemaCommitPasses(args) {
   );
   const pendingConstructedGraphValidations = materializeAuthoredValueSeeds({
     document: seedPostDocument,
+    previousMembers: args.document.members,
     prepared,
     authoredValueSeeds: args.authoredValueSeeds ?? [],
     contentHashHeads: args.contentHashHeads,
@@ -122029,6 +122030,11 @@ function materializeAuthoredValueSeeds(args) {
   if (seedByMemberId.size !== args.authoredValueSeeds.length) {
     throw new Error("Schema commit contains duplicate member value seeds.");
   }
+  const previousSeedMembers = /* @__PURE__ */ new Map();
+  for (const member of args.previousMembers) {
+    if (seedByMemberId.has(member.id))
+      previousSeedMembers.set(member.id, member);
+  }
   const evaluationDocument = withProspectiveAuthoredSeedValues(
     args.document,
     args.authoredValueSeeds
@@ -122047,6 +122053,7 @@ function materializeAuthoredValueSeeds(args) {
       consumedSeedMemberIds.add(member.id);
       materializeInstanceDefaultSeed({
         document: evaluationDocument,
+        previousMember: previousSeedMembers.get(member.id),
         documentIndex,
         prepared: args.prepared,
         pendingGraphValidations,
@@ -122201,6 +122208,7 @@ function materializeAuthoredValueSeeds(args) {
     if (member !== void 0 && isStaticMember(member) !== true && !isProjectRootMember(args.document.project, member.id)) {
       materializeInstanceDefaultSeed({
         document: evaluationDocument,
+        previousMember: previousSeedMembers.get(member.id),
         documentIndex,
         prepared: args.prepared,
         pendingGraphValidations,
@@ -122535,19 +122543,42 @@ function materializeInstanceDefaultSeed(args) {
     ...args.member,
     defaultValue: seedValueContent(args.seed)
   };
+  const heldChildren = args.documentIndex.heldChildIdsByParentId;
+  const previousHeld = heldChildren.get(args.seed.valueId);
+  const declarationDefault = !isProjectRootMember(
+    args.document.project,
+    args.member.id
+  );
+  if (declarationDefault) {
+    heldChildren.set(
+      args.seed.valueId,
+      storedChildIds(
+        args.previousMember && !isStaticMember(args.previousMember) ? args.previousMember.defaultValue?.value : void 0
+      )
+    );
+  }
+  let materialized;
+  try {
+    materialized = materializeAuthoredStaticSeed({
+      document: args.document,
+      documentIndex: args.documentIndex,
+      member: materializationMember,
+      seed: { ...args.seed, values: args.seed.values },
+      allowExistingRows: args.memberChange === null || args.memberChange.change.operation === "update"
+    });
+  } finally {
+    if (declarationDefault) {
+      if (previousHeld === void 0) heldChildren.delete(args.seed.valueId);
+      else heldChildren.set(args.seed.valueId, previousHeld);
+    }
+  }
   const {
     createdValues,
     localizedTexts,
     storageKeyDeclarations,
     rootValue,
     existingValueIds
-  } = materializeAuthoredStaticSeed({
-    document: args.document,
-    documentIndex: args.documentIndex,
-    member: materializationMember,
-    seed: { ...args.seed, values: args.seed.values },
-    allowExistingRows: args.memberChange === null || args.memberChange.change.operation === "update"
-  });
+  } = materialized;
   stampCreatedValuesMapKey(createdValues, null);
   applyDeclaredStorageKeyOverrides({
     createdValues,
@@ -137476,18 +137507,32 @@ function lowerRowBackedDefault(context, member, backed, baseData3, binding) {
         environment
       ))
         continue;
-      body[assignment.name] = lowerSeedChild(
+      const rows = /* @__PURE__ */ new Map();
+      const localizedTexts = /* @__PURE__ */ new Map();
+      const childId = lowerSeedChild(
         context,
         childValueMember,
         assignment.value,
         binding,
         `${binding.label}.${assignment.name}`,
-        /* @__PURE__ */ new Map(),
-        /* @__PURE__ */ new Map(),
+        rows,
+        localizedTexts,
         void 0,
         environment,
         childSlice
       );
+      if (member.partial !== true && !(assignment.value.kind === "ident" && context.localValues.has(assignment.value.name)) && freshChildReplaysDeclaredDefault({
+        context,
+        childMember,
+        childId,
+        rows,
+        localizedTexts,
+        environment
+      })) {
+        deleteSeedRow(context, rows, childId);
+        continue;
+      }
+      body[assignment.name] = childId;
     }
     return {
       value: body,
@@ -205758,7 +205803,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.82.1");
+    console.log("0.82.2");
     return;
   }
   if (args.command === null || args.command === "help" || args.command === "--help" || args.command === "-h") {

package/dist/test-worker.mjs

@@ -61885,7 +61885,7 @@ var INITIALIZER_KEY_PLACEHOLDER = new RegExp(
 
 // ../packages/manifest/schema-contract/registry.mjs
 var PROJECT_SCHEMA_CONTRACT = Object.freeze({
-  cliVersion: "0.82.1",
+  cliVersion: "0.82.2",
   projectFileUploadBatchSize: 32,
   documentRecords: {
     member: {

package/package.json

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

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

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

package/skills/neocompose-cli/references/values-identities-and-references.md

@@ -265,6 +265,10 @@ that instance. Leaving the emitted construction unchanged creates no redundant
 value rows. A nullable member with no default remains bare; an explicit null
 default emits `= null`.
 
+Spelling an unchanged child default, such as `FPS = 8`, stays a no-op after
+push even when sibling fields own stored rows. Adding a different child default
+preserves those siblings' row identities; it does not transfer their ownership.
+
 Omitted members are virtual projections of the instance's recorded constructor
 or variant. Later edits to that source update every omitted member. Writing a
 member creates a sparse override with its deterministic projected ID; removing