fix(tools): commit director tool activations onto the infer wire

LSP/onToolsActivate was only calling activate() so those names stayed off the provider tools array too.

Author: TheGreatAxios

src/exec/runner.ts

@@ -903,20 +903,21 @@ export async function runExec(config: Config): Promise<ExecResult> {
     // refuses MCP/present/plugin names the result just told the model to
     // invoke. Under a closed overlay the promoter only activates allowed
     // names, so outside-allow tools can never become advertised or callable.
+    const commitPromotedWire = (): void => {
+      if (flushPromotions()) {
+        directorHolder.instance?.updateToolDefinitions(
+          computeAdvertised(agentToolset.dynamicRunner.currentDefinitions()),
+        );
+      }
+    };
     agentToolset.setToolPromoter(
       createExecToolPromoter({
         activate: (names) => activatedToolNames.activate(names),
         isAllowed: (name) => isExecOverlayToolAllowed(overlay, name),
         persist: () => {
           void persist("running");
         },
-        commitWire: () => {
-          if (flushPromotions()) {
-            directorHolder.instance?.updateToolDefinitions(
-              computeAdvertised(agentToolset.dynamicRunner.currentDefinitions()),
-            );
-          }
-        },
+        commitWire: commitPromotedWire,
       }),
     );
 
@@ -1010,7 +1011,10 @@ export async function runExec(config: Config): Promise<ExecResult> {
               tasks: tasks.map((t) => `${t.status}:${t.title}`).join(", "),
             });
           },
-          onToolsActivate: (names) => activatedToolNames.activate(names),
+          onToolsActivate: (names) => {
+            if (!activatedToolNames.activate(names)) return;
+            commitPromotedWire();
+          },
         },
         (message, fields) => logger.debug(message, fields),
       );

src/tui/runner/exit.ts

@@ -277,6 +277,17 @@ export async function createRunLifecycle(
 }> {
   const { persistRunSnapshot } = createRunPersistence(state, services);
   state.persistRunSnapshot = persistRunSnapshot;
+  const activateAndCommitWire = (names: string[]): void => {
+    if (!services.activatedToolNames.activate(names)) return;
+    if (services.flushPromotions()) {
+      services.directorHolder.instance?.updateToolDefinitions(
+        services.computeAdvertised(
+          services.toolset.dynamicRunner.currentDefinitions(),
+        ),
+      );
+    }
+    void persistRunSnapshot("running");
+  };
   const stopHeartbeat = startRunHeartbeat({
     shouldTick: () => !services.crashGuard.isFinalized(),
     tick: () => persistRunSnapshot("running"),
@@ -340,7 +351,7 @@ export async function createRunLifecycle(
       event,
       {
         onTasksChanged: (tasks) => services.emitter.emit("tasks", tasks),
-        onToolsActivate: (names) => services.activatedToolNames.activate(names),
+        onToolsActivate: (names) => activateAndCommitWire(names),
       },
       (message, fields) => tuiLogger.debug(message, fields),
     );
@@ -412,18 +423,7 @@ export async function createRunLifecycle(
   // could not emit those calls. Cache prefix growth on discovery is the cost
   // of making promotion actually work.
   const promoteTools = (names: string[]): void => {
-    if (!services.activatedToolNames.activate(names)) return;
-    if (services.flushPromotions()) {
-      services.directorHolder.instance?.updateToolDefinitions(
-        services.computeAdvertised(
-          services.toolset.dynamicRunner.currentDefinitions(),
-        ),
-      );
-    }
-    // Activation is model-visible contract — persist it now so a crash or
-    // restart before the next turn boundary does not strand the transcript's
-    // "these tools are available" record.
-    void persistRunSnapshot("running");
+    activateAndCommitWire(names);
   };
   services.toolset.setToolPromoter(promoteTools);