@neocompose/cli 0.57.2 → 0.57.3

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [0.57.3] - 2026-09-24
+
+- Status, pull, and push no longer copy a shared row passed to a constructor when they check a stored construction. Before, a constructor argument that named a shared row, such as a schedule event's `area`, copied that row's whole graph for every construction. Once an area grid held a few placed characters, this failed with "NeoScript constructed Session row limit of 4096 exceeded."
+- Dry run and push no longer read a shared constructor argument's current graph when they check a stored construction. Before, a push that deleted a variant still placed in that graph, such as a character in the area a schedule event names, failed with "Variant '…' does not exist in this project."
+- `neo dialogue dryrun` evaluates conditions that have the dialogue context in scope. Before, every such condition failed with "NeoScript getter expected 1 argument(s), got 0."
+
 ## [0.57.2] - 2026-09-23
 
 - Removing a variant's last `overrides:` argument deletes its stored override row. Dry run and push accept the edit, including variants whose initializer is an inline delegate.

package/dist/neo.mjs

@@ -72067,7 +72067,9 @@ function evaluateNSGetterWithEffects(getter, ctx, argumentValues = [], options =
   const writes = runtimeCtx.__executionState?.writes ?? requestedWrites;
   const creationCheckpoint = sessionCreationCheckpoint(runtimeCtx);
   const scope = createTopLevelScope(runtimeCtx);
-  const parameters = getter.parameters.slice(2);
+  const parameters = getter.parameters.slice(
+    getter.parameters[2]?.id === DIALOGUE_CONTEXT_PARAMETER_ID ? 3 : 2
+  );
   if (parameters.length !== argumentValues.length) {
     throw new NSGetterRuntimeError(
       `NeoScript getter expected ${parameters.length} argument(s), got ${argumentValues.length}.`
@@ -72942,7 +72944,7 @@ function createTopLevelScope(ctx) {
   scope.setLocal("__this__", ctx.thisValue);
   scope.setLocal("__root__", ctx.rootValue);
   scope.setLocal(
-    "__context__",
+    DIALOGUE_CONTEXT_PARAMETER_ID,
     ctx.dialogueContext ?? {
       primary: null,
       trigger: null
@@ -81208,7 +81210,7 @@ function collectionEntryUsesReferenceResolution(collection, key, ctx) {
   );
   return referenceIndexes === void 0 || referenceIndexes.has(key);
 }
-var DELEGATE_LEXICAL_THIS, DELEGATE_LEXICAL_ROOT, NonCatchableNSGetterRuntimeError, NativeFunctionDelegateUnavailableError, CorruptNeoScriptIRError, NeoScriptResourceLimitError, NeoScriptWallClockTimeoutError, DEFAULT_NEO_SCRIPT_EXECUTION_BUDGET_LIMITS, liveListIndexesByProject, evaluatorOwnershipCachesByBase, MAX_CONSTRUCTION_DEPTH, MAX_LOOP_ITERATIONS, resolutionCacheByMembers, NO_SCHEMA_REVISION, LazyValueOverlay, READONLY_FOREACH_BINDING_ERROR, READONLY_CATCH_BINDING_ERROR, NS_MATH_OP_NAMES, DOUBLE_INTEGRAL_MAGNITUDE, EMPTY_SUPPLIED_CONSTRUCTOR_FIELDS;
+var DELEGATE_LEXICAL_THIS, DELEGATE_LEXICAL_ROOT, NonCatchableNSGetterRuntimeError, NativeFunctionDelegateUnavailableError, CorruptNeoScriptIRError, NeoScriptResourceLimitError, NeoScriptWallClockTimeoutError, DEFAULT_NEO_SCRIPT_EXECUTION_BUDGET_LIMITS, liveListIndexesByProject, evaluatorOwnershipCachesByBase, MAX_CONSTRUCTION_DEPTH, MAX_LOOP_ITERATIONS, resolutionCacheByMembers, NO_SCHEMA_REVISION, LazyValueOverlay, READONLY_FOREACH_BINDING_ERROR, READONLY_CATCH_BINDING_ERROR, DIALOGUE_CONTEXT_PARAMETER_ID, NS_MATH_OP_NAMES, DOUBLE_INTEGRAL_MAGNITUDE, EMPTY_SUPPLIED_CONSTRUCTOR_FIELDS;
 var init_evaluateNSGetter = __esm({
   "../src/runtime/neoscript/evaluateNSGetter.ts"() {
     "use strict";
@@ -81377,6 +81379,7 @@ var init_evaluateNSGetter = __esm({
     };
     READONLY_FOREACH_BINDING_ERROR = "Cannot assign to a read-only foreach iterator binding.";
     READONLY_CATCH_BINDING_ERROR = "Cannot assign to a read-only catch message binding.";
+    DIALOGUE_CONTEXT_PARAMETER_ID = "__context__";
     NS_MATH_OP_NAMES = {
       min: "Min",
       max: "Max",
@@ -117187,12 +117190,13 @@ function materializedInitializerReconciliationFailuresV4(args) {
     const replay = replayRows.get(reconciliation.valueId);
     const replayArgs = replay !== void 0 && isLiteralValueContent(replay) ? replay.constructorArgs : void 0;
     const currentRoot = currentRows.get(reconciliation.valueId);
+    const [storedArgs, replayedArgs] = isObjectRecord2(replayArgs) ? withoutSharedArguments(reconciliation.storedConstructorArgs, replayArgs) : [reconciliation.storedConstructorArgs, replayArgs];
     const currentNormalized = currentRoot === void 0 ? void 0 : currentReader.normalizeConstructorArguments(
-      reconciliation.storedConstructorArgs,
+      storedArgs,
       currentRoot,
       owner
     );
-    const replayNormalized = replay === void 0 || !isObjectRecord2(replayArgs) ? void 0 : currentReader.withRowOverrides(replayRows).normalizeConstructorArguments(replayArgs, replay, owner);
+    const replayNormalized = replay === void 0 || !isObjectRecord2(replayedArgs) ? void 0 : currentReader.withRowOverrides(replayRows).normalizeConstructorArguments(replayedArgs, replay, owner);
     const argumentsMatch = isObjectRecord2(replayArgs) && currentRoot !== void 0 && replay !== void 0 && effectiveValuesEqual(currentNormalized, replayNormalized);
     if (argumentsMatch) {
       continue;
@@ -117213,6 +117217,16 @@ function materializedInitializerReconciliationFailuresV4(args) {
   }
   return failures;
 }
+function withoutSharedArguments(stored, replayed) {
+  const storedRest = { ...stored };
+  const replayedRest = { ...replayed };
+  for (const [slot, value] of Object.entries(stored)) {
+    if (typeof value !== "string" || replayed[slot] !== value) continue;
+    delete storedRest[slot];
+    delete replayedRest[slot];
+  }
+  return [storedRest, replayedRest];
+}
 function reconciliationRecords(workspace, changes, seeds = /* @__PURE__ */ new Map()) {
   return prospectiveInitializerRecordsV4({
     workspace,
@@ -127418,7 +127432,13 @@ function storedConstructorArgumentSource(context, type, value, environment, visi
         );
       }
       if (cloneAggregateArguments) {
-        return storedAggregateCloneSource(context, type, value, environment);
+        return storedAggregateReplaySource(
+          context,
+          type,
+          value,
+          environment,
+          constructorOwnsGraph
+        );
       }
       return storedTypedRowSource(
         context,
@@ -127661,14 +127681,18 @@ function requiredGenericBindingMemberId(environment, genericParamId) {
   }
   return bindingId;
 }
-function storedAggregateCloneSource(context, type, valueId, environment) {
+function storedAggregateReplaySource(context, type, valueId, environment, constructorOwnsGraph) {
   if (!context.values.has(valueId)) {
     throw new Error(
       `Stored ${type.kind} constructor argument references missing value row ${valueId}.`
     );
   }
   const typeName = storedSchemaTypeName(context, type, environment, valueId);
-  return `Reference<${typeName}>(id: ${quote5(valueId)}).Clone()`;
+  const reference2 = `Reference<${typeName}>(id: ${quote5(valueId)})`;
+  return isSharedConstructorArgument(context, valueId, constructorOwnsGraph) ? reference2 : `${reference2}.Clone()`;
+}
+function isSharedConstructorArgument(context, valueId, constructorOwnsGraph) {
+  return !constructorOwnsGraph && context.readMaterializedGraph().independentlyOwnedValueIds.has(valueId);
 }
 function storedSchemaTypeName(context, type, environment, valueId) {
   switch (type.kind) {
@@ -127758,7 +127782,7 @@ function storedTypedRowSource(context, type, valueId, environment, visited, writ
       `Stored ${type.kind} constructor argument references missing value row ${valueId}.`
     );
   }
-  if (!constructorOwnsGraph && context.readMaterializedGraph().independentlyOwnedValueIds.has(valueId)) {
+  if (isSharedConstructorArgument(context, valueId, constructorOwnsGraph)) {
     return `Reference<${storedSchemaTypeName(context, type, environment, valueId)}>(id: ${quote5(valueId)})`;
   }
   if (context.activeStoredConstructorArguments.has(valueId)) {
@@ -140421,7 +140445,7 @@ var init_registry2 = __esm({
   "schema-contract/registry.mjs"() {
     "use strict";
     PROJECT_SCHEMA_CONTRACT = Object.freeze({
-      cliVersion: "0.57.2",
+      cliVersion: "0.57.3",
       projectFileUploadBatchSize: 32,
       documentRecords: {
         member: {
@@ -148242,7 +148266,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.57.2");
+    console.log("0.57.3");
     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.57.2",
+  "version": "0.57.3",
   "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.57.2 -->
+<!-- reviewed-through-cli: 0.57.3 -->
 
 # Neo Compose CLI
 

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

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