gsd-pi 2.3.9 → 2.3.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gsd-pi",
-  "version": "2.3.9",
+  "version": "2.3.10",
   "description": "GSD — Get Shit Done coding agent",
   "license": "MIT",
   "repository": {
@@ -44,7 +44,9 @@
     "prepublishOnly": "npm run sync-pkg-version && npm run build"
   },
   "dependencies": {
+    "@clack/prompts": "^1.1.0",
     "@mariozechner/pi-coding-agent": "^0.57.1",
+    "picocolors": "^1.1.1",
     "playwright": "^1.58.2"
   },
   "devDependencies": {

package/scripts/postinstall.js

@@ -1,18 +1,38 @@
 #!/usr/bin/env node
-import { execSync } from 'child_process'
+
+import { exec as execCb } from 'child_process'
 import { createRequire } from 'module'
 import os from 'os'
-import { fileURLToPath } from 'url'
 import { dirname, resolve } from 'path'
+import { fileURLToPath } from 'url'
 
 const __dirname = dirname(fileURLToPath(import.meta.url))
 const require = createRequire(import.meta.url)
 const pkg = require(resolve(__dirname, '..', 'package.json'))
+const cwd = resolve(__dirname, '..')
+
+// ---------------------------------------------------------------------------
+// Async exec helper — captures stdout+stderr, never inherits to terminal
+// ---------------------------------------------------------------------------
+function run(cmd, options = {}) {
+  return new Promise((resolve) => {
+    execCb(cmd, { cwd, ...options }, (error, stdout, stderr) => {
+      resolve({ ok: !error, stdout, stderr, error })
+    })
+  })
+}
 
-// Colors
+// ---------------------------------------------------------------------------
+// Redirect stdout → stderr so npm always shows postinstall output.
+// npm ≥7 suppresses stdout from lifecycle scripts by default; stderr is
+// always forwarded. Clack writes to process.stdout, so we reroute it.
+// ---------------------------------------------------------------------------
+process.stdout.write = process.stderr.write.bind(process.stderr)
+
+// ---------------------------------------------------------------------------
+// ASCII banner — printed before clack UI for brand recognition
+// ---------------------------------------------------------------------------
 const cyan    = '\x1b[36m'
-const green   = '\x1b[32m'
-const yellow  = '\x1b[33m'
 const dim     = '\x1b[2m'
 const reset   = '\x1b[0m'
 
@@ -27,58 +47,81 @@ const banner =
   '   ╚═════╝ ╚══════╝╚═════╝ ' +
   reset + '\n' +
   '\n' +
-  `  Get Shit Done ${dim}v${pkg.version}${reset}\n` +
-  `  A standalone coding agent that plans, executes, and ships.\n` +
-  '\n' +
-  `  ${green}✓${reset} Installed successfully\n` +
-  `  ${dim}Run ${reset}${cyan}gsd${reset}${dim} to get started.${reset}\n`
-
-function run(command, options = {}) {
-  return execSync(command, {
-    cwd: resolve(__dirname, '..'),
-    encoding: 'utf8',
-    stdio: ['ignore', 'pipe', 'pipe'],
-    ...options,
-  })
-}
+  `  Get Shit Done ${dim}v${pkg.version}${reset}\n`
 
-function printCaptured(output) {
-  if (output) process.stderr.write(output)
-}
+// ---------------------------------------------------------------------------
+// Main — wrapped in async IIFE, with graceful fallback if clack fails
+// ---------------------------------------------------------------------------
+;(async () => {
+  process.stderr.write(banner)
 
-process.stderr.write(banner)
-
-// Apply patches to upstream dependencies (non-fatal)
-try {
-  const output = run('npx patch-package')
-  printCaptured(output)
-  process.stderr.write(`\n  ${green}✓${reset} Patches applied\n`)
-} catch (error) {
-  printCaptured(error.stdout)
-  printCaptured(error.stderr)
-  process.stderr.write(`\n  ${yellow}⚠${reset}  Failed to apply patches — run ${cyan}npx patch-package${reset} manually\n`)
-}
+  let p, pc
 
-// Install Playwright chromium for browser tools (non-fatal).
-// We intentionally avoid --with-deps here because install scripts should not
-// block on interactive sudo prompts. Playwright validates host requirements
-// after download; if Linux libs are missing, suggest the explicit follow-up.
-try {
-  const output = run('npx playwright install chromium')
-  printCaptured(output)
-  process.stderr.write(`\n  ${green}✓${reset} Browser tools ready\n\n`)
-} catch (error) {
-  const output = `${error.stdout ?? ''}${error.stderr ?? ''}`
-  printCaptured(output)
-
-  if (os.platform() === 'linux' && output.includes('Host system is missing dependencies to run browsers.')) {
-    process.stderr.write(
-      `\n  ${yellow}⚠${reset}  Browser downloaded, but Linux system dependencies are missing.\n` +
-      `  ${dim}Run ${reset}${cyan}sudo npx playwright install-deps chromium${reset}${dim} to finish setup.${reset}\n\n`
-    )
+  try {
+    p = await import('@clack/prompts')
+    pc = (await import('picocolors')).default
+  } catch {
+    // Clack or picocolors unavailable — fall back to minimal output
+    process.stderr.write(`  Run gsd to get started.\n\n`)
+    await run('npx patch-package')
+    await run('npx playwright install chromium')
+    return
+  }
+
+  // --- Branded intro -------------------------------------------------------
+  p.intro('Setup')
+
+  const results = []
+  const s = p.spinner()
+
+  // --- Step 1: Apply patches -----------------------------------------------
+  s.start('Applying patches…')
+  const patchResult = await run('npx patch-package')
+  if (patchResult.ok) {
+    s.stop('Patches applied')
+    results.push({ label: 'Patches applied', ok: true })
   } else {
-    process.stderr.write(
-      `\n  ${yellow}⚠${reset}  Browser tools unavailable — run ${cyan}npx playwright install chromium${reset} to enable\n\n`
-    )
+    s.stop(pc.yellow('Patches — skipped (non-fatal)'))
+    results.push({
+      label: 'Patches skipped — run ' + pc.cyan('npx patch-package') + ' manually',
+      ok: false,
+    })
   }
-}
+
+  // --- Step 2: Playwright browser ------------------------------------------
+  // Avoid --with-deps: install scripts should not block on interactive sudo
+  // prompts. If Linux libs are missing, suggest the explicit follow-up.
+  s.start('Setting up browser tools…')
+  const pwResult = await run('npx playwright install chromium')
+  if (pwResult.ok) {
+    s.stop('Browser tools ready')
+    results.push({ label: 'Browser tools ready', ok: true })
+  } else {
+    const output = `${pwResult.stdout ?? ''}${pwResult.stderr ?? ''}`
+    if (os.platform() === 'linux' && output.includes('Host system is missing dependencies to run browsers.')) {
+      s.stop(pc.yellow('Browser downloaded, missing Linux deps'))
+      results.push({
+        label: 'Run ' + pc.cyan('sudo npx playwright install-deps chromium') + ' to finish setup',
+        ok: false,
+      })
+    } else {
+      s.stop(pc.yellow('Browser tools — skipped (non-fatal)'))
+      results.push({
+        label: 'Browser tools unavailable — run ' + pc.cyan('npx playwright install chromium'),
+        ok: false,
+      })
+    }
+  }
+
+  // --- Summary note --------------------------------------------------------
+  const lines = results.map(
+    (r) => (r.ok ? pc.green('✓') : pc.yellow('⚠')) + ' ' + r.label
+  )
+  lines.push('')
+  lines.push('Run ' + pc.cyan('gsd') + ' to get started.')
+
+  p.note(lines.join('\n'), 'Installed')
+
+  // --- Outro ---------------------------------------------------------------
+  p.outro(pc.green('Done!'))
+})()

package/src/resources/extensions/bg-shell/index.ts

@@ -48,6 +48,7 @@ import { createConnection } from "node:net";
 import { randomUUID } from "node:crypto";
 import { writeFileSync, readFileSync, existsSync, mkdirSync } from "node:fs";
 import { join } from "node:path";
+import { shortcutDesc } from "../shared/terminal.js";
 import { createRequire } from "node:module";
 
 // ── Windows VT Input Restoration ────────────────────────────────────────────
@@ -2356,7 +2357,7 @@ export default function (pi: ExtensionAPI) {
 	// ── Ctrl+Alt+B shortcut ──────────────────────────────────────────────
 
 	pi.registerShortcut(Key.ctrlAlt("b"), {
-		description: "Open background process manager",
+		description: shortcutDesc("Open background process manager", "/bg"),
 		handler: async (ctx) => {
 			latestCtx = ctx;
 			await ctx.ui.custom<void>(

package/src/resources/extensions/gsd/index.ts

@@ -47,6 +47,7 @@ import {
 import { Key } from "@mariozechner/pi-tui";
 import { join } from "node:path";
 import { existsSync } from "node:fs";
+import { shortcutDesc } from "../shared/terminal.js";
 import { Text } from "@mariozechner/pi-tui";
 
 // ── ASCII logo ────────────────────────────────────────────────────────────
@@ -184,10 +185,8 @@ export default function (pi: ExtensionAPI) {
   });
 
   // ── Ctrl+Alt+G shortcut — GSD dashboard overlay ────────────────────────
-  // Requires Kitty keyboard protocol or modifyOtherKeys support.
-  // Terminals without support (macOS Terminal.app, JetBrains): use /gsd status instead.
   pi.registerShortcut(Key.ctrlAlt("g"), {
-    description: "Open GSD dashboard (or use /gsd status)",
+    description: shortcutDesc("Open GSD dashboard", "/gsd status"),
     handler: async (ctx) => {
       // Only show if .gsd/ exists
       if (!existsSync(join(process.cwd(), ".gsd"))) {

package/src/resources/extensions/shared/terminal.ts

@@ -0,0 +1,23 @@
+/**
+ * Terminal capability detection for keyboard shortcut support.
+ *
+ * Ctrl+Alt shortcuts require the Kitty keyboard protocol or modifyOtherKeys.
+ * Terminals that lack this support silently swallow the key combos.
+ */
+
+const UNSUPPORTED_TERMS = ["apple_terminal"];
+
+export function supportsCtrlAltShortcuts(): boolean {
+  const term = (process.env.TERM_PROGRAM || "").toLowerCase();
+  const jetbrains = (process.env.TERMINAL_EMULATOR || "").toLowerCase().includes("jetbrains");
+  return !UNSUPPORTED_TERMS.some((t) => term.includes(t)) && !jetbrains;
+}
+
+/**
+ * Returns a shortcut description that includes a slash-command fallback hint
+ * when the current terminal likely can't fire Ctrl+Alt combos.
+ */
+export function shortcutDesc(base: string, fallbackCmd: string): string {
+  if (supportsCtrlAltShortcuts()) return base;
+  return `${base} — shortcut may not work in this terminal, use ${fallbackCmd}`;
+}

package/src/resources/extensions/voice/index.ts

@@ -1,4 +1,5 @@
 import type { ExtensionAPI, ExtensionContext } from "@mariozechner/pi-coding-agent";
+import { shortcutDesc } from "../shared/terminal.js";
 import type { AssistantMessage } from "@mariozechner/pi-ai";
 import { isKeyRelease, Key, matchesKey, truncateToWidth, visibleWidth } from "@mariozechner/pi-tui";
 import { spawn, execSync, type ChildProcess } from "node:child_process";
@@ -131,7 +132,7 @@ export default function (pi: ExtensionAPI) {
 	});
 
 	pi.registerShortcut("ctrl+alt+v", {
-		description: "Toggle voice mode",
+		description: shortcutDesc("Toggle voice mode", "/voice"),
 		handler: async (ctx) => toggleVoice(ctx),
 	});