@dreb/coding-agent 2.19.0 → 2.19.1

package/dist/core/tools/subagent.js

@@ -9,6 +9,7 @@ import { Type } from "@sinclair/typebox";
 import { CONFIG_DIR_NAME, getPackageDir, getSubagentSessionsDir } from "../../config.js";
 import { keyHint } from "../../modes/interactive/components/keybinding-hints.js";
 import { attachJsonlLineReader } from "../../modes/rpc/jsonl.js";
+import { log } from "../logger.js";
 import { resolveCliModel } from "../model-resolver.js";
 import { getTextOutput, invalidArgText, str } from "./render-utils.js";
 import { wrapToolDefinition } from "./tool-definition-wrapper.js";
@@ -87,20 +88,20 @@ function loadAgentsFromDir(dir, agents) {
                 const content = readFileSync(join(dir, file), "utf-8");
                 const parsed = parseAgentFrontmatter(content);
                 if (!parsed.ok) {
-                    console.error(`[subagent] Skipping agent file ${join(dir, file)}: ${parsed.error}`);
+                    log.warn(`[subagent] Skipping agent file ${join(dir, file)}: ${parsed.error}`);
                 }
                 else {
                     agents.set(parsed.config.name, parsed.config);
                 }
             }
             catch (err) {
-                console.error(`[subagent] Could not read agent file ${join(dir, file)}: ${err instanceof Error ? err.message : String(err)}`);
+                log.warn(`[subagent] Could not read agent file ${join(dir, file)}: ${err instanceof Error ? err.message : String(err)}`);
             }
         }
     }
     catch (err) {
         if (err.code !== "ENOENT") {
-            console.error(`[subagent] Could not read agents directory ${dir}: ${err instanceof Error ? err.message : String(err)}`);
+            log.warn(`[subagent] Could not read agents directory ${dir}: ${err instanceof Error ? err.message : String(err)}`);
         }
     }
 }
