@neocompose/cli 0.33.0 → 0.33.2

package/CHANGELOG.md

@@ -1,5 +1,28 @@
 # Changelog
 
+## [0.33.2] - 2026-08-19
+
+### Fixed
+
+- Preserve authored non-null generic fields while projecting the local schema
+  manifest used by `neo script check --all`, so a `T` field read remains `T`
+  instead of becoming `T?` from the storage placeholder's structural flags.
+- Compile prospective functions, properties, migrations, and ad-hoc scripts
+  against the complete project context, including declared constructors,
+  project files, and variants. `neo test` now resolves required Class-header
+  constructors instead of incorrectly falling back to their stored fields.
+
+## [0.33.1] - 2026-08-19
+
+### Fixed
+
+- Preserve an authored non-null `T` member while recompiling an open generic
+  Class from its lowered project records, so `neo test` no longer rejects
+  valid generic getters as returning `T?` during animation declaration replay.
+- Identify the actual NeoScript property and getter or setter position when
+  prospective replay compilation fails, instead of reporting only the
+  unrelated animation value that triggered whole-document validation.
+
 ## [0.33.0] - 2026-08-19
 
 ### Added

package/dist/neo.mjs

@@ -32824,7 +32824,7 @@ function memberType(member, environment, visiting, field) {
     return typeParameter(
       string2(member.genericParamId, `${field}.genericParamId`),
       environment,
-      nullable
+      false
     );
   }
   return { ...unknownType(), nullable };
@@ -61279,7 +61279,14 @@ function memberRuntimeType(record3, context, seen = /* @__PURE__ */ new Set(), g
       }
       return genericParameterLanguageType(
         genericParamId,
-        required2,
+        // A Generic schema member is a value-less placeholder, so its stored
+        // `required` is always false even when the authored slot is spelled
+        // `T`. Project-source declarations forbid `T?` member slots because
+        // the concrete binding owns storage nullability. While compiling the
+        // open declaring Class, preserve the authored `T` surface instead of
+        // turning every read into `T?`; closed receivers still take the
+        // concrete binding branch above and retain its real nullability.
+        true,
         context,
         genericEnvironment
       );
@@ -94844,7 +94851,7 @@ function replayAnimationDeclarationInitializersV4(records2, document, fallbackDo
       documentBodiesCompiled = true;
     } catch (error) {
       throw new Error(
-        `Animation declaration replay failed for value "${row.id}": ${error instanceof Error ? error.message : String(error)}`,
+        `Animation declaration replay failed for value "${row.id}": ${describePulledProjectBodyCompileError(error)}`,
         { cause: error }
       );
     }
@@ -96098,6 +96105,16 @@ var init_server_preparation_preflight = __esm({
 });
 
 // src/project-source/initializer-replay.ts
