gsd-pi 2.81.0 → 2.82.0-dev.20138ae42

package/dist/resources/extensions/gsd/verification-gate.js

@@ -3,7 +3,7 @@
 // Discovery order (D003): preference → task plan verify → package.json scripts.
 // First non-empty source wins.
 import { spawnSync } from "node:child_process";
-import { existsSync, readFileSync } from "node:fs";
+import { existsSync, readFileSync, readdirSync } from "node:fs";
 import { join, basename } from "node:path";
 import { DEFAULT_COMMAND_TIMEOUT_MS } from "./constants.js";
 import { rewriteCommandWithRtk } from "../shared/rtk.js";
@@ -27,7 +27,8 @@ const PACKAGE_SCRIPT_KEYS = ["typecheck", "lint", "test"];
  *   1. Explicit preference commands
  *   2. Task plan verify field (split on &&)
  *   3. package.json scripts (typecheck, lint, test)
- *   4. None found
+ *   4. Python pytest project markers
+ *   5. None found
  */
 export function discoverCommands(options) {
     // 1. Preference commands
@@ -72,9 +73,57 @@ export function discoverCommands(options) {
             // Malformed package.json — fall through to "none"
         }
     }
-    // 4. Nothing found
+    const pythonCommand = discoverPythonPytestCommand(options.cwd);
+    if (pythonCommand) {
+        return { commands: [pythonCommand], source: "python-project" };
+    }
+    // 5. Nothing found
     return { commands: [], source: "none" };
 }
+function discoverPythonPytestCommand(cwd) {
+    const hasPythonTestFiles = hasPythonTests(join(cwd, "tests"));
+    const hasPytestConfig = existsSync(join(cwd, "pytest.ini"));
+    const pyprojectPath = join(cwd, "pyproject.toml");
+    const hasPyproject = existsSync(pyprojectPath);
+    if (!hasPythonTestFiles && !hasPytestConfig && !hasPyproject) {
+        return null;
+    }
+    if (hasPytestConfig || hasPythonTestFiles) {
+        return "python3 -m pytest";
+    }
+    try {
+        const pyproject = readFileSync(pyprojectPath, "utf-8");
+        if (pyproject.includes("[tool.pytest]") ||
+            pyproject.includes("[tool.pytest.") ||
+            pyproject.includes("[pytest]") ||
+            pyproject.includes("[tool:pytest]")) {
+            return "python3 -m pytest";
+        }
+    }
+    catch {
+        // Ignore unreadable pyproject.toml and fall through.
+    }
+    return null;
+}
+function hasPythonTests(dir) {
+    let entries;
+    try {
+        entries = readdirSync(dir, { withFileTypes: true });
+    }
+    catch {
+        return false;
+    }
+    for (const entry of entries) {
+        const path = join(dir, entry.name);
+        if (entry.isDirectory() && hasPythonTests(path)) {
+            return true;
+        }
+        if (entry.isFile() && /^test_.*\.py$|^.*_test\.py$/.test(entry.name)) {
+            return true;
+        }
+    }
+    return false;
+}
 // ─── Failure Context Formatting ──────────────────────────────────────────────
 /** Maximum chars of stderr to include per failed check in failure context. */
 const MAX_STDERR_PER_CHECK = 2_000;
@@ -112,7 +161,7 @@ export function formatFailureContext(result) {
 }
 // ─── Gate Execution ─────────────────────────────────────────────────────────
 /** Characters that indicate shell injection when found in a command string. */
