feat(terminal): add default command output collapse setting

packages/types/src/__tests__/global-settings.test.ts

@@ -38,3 +38,16 @@ describe("alwaysDenyUnapprovedCommands global setting", () => {
 		expect(() => globalSettingsSchema.parse({ alwaysDenyUnapprovedCommands: "true" })).toThrow()
 	})
 })
+
+describe("collapseCommandOutputByDefault global setting", () => {
+	it.each([true, false, undefined])("round-trips %s through the export/import schema", (value) => {
+		const settings = globalSettingsSchema.parse({ collapseCommandOutputByDefault: value })
+		expect(globalSettingsSchema.parse(JSON.parse(JSON.stringify(settings))).collapseCommandOutputByDefault).toBe(
+			value,
+		)
+		expect(GLOBAL_SETTINGS_KEYS).toContain("collapseCommandOutputByDefault")
+	})
+	it("rejects non-boolean values", () => {
+		expect(globalSettingsSchema.safeParse({ collapseCommandOutputByDefault: "true" }).success).toBe(false)
+	})
+})

packages/types/src/global-settings.ts

@@ -49,6 +49,9 @@
  */
 export const DEFAULT_DIFF_FUZZY_THRESHOLD = 1.0
 
+// Preserve the existing terminal-dependent output display until users opt in.
+export const DEFAULT_COLLAPSE_COMMAND_OUTPUT_BY_DEFAULT = false
+
 export const DEFAULT_DESTRUCTIVE_COMMAND_GUARD_ENABLED = false
 
 /**
@@ -236,6 +239,7 @@
 	terminalOutputPreviewSize: z.enum(["small", "medium", "large"]).optional(),
 	terminalShellIntegrationTimeout: z.number().optional(),
 	terminalShellIntegrationDisabled: z.boolean().optional(),
+	collapseCommandOutputByDefault: z.boolean().optional(),
 	terminalCommandDelay: z.number().optional(),
 	terminalPowershellCounter: z.boolean().optional(),
 	terminalZshClearEolMark: z.boolean().optional(),

packages/types/src/vscode-extension-host.ts

@@ -310,6 +310,7 @@ export type ExtensionState = Pick<
 	| "terminalOutputPreviewSize"
 	| "terminalShellIntegrationTimeout"
 	| "terminalShellIntegrationDisabled"
+	| "collapseCommandOutputByDefault"
 	| "terminalCommandDelay"
 	| "terminalPowershellCounter"
 	| "terminalZshClearEolMark"

src/core/webview/ClineProvider.ts

@@ -48,6 +48,7 @@ import {
 	DEFAULT_DIFF_FUZZY_THRESHOLD,
 	DEFAULT_DESTRUCTIVE_COMMAND_GUARD_ENABLED,
 	DEFAULT_ALWAYS_DENY_UNAPPROVED_COMMANDS,
+	DEFAULT_COLLAPSE_COMMAND_OUTPUT_BY_DEFAULT,
 	DEFAULT_AUTO_CLOSE_ZOO_OPENED_FILES,
 	DEFAULT_AUTO_CLOSE_ZOO_OPENED_FILES_AFTER_USER_EDITED,
 	DEFAULT_AUTO_CLOSE_ZOO_OPENED_NEW_FILES,
@@ -2751,6 +2752,7 @@ export class ClineProvider
 			diffFuzzyThreshold,
 			terminalShellIntegrationTimeout,
 			terminalShellIntegrationDisabled,
+			collapseCommandOutputByDefault,
 			terminalCommandDelay,
 			terminalPowershellCounter,
 			terminalZshClearEolMark,
@@ -2923,6 +2925,8 @@ export class ClineProvider
 			diffFuzzyThreshold: diffFuzzyThreshold ?? DEFAULT_DIFF_FUZZY_THRESHOLD,
 			terminalShellIntegrationTimeout: terminalShellIntegrationTimeout ?? Terminal.defaultShellIntegrationTimeout,
 			terminalShellIntegrationDisabled: terminalShellIntegrationDisabled ?? true,
+			collapseCommandOutputByDefault:
+				collapseCommandOutputByDefault ?? DEFAULT_COLLAPSE_COMMAND_OUTPUT_BY_DEFAULT,
 			terminalCommandDelay: terminalCommandDelay ?? 0,
 			terminalPowershellCounter: terminalPowershellCounter ?? false,
 			terminalZshClearEolMark: terminalZshClearEolMark ?? true,
@@ -3159,6 +3163,8 @@ export class ClineProvider
 			terminalShellIntegrationTimeout:
 				stateValues.terminalShellIntegrationTimeout ?? Terminal.defaultShellIntegrationTimeout,
 			terminalShellIntegrationDisabled: stateValues.terminalShellIntegrationDisabled ?? true,
+			collapseCommandOutputByDefault:
+				stateValues.collapseCommandOutputByDefault ?? DEFAULT_COLLAPSE_COMMAND_OUTPUT_BY_DEFAULT,
 			terminalCommandDelay: stateValues.terminalCommandDelay ?? 0,
 			terminalPowershellCounter: stateValues.terminalPowershellCounter ?? false,
 			terminalZshClearEolMark: stateValues.terminalZshClearEolMark ?? true,

src/core/webview/__tests__/ClineProvider.spec.ts

@@ -2226,6 +2226,16 @@ describe("ClineProvider", () => {
 		expect(state.writeDelayMs).toBe(DEFAULT_WRITE_DELAY_MS)
 	})
 
+	test.each([true, false, undefined])(
+		"returns the saved output collapse preference (%s) to the webview",
+		async (value) => {
+			await provider.resolveWebviewView(mockWebviewView)
+			await provider.contextProxy.setValue("collapseCommandOutputByDefault", value)
+			expect((await provider.getState()).collapseCommandOutputByDefault).toBe(value ?? false)
+			expect((await provider.getStateToPostToWebview()).collapseCommandOutputByDefault).toBe(value ?? false)
+		},
+	)
+
 	test("getState applies fallback defaults for write, diff, and terminal settings", async () => {
 		// Mock globalState.get to return undefined for the fallback settings
 		// (typed reassignment — Memento.get has no mock type, same pattern as the

src/core/webview/__tests__/webviewMessageHandler.spec.ts

@@ -2971,6 +2971,31 @@ describe("webviewMessageHandler - telemetrySetting", () => {
 	})
 })
 
+describe("webviewMessageHandler - collapseCommandOutputByDefault", () => {
+	beforeEach(() => {
+		vi.clearAllMocks()
+	})
+
+	it.each([true, false, undefined])(
+		"persists the output collapse preference (%s) without changing terminal mode",
+		async (value) => {
+			await webviewMessageHandler(mockClineProvider, {
+				type: "updateSettings",
+				updatedSettings: { collapseCommandOutputByDefault: value },
+			})
+			expect(mockClineProvider.contextProxy.setValue).toHaveBeenCalledWith(
+				"collapseCommandOutputByDefault",
+				value,
+			)
+			expect(mockClineProvider.contextProxy.setValue).not.toHaveBeenCalledWith(
+				"terminalShellIntegrationDisabled",
+				expect.anything(),
+			)
+			expect(mockClineProvider.postStateToWebview).toHaveBeenCalledTimes(1)
+		},
+	)
+})
+
 describe("updateProfileModel", () => {
 	let updateProfileModel: Mock<ClineProvider["updateProfileModel"]>
 

webview-ui/playwright/gallery/stories.tsx

@@ -12,6 +12,33 @@ const mermaidGantt = `gantt
     Ship release :active, release, after scope, 3d`
 
 export const stories: Record<string, Story> = {
+	"terminal-settings": async () => {
+		const [{ AppProviders }, { TerminalSettings }] = await Promise.all([
+			import("../AppProviders"),
+			import("@/components/settings/TerminalSettings"),
+		])
+		function Settings() {
+			const [collapsed, setCollapsed] = useState(false)
+			return (
+				<div
+					className="w-[420px] p-4 bg-vscode-editor-background text-vscode-foreground"
+					data-testid="terminal-settings-story">
+					<TerminalSettings
+						terminalShellIntegrationDisabled={true}
+						collapseCommandOutputByDefault={collapsed}
+						setCachedStateField={(field, value) => {
+							if (field === "collapseCommandOutputByDefault") setCollapsed(Boolean(value))
+						}}
+					/>
+				</div>
+			)
+		}
+		return (
+			<AppProviders>
+				<Settings />
+			</AppProviders>
+		)
+	},
 	"bedrock-output-budget": async ({ model = "anthropic.claude-opus-5" }) => {
 		const [{ AppProviders }, { ThinkingBudget }, { bedrockModels, providerIdentifiers }] = await Promise.all([
 			import("../AppProviders"),

webview-ui/src/components/chat/CommandExecution.tsx

@@ -3,7 +3,12 @@
 import { t } from "i18next"
 import { ChevronDown, OctagonX } from "lucide-react"
 
-import { type ExtensionMessage, type CommandExecutionStatus, commandExecutionStatusSchema } from "@roo-code/types"
+import {
+	type ExtensionMessage,
+	type CommandExecutionStatus,
+	commandExecutionStatusSchema,
+	DEFAULT_COLLAPSE_COMMAND_OUTPUT_BY_DEFAULT,
+} from "@roo-code/types"
 
 import { safeJsonParse } from "@roo/core"
 import { COMMAND_OUTPUT_STRING } from "@roo/combineCommandSequences"
@@ -42,6 +47,7 @@
 export const CommandExecution = ({ executionId, text, icon, title, isDenied = false }: CommandExecutionProps) => {
 	const {
 		terminalShellIntegrationDisabled = false,
+		collapseCommandOutputByDefault = DEFAULT_COLLAPSE_COMMAND_OUTPUT_BY_DEFAULT,
 		destructiveCommandGuardEnabled = false,
 		allowedCommands = [],
 		deniedCommands = [],
@@ -51,9 +57,8 @@
 
 	const { command, output: parsedOutput } = useMemo(() => parseCommandAndOutput(text), [text])
 
-	// If we aren't opening the VSCode terminal for this command then we default
-	// to expanding the command execution output.
-	const [isExpanded, setIsExpanded] = useState(terminalShellIntegrationDisabled)
+	// Apply the preference on mount. Manual toggles remain in control afterwards.
+	const [isExpanded, setIsExpanded] = useState(!collapseCommandOutputByDefault && terminalShellIntegrationDisabled)
 	const [streamingOutput, setStreamingOutput] = useState("")
 	// Initialize from the module-level cache so that components mounting after
 	// the "started" event was delivered still show the running indicator.
@@ -159,7 +164,9 @@
 							// Not a terminal state -- signals a mid-execution retry
 							// via execa after a shell integration failure. A new
 							// "started" event will follow, so leave the cache intact.
-							setIsExpanded(true)
+							if (!collapseCommandOutputByDefault) {
+								setIsExpanded(true)
+							}
 							break
 						case "output":
 							setStreamingOutput(data.output)
@@ -171,7 +178,7 @@
 				}
 			}
 		},
-		[executionId],
+		[executionId, collapseCommandOutputByDefault],
 	)
 
 	useEvent("message", onMessage)

webview-ui/src/components/chat/__tests__/CommandExecution.collapse.spec.tsx

@@ -0,0 +1,73 @@
+import { act, fireEvent, screen, renderWithExtensionState } from "@/utils/test-utils"
+import { CommandExecution } from "../CommandExecution"
+
+vi.mock("@/utils/vscode", () => ({ vscode: { postMessage: vi.fn() } }))
+vi.mock("../../common/CodeBlock", () => ({
+	default: ({ source }: { source: string }) => <div>{source}</div>,
+}))
+vi.mock("../TerminalOutput", () => ({
+	TerminalOutput: ({ content }: { content: string }) => <div data-testid="terminal-output">{content}</div>,
+}))
+vi.mock("../CommandPatternSelector", () => ({ CommandPatternSelector: () => null }))
+
+const sendStatus = (data: Record<string, unknown>) => {
+	act(() =>
+		window.dispatchEvent(
+			new MessageEvent("message", {
+				data: {
+					type: "commandExecutionStatus",
+					text: JSON.stringify({ executionId: "collapse-test", ...data }),
+				},
+			}),
+		),
+	)
+}
+const setup = (collapseCommandOutputByDefault?: boolean, terminalShellIntegrationDisabled = true) =>
+	renderWithExtensionState(
+		<CommandExecution executionId="collapse-test" text="npm test\nOutput:\nInitial output" />,
+		{
+			state: { collapseCommandOutputByDefault, terminalShellIntegrationDisabled },
+		},
+	)
+const outputContainer = () => screen.getByTestId("terminal-output").parentElement
+
+describe("command output collapse preference", () => {
+	it.each([
+		[true, true, true],
+		[true, false, true],
+		[false, true, false],
+		[false, false, true],
+		[undefined, true, false],
+		[undefined, false, true],
+	])("applies preference %s with inline mode %s", (preference, inline, collapsed) => {
+		setup(preference, inline)
+		expect(outputContainer()).toHaveClass(collapsed ? "max-h-0" : "max-h-[100%]")
+	})
+	it("keeps streaming and completed output collapsed until manually expanded", () => {
+		setup(true)
+		sendStatus({ status: "started", pid: 123 })
+		sendStatus({ status: "output", output: "Streaming output" })
+		expect(screen.getByTestId("terminal-output")).toHaveTextContent("Streaming output")
+		expect(outputContainer()).toHaveClass("max-h-0")
+		fireEvent.click(screen.getAllByRole("button").at(-1)!)
+		expect(outputContainer()).not.toHaveClass("max-h-0")
+		sendStatus({ status: "output", output: "More output" })
+		sendStatus({ status: "exited", exitCode: 0 })
+		expect(outputContainer()).not.toHaveClass("max-h-0")
+		fireEvent.click(screen.getAllByRole("button").at(-1)!)
+		expect(outputContainer()).toHaveClass("max-h-0")
+	})
+
+	it("preserves a manual expansion through fallback and new output", () => {
+		setup(true, false)
+		fireEvent.click(screen.getAllByRole("button").at(-1)!)
+		sendStatus({ status: "fallback" })
+		sendStatus({ status: "output", output: "Fallback output" })
+		expect(outputContainer()).not.toHaveClass("max-h-0")
+	})
+	it.each([true, false, undefined])("respects preference %s during shell integration fallback", (preference) => {
+		setup(preference, false)
+		sendStatus({ status: "fallback" })
+		expect(outputContainer()).toHaveClass(preference ? "max-h-0" : "max-h-[100%]")
+	})
+})

webview-ui/src/components/settings/SettingsView.tsx

@@ -188,6 +188,7 @@ const SettingsView = forwardRef<SettingsViewRef, SettingsViewProps>(({ onDone, t
 		terminalOutputPreviewSize,
 		terminalShellIntegrationTimeout,
 		terminalShellIntegrationDisabled, // Added from upstream
+		collapseCommandOutputByDefault,
 		terminalCommandDelay,
 		terminalPowershellCounter,
 		terminalZshClearEolMark,
@@ -416,6 +417,7 @@ const SettingsView = forwardRef<SettingsViewRef, SettingsViewProps>(({ onDone, t
 					diffFuzzyThreshold,
 					terminalShellIntegrationTimeout: terminalShellIntegrationTimeout ?? 30_000,
 					terminalShellIntegrationDisabled,
+					collapseCommandOutputByDefault,
 					terminalCommandDelay,
 					terminalPowershellCounter,
 					terminalZshClearEolMark,
@@ -897,6 +899,7 @@ const SettingsView = forwardRef<SettingsViewRef, SettingsViewProps>(({ onDone, t
 								terminalOutputPreviewSize={terminalOutputPreviewSize}
 								terminalShellIntegrationTimeout={terminalShellIntegrationTimeout}
 								terminalShellIntegrationDisabled={terminalShellIntegrationDisabled}
+								collapseCommandOutputByDefault={collapseCommandOutputByDefault}
 								terminalCommandDelay={terminalCommandDelay}
 								terminalPowershellCounter={terminalPowershellCounter}
 								terminalZshClearEolMark={terminalZshClearEolMark}

webview-ui/src/components/settings/TerminalSettings.tsx

@@ -7,7 +7,11 @@
 import { useEvent, useMount } from "react-use"
 import { Terminal } from "lucide-react"
 
-import { type ExtensionMessage, type TerminalOutputPreviewSize } from "@roo-code/types"
+import {
+	type ExtensionMessage,
+	type TerminalOutputPreviewSize,
+	DEFAULT_COLLAPSE_COMMAND_OUTPUT_BY_DEFAULT,
+} from "@roo-code/types"
 
 import { cn } from "@/lib/utils"
 import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue, Slider, Button } from "@/components/ui"
@@ -21,6 +25,7 @@
 	terminalOutputPreviewSize?: TerminalOutputPreviewSize
 	terminalShellIntegrationTimeout?: number
 	terminalShellIntegrationDisabled?: boolean
+	collapseCommandOutputByDefault?: boolean
 	terminalCommandDelay?: number
 	terminalPowershellCounter?: boolean
 	terminalZshClearEolMark?: boolean
@@ -33,6 +38,7 @@
 		| "terminalOutputPreviewSize"
 		| "terminalShellIntegrationTimeout"
 		| "terminalShellIntegrationDisabled"
+		| "collapseCommandOutputByDefault"
 		| "terminalCommandDelay"
 		| "terminalPowershellCounter"
 		| "terminalZshClearEolMark"
@@ -51,6 +57,7 @@
 	terminalOutputPreviewSize,
 	terminalShellIntegrationTimeout,
 	terminalShellIntegrationDisabled,
+	collapseCommandOutputByDefault,
 	terminalCommandDelay,
 	terminalPowershellCounter,
 	terminalZshClearEolMark,
@@ -151,6 +158,26 @@
 					</div>
 				</div>
 
+				<SearchableSetting
+					settingId="terminal-collapse-command-output"
+					section="terminal"
+					label={t("settings:terminal.collapseCommandOutput.label")}>
+					<VSCodeCheckbox
+						checked={collapseCommandOutputByDefault ?? DEFAULT_COLLAPSE_COMMAND_OUTPUT_BY_DEFAULT}
+						onChange={(event) =>
+							setCachedStateField(
+								"collapseCommandOutputByDefault",
+								(event.target as HTMLInputElement).checked,
+							)
+						}
+						data-testid="terminal-collapse-command-output-checkbox">
+						<span className="font-medium">{t("settings:terminal.collapseCommandOutput.label")}</span>
+					</VSCodeCheckbox>
+					<div className="text-vscode-descriptionForeground text-sm mt-1">
+						{t("settings:terminal.collapseCommandOutput.description")}
+					</div>
+				</SearchableSetting>
+
 				{/* Advanced Settings */}
 				<div className="flex flex-col gap-3">
 					<div className="flex flex-col gap-1">