wave-agent-sdk 0.18.4 → 0.18.5

package/builtin/skills/settings/ENV.md

@@ -28,7 +28,7 @@ Wave uses several environment variables to control its core functionality.
 | `WAVE_MODEL` | The primary AI model to use for the agent. | `gemini-3-flash` |
 | `WAVE_FAST_MODEL` | The fast AI model to use for quick tasks. | `gemini-2.5-flash` |
 | `WAVE_MAX_INPUT_TOKENS` | Maximum number of input tokens allowed. | `200000` |
-| `WAVE_MAX_OUTPUT_TOKENS` | Maximum number of output tokens allowed. | `16384` |
+| `WAVE_MAX_OUTPUT_TOKENS` | Maximum number of output tokens allowed. | `32000` |
 | `WAVE_DISABLE_AUTO_MEMORY` | Set to `1` or `true` to disable the auto-memory feature. | `false` |
 | `WAVE_AUTO_MEMORY_FREQUENCY` | Auto memory update frequency. `1` = every turn, `2` = every 2 turns, etc. | `1` |
 | `WAVE_TASK_LIST_ID` | Explicitly set the task list ID for the session. | (Session ID) |

package/builtin/skills/settings/MEMORY.md

@@ -0,0 +1,76 @@
+# Wave Memory
+
+Wave provides multiple memory layers to give the agent context-specific instructions and knowledge. Memory files are standard Markdown files loaded automatically at startup.
+
+## Memory Layers
+
+Wave looks for memory in the following locations, loaded in order of increasing priority:
+
+1. **User Memory**: `~/.wave/AGENTS.md` — Global instructions across all projects
+2. **Project Memory**: `AGENTS.md` in the project root — Project-specific instructions
+3. **Memory Rules**: `.wave/rules/*.md` and `~/.wave/rules/*.md` — Modular, path-scoped rules (see below)
+
+> `CLAUDE.md` is also supported as a fallback for both user and project memory, for compatibility with existing repositories.
+
+## User Memory
+
+Stored at `~/.wave/AGENTS.md`. Use it for cross-project preferences and global instructions, e.g. "always write tests for new features", "prefer functional style".
+
+## Project Memory
+
+### Project Memory File
+
+The `AGENTS.md` file in the project root is the primary project-level memory. It is checked into the repository and shared with all contributors. Use it for:
+
+- Build and test commands (e.g. "use pnpm not npm")
+- Project structure and architecture conventions
+- Coding standards and patterns
+
+### Memory Rules
+
+Memory rules are modular Markdown files that provide path-scoped instructions. They are discovered in:
+
+- **Project scope**: `.wave/rules/*.md` (checked into the repo)
+- **User scope**: `~/.wave/rules/*.md` (personal, not shared)
+
+Each file can optionally include YAML frontmatter to scope rules to specific file paths:
+
+```markdown
+---
+paths:
+  - "src/api/**/*.ts"
+  - "src/services/**/*.ts"
+---
+
+# API and Service Guidelines
+
+- Always use `async/await` for asynchronous operations.
+- Use `Zod` for input validation.
+```
+
+#### YAML Frontmatter Fields
+
+- `paths`: (Optional) A list of glob patterns. The rules in this file will only be active when the agent is working with files that match these patterns. If omitted, the rules are always active.
+- `priority`: (Optional) A number controlling rule precedence. Higher priority rules override lower ones on conflict.
+
+#### How Memory Rules are Loaded
+
+- Wave recursively discovers all `.md` files in `.wave/rules/` and `~/.wave/rules/`.
+- **Path-specific activation**: If a rule has a `paths` field, it is only included in the agent's context when a file being read or modified matches the glob patterns.
+- **Unconditional rules**: Rules without a `paths` field are always active.
+- **Priority**: Project-level rules take priority over user-level rules if there is a conflict.
+
+#### Best Practices
+
+- **Keep rules focused**: Create separate files for different topics (e.g. `testing.md`, `ui-components.md`).
+- **Leverage path scoping**: Use the `paths` field to keep the agent's context window clean and relevant.
+- **Share with your team**: Commit `.wave/rules/` to your git repository.
+
+## Auto-Memory
+
+In addition to manual memory files, Wave has an **auto-memory** feature that automatically extracts and remembers important information across sessions. This is stored in `~/.wave/projects/<project-id>/memory/MEMORY.md`.
+
+You can control auto-memory in `settings.json`:
+
+- `autoMemoryEnabled`: Enable or disable auto-memory (default: `true`).
+- `autoMemoryFrequency`: Frequency of auto-memory extraction turns (default: `1`).

package/builtin/skills/settings/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: settings
-description: Manage Wave settings and get guidance on settings.json, hooks, environment variables, permissions, MCP servers, memory rules, skills, subagents, plugins, and plugin marketplaces. Use this when the user wants to view, update, or learn how to configure Wave.
+description: Manage Wave settings and get guidance on settings.json, hooks, environment variables, permissions, MCP servers, memory, skills, subagents, plugins, and plugin marketplaces. Use this when the user wants to view, update, or learn how to configure Wave.
 ---
 
 # Wave Settings Skill
@@ -79,9 +79,9 @@ For detailed model configuration, see [MODELS.md](${WAVE_SKILL_DIR}/MODELS.md).
 Connect to external servers to provide additional tools and context.
 For detailed MCP configuration, see [MCP.md](${WAVE_SKILL_DIR}/MCP.md).
 
-### 6. Memory Rules
-Provide context-specific instructions and guidelines to the agent.
-For detailed memory rules configuration, see [MEMORY_RULES.md](${WAVE_SKILL_DIR}/MEMORY_RULES.md).
+### 6. Memory
+Provide context-specific instructions and knowledge to the agent through user memory, project memory, memory rules, and auto-memory.
+For detailed memory configuration, see [MEMORY.md](${WAVE_SKILL_DIR}/MEMORY.md).
 
 ### 7. Skills
 Extend Wave's functionality by creating custom skills.

