neoctl 0.2.33 → 0.2.35

package/dist/web/index.js

@@ -389,6 +389,7 @@ export class WebRepl {
     pendingUserImageEchoMessageId;
     runtimeContextRevision = 0;
     runtimeContextPublishQueue = Promise.resolve();
+    pendingRuntimeConfiguration = false;
     backgroundSessionRuns = new Map();
     suppressReattachedStreaming = new Set();
     constructor(runtime) {
@@ -513,8 +514,6 @@ export class WebRepl {
         };
     }
     async setGlobalTools(value) {
-        if (this.busy)
-            return actionFailure("TOOL_UPDATE_BLOCKED", "cannot change global tools while a response is running");
         if (!isRecord(value))
             return actionFailure("INVALID_REQUEST", "tool overrides must be an object");
         const support = this.runtime.toolSupport;
@@ -525,10 +524,12 @@ export class WebRepl {
             return normalized;
         try {
             support.globalOverrides = normalized.value;
-            this.syncPluginToolRegistration();
-            applyToolOverrides(this.runtime.tools, support.catalog, support.globalOverrides, support.sessionOverrides);
             if (support.persistGlobal)
                 await support.persistGlobal(support.globalOverrides);
+            if (this.busy)
+                this.pendingRuntimeConfiguration = true;
+            else
+                await this.applyRuntimeConfiguration();
             await this.refreshToolConfiguration();
             return { ok: true, state: this.globalTools() };
         }
@@ -537,8 +538,6 @@ export class WebRepl {
         }
     }
     async setSessionTools(value) {
-        if (this.busy)
-            return actionFailure("TOOL_UPDATE_BLOCKED", "cannot change session tools while a response is running");
         if (!isRecord(value))
             return actionFailure("INVALID_REQUEST", "tool overrides must be an object");
         const support = this.runtime.toolSupport;
@@ -562,8 +561,6 @@ export class WebRepl {
         await this.applySessionToolOverrides(await support.resolveSession(sessionId), false);
     }
     async setSessionPlugins(value) {
-        if (this.busy)
-            return actionFailure("PLUGIN_UPDATE_BLOCKED", "cannot change session plugins while a response is running");
         if (!isRecord(value))
             return actionFailure("INVALID_REQUEST", "plugin overrides must be an object");
         const support = this.runtime.pluginSupport;
@@ -627,11 +624,13 @@ export class WebRepl {
         if (!support)
             return;
         support.sessionOverrides = normalizeToolOverrides(overrides, support.catalog);
-        this.syncPluginToolRegistration();
-        applyToolOverrides(this.runtime.tools, support.catalog, support.globalOverrides, support.sessionOverrides);
         const sessionId = this.runtime.engine.snapshot().session?.sessionId;
         if (persist && sessionId && support.persistSession)
             await support.persistSession(sessionId, support.sessionOverrides);
+        if (this.busy)
+            this.pendingRuntimeConfiguration = true;
+        else
+            await this.applyRuntimeConfiguration();
         await this.refreshToolConfiguration();
     }
     syncPluginToolRegistration() {
@@ -646,6 +645,18 @@ export class WebRepl {
                 this.runtime.tools.register(tool);
         }
     }
+    async applyRuntimeConfiguration() {
+        this.syncPluginToolRegistration();
+        const pluginSupport = this.runtime.pluginSupport;
+        if (pluginSupport) {
+            const activePlugins = activeWebRuntimePlugins(pluginSupport.catalog, pluginSupport.overrides);
+            this.runtime.engine.setRuntimePlugins([...pluginSupport.basePluginIds, ...activePlugins.map((plugin) => plugin.id)], [...pluginSupport.basePromptSections, ...activePlugins.flatMap((plugin) => plugin.promptSections ?? [])]);
+        }
+        const toolSupport = this.runtime.toolSupport;
+        if (toolSupport)
+            applyToolOverrides(this.runtime.tools, toolSupport.catalog, toolSupport.globalOverrides, toolSupport.sessionOverrides);
+        this.pendingRuntimeConfiguration = false;
+    }
     async refreshToolConfiguration() {
         const metrics = await this.runtime.engine.contextMetrics();
         this.runtime.initialMetrics = metrics;
@@ -658,21 +669,14 @@ export class WebRepl {
         if (!support)
             return;
         const normalized = normalizePluginOverrides(overrides, support.catalog);
-        for (const plugin of support.catalog)
-            for (const tool of plugin.tools)
-                this.runtime.tools.unregister(tool.name);
-        const activePlugins = activeWebRuntimePlugins(support.catalog, normalized);
-        for (const plugin of activePlugins)
-            for (const tool of plugin.tools)
-                this.runtime.tools.register(tool);
         support.overrides = normalized;
-        const toolSupport = this.runtime.toolSupport;
-        if (toolSupport)
-            applyToolOverrides(this.runtime.tools, toolSupport.catalog, toolSupport.globalOverrides, toolSupport.sessionOverrides);
-        this.runtime.engine.setRuntimePlugins([...support.basePluginIds, ...activePlugins.map((plugin) => plugin.id)], [...support.basePromptSections, ...activePlugins.flatMap((plugin) => plugin.promptSections ?? [])]);
         const sessionId = this.runtime.engine.snapshot().session?.sessionId;
         if (persist && sessionId && support.persist)
             await support.persist(sessionId, normalized);
+        if (this.busy)
+            this.pendingRuntimeConfiguration = true;
+        else
+            await this.applyRuntimeConfiguration();
         const metrics = await this.runtime.engine.contextMetrics();
         this.runtime.initialMetrics = metrics;
         this.setStatus({ ...this.status, metrics, activityTick: this.status.activityTick + 1 });
@@ -723,7 +727,11 @@ export class WebRepl {
         this.queuedAttachments = [...(this.queuedAttachments ?? []), ...attachments].slice(0, 20);
     }
     startRun(text, attachments = []) {
-        const run = this.handleCommandOrPrompt(text, attachments).catch((error) => {
+        const run = (async () => {
+            if (this.pendingRuntimeConfiguration)
+                await this.applyRuntimeConfiguration();
+            await this.handleCommandOrPrompt(text, attachments);
+        })().catch((error) => {
             this.append({ kind: "error", text: error instanceof Error ? error.message : String(error) });
             this.setBusy(false);
             this.setStatus({ ...this.status, phase: "ready", detail: undefined });
@@ -884,8 +892,6 @@ export class WebRepl {
         }
     }
     async setContextWindowK(value) {
-        if (this.busy)
-            return actionFailure("CONTEXT_WINDOW_UPDATE_BLOCKED", "cannot change context window while a response is running");
         if (typeof value !== "string" || !/^\d+$/u.test(value) || Number(value) <= 0) {
             return actionFailure("CONTEXT_WINDOW_INVALID", "context window must be a positive integer in k");
         }