+function describePulledProjectBodyCompileError(error) {
+  const fallback = error instanceof Error ? error.message : String(error);
+  if (!(error instanceof CompileError)) return fallback;
+  const site = pulledBodyCompileSites.get(error);
+  if (site === void 0) return fallback;
+  const position = `${error.line}:${error.column}`;
+  const detailPrefix = `${position}: `;
+  const detail = error.message.startsWith(detailPrefix) ? error.message.slice(detailPrefix.length) : error.message;
+  return `Member "${site.memberName}" (${site.memberId}) ${error.unit ?? "body"} failed at ${position}: ${detail}`;
+}
 function replayStoredConstructionV4(args) {
   const document = args.document ?? readPulledProjectDocumentV4(args.records);
   const compilationProject = args.compilationProject ?? (args.compileDocumentBodies === true ? createNeoScriptCompilationProject({
@@ -96343,11 +96360,21 @@ function compilePulledProjectDocumentForEvaluationV4(document, compilationProjec
     )
   );
   for (const member of compileArgs.members) {
-    compileAuthoredMemberBodies({
-      ...compileArgs,
-      member,
-      thisClass: ownerClassByMemberId.get(member.id) ?? null
-    });
+    try {
+      compileAuthoredMemberBodies({
+        ...compileArgs,
+        member,
+        thisClass: ownerClassByMemberId.get(member.id) ?? null
+      });
+    } catch (error) {
+      if (error instanceof CompileError) {
+        pulledBodyCompileSites.set(error, {
+          memberId: member.id,
+          memberName: member.name
+        });
+      }
+      throw error;
+    }
   }
   for (const site of valueInitializerCompilationSites(document).values()) {
     if (isInitValueContent(site.row) && site.row.init.compiled !== void 0) {
@@ -96509,7 +96536,7 @@ function replayWorkspace(records2) {
     state: { records: stateRecords }
   };
 }
-var pulledValueInitializerCompilationSites;
+var pulledBodyCompileSites, pulledValueInitializerCompilationSites;
 var init_initializer_replay = __esm({
   "src/project-source/initializer-replay.ts"() {
     "use strict";
@@ -96524,6 +96551,7 @@ var init_initializer_replay = __esm({
     init_constructors2();
     init_neoscript_evaluator();
     init_compiler_adapter();
+    pulledBodyCompileSites = /* @__PURE__ */ new WeakMap();
     pulledValueInitializerCompilationSites = /* @__PURE__ */ new WeakMap();
   }
 });
@@ -107110,6 +107138,7 @@ function createCliNeoScriptContext(options) {
   const members = [...options.documents.members];
   const vm = {
     project: options.documents.project,
+    projectFiles: [...options.documents.projectFiles ?? []],
     members,
     classes: [...options.documents.classes],
     enums: [...options.documents.enums],
@@ -107129,7 +107158,9 @@ function createCliNeoScriptContext(options) {
     ...options.functionReturnTypeInfo ? { functionReturnTypeInfo: options.functionReturnTypeInfo } : {},
     ...options.functionArguments ? { functionArguments: options.functionArguments } : {},
     ...options.functionDeferred !== void 0 ? { functionDeferred: options.functionDeferred } : {},
-    ...options.functionName ? { functionName: options.functionName } : {}
+    ...options.functionName ? { functionName: options.functionName } : {},
+    ...options.documents.variants?.length ? { variants: [...options.documents.variants] } : {},
+    ...options.documents.variantFolders?.length ? { variantFolders: [...options.documents.variantFolders] } : {}
   });
   if (!options.manifest) return compilerContext;
   const projection = projectSchemaManifest(options.manifest, {
@@ -107768,11 +107799,14 @@ function readDocumentArrays(raw) {
   }
   return {
     project: raw.project,
+    projectFiles: arrayOf2("projectFiles"),
     members: arrayOf2("members"),
     classes: arrayOf2("classes"),
     constructors: arrayOf2("constructors"),
     enums: arrayOf2("enums"),
     interfaces: arrayOf2("interfaces"),
+    variants: arrayOf2("variants"),
+    variantFolders: arrayOf2("variantFolders"),
     values: arrayOf2("values"),
     dialogues: arrayOf2("dialogues"),
     dialogueRecords: arrayOf2("dialogueRecords"),
@@ -107780,7 +107814,6 @@ function readDocumentArrays(raw) {
     dialogueGroups: arrayOf2("dialogueGroups"),
     priorityGroups: arrayOf2("priorityGroups"),
     migrations: arrayOf2("migrations"),
-    projectFiles: arrayOf2("projectFiles"),
     localizationConfig: raw.localizationConfig,
     localizedTexts: arrayOf2("localizedTexts")
   };
@@ -108383,12 +108416,19 @@ async function runScript(workspace, command, options, dependencies = {}) {
     unit,
     code: source
   });
-  const compileContext = {
+  const compilerProject = {
     project: document.project,
+    projectFiles: document.projectFiles,
     members: document.members,
     classes: document.classes,
+    constructors: document.constructors,
     enums: document.enums,
     interfaces: document.interfaces,
+    variants: document.variants,
+    variantFolders: document.variantFolders
+  };
+  const compileContext = {
+    ...compilerProject,
     thisClass,
     returnTypeInfo: parseReturns(options.returns, document),
     dialogueContext: null
@@ -108437,11 +108477,7 @@ async function runScript(workspace, command, options, dependencies = {}) {
       }
       compiled = compileNeoScriptBodyOrThrow(
         () => compileNSFunction(source, {
-          project: document.project,
-          members: document.members,
-          classes: document.classes,
-          enums: document.enums,
-          interfaces: document.interfaces,
+          ...compilerProject,
           thisClass,
           returnTypeInfo: nsFunctionDefinition.returnTypeInfo,
           argumentTypes: nsFunctionDefinition.argumentTypes,
@@ -108457,11 +108493,7 @@ async function runScript(workspace, command, options, dependencies = {}) {
       }
       compiled = compileNeoScriptBodyOrThrow(
         () => compileNSSetter(source, {
-          project: document.project,
-          members: document.members,
-          classes: document.classes,
-          enums: document.enums,
-          interfaces: document.interfaces,
+          ...compilerProject,
           thisClass,
           valueTypeInfo: resolveScriptPropertyReturnTypeInfo(
             setterMember,
@@ -112370,10 +112402,27 @@ async function buildPostPushCompileSchema(workspace, status) {
   }
   return {
     project,
+    projectFiles: [...bucket("project-file").values()],
     members: [...bucket("member").values()],
     classes: [...bucket("class").values()],
+    constructors: [...bucket("constructor").values()],
     enums: [...bucket("enum").values()],
-    interfaces: [...bucket("interface").values()]
+    interfaces: [...bucket("interface").values()],
+    variants: [...bucket("variant").values()],
+    variantFolders: [...bucket("variant-folder").values()]
+  };
+}
+function postPushCompilerProject(schema) {
+  return {
+    project: schema.project,
+    projectFiles: schema.projectFiles,
+    members: schema.members,
+    classes: schema.classes,
+    constructors: schema.constructors,
+    enums: schema.enums,
+    interfaces: schema.interfaces,
+    variants: schema.variants,
+    variantFolders: schema.variantFolders
   };
 }
 function containingClass(schema, memberId) {
@@ -112405,11 +112454,7 @@ function compileNSPropertyChange(schema, memberData, locator) {
     const code = memberData.code;
     next.getter = compileNeoScriptBodyOrThrow(
       () => compileNSGetter2(code, {
-        project: schema.project,
-        members: schema.members,
-        classes: schema.classes,
-        enums: schema.enums,
-        interfaces: schema.interfaces,
+        ...postPushCompilerProject(schema),
         thisClass,
         returnTypeInfo,
         dialogueContext: null,
@@ -112426,11 +112471,7 @@ function compileNSPropertyChange(schema, memberData, locator) {
     const setterCode = memberData.setterCode;
     next.setter = compileNeoScriptBodyOrThrow(
       () => compileNSSetter2(setterCode, {
-        project: schema.project,
-        members: schema.members,
-        classes: schema.classes,
-        enums: schema.enums,
-        interfaces: schema.interfaces,
+        ...postPushCompilerProject(schema),
         thisClass,
         valueTypeInfo: returnTypeInfo,
         implicitMemberAccess: true,
@@ -112455,11 +112496,7 @@ function compileNSFunctionChange(schema, memberData, locator) {
   const thisClass = containingClass(schema, memberData.id);
   next.action = compileNeoScriptBodyOrThrow(
     () => compileNSFunction2(code, {
-      project: schema.project,
-      members: schema.members,
-      classes: schema.classes,
-      enums: schema.enums,
-      interfaces: schema.interfaces,
+      ...postPushCompilerProject(schema),
       thisClass,
       returnTypeInfo: contract.returnTypeInfo,
       argumentTypes: contract.argumentTypes,
@@ -112799,11 +112836,7 @@ function compileMigrationAction(schema, migrationData, locator) {
   const code = String(migrationData.code);
   return compileNeoScriptBodyOrThrow(
     () => compileNSVoidBody2(code, {
-      project: schema.project,
-      members: schema.members,
-      classes: schema.classes,
-      enums: schema.enums,
-      interfaces: schema.interfaces,
+      ...postPushCompilerProject(schema),
       thisClass,
       dialogueContext: null,
       migrationContext: true
@@ -112911,7 +112944,7 @@ var init_registry2 = __esm({
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
       formatVersion: 3,
       contractVersion: "3.13",
-      cliVersion: "0.33.0",
+      cliVersion: "0.33.2",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {
@@ -116784,10 +116817,14 @@ async function runPendingMigrations(workspace, client, raw, migrations, onlyRef,
     const compiled = pinned !== void 0 && pinned !== null ? pinned : compileNeoScriptBodyOrThrow(
       () => compileNSVoidBody(migrationCode, {
         project: document.project,
+        projectFiles: document.projectFiles,
         members: document.members,
         classes: document.classes,
+        constructors: document.constructors,
         enums: document.enums,
         interfaces: document.interfaces,
+        variants: document.variants,
+        variantFolders: document.variantFolders,
         thisClass,
         dialogueContext: null,
         migrationContext: true
@@ -119506,7 +119543,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.33.0");
+    console.log("0.33.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.33.0",
+  "version": "0.33.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.33.0 -->
+<!-- reviewed-through-cli: 0.33.2 -->
 
 # 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.33.0 -->
+<!-- reviewed-through-cli: 0.33.2 -->
 ```
 
 The quoted version above is checked too, so this instruction cannot go stale