wave-agent-sdk 0.0.8 → 0.0.10

package/dist/tools/bashTool.js

@@ -1,5 +1,7 @@
 import { spawn } from "child_process";
 import { logger } from "../utils/globalLogger.js";
+import { stripAnsiColors } from "../utils/stringUtils.js";
+import { handleLargeOutput } from "../utils/largeOutputHandler.js";
 /**
  * Bash command execution tool - supports both foreground and background execution
  */
@@ -37,6 +39,7 @@ export const bashTool = {
     execute: async (args, context) => {
         const command = args.command;
         const runInBackground = args.run_in_background;
+        const description = args.description;
         // Set default timeout: 60s for foreground, no timeout for background
         const timeout = args.timeout ??
             (runInBackground ? undefined : 60000);
@@ -56,6 +59,36 @@ export const bashTool = {
                 error: "Timeout must be a number between 0 and 600000 milliseconds",
             };
         }
+        // Permission check after validation but before real operation
+        if (context.permissionManager &&
+            context.permissionMode &&
+            context.permissionMode !== "bypassPermissions") {
+            if (context.permissionManager.isRestrictedTool("Bash")) {
+                try {
+                    const permissionContext = context.permissionManager.createContext("Bash", context.permissionMode, context.canUseToolCallback, {
+                        command,
+                        description,
+                        run_in_background: runInBackground,
+                        timeout,
+                    });
+                    const permissionResult = await context.permissionManager.checkPermission(permissionContext);
+                    if (permissionResult.behavior === "deny") {
+                        return {
+                            success: false,
+                            content: "",
+                            error: `Bash operation denied by user, reason: ${permissionResult.message || "No reason provided"}`,
+                        };
+                    }
+                }
+                catch {
+                    return {
+                        success: false,
+                        content: "",
+                        error: "Permission check failed",
+                    };
+                }
+            }
+        }
         if (runInBackground) {
             // Background execution
             const backgroundBashManager = context?.backgroundBashManager;
@@ -147,16 +180,19 @@ export const bashTool = {
                     handleAbort();
                     return;
                 }
-                context.abortSignal.addEventListener("abort", () => handleAbort());
+                // Use { once: true } to prevent listener accumulation on signal reuse
+                context.abortSignal.addEventListener("abort", () => handleAbort(), {
+                    once: true,
+                });
             }
             child.stdout?.on("data", (data) => {
                 if (!isAborted) {
-                    outputBuffer += data.toString();
+                    outputBuffer += stripAnsiColors(data.toString());
                 }
             });
             child.stderr?.on("data", (data) => {
                 if (!isAborted) {
-                    errorBuffer += data.toString();
+                    errorBuffer += stripAnsiColors(data.toString());
                 }
             });
             child.on("exit", (code) => {
@@ -166,12 +202,28 @@ export const bashTool = {
                     }
                     const exitCode = code ?? 0;
                     const combinedOutput = outputBuffer + (errorBuffer ? "\n" + errorBuffer : "");
-                    resolve({
-                        success: exitCode === 0,
-                        content: combinedOutput || `Command executed with exit code: ${exitCode}`,
-                        error: exitCode !== 0
-                            ? `Command failed with exit code: ${exitCode}`
-                            : undefined,
+                    // Handle large output by writing to temp file if needed
+                    const finalOutput = combinedOutput || `Command executed with exit code: ${exitCode}`;
+                    handleLargeOutput(finalOutput)
+                        .then(({ content, filePath }) => {
+                        resolve({
+                            success: exitCode === 0,
+                            content,
+                            filePath,
+                            error: exitCode !== 0
+                                ? `Command failed with exit code: ${exitCode}`
+                                : undefined,
+                        });
+                    })
+                        .catch((error) => {
+                        logger.warn(`Error handling large output: ${error}`);
+                        resolve({
+                            success: exitCode === 0,
+                            content: finalOutput,
+                            error: exitCode !== 0
+                                ? `Command failed with exit code: ${exitCode}`
+                                : undefined,
+                        });
                     });
                 }
             });
@@ -190,9 +242,13 @@ export const bashTool = {
         });
     },
     formatCompactParams: (params) => {
+        const description = params.description;
         const command = params.command;
         const runInBackground = params.run_in_background;
-        return `${command}${runInBackground ? " background" : ""}`;
+        if (description) {
+            return description;
+        }
+        return `${command}${runInBackground ? " (background)" : ""}`;
     },
 };
 /**
@@ -257,14 +313,17 @@ export const bashOutputTool = {
         }
         let content = "";
         if (output.stdout) {
-            content += output.stdout;
+            content += stripAnsiColors(output.stdout);
         }
         if (output.stderr) {
-            content += (content ? "\n" : "") + output.stderr;
+            content += (content ? "\n" : "") + stripAnsiColors(output.stderr);
         }
+        const finalContent = content || "No output available";
+        const { content: processedContent, filePath } = await handleLargeOutput(finalContent);
         return {
             success: true,
-            content: content || "No output available",
+            content: processedContent,
+            filePath,
             shortResult: `${bashId}: ${output.status}${shell.exitCode !== undefined ? ` (${shell.exitCode})` : ""}`,
             error: undefined,
         };

package/dist/tools/deleteFileTool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"deleteFileTool.d.ts","sourceRoot":"","sources":["../../src/tools/deleteFileTool.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,UAAU,EAA2B,MAAM,YAAY,CAAC;AAGtE;;GAEG;AACH,eAAO,MAAM,cAAc,EAAE,UAyE5B,CAAC"}
+{"version":3,"file":"deleteFileTool.d.ts","sourceRoot":"","sources":["../../src/tools/deleteFileTool.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,UAAU,EAA2B,MAAM,YAAY,CAAC;AAGtE;;GAEG;AACH,eAAO,MAAM,cAAc,EAAE,UA6G5B,CAAC"}

package/dist/tools/deleteFileTool.js

@@ -37,6 +37,31 @@ export const deleteFileTool = {
         }
         try {
             const filePath = resolvePath(targetFile, context.workdir);
+            // Permission check after validation but before real operation
+            if (context.permissionManager &&
+                context.permissionMode &&
+                context.permissionMode !== "bypassPermissions") {
+                if (context.permissionManager.isRestrictedTool("Delete")) {
+                    try {
+                        const permissionContext = context.permissionManager.createContext("Delete", context.permissionMode, context.canUseToolCallback, { target_file: targetFile });
+                        const permissionResult = await context.permissionManager.checkPermission(permissionContext);
+                        if (permissionResult.behavior === "deny") {
+                            return {
+                                success: false,
+                                content: "",
+                                error: `Delete operation denied by user, reason: ${permissionResult.message || "No reason provided"}`,
+                            };
+                        }
+                    }
+                    catch {
+                        return {
+                            success: false,
+                            content: "",
+                            error: "Permission check failed",
+                        };
+                    }
+                }
+            }
             // Delete file
             await unlink(filePath);
             logger.debug(`Successfully deleted file: ${filePath}`);

package/dist/tools/editTool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"editTool.d.ts","sourceRoot":"","sources":["../../src/tools/editTool.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,UAAU,EAA2B,MAAM,YAAY,CAAC;AAetE;;GAEG;AACH,eAAO,MAAM,QAAQ,EAAE,UAqKtB,CAAC"}
+{"version":3,"file":"editTool.d.ts","sourceRoot":"","sources":["../../src/tools/editTool.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,UAAU,EAA2B,MAAM,YAAY,CAAC;AActE;;GAEG;AACH,eAAO,MAAM,QAAQ,EAAE,UAwMtB,CAAC"}

package/dist/tools/editTool.js

@@ -1,7 +1,6 @@
 import { readFile, writeFile } from "fs/promises";
 import { logger } from "../utils/globalLogger.js";
 import { resolvePath, getDisplayPath } from "../utils/path.js";
-import { diffLines } from "diff";
 /**
  * Format compact parameter display
  */
