From d79478689fb0bc6ef54f782f7251fe787a536c23 Mon Sep 17 00:00:00 2001 From: Adib234 <30782825+Adib234@users.noreply.github.com> Date: Wed, 4 Feb 2026 14:43:25 -0500 Subject: [PATCH] Stop showing an error message in `/plan` (#18333) --- packages/cli/src/ui/commands/planCommand.test.ts | 15 --------------- packages/cli/src/ui/commands/planCommand.ts | 4 ---- 2 files changed, 19 deletions(-) diff --git a/packages/cli/src/ui/commands/planCommand.test.ts b/packages/cli/src/ui/commands/planCommand.test.ts index 410694b2ed..af556ae255 100644 --- a/packages/cli/src/ui/commands/planCommand.test.ts +++ b/packages/cli/src/ui/commands/planCommand.test.ts @@ -92,21 +92,6 @@ describe('planCommand', () => { ); }); - it('should show "No approved plan found" if no approved plan path in config', async () => { - vi.mocked(mockContext.services.config!.isPlanEnabled).mockReturnValue(true); - vi.mocked(mockContext.services.config!.getApprovedPlanPath).mockReturnValue( - undefined, - ); - - if (!planCommand.action) throw new Error('Action missing'); - await planCommand.action(mockContext, ''); - - expect(coreEvents.emitFeedback).toHaveBeenCalledWith( - 'error', - 'No approved plan found. Please create and approve a plan first.', - ); - }); - it('should display the approved plan from config', async () => { const mockPlanPath = '/mock/plans/dir/approved-plan.md'; vi.mocked(mockContext.services.config!.isPlanEnabled).mockReturnValue(true); diff --git a/packages/cli/src/ui/commands/planCommand.ts b/packages/cli/src/ui/commands/planCommand.ts index 53fad50c65..c64b0048f4 100644 --- a/packages/cli/src/ui/commands/planCommand.ts +++ b/packages/cli/src/ui/commands/planCommand.ts @@ -37,10 +37,6 @@ export const planCommand: SlashCommand = { const approvedPlanPath = config.getApprovedPlanPath(); if (!approvedPlanPath) { - coreEvents.emitFeedback( - 'error', - 'No approved plan found. Please create and approve a plan first.', - ); return; }