package/dist/managers/mcpManager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"mcpManager.d.ts","sourceRoot":"","sources":["../../src/managers/mcpManager.ts"],"names":[],"mappings":"AAOA,OAAO,EAAE,0BAA0B,EAAE,MAAM,qBAAqB,CAAC;AAEjE,OAAO,KAAK,EAAE,UAAU,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAC7E,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAClD,OAAO,KAAK,EACV,MAAM,EACN,eAAe,EACf,SAAS,EACT,OAAO,EACP,eAAe,EAChB,MAAM,mBAAmB,CAAC;AAQ3B,MAAM,WAAW,mBAAmB;IAClC,kBAAkB,CAAC,EAAE,CAAC,OAAO,EAAE,eAAe,EAAE,KAAK,IAAI,CAAC;CAC3D;AAID,MAAM,WAAW,iBAAiB;IAChC,SAAS,CAAC,EAAE,mBAAmB,CAAC;IAChC,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,iEAAiE;IACjE,UAAU,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,eAAe,CAAC,CAAC;CAC9C;AAQD,wBAAgB,aAAa,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAUnD;AAED;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,SAAS,GAAG,SAAS,CAsC7D;AAED,qBAAa,UAAU;IAanB,OAAO,CAAC,SAAS;IAZnB,OAAO,CAAC,MAAM,CAA0B;IACxC,OAAO,CAAC,OAAO,CAA2C;IAC1D,OAAO,CAAC,WAAW,CAAyC;IAC5D,OAAO,CAAC,UAAU,CAAc;IAChC,OAAO,CAAC,OAAO,CAAc;IAC7B,OAAO,CAAC,SAAS,CAAsB;IACvC,OAAO,CAAC,UAAU,CAA8C;IAEhE,OAAO,CAAC,eAAe,CAA0C;IACjE,OAAO,CAAC,iBAAiB,CAAkC;gBAGjD,SAAS,EAAE,SAAS,EAC5B,OAAO,GAAE,iBAAsB;IAMjC;;OAEG;IACG,UAAU,CACd,OAAO,EAAE,MAAM,EACf,WAAW,GAAE,OAAe,GAC3B,OAAO,CAAC,IAAI,CAAC;IAgDV,kBAAkB,IAAI,OAAO,CAAC,SAAS,GAAG,IAAI,CAAC;IAO/C,UAAU,IAAI,OAAO,CAAC,SAAS,GAAG,IAAI,CAAC;IAgEvC,UAAU,CAAC,MAAM,EAAE,SAAS,GAAG,OAAO,CAAC,OAAO,CAAC;IAWrD,SAAS,IAAI,SAAS,GAAG,IAAI;IAI7B,aAAa,IAAI,eAAe,EAAE;IAIlC,SAAS,CAAC,IAAI,EAAE,MAAM,GAAG,eAAe,GAAG,SAAS;IAIpD,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,CAAC,eAAe,CAAC,GAAG,IAAI;IASzE,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,eAAe,GAAG,OAAO;IAuDzD,YAAY,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO;IAgB7B,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAuPnD;;;OAGG;IACH,OAAO,CAAC,iBAAiB;IA8BzB;;;OAGG;YACW,qBAAqB;IA4CnC,OAAO,CAAC,eAAe;IASjB,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IA8BtD,oBAAoB,IAAI,OAAO,EAAE;IAW3B,cAAc,CAClB,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,OAAO,CAAC,EAAE,WAAW,GACpB,OAAO,CAAC;QACT,OAAO,EAAE,OAAO,CAAC;QACjB,OAAO,EAAE,MAAM,CAAC;QAChB,UAAU,CAAC,EAAE,MAAM,CAAC;QACpB,MAAM,CAAC,EAAE,KAAK,CAAC;YAAE,IAAI,EAAE,MAAM,CAAC;YAAC,SAAS,CAAC,EAAE,MAAM,CAAA;SAAE,CAAC,CAAC;KACtD,CAAC;YAsDY,uBAAuB;IA8D/B,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAa9B;;OAEG;IACH,iBAAiB,IAAI,UAAU,EAAE;IA6BjC;;OAEG;IACH,iBAAiB,IAAI,0BAA0B,EAAE;IAIjD;;OAEG;IACG,wBAAwB,CAC5B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,OAAO,EAAE,WAAW,GACnB,OAAO,CAAC,UAAU,CAAC;IActB;;OAEG;IACH,SAAS,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO;CAcjC"}
+{"version":3,"file":"mcpManager.d.ts","sourceRoot":"","sources":["../../src/managers/mcpManager.ts"],"names":[],"mappings":"AAOA,OAAO,EAAE,0BAA0B,EAAE,MAAM,qBAAqB,CAAC;AAEjE,OAAO,KAAK,EAAE,UAAU,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAC7E,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAClD,OAAO,KAAK,EACV,MAAM,EACN,eAAe,EACf,SAAS,EACT,OAAO,EACP,eAAe,EAChB,MAAM,mBAAmB,CAAC;AAQ3B,MAAM,WAAW,mBAAmB;IAClC,kBAAkB,CAAC,EAAE,CAAC,OAAO,EAAE,eAAe,EAAE,KAAK,IAAI,CAAC;CAC3D;AAID,MAAM,WAAW,iBAAiB;IAChC,SAAS,CAAC,EAAE,mBAAmB,CAAC;IAChC,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,iEAAiE;IACjE,UAAU,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,eAAe,CAAC,CAAC;CAC9C;AAQD,wBAAgB,aAAa,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAUnD;AAED;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,SAAS,GAAG,SAAS,CAsC7D;AAED,qBAAa,UAAU;IAanB,OAAO,CAAC,SAAS;IAZnB,OAAO,CAAC,MAAM,CAA0B;IACxC,OAAO,CAAC,OAAO,CAA2C;IAC1D,OAAO,CAAC,WAAW,CAAyC;IAC5D,OAAO,CAAC,UAAU,CAAc;IAChC,OAAO,CAAC,OAAO,CAAc;IAC7B,OAAO,CAAC,SAAS,CAAsB;IACvC,OAAO,CAAC,UAAU,CAA8C;IAEhE,OAAO,CAAC,eAAe,CAA0C;IACjE,OAAO,CAAC,iBAAiB,CAAkC;gBAGjD,SAAS,EAAE,SAAS,EAC5B,OAAO,GAAE,iBAAsB;IAMjC;;OAEG;IACG,UAAU,CACd,OAAO,EAAE,MAAM,EACf,WAAW,GAAE,OAAe,GAC3B,OAAO,CAAC,IAAI,CAAC;IAgDV,kBAAkB,IAAI,OAAO,CAAC,SAAS,GAAG,IAAI,CAAC;IAO/C,UAAU,IAAI,OAAO,CAAC,SAAS,GAAG,IAAI,CAAC;IAgEvC,UAAU,CAAC,MAAM,EAAE,SAAS,GAAG,OAAO,CAAC,OAAO,CAAC;IAWrD,SAAS,IAAI,SAAS,GAAG,IAAI;IAI7B,aAAa,IAAI,eAAe,EAAE;IAIlC,SAAS,CAAC,IAAI,EAAE,MAAM,GAAG,eAAe,GAAG,SAAS;IAIpD,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,CAAC,eAAe,CAAC,GAAG,IAAI;IASzE,SAAS,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,eAAe,GAAG,OAAO;IAuDzD,YAAY,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO;IAgB7B,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAkQnD;;;OAGG;IACH,OAAO,CAAC,iBAAiB;IA8BzB;;;OAGG;YACW,qBAAqB;IA4CnC,OAAO,CAAC,eAAe;IASjB,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IA8BtD,oBAAoB,IAAI,OAAO,EAAE;IAW3B,cAAc,CAClB,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,OAAO,CAAC,EAAE,WAAW,GACpB,OAAO,CAAC;QACT,OAAO,EAAE,OAAO,CAAC;QACjB,OAAO,EAAE,MAAM,CAAC;QAChB,UAAU,CAAC,EAAE,MAAM,CAAC;QACpB,MAAM,CAAC,EAAE,KAAK,CAAC;YAAE,IAAI,EAAE,MAAM,CAAC;YAAC,SAAS,CAAC,EAAE,MAAM,CAAA;SAAE,CAAC,CAAC;KACtD,CAAC;YAsDY,uBAAuB;IA8D/B,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAa9B;;OAEG;IACH,iBAAiB,IAAI,UAAU,EAAE;IA6BjC;;OAEG;IACH,iBAAiB,IAAI,0BAA0B,EAAE;IAIjD;;OAEG;IACG,wBAAwB,CAC5B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,OAAO,EAAE,WAAW,GACnB,OAAO,CAAC,UAAU,CAAC;IActB;;OAEG;IACH,SAAS,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO;CAcjC"}

