gsd-pi 2.80.0-dev.c5f2443b3 → 2.80.0-dev.e146beb20

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";
+}

package/packages/pi-coding-agent/src/resources/extensions/memory/storage-safety-guard.test.ts

@@ -0,0 +1,14 @@
+import assert from "node:assert/strict";
+import { readFileSync } from "node:fs";
+import { join } from "node:path";
+import test from "node:test";
+
+test("MemoryStorage persists sql.js snapshots through the atomic DB snapshot writer", () => {
+	const src = readFileSync(
+		join(process.cwd(), "packages", "pi-coding-agent", "src", "resources", "extensions", "memory", "storage.ts"),
+		"utf-8",
+	);
+
+	assert.match(src, /atomicWriteDbSnapshotSync\(/, "storage must use atomic DB snapshot writes");
+	assert.doesNotMatch(src, /writeFileSync\(this\.dbPath/, "direct live DB overwrite is forbidden");
+});

package/packages/pi-coding-agent/src/resources/extensions/memory/storage.ts

@@ -9,8 +9,9 @@
 
 import initSqlJs, { type Database as SqlJsDatabase } from "sql.js";
 import { randomUUID } from "crypto";
-import { existsSync, mkdirSync, readFileSync, writeFileSync } from "fs";
+import { existsSync, mkdirSync, readFileSync } from "fs";
 import { dirname } from "path";
+import { atomicWriteDbSnapshotSync } from "../../../core/db-snapshot.js";
 
 export interface ThreadRow {
 	thread_id: string;
@@ -74,7 +75,7 @@ export class MemoryStorage {
 
 	private persist(): void {
 		const data = this.db.export();
-		writeFileSync(this.dbPath, Buffer.from(data));
+		atomicWriteDbSnapshotSync(this.dbPath, data);
 	}
 
 	private schedulePersist(): void {