@caupulican/pi-adaptative 0.86.2 → 0.86.5

package/dist/modes/interactive/components/settings-selector.js

@@ -1,6 +1,7 @@
 import { Container, getCapabilities, getKeybindings, Input, SelectList, SettingsList, Spacer, Text, } from "@caupulican/pi-tui";
+import { DEFAULT_WORKER_FLEET_LIMITS } from "../../../core/delegation/worker-fleet-limits.js";
 import { formatHttpIdleTimeoutMs, HTTP_IDLE_TIMEOUT_CHOICES } from "../../../core/http-dispatcher.js";
-import { DEFAULT_AUTONOMY_GOAL_AUTO_CONTINUE, DEFAULT_AUTONOMY_GOAL_AUTO_CONTINUE_DELAY_MS, DEFAULT_AUTONOMY_GOAL_CONTINUE_MAX_WALL_CLOCK_MINUTES, DEFAULT_AUTONOMY_GOAL_CONTINUE_TURNS, DEFAULT_AUTONOMY_MAX_STALL_TURNS, DEFAULT_RESEARCH_LANE_ENABLED, DEFAULT_RESEARCH_LANE_IDLE_DELAY_MS, DEFAULT_RESEARCH_LANE_MAX_RUNS_PER_SESSION, DEFAULT_RESEARCH_LANE_MAX_USD, DEFAULT_RESEARCH_LANE_MAX_WALL_CLOCK_MS, DEFAULT_WORKER_DELEGATION_ENABLED, DEFAULT_WORKER_DELEGATION_MAX_USD, DEFAULT_WORKER_DELEGATION_MAX_WALL_CLOCK_MS, DEFAULT_WORKER_DELEGATION_WRITE_ENABLED, } from "../../../core/settings-manager.js";
+import { DEFAULT_AUTONOMY_GOAL_AUTO_CONTINUE, DEFAULT_AUTONOMY_GOAL_AUTO_CONTINUE_DELAY_MS, DEFAULT_AUTONOMY_GOAL_CONTINUE_MAX_WALL_CLOCK_MINUTES, DEFAULT_AUTONOMY_GOAL_CONTINUE_TURNS, DEFAULT_AUTONOMY_MAX_STALL_TURNS, DEFAULT_RESEARCH_LANE_ENABLED, DEFAULT_RESEARCH_LANE_IDLE_DELAY_MS, DEFAULT_RESEARCH_LANE_MAX_RUNS_PER_SESSION, DEFAULT_RESEARCH_LANE_MAX_USD, DEFAULT_RESEARCH_LANE_MAX_WALL_CLOCK_MS, DEFAULT_WORKER_DELEGATION_ENABLED, DEFAULT_WORKER_DELEGATION_MAX_CONCURRENT, DEFAULT_WORKER_DELEGATION_MAX_USD, DEFAULT_WORKER_DELEGATION_MAX_WALL_CLOCK_MS, DEFAULT_WORKER_DELEGATION_WRITE_ENABLED, MAX_WORKER_DELEGATION_MAX_USD, } from "../../../core/settings-manager.js";
 import { getSelectListTheme, getSettingsListTheme, theme } from "../theme/theme.js";
 import { DynamicBorder } from "./dynamic-border.js";
 import { keyDisplayText } from "./keybinding-hints.js";
