gsd-pi 2.79.0 → 2.80.0-dev.0aaa4fc17

package/packages/pi-coding-agent/src/modes/interactive/slash-command-handlers.test.ts

@@ -0,0 +1,95 @@
+// GSD2 - Slash command tests for interactive TUI settings commands
+
+import assert from "node:assert/strict";
+import { describe, it } from "node:test";
+import { Container } from "@gsd/pi-tui";
+import { SettingsManager } from "../../core/settings-manager.js";
+import { dispatchSlashCommand, type SlashCommandContext } from "./slash-command-handlers.js";
+
+function makeContext(settingsManager = SettingsManager.inMemory()): SlashCommandContext {
+	const statuses: string[] = [];
+	const warnings: string[] = [];
+	const renders: string[] = [];
+	return {
+		session: {} as never,
+		ui: {} as never,
+		keybindings: {} as never,
+		chatContainer: new Container(),
+		statusContainer: new Container(),
+		editorContainer: new Container(),
+		headerContainer: new Container(),
+		pendingMessagesContainer: new Container(),
+		editor: {} as never,
+		defaultEditor: {} as never,
+		sessionManager: {} as never,
+		settingsManager,
+		invalidateFooter() {},
+		showStatus(message: string) {
+			statuses.push(message);
+		},
+		showError(message: string) {
+			throw new Error(message);
+		},
+		showWarning(message: string) {
+			warnings.push(message);
+		},
+		showSelector() {},
+		updateEditorBorderColor() {},
+		getMarkdownThemeWithSettings: () => ({} as never),
+		requestRender() {
+			renders.push("render");
+		},
+		updateTerminalTitle() {},
+		showSettingsSelector() {},
+		showModelsSelector: async () => {},
+		handleModelCommand: async () => {},
+		showUserMessageSelector() {},
+		showTreeSelector() {},
+		showProviderManager() {},
+		showOAuthSelector: async () => {},
+		showSessionSelector() {},
+		handleClearCommand: async () => {},
+		handleReloadCommand: async () => {},
+		handleDebugCommand() {},
+		shutdown: async () => {},
+		executeCompaction: async () => undefined,
+		handleBashCommand: async () => {},
+		_testStatuses: statuses,
+		_testWarnings: warnings,
+		_testRenders: renders,
+	} as SlashCommandContext & {
+		_testStatuses: string[];
+		_testWarnings: string[];
+		_testRenders: string[];
+	};
+}
+
+describe("dispatchSlashCommand /tui", () => {
+	it("persists /tui mode validation to terminal adaptive mode", async () => {
+		const settingsManager = SettingsManager.inMemory();
+		const ctx = makeContext(settingsManager) as SlashCommandContext & {
+			_testStatuses: string[];
+			_testRenders: string[];
+		};
+
+		const handled = await dispatchSlashCommand("/tui mode validation", ctx);
+
+		assert.equal(handled, true);
+		assert.equal(settingsManager.getAdaptiveMode(), "validation");
+		assert.deepEqual(ctx._testStatuses, ["TUI mode: validation"]);
+		assert.equal(ctx._testRenders.length, 1);
+	});
+
+	it("rejects unknown TUI modes without changing settings", async () => {
+		const settingsManager = SettingsManager.inMemory({ terminal: { adaptiveMode: "workflow" } });
+		const ctx = makeContext(settingsManager) as SlashCommandContext & {
+			_testWarnings: string[];
+		};
+
+		const handled = await dispatchSlashCommand("/tui mode poster", ctx);
+
+		assert.equal(handled, true);
+		assert.equal(settingsManager.getAdaptiveMode(), "workflow");
+		assert.match(ctx._testWarnings[0], /Usage: \/tui mode/);
+	});
+});

package/packages/pi-coding-agent/src/modes/interactive/slash-command-handlers.ts

@@ -31,7 +31,7 @@ import {
 import type { AgentSession } from "../../core/agent-session.js";
 import type { AppAction, KeybindingsManager } from "../../core/keybindings.js";
 import type { SessionManager } from "../../core/session-manager.js";
-import type { SettingsManager } from "../../core/settings-manager.js";
+import type { AdaptiveTuiMode, SettingsManager } from "../../core/settings-manager.js";
 import { copyToClipboard } from "../../utils/clipboard.js";
 import { getChangelogPath, parseChangelog } from "../../utils/changelog.js";
 import { ArminComponent } from "./components/armin.js";
@@ -235,6 +235,10 @@ export async function dispatchSlashCommand(
 		await ctx.handleBashCommand(command, { loginShell: true });
 		return true;
 	}
+	if (text === "/tui" || text.startsWith("/tui ")) {
+		handleTuiCommand(text, ctx);
+		return true;
+	}
 
 	return false;
 }
