wave-agent-sdk 0.5.0 → 0.6.2

package/dist/services/session.js

@@ -62,6 +62,7 @@ async function updateSessionIndex(projectDirPath, metadata) {
         ...rest,
         lastActiveAt: metadata.lastActiveAt.toISOString(),
         firstMessage: metadata.firstMessage || index.sessions[id]?.firstMessage,
+        parentSessionId: metadata.parentSessionId,
     };
     index.lastUpdated = new Date().toISOString();
     await fs.writeFile(indexPath, JSON.stringify(index, null, 2), "utf8");
@@ -126,7 +127,7 @@ export async function createSession(sessionId, workdir, sessionType = "main") {
  * @param workdir - Working directory for the session
  * @param sessionType - Type of session ("main" or "subagent", defaults to "main")
  */
-export async function appendMessages(sessionId, newMessages, workdir, sessionType = "main") {
+export async function appendMessages(sessionId, newMessages, workdir, sessionType = "main", rootSessionId, parentSessionId) {
     // Do not save session files in test environment
     if (process.env.NODE_ENV === "test") {
         return;
@@ -174,6 +175,8 @@ export async function appendMessages(sessionId, newMessages, workdir, sessionTyp
     }
     await updateSessionIndex(projectDir.encodedPath, {
         id: sessionId,
+        rootSessionId,
+        parentSessionId,
         sessionType,
         workdir,
         lastActiveAt: new Date(lastMessage.timestamp),
@@ -202,8 +205,25 @@ export async function loadSessionFromJsonl(sessionId, workdir, sessionType = "ma
         }
         // Extract metadata from messages
         const lastMessage = messages[messages.length - 1];
+        // Try to get rootSessionId and parentSessionId from index
+        let rootSessionId;
+        let parentSessionId;
+        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);
+            rootSessionId = index.sessions[sessionId]?.rootSessionId;
+            parentSessionId = index.sessions[sessionId]?.parentSessionId;
+        }
+        catch {
+            // Ignore index errors
+        }
         const sessionData = {
             id: sessionId,
+            rootSessionId: rootSessionId || sessionId,
+            parentSessionId,
             messages: messages.map((msg) => {
                 // Remove timestamp property for backward compatibility
                 const { timestamp: _ignored, ...messageWithoutTimestamp } = msg;
@@ -644,3 +664,39 @@ export async function handleSessionRestoration(restoreSessionId, continueLastSes
         process.exit(1);
     }
 }
+/**
+ * Load the full message thread by following parentSessionId links
+ * @param currentSessionId - The ID of the current session
+ * @param workdir - Working directory for the session
+ * @returns Promise that resolves to an array of all messages in the thread
+ */
+export async function loadFullMessageThread(currentSessionId, workdir) {
+    const sessionIds = [];
+    let currentId = currentSessionId;
+    const allMessages = [];
+    while (currentId) {
+        const sessionData = await loadSessionFromJsonl(currentId, workdir);
+        if (!sessionData)
+            break;
+        sessionIds.unshift(currentId);
+        // Add messages from this session to the beginning of the list
+        // But skip the "compress" block if it's not the first session in our traversal (which is the latest)
+        // Actually, we should probably keep all messages and let the UI/logic handle it.
+        // But wait, if we are concatenating, the "compress" block in session N summarizes session N-1.
+        // So if we have session N-1 and session N, we should probably skip the compress block in session N.
+        const messages = sessionData.messages;
+        if (allMessages.length > 0) {
+            // If we already have messages (from "later" sessions),
+            // we are now adding messages from an "earlier" session.
+            // The later session's first message might be a "compress" block.
+            if (allMessages[0].blocks.some((b) => b.type === "compress")) {
+                // Remove the compress block from the later session's messages
+                // because we are now providing the actual messages it summarized.
+                allMessages.shift();
+            }
+        }
+        allMessages.unshift(...messages);
+        currentId = sessionData.parentSessionId;
+    }
+    return { messages: allMessages, sessionIds };
+}

package/dist/services/taskManager.d.ts

@@ -0,0 +1,25 @@
+import { EventEmitter } from "events";
+import { Task } from "../types/tasks.js";
+export declare class TaskManager extends EventEmitter {
+    private readonly baseDir;
+    private taskListId;
+    constructor(taskListId: string);
+    getTaskListId(): string;
+    setTaskListId(taskListId: string): void;
+    private getSessionDir;
+    getTaskPath(taskId: string): string;
+    private getLockPath;
+    ensureSessionDir(): Promise<void>;
+    private withLock;
+    private validateTask;
+    createTask(task: Omit<Task, "id">): Promise<string>;
+    getTask(taskId: string): Promise<Task | null>;
+    updateTask(task: Task): Promise<void>;
+    listTasks(): Promise<Task[]>;
+    getNextTaskId(): Promise<string>;
+    /**
+     * Refreshes the task list by re-reading tasks from disk and emitting a change event.
+     */
+    refreshTasks(): Promise<void>;
+}
+//# sourceMappingURL=taskManager.d.ts.map

package/dist/services/taskManager.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"taskManager.d.ts","sourceRoot":"","sources":["../../src/services/taskManager.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,YAAY,EAAE,MAAM,QAAQ,CAAC;AACtC,OAAO,EAAE,IAAI,EAAE,MAAM,mBAAmB,CAAC;AAGzC,qBAAa,WAAY,SAAQ,YAAY;IAC3C,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAS;IACjC,OAAO,CAAC,UAAU,CAAS;gBAEf,UAAU,EAAE,MAAM;IAMvB,aAAa,IAAI,MAAM;IAIvB,aAAa,CAAC,UAAU,EAAE,MAAM,GAAG,IAAI;IAI9C,OAAO,CAAC,aAAa;IAId,WAAW,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM;IAI1C,OAAO,CAAC,WAAW;IAIb,gBAAgB,IAAI,OAAO,CAAC,IAAI,CAAC;YAIzB,QAAQ;IA2CtB,OAAO,CAAC,YAAY;IAed,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,IAAI,EAAE,IAAI,CAAC,GAAG,OAAO,CAAC,MAAM,CAAC;IAcnD,OAAO,CAAC,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,GAAG,IAAI,CAAC;IAmB7C,UAAU,CAAC,IAAI,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC;IAWrC,SAAS,IAAI,OAAO,CAAC,IAAI,EAAE,CAAC;IA+B5B,aAAa,IAAI,OAAO,CAAC,MAAM,CAAC;IActC;;OAEG;IACU,YAAY,IAAI,OAAO,CAAC,IAAI,CAAC;CAG3C"}

package/dist/services/taskManager.js

@@ -0,0 +1,164 @@
+import { promises as fs } from "fs";
+import { join } from "path";
+import { homedir } from "os";
+import { EventEmitter } from "events";
+import { logger } from "../utils/globalLogger.js";
+export class TaskManager extends EventEmitter {
+    constructor(taskListId) {
+        super();
+        this.taskListId = taskListId;
+        this.baseDir = join(homedir(), ".wave", "tasks");
+    }
+    getTaskListId() {
+        return this.taskListId;
+    }
+    setTaskListId(taskListId) {
+        this.taskListId = taskListId;
+    }
+    getSessionDir() {
+        return join(this.baseDir, this.taskListId);
+    }
+    getTaskPath(taskId) {
+        return join(this.getSessionDir(), `${taskId}.json`);
+    }
+    getLockPath() {
+        return join(this.getSessionDir(), `.lock`);
+    }
+    async ensureSessionDir() {
+        await fs.mkdir(this.getSessionDir(), { recursive: true });
+    }
+    async withLock(operation) {
+        const lockPath = this.getLockPath();
+        let lockHandle;
+        const maxRetries = 100;
+        const retryDelay = process.env.NODE_ENV === "test" ? 10 : 100;
+        await this.ensureSessionDir();
+        for (let i = 0; i < maxRetries; i++) {
+            try {
+                lockHandle = await fs.open(lockPath, "wx");
+                break;
+            }
+            catch (error) {
+                if (error.code === "EEXIST") {
+                    if (i === maxRetries - 1) {
+                        throw new Error(`Could not acquire lock for task list ${this.taskListId} after ${maxRetries} retries`);
+                    }
+                    await new Promise((resolve) => setTimeout(resolve, retryDelay));
+                    continue;
+                }
+                throw error;
+            }
+        }
+        try {
+            return await operation();
+        }
+        finally {
+            if (lockHandle) {
+                await lockHandle.close();
+            }
+            try {
+                await fs.unlink(lockPath);
+            }
+            catch (error) {
+                logger.error(`Failed to release lock for task list ${this.taskListId}:`, error);
+            }
+        }
+    }
+    validateTask(task) {
+        if (!task.id || typeof task.id !== "string")
+            throw new Error("Invalid task ID");
+        if (!task.subject || typeof task.subject !== "string")
+            throw new Error("Invalid task subject");
+        if (!task.description || typeof task.description !== "string")
+            throw new Error("Invalid task description");
+        if (!task.status ||
+            !["pending", "in_progress", "completed", "deleted"].includes(task.status)) {
+            throw new Error(`Invalid task status: ${task.status}`);
+        }
+    }
+    async createTask(task) {
+        return await this.withLock(async () => {
+            const taskId = await this.getNextTaskId();
+            const fullTask = { ...task, id: taskId };
+            this.validateTask(fullTask);
+            const taskPath = this.getTaskPath(taskId);
+            const content = JSON.stringify(fullTask, null, 2);
+            await fs.writeFile(taskPath, content, "utf8");
+            this.emit("tasksChange", this.taskListId);
+            logger.info(`Task ${taskId} created in task list ${this.taskListId}`);
+            return taskId;
+        });
+    }
+    async getTask(taskId) {
+        const taskPath = this.getTaskPath(taskId);
+        try {
+            const content = await fs.readFile(taskPath, "utf8");
+            try {
+                return JSON.parse(content.trim());
+            }
+            catch (parseError) {
+                logger.error(`Failed to parse task file ${taskPath}:`, parseError);
+                logger.error(`Corrupted content: ${content}`);
+                throw parseError;
+            }
+        }
+        catch (error) {
+            if (error.code === "ENOENT") {
+                return null;
+            }
+            throw error;
+        }
+    }
+    async updateTask(task) {
+        this.validateTask(task);
+        await this.withLock(async () => {
+            const taskPath = this.getTaskPath(task.id);
+            const content = JSON.stringify(task, null, 2);
+            await fs.writeFile(taskPath, content, "utf8");
+            this.emit("tasksChange", this.taskListId);
+            logger.info(`Task ${task.id} updated in task list ${this.taskListId}`);
+        });
+    }
+    async listTasks() {
+        const sessionDir = this.getSessionDir();
+        try {
+            const files = await fs.readdir(sessionDir);
+            const taskFiles = files.filter((f) => f.endsWith(".json"));
+            const tasks = await Promise.all(taskFiles.map(async (file) => {
+                const taskPath = join(sessionDir, file);
+                try {
+                    const content = await fs.readFile(taskPath, "utf8");
+                    return JSON.parse(content.trim());
+                }
+                catch (error) {
+                    logger.error(`Failed to read or parse task file ${taskPath}:`, error);
+                    return null;
+                }
+            }));
+            return tasks.filter((t) => t !== null);
+        }
+        catch (error) {
+            if (error.code === "ENOENT") {
+                return [];
+            }
+            throw error;
+        }
+    }
+    async getNextTaskId() {
+        const tasks = await this.listTasks();
+        if (tasks.length === 0) {
+            return "1";
+        }
+        const ids = tasks.map((t) => parseInt(t.id, 10)).filter((id) => !isNaN(id));
+        if (ids.length === 0) {
+            return (tasks.length + 1).toString();
+        }
+        return (Math.max(...ids) + 1).toString();
+    }
+    /**
+     * Refreshes the task list by re-reading tasks from disk and emitting a change event.
+     */
+    async refreshTasks() {
+        this.emit("tasksChange", this.taskListId);
+    }
+}

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

@@ -1 +1 @@
-{"version":3,"file":"askUserQuestion.d.ts","sourceRoot":"","sources":["../../src/tools/askUserQuestion.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;AAOxC,eAAO,MAAM,mBAAmB,EAAE,UAwHjC,CAAC"}
+{"version":3,"file":"askUserQuestion.d.ts","sourceRoot":"","sources":["../../src/tools/askUserQuestion.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;AAIxC,eAAO,MAAM,mBAAmB,EAAE,UAiJjC,CAAC"}

package/dist/tools/askUserQuestion.js

@@ -1,23 +1,11 @@
-import { ASK_USER_QUESTION_TOOL_NAME, EXIT_PLAN_MODE_TOOL_NAME, } from "../constants/tools.js";
+import { ASK_USER_QUESTION_TOOL_NAME } from "../constants/tools.js";
 export const askUserQuestionTool = {
     name: ASK_USER_QUESTION_TOOL_NAME,
     config: {
         type: "function",
         function: {
             name: ASK_USER_QUESTION_TOOL_NAME,
-            description: `Asks the user multiple choice questions to gather information, clarify ambiguity, understand preferences, make decisions or offer them choices.
-Use this tool when you need to ask the user questions during execution. This allows you to:
-1. Gather user preferences or requirements
-2. Clarify ambiguous instructions
-3. Get decisions on implementation choices as you work
-4. Offer choices to the user about what direction to take.
-
-Usage notes:
-- Users will always be able to select "Other" to provide custom text input
-- Use multiSelect: true to allow multiple answers to be selected for a question
-- If you recommend a specific option, make that the first option in the list and add "(Recommended)" at the end of the label
-
-Plan mode note: In plan mode, use this tool to clarify requirements or choose between approaches BEFORE finalizing your plan. Do NOT use this tool to ask "Is my plan ready?" or "Should I proceed?" - use ${EXIT_PLAN_MODE_TOOL_NAME} for plan approval.`,
+            description: "Asks the user multiple choice questions to gather information, clarify ambiguity, understand preferences, make decisions or offer them choices.",
             parameters: {
                 type: "object",
                 properties: {
@@ -25,36 +13,34 @@ Plan mode note: In plan mode, use this tool to clarify requirements or choose be
                         type: "array",
                         minItems: 1,
                         maxItems: 4,
+                        description: "Questions to ask the user (1-4 questions)",
                         items: {
                             type: "object",
                             properties: {
                                 question: {
                                     type: "string",
-                                    description: "The complete question to ask the user.",
+                                    description: 'The complete question to ask the user. Should be clear, specific, and end with a question mark. Example: "Which library should we use for date formatting?" If multiSelect is true, phrase it accordingly, e.g. "Which features do you want to enable?"',
                                 },
                                 header: {
                                     type: "string",
                                     maxLength: 12,
-                                    description: "Very short label displayed as a chip/tag (max 12 chars).",
+                                    description: `Very short label displayed as a chip/tag (max 12 chars). Examples: "Auth method", "Library", "Approach".`,
                                 },
                                 options: {
                                     type: "array",
                                     minItems: 2,
                                     maxItems: 4,
+                                    description: "The available choices for this question. Must have 2-4 options. Each option should be a distinct, mutually exclusive choice (unless multiSelect is enabled). There should be no 'Other' option, that will be provided automatically.",
                                     items: {
                                         type: "object",
                                         properties: {
                                             label: {
                                                 type: "string",
-                                                description: "The display text for this option.",
+                                                description: "The display text for this option that the user will see and select. Should be concise (1-5 words) and clearly describe the choice.",
                                             },
                                             description: {
                                                 type: "string",
-                                                description: "Explanation of what this option means.",
-                                            },
-                                            isRecommended: {
-                                                type: "boolean",
-                                                description: "Whether this option is recommended.",
+                                                description: "Explanation of what this option means or what will happen if chosen. Useful for providing context about trade-offs or implications.",
                                             },
                                         },
                                         required: ["label"],
@@ -63,23 +49,51 @@ Plan mode note: In plan mode, use this tool to clarify requirements or choose be
                                 multiSelect: {
                                     type: "boolean",
                                     default: false,
-                                    description: "Allow multiple answers to be selected.",
+                                    description: "Set to true to allow the user to select multiple options instead of just one. Use when choices are not mutually exclusive.",
                                 },
                             },
                             required: ["question", "header", "options"],
                         },
                     },
+                    answers: {
+                        type: "object",
+                        additionalProperties: { type: "string" },
+                        description: "User answers collected by the permission component",
+                    },
+                    metadata: {
+                        type: "object",
+                        properties: {
+                            source: {
+                                type: "string",
+                                description: 'Optional identifier for the source of this question (e.g., "remember" for /remember command). Used for analytics tracking.',
+                            },
+                        },
+                        description: "Optional metadata for the question",
+                    },
                 },
                 required: ["questions"],
             },
         },
     },
+    prompt: () => `Use this tool when you need to ask the user questions during execution. This allows you to:
+1. Gather user preferences or requirements
+2. Clarify ambiguous instructions
+3. Get decisions on implementation choices as you work
+4. Offer choices to the user about what direction to take.
+
+Usage notes:
+- Users will always be able to select "Other" to provide custom text input
+- Use multiSelect: true to allow multiple answers to be selected for a question
+- If you recommend a specific option, make that the first option in the list and add "(Recommended)" at the end of the label
+
+Plan mode note: In plan mode, use this tool to clarify requirements or choose between approaches BEFORE finalizing your plan. Do NOT use this tool to ask "Is my plan ready?" or "Should I proceed?" - use ExitPlanMode for plan approval.
+`,
     execute: async (args, context) => {
-        const { questions } = args;
+        const { questions, answers: existingAnswers, metadata, } = args;
         if (!context.permissionManager) {
             throw new Error(`Permission manager is required for ${ASK_USER_QUESTION_TOOL_NAME} tool`);
         }
-        const permissionContext = context.permissionManager.createContext(ASK_USER_QUESTION_TOOL_NAME, context.permissionMode || "default", context.canUseToolCallback, { questions });
+        const permissionContext = context.permissionManager.createContext(ASK_USER_QUESTION_TOOL_NAME, context.permissionMode || "default", context.canUseToolCallback, { questions, answers: existingAnswers, metadata });
         permissionContext.hidePersistentOption = true; // Always hide persistent option for questions
         const decision = await context.permissionManager.checkPermission(permissionContext);
         if (decision.behavior === "deny") {

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

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

package/dist/tools/bashTool.js

@@ -83,6 +83,9 @@ Usage notes:
             },
         },
     },
+    prompt: () => `
+- Reserve bash tools exclusively for actual system commands and terminal operations that require shell execution. NEVER use bash echo or other command-line tools to communicate thoughts, explanations, or instructions to the user. Output all communication directly in your response text instead.
+- When making multiple bash tool calls, you MUST send a single message with multiple tools calls to run the calls in parallel. For example, if you need to run "git status" and "git diff", send a single message with two tool calls in parallel.`,
     execute: async (args, context) => {
         const command = args.command;
         const runInBackground = args.run_in_background;
@@ -179,8 +182,9 @@ Usage notes:
                             const taskId = backgroundTaskManager.adoptProcess(child, command, outputBuffer, errorBuffer);
                             resolve({
                                 success: true,
-                                content: `Command moved to background with ID: ${taskId}. Use TaskOutput tool with task_id="${taskId}" to monitor output.`,
+                                content: `Command moved to background with ID: ${taskId}.`,
                                 shortResult: `Process ${taskId} backgrounded`,
+                                isManuallyBackgrounded: true,
                             });
                         }
                         else {
@@ -247,28 +251,26 @@ Usage notes:
             // Handle abort signal from context
             if (context?.abortSignal) {
                 if (context.abortSignal.aborted) {
-                    if (!isBackgrounded) {
-                        handleAbort();
-                    }
+                    handleAbort();
                     return;
                 }
                 // Use { once: true } to prevent listener accumulation on signal reuse
-                context.abortSignal.addEventListener("abort", () => {
-                    if (!isBackgrounded) {
-                        handleAbort();
-                    }
-                }, {
+                context.abortSignal.addEventListener("abort", () => handleAbort(), {
                     once: true,
                 });
             }
             child.stdout?.on("data", (data) => {
                 if (!isAborted && !isBackgrounded) {
-                    outputBuffer += stripAnsiColors(data.toString());
+                    const chunk = stripAnsiColors(data.toString());
+                    outputBuffer += chunk;
+                    context.onShortResultUpdate?.(chunk.trim().split("\n").pop() || "");
                 }
             });
             child.stderr?.on("data", (data) => {
                 if (!isAborted && !isBackgrounded) {
-                    errorBuffer += stripAnsiColors(data.toString());
+                    const chunk = stripAnsiColors(data.toString());
+                    errorBuffer += chunk;
+                    context.onShortResultUpdate?.(chunk.trim().split("\n").pop() || "");
                 }
             });
             child.on("exit", (code) => {

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;AAgBtE;;GAEG;AACH,eAAO,MAAM,QAAQ,EAAE,UAiNtB,CAAC"}
+{"version":3,"file":"editTool.d.ts","sourceRoot":"","sources":["../../src/tools/editTool.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,UAAU,EAA2B,MAAM,YAAY,CAAC;AAgBtE;;GAEG;AACH,eAAO,MAAM,QAAQ,EAAE,UAmNtB,CAAC"}

package/dist/tools/editTool.js

@@ -16,11 +16,12 @@ function formatCompactParams(args, context) {
 export const editTool = {
     name: EDIT_TOOL_NAME,
     formatCompactParams,
+    prompt: () => `Performs exact string replacements in files. \n\nUsage:\n- You must use your \`${READ_TOOL_NAME}\` tool at least once in the conversation before editing. This tool will error if you attempt an edit without reading the file. \n- When editing text from read_file tool output, ensure you preserve the exact indentation (tabs/spaces) as it appears AFTER the line number prefix. The line number prefix format is: spaces + line number + tab. Everything after that tab is the actual file content to match. Never include any part of the line number prefix in the old_string or new_string.\n- ALWAYS prefer editing existing files in the codebase. NEVER write new files unless explicitly required.\n- Only use emojis if the user explicitly requests it. Avoid adding emojis to files unless asked.\n- The edit will FAIL if \`old_string\` is not unique in the file. Either provide a larger string with more surrounding context to make it unique or use \`replace_all\` to change every instance of \`old_string\`. \n- Use \`replace_all\` for replacing and renaming strings across the file. This parameter is useful if you want to rename a variable for instance.`,
     config: {
         type: "function",
         function: {
             name: EDIT_TOOL_NAME,
-            description: `Performs exact string replacements in files. \n\nUsage:\n- You must use your \`${READ_TOOL_NAME}\` tool at least once in the conversation before editing. This tool will error if you attempt an edit without reading the file. \n- When editing text from read_file tool output, ensure you preserve the exact indentation (tabs/spaces) as it appears AFTER the line number prefix. The line number prefix format is: spaces + line number + tab. Everything after that tab is the actual file content to match. Never include any part of the line number prefix in the old_string or new_string.\n- ALWAYS prefer editing existing files in the codebase. NEVER write new files unless explicitly required.\n- Only use emojis if the user explicitly requests it. Avoid adding emojis to files unless asked.\n- The edit will FAIL if \`old_string\` is not unique in the file. Either provide a larger string with more surrounding context to make it unique or use \`replace_all\` to change every instance of \`old_string\`. \n- Use \`replace_all\` for replacing and renaming strings across the file. This parameter is useful if you want to rename a variable for instance.`,
+            description: "A tool for editing files",
             parameters: {
                 type: "object",
                 properties: {

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

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

package/dist/tools/exitPlanMode.js

@@ -10,7 +10,7 @@ export const exitPlanModeTool = {
         type: "function",
         function: {
             name: EXIT_PLAN_MODE_TOOL_NAME,
-            description: "Use this tool when you are in plan mode and have finished writing your plan to the plan file and are ready for user approval. This tool will read the plan from the file specified in the system message and present it to the user for confirmation. You should have already written your plan to that file before calling this tool.",
+            description: "Prompts the user to exit plan mode and start coding",
             parameters: {
                 type: "object",
                 properties: {},
@@ -19,6 +19,30 @@ export const exitPlanModeTool = {
             },
         },
     },
+    prompt: () => `Use this tool when you are in plan mode and have finished writing your plan to the plan file and are ready for user approval.
+
+## How This Tool Works
+- You should have already written your plan to the plan file specified in the plan mode system message
+- This tool does NOT take the plan content as a parameter - it will read the plan from the file you wrote
+- This tool simply signals that you're done planning and ready for the user to review and approve
+- The user will see the contents of your plan file when they review it
+
+## When to Use This Tool
+IMPORTANT: Only use this tool when the task requires planning the implementation steps of a task that requires writing code. For research tasks where you're gathering information, searching files, reading files or in general trying to understand the codebase - do NOT use this tool.
+
+## Before Using This Tool
+Ensure your plan is complete and unambiguous:
+- If you have unresolved questions about requirements or approach, use AskUserQuestion first (in earlier phases)
+- Once your plan is finalized, use THIS tool to request approval
+
+**Important:** Do NOT use AskUserQuestion to ask "Is this plan okay?" or "Should I proceed?" - that's exactly what THIS tool does. ExitPlanMode inherently requests user approval of your plan.
+
+## Examples
+
+1. Initial task: "Search for and understand the implementation of vim mode in the codebase" - Do not use the exit plan mode tool because you are not planning the implementation steps of a task.
+2. Initial task: "Help me implement yank mode for vim" - Use the exit plan mode tool after you have finished planning the implementation steps of the task.
+3. Initial task: "Add a new feature to handle user authentication" - If unsure about auth method (OAuth, JWT, etc.), use AskUserQuestion first, then use exit plan mode tool after clarifying the approach.
+`,
     execute: async (_args, context) => {
         try {
             if (!context.permissionManager) {
@@ -53,7 +77,7 @@ export const exitPlanModeTool = {
             if (permissionResult.behavior === "deny") {
                 return {
                     success: false,
-                    content: permissionResult.message || "Plan rejected by user",
+                    content: `User feedback: ${permissionResult.message || "Plan rejected by user"}. Please update your proposal accordingly.`,
                     error: permissionResult.message ? undefined : "Plan rejected by user",
                 };
             }

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

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

package/dist/tools/globTool.js

@@ -2,7 +2,7 @@ import { glob } from "glob";
 import { stat } from "fs/promises";
 import { resolvePath, getDisplayPath } from "../utils/path.js";
 import { getGlobIgnorePatterns } from "../utils/fileFilter.js";
-import { GLOB_TOOL_NAME, TASK_TOOL_NAME } from "../constants/tools.js";
+import { GLOB_TOOL_NAME } from "../constants/tools.js";
 /**
  * Glob Tool Plugin - Fast file pattern matching
  */
@@ -12,7 +12,7 @@ export const globTool = {
         type: "function",
         function: {
             name: GLOB_TOOL_NAME,
-            description: `- Fast file pattern matching tool that works with any codebase size\n- Supports glob patterns like "**/*.js" or "src/**/*.ts"\n- Returns matching file paths sorted by modification time\n- Use this tool when you need to find files by name patterns\n- When you are doing an open ended search that may require multiple rounds of globbing and grepping, use the ${TASK_TOOL_NAME} tool instead\n- You have the capability to call multiple tools in a single response. It is always better to speculatively perform multiple searches as a batch that are potentially useful.`,
+            description: "Fast file pattern matching tool that works with any codebase size",
             parameters: {
                 type: "object",
                 properties: {
@@ -29,6 +29,12 @@ export const globTool = {
             },
         },
     },
+    prompt: () => `- Fast file pattern matching tool that works with any codebase size
+- Supports glob patterns like "**/*.js" or "src/**/*.ts"
+- Returns matching file paths sorted by modification time
+- Use this tool when you need to find files by name patterns
+- When you are doing an open ended search that may require multiple rounds of globbing and grepping, use the Agent tool instead
+- You can call multiple tools in a single response. It is always better to speculatively perform multiple searches in parallel if they are potentially useful.`,
     execute: async (args, context) => {
         const pattern = args.pattern;
         const searchPath = args.path;

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

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