package/dist/managers/mcpManager.js

@@ -363,28 +363,35 @@ export class McpManager {
                     cwd: this.workdir, // Use the agent's workdir as the process working directory
                     stderr: "pipe", // Pipe stderr to capture it
                 });
-                // Handle stderr output for StdioClientTransport
+                // Buffer stderr silently (MCP servers use stderr for all logging;
+                // stdout is reserved for JSON-RPC). Only dump it on connection
+                // success (once) or failure — not line-by-line in real time.
+                // Aligns with Claude Code's approach to avoid noise from [INFO],
+                // [DEBUG], Warning: lines that are not actual errors.
                 const stderr = transport.stderr;
+                let stderrOutput = "";
                 if (stderr) {
-                    let buffer = "";
                     stderr.on("data", (chunk) => {
-                        buffer += chunk.toString();
-                        const lines = buffer.split("\n");
-                        buffer = lines.pop() || "";
-                        for (const line of lines) {
-                            if (line.trim()) {
-                                logger?.error(`[MCP Server ${name}] ${line}`);
-                            }
-                        }
-                    });
-                    stderr.on("end", () => {
-                        if (buffer.trim()) {
-                            logger?.error(`[MCP Server ${name}] ${buffer}`);
+                        if (stderrOutput.length < 64 * 1024 * 1024) {
+                            stderrOutput += chunk.toString();
                         }
                     });
                 }
                 client = createClient();
-                await client.connect(transport);
+                try {
+                    await client.connect(transport);
+                }
+                catch (error) {
+                    if (stderrOutput.trim()) {
+                        logger?.error(`[MCP Server ${name}] Server stderr: ${stderrOutput.trim()}`);
+                    }
+                    throw error;
+                }
+                // Dump accumulated stderr once after successful connection
+                if (stderrOutput.trim()) {
+                    logger?.debug(`[MCP Server ${name}] Server stderr: ${stderrOutput.trim()}`);
+                    stderrOutput = "";
+                }
                 const toolsResponse = await client.listTools();
                 tools =
                     toolsResponse.tools?.map((tool) => ({

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

@@ -1 +1 @@
-{"version":3,"file":"bashTool.d.ts","sourceRoot":"","sources":["../../src/tools/bashTool.ts"],"names":[],"mappings":"AASA,OAAO,KAAK,EAAE,UAAU,EAA2B,MAAM,YAAY,CAAC;AAuBtE;;GAEG;AACH,eAAO,MAAM,QAAQ,EAAE,UAmiBtB,CAAC"}
+{"version":3,"file":"bashTool.d.ts","sourceRoot":"","sources":["../../src/tools/bashTool.ts"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,UAAU,EAA2B,MAAM,YAAY,CAAC;AAuBtE;;GAEG;AACH,eAAO,MAAM,QAAQ,EAAE,UA2iBtB,CAAC"}

package/dist/tools/bashTool.js

@@ -4,6 +4,7 @@ import * as path from "path";
 import * as os from "os";
 import { logger } from "../utils/globalLogger.js";
 import { resolveShellPath } from "../utils/shellResolver.js";
+import { toPosixPath } from "../utils/path.js";
 import { stripAnsiColors } from "../utils/stringUtils.js";
 import { processToolResult } from "../utils/toolResultStorage.js";
 import { BASH_MAX_OUTPUT_CHARS } from "../constants/toolLimits.js";
@@ -211,7 +212,8 @@ The working directory persists between commands. Try to maintain your current wo
         return new Promise((resolve) => {
             // Create a temporary file to store the CWD
             const tempCwdFile = path.join(os.tmpdir(), `wave_cwd_${Date.now()}_${Math.random().toString(36).substring(2, 11)}.tmp`);
-            const wrappedCommand = `${command} && pwd -P >| ${tempCwdFile}`;
+            const tempCwdFileForBash = toPosixPath(tempCwdFile);
+            const wrappedCommand = `${command} && pwd -P >| ${tempCwdFileForBash}`;
             const child = spawn(wrappedCommand, {
                 shell: shellPath || true,
                 stdio: "pipe",
@@ -226,6 +228,17 @@ The working directory persists between commands. Try to maintain your current wo
             let isAborted = false;
             let isBackgrounded = false;
             let isFinished = false;
+            // Best-effort cleanup of the temp CWD file — used by abort/error/exit paths
+            const cleanupTempFile = () => {
+                try {
+                    if (fs.existsSync(tempCwdFile)) {
+                        fs.unlinkSync(tempCwdFile);
+                    }
+                }
+                catch {
+                    // ignore — best-effort cleanup
+                }
+            };
             const updateRealtimeResults = () => {
                 if (isAborted || isBackgrounded || isFinished)
                     return;
@@ -352,6 +365,7 @@ The working directory persists between commands. Try to maintain your current wo
                             }
                         }
                     }
+                    cleanupTempFile();
                     const processedOutput = processToolResult(outputBuffer + (errorBuffer ? "\n" + errorBuffer : ""), BASH_MAX_OUTPUT_CHARS, "bash");
                     resolve({
                         success: false,
@@ -410,15 +424,7 @@ The working directory persists between commands. Try to maintain your current wo
                         newCwd = undefined;
                     }
                     finally {
-                        // Ensure temp file is cleaned up even if reading fails
-                        try {
-                            if (fs.existsSync(tempCwdFile)) {
-                                fs.unlinkSync(tempCwdFile);
-                            }
-                        }
-                        catch (fileError) {
-                            logger.error("Failed to clean up temp CWD file:", fileError);
-                        }
+                        cleanupTempFile();
                     }
                     // If CWD changed, call the onCwdChange callback and add notification
                     let cwdMessage;
@@ -462,6 +468,7 @@ The working directory persists between commands. Try to maintain your current wo
                     if (timeoutHandle) {
                         clearTimeout(timeoutHandle);
                     }
+                    cleanupTempFile();
                     resolve({
                         success: false,
                         content: "",

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

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

package/dist/tools/editTool.js

@@ -95,9 +95,11 @@ Usage:
         }
         // Trigger conditional rule loading for this file
         context.messageManager?.triggerFileRead(filePath);
-        // Enforce read-before-edit: the file must have been read first
-        if (context.messageManager &&
-            !context.messageManager.hasFileBeenRead(filePath)) {
+        // Enforce read-before-edit: the file must have been read or written first.
+        // readFileState is populated by Read, Write, and Edit tools — single source
+        // of truth, aligned with Claude Code's readFileState approach.
+        const resolvedPath = resolvePath(filePath, context.workdir);
+        if (!context.readFileState?.has(resolvedPath)) {
             return {
                 success: false,
                 content: "",
@@ -105,7 +107,6 @@ Usage:
             };
         }
         try {
-            const resolvedPath = resolvePath(filePath, context.workdir);
             // Read file content
             let originalContent;
             try {

package/dist/utils/constants.d.ts

@@ -22,5 +22,5 @@ export declare const USER_MEMORY_FILE: string;
  * AI related constants
  */
 export declare const DEFAULT_WAVE_MAX_INPUT_TOKENS = 200000;
-export declare const DEFAULT_WAVE_MAX_OUTPUT_TOKENS = 16384;
+export declare const DEFAULT_WAVE_MAX_OUTPUT_TOKENS = 32000;
 //# sourceMappingURL=constants.d.ts.map

package/dist/utils/constants.js

@@ -24,4 +24,4 @@ export const USER_MEMORY_FILE = path.join(DATA_DIRECTORY, "AGENTS.md");
  * AI related constants
  */
 export const DEFAULT_WAVE_MAX_INPUT_TOKENS = 200000; // Default token limit
-export const DEFAULT_WAVE_MAX_OUTPUT_TOKENS = 16384; // Default output token limit
+export const DEFAULT_WAVE_MAX_OUTPUT_TOKENS = 32000; // Default output token limit (aligned with Claude Code)

package/dist/utils/gitUtils.js

@@ -1,6 +1,6 @@
 import * as path from "node:path";
 import * as fsSync from "node:fs";
-import { execSync } from "node:child_process";
+import { execFileSync } from "node:child_process";
 /**
  * Check if a directory is a git repository
  * @param dirPath Directory path
@@ -30,7 +30,7 @@ export function isGitRepository(dirPath) {
  */
 export function getGitRepoRoot(cwd) {
     try {
-        return execSync("git rev-parse --show-toplevel", {
+        return execFileSync("git", ["rev-parse", "--show-toplevel"], {
             cwd,
             encoding: "utf8",
             stdio: ["ignore", "pipe", "ignore"],
@@ -47,7 +47,7 @@ export function getGitRepoRoot(cwd) {
  */
 export function getGitCommonDir(cwd) {
     try {
-        const commonDir = execSync("git rev-parse --git-common-dir", {
+        const commonDir = execFileSync("git", ["rev-parse", "--git-common-dir"], {
             cwd,
             encoding: "utf8",
             stdio: ["ignore", "pipe", "ignore"],
@@ -65,7 +65,7 @@ export function getGitCommonDir(cwd) {
  */
 export function getGitMainRepoRoot(cwd) {
     try {
-        const output = execSync("git worktree list --porcelain", {
+        const output = execFileSync("git", ["worktree", "list", "--porcelain"], {
             cwd,
             encoding: "utf8",
             stdio: ["ignore", "pipe", "ignore"],
@@ -223,7 +223,7 @@ export function getDefaultRemoteBranch(cwd) {
  */
 export function hasUncommittedChanges(cwd) {
     try {
-        const status = execSync("git status --porcelain", {
+        const status = execFileSync("git", ["status", "--porcelain"], {
             cwd,
             encoding: "utf8",
             stdio: ["ignore", "pipe", "ignore"],
@@ -243,7 +243,7 @@ export function hasUncommittedChanges(cwd) {
 export function hasNewCommits(cwd, baseBranch) {
     try {
         const range = baseBranch ? `${baseBranch}..HEAD` : "@{u}..HEAD";
-        const log = execSync(`git log ${range} --oneline`, {
+        const log = execFileSync("git", ["log", range, "--oneline"], {
             cwd,
             encoding: "utf8",
             stdio: ["ignore", "pipe", "ignore"],

package/dist/utils/openaiClient.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"openaiClient.d.ts","sourceRoot":"","sources":["../../src/utils/openaiClient.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,sCAAsC,EACtC,mCAAmC,EACnC,mBAAmB,EACnB,cAAc,EACf,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAyBnD,KAAK,YAAY,GACb,sCAAsC,GACtC,mCAAmC,CAAC;AAExC,UAAU,WAAW,CAAC,CAAC;IACrB,IAAI,EAAE,CAAC,CAAC;IACR,QAAQ,EAAE,QAAQ,CAAC;CACpB;AAED,UAAU,UAAU,CAAC,CAAC,CAAE,SAAQ,OAAO,CAAC,CAAC,CAAC;IACxC,YAAY,IAAI,OAAO,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,CAAC;CACzC;AAED,qBAAa,YAAY;IACX,OAAO,CAAC,MAAM;gBAAN,MAAM,EAAE,aAAa;IAEzC,IAAI,IAAI;;qBAGO,CAAC,SAAS,YAAY,UACrB,CAAC,YACC;gBAAE,MAAM,CAAC,EAAE,WAAW,CAAA;aAAE,KACjC,UAAU,CACX,CAAC,SAAS,mCAAmC,GACzC,aAAa,CAAC,mBAAmB,CAAC,GAClC,cAAc,CACnB;;MA2BN;YAEa,OAAO;YA0IN,oBAAoB;CAyCpC"}
+{"version":3,"file":"openaiClient.d.ts","sourceRoot":"","sources":["../../src/utils/openaiClient.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,sCAAsC,EACtC,mCAAmC,EACnC,mBAAmB,EACnB,cAAc,EACf,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAyBnD,KAAK,YAAY,GACb,sCAAsC,GACtC,mCAAmC,CAAC;AAExC,UAAU,WAAW,CAAC,CAAC;IACrB,IAAI,EAAE,CAAC,CAAC;IACR,QAAQ,EAAE,QAAQ,CAAC;CACpB;AAED,UAAU,UAAU,CAAC,CAAC,CAAE,SAAQ,OAAO,CAAC,CAAC,CAAC;IACxC,YAAY,IAAI,OAAO,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,CAAC;CACzC;AAED,qBAAa,YAAY;IACX,OAAO,CAAC,MAAM;gBAAN,MAAM,EAAE,aAAa;IAEzC,IAAI,IAAI;;qBAGO,CAAC,SAAS,YAAY,UACrB,CAAC,YACC;gBAAE,MAAM,CAAC,EAAE,WAAW,CAAA;aAAE,KACjC,UAAU,CACX,CAAC,SAAS,mCAAmC,GACzC,aAAa,CAAC,mBAAmB,CAAC,GAClC,cAAc,CACnB;;MA2BN;YAEa,OAAO;YAuHN,oBAAoB;CAyCpC"}

package/dist/utils/openaiClient.js

@@ -91,36 +91,16 @@ export class OpenAIClient {
                     };
                 }
             }
-            let errorBody;
+            let responseText = "";
             try {
-                const text = await response.text();
-                try {
-                    errorBody = JSON.parse(text);
-                }
-                catch (e) {
-                    logger.error("Failed to parse error response body as JSON", {
-                        error: e,
-                        text,
-                    });
-                    errorBody = text;
-                }
+                responseText = await response.text();
             }
             catch (e) {
                 logger.error("Failed to read error response text", { error: e });
-                errorBody = {};
             }
-            const error = new Error(typeof errorBody === "object" &&
-                errorBody !== null &&
-                "error" in errorBody &&
-                typeof errorBody.error === "object" &&
-                errorBody.error !== null &&
-                "message" in errorBody.error
-                ? String(errorBody.error.message)
-                : typeof errorBody === "string"
-                    ? errorBody
-                    : response.statusText);
+            const error = new Error(`HTTP ${response.status}${response.statusText ? ` ${response.statusText}` : ""}: ${responseText}`);
             error.status = response.status;
-            error.body = errorBody;
+            error.body = responseText;
             const retryableStatus = response.status === 429 ||
                 (response.status >= 500 && response.status !== 501);
             if (retryableStatus && attempt < MAX_RETRIES) {
@@ -136,7 +116,7 @@ export class OpenAIClient {
             logger.error("OpenAI API Error:", {
                 status: response.status,
                 statusText: response.statusText,
-                errorBody,
+                body: responseText,
             });
             throw error;
         }

package/dist/utils/path.d.ts

@@ -12,4 +12,11 @@ export declare function resolvePath(filePath: string, workdir: string): string;
  * @returns Path for display (relative or absolute)
  */
 export declare function getDisplayPath(filePath: string, workdir: string): string;
+/**
+ * Convert backslashes to forward slashes for shell compatibility on Windows.
+ * On Windows, os.tmpdir() returns paths with backslashes (e.g., C:\Users\foo\Temp),
+ * which are treated as escape characters when interpolated into shell command strings.
+ * Returns the path as-is on non-Windows platforms.
+ */
+export declare function toPosixPath(p: string): string;
 //# sourceMappingURL=path.d.ts.map

package/dist/utils/path.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"path.d.ts","sourceRoot":"","sources":["../../src/utils/path.ts"],"names":[],"mappings":"AAIA;;;;;GAKG;AACH,wBAAgB,WAAW,CAAC,QAAQ,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAarE;AAED;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAwBxE"}
+{"version":3,"file":"path.d.ts","sourceRoot":"","sources":["../../src/utils/path.ts"],"names":[],"mappings":"AAIA;;;;;GAKG;AACH,wBAAgB,WAAW,CAAC,QAAQ,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAarE;AAED;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAwBxE;AAED;;;;;GAKG;AACH,wBAAgB,WAAW,CAAC,CAAC,EAAE,MAAM,GAAG,MAAM,CAE7C"}

package/dist/utils/path.js

@@ -46,3 +46,12 @@ export function getDisplayPath(filePath, workdir) {
     }
     return filePath;
 }
+/**
+ * Convert backslashes to forward slashes for shell compatibility on Windows.
+ * On Windows, os.tmpdir() returns paths with backslashes (e.g., C:\Users\foo\Temp),
+ * which are treated as escape characters when interpolated into shell command strings.
+ * Returns the path as-is on non-Windows platforms.
+ */
+export function toPosixPath(p) {
+    return process.platform === "win32" ? p.replace(/\\/g, "/") : p;
+}

package/dist/utils/worktreeUtils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"worktreeUtils.d.ts","sourceRoot":"","sources":["../../src/utils/worktreeUtils.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAQH,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,OAAO,CAAC;IACf,mFAAmF;IACnF,kBAAkB,CAAC,EAAE,MAAM,CAAC;CAC7B;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,CAmBvD;AAED;;GAEG;AACH,wBAAgB,oBAAoB,IAAI,MAAM,CA6B7C;AAED;;GAEG;AACH,wBAAgB,aAAa,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,CAKjD;AAED;;GAEG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,YAAY,CAuJtE;AAED;;GAEG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,YAAY,GAAG,IAAI,CA+DvD;AAED;;;GAGG;AACH,wBAAgB,oBAAoB,CAClC,YAAY,EAAE,MAAM,EACpB,kBAAkB,EAAE,MAAM,GAAG,SAAS,GACrC;IAAE,YAAY,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CA8BlD"}
+{"version":3,"file":"worktreeUtils.d.ts","sourceRoot":"","sources":["../../src/utils/worktreeUtils.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAQH,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,OAAO,CAAC;IACf,mFAAmF;IACnF,kBAAkB,CAAC,EAAE,MAAM,CAAC;CAC7B;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,CAmBvD;AAED;;GAEG;AACH,wBAAgB,oBAAoB,IAAI,MAAM,CA6B7C;AAED;;GAEG;AACH,wBAAgB,aAAa,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,CAKjD;AAED;;GAEG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,YAAY,CA6JtE;AAED;;GAEG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,YAAY,GAAG,IAAI,CAmEvD;AAED;;;GAGG;AACH,wBAAgB,oBAAoB,CAClC,YAAY,EAAE,MAAM,EACpB,kBAAkB,EAAE,MAAM,GAAG,SAAS,GACrC;IAAE,YAAY,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CAsClD"}

package/dist/utils/worktreeUtils.js

@@ -2,7 +2,7 @@
  * Git worktree creation and removal utilities for the SDK.
  * Used by EnterWorktree and ExitWorktree tools.
  */
-import { execSync } from "node:child_process";
+import { execFileSync } from "node:child_process";
 import * as path from "node:path";
 import * as fs from "node:fs";
 import { getGitMainRepoRoot, getDefaultRemoteBranch } from "./gitUtils.js";
@@ -61,7 +61,7 @@ export function generateWorktreeName() {
  * Get the current HEAD commit SHA.
  */
 export function getHeadCommit(cwd) {
-    return execSync(`git -C "${cwd}" rev-parse HEAD`, {
+    return execFileSync("git", ["-C", cwd, "rev-parse", "HEAD"], {
         encoding: "utf8",
         stdio: ["ignore", "pipe", "ignore"],
     }).trim();
@@ -97,7 +97,7 @@ export function createWorktree(name, cwd) {
     }
     try {
         // Create worktree and branch
-        execSync(`git worktree add -b ${branchName} "${worktreePath}" ${baseBranch}`, {
+        execFileSync("git", ["worktree", "add", "-b", branchName, worktreePath, baseBranch], {
             cwd: repoRoot,
             stdio: ["ignore", "pipe", "pipe"],
             env: {
@@ -120,7 +120,7 @@ export function createWorktree(name, cwd) {
         if (stderr.includes("already exists")) {
             // Branch exists but worktree doesn't — attach to existing branch
             try {
-                execSync(`git worktree add "${worktreePath}" ${branchName}`, {
+                execFileSync("git", ["worktree", "add", worktreePath, branchName], {
                     cwd: repoRoot,
                     stdio: ["ignore", "pipe", "pipe"],
                     env: {
@@ -147,7 +147,7 @@ export function createWorktree(name, cwd) {
             // Base branch not fetched yet — try fetching then retrying
             const branchNameOnly = baseBranch.split("/").pop();
             try {
-                execSync(`git fetch origin ${branchNameOnly}`, {
+                execFileSync("git", ["fetch", "origin", branchNameOnly], {
                     cwd: repoRoot,
                     stdio: ["ignore", "pipe", "pipe"],
                     env: {
@@ -156,7 +156,7 @@ export function createWorktree(name, cwd) {
                         GIT_ASKPASS: "",
                     },
                 });
-                execSync(`git worktree add -b ${branchName} "${worktreePath}" ${baseBranch}`, {
+                execFileSync("git", ["worktree", "add", "-b", branchName, worktreePath, baseBranch], {
                     cwd: repoRoot,
                     stdio: ["ignore", "pipe", "pipe"],
                     env: {
@@ -177,7 +177,7 @@ export function createWorktree(name, cwd) {
             catch {
                 // Fetch or retry failed — fall back to HEAD
                 try {
-                    execSync(`git worktree add -b ${branchName} "${worktreePath}" HEAD`, {
+                    execFileSync("git", ["worktree", "add", "-b", branchName, worktreePath, "HEAD"], {
                         cwd: repoRoot,
                         stdio: ["ignore", "pipe", "pipe"],
                         env: {
@@ -212,7 +212,7 @@ export function removeWorktree(info) {
         // Get current branch in worktree before removing
         let currentBranch;
         try {
-            currentBranch = execSync(`git rev-parse --abbrev-ref HEAD`, {
+            currentBranch = execFileSync("git", ["rev-parse", "--abbrev-ref", "HEAD"], {
                 cwd: info.path,
                 encoding: "utf8",
                 stdio: ["ignore", "pipe", "ignore"],
@@ -222,13 +222,13 @@ export function removeWorktree(info) {
             // Ignore errors
         }
         // Remove worktree
-        execSync(`git worktree remove --force "${info.path}"`, {
+        execFileSync("git", ["worktree", "remove", "--force", info.path], {
             cwd: repoRoot,
             stdio: ["ignore", "pipe", "pipe"],
         });
         // Delete worktree branch
         try {
-            execSync(`git branch -D ${info.branch}`, {
+            execFileSync("git", ["branch", "-D", info.branch], {
                 cwd: repoRoot,
                 stdio: ["ignore", "pipe", "pipe"],
             });
@@ -246,7 +246,7 @@ export function removeWorktree(info) {
                 currentBranch !== "main" &&
                 currentBranch !== "master") {
                 try {
-                    execSync(`git branch -D ${currentBranch}`, {
+                    execFileSync("git", ["branch", "-D", currentBranch], {
                         cwd: repoRoot,
                         stdio: ["ignore", "pipe", "pipe"],
                     });
@@ -271,7 +271,7 @@ export function removeWorktree(info) {
  */
 export function countWorktreeChanges(worktreePath, originalHeadCommit) {
     try {
-        const statusOutput = execSync(`git -C "${worktreePath}" status --porcelain`, {
+        const statusOutput = execFileSync("git", ["-C", worktreePath, "status", "--porcelain"], {
             encoding: "utf8",
             stdio: ["ignore", "pipe", "ignore"],
         });
@@ -281,7 +281,13 @@ export function countWorktreeChanges(worktreePath, originalHeadCommit) {
         if (!originalHeadCommit) {
             return null;
         }
-        const revListOutput = execSync(`git -C "${worktreePath}" rev-list --count ${originalHeadCommit}..HEAD`, {
+        const revListOutput = execFileSync("git", [
+            "-C",
+            worktreePath,
+            "rev-list",
+            "--count",
+            `${originalHeadCommit}..HEAD`,
+        ], {
             encoding: "utf8",
             stdio: ["ignore", "pipe", "ignore"],
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wave-agent-sdk",
-  "version": "0.18.4",
+  "version": "0.18.5",
   "description": "SDK for building AI-powered development tools and agents",
   "keywords": [
     "ai",

package/src/managers/mcpManager.ts

@@ -484,29 +484,40 @@ export class McpManager {
           stderr: "pipe", // Pipe stderr to capture it
         });
 
-        // Handle stderr output for StdioClientTransport
+        // Buffer stderr silently (MCP servers use stderr for all logging;
+        // stdout is reserved for JSON-RPC). Only dump it on connection
+        // success (once) or failure — not line-by-line in real time.
+        // Aligns with Claude Code's approach to avoid noise from [INFO],
+        // [DEBUG], Warning: lines that are not actual errors.
         const stderr = (transport as StdioClientTransport).stderr;
+        let stderrOutput = "";
         if (stderr) {
-          let buffer = "";
           stderr.on("data", (chunk: Buffer) => {
-            buffer += chunk.toString();
-            const lines = buffer.split("\n");
-            buffer = lines.pop() || "";
-            for (const line of lines) {
-              if (line.trim()) {
-                logger?.error(`[MCP Server ${name}] ${line}`);
-              }
-            }
-          });
-          stderr.on("end", () => {
-            if (buffer.trim()) {
-              logger?.error(`[MCP Server ${name}] ${buffer}`);
+            if (stderrOutput.length < 64 * 1024 * 1024) {
+              stderrOutput += chunk.toString();
             }
           });
         }
 
         client = createClient();
-        await client.connect(transport);
+        try {
+          await client.connect(transport);
+        } catch (error) {
+          if (stderrOutput.trim()) {
+            logger?.error(
+              `[MCP Server ${name}] Server stderr: ${stderrOutput.trim()}`,
+            );
+          }
+          throw error;
+        }
+
+        // Dump accumulated stderr once after successful connection
+        if (stderrOutput.trim()) {
+          logger?.debug(
+            `[MCP Server ${name}] Server stderr: ${stderrOutput.trim()}`,
+          );
+          stderrOutput = "";
+        }
 
         const toolsResponse = await client.listTools();
         tools =

package/src/tools/bashTool.ts

@@ -4,6 +4,7 @@ import * as path from "path";
 import * as os from "os";
 import { logger } from "../utils/globalLogger.js";
 import { resolveShellPath } from "../utils/shellResolver.js";
+import { toPosixPath } from "../utils/path.js";
 import { stripAnsiColors } from "../utils/stringUtils.js";
 import { processToolResult } from "../utils/toolResultStorage.js";
 import { BASH_MAX_OUTPUT_CHARS } from "../constants/toolLimits.js";
@@ -246,7 +247,8 @@ The working directory persists between commands. Try to maintain your current wo
         os.tmpdir(),
         `wave_cwd_${Date.now()}_${Math.random().toString(36).substring(2, 11)}.tmp`,
       );
-      const wrappedCommand = `${command} && pwd -P >| ${tempCwdFile}`;
+      const tempCwdFileForBash = toPosixPath(tempCwdFile);
+      const wrappedCommand = `${command} && pwd -P >| ${tempCwdFileForBash}`;
 
       const child: ChildProcess = spawn(wrappedCommand, {
         shell: shellPath || true,
@@ -264,6 +266,17 @@ The working directory persists between commands. Try to maintain your current wo
       let isBackgrounded = false;
       let isFinished = false;
 
+      // Best-effort cleanup of the temp CWD file — used by abort/error/exit paths
+      const cleanupTempFile = () => {
+        try {
+          if (fs.existsSync(tempCwdFile)) {
+            fs.unlinkSync(tempCwdFile);
+          }
+        } catch {
+          // ignore — best-effort cleanup
+        }
+      };
+
       const updateRealtimeResults = () => {
         if (isAborted || isBackgrounded || isFinished) return;
 
@@ -420,6 +433,8 @@ The working directory persists between commands. Try to maintain your current wo
             }
           }
 
+          cleanupTempFile();
+
           const processedOutput = processToolResult(
             outputBuffer + (errorBuffer ? "\n" + errorBuffer : ""),
             BASH_MAX_OUTPUT_CHARS,
@@ -491,14 +506,7 @@ The working directory persists between commands. Try to maintain your current wo
             );
             newCwd = undefined;
           } finally {
-            // Ensure temp file is cleaned up even if reading fails
-            try {
-              if (fs.existsSync(tempCwdFile)) {
-                fs.unlinkSync(tempCwdFile);
-              }
-            } catch (fileError) {
-              logger.error("Failed to clean up temp CWD file:", fileError);
-            }
+            cleanupTempFile();
           }
 
           // If CWD changed, call the onCwdChange callback and add notification
@@ -560,6 +568,7 @@ The working directory persists between commands. Try to maintain your current wo
           if (timeoutHandle) {
             clearTimeout(timeoutHandle);
           }
+          cleanupTempFile();
           resolve({
             success: false,
             content: "",

package/src/tools/editTool.ts

@@ -112,11 +112,11 @@ Usage:
     // Trigger conditional rule loading for this file
     context.messageManager?.triggerFileRead(filePath);
 
-    // Enforce read-before-edit: the file must have been read first
-    if (
-      context.messageManager &&
-      !context.messageManager.hasFileBeenRead(filePath)
-    ) {
+    // Enforce read-before-edit: the file must have been read or written first.
+    // readFileState is populated by Read, Write, and Edit tools — single source
+    // of truth, aligned with Claude Code's readFileState approach.
+    const resolvedPath = resolvePath(filePath, context.workdir);
+    if (!context.readFileState?.has(resolvedPath)) {
       return {
         success: false,
         content: "",
@@ -125,8 +125,6 @@ Usage:
     }
 
     try {
-      const resolvedPath = resolvePath(filePath, context.workdir);
-
       // Read file content
       let originalContent: string;
       try {

package/src/utils/constants.ts

@@ -30,4 +30,4 @@ export const USER_MEMORY_FILE = path.join(DATA_DIRECTORY, "AGENTS.md");
  * AI related constants
  */
 export const DEFAULT_WAVE_MAX_INPUT_TOKENS = 200000; // Default token limit
-export const DEFAULT_WAVE_MAX_OUTPUT_TOKENS = 16384; // Default output token limit
+export const DEFAULT_WAVE_MAX_OUTPUT_TOKENS = 32000; // Default output token limit (aligned with Claude Code)

package/src/utils/gitUtils.ts

@@ -1,6 +1,6 @@
 import * as path from "node:path";
 import * as fsSync from "node:fs";
-import { execSync } from "node:child_process";
+import { execFileSync } from "node:child_process";
 
 /**
  * Check if a directory is a git repository
@@ -31,7 +31,7 @@ export function isGitRepository(dirPath: string): string {
  */
 export function getGitRepoRoot(cwd: string): string {
   try {
-    return execSync("git rev-parse --show-toplevel", {
+    return execFileSync("git", ["rev-parse", "--show-toplevel"], {
       cwd,
       encoding: "utf8",
       stdio: ["ignore", "pipe", "ignore"],
@@ -48,7 +48,7 @@ export function getGitRepoRoot(cwd: string): string {
  */
 export function getGitCommonDir(cwd: string): string {
   try {
-    const commonDir = execSync("git rev-parse --git-common-dir", {
+    const commonDir = execFileSync("git", ["rev-parse", "--git-common-dir"], {
       cwd,
       encoding: "utf8",
       stdio: ["ignore", "pipe", "ignore"],
@@ -66,7 +66,7 @@ export function getGitCommonDir(cwd: string): string {
  */
 export function getGitMainRepoRoot(cwd: string): string {
   try {
-    const output = execSync("git worktree list --porcelain", {
+    const output = execFileSync("git", ["worktree", "list", "--porcelain"], {
       cwd,
       encoding: "utf8",
       stdio: ["ignore", "pipe", "ignore"],
@@ -229,7 +229,7 @@ export function getDefaultRemoteBranch(cwd: string): string {
  */
 export function hasUncommittedChanges(cwd: string): boolean {
   try {
-    const status = execSync("git status --porcelain", {
+    const status = execFileSync("git", ["status", "--porcelain"], {
       cwd,
       encoding: "utf8",
       stdio: ["ignore", "pipe", "ignore"],
@@ -249,7 +249,7 @@ export function hasUncommittedChanges(cwd: string): boolean {
 export function hasNewCommits(cwd: string, baseBranch?: string): boolean {
   try {
     const range = baseBranch ? `${baseBranch}..HEAD` : "@{u}..HEAD";
-    const log = execSync(`git log ${range} --oneline`, {
+    const log = execFileSync("git", ["log", range, "--oneline"], {
       cwd,
       encoding: "utf8",
       stdio: ["ignore", "pipe", "ignore"],

package/src/utils/openaiClient.ts

@@ -166,37 +166,18 @@ export class OpenAIClient {
         }
       }
 
-      let errorBody: unknown;
+      let responseText = "";
       try {
-        const text = await response.text();
-        try {
-          errorBody = JSON.parse(text);
-        } catch (e) {
-          logger.error("Failed to parse error response body as JSON", {
-            error: e,
-            text,
-          });
-          errorBody = text;
-        }
+        responseText = await response.text();
       } catch (e) {
         logger.error("Failed to read error response text", { error: e });
-        errorBody = {};
       }
 
       const error = new Error(
-        typeof errorBody === "object" &&
-        errorBody !== null &&
-        "error" in errorBody &&
-        typeof (errorBody as { error: unknown }).error === "object" &&
-        (errorBody as { error: object }).error !== null &&
-        "message" in (errorBody as { error: { message: unknown } }).error
-          ? String((errorBody as { error: { message: string } }).error.message)
-          : typeof errorBody === "string"
-            ? errorBody
-            : response.statusText,
+        `HTTP ${response.status}${response.statusText ? ` ${response.statusText}` : ""}: ${responseText}`,
       ) as Error & { status?: number; body?: unknown };
       error.status = response.status;
-      error.body = errorBody;
+      error.body = responseText;
 
       const retryableStatus =
         response.status === 429 ||
@@ -215,7 +196,7 @@ export class OpenAIClient {
       logger.error("OpenAI API Error:", {
         status: response.status,
         statusText: response.statusText,
-        errorBody,
+        body: responseText,
       });
       throw error;
     }

package/src/utils/path.ts

@@ -54,3 +54,13 @@ export function getDisplayPath(filePath: string, workdir: string): string {
   }
   return filePath;
 }
+
+/**
+ * Convert backslashes to forward slashes for shell compatibility on Windows.
+ * On Windows, os.tmpdir() returns paths with backslashes (e.g., C:\Users\foo\Temp),
+ * which are treated as escape characters when interpolated into shell command strings.
+ * Returns the path as-is on non-Windows platforms.
+ */
+export function toPosixPath(p: string): string {
+  return process.platform === "win32" ? p.replace(/\\/g, "/") : p;
+}

package/src/utils/worktreeUtils.ts

@@ -3,7 +3,7 @@
  * Used by EnterWorktree and ExitWorktree tools.
  */
 
-import { execSync } from "node:child_process";
+import { execFileSync } from "node:child_process";
 import * as path from "node:path";
 import * as fs from "node:fs";
 import { getGitMainRepoRoot, getDefaultRemoteBranch } from "./gitUtils.js";
@@ -81,7 +81,7 @@ export function generateWorktreeName(): string {
  * Get the current HEAD commit SHA.
  */
 export function getHeadCommit(cwd: string): string {
-  return execSync(`git -C "${cwd}" rev-parse HEAD`, {
+  return execFileSync("git", ["-C", cwd, "rev-parse", "HEAD"], {
     encoding: "utf8",
     stdio: ["ignore", "pipe", "ignore"],
   }).trim();
@@ -125,8 +125,9 @@ export function createWorktree(name: string, cwd: string): WorktreeInfo {
 
   try {
     // Create worktree and branch
-    execSync(
-      `git worktree add -b ${branchName} "${worktreePath}" ${baseBranch}`,
+    execFileSync(
+      "git",
+      ["worktree", "add", "-b", branchName, worktreePath, baseBranch],
       {
         cwd: repoRoot,
         stdio: ["ignore", "pipe", "pipe"],
@@ -151,7 +152,7 @@ export function createWorktree(name: string, cwd: string): WorktreeInfo {
     if (stderr.includes("already exists")) {
       // Branch exists but worktree doesn't — attach to existing branch
       try {
-        execSync(`git worktree add "${worktreePath}" ${branchName}`, {
+        execFileSync("git", ["worktree", "add", worktreePath, branchName], {
           cwd: repoRoot,
           stdio: ["ignore", "pipe", "pipe"],
           env: {
@@ -181,7 +182,7 @@ export function createWorktree(name: string, cwd: string): WorktreeInfo {
       // Base branch not fetched yet — try fetching then retrying
       const branchNameOnly = baseBranch.split("/").pop()!;
       try {
-        execSync(`git fetch origin ${branchNameOnly}`, {
+        execFileSync("git", ["fetch", "origin", branchNameOnly], {
           cwd: repoRoot,
           stdio: ["ignore", "pipe", "pipe"],
           env: {
@@ -190,8 +191,9 @@ export function createWorktree(name: string, cwd: string): WorktreeInfo {
             GIT_ASKPASS: "",
           },
         });
-        execSync(
-          `git worktree add -b ${branchName} "${worktreePath}" ${baseBranch}`,
+        execFileSync(
+          "git",
+          ["worktree", "add", "-b", branchName, worktreePath, baseBranch],
           {
             cwd: repoRoot,
             stdio: ["ignore", "pipe", "pipe"],
@@ -213,15 +215,19 @@ export function createWorktree(name: string, cwd: string): WorktreeInfo {
       } catch {
         // Fetch or retry failed — fall back to HEAD
         try {
-          execSync(`git worktree add -b ${branchName} "${worktreePath}" HEAD`, {
-            cwd: repoRoot,
-            stdio: ["ignore", "pipe", "pipe"],
-            env: {
-              ...process.env,
-              GIT_TERMINAL_PROMPT: "0",
-              GIT_ASKPASS: "",
+          execFileSync(
+            "git",
+            ["worktree", "add", "-b", branchName, worktreePath, "HEAD"],
+            {
+              cwd: repoRoot,
+              stdio: ["ignore", "pipe", "pipe"],
+              env: {
+                ...process.env,
+                GIT_TERMINAL_PROMPT: "0",
+                GIT_ASKPASS: "",
+              },
             },
-          });
+          );
           return {
             name,
             path: worktreePath,
@@ -253,24 +259,28 @@ export function removeWorktree(info: WorktreeInfo): void {
     // Get current branch in worktree before removing
     let currentBranch: string | undefined;
     try {
-      currentBranch = execSync(`git rev-parse --abbrev-ref HEAD`, {
-        cwd: info.path,
-        encoding: "utf8",
-        stdio: ["ignore", "pipe", "ignore"],
-      }).trim();
+      currentBranch = execFileSync(
+        "git",
+        ["rev-parse", "--abbrev-ref", "HEAD"],
+        {
+          cwd: info.path,
+          encoding: "utf8",
+          stdio: ["ignore", "pipe", "ignore"],
+        },
+      ).trim();
     } catch {
       // Ignore errors
     }
 
     // Remove worktree
-    execSync(`git worktree remove --force "${info.path}"`, {
+    execFileSync("git", ["worktree", "remove", "--force", info.path], {
       cwd: repoRoot,
       stdio: ["ignore", "pipe", "pipe"],
     });
 
     // Delete worktree branch
     try {
-      execSync(`git branch -D ${info.branch}`, {
+      execFileSync("git", ["branch", "-D", info.branch], {
         cwd: repoRoot,
         stdio: ["ignore", "pipe", "pipe"],
       });
@@ -293,7 +303,7 @@ export function removeWorktree(info: WorktreeInfo): void {
         currentBranch !== "master"
       ) {
         try {
-          execSync(`git branch -D ${currentBranch}`, {
+          execFileSync("git", ["branch", "-D", currentBranch], {
             cwd: repoRoot,
             stdio: ["ignore", "pipe", "pipe"],
           });
@@ -320,8 +330,9 @@ export function countWorktreeChanges(
   originalHeadCommit: string | undefined,
 ): { changedFiles: number; commits: number } | null {
   try {
-    const statusOutput = execSync(
-      `git -C "${worktreePath}" status --porcelain`,
+    const statusOutput = execFileSync(
+      "git",
+      ["-C", worktreePath, "status", "--porcelain"],
       {
         encoding: "utf8",
         stdio: ["ignore", "pipe", "ignore"],
@@ -335,8 +346,15 @@ export function countWorktreeChanges(
       return null;
     }
 
-    const revListOutput = execSync(
-      `git -C "${worktreePath}" rev-list --count ${originalHeadCommit}..HEAD`,
+    const revListOutput = execFileSync(
+      "git",
+      [
+        "-C",
+        worktreePath,
+        "rev-list",
+        "--count",
+        `${originalHeadCommit}..HEAD`,
+      ],
       {
         encoding: "utf8",
         stdio: ["ignore", "pipe", "ignore"],

package/builtin/skills/settings/MEMORY_RULES.md

@@ -1,60 +0,0 @@
-# Wave Memory Rules Configuration
-
-Memory rules allow you to provide context-specific instructions and guidelines to the agent. This document explains how to create and manage memory rules in Wave.
-
-## What are Memory Rules?
-
-Memory rules are Markdown files that contain instructions for the agent. They are used to:
-- Enforce coding styles and conventions.
-- Provide project-specific context (e.g., "always use pnpm").
-- Define architectural patterns and best practices.
-- Share common knowledge across the team.
-
-## Creating Memory Rules
-
-Wave looks for memory rules in the following locations:
-
-1.  **User Scope**: `~/.wave/rules/*.md` (Global memory rules)
-2.  **Project Scope**: `.wave/rules/*.md` (Project-specific memory rules)
-3.  **Project Root**: `AGENTS.md` (Legacy project-level memory rules)
-
-### File Structure
-
-A memory rule file is a standard Markdown file. It can optionally include YAML frontmatter to scope the rules to specific file paths.
-
-```markdown
----
-paths:
-  - "src/api/**/*.ts"
-  - "src/services/**/*.ts"
----
-
-# API and Service Guidelines
-
-- Always use `async/await` for asynchronous operations.
-- Use `Zod` for input validation.
-- Follow the repository pattern for data access.
-```
-
-### YAML Frontmatter Fields
-
-- `paths`: (Optional) A list of glob patterns. The rules in this file will only be active when the agent is working with files that match these patterns. If omitted, the rules are always active.
-
-## How Memory Rules are Loaded
-
-Wave automatically discovers and loads all `.md` files in the `.wave/rules/` directory and its immediate subdirectories.
-
-- **Path-Specific Activation**: If a memory rule has a `paths` field, it is only included in the agent's context if *any* file currently being read or modified matches the glob patterns.
-- **Union of Rules**: If multiple files are in context, Wave activates the union of all matching memory rules.
-- **Priority**: Project-level memory rules take priority over user-level memory rules if there is a conflict.
-
-## Best Practices
-
-- **Keep rules focused**: Create separate files for different topics (e.g., `testing.md`, `ui-components.md`).
-- **Use clear instructions**: Write rules in a way that is easy for the agent to understand and follow.
-- **Leverage path scoping**: Use the `paths` field to keep the agent's context window clean and relevant.
-- **Share rules with your team**: Commit `.wave/rules/` to your git repository to ensure everyone on the team has the same context.
-
-## Auto-Memory
-
-In addition to manual memory rules, Wave also has an **auto-memory** feature that automatically remembers important information across sessions. This is stored in `~/.wave/projects/<project-id>/memory/MEMORY.md`. You can disable this feature in `settings.json` by setting `"autoMemoryEnabled": false`.