@@ -665,6 +669,25 @@ function handleEditModeCommand(arg: string | undefined, ctx: SlashCommandContext
 	ctx.showStatus(`Edit mode: ${next}${next === "hashline" ? " (LINE#ID anchored edits)" : " (text-match edits)"}`);
 }
 
+function handleTuiCommand(text: string, ctx: SlashCommandContext): void {
+	const parts = text.trim().split(/\s+/);
+	const mode = parts[1] === "mode" ? parts[2] : parts[1];
+	const valid: AdaptiveTuiMode[] = ["auto", "chat", "workflow", "validation", "debug", "compact"];
+
+	if (!mode) {
+		ctx.showStatus(`TUI mode: ${ctx.settingsManager.getAdaptiveMode()}`);
+		return;
+	}
+	if (!valid.includes(mode as AdaptiveTuiMode)) {
+		ctx.showWarning(`Usage: /tui mode ${valid.join("|")}`);
+		return;
+	}
+
+	ctx.settingsManager.setAdaptiveMode(mode as AdaptiveTuiMode);
+	ctx.showStatus(`TUI mode: ${mode}`);
+	ctx.requestRender();
+}
+
 function handleArminSaysHi(ctx: SlashCommandContext): void {
 	ctx.chatContainer.addChild(new Spacer(1));
 	ctx.chatContainer.addChild(new ArminComponent(ctx.ui));

package/packages/pi-coding-agent/src/modes/interactive/theme/theme-schema.ts

@@ -70,6 +70,19 @@ export const ThemeJsonSchema = Type.Object({
     thinkingXhigh: ColorValueSchema,
     // Bash Mode (1 color)
     bashMode: ColorValueSchema,
+    // Adaptive TUI semantic tokens (optional; defaults are derived from core colors)
+    surfaceBorder: Type.Optional(ColorValueSchema),
+    surfaceMuted: Type.Optional(ColorValueSchema),
+    surfaceTitle: Type.Optional(ColorValueSchema),
+    surfaceAccent: Type.Optional(ColorValueSchema),
+    toolRunning: Type.Optional(ColorValueSchema),
+    toolSuccess: Type.Optional(ColorValueSchema),
+    toolError: Type.Optional(ColorValueSchema),
+    toolMuted: Type.Optional(ColorValueSchema),
+    modeWorkflow: Type.Optional(ColorValueSchema),
+    modeValidation: Type.Optional(ColorValueSchema),
+    modeDebug: Type.Optional(ColorValueSchema),
+    modeCompact: Type.Optional(ColorValueSchema),
   }),
   export: Type.Optional(
     Type.Object({

package/packages/pi-coding-agent/src/modes/interactive/theme/theme.ts

@@ -69,7 +69,19 @@ export type ThemeColor =
 	| "thinkingMedium"
 	| "thinkingHigh"
 	| "thinkingXhigh"
-	| "bashMode";
+	| "bashMode"
+	| "surfaceBorder"
+	| "surfaceMuted"
+	| "surfaceTitle"
+	| "surfaceAccent"
+	| "toolRunning"
+	| "toolSuccess"
+	| "toolError"
+	| "toolMuted"
+	| "modeWorkflow"
+	| "modeValidation"
+	| "modeDebug"
+	| "modeCompact";
 
 export type ThemeBg =
 	| "selectedBg"
@@ -374,6 +386,24 @@ function getBuiltinThemes(): Record<string, ThemeJson> {
 	return builtinThemes;
 }
 
+function withSemanticColorDefaults(colors: ThemeJson["colors"]): ThemeJson["colors"] {
+	return {
+		...colors,
+		surfaceBorder: colors.surfaceBorder ?? colors.border,
+		surfaceMuted: colors.surfaceMuted ?? colors.borderMuted,
+		surfaceTitle: colors.surfaceTitle ?? colors.toolTitle,
+		surfaceAccent: colors.surfaceAccent ?? colors.borderAccent,
+		toolRunning: colors.toolRunning ?? colors.warning,
+		toolSuccess: colors.toolSuccess ?? colors.success,
+		toolError: colors.toolError ?? colors.error,
+		toolMuted: colors.toolMuted ?? colors.muted,
+		modeWorkflow: colors.modeWorkflow ?? colors.accent,
+		modeValidation: colors.modeValidation ?? colors.warning,
+		modeDebug: colors.modeDebug ?? colors.error,
+		modeCompact: colors.modeCompact ?? colors.muted,
+	};
+}
+
 export function getAvailableThemes(): string[] {
 	const themes = new Set<string>(Object.keys(getBuiltinThemes()));
 	const customThemesDir = getCustomThemesDir();
@@ -493,7 +523,7 @@ function loadThemeJson(name: string): ThemeJson {
 
 function createTheme(themeJson: ThemeJson, mode?: ColorMode, sourcePath?: string): Theme {
 	const colorMode = mode ?? detectColorMode();
-	const resolvedColors = resolveThemeColors(themeJson.colors, themeJson.vars);
+	const resolvedColors = resolveThemeColors(withSemanticColorDefaults(themeJson.colors), themeJson.vars);
 	const fgColors: Record<ThemeColor, string | number> = {} as Record<ThemeColor, string | number>;
 	const bgColors: Record<ThemeBg, string | number> = {} as Record<ThemeBg, string | number>;
 	const bgColorKeys: Set<string> = new Set([

package/packages/pi-coding-agent/src/modes/interactive/theme/themes.ts

@@ -1,3 +1,4 @@
+// GSD2 TUI - Built-in terminal theme definitions.
 /**
  * Built-in theme definitions.
  *
@@ -18,27 +19,30 @@ import type { ThemeJson } from "./theme-schema.js";
 const dark: ThemeJson = {
 	name: "dark",
 	vars: {
-		cyan: "#22d3ee",
-		blue: "#60a5fa",
-		green: "#34d399",
-		red: "#fb7185",
-		yellow: "#facc15",
-		gray: "#a1a1aa",
-		dimGray: "#8b93a7",
-		darkGray: "#4b5563",
-		accent: "#2dd4bf",
-		selectedBg: "#1f2a44",
-		userMsgBg: "#1e2535",
-		toolPendingBg: "#1f2330",
-		toolSuccessBg: "#1f3128",
-		toolErrorBg: "#3a202a",
-		customMsgBg: "#2a2140",
+		cyan: "#5cc8c8",
+		blue: "#79b8ff",
+		green: "#7ad38b",
+		red: "#f27b7b",
+		yellow: "#d6b85f",
+		gray: "#87949b",
+		dimGray: "#657178",
+		darkGray: "#303a40",
+		line: "#52616a",
+		lineSoft: "#303a40",
+		textSoft: "#cbd6dc",
+		accent: "#5cc8c8",
+		selectedBg: "#1d282a",
+		userMsgBg: "#202a2f",
+		toolPendingBg: "#252721",
+		toolSuccessBg: "#1c251f",
+		toolErrorBg: "#251e20",
+		customMsgBg: "#202a2f",
 	},
 	colors: {
 		accent: "accent",
-		border: "blue",
-		borderAccent: "cyan",
-		borderMuted: "darkGray",
+		border: "line",
+		borderAccent: "accent",
+		borderMuted: "lineSoft",
 		success: "green",
 		error: "red",
 		warning: "yellow",
@@ -49,16 +53,21 @@ const dark: ThemeJson = {
 
 		selectedBg: "selectedBg",
 		userMessageBg: "userMsgBg",
-		userMessageText: "gray",
-		assistantMessageText: "",
+		userMessageText: "textSoft",
+		assistantMessageText: "textSoft",
 		customMessageBg: "customMsgBg",
-		customMessageText: "",
-		customMessageLabel: "#c084fc",
+		customMessageText: "textSoft",
+		customMessageLabel: "blue",
 		toolPendingBg: "toolPendingBg",
 		toolSuccessBg: "toolSuccessBg",
 		toolErrorBg: "toolErrorBg",
-		toolTitle: "#8a8a8a",
-		toolOutput: "#6e6e6e",
+		toolTitle: "gray",
+		toolOutput: "textSoft",
+		surfaceTitle: "accent",
+		surfaceAccent: "accent",
+		toolRunning: "yellow",
+		toolSuccess: "green",
+		toolError: "red",
 
 		mdHeading: "#fbbf24",
 		mdLink: "#93c5fd",
@@ -95,9 +104,9 @@ const dark: ThemeJson = {
 		bashMode: "green",
 	},
 	export: {
-		pageBg: "#101522",
-		cardBg: "#171f33",
-		infoBg: "#3b321d",
+		pageBg: "#111416",
+		cardBg: "#1e2529",
+		infoBg: "#252721",
 	},
 };
 

package/packages/pi-coding-agent/src/modes/interactive/tui-mode.test.ts

@@ -0,0 +1,65 @@
+// GSD2 - Tests for adaptive TUI mode selection
+
+import assert from "node:assert/strict";
+import { describe, test } from "node:test";
+import stripAnsi from "strip-ansi";
+
+import { AdaptiveLayoutComponent } from "./components/adaptive-layout.js";
+import { initTheme } from "./theme/theme.js";
+import { resolveTuiMode } from "./tui-mode.js";
+
+initTheme("dark", false);
+
+describe("resolveTuiMode", () => {
+	test("explicit overrides beat auto selection", () => {
+		assert.equal(
+			resolveTuiMode({ terminalWidth: 60, override: "debug", gsdPhase: "validating-milestone" }),
+			"debug",
+		);
+	});
+
+	test("prioritizes compact layouts on narrow terminals", () => {
+		assert.equal(
+			resolveTuiMode({ terminalWidth: 60, override: "auto", hasBlockingError: true, gsdPhase: "validating-milestone" }),
+			"compact",
+		);
+	});
+
+	test("uses debug mode for blocking errors on roomy terminals", () => {
+		assert.equal(resolveTuiMode({ terminalWidth: 100, hasBlockingError: true }), "debug");
+	});
+
+	test("uses validation mode for validation and completion phases", () => {
+		assert.equal(resolveTuiMode({ terminalWidth: 100, gsdPhase: "validating-milestone" }), "validation");
+		assert.equal(resolveTuiMode({ terminalWidth: 100, gsdPhase: "complete-milestone" }), "validation");
+	});
+
+	test("uses workflow mode when tools or non-validation phases are active", () => {
+		assert.equal(resolveTuiMode({ terminalWidth: 100, activeToolCount: 1 }), "workflow");
+		assert.equal(resolveTuiMode({ terminalWidth: 100, gsdPhase: "execute-phase" }), "workflow");
+	});
+
+	test("falls back to chat mode for plain conversation", () => {
+		assert.equal(resolveTuiMode({ terminalWidth: 100 }), "chat");
+	});
+});
+
+describe("AdaptiveLayoutComponent", () => {
+	test("renders workflow layout with prototype rule frames", () => {
+		const layout = new AdaptiveLayoutComponent(() => ({
+			override: "workflow",
+			activeToolCount: 2,
+			gsdPhase: "execute-task",
+			sessionName: "main",
+			cwd: "/Users/example/project",
+		}));
+
+		const plain = layout.render(120).map(stripAnsi);
+
+		assert.match(plain[0], /^─+/, "workflow layout should start with a rule frame");
+		assert.ok(plain.some((line) => line.includes("GSD Command Center")), "workflow title should render");
+		assert.ok(plain.some((line) => line.includes("signals")), "inspector title should render");
+		assert.ok(plain.some((line) => line.includes("│ Active")), "body rows should keep prototype gutter");
+		assert.ok(!plain.some((line) => /[╭╮╰╯]/.test(line)), "workflow layout should not use rounded box corners");
+	});
+});

package/packages/pi-coding-agent/src/modes/interactive/tui-mode.ts

@@ -0,0 +1,29 @@
+// GSD2 - Adaptive mode selection for the interactive terminal UI
+
+export type TuiMode = "chat" | "workflow" | "validation" | "debug" | "compact";
+export type TuiAdaptiveMode = "auto" | TuiMode;
+
+export interface TuiModeContext {
+	terminalWidth: number;
+	override?: TuiAdaptiveMode;
+	gsdPhase?: string;
+	activeToolCount?: number;
+	hasBlockingError?: boolean;
+}
+
+export function resolveTuiMode(context: TuiModeContext): TuiMode {
+	if (context.override && context.override !== "auto") return context.override;
+	if (context.terminalWidth < 72) return "compact";
+	if (context.hasBlockingError) return "debug";
+
+	const phase = context.gsdPhase?.toLowerCase() ?? "";
+	if (phase.includes("validat") || phase.includes("complete") || phase.includes("review")) {
+		return "validation";
+	}
+
+	if ((context.activeToolCount ?? 0) > 0 || phase.length > 0) {
+		return "workflow";
+	}
+
+	return "chat";
+}