@neocompose/cli 0.26.0 → 0.26.1

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## [0.26.1] - 2026-08-10
+
+### Fixed
+
+- Preserve concrete generic bindings while replaying nested constructors in class initializers, including generic entries inside inherited collection defaults. `neo status` and push preparation no longer reject a closed `NeoAnimationSegmentFrame<T>` as an open generic or report unchanged construction arguments as conflicting.
+- Treat assignment to a single-value Lookup as a write to its owning selection row, while continuing to enforce the referenced collection's storage on writes through the selected value. Save-backed selections can now point at Immutable catalog entries without becoming immutable themselves.
+- Rewrite reconstructed interface-member symbols through their interface owner's verified pending id instead of trying to assign the composite compiler symbol its own record id.
+- Accept a constrained open generic member as a covariant implementation of a get-only interface property while continuing to reject that unsafe substitution for settable properties.
+
 ## [0.26.0] - 2026-08-09
 
 ### Added

package/dist/neo.mjs

@@ -10150,7 +10150,9 @@ var init_strict_resolver = __esm({
                 pointer: operand.pointer
               },
               type: isNullable(operand.type) ? { ...operand.type, nullable: false } : operand.type,
-              ...operand.writability ? { writability: operand.writability } : {}
+              ...operand.writability ? { writability: operand.writability } : {},
+              ...operand.entryWritability ? { entryWritability: operand.entryWritability } : {},
+              ...operand.writeRoot ? { writeRoot: operand.writeRoot } : {}
             };
           }
           case "coalesce": {
@@ -10999,7 +11001,7 @@ var init_strict_resolver = __esm({
             };
           }
           const writability = this.memberWritability(member, receiver);
-          const entryWritability = member.lookup?.multiselect === true ? this.lookupEntryWritability(member, receiver) : void 0;
+          const entryWritability = member.lookup ? this.lookupEntryWritability(member, receiver) : void 0;
           const lookupWire = member.lookup?.multiselect === true && memberType2.kind === "set" ? {
             type: 9 /* Lookup */,
             required: !isNullable(memberType2),
@@ -11094,12 +11096,9 @@ var init_strict_resolver = __esm({
         }
         if (!member.writable) return void 0;
         if (!member.lookup) {
-          return member.writability ? toWritability(member.writability) : receiver.writability ?? "save" /* Save */;
+          return member.writability ? toWritability(member.writability) : receiver.entryWritability ?? receiver.writability ?? "save" /* Save */;
         }
-        if (member.lookup.multiselect) {
-          return member.writability ? toWritability(member.writability) : receiver.writability ?? "save" /* Save */;
-        }
-        return this.lookupEntryWritability(member, receiver);
+        return member.writability ? toWritability(member.writability) : receiver.writability ?? "save" /* Save */;
       }
       lookupEntryWritability(member, receiver) {
         if (!member.lookup) return "readOnly" /* ReadOnly */;
@@ -42908,6 +42907,7 @@ function evaluateLiteralContainer(args) {
   }
   return {
     literal: literal2,
+    ...evaluated.genericBindings === void 0 ? {} : { genericBindings: evaluated.genericBindings },
     ...evaluated.provisionalRootId === void 0 ? {} : { provisionalRootId: evaluated.provisionalRootId },
     ...evaluated.existingValueRow === void 0 ? {} : { existingValueRow: evaluated.existingValueRow }
   };
@@ -42930,6 +42930,9 @@ function buildDefaultMemberValue(args) {
       return evaluated.existingValueRow;
     }
     const value2 = buildNewValue(args.projectId, evaluated.literal);
+    if (evaluated.genericBindings !== void 0) {
+      value2.genericBindings = { ...evaluated.genericBindings };
+    }
     if (evaluated.provisionalRootId !== void 0) {
       retargetDelegateReceiverValueIds(
         [value2, ...args.createdValues],
@@ -43316,6 +43319,9 @@ function cloneDefaultValueForMember(args) {
       return evaluated.existingValueRow;
     }
     const evaluatedRow = buildNewValue(args.projectId, evaluated.literal);
+    if (evaluated.genericBindings !== void 0) {
+      evaluatedRow.genericBindings = { ...evaluated.genericBindings };
+    }
     if (evaluated.provisionalRootId !== void 0) {
       retargetDelegateReceiverValueIds(
         [evaluatedRow, ...args.createdValues],
@@ -62168,6 +62174,43 @@ function validateConstructorFieldSlot(args) {
     requiredWithoutDefault: member.required && member.defaultValue == null
   };
 }
+function constructionGenericSlotsForMember(rawMember, ctx, instanceEnv, visitedMemberIds = /* @__PURE__ */ new Set()) {
+  const member = substituteMember(
+    rawMember,
+    instanceEnv,
+    ctx.vm.members
+  );
+  const memberId = Reflect.get(member, "id");
+  if (typeof memberId === "string" && visitedMemberIds.has(memberId)) {
+    return [];
+  }
+  const nextVisited = new Set(visitedMemberIds);
+  if (typeof memberId === "string") nextVisited.add(memberId);
+  if (isMemberClassBase(member)) {
+    const classArguments2 = member.classArguments ?? {};
+    return Object.keys(classArguments2).length === 0 ? [] : [{ classId: member.classId, classArguments: classArguments2 }];
+  }
+  if (!isMemberListBase(member) && !isMemberDictionaryBase(member)) return [];
+  const entryMember = evalMemberById(ctx.vm, member.entryMemberId);
+  if (entryMember === null) return [];
+  return constructionGenericSlotsForMember(
+    resolveMember2(entryMember, ctx.vm.members),
+    ctx,
+    instanceEnv,
+    nextVisited
+  );
+}
+function withConstructionGenericSlots(member, ctx, instanceEnv, evaluate) {
+  const slots = constructionGenericSlotsForMember(member, ctx, instanceEnv);
+  if (slots.length === 0) return evaluate();
+  const previousSlots = ctx.__constructionGenericSlots;
+  ctx.__constructionGenericSlots = [...previousSlots ?? [], ...slots];
+  try {
+    return evaluate();
+  } finally {
+    ctx.__constructionGenericSlots = previousSlots;
+  }
+}
 function evaluateConstructorFields(descriptor, scope, ctx) {
   const schemaClass2 = descriptor.schemaClass;
   const evaluatedValues = descriptor.fields.map((validated) => {
@@ -62176,7 +62219,13 @@ function evaluateConstructorFields(descriptor, scope, ctx) {
         `Constructor field '${validated.schemaKey}' on '${schemaClass2.name}' has no call-site expression to evaluate.`
       );
     }
-    return evalPointer(validated.valuePointer, scope, ctx);
+    const valuePointer = validated.valuePointer;
+    return withConstructionGenericSlots(
+      validated.member,
+      ctx,
+      descriptor.instanceEnv,
+      () => evalPointer(valuePointer, scope, ctx)
+    );
   });
   return descriptor.fields.map((validated, index) => {
     let value = evaluatedValues[index];
@@ -62214,6 +62263,14 @@ function syntheticConstructorMember(schemaClass2, classArguments2) {
     accessModifierKind: "public"
   };
 }
+function stampConstructedRootGenericBindings(root, descriptor) {
+  const stamp = {};
+  for (const parameter4 of descriptor.schemaClass.genericParams ?? []) {
+    const binding = descriptor.instanceEnv.get(parameter4.id);
+    if (binding?.kind === "member") stamp[parameter4.id] = binding.memberId;
+  }
+  if (Object.keys(stamp).length > 0) root.genericBindings = stamp;
+}
 function applyConstructorFields(args) {
   const {
     root,
@@ -62551,7 +62608,8 @@ function constructClassValueWithinFrame(descriptor, scope, ctx) {
           valuesByClassId: /* @__PURE__ */ new Map(),
           missingScopeReason: "no-constructor"
         },
-        classId
+        classId,
+        descriptor.instanceEnv
       ),
       constructorRoot: {
         providedSchemaKeys: new Set(
@@ -62560,6 +62618,7 @@ function constructClassValueWithinFrame(descriptor, scope, ctx) {
       }
     });
     root.classId = classId;
+    stampConstructedRootGenericBindings(root, descriptor);
     if (typeof root.value !== "object" || root.value === null || Array.isArray(root.value)) {
       throw new Error(
         `Class constructor for '${schemaClass2.name}' produced a non-record root.`
@@ -62918,7 +62977,7 @@ function evaluateBaseInitializerFields(args) {
     return { validated, value };
   });
 }
-function constructionInitEvaluator(ctx, createdValues, argumentScopes, constructedClassId) {
+function constructionInitEvaluator(ctx, createdValues, argumentScopes, constructedClassId, instanceEnv) {
   const indexes = constructorInitializerIndexes(ctx);
   const inheritanceChain = resolveInheritanceChain(
     constructedClassId,
@@ -62940,36 +62999,19 @@ function constructionInitEvaluator(ctx, createdValues, argumentScopes, construct
   };
   return (member, init, sourceValueId) => {
     const evaluate = (argumentValues = []) => {
-      if (!isMemberClassBase(member) || Object.keys(member.classArguments ?? {}).length === 0) {
-        return evaluateInitializerInContext(
-          init,
-          member,
-          ctx,
-          createdValues,
-          argumentValues,
-          sourceValueId ?? null
-        );
-      }
-      const previousSlots = ctx.__constructionGenericSlots;
-      ctx.__constructionGenericSlots = [
-        ...previousSlots ?? [],
-        {
-          classId: member.classId,
-          classArguments: member.classArguments ?? {}
-        }
-      ];
-      try {
-        return evaluateInitializerInContext(
+      return withConstructionGenericSlots(
+        member,
+        ctx,
+        instanceEnv,
+        () => evaluateInitializerInContext(
           init,
           member,
           ctx,
           createdValues,
           argumentValues,
           sourceValueId ?? null
-        );
-      } finally {
-        ctx.__constructionGenericSlots = previousSlots;
-      }
+        )
+      );
     };
     if (init.compiled === void 0) {
       return evaluate();
@@ -63172,6 +63214,7 @@ function evaluateInitializerInContext(init, member, ctx, createdValues, argument
   return {
     value: result.value,
     classId: rootRow.classId ?? null,
+    ...rootRow.genericBindings === void 0 ? {} : { genericBindings: { ...rootRow.genericBindings } },
     provisionalRootId: rootRow.id,
     ...constructorArgs === void 0 ? {} : { constructorArgs: structuredClone(constructorArgs) }
   };
@@ -63407,7 +63450,8 @@ function constructDeclaredClassValueWithinFrame(descriptor, record3, info, scope
         ctx,
         createdValues,
         preparedArgumentScopes,
-        classId
+        classId,
+        descriptor.instanceEnv
       ),
       constructorRoot: {
         providedSchemaKeys: new Set(
@@ -63417,6 +63461,7 @@ function constructDeclaredClassValueWithinFrame(descriptor, record3, info, scope
       }
     });
     root.classId = classId;
+    stampConstructedRootGenericBindings(root, descriptor);
   } catch (error) {
     if (error instanceof NSGetterRuntimeError) throw error;
     throw new NSGetterRuntimeError(
@@ -64752,6 +64797,7 @@ function evaluateMemberInitializer(args) {
   return {
     value: result.value,
     classId: rootRow.classId ?? null,
+    ...rootRow.genericBindings === void 0 ? {} : { genericBindings: { ...rootRow.genericBindings } },
     provisionalRootId: rootRow.id,
     ...constructorArgs === void 0 ? {} : { constructorArgs: structuredClone(constructorArgs) }
   };
@@ -64811,6 +64857,7 @@ function materializeInitializerValue(args) {
     ...envelope,
     value: evaluated.value,
     ...evaluated.classId === null ? {} : { classId: evaluated.classId },
+    ...evaluated.genericBindings === void 0 ? {} : { genericBindings: { ...evaluated.genericBindings } },
     ...evaluated.constructorArgs === void 0 ? {} : { constructorArgs: structuredClone(evaluated.constructorArgs) }
   };
   if (evaluated.provisionalRootId !== void 0) {
@@ -74550,6 +74597,13 @@ function assertProjectInterfaceDocumentValid(view, options = {}) {
   const classesById = new Map(
     classes.map((schemaClass2) => [schemaClass2.id, schemaClass2])
   );
+  const genericParamConstraintsById = new Map(
+    classes.flatMap(
+      (schemaClass2) => (schemaClass2.genericParams ?? []).map(
+        (param) => [param.id, param.constraint]
+      )
+    )
+  );
   const interfacesById = new Map(
     interfaces.map((neoInterface) => [neoInterface.id, neoInterface])
   );
@@ -75439,6 +75493,33 @@ function assertProjectInterfaceDocumentValid(view, options = {}) {
     }
     return { type: resolved.kind, required: required2 };
   };
+  const constrainedGenericTypeInfo = (typeInfo) => {
+    if (!isRecord7(typeInfo) || typeInfo.type !== 21 || typeof typeInfo.genericParamId !== "string") {
+      return null;
+    }
+    const constraint = genericParamConstraintsById.get(typeInfo.genericParamId);
+    if (!constraint) return null;
+    return constraint.kind === "class" ? {
+      type: 7,
+      required: typeInfo.required === true,
+      classId: constraint.classId
+    } : {
+      type: 8,
+      required: typeInfo.required === true,
+      enumId: constraint.enumId
+    };
+  };
+  const constrainedGenericMemberTypeInfo = (member, schemaClass2) => {
+    const resolved = substituteMemberForClass(member, schemaClass2);
+    if (resolved.kind !== 21 || typeof resolved.genericParamId !== "string") {
+      return null;
+    }
+    return constrainedGenericTypeInfo({
+      type: 21,
+      required: true,
+      genericParamId: resolved.genericParamId
+    });
+  };
   const substituteTypeInfoForClass = (typeInfo, schemaClass2) => {
     if (!isRecord7(typeInfo)) return typeInfo;
     if (typeInfo.type === 21 && typeof typeInfo.genericParamId === "string") {
@@ -75509,7 +75590,8 @@ function assertProjectInterfaceDocumentValid(view, options = {}) {
     if (resolved.kind === 13 || resolved.kind === 23) {
       return "Schema member is a Function but the interface declares a property.";
     }
-    const implementationType = memberTypeInfo(member, schemaClass2);
+    const storedType = memberTypeInfo(member, schemaClass2);
+    const implementationType = declaration.settable === true ? storedType : storedType ? constrainedGenericTypeInfo(storedType) ?? storedType : constrainedGenericMemberTypeInfo(member, schemaClass2);
     if (!implementationType) return "Schema member type could not be resolved.";
     const matches = declaration.settable === true ? invariantTypeInfo(implementationType, declaration.typeInfo) : typeInfoAssignable(implementationType, declaration.typeInfo);
     if (!matches) return "Property type does not conform.";
@@ -98877,8 +98959,13 @@ function assignPendingIds(changes, authoredValueSeeds, reconstructed3, localInit
     return fresh;
   };
   const collect = (value) => {
-    if (typeof value === "string" && isPendingId(value)) {
-      assign(value);
+    if (typeof value === "string") {
+      const interfaceOwner = pendingInterfaceMemberSymbolOwner(value);
+      if (interfaceOwner !== null) {
+        assign(interfaceOwner);
+        return;
+      }
+      if (isPendingId(value)) assign(value);
       return;
     }
     if (Array.isArray(value)) {
@@ -99032,6 +99119,18 @@ function assignPendingIds(changes, authoredValueSeeds, reconstructed3, localInit
     }
   };
 }
+function pendingInterfaceMemberSymbolOwner(value) {
+  const marker = ":member:";
+  const markerIndex = value.indexOf(marker);
+  if (markerIndex < 0) return null;
+  const owner = value.slice(0, markerIndex);
+  const parts = owner.split(":");
+  if (parts.length !== 6) return null;
+  if (parts[0] !== "__pending__" || parts[1] !== "interface") return null;
+  if (!/^\d+$/u.test(parts[3] ?? "")) return null;
+  if (!/^\d+$/u.test(parts[4] ?? "")) return null;
+  return owner;
+}
 function readAcceptedProjectVersionCommitResponse(value) {
   if (!isObjectRecord2(value) || value.kind !== "accepted") return null;
   if (typeof value.transactionId !== "string") {
@@ -101612,7 +101711,7 @@ var init_registry2 = __esm({
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
       formatVersion: 3,
       contractVersion: "3.9",
-      cliVersion: "0.26.0",
+      cliVersion: "0.26.1",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {
@@ -107920,7 +108019,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.26.0");
+    console.log("0.26.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.26.0",
+  "version": "0.26.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.26.0 -->
+<!-- reviewed-through-cli: 0.26.1 -->
 
 # Neo Compose CLI
 

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

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