wave-agent-sdk 0.9.7 → 0.10.1

package/src/managers/backgroundTaskManager.ts

@@ -1,4 +1,7 @@
 import { spawn, type ChildProcess } from "child_process";
+import * as os from "os";
+import * as fs from "fs";
+import * as path from "path";
 import { BackgroundTask, BackgroundShell } from "../types/processes.js";
 import { stripAnsiColors } from "../utils/stringUtils.js";
 import { logger } from "../utils/globalLogger.js";
@@ -64,6 +67,10 @@ export class BackgroundTaskManager {
       },
     });
 
+    // Create log file
+    const logPath = path.join(os.tmpdir(), `wave-task-${id}.log`);
+    const logStream = fs.createWriteStream(logPath, { flags: "a" });
+
     const shell: BackgroundShell = {
       id,
       type: "shell",
@@ -73,8 +80,10 @@ export class BackgroundTaskManager {
       status: "running",
       stdout: "",
       stderr: "",
+      outputPath: logPath,
       onStop: () => {
         try {
+          logStream.end();
           if (child.pid) {
             process.kill(-child.pid, "SIGTERM");
             setTimeout(() => {
@@ -109,12 +118,20 @@ export class BackgroundTaskManager {
     }
 
     const onStdout = (data: Buffer | string) => {
-      shell.stdout += stripAnsiColors(data.toString());
+      const stripped = stripAnsiColors(data.toString());
+      shell.stdout += stripped;
+      if (logStream.writable) {
+        logStream.write(stripped);
+      }
       this.notifyTasksChange();
     };
 
     const onStderr = (data: Buffer | string) => {
-      shell.stderr += stripAnsiColors(data.toString());
+      const stripped = stripAnsiColors(data.toString());
+      shell.stderr += stripped;
+      if (logStream.writable) {
+        logStream.write(stripped);
+      }
       this.notifyTasksChange();
     };
 
@@ -122,6 +139,9 @@ export class BackgroundTaskManager {
       if (timeoutHandle) {
         clearTimeout(timeoutHandle);
       }
+      if (logStream.writable) {
+        logStream.end();
+      }
       shell.status = code === 0 ? "completed" : "failed";
       shell.exitCode = code ?? 0;
       shell.endTime = Date.now();
@@ -133,8 +153,13 @@ export class BackgroundTaskManager {
       if (timeoutHandle) {
         clearTimeout(timeoutHandle);
       }
+      const stripped = `\nProcess error: ${stripAnsiColors(error.message)}`;
       shell.status = "failed";
-      shell.stderr += `\nProcess error: ${stripAnsiColors(error.message)}`;
+      shell.stderr += stripped;
+      if (logStream.writable) {
+        logStream.write(stripped);
+        logStream.end();
+      }
       shell.exitCode = 1;
       shell.endTime = Date.now();
       shell.runtime = shell.endTime - startTime;
@@ -154,6 +179,7 @@ export class BackgroundTaskManager {
       if (timeoutHandle) {
         clearTimeout(timeoutHandle);
       }
+      logStream.end();
       this.tasks.delete(id);
       this.notifyTasksChange();
     };
@@ -170,6 +196,18 @@ export class BackgroundTaskManager {
     const id = this.generateId();
     const startTime = Date.now();
 
+    // Create log file
+    const logPath = path.join(os.tmpdir(), `wave-task-${id}.log`);
+    const logStream = fs.createWriteStream(logPath, { flags: "a" });
+
+    // Write initial output to log file
+    if (initialStdout) {
+      logStream.write(stripAnsiColors(initialStdout));
+    }
+    if (initialStderr) {
+      logStream.write(stripAnsiColors(initialStderr));
+    }
+
     const shell: BackgroundShell = {
       id,
       type: "shell",
@@ -179,8 +217,10 @@ export class BackgroundTaskManager {
       status: "running",
       stdout: initialStdout,
       stderr: initialStderr,
+      outputPath: logPath,
       onStop: () => {
         try {
+          logStream.end();
           if (child.pid) {
             process.kill(-child.pid, "SIGTERM");
             setTimeout(() => {
@@ -205,16 +245,27 @@ export class BackgroundTaskManager {
     this.notifyTasksChange();
 
     child.stdout?.on("data", (data) => {
-      shell.stdout += stripAnsiColors(data.toString());
+      const stripped = stripAnsiColors(data.toString());
+      shell.stdout += stripped;
+      if (logStream.writable) {
+        logStream.write(stripped);
+      }
       this.notifyTasksChange();
     });
 
     child.stderr?.on("data", (data) => {
-      shell.stderr += stripAnsiColors(data.toString());
+      const stripped = stripAnsiColors(data.toString());
+      shell.stderr += stripped;
+      if (logStream.writable) {
+        logStream.write(stripped);
+      }
       this.notifyTasksChange();
     });
 
     child.on("exit", (code) => {
+      if (logStream.writable) {
+        logStream.end();
+      }
       shell.status = code === 0 ? "completed" : "failed";
       shell.exitCode = code ?? 0;
       shell.endTime = Date.now();
@@ -223,8 +274,13 @@ export class BackgroundTaskManager {
     });
 
     child.on("error", (error) => {
+      const stripped = `\nProcess error: ${stripAnsiColors(error.message)}`;
       shell.status = "failed";
-      shell.stderr += `\nProcess error: ${stripAnsiColors(error.message)}`;
+      shell.stderr += stripped;
+      if (logStream.writable) {
+        logStream.write(stripped);
+        logStream.end();
+      }
       shell.exitCode = 1;
       shell.endTime = Date.now();
       shell.runtime = shell.endTime - startTime;

package/src/managers/messageManager.ts

@@ -12,7 +12,7 @@ import {
   type AgentToolBlockUpdateParams,
   generateMessageId,
 } from "../utils/messageOperations.js";
-import type { Message, Usage, SlashCommand } from "../types/index.js";
+import type { Message, Usage } from "../types/index.js";
 import { join } from "path";
 import {
   appendMessages,
@@ -55,7 +55,6 @@ export interface MessageManagerCallbacks {
   onAddBangMessage?: (command: string) => void;
   onUpdateBangMessage?: (command: string, output: string) => void;
   onCompleteBangMessage?: (command: string, exitCode: number) => void;
-  onSlashCommandsChange?: (commands: SlashCommand[]) => void;
   onInfoBlockAdded?: (content: string) => void;
   // Rewind callbacks
   onShowRewind?: () => void;
@@ -297,13 +296,6 @@ export class MessageManager {
     this.callbacks.onShowRewind?.();
   }
 
-  /**
-   * Trigger slash commands change callback
-   */
-  public triggerSlashCommandsChange(commands: SlashCommand[]): void {
-    this.callbacks.onSlashCommandsChange?.(commands);
-  }
-
   // Initialize state from session data
   public initializeFromSession(sessionData: SessionData): void {
     this.setSessionId(sessionData.id);

package/src/managers/permissionManager.ts

@@ -69,6 +69,7 @@ const DEFAULT_ALLOWED_RULES = [
   "Bash(whoami*)",
   "Bash(date*)",
   "Bash(uptime*)",
+  "Bash(wc -l*)",
 ];
 
 import { logger } from "../utils/globalLogger.js";
@@ -457,6 +458,7 @@ export class PermissionManager {
     permissionMode: PermissionMode,
     callback?: PermissionCallback,
     toolInput?: Record<string, unknown>,
+    toolCallId?: string,
   ): ToolPermissionContext {
     let suggestedPrefix: string | undefined;
     if (toolName === BASH_TOOL_NAME && toolInput?.command) {
@@ -475,6 +477,7 @@ export class PermissionManager {
       canUseToolCallback: callback,
       toolInput,
       suggestedPrefix,
+      toolCallId,
     };
 
     // Set hidePersistentOption for out-of-bounds file operations
@@ -512,8 +515,8 @@ export class PermissionManager {
             return true;
           }
 
-          // Check out-of-bounds for cd and ls
-          if (cmd === "cd" || cmd === "ls") {
+          // Check out-of-bounds for cd
+          if (cmd === "cd") {
             const pathArgs =
               (args.match(/(?:[^\s"']+|"[^"]*"|'[^']*')+/g) || []).filter(
                 (arg) => !arg.startsWith("-"),
@@ -642,19 +645,24 @@ export class PermissionManager {
               const args = commandMatch[2]?.trim() || "";
 
               if (SAFE_COMMANDS.includes(cmd)) {
-                if (cmd === "pwd" || cmd === "true" || cmd === "false") {
+                if (
+                  cmd === "pwd" ||
+                  cmd === "true" ||
+                  cmd === "false" ||
+                  cmd === "ls"
+                ) {
                   return true;
                 }
 
                 if (workdir) {
-                  // For cd and ls, check paths
+                  // For cd, check paths
                   const pathArgs =
                     (args.match(/(?:[^\s"']+|"[^"]*"|'[^']*')+/g) || []).filter(
                       (arg) => !arg.startsWith("-"),
                     ) || [];
 
                   if (pathArgs.length === 0) {
-                    // cd or ls without arguments operates on current dir (workdir)
+                    // cd without arguments operates on current dir (workdir)
                     return true;
                   }
 
@@ -739,10 +747,15 @@ export class PermissionManager {
         const args = commandMatch[2]?.trim() || "";
 
         if (SAFE_COMMANDS.includes(cmd)) {
-          if (cmd === "pwd" || cmd === "true" || cmd === "false") {
+          if (
+            cmd === "pwd" ||
+            cmd === "true" ||
+            cmd === "false" ||
+            cmd === "ls"
+          ) {
             isSafe = true;
           } else {
-            // For cd and ls, check paths
+            // For cd, check paths
             const pathArgs =
               (args.match(/(?:[^\s"']+|"[^"]*"|'[^']*')+/g) || []).filter(
                 (arg) => !arg.startsWith("-"),
@@ -775,7 +788,7 @@ export class PermissionManager {
             continue;
           }
 
-          if (cmd === "cd" || cmd === "ls") {
+          if (cmd === "cd") {
             const pathArgs =
               (args.match(/(?:[^\s"']+|"[^"]*"|'[^']*')+/g) || []).filter(
                 (arg) => !arg.startsWith("-"),

package/src/managers/skillManager.ts

@@ -243,8 +243,18 @@ export class SkillManager {
       const entries = await readdir(skillsPath, { withFileTypes: true });
 
       for (const entry of entries) {
+        const fullPath = join(skillsPath, entry.name);
         if (entry.isDirectory()) {
-          directories.push(join(skillsPath, entry.name));
+          directories.push(fullPath);
+        } else if (entry.isSymbolicLink()) {
+          try {
+            const s = await stat(fullPath);
+            if (s.isDirectory()) {
+              directories.push(fullPath);
+            }
+          } catch {
+            // Ignore broken symlinks or other errors
+          }
         }
       }
     } catch {

package/src/managers/slashCommandManager.ts

@@ -277,7 +277,6 @@ export class SlashCommandManager {
    */
   public registerCommand(command: SlashCommand): void {
     this.commands.set(command.id, command);
-    this.messageManager.triggerSlashCommandsChange(this.getCommands());
   }
 
   /**

package/src/managers/subagentManager.ts

@@ -1,4 +1,7 @@
 import { randomUUID } from "crypto";
+import * as os from "os";
+import * as fs from "fs";
+import * as path from "path";
 import type { SubagentConfiguration } from "../utils/subagentParser.js";
 import type { Message, Usage } from "../types/index.js";
 import { AIManager } from "./aiManager.js";
@@ -70,6 +73,7 @@ export interface SubagentInstance {
   backgroundTaskId?: string; // ID of the background task if transitioned
   onUpdate?: () => void; // Optional callback for real-time updates
   model?: string; // Optional model override
+  logStream?: fs.WriteStream; // Optional log stream for background tasks
 }
 
 export interface SubagentManagerOptions {
@@ -274,6 +278,11 @@ export class SubagentManager {
         const taskId = backgroundTaskManager.generateId();
         const startTime = Date.now();
 
+        // Create log file
+        const logPath = path.join(os.tmpdir(), `wave-subagent-${taskId}.log`);
+        const logStream = fs.createWriteStream(logPath, { flags: "a" });
+        instance.logStream = logStream;
+
         backgroundTaskManager.addTask({
           id: taskId,
           type: "subagent",
@@ -282,8 +291,10 @@ export class SubagentManager {
           description: instance.description,
           stdout: "",
           stderr: "",
+          outputPath: logPath,
           subagentId: instance.subagentId,
           onStop: () => {
+            instance.logStream?.end();
             instance.aiManager.abortAIMessage();
             this.cleanupInstance(instance.subagentId);
           },
@@ -345,6 +356,11 @@ export class SubagentManager {
     const taskId = backgroundTaskManager.generateId();
     const startTime = Date.now();
 
+    // Create log file
+    const logPath = path.join(os.tmpdir(), `wave-subagent-${taskId}.log`);
+    const logStream = fs.createWriteStream(logPath, { flags: "a" });
+    instance.logStream = logStream;
+
     backgroundTaskManager.addTask({
       id: taskId,
       type: "subagent",
@@ -353,8 +369,10 @@ export class SubagentManager {
       description: instance.description,
       stdout: "",
       stderr: "",
+      outputPath: logPath,
       subagentId: instance.subagentId,
       onStop: () => {
+        instance.logStream?.end();
         instance.aiManager.abortAIMessage();
         this.cleanupInstance(instance.subagentId);
       },
@@ -446,6 +464,7 @@ export class SubagentManager {
 
       // If this was transitioned to background, update the background task
       if (instance.backgroundTaskId && backgroundTaskManager) {
+        instance.logStream?.end();
         const task = backgroundTaskManager.getTask(instance.backgroundTaskId);
         if (task) {
           task.status = "completed";
@@ -465,6 +484,7 @@ export class SubagentManager {
 
       // If this was transitioned to background, update the background task with error
       if (instance.backgroundTaskId && backgroundTaskManager) {
+        instance.logStream?.end();
         const task = backgroundTaskManager.getTask(instance.backgroundTaskId);
         if (task) {
           task.status = "failed";
@@ -599,6 +619,17 @@ export class SubagentManager {
               instance.lastTools.shift();
             }
             instance.onUpdate?.();
+
+            // Log tool execution to file
+            if (instance.logStream) {
+              const compactParams = (params.parameters || "{}").substring(
+                0,
+                100,
+              );
+              instance.logStream.write(
+                `[${new Date().toISOString()}] Running tool: ${params.name} with params: ${compactParams}${compactParams.length >= 100 ? "..." : ""}\n`,
+              );
+            }
           }
         }
 

package/src/managers/toolManager.ts

@@ -195,6 +195,7 @@ class ToolManager {
         ? this.container.get<SkillManager>("SkillManager")
         : undefined,
       sessionId: context.sessionId,
+      toolCallId: context.toolCallId,
     };
 
     logger?.debug("Executing tool with enhanced context", {

package/src/services/session.ts

@@ -798,6 +798,47 @@ export async function getFirstMessageContent(
   }
 }
 
+/**
+ * Delete a session
+ * @param sessionId - UUID session identifier
+ * @param workdir - Working directory for the session
+ * @param sessionType - Type of session ("main" or "subagent", defaults to "main")
+ */
+export async function deleteSession(
+  sessionId: string,
+  workdir: string,
+  sessionType: "main" | "subagent" = "main",
+): Promise<void> {
+  const filePath = await generateSessionFilePath(
+    sessionId,
+    workdir,
+    sessionType,
+  );
+  try {
+    await fs.unlink(filePath);
+  } catch (error) {
+    // Ignore if file doesn't exist
+    if ((error as NodeJS.ErrnoException).code !== "ENOENT") {
+      throw error;
+    }
+  }
+
+  // Also remove from index
+  try {
+    const encoder = new PathEncoder();
+    const projectDir = await encoder.getProjectDirectory(workdir, SESSION_DIR);
+    const indexPath = join(projectDir.encodedPath, SESSION_INDEX_FILENAME);
+    const indexContent = await fs.readFile(indexPath, "utf8");
+    const index = JSON.parse(indexContent) as SessionIndex;
+    if (index.sessions[sessionId]) {
+      delete index.sessions[sessionId];
+      await fs.writeFile(indexPath, JSON.stringify(index, null, 2), "utf8");
+    }
+  } catch {
+    // Ignore index errors
+  }
+}
+
 /**
  * Truncate content to a maximum length, adding ellipsis if truncated
  * @param content - The content to truncate

package/src/tools/agentTool.ts

@@ -182,10 +182,14 @@ When using the Agent tool, you must specify a subagent_type parameter to select
               id: instance.subagentId,
               backgroundHandler: async () => {
                 isBackgrounded = true;
-                await subagentManager.backgroundInstance(instance.subagentId);
+                const taskId = await subagentManager.backgroundInstance(
+                  instance.subagentId,
+                );
+                const task = context.backgroundTaskManager?.getTask(taskId);
+                const outputPath = task?.outputPath;
                 resolve({
                   success: true,
-                  content: "Agent backgrounded",
+                  content: `Agent backgrounded with ID: ${taskId}.${outputPath ? ` Real-time output: ${outputPath}` : ""}`,
                   shortResult: "Agent backgrounded",
                   isManuallyBackgrounded: true,
                 });
@@ -206,9 +210,11 @@ When using the Agent tool, you must specify a subagent_type parameter to select
             }
 
             if (run_in_background) {
+              const task = context.backgroundTaskManager?.getTask(result);
+              const outputPath = task?.outputPath;
               resolve({
                 success: true,
-                content: `Agent started in background with ID: ${result}`,
+                content: `Agent started in background with ID: ${result}.${outputPath ? ` Real-time output: ${outputPath}` : ""}`,
                 shortResult: `Agent started in background: ${result}`,
               });
               return;

package/src/tools/askUserQuestion.ts

@@ -106,6 +106,15 @@ Plan mode note: In plan mode, use this tool to clarify requirements or choose be
       metadata,
     } = args as unknown as AskUserQuestionInput;
 
+    if (!questions || !Array.isArray(questions) || questions.length === 0) {
+      return {
+        success: false,
+        content: "",
+        error:
+          "The 'questions' parameter is missing or empty. Please use the correct schema: { questions: [{ question, header, options, multiSelect? }] }",
+      };
+    }
+
     if (!context.permissionManager) {
       throw new Error(
         `Permission manager is required for ${ASK_USER_QUESTION_TOOL_NAME} tool`,
@@ -117,6 +126,7 @@ Plan mode note: In plan mode, use this tool to clarify requirements or choose be
       context.permissionMode || "default",
       context.canUseToolCallback,
       { questions, answers: existingAnswers, metadata },
+      context.toolCallId,
     );
     permissionContext.hidePersistentOption = true; // Always hide persistent option for questions
 

package/src/tools/bashTool.ts

@@ -145,6 +145,7 @@ Usage notes:
             timeout,
             workdir: context.workdir,
           },
+          context.toolCallId,
         );
         const permissionResult =
           await context.permissionManager.checkPermission(permissionContext);
@@ -177,9 +178,11 @@ Usage notes:
       }
 
       const { id: taskId } = backgroundTaskManager.startShell(command, timeout);
+      const task = backgroundTaskManager.getTask(taskId);
+      const outputPath = task?.outputPath;
       return {
         success: true,
-        content: `Command started in background with ID: ${taskId}. Use TaskOutput tool with task_id="${taskId}" to monitor output.`,
+        content: `Command started in background with ID: ${taskId}.${outputPath ? ` Real-time output: ${outputPath}` : ` Use TaskOutput tool with task_id="${taskId}" to monitor output.`}`,
         shortResult: `Background process ${taskId} started`,
       };
     }
@@ -220,9 +223,11 @@ Usage notes:
                 outputBuffer,
                 errorBuffer,
               );
+              const task = backgroundTaskManager.getTask(taskId);
+              const outputPath = task?.outputPath;
               resolve({
                 success: true,
-                content: `Command moved to background with ID: ${taskId}.`,
+                content: `Command moved to background with ID: ${taskId}.${outputPath ? ` Real-time output: ${outputPath}` : ""}`,
                 shortResult: `Process ${taskId} backgrounded`,
                 isManuallyBackgrounded: true,
               });

package/src/tools/editTool.ts

@@ -174,6 +174,7 @@ Usage:
               replace_all: replaceAll,
               startLineNumber,
             },
+            context.toolCallId,
           );
           const permissionResult =
             await context.permissionManager.checkPermission(permissionContext);

package/src/tools/exitPlanMode.ts

@@ -86,6 +86,7 @@ Ensure your plan is complete and unambiguous:
         context.permissionMode || "plan",
         context.canUseToolCallback,
         { plan_content: planContent },
+        context.toolCallId,
       );
 
       const permissionResult =

package/src/tools/globTool.ts

@@ -1,70 +1,13 @@
-import { spawn } from "child_process";
-import { rgPath } from "../utils/ripgrep.js";
+import { glob } from "glob";
 import { stat } from "fs/promises";
 import type { ToolPlugin, ToolResult, ToolContext } from "./types.js";
 import { resolvePath, getDisplayPath } from "../utils/path.js";
-import { getAllIgnorePatterns } from "../utils/fileFilter.js";
 import { GLOB_TOOL_NAME } from "../constants/tools.js";
 
 /**
  * Maximum number of files returned by glob tool
  */
-const MAX_GLOB_RESULTS = 1000;
-
-/**
- * Execute ripgrep to find files matching a pattern
- */
-async function runRipgrep(pattern: string, workdir: string): Promise<string[]> {
-  if (!rgPath) {
-    throw new Error("ripgrep is not available");
-  }
-
-  const ignorePatterns = getAllIgnorePatterns();
-  const rgArgs = ["--files", "--color=never", "--hidden", "--glob", pattern];
-
-  for (const ignorePattern of ignorePatterns) {
-    rgArgs.push("--glob", `!${ignorePattern}`);
-  }
-
-  return new Promise<string[]>((resolve, reject) => {
-    const child = spawn(rgPath, rgArgs, {
-      cwd: workdir,
-      stdio: ["ignore", "pipe", "pipe"],
-    });
-
-    let stdout = "";
-    let stderr = "";
-
-    child.stdout?.on("data", (data) => {
-      stdout += data.toString();
-    });
-
-    child.stderr?.on("data", (data) => {
-      stderr += data.toString();
-    });
-
-    child.on("close", (code) => {
-      if (code !== 0 && code !== 1) {
-        reject(
-          new Error(
-            `ripgrep failed with code ${code}: ${stderr || "Unknown error"}`,
-          ),
-        );
-        return;
-      }
-      const files = stdout
-        .trim()
-        .split("\n")
-        .filter((f) => f.length > 0)
-        .map((f) => f.replace(/\\/g, "/")); // Normalize to forward slashes
-      resolve(files);
-    });
-
-    child.on("error", (err) => {
-      reject(err);
-    });
-  });
-}
+const MAX_GLOB_RESULTS = 100;
 
 /**
  * Glob Tool Plugin - Fast file pattern matching
@@ -122,8 +65,13 @@ export const globTool: ToolPlugin = {
         ? resolvePath(searchPath, context.workdir)
         : context.workdir;
 
-      // Execute glob search using ripgrep
-      const matches = await runRipgrep(pattern, workdir);
+      // Execute glob search using glob package
+      const matches = await glob(pattern, {
+        cwd: workdir,
+        nodir: true,
+        dot: true,
+        ignore: ["**/.git/**"],
+      });
 
       if (matches.length === 0) {
         return {

package/src/tools/grepTool.ts

@@ -1,6 +1,5 @@
 import type { ToolPlugin, ToolResult, ToolContext } from "./types.js";
 import { spawn } from "child_process";
-import { getAllIgnorePatterns } from "../utils/fileFilter.js";
 import { rgPath } from "../utils/ripgrep.js";
 import { getDisplayPath } from "../utils/path.js";
 import {
@@ -188,12 +187,6 @@ export const grepTool: ToolPlugin = {
         rgArgs.push("--glob", globPattern);
       }
 
-      // Get common ignore rules
-      const ignorePatterns = getAllIgnorePatterns();
-      for (const exclude of ignorePatterns) {
-        rgArgs.push("--glob", `!${exclude}`);
-      }
-
       // Add search pattern - use -e parameter to avoid patterns starting with - being mistaken as command line options
       rgArgs.push("-e", pattern);
 

package/src/tools/types.ts

@@ -79,6 +79,8 @@ export interface ToolContext {
   skillManager?: import("../managers/skillManager.js").SkillManager;
   /** Current session ID */
   sessionId?: string;
+  /** The ID of the current tool call */
+  toolCallId?: string;
   /** Callback to update the short result of the current tool block */
   onShortResultUpdate?: (shortResult: string) => void;
 }