@@ -123,6 +122,36 @@ export const editTool = {
                 newContent = originalContent.replace(oldString, newString);
                 replacementCount = 1;
             }
+            // Permission check after validation but before real operation
+            if (context.permissionManager &&
+                context.permissionMode &&
+                context.permissionMode !== "bypassPermissions") {
+                if (context.permissionManager.isRestrictedTool("Edit")) {
+                    try {
+                        const permissionContext = context.permissionManager.createContext("Edit", context.permissionMode, context.canUseToolCallback, {
+                            file_path: filePath,
+                            old_string: oldString,
+                            new_string: newString,
+                            replace_all: replaceAll,
+                        });
+                        const permissionResult = await context.permissionManager.checkPermission(permissionContext);
+                        if (permissionResult.behavior === "deny") {
+                            return {
+                                success: false,
+                                content: "",
+                                error: `Edit operation denied by user, reason: ${permissionResult.message || "No reason provided"}`,
+                            };
+                        }
+                    }
+                    catch {
+                        return {
+                            success: false,
+                            content: "",
+                            error: "Permission check failed",
+                        };
+                    }
+                }
+            }
             // Write file
             try {
                 await writeFile(resolvedPath, newContent, "utf-8");
@@ -134,8 +163,6 @@ export const editTool = {
                     error: `Failed to write file: ${writeError instanceof Error ? writeError.message : String(writeError)}`,
                 };
             }
-            // Generate diff information
-            const diffResult = diffLines(originalContent, newContent);
             const shortResult = replaceAll
                 ? `Replaced ${replacementCount} instances`
                 : "Text replaced successfully";
@@ -145,9 +172,6 @@ export const editTool = {
                 content: shortResult,
                 shortResult,
                 filePath: resolvedPath,
-                originalContent,
-                newContent,
-                diffResult,
             };
         }
         catch (error) {

package/dist/tools/lspTool.d.ts

@@ -0,0 +1,6 @@
+import type { ToolPlugin } from "./types.js";
+/**
+ * LSP tool plugin - interact with LSP servers for code intelligence
+ */
+export declare const lspTool: ToolPlugin;
+//# sourceMappingURL=lspTool.d.ts.map

package/dist/tools/lspTool.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"lspTool.d.ts","sourceRoot":"","sources":["../../src/tools/lspTool.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAA2B,MAAM,YAAY,CAAC;AA8dtE;;GAEG;AACH,eAAO,MAAM,OAAO,EAAE,UAsRrB,CAAC"}