fixing failing tests

Author: igorcosta

src/modes/planMode/PlanModeManager.ts

@@ -44,6 +44,7 @@ const READ_ONLY_TOOLS = [
   'tools_registry',
   'tool_search',
   'plan',
+  'exit_plan_mode',
   'ask_followup_question',
 ];
 

tests/modes/planMode/PlanModeManager.spec.ts

@@ -315,13 +315,14 @@ describe('PlanModeManager', () => {
       expect(tools).not.toContain('run_command');
     });
 
-    it('should include plan and ask_followup_question in read-only tools', async () => {
+    it('should include plan approval tools in read-only tools', async () => {
       const { PlanModeManager } = await import('../../../src/modes/planMode/PlanModeManager.js');
       const manager = new PlanModeManager();
 
       const tools = manager.getReadOnlyTools();
 
       expect(tools).toContain('plan');
+      expect(tools).toContain('exit_plan_mode');
       expect(tools).toContain('ask_followup_question');
     });
   });

tests/planMode.integration.spec.ts

@@ -369,6 +369,7 @@ describe('PlanModeManager tool filtering', () => {
     // Should include plan-related tools (plan is allowed in read-only list
     // when plan mode is enabled; it's gated at the ToolManager level)
     expect(tools).toContain('plan');
+    expect(tools).toContain('exit_plan_mode');
     expect(tools).toContain('ask_followup_question');
 
     // Should NOT include write operations