-const SHELL_INJECTION_PATTERN = /[;|`]|\$\(/;
+const SHELL_INJECTION_PATTERN = /[;|`<>]|\$\(/;
 /**
  * Known executable first-tokens that are safe to run.
  * Lowercase commands, common build/test tools, and npm/yarn/pnpm invocations.
@@ -148,6 +197,7 @@ const KNOWN_COMMAND_PREFIXES = new Set([
  * Heuristics (any true → prose-like):
  *   1. First token starts with an uppercase letter and the string has 4+ words
  *   2. String contains commas followed by spaces (prose clause structure)
+ *   3. First token has no ASCII letters or digits and the string has 4+ words
  */
 export function isLikelyCommand(cmd) {
     const trimmed = cmd.trim();
@@ -173,16 +223,27 @@ export function isLikelyCommand(cmd) {
     // First token has uppercase letters and no path separators → prose
     if (/[A-Z]/.test(firstToken) && !firstToken.includes("/"))
         return false;
+    // Non-ASCII prose with multiple words should not be executed as a command.
+    if (!/[A-Za-z0-9]/.test(firstToken) && tokens.length >= 4)
+        return false;
     return true;
 }
 /**
  * Validate a command string for obvious shell injection patterns.
  * Returns the command unchanged if safe, or null if suspicious.
  */
+export function validateVerificationCommand(cmd) {
+    if (SHELL_INJECTION_PATTERN.test(cmd)) {
+        return { ok: false, reason: "contains shell control syntax such as pipes, redirects, semicolons, backticks, or command substitution" };
+    }
+    if (!isLikelyCommand(cmd)) {
+        return { ok: false, reason: "does not look like a runnable command" };
+    }
+    return { ok: true };
+}
 function sanitizeCommand(cmd) {
-    if (SHELL_INJECTION_PATTERN.test(cmd))
-        return null;
-    if (!isLikelyCommand(cmd))
+    const validation = validateVerificationCommand(cmd);
+    if (!validation.ok)
         return null;
     return cmd;
 }

package/dist/resources/extensions/gsd/verification-verdict.js

@@ -0,0 +1,26 @@
+// Project/App: GSD-2
+// File Purpose: Host-owned verification verdict policy for auto-mode units.
+export function decideVerificationVerdict(unitType, result) {
+    if (unitType === "execute-task" && result.discoverySource === "none" && result.checks.length === 0) {
+        return {
+            passed: false,
+            reason: "no-host-checks",
+            retryable: false,
+            failureContext: "No runnable host-owned verification command was discovered. Add project verification_commands or a runnable task-plan Verify command before completing this execute-task.",
+        };
+    }
+    if (!result.passed) {
+        return {
+            passed: false,
+            reason: "checks-failed",
+            retryable: true,
+            failureContext: "",
+        };
+    }
+    return {
+        passed: true,
+        reason: "passed",
+        retryable: false,
+        failureContext: "",
+    };
+}

package/dist/resources/extensions/gsd/watch/header-renderer.js

@@ -1,5 +1,5 @@
-// GSD Watch — Header renderer: ASCII logo, session info, MCP status, remote questions
-// Copyright (c) 2026 Jeremy McSpadden <jeremy@fluxlabs.net>
+// Project/App: GSD-2
+// File Purpose: Watch-mode terminal header and splash renderer for GSD project status.
 import { execFileSync } from "node:child_process";
 import { existsSync, readFileSync } from "node:fs";
 import { homedir } from "node:os";
@@ -7,25 +7,57 @@ import { join } from "node:path";
 import { visibleWidth, truncateToWidth } from "@gsd/pi-tui";
 import { loadEffectiveGSDPreferences } from "../preferences.js";
 import { gsdHome } from "../gsd-home.js";
+import { splashPalette } from "./splash-palette.js";
 // ─── Constants ────────────────────────────────────────────────────────────────
-/**
- * GSD ASCII logo — inlined here because the canonical src/logo.ts is outside
- * the resources rootDir and cannot be imported directly.
- */
 const GSD_LOGO = [
-    '   ██████╗ ███████╗██████╗ ',
-    '  ██╔════╝ ██╔════╝██╔══██╗',
-    '  ██║  ███╗███████╗██║  ██║',
-    '  ██║   ██║╚════██║██║  ██║',
-    '  ╚██████╔╝███████║██████╔╝',
-    '   ╚═════╝ ╚══════╝╚═════╝ ',
+    "   ██████╗ ███████╗██████╗ ",
+    "  ██╔════╝ ██╔════╝██╔══██╗",
+    "  ██║  ███╗███████╗██║  ██║",
+    "  ██║   ██║╚════██║██║  ██║",
+    "  ╚██████╔╝███████║██████╔╝",
+    "   ╚═════╝ ╚══════╝╚═════╝ ",
 ];
-/** Separator character for the horizontal divider line. */
-const SEPARATOR_CHAR = "─";
-/** Vertical bar between logo and info panel. */
-const PANEL_DIVIDER = "│";
 /** Label column width for Model/Provider/Directory/Branch rows. */
 const LABEL_COL_WIDTH = 10;
+const ANSI_RESET = "\x1b[0m";
+const ANSI_BOLD = "\x1b[1m";
+function rgb(hex) {
+    const cleaned = hex.replace("#", "");
+    const r = Number.parseInt(cleaned.slice(0, 2), 16);
+    const g = Number.parseInt(cleaned.slice(2, 4), 16);
+    const b = Number.parseInt(cleaned.slice(4, 6), 16);
+    return `\x1b[38;2;${r};${g};${b}m`;
+}
+const colors = {
+    accent: rgb(splashPalette.accent),
+    border: rgb(splashPalette.border),
+    muted: rgb(splashPalette.muted),
+    dim: rgb(splashPalette.dim),
+    text: rgb(splashPalette.text),
+    success: rgb(splashPalette.success),
+};
+function color(text, colorCode) {
+    return `${colorCode}${text}${ANSI_RESET}`;
+}
+function bold(text) {
+    return `${ANSI_BOLD}${text}${ANSI_RESET}`;
+}
+function padVisible(text, width) {
+    const clipped = truncateToWidth(text, Math.max(0, width), "");
+    return clipped + " ".repeat(Math.max(0, width - visibleWidth(clipped)));
+}
+function rightAlign(left, right, width) {
+    if (!right)
+        return truncateToWidth(left, width, "");
+    const gap = Math.max(1, width - visibleWidth(left) - visibleWidth(right));
+    return truncateToWidth(left + " ".repeat(gap) + right, width, "");
+}
+function frameLine(content, width) {
+    if (width < 3)
+        return truncateToWidth(content, Math.max(0, width), "");
+    const innerWidth = Math.max(0, width - 2);
+    return `${color("│", colors.border)}${padVisible(content, innerWidth)}${color("│", colors.border)}`;
+}
 // ─── Data Readers ─────────────────────────────────────────────────────────────
 /**
  * Read the configured execution model from GSD preferences.
@@ -147,12 +179,11 @@ export function gatherHeaderData(projectRoot) {
  * Returns empty string if value is empty.
  */
 function formatInfoLine(label, value, availableWidth) {
-    const bold = `\x1b[1m${label}\x1b[0m`;
-    const labelVis = visibleWidth(bold);
-    const padding = " ".repeat(Math.max(1, LABEL_COL_WIDTH - labelVis));
+    const labelText = bold(color(label, colors.text));
+    const padding = " ".repeat(Math.max(1, LABEL_COL_WIDTH - label.length));
     const maxValueWidth = Math.max(1, availableWidth - LABEL_COL_WIDTH);
     const truncValue = truncateToWidth(value, maxValueWidth, "…");
-    return bold + padding + truncValue;
+    return labelText + padding + color(truncValue, colors.muted);
 }
 /**
  * Format MCP server names as a dot-separated row with checkmarks.
@@ -164,7 +195,7 @@ export function formatMcpRow(servers, width) {
     // Capitalize first letter of each server name
     const items = servers.map(s => {
         const cap = s.charAt(0).toUpperCase() + s.slice(1);
-        return `${cap} ✓`;
+        return `${color(cap, colors.accent)} ${color("✓", colors.success)}`;
     });
     const full = items.join("  ·  ");
     if (visibleWidth(full) <= width)
@@ -175,70 +206,62 @@ export function formatMcpRow(servers, width) {
 /**
  * Render the full header as an array of terminal-safe strings.
  *
- * Layout: GSD ASCII logo on the left, info panel on the right separated by │.
- * Below: MCP server row, remote questions row, separator line.
+ * Layout: compact GSD mark on the left with a command-center status panel on
+ * the right. This keeps the splash visual while making the actionable command
+ * and workspace state easier to scan.
  */
 export function renderHeaderLines(data, width) {
-    const lines = [];
-    // Logo is 6 lines tall. Info panel has: title + blank + model + provider + directory + branch = 6 lines
+    if (width < 40)
+        return renderStackedHeader(data, width);
+    const outerWidth = width;
+    const innerWidth = Math.max(0, outerWidth - 2);
     const logoLines = GSD_LOGO;
-    const logoWidth = Math.max(...logoLines.map(l => visibleWidth(l)));
-    // Calculate available width for the info panel
-    // Layout: logo + " " + "│" + " " = logoWidth + 3
-    const dividerOverhead = 3; // " │ "
-    const infoPanelWidth = width - logoWidth - dividerOverhead;
-    // If terminal is too narrow for side-by-side, fall back to stacked layout
-    if (infoPanelWidth < 20) {
+    const logoWidth = Math.max(...logoLines.map((line) => visibleWidth(line)));
+    const gap = ` ${color("│", colors.dim)} `;
+    const panelWidth = innerWidth - logoWidth - visibleWidth(gap);
+    if (panelWidth < 44) {
         return renderStackedHeader(data, width);
     }
-    // Build info panel lines (6 lines to match logo height)
-    const infoLines = [
-        `\x1b[1mGet Shit Done\x1b[0m`,
-        "",
-        formatInfoLine("Model", data.model, infoPanelWidth),
-        formatInfoLine("Provider", data.provider, infoPanelWidth),
-        formatInfoLine("Directory", data.directory, infoPanelWidth),
-        formatInfoLine("Branch", data.branch, infoPanelWidth),
+    const mcpRow = formatMcpRow(data.mcpServers, Math.max(1, panelWidth - 9)) || color("none configured", colors.dim);
+    const panelLines = [
+        rightAlign(`${color("GSD", colors.accent)} ${bold(color("Project Console", colors.text))}`, color("idle", colors.muted), panelWidth),
+        rightAlign(`${color("Project", colors.muted)} ${color(data.directory, colors.text)}`, `${color("Command", colors.muted)} ${color("/gsd start", colors.accent)}`, panelWidth),
+        rightAlign(`${color("Branch", colors.muted)} ${color(data.branch, colors.text)}`, `${color("Mode", colors.muted)} ${color("manual", colors.text)}`, panelWidth),
+        rightAlign(`${color("MCP", colors.muted)} ${mcpRow}`, `${color("Model", colors.muted)} ${color(data.model, colors.text)}`, panelWidth),
+        rightAlign(`${color("Provider", colors.muted)} ${color(data.provider, colors.text)}`, `${color("Workspace", colors.muted)} ${color(data.directory, colors.text)}`, panelWidth),
+        rightAlign(color("/gsd to begin", colors.accent), `${color("/gsd templates", colors.muted)}  ${color("/gsd help", colors.muted)}`, panelWidth),
     ];
-    // Merge logo and info panel side by side
-    const maxLines = Math.max(logoLines.length, infoLines.length);
-    for (let i = 0; i < maxLines; i++) {
-        const logoLine = i < logoLines.length ? logoLines[i] : "";
-        const infoLine = i < infoLines.length ? infoLines[i] : "";
-        // Pad logo line to consistent width
-        const logoPad = " ".repeat(Math.max(0, logoWidth - visibleWidth(logoLine)));
-        lines.push(`${logoLine}${logoPad} ${PANEL_DIVIDER} ${infoLine}`);
-    }
-    // Blank line after logo+info block
-    lines.push("");
-    // MCP server row
-    const mcpRow = formatMcpRow(data.mcpServers, width);
-    if (mcpRow) {
-        lines.push(` ${mcpRow}`);
+    const lines = [
+        color("╭" + "─".repeat(Math.max(0, outerWidth - 2)) + "╮", colors.border),
+    ];
+    for (let i = 0; i < logoLines.length; i++) {
+        const logo = padVisible(color(logoLines[i], colors.border), logoWidth);
+        lines.push(frameLine(`${logo}${gap}${panelLines[i] ?? ""}`, outerWidth));
     }
-    // Separator line
-    lines.push(SEPARATOR_CHAR.repeat(width));
+    lines.push(color("╰" + "─".repeat(Math.max(0, outerWidth - 2)) + "╯", colors.border));
     return lines;
 }
 /**
  * Fallback stacked layout for narrow terminals (< 20 cols for info panel).
  */
 function renderStackedHeader(data, width) {
-    const lines = [];
+    const outerWidth = Math.max(0, width);
+    if (outerWidth < 3) {
+        return [color(truncateToWidth("GSD", outerWidth, ""), colors.accent)];
+    }
+    const lines = [color("╭" + "─".repeat(Math.max(0, outerWidth - 2)) + "╮", colors.border)];
     // Title
-    lines.push(`\x1b[1mGet Shit Done\x1b[0m`);
-    lines.push("");
+    lines.push(frameLine(`${color("GSD", colors.accent)} ${bold(color("Project Console", colors.text))}`, outerWidth));
     // Info
-    lines.push(formatInfoLine("Model", data.model, width));
-    lines.push(formatInfoLine("Provider", data.provider, width));
-    lines.push(formatInfoLine("Directory", data.directory, width));
-    lines.push(formatInfoLine("Branch", data.branch, width));
-    lines.push("");
+    lines.push(frameLine(formatInfoLine("Project", data.directory, outerWidth - 2), outerWidth));
+    lines.push(frameLine(formatInfoLine("Command", "/gsd start", outerWidth - 2), outerWidth));
+    lines.push(frameLine(formatInfoLine("Branch", data.branch, outerWidth - 2), outerWidth));
+    lines.push(frameLine(formatInfoLine("Model", data.model, outerWidth - 2), outerWidth));
     // MCP
-    const mcpRow = formatMcpRow(data.mcpServers, width);
+    const mcpRow = formatMcpRow(data.mcpServers, Math.max(1, outerWidth - 7));
     if (mcpRow)
-        lines.push(` ${mcpRow}`);
-    // Separator
-    lines.push(SEPARATOR_CHAR.repeat(width));
+        lines.push(frameLine(`${color("MCP", colors.muted)} ${mcpRow}`, outerWidth));
+    lines.push(frameLine(`${color("/gsd to begin", colors.accent)}  ${color("/gsd help", colors.muted)}`, outerWidth));
+    lines.push(color("╰" + "─".repeat(Math.max(0, outerWidth - 2)) + "╯", colors.border));
     return lines;
 }

package/dist/resources/extensions/gsd/watch/splash-palette.js

@@ -0,0 +1,10 @@
+// Project/App: GSD-2
+// File Purpose: Shared RGB palette for the pre-theme GSD watch splash.
+export const splashPalette = {
+    accent: "#8db7ff",
+    border: "#a7ba78",
+    muted: "#7d889f",
+    dim: "#4d5870",
+    text: "#dce4f2",
+    success: "#a8c978",
+};

package/dist/resources/extensions/gsd/workflow-mcp.js

@@ -266,7 +266,7 @@ export function getRequiredWorkflowToolsForGuidedUnit(unitType) {
         case "execute-task":
             return ["gsd_task_complete"];
         case "complete-slice":
-            return ["gsd_slice_complete"];
+            return ["gsd_slice_complete", "gsd_task_reopen", "gsd_replan_slice"];
         default:
             return [];
     }
@@ -294,7 +294,7 @@ export function getRequiredWorkflowToolsForAutoUnit(unitType) {
         case "reactive-execute":
             return ["gsd_task_complete"];
         case "complete-slice":
-            return ["gsd_slice_complete"];
+            return ["gsd_slice_complete", "gsd_task_reopen", "gsd_replan_slice"];
         case "replan-slice":
             return ["gsd_replan_slice"];
         case "reassess-roadmap":

package/dist/resources/extensions/gsd/workflow-projections.js

@@ -150,11 +150,11 @@ export function renderSummaryContent(taskRow, sliceId, milestoneId, evidence) {
     }
     // ── Frontmatter (YAML list format, matches parseSummary() expectations) ──
     const keyFilesYaml = taskRow.key_files && taskRow.key_files.length > 0
-        ? taskRow.key_files.map(f => `  - ${f}`).join("\n")
-        : "  - (none)";
+        ? `\n${taskRow.key_files.map(f => `  - ${f}`).join("\n")}`
+        : " []";
     const keyDecisionsYaml = taskRow.key_decisions && taskRow.key_decisions.length > 0
-        ? taskRow.key_decisions.map(d => `  - ${d}`).join("\n")
-        : "  - (none)";
+        ? `\n${taskRow.key_decisions.map(d => `  - ${d}`).join("\n")}`
+        : " []";
     // Derive verification_result from evidence if available
     const evidenceList = evidence ?? [];
     const allPassed = evidenceList.length > 0 &&
@@ -182,10 +182,8 @@ export function renderSummaryContent(taskRow, sliceId, milestoneId, evidence) {
 id: ${taskRow.id}
 parent: ${sliceId}
 milestone: ${milestoneId}
-key_files:
-${keyFilesYaml}
-key_decisions:
-${keyDecisionsYaml}
+key_files:${keyFilesYaml}
+key_decisions:${keyDecisionsYaml}
 duration: ${taskRow.duration || ""}
 verification_result: ${verificationResult}
 completed_at: ${taskRow.completed_at || ""}