@@ -738,6 +739,7 @@ class WorkerDelegationSettingsSubmenu extends SettingsListSubmenu {
         this.state = {
             ...settings,
             enabled: settings.enabled ?? DEFAULT_WORKER_DELEGATION_ENABLED,
+            maxConcurrent: settings.maxConcurrent ?? DEFAULT_WORKER_DELEGATION_MAX_CONCURRENT,
             maxUsd: settings.maxUsd ?? DEFAULT_WORKER_DELEGATION_MAX_USD,
             maxWallClockMs: settings.maxWallClockMs ?? DEFAULT_WORKER_DELEGATION_MAX_WALL_CLOCK_MS,
         };
@@ -762,14 +764,15 @@ class WorkerDelegationSettingsSubmenu extends SettingsListSubmenu {
                 label: "Budget per agent tree",
                 description: "Cumulative USD budget shared by a root agent and every descendant; 0 disables this ceiling",
                 currentValue: String(this.state.maxUsd),
-                submenu: (_currentValue, done) => new TextInputSubmenu("Agent Tree USD Budget", "Enter any nonnegative finite number. Zero disables the settings-level cost ceiling.", String(this.state.maxUsd), (value) => {
+                submenu: (_currentValue, done) => new TextInputSubmenu("Agent Tree USD Budget", `Enter a nonnegative finite number no greater than ${MAX_WORKER_DELEGATION_MAX_USD}. Zero disables the settings-level cost ceiling.`, String(this.state.maxUsd), (value) => {
                     const maxUsd = Number(value.trim());
-                    if (Number.isFinite(maxUsd) && maxUsd >= 0) {
+                    if (Number.isFinite(maxUsd) && maxUsd >= 0 && maxUsd <= MAX_WORKER_DELEGATION_MAX_USD) {
                         this.state = { ...this.state, maxUsd };
-                        onChange({ ...this.state }, this.scope);
+                        done(String(maxUsd));
+                        return;
                     }
-                    done(String(this.state.maxUsd));
-                }, () => done(), "nonnegative number; invalid input keeps the current value"),
+                    done();
+                }, () => done(), `number in [0, ${MAX_WORKER_DELEGATION_MAX_USD}]; invalid input keeps the current value`),
             },
             {
                 id: "worker-delegation-max-wall-clock-ms",
@@ -780,9 +783,10 @@ class WorkerDelegationSettingsSubmenu extends SettingsListSubmenu {
                     const maxWallClockMs = Number(value.trim());
                     if (Number.isSafeInteger(maxWallClockMs) && maxWallClockMs >= 0) {
                         this.state = { ...this.state, maxWallClockMs };
-                        onChange({ ...this.state }, this.scope);
+                        done(String(maxWallClockMs));
+                        return;
                     }
-                    done(String(this.state.maxWallClockMs));
+                    done();
                 }, () => done(), "nonnegative safe integer; invalid input keeps the current value"),
             },
             {
@@ -795,15 +799,16 @@ class WorkerDelegationSettingsSubmenu extends SettingsListSubmenu {
             {
                 id: "worker-delegation-max-concurrent",
                 label: "Max concurrent workers",
-                description: "Global scheduler concurrency; delegation depth and fan-out have no framework cap",
-                currentValue: String(this.state.maxConcurrent ?? 1),
-                submenu: (_currentValue, done) => new TextInputSubmenu("Global Worker Concurrency", "Enter any positive safe integer. This limits simultaneously running agents, not tree depth or fan-out.", String(this.state.maxConcurrent ?? 1), (value) => {
+                description: `Global scheduler concurrency; fixed fleet caps are depth ${DEFAULT_WORKER_FLEET_LIMITS.maxDepth}, ${DEFAULT_WORKER_FLEET_LIMITS.maxChildrenPerAgent} children/agent, and ${DEFAULT_WORKER_FLEET_LIMITS.maxAgentsPerSession} session agents`,
+                currentValue: String(this.state.maxConcurrent),
+                submenu: (_currentValue, done) => new TextInputSubmenu("Global Worker Concurrency", `Enter any positive safe integer. This limits running agents; fixed safety ceilings separately bound depth ${DEFAULT_WORKER_FLEET_LIMITS.maxDepth}, ${DEFAULT_WORKER_FLEET_LIMITS.maxChildrenPerAgent} direct children, ${DEFAULT_WORKER_FLEET_LIMITS.maxAgentsPerSession} session agents, and ${DEFAULT_WORKER_FLEET_LIMITS.maxQueuedDispatches} queued dispatches.`, String(this.state.maxConcurrent), (value) => {
                     const maxConcurrent = Number(value.trim());
                     if (Number.isSafeInteger(maxConcurrent) && maxConcurrent > 0) {
                         this.state = { ...this.state, maxConcurrent };
-                        onChange({ ...this.state }, this.scope);
+                        done(String(maxConcurrent));
+                        return;
                     }
-                    done(String(this.state.maxConcurrent ?? 1));
+                    done();
                 }, () => done(), "positive safe integer; invalid input keeps the current value"),
             },
         ];