chore: rename refactor - may not use this

Author: bhouston

docs/tools/agent-tools.md

@@ -2,15 +2,15 @@
 
 The agent tools provide ways to create and interact with sub-agents. There are two approaches available:
 
-1. The original `subAgent` tool (synchronous, blocking)
+1. The original `agentExecute` tool (synchronous, blocking)
 2. The new `agentStart` and `agentMessage` tools (asynchronous, non-blocking)
 
-## subAgent Tool
+## agentExecute Tool
 
-The `subAgent` tool creates a sub-agent that runs synchronously until completion. The parent agent waits for the sub-agent to complete before continuing.
+The `agentExecute` tool creates a sub-agent that runs synchronously until completion. The parent agent waits for the sub-agent to complete before continuing.
 
 ```typescript
-subAgent({
+agentExecute({
   description: "A brief description of the sub-agent's purpose",
   goal: 'The main objective that the sub-agent needs to achieve',
   projectContext: 'Context about the problem or environment',
@@ -123,7 +123,7 @@ while (!agent1Completed || !agent2Completed) {
 
 ## Choosing Between Approaches
 
-- Use `subAgent` for simpler tasks where blocking execution is acceptable
+- Use `agentExecute` for simpler tasks where blocking execution is acceptable
 - Use `agentStart` and `agentMessage` for:
   - Parallel execution of multiple sub-agents
   - Tasks where you need to monitor progress

packages/agent/CHANGELOG.md

@@ -1,9 +1,8 @@
 # [mycoder-agent-v1.4.2](https://github.com/drivecore/mycoder/compare/mycoder-agent-v1.4.1...mycoder-agent-v1.4.2) (2025-03-14)
 
-
 ### Bug Fixes
 
-* improve profiling ([79a3df2](https://github.com/drivecore/mycoder/commit/79a3df2db13b8372666c6604ebe1666d33663be9))
+- improve profiling ([79a3df2](https://github.com/drivecore/mycoder/commit/79a3df2db13b8372666c6604ebe1666d33663be9))
 
 # [mycoder-agent-v1.4.1](https://github.com/drivecore/mycoder/compare/mycoder-agent-v1.4.0...mycoder-agent-v1.4.1) (2025-03-14)
 
@@ -56,7 +55,7 @@
 - llm choice working well for openai, anthropic and ollama ([68d34ab](https://github.com/drivecore/mycoder/commit/68d34abf8a73ed533a072359ce334a9364753425))
 - **openai:** add OpenAI dependency to agent package and enable provider in config ([30b0807](https://github.com/drivecore/mycoder/commit/30b0807d4f3ecdd24f53b7ee4160645a4ed10444))
 - replace @semantic-release/npm with @anolilab/semantic-release-pnpm to properly resolve workspace references ([bacb51f](https://github.com/drivecore/mycoder/commit/bacb51f637f2b2d3b1039bdfdbd33e3d704b6cde))
-- up subagent iterations to 200 from 50 ([b405f1e](https://github.com/drivecore/mycoder/commit/b405f1e6d62eb5304dc1aa6c0ff28dc49dc67dce))
+- up agentExecute iterations to 200 from 50 ([b405f1e](https://github.com/drivecore/mycoder/commit/b405f1e6d62eb5304dc1aa6c0ff28dc49dc67dce))
 
 ### Features
 

packages/agent/src/core/backgroundTools.cleanup.test.ts

@@ -1,9 +1,9 @@
 import { describe, expect, it, vi, beforeEach, afterEach } from 'vitest';
 
 // Import mocked modules
+import { agentStates } from '../tools/agent/agentStart.js';
 import { BrowserManager } from '../tools/browser/BrowserManager.js';
-import { agentStates } from '../tools/interaction/agentStart.js';
-import { shellTracker } from '../tools/system/ShellTracker.js';
+import { shellTracker } from '../tools/shell/ShellTracker.js';
 
 import { BackgroundTools, BackgroundToolStatus } from './backgroundTools';
 import { Tool } from './types';
@@ -43,7 +43,7 @@ vi.mock('../tools/browser/BrowserManager.js', () => {
   };
 });
 
-vi.mock('../tools/system/ShellTracker.js', () => {
+vi.mock('../tools/shell/ShellTracker.js', () => {
   return {
     shellTracker: {
       processStates: new Map<string, MockProcessState>(),
@@ -52,7 +52,7 @@ vi.mock('../tools/system/ShellTracker.js', () => {
   };
 });
 
-vi.mock('../tools/interaction/agentStart.js', () => {
+vi.mock('../tools/agent/agentStart.js', () => {
   return {
     agentStates: new Map<string, MockAgentState>(),
   };

packages/agent/src/core/backgroundTools.ts

@@ -1,9 +1,9 @@
 import { v4 as uuidv4 } from 'uuid';
 
 // These imports will be used by the cleanup method
+import { agentStates } from '../tools/agent/agentStart.js';
 import { BrowserManager } from '../tools/browser/BrowserManager.js';
-import { agentStates } from '../tools/interaction/agentStart.js';
-import { shellTracker } from '../tools/system/ShellTracker.js';
+import { shellTracker } from '../tools/shell/ShellTracker.js';
 
 // Types of background processes we can track
 export enum BackgroundToolType {
@@ -157,7 +157,7 @@ export class BackgroundTools {
       this.cleanupBrowserSession(tool),
     );
     const agentCleanupPromises = agentTools.map((tool) =>
-      this.cleanupSubAgent(tool),
+      this.cleanupagentExecute(tool),
     );
 
     // Clean up shell processes using ShellTracker
@@ -193,7 +193,7 @@ export class BackgroundTools {
    * Cleans up a sub-agent
    * @param tool The agent tool to clean up
    */
-  private async cleanupSubAgent(tool: AgentBackgroundTool): Promise<void> {
+  private async cleanupagentExecute(tool: AgentBackgroundTool): Promise<void> {
     try {
       const agentState = agentStates.get(tool.id);
       if (agentState && !agentState.aborted) {

packages/agent/src/core/toolAgent/config.ts

@@ -146,7 +146,7 @@ export function getDefaultSystemPrompt(toolContext: ToolContext): string {
     githubModeInstructions,
     '',
     'You prefer to call tools in parallel when possible because it leads to faster execution and less resource usage.',
-    'When done, call the sequenceComplete tool with your results to indicate that the sequence has completed.',
+    'When done, call the agentComplete tool with your results to indicate that the sequence has completed.',
     '',
     'For coding tasks:',
     '0. Try to break large tasks into smaller sub-tasks that can be completed and verified sequentially.',

packages/agent/src/core/toolAgent/toolAgentCore.test.ts

@@ -13,7 +13,7 @@ describe('toolAgentCore empty response detection', () => {
         content: [
           {
             type: 'text',
-            text: 'I notice you sent an empty response. If you are done with your tasks, please call the sequenceComplete tool with your results. If you are waiting for other tools to complete, you can use the sleep tool to wait before checking again.',
+            text: 'I notice you sent an empty response. If you are done with your tasks, please call the agentComplete tool with your results. If you are waiting for other tools to complete, you can use the sleep tool to wait before checking again.',
           },
         ],
       });

packages/agent/src/core/toolAgent/toolAgentCore.ts

@@ -100,7 +100,7 @@ export const toolAgent = async (
       messages.push({
         role: 'user',
         content:
-          'I notice you sent an empty response. If you are done with your tasks, please call the sequenceComplete tool with your results. If you are waiting for other tools to complete, you can use the sleep tool to wait before checking again.',
+          'I notice you sent an empty response. If you are done with your tasks, please call the agentComplete tool with your results. If you are waiting for other tools to complete, you can use the sleep tool to wait before checking again.',
       });
       continue;
     }
@@ -129,8 +129,12 @@ export const toolAgent = async (
       );
 
       // Execute the tools and get results
-      const { sequenceCompleted, completionResult, respawn } =
-        await executeTools(toolCalls, tools, messages, localContext);
+      const { agentCompleted, completionResult, respawn } = await executeTools(
+        toolCalls,
+        tools,
+        messages,
+        localContext,
+      );
 
       if (respawn) {
         logger.info('Respawning agent with new context');
@@ -143,7 +147,7 @@ export const toolAgent = async (
         continue;
       }
 
-      if (sequenceCompleted) {
+      if (agentCompleted) {
         const result: ToolAgentResult = {
           result: completionResult ?? 'Sequence explicitly completed',
           interactions,

packages/agent/src/core/toolAgent/toolExecutor.ts

@@ -32,7 +32,7 @@ export async function executeTools(
   context: ToolContext,
 ): Promise<ToolCallResult> {
   if (toolCalls.length === 0) {
-    return { sequenceCompleted: false, toolResults: [] };
+    return { agentCompleted: false, toolResults: [] };
   }
 
   const { logger } = context;
@@ -46,7 +46,7 @@ export async function executeTools(
     addToolResultToMessages(messages, respawnCall.id, { success: true }, false);
 
     return {
-      sequenceCompleted: false,
+      agentCompleted: false,
       toolResults: [
         {
           toolCallId: respawnCall.id,
@@ -97,19 +97,19 @@ export async function executeTools(
     }),
   );
 
-  const sequenceCompletedTool = toolResults.find(
-    (r) => r.toolName === 'sequenceComplete',
+  const agentCompletedTool = toolResults.find(
+    (r) => r.toolName === 'agentComplete',
   );
-  const completionResult = sequenceCompletedTool
-    ? (sequenceCompletedTool.result as { result: string }).result
+  const completionResult = agentCompletedTool
+    ? (agentCompletedTool.result as { result: string }).result
     : undefined;
 
-  if (sequenceCompletedTool) {
+  if (agentCompletedTool) {
     logger.verbose('Sequence completed', { completionResult });
   }
 
   return {
-    sequenceCompleted: sequenceCompletedTool !== undefined,
+    agentCompleted: agentCompletedTool !== undefined,
     completionResult,
     toolResults,
   };

packages/agent/src/core/toolAgent/types.ts

@@ -7,7 +7,7 @@ export interface ToolAgentResult {
 }
 
 export interface ToolCallResult {
-  sequenceCompleted: boolean;
+  agentCompleted: boolean;
   completionResult?: string;
   toolResults: unknown[];
   respawn?: { context: string };

packages/agent/src/index.ts

@@ -3,26 +3,27 @@
 export * from './tools/io/fetch.js';
 
 // Tools - System
-export * from './tools/system/shellStart.js';
+export * from './tools/shell/shellStart.js';
 export * from './tools/system/sleep.js';
-export * from './tools/system/respawn.js';
-export * from './tools/system/sequenceComplete.js';
-export * from './tools/system/shellMessage.js';
-export * from './tools/system/shellExecute.js';
+export * from './tools/agent/agentComplete.js';
+export * from './tools/shell/shellMessage.js';
+export * from './tools/shell/shellExecute.js';
 export * from './tools/system/listBackgroundTools.js';
-export * from './tools/system/listShells.js';
-export * from './tools/system/ShellTracker.js';
+export * from './tools/shell/listShells.js';
+export * from './tools/shell/ShellTracker.js';
 
 // Tools - Browser
 export * from './tools/browser/BrowserManager.js';
 export * from './tools/browser/types.js';
-export * from './tools/browser/browseMessage.js';
-export * from './tools/browser/browseStart.js';
+export * from './tools/browser/sessionMessage.js';
+export * from './tools/browser/sessionStart.js';
 export * from './tools/browser/PageController.js';
 export * from './tools/browser/BrowserAutomation.js';
 
 // Tools - Interaction
-export * from './tools/interaction/subAgent.js';
+export * from './tools/agent/agentExecute.js';
+export * from './tools/agent/agentStart.js';
+export * from './tools/agent/agentMessage.js';
 export * from './tools/interaction/userPrompt.js';
 
 // Core