feat: remove respawn capability, it wasn't being used anyhow.

bhouston · bhouston · commit 8e086b46bd08 · 2025-03-18T15:40:30.000-04:00
diff --git a/docs/github-comment-commands.md b/docs/github-comment-commands.md
diff --git a/docs/release-process.md b/docs/release-process.md
diff --git a/docs/tools/agent-tools.md b/docs/tools/agent-tools.md
diff --git a/packages/agent/src/core/toolAgent/toolAgentCore.ts b/packages/agent/src/core/toolAgent/toolAgentCore.ts
@@ -157,24 +157,13 @@ export const toolAgent = async (
       );
 
       // Execute the tools and get results
-      const { agentDoned, completionResult, respawn } = await executeTools(
+      const { agentDoned, completionResult } = await executeTools(
         toolCalls,
         tools,
         messages,
         localContext,
       );
 
-      if (respawn) {
-        logger.info('Respawning agent with new context');
-        // Reset messages to just the new context
-        messages.length = 0;
-        messages.push({
-          role: 'user',
-          content: respawn.context,
-        });
-        continue;
-      }
-
       if (agentDoned) {
         const result: ToolAgentResult = {
           result: completionResult ?? 'Sequence explicitly completed',
diff --git a/packages/agent/src/core/toolAgent/toolExecutor.ts b/packages/agent/src/core/toolAgent/toolExecutor.ts
@@ -39,27 +39,6 @@ export async function executeTools(
 
   logger.verbose(`Executing ${toolCalls.length} tool calls`);
 
-  // Check for respawn tool call
-  const respawnCall = toolCalls.find((call) => call.name === 'respawn');
-  if (respawnCall) {
-    // Add the tool result to messages
-    addToolResultToMessages(messages, respawnCall.id, { success: true }, false);
-
-    return {
-      agentDoned: false,
-      toolResults: [
-        {
-          toolCallId: respawnCall.id,
-          toolName: respawnCall.name,
-          result: { success: true },
-        },
-      ],
-      respawn: {
-        context: JSON.parse(respawnCall.content).respawnContext,
-      },
-    };
-  }
-
   const toolResults = await Promise.all(
     toolCalls.map(async (call) => {
       let toolResult = '';
diff --git a/packages/agent/src/core/toolAgent/types.ts b/packages/agent/src/core/toolAgent/types.ts
@@ -10,7 +10,6 @@ export interface ToolCallResult {
   agentDoned: boolean;
   completionResult?: string;
   toolResults: unknown[];
-  respawn?: { context: string };
 }
 
 export type ErrorResult = {
diff --git a/packages/agent/src/index.ts b/packages/agent/src/index.ts
@@ -4,7 +4,7 @@ export * from './tools/fetch/fetch.js';
 
 // Tools - System
 export * from './tools/shell/shellStart.js';
-export * from './tools/sleep/sleep.js';
+export * from './tools/sleep/wait.js';
 export * from './tools/agent/agentDone.js';
 export * from './tools/shell/shellMessage.js';
 export * from './tools/shell/shellExecute.js';
diff --git a/packages/agent/src/tools/getTools.ts b/packages/agent/src/tools/getTools.ts
@@ -15,7 +15,7 @@ import { sessionStartTool } from './session/sessionStart.js';
 import { listShellsTool } from './shell/listShells.js';
 import { shellMessageTool } from './shell/shellMessage.js';
 import { shellStartTool } from './shell/shellStart.js';
-import { sleepTool } from './sleep/sleep.js';
+import { waitTool } from './sleep/wait.js';
 import { textEditorTool } from './textEditor/textEditor.js';
 
 // Import these separately to avoid circular dependencies
@@ -49,7 +49,7 @@ export function getTools(options?: GetToolsOptions): Tool[] {
     sessionMessageTool as unknown as Tool,
     listSessionsTool as unknown as Tool,
 
-    sleepTool as unknown as Tool,
+    waitTool as unknown as Tool,
   ];
 
   // Only include userPrompt tool if enabled
diff --git a/packages/agent/src/tools/sleep/wait.test.ts b/packages/agent/src/tools/sleep/wait.test.ts
@@ -3,7 +3,7 @@ import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { ToolContext } from '../../core/types';
 import { getMockToolContext } from '../getTools.test';
 
-import { sleepTool } from './sleep';
+import { waitTool } from './wait';
 
 const toolContext: ToolContext = getMockToolContext();
 
@@ -13,7 +13,7 @@ describe('sleep tool', () => {
   });
 
   it('should sleep for the specified duration', async () => {
-    const sleepPromise = sleepTool.execute({ seconds: 2 }, toolContext);
+    const sleepPromise = waitTool.execute({ seconds: 2 }, toolContext);
 
     await vi.advanceTimersByTimeAsync(2000);
     const result = await sleepPromise;
@@ -23,13 +23,13 @@ describe('sleep tool', () => {
 
   it('should reject negative sleep duration', async () => {
     await expect(
-      sleepTool.execute({ seconds: -1 }, toolContext),
+      waitTool.execute({ seconds: -1 }, toolContext),
     ).rejects.toThrow();
   });
 
   it('should reject sleep duration over 1 hour', async () => {
     await expect(
-      sleepTool.execute({ seconds: 3601 }, toolContext),
+      waitTool.execute({ seconds: 3601 }, toolContext),
     ).rejects.toThrow();
   });
 });
diff --git a/packages/agent/src/tools/sleep/wait.ts b/packages/agent/src/tools/sleep/wait.ts
@@ -18,8 +18,8 @@ const returnsSchema = z.object({
   sleptFor: z.number().describe('Actual number of seconds slept'),
 });
 
-export const sleepTool: Tool = {
-  name: 'sleep',
+export const waitTool: Tool = {
+  name: 'wait',
   description:
     'Pauses execution for the specified number of seconds, useful when waiting for async tools to make progress before checking on them',
   logPrefix: '💤',

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,6 @@ export interface ToolCallResult {`
`10`	`10`	`agentDoned: boolean;`
`11`	`11`	`completionResult?: string;`
`12`	`12`	`toolResults: unknown[];`
`13`		`- respawn?: { context: string };`
`14`	`13`	`}`
`15`	`14`
`16`	`15`	`export type ErrorResult = {`