@neocompose/cli 0.68.3 → 0.68.4

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## [0.68.4] - 2026-10-05
+
+- An empty header base clause such as `class Child(string name) : Base() { Name = name } {}` uses the base's member initializers when its declared constructors require arguments. The base block must settle remaining required members. Zero-argument and fully defaulted base constructors still run; required base constructors and declared `: base()` calls still require valid arguments. Fixes [#1186](https://github.com/ryanbliss/neo-compose/issues/1186). Use SDK 0.57.4 for matching Unity runtime behavior.
+
 ## [0.68.3] - 2026-10-05
 
 - `neo push --dry-run` preserves valid authored UUID-v4 IDs on new nested member and dictionary values, including values named by local declarations. Previously, verification dropped the ID and failed with an internal "has no pending id assignment" error. Status and push report malformed new member-value IDs at the annotation, name the affected `Class.Member`, and suggest removing `@id` or providing an unused UUID v4 ([#1189](https://github.com/ryanbliss/neo-compose/issues/1189)).

package/dist/neo.mjs

@@ -22862,6 +22862,11 @@ var init_project_source_variants = __esm({
 });
 
 // ../packages/neoscript-language/src/project-source-settlement.ts
+function requiredConstructorParameters(declaration) {
+  if (declaration.headerParameters !== void 0)
+    return declaration.headerParameters;
+  return declaration.initBlocks.length > 0 ? [] : void 0;
+}
 function buildNeoConstructionIndex(documents) {
   const classes = /* @__PURE__ */ new Map();
   const interfaceNames = /* @__PURE__ */ new Set();
@@ -22933,8 +22938,9 @@ function unsettledRequiredMembers(index, className, settledNames, typeArguments
 function classConstructionContract(index, className) {
   const declaration = index.classes.get(className)?.declaration;
   if (!declaration) return null;
-  if (declaration.headerParameters !== void 0) {
-    return { kind: "required", parameters: declaration.headerParameters };
+  const parameters = requiredConstructorParameters(declaration);
+  if (parameters !== void 0) {
+    return { kind: "required", parameters };
   }
   if (declaration.constructors.length > 0) {
     return { kind: "declared", constructors: declaration.constructors };
@@ -22998,7 +23004,7 @@ function validateBaseConstruction(index, entry, diagnostics) {
   }
   for (const diagnostic of validateNeoConstructionSite(index, {
     className: baseClause.type.name,
-    site: "base",
+    site: "headerBase",
     typeArguments: baseClause.type.typeArguments.map(typeShape),
     argumentNames: (baseClause.arguments ?? []).map(
       (argument2) => argument2.name ?? null
@@ -23024,11 +23030,10 @@ function validateBaseConstruction(index, entry, diagnostics) {
   }
 }
 function validateUnconstructedBase(entry, base, range2, diagnostics) {
-  const parameters = base.declaration.headerParameters;
+  const parameters = requiredConstructorParameters(base.declaration);
   if (parameters === void 0) {
-    if (entry.declaration.headerParameters === void 0 && entry.declaration.initBlocks.length === 0)
-      return;
-    pushBaseCall(entry, base, [], () => range2, diagnostics);
+    if (requiredConstructorParameters(entry.declaration) === void 0) return;
+    pushBaseCall(entry, base, [], () => range2, diagnostics, "headerBase");
     return;
   }
   if (parameterListIsFullyDefaulted(parameters)) return;
@@ -23047,7 +23052,7 @@ function validateUnconstructedBase(entry, base, range2, diagnostics) {
 function validateDeclaredBaseCalls(entry, base, diagnostics) {
   for (const constructor2 of entry.declaration.constructors) {
     const baseArguments = constructor2.baseArguments ?? [];
-    if (constructor2.baseArguments === void 0 && base.declaration.headerParameters !== void 0) {
+    if (constructor2.baseArguments === void 0 && requiredConstructorParameters(base.declaration) !== void 0) {
       continue;
     }
     pushBaseCall(
@@ -23062,13 +23067,13 @@ function validateDeclaredBaseCalls(entry, base, diagnostics) {
     );
   }
 }
-function pushBaseCall(entry, base, argumentNames, rangeOf, diagnostics) {
+function pushBaseCall(entry, base, argumentNames, rangeOf, diagnostics, site = "base") {
   const siteDiagnostics = [];
   resolveInvokedConstructor(
     base.declaration,
     {
       className: base.declaration.name,
-      site: "base",
+      site,
       argumentNames,
       initializerNames: []
     },
@@ -23127,7 +23132,7 @@ function validateDeclaredConstructionPaths(index, entry, diagnostics) {
     []
   ).filter((member) => owned || member.declaringClassName === declaration.name);
   if (outstanding.length === 0) return;
-  if (declaration.headerParameters !== void 0) {
+  if (requiredConstructorParameters(declaration) !== void 0) {
     const initBlock = declaration.initBlocks[0];
     const assigned = assignedOnEveryPath(initBlock?.body.text ?? "{}");
     const unsettled = filterUnassigned(outstanding, assigned);
@@ -23168,13 +23173,30 @@ function filterUnassigned(members, assigned) {
 }
 function declarationSettledMembers(index, entry) {
   const settled = /* @__PURE__ */ new Set();
-  const baseClause = entry.declaration.baseClause;
-  if (!baseClause) return settled;
-  for (const field of baseClause.initializerFields ?? []) {
+  const declaration = entry.declaration;
+  const baseClause = declaration.baseClause;
+  for (const field of baseClause?.initializerFields ?? []) {
     settled.add(field.name.toLowerCase());
   }
-  if ((baseClause.arguments ?? []).length === 0) return settled;
-  for (const member of requiredMembersForClass(index, baseClause.type.name)) {
+  if (baseClause === void 0 && requiredConstructorParameters(declaration) === void 0)
+    return settled;
+  const baseName = baseClause?.type.name ?? declaration.baseTypes[0]?.name;
+  const base = baseName === void 0 ? void 0 : index.classes.get(baseName);
+  if (base === void 0) return settled;
+  if (!resolveInvokedConstructor(
+    base.declaration,
+    {
+      className: base.declaration.name,
+      site: "headerBase",
+      argumentNames: (baseClause?.arguments ?? []).map(
+        (argument2) => argument2.name ?? null
+      ),
+      initializerNames: []
+    },
+    []
+  ))
+    return settled;
+  for (const member of requiredMembersForClass(index, base.declaration.name)) {
     settled.add(member.name.toLowerCase());
   }
   return settled;
@@ -23192,7 +23214,7 @@ function baseClauseRange(declaration, anchor) {
   return baseClause.range;
 }
 function resolveInvokedConstructor(declaration, call, diagnostics) {
-  const headerParameters = declaration.headerParameters;
+  const headerParameters = requiredConstructorParameters(declaration);
   if (headerParameters !== void 0) {
     if (call.argumentNames.length > 0) {
       validateArgumentsAgainstParameters(
@@ -23228,7 +23250,7 @@ function resolveInvokedConstructor(declaration, call, diagnostics) {
     const parameterless = declaration.constructors.some(
       (constructor2) => parameterListIsFullyDefaulted(constructor2.parameters)
     );
-    if (parameterless || call.site === "new") return parameterless;
+    if (parameterless || call.site !== "base") return parameterless;
     diagnostics.push({
       code: "base-requires-construction",
       message: `The base call must pass arguments: '${declaration.name}' declares no parameterless constructor \u2014 candidates: ${describeConstructors(declaration)}.`,
@@ -23245,7 +23267,7 @@ function resolveInvokedConstructor(declaration, call, diagnostics) {
     );
     return true;
   }
-  if (call.site === "base") {
+  if (call.site !== "new") {
     let positional = false;
     call.argumentNames.forEach((name, index) => {
       if (name !== null) return;
@@ -31690,7 +31712,7 @@ var init_registry = __esm({
   "../packages/manifest/schema-contract/registry.mjs"() {
     "use strict";
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
-      cliVersion: "0.68.3",
+      cliVersion: "0.68.4",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {
@@ -79315,6 +79337,8 @@ function resolveBaseConstructorRecord(record6, ctx) {
   );
   if (resolution.kind === "resolved") return resolution.overload;
   if (resolution.kind === "noMatch" && baseArguments.length === 0) {
+    if (owningClass.requiredConstructorId === record6.id && evaluatorSchemaIndex(ctx).class(baseClassId)?.requiredConstructorId == null)
+      return null;
     throw new NSGetterRuntimeError(
       `Constructor '${record6.id}' on '${owningClass.name}' must call ': base(...)': its base class declares constructors but none parameterless.`
     );
@@ -88861,6 +88885,8 @@ function resolveBaseConstructorOverload(args, owner) {
   );
   if (resolution.kind === "resolved") return resolution.overload;
   if (resolution.kind === "noMatch" && baseArguments.length === 0) {
+    if (owner.requiredConstructorId === args.constructor.id && baseClass.requiredConstructorId == null)
+      return null;
     throw new Error(
       `Constructor "${args.constructor.id}" must declare a base call: base class "${baseClass.name}" declares constructors but none parameterless \u2014 candidates: ${describeConstructorOverloads(candidates)}.`
     );
@@ -165112,7 +165138,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.68.3");
+    console.log("0.68.4");
     return;
   }
   if (args.command === null || args.command === "help" || args.command === "--help" || args.command === "-h") {

package/dist/test-worker.mjs

@@ -22401,6 +22401,8 @@ function resolveBaseConstructorRecord(record3, ctx) {
   );
   if (resolution.kind === "resolved") return resolution.overload;
   if (resolution.kind === "noMatch" && baseArguments.length === 0) {
+    if (owningClass.requiredConstructorId === record3.id && evaluatorSchemaIndex(ctx).class(baseClassId)?.requiredConstructorId == null)
+      return null;
     throw new NSGetterRuntimeError(
       `Constructor '${record3.id}' on '${owningClass.name}' must call ': base(...)': its base class declares constructors but none parameterless.`
     );

package/package.json

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

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

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

package/skills/neocompose-cli/references/declarations-and-construction.md

@@ -385,6 +385,14 @@ class Child(string name) : Base {
 }
 ```
 
+A header base clause with no arguments, such as `: Base()` or `: Base`,
+runs a zero-argument or fully defaulted base constructor when available.
+Otherwise, if the base has only declared constructors, it uses the member
+initializers and base block without running a base constructor. The base block
+must settle any remaining required members. A required base constructor cannot
+be skipped. A declared constructor's `: base()` must call a base constructor
+when the base declares one.
+
 Bind generic base arguments explicitly when needed. Reject a subclass that
 omits required base arguments or leaves any inherited required member
 unsettled.