test(cli): improve coverage for autonomous mode and validation paths

Add comprehensive test coverage for:
- AskDispatcher error handling and autonomous mode paths
- run.ts provider base URL validation in non-autonomous mode
- Disabled ask handling mode
- Partial message handling
- Unknown ask types with onInputRequired callback
- Error recovery (removing asks from handled set on error)
- API request failure handling with onInputRequired
- Followup questions with onInputRequired in autonomous mode

This addresses codecov/patch coverage gaps by testing previously
uncovered branches in autonomous mode execution and validation.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: taltas

apps/cli/src/agent/__tests__/ask-dispatcher.test.ts

@@ -0,0 +1,173 @@
+import type { ClineMessage, ClineAsk } from "@roo-code/types"
+import { AskDispatcher } from "../ask-dispatcher.js"
+import type { OutputManager } from "../output-manager.js"
+import type { PromptManager } from "../prompt-manager.js"
+
+const createMockOutputManager = (): OutputManager =>
+	({
+		output: vi.fn(),
+		markDisplayed: vi.fn(),
+	}) as unknown as OutputManager
+
+const createMockPromptManager = (): PromptManager =>
+	({
+		promptForInput: vi.fn(),
+		promptForYesNo: vi.fn(),
+		promptWithTimeout: vi.fn(),
+	}) as unknown as PromptManager
+
+describe("AskDispatcher", () => {
+	let mockOutputManager: OutputManager
+	let mockPromptManager: PromptManager
+	let sendMessageMock: ReturnType<typeof vi.fn>
+
+	beforeEach(() => {
+		mockOutputManager = createMockOutputManager()
+		mockPromptManager = createMockPromptManager()
+		sendMessageMock = vi.fn()
+	})
+
+	describe("handleAsk - disabled mode", () => {
+		it("returns handled=false when disabled", async () => {
+			const dispatcher = new AskDispatcher({
+				outputManager: mockOutputManager,
+				promptManager: mockPromptManager,
+				sendMessage: sendMessageMock,
+				disabled: true,
+			})
+
+			const message: ClineMessage = {
+				ts: 1,
+				type: "ask",
+				ask: "followup",
+				text: "test",
+				partial: false,
+			} as ClineMessage
+
+			const result = await dispatcher.handleAsk(message)
+			expect(result.handled).toBe(false)
+		})
+	})
+
+	describe("handleAsk - partial messages", () => {
+		it("skips partial messages", async () => {
+			const dispatcher = new AskDispatcher({
+				outputManager: mockOutputManager,
+				promptManager: mockPromptManager,
+				sendMessage: sendMessageMock,
+			})
+
+			const message: ClineMessage = {
+				ts: 1,
+				type: "ask",
+				ask: "followup",
+				text: "test",
+				partial: true,
+			} as ClineMessage
+
+			const result = await dispatcher.handleAsk(message)
+			expect(result.handled).toBe(false)
+		})
+	})
+
+	describe("handleAsk - unknown ask type in non-interactive mode", () => {
+		it("calls onInputRequired for unknown ask types", async () => {
+			const onInputRequired = vi.fn()
+			const dispatcher = new AskDispatcher({
+				outputManager: mockOutputManager,
+				promptManager: mockPromptManager,
+				sendMessage: sendMessageMock,
+				nonInteractive: true,
+				onInputRequired,
+			})
+
+			const message: ClineMessage = {
+				ts: 1,
+				type: "ask",
+				ask: "unknown_ask_type" as ClineAsk,
+				text: "test message",
+				partial: false,
+			} as ClineMessage
+
+			const result = await dispatcher.handleAsk(message)
+			expect(result.handled).toBe(true)
+			expect(onInputRequired).toHaveBeenCalledWith("unknown_ask_type", "test message")
+		})
+	})
+
+	describe("handleAsk - error handling", () => {
+		it("removes ask from handled set on error", async () => {
+			const dispatcher = new AskDispatcher({
+				outputManager: mockOutputManager,
+				promptManager: mockPromptManager,
+				sendMessage: () => {
+					throw new Error("sendMessage error")
+				},
+			})
+
+			const message: ClineMessage = {
+				ts: 1,
+				type: "ask",
+				ask: "command_output",
+				text: "test",
+				partial: false,
+			} as ClineMessage
+
+			const result = await dispatcher.handleAsk(message)
+			expect(result.handled).toBe(false)
+			expect(result.error).toBeInstanceOf(Error)
+
+			// Should be able to handle again after error
+			expect(dispatcher.isHandled(1)).toBe(false)
+		})
+	})
+
+	describe("api_req_failed handling", () => {
+		it("calls onInputRequired when provided", async () => {
+			const onInputRequired = vi.fn()
+			const dispatcher = new AskDispatcher({
+				outputManager: mockOutputManager,
+				promptManager: mockPromptManager,
+				sendMessage: sendMessageMock,
+				onInputRequired,
+			})
+
+			const message: ClineMessage = {
+				ts: 1,
+				type: "ask",
+				ask: "api_req_failed",
+				text: "API error",
+				partial: false,
+			} as ClineMessage
+
+			const result = await dispatcher.handleAsk(message)
+			expect(result.handled).toBe(true)
+			expect(onInputRequired).toHaveBeenCalledWith("api_req_failed", "API error")
+		})
+	})
+
+	describe("followup handling with onInputRequired", () => {
+		it("calls onInputRequired in non-interactive mode", async () => {
+			const onInputRequired = vi.fn()
+			const dispatcher = new AskDispatcher({
+				outputManager: mockOutputManager,
+				promptManager: mockPromptManager,
+				sendMessage: sendMessageMock,
+				nonInteractive: true,
+				onInputRequired,
+			})
+
+			const message: ClineMessage = {
+				ts: 1,
+				type: "ask",
+				ask: "followup",
+				text: JSON.stringify({ question: "What next?", suggest: [{ answer: "Continue" }] }),
+				partial: false,
+			} as ClineMessage
+
+			const result = await dispatcher.handleAsk(message)
+			expect(result.handled).toBe(true)
+			expect(onInputRequired).toHaveBeenCalledWith("followup", "What next?")
+		})
+	})
+})

apps/cli/src/commands/cli/__tests__/run.test.ts

@@ -389,6 +389,21 @@ describe("run command validation", () => {
 		})
 	})
 
+	describe("providerBaseUrl validation", () => {
+		it("should reject providerBaseUrl with non-openrouter provider in non-autonomous mode", async () => {
+			const flags = createFlagOptions({
+				providerBaseUrl: "https://custom-base-url.com",
+				provider: "anthropic",
+				print: true,
+			})
+
+			await expect(run("test", flags)).rejects.toThrow("process.exit: 1")
+			expect(consoleErrorSpy).toHaveBeenCalledWith(
+				expect.stringContaining("--provider-base-url is currently supported only with --provider openrouter"),
+			)
+		})
+	})
+
 	describe("prompt file validation", () => {
 		it("should reject non-existent prompt file", async () => {
 			const flags = createFlagOptions({