@@ -141,7 +142,7 @@ function findDrebBinary() {
 }
 async function spawnSubagent(agentConfig, task, cwd, signal, onProgress, parentProvider, sessionDir) {
     const drebBin = findDrebBinary();
-    console.error(`[subagent] spawn: agent=${agentConfig.name} cwd=${cwd}`);
+    log.debug(`[subagent] spawn: agent=${agentConfig.name} cwd=${cwd}`);
     // Validate cwd exists — spawn() throws a misleading ENOENT blaming the
     // binary when the cwd is invalid, making the real cause hard to diagnose
     if (!existsSync(cwd)) {
@@ -227,12 +228,12 @@ async function spawnSubagent(agentConfig, task, cwd, signal, onProgress, parentP
             }
         });
         proc.stderr?.on("error", (err) => {
-            console.error(`[subagent] stderr stream error (agent=${agentConfig.name}): ${err.message}`);
+            log.warn(`[subagent] stderr stream error (agent=${agentConfig.name}): ${err.message}`);
         });
         // Parse JSONL events from stdout
         if (proc.stdout) {
             proc.stdout.on("error", (err) => {
-                console.error(`[subagent] stdout stream error (agent=${agentConfig.name}): ${err.message}`);
+                log.warn(`[subagent] stdout stream error (agent=${agentConfig.name}): ${err.message}`);
             });
             attachJsonlLineReader(proc.stdout, (line) => {
                 if (!line.trim())
@@ -248,7 +249,7 @@ async function spawnSubagent(agentConfig, task, cwd, signal, onProgress, parentP
                     // (e.g. startup errors printed before JSONL mode begins)
                     plainStdoutLines.push(line.trim());
                     if (line.trim().startsWith("{")) {
-                        console.error(`[subagent] Failed to parse JSONL event: ${line.slice(0, 200)}`);
+                        log.warn(`[subagent] Failed to parse JSONL event: ${line.slice(0, 200)}`);
                     }
                     return;
                 }
@@ -304,7 +305,7 @@ async function spawnSubagent(agentConfig, task, cwd, signal, onProgress, parentP
             signal?.removeEventListener("abort", onAbort);
             const exitCode = code ?? 1;
             const stderr = stderrChunks.join("");
-            console.error(`[subagent] close: agent=${agentConfig.name} exit=${exitCode} messages=${collectedMessages.length}${exitCode !== 0 ? ` stderr=${stderr.slice(0, 200)} stdout=${plainStdoutLines.join("|").slice(0, 200)}` : ""}`);
+            log.debug(`[subagent] close: agent=${agentConfig.name} exit=${exitCode} messages=${collectedMessages.length}${exitCode !== 0 ? ` stderr=${stderr.slice(0, 200)} stdout=${plainStdoutLines.join("|").slice(0, 200)}` : ""}`);
             // Extract final text output from collected assistant messages
             const outputParts = [];
             for (const msg of collectedMessages) {
@@ -375,12 +376,12 @@ export function discoverSessionFile(sessionDir, agentName) {
             }
         }
         if (best) {
-            console.error(`[subagent] session file: ${best.path} (agent=${agentName})`);
+            log.debug(`[subagent] session file: ${best.path} (agent=${agentName})`);
             return best.path;
         }
     }
     catch (err) {
-        console.error(`[subagent] failed to discover session file (agent=${agentName}): ${err instanceof Error ? err.message : String(err)}`);
+        log.warn(`[subagent] failed to discover session file (agent=${agentName}): ${err instanceof Error ? err.message : String(err)}`);
     }
     return undefined;
 }
@@ -570,7 +571,7 @@ export async function resolveModelForSubagentSpawn(models, parentProvider, regis
             lastError = resolved.error;
             const reason = compactErrorReason(resolved.error);
             skippedModels.push({ model: modelStr, reason });
-            console.error(`[subagent] Model "${modelStr}" unavailable (${reason}). Trying next fallback...`);
+            log.warn(`[subagent] Model "${modelStr}" unavailable (${reason}). Trying next fallback...`);
             continue;
         }
         const modelObj = resolved.provider ? registry.find(resolved.provider, resolved.modelId) : undefined;
@@ -584,11 +585,11 @@ export async function resolveModelForSubagentSpawn(models, parentProvider, regis
             if (!probe.ok) {
                 lastError = probe.reason;
                 skippedModels.push({ model: modelStr, reason: probe.reason });
-                console.error(`[subagent] Model "${modelStr}" failed probe (${probe.reason}). Trying next fallback...`);
+                log.warn(`[subagent] Model "${modelStr}" failed probe (${probe.reason}). Trying next fallback...`);
                 continue;
             }
         }
-        console.error(`[subagent] Using model "${resolved.modelId}" for subagent.`);
+        log.debug(`[subagent] Using model "${resolved.modelId}" for subagent.`);
         return { ...resolved, skippedModels };
     }
     if (signal?.aborted)
@@ -597,7 +598,7 @@ export async function resolveModelForSubagentSpawn(models, parentProvider, regis
         const parentResolved = resolveModelStringSingle(parentModel, parentProvider, registry);
         if (parentResolved.ok) {
             const warning = `Agent preferred models were unavailable. Falling back to parent model "${parentResolved.modelId}".`;
-            console.error(`[subagent] ${warning}`);
+            log.warn(`[subagent] ${warning}`);
             return { ...parentResolved, warning, skippedModels };
         }
         lastError = parentResolved.error;
@@ -1019,7 +1020,7 @@ export function createSubagentToolDefinition(cwd, options) {
                             onBackgroundComplete(agentId, result, bgSignal.aborted);
                         }
                         catch (err) {
-                            console.error(`[subagent] onBackgroundComplete threw for agent ${agentId}: ${err instanceof Error ? err.message : String(err)}. Background result lost.`);
+                            log.warn(`[subagent] onBackgroundComplete threw for agent ${agentId}: ${err instanceof Error ? err.message : String(err)}. Background result lost.`);
                         }
                     };
                     const run = async () => {
@@ -1051,7 +1052,7 @@ export function createSubagentToolDefinition(cwd, options) {
                         }
                     };
                     run().catch((err) => {
-                        console.error(`[subagent] Unhandled background error (${agentId}): ${err instanceof Error ? err.message : String(err)}`);
+                        log.warn(`[subagent] Unhandled background error (${agentId}): ${err instanceof Error ? err.message : String(err)}`);
                         const entry = backgroundAgentRegistry.get(agentId);
                         if (entry && entry.status === "running")
                             entry.status = "failed";
@@ -1067,7 +1068,7 @@ export function createSubagentToolDefinition(cwd, options) {
                             }, bgSignal.aborted);
                         }
                         catch (notifyErr) {
-                            console.error(`[subagent] CRITICAL: Last-resort notification failed for ${agentId}: ${notifyErr instanceof Error ? notifyErr.message : String(notifyErr)}`);
+                            log.error(`[subagent] CRITICAL: Last-resort notification failed for ${agentId}: ${notifyErr instanceof Error ? notifyErr.message : String(notifyErr)}`);
                         }
                     });
                     return agentId;