@neocompose/cli 0.34.0 → 0.34.2

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## [0.34.2] - 2026-08-19
+
+### Fixed
+
+- Treat unknown top-level document fields with a JSON `null` value as absent
+  while continuing to reject unknown non-null authored data, so harmless fields
+  emitted by an older CLI do not block project commands.
+
+## [0.34.1] - 2026-08-19
+
+### Fixed
+
+- Accept the advertised positional `NeoLookupVariantFolder<TObject, TValue>`
+  constructor form and lower it identically to named `path`, `collection`, and
+  `collectionValue` arguments.
+- Preserve lookup-variant member `TValue` through the CLI/server document
+  boundary under the canonical `valueTypeInfo` field, including an open generic
+  parameter authored by an enclosing class.
+- Give `neo test` the candidate project's variants and variant folders, so
+  variants declared in the same pending push resolve alongside root values and
+  declared list indexes.
+
 ## [0.34.0] - 2026-08-19
 
 ### Added

package/dist/neo.mjs

@@ -18762,7 +18762,7 @@ var init_project_schema_contract_generated = __esm({
   "../packages/neoscript-language/src/project-schema-contract.generated.ts"() {
     "use strict";
     PROJECT_SCHEMA_MANIFEST_FORMAT_VERSION = 3;
-    PROJECT_SCHEMA_MANIFEST_CONTRACT_VERSION = "3.13";
+    PROJECT_SCHEMA_MANIFEST_CONTRACT_VERSION = "3.14";
     PROJECT_FILE_UPLOAD_BATCH_SIZE = 32;
     NEO_PROJECT_SOURCE_RECORD_CONTRACT = {
       "recordFields": {
@@ -18804,7 +18804,7 @@ var init_project_schema_contract_generated = __esm({
           "collectionValueId",
           "declaredTypeInfo",
           "targetTypeInfo",
-          "variantValueTypeInfo",
+          "valueTypeInfo",
           "dialogueGroupId",
           "code",
           "setterCode",
@@ -20934,10 +20934,20 @@ var init_project_source_construction_diagnostics = __esm({
 });
 
 // ../packages/neoscript-language/src/project-source-variants.ts
+function neoLookupVariantFolderArgument(expression, name) {
+  const index = expression.args.findIndex((_, argumentIndex) => {
+    const authoredName = expression.argumentNames?.[argumentIndex];
+    return (authoredName ?? NEO_LOOKUP_VARIANT_FOLDER_CONSTRUCTOR_ARGUMENTS[argumentIndex]?.name ?? null) === name;
+  });
+  return index < 0 ? void 0 : expression.args[index];
+}
 function neoVariantConstructionSiteCall(className, argumentNames, initializerNames) {
   if (className === NEO_LOOKUP_VARIANT_FOLDER_TYPE_NAME) {
+    const logicalNames = argumentNames.map(
+      (name, index) => name ?? NEO_LOOKUP_VARIANT_FOLDER_CONSTRUCTOR_ARGUMENTS[index]?.name ?? null
+    );
     return {
-      argumentNames: argumentNames.filter(
+      argumentNames: logicalNames.filter(
         (name) => name !== "collection" && name !== "collectionValue"
       ),
       initializerNames
@@ -21001,21 +21011,14 @@ function neoVariantFolderAnnotationSymbol(annotations) {
 function folderPath(declaration) {
   const expression = parseInitializer(declaration);
   if (expression === null || expression.kind !== "new") return null;
-  const pathIndex = expression.argumentNames?.findIndex(
-    (name) => name === "path"
-  );
-  const argument2 = expression.args[pathIndex !== void 0 && pathIndex >= 0 ? pathIndex : 0];
+  const argument2 = neoLookupVariantFolderArgument(expression, "path");
   if (argument2 === void 0 || argument2.kind !== "litString") return null;
   return argument2.value;
 }
 function folderCollectionPath(declaration) {
   const expression = parseInitializer(declaration);
   if (expression === null || expression.kind !== "new") return null;
-  const index = expression.argumentNames?.findIndex(
-    (name) => name === "collection"
-  );
-  if (index === void 0 || index < 0) return null;
-  const argument2 = expression.args[index];
+  const argument2 = neoLookupVariantFolderArgument(expression, "collection");
   if (argument2 === void 0) return null;
   const parts = [];
   let cursor = argument2;
@@ -21746,7 +21749,7 @@ function variantDiagnostic(entry, range2, code, message) {
     message
   };
 }
-var NEO_VARIANT_CONSTRUCTOR_ARGUMENTS, NEO_VARIANT_MEMBER_ARGUMENTS, dottedScope;
+var NEO_VARIANT_CONSTRUCTOR_ARGUMENTS, NEO_LOOKUP_VARIANT_FOLDER_CONSTRUCTOR_ARGUMENTS, NEO_VARIANT_MEMBER_ARGUMENTS, dottedScope;
 var init_project_source_variants = __esm({
   "../packages/neoscript-language/src/project-source-variants.ts"() {
     "use strict";
@@ -21760,6 +21763,11 @@ var init_project_source_variants = __esm({
       "overrides",
       "childOverrides"
     ];
+    NEO_LOOKUP_VARIANT_FOLDER_CONSTRUCTOR_ARGUMENTS = [
+      { name: "path", type: "string" },
+      { name: "collection", type: "List<TValue>" },
+      { name: "collectionValue", type: "Reference<List<TValue>>" }
+    ];
     NEO_VARIANT_MEMBER_ARGUMENTS = /* @__PURE__ */ new Map([
       ["overrides", "Overrides"],
       ["childOverrides", "ChildOverrides"]
@@ -23847,7 +23855,8 @@ function validateExpression(expression, expected, scope, environment, uri, range
     ) ?? [];
     const overload = declaration && declaration.constructors.length > 0 ? matchingDeclaredConstructor(declaration, expression.argumentNames) : null;
     expression.args.forEach((argument2, index) => {
-      const named = expression.argumentNames?.[index];
+      const authoredName = expression.argumentNames?.[index];
+      const named = declaration.name === NEO_LOOKUP_VARIANT_FOLDER_TYPE_NAME ? authoredName ?? NEO_LOOKUP_VARIANT_FOLDER_CONSTRUCTOR_ARGUMENTS[index]?.name : authoredName;
       if (declaration.name === NEO_LOOKUP_VARIANT_FOLDER_TYPE_NAME && named === "collection") {
         return;
       }
@@ -29003,11 +29012,7 @@ function projectConstructorArgumentCompletions(analysis, document, position) {
 }
 function lookupVariantFolderArgumentFields(typeName) {
   if (typeName !== NEO_LOOKUP_VARIANT_FOLDER_TYPE_NAME) return null;
-  return [
-    { name: "path", type: "string" },
-    { name: "collection", type: "List<TValue>" },
-    { name: "collectionValue", type: "Reference<List<TValue>>" }
-  ];
+  return NEO_LOOKUP_VARIANT_FOLDER_CONSTRUCTOR_ARGUMENTS;
 }
 function declaredConstructorNamedArguments(analysis, typeName, used) {
   const contract = classConstructionContract(
@@ -34866,7 +34871,7 @@ var init_document_contracts = __esm({
           "collectionValueId",
           "declaredTypeInfo",
           "targetTypeInfo",
-          "variantValueTypeInfo",
+          "valueTypeInfo",
           "dialogueGroupId",
           "code",
           "setterCode",
@@ -34905,6 +34910,7 @@ var init_document_contracts = __esm({
           collectionValueId: "missingIsNullOnFullMember",
           declaredTypeInfo: "nullIsAbsent",
           targetTypeInfo: "nullIsAbsent",
+          valueTypeInfo: "nullIsAbsent",
           dialogueGroupId: "missingIsNullOnFullMember",
           templateId: "missingIsNullOnFullMember",
           setterCode: "nullIsAbsent",
@@ -35166,8 +35172,9 @@ function fieldDisposition(recordKind, field) {
   return null;
 }
 function assertKnownDocumentFields(recordKind, value) {
-  for (const field of Object.keys(value)) {
+  for (const [field, fieldValue] of Object.entries(value)) {
     if (fieldDisposition(recordKind, field) !== null) continue;
+    if (fieldValue === null) continue;
     throw new SchemaDocumentContractError(recordKind, field);
   }
 }
@@ -35182,6 +35189,7 @@ function partitionDocumentFields(recordKind, value) {
     if (fieldValue === void 0) continue;
     const disposition = fieldDisposition(recordKind, field);
     if (disposition === null) {
+      if (fieldValue === null) continue;
       throw new SchemaDocumentContractError(recordKind, field);
     }
     result[disposition][field] = fieldValue;
@@ -35192,6 +35200,11 @@ function normalizeDocumentFields(recordKind, value) {
   assertKnownDocumentFields(recordKind, value);
   const contract = SCHEMA_DOCUMENT_FIELD_CONTRACTS[recordKind];
   const normalized = deepOmitUndefined(value);
+  for (const [field, fieldValue] of Object.entries(normalized)) {
+    if (fieldValue === null && fieldDisposition(recordKind, field) === null) {
+      delete normalized[field];
+    }
+  }
   for (const [field, rule] of Object.entries(contract.normalization)) {
     const current = normalized[field];
     if (rule === "nullIsAbsent") {
@@ -36335,10 +36348,10 @@ function memberFromDocument(record3, members, owners, classNames, context) {
         record3,
         "targetTypeInfo"
       ),
-      valueType: field("variantValueTypeInfo", null) === null ? null : documentTypeToManifest(
-        field("variantValueTypeInfo", void 0),
+      valueType: field("valueTypeInfo", null) === null ? null : documentTypeToManifest(
+        field("valueTypeInfo", void 0),
         record3,
-        "variantValueTypeInfo"
+        "valueTypeInfo"
       )
     };
   }
@@ -36488,7 +36501,7 @@ function memberToDocumentFields(member, baseData3) {
   }
   if (member.kind === "variant") {
     fields.targetTypeInfo = manifestTypeToDocument(member.targetType);
-    fields.variantValueTypeInfo = member.valueType === null ? null : manifestTypeToDocument(member.valueType);
+    fields.valueTypeInfo = member.valueType === null ? null : manifestTypeToDocument(member.valueType);
   }
   if (member.kind === "generic") {
     fields.genericParamId = member.genericParamId;
@@ -39692,10 +39705,10 @@ function assertMember2(value, path) {
         member.valueType,
         `${path}.valueType`
       ).kind;
-      if (valueKind !== "class") {
+      if (valueKind !== "class" && valueKind !== "generic") {
         invalid(
           `${path}.valueType.kind`,
-          "a lookup variant member's TValue must be a collection entry class."
+          "a lookup variant member's TValue must be a collection entry class or an open generic parameter."
         );
       }
     }
@@ -44154,6 +44167,11 @@ function isMemberVariantBase(value) {
   if (v.targetTypeInfo.type !== 7 /* Class */ && v.targetTypeInfo.type !== 21 /* Generic */) {
     return false;
   }
+  if (v.valueTypeInfo !== void 0 && v.valueTypeInfo !== null && (!isNSTypeInfo(v.valueTypeInfo) || // P68's concrete folder binding resolves to a Class, while an unclosed
+  // owner class can still spell the same TValue as its Generic parameter.
+  v.valueTypeInfo.type !== 7 /* Class */ && v.valueTypeInfo.type !== 21 /* Generic */)) {
+    return false;
+  }
   const defaultValue = v.defaultValue;
   if (defaultValue === void 0 || defaultValue === null) return true;
   if (!isMemberValueBase(defaultValue)) return false;
@@ -60262,13 +60280,11 @@ function folderBinding(context, global) {
       `Lookup variant folder ${global.name} must be declared with new(...).`
     );
   }
-  const named = /* @__PURE__ */ new Map();
-  expression.args.forEach((argument2, index) => {
-    const name = expression.argumentNames?.[index];
-    if (name !== void 0 && name !== null) named.set(name, argument2);
-  });
-  const collection = named.get("collection");
-  const collectionValue = named.get("collectionValue");
+  const collection = neoLookupVariantFolderArgument(expression, "collection");
+  const collectionValue = neoLookupVariantFolderArgument(
+    expression,
+    "collectionValue"
+  );
   const collectionPath = collection === void 0 ? null : dottedPath(collection);
   if (collectionPath === null) {
     throw new Error(
@@ -113164,8 +113180,8 @@ var init_registry2 = __esm({
     "use strict";
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
       formatVersion: 3,
-      contractVersion: "3.13",
-      cliVersion: "0.34.0",
+      contractVersion: "3.14",
+      cliVersion: "0.34.2",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {
@@ -113207,7 +113223,7 @@ var init_registry2 = __esm({
             "collectionValueId",
             "declaredTypeInfo",
             "targetTypeInfo",
-            "variantValueTypeInfo",
+            "valueTypeInfo",
             "dialogueGroupId",
             "code",
             "setterCode",
@@ -113246,6 +113262,7 @@ var init_registry2 = __esm({
             collectionValueId: "missingIsNullOnFullMember",
             declaredTypeInfo: "nullIsAbsent",
             targetTypeInfo: "nullIsAbsent",
+            valueTypeInfo: "nullIsAbsent",
             dialogueGroupId: "missingIsNullOnFullMember",
             templateId: "missingIsNullOnFullMember",
             setterCode: "nullIsAbsent",
@@ -114652,6 +114669,8 @@ function sharedEvaluatorBase(rawDocument) {
     enums: document.enums,
     interfaces: document.interfaces,
     constructors,
+    variants: document.variants,
+    variantFolders: document.variantFolders,
     projectFiles: document.projectFiles,
     localizationConfig: document.localizationConfig,
     localizedTexts: document.localizedTexts,
@@ -119764,7 +119783,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.34.0");
+    console.log("0.34.2");
     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.34.0",
+  "version": "0.34.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.34.0 -->
+<!-- reviewed-through-cli: 0.34.2 -->
 
 # Neo Compose CLI
 
@@ -47,8 +47,9 @@ Treat pull followed immediately by status or dry-run as a semantic no-op. If it
 reports project changes, stop and report a round-trip bug instead of pushing.
 
 The compiler reports file:line:column diagnostics and fails closed. Unknown
-authorable server fields require a CLI contract update; never preserve them as
-opaque JSON or guessed syntax.
+non-null authorable server fields require a CLI contract update; never preserve
+them as opaque JSON or guessed syntax. An unknown top-level document field whose
+value is exactly JSON `null` is treated as absent.
 
 ## Honor hard authoring invariants
 

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.34.0 -->
+<!-- reviewed-through-cli: 0.34.2 -->
 ```
 
 The quoted version above is checked too, so this instruction cannot go stale