feat(notification): add showHostNotification function for user alerts

feat(tools): remove terminal tools and related imports

Author: AndyLiner13

src/client-pipe.ts

@@ -8,14 +8,7 @@
  * Client Pipe Client
  *
  * Connects to the Client extension's pipe server (Extension Development Host)
- * to interact with terminal, output channel, and VS Code command APIs.
- *
- * Terminal methods (single-terminal model):
- * - terminal.run: Run a command, wait for completion/prompt/timeout
- * - terminal.input: Send input to a waiting prompt
- * - terminal.state: Check current terminal state
- * - terminal.kill: Send Ctrl+C to stop the running process
- * - terminal.listAll: List all terminals (tracked + untracked)
+ * to interact with output channel, and VS Code command APIs.
  *
  * Output methods:
  * - output.listChannels: List VS Code output channels
@@ -36,8 +29,6 @@ const CLIENT_PIPE_PATH = IS_WINDOWS
   : '/tmp/vscode-devtools-client.sock';
 
 const DEFAULT_TIMEOUT_MS = 10_000;
-// Terminal operations wait up to 35s so the 30s command timeout finishes first
-const TERMINAL_TIMEOUT_MS = 35_000;
 
 // ── Types ────────────────────────────────────────────────
 
@@ -67,25 +58,6 @@ function isJsonRpcResponse(value: unknown): value is JsonRpcResponse {
   );
 }
 
-export type TerminalStatus =
-  | 'idle'
-  | 'running'
-  | 'completed'
-  | 'waiting_for_input'
-  | 'timeout';
-
-export type WaitMode = 'completion' | 'background';
-
-export interface ActiveProcess {
-  terminalName: string;
-  pid?: number;
-  command: string;
-  status: TerminalStatus;
-  startedAt: string;
-  durationMs: number;
-  exitCode?: number;
-}
-
 export type ProcessStatus = 'running' | 'completed' | 'killed' | 'orphaned';
 
 export interface ChildProcessInfo {
@@ -115,20 +87,6 @@ export interface ProcessLedgerSummary {
   sessionId: string;
 }
 
-export interface TerminalRunResult {
-  status: TerminalStatus;
-  output: string;
-  shell?: string;
-  cwd?: string;
-  exitCode?: number;
-  prompt?: string;
-  pid?: number;
-  name?: string;
-  durationMs?: number;
-  activeProcesses?: ActiveProcess[];
-  terminalSessions?: TerminalSessionInfo[];
-}
-
 export interface TerminalSessionInfo {
   name: string;
   shell?: string;
@@ -254,81 +212,6 @@ function sendClientRequest(
   });
 }
 
-// ── Terminal Methods (Multi-Terminal Model) ─────────────
-
-/**
- * Run a PowerShell command in a named terminal.
- * Creates terminal if needed, rejects with state if busy.
- * Waits for completion, prompt detection, or timeout.
- *
- * @param command The PowerShell command to execute
- * @param cwd Absolute path to working directory for command execution
- * @param timeout Max wait time in milliseconds (default: 120000)
- * @param name Terminal name (default: 'default')
- * @param waitMode 'completion' blocks until done; 'background' returns immediately
- */
-export async function terminalRun(
-  command: string,
-  cwd: string,
-  timeout?: number,
-  name?: string,
-  waitMode?: WaitMode,
-): Promise<TerminalRunResult> {
-  const result = await sendClientRequest(
-    'terminal.run',
-    {command, cwd, timeout, name, waitMode},
-    TERMINAL_TIMEOUT_MS,
-  );
-  assertResult<TerminalRunResult>(result, 'terminal.run');
-  return result;
-}
-
-/**
- * Send input to a terminal waiting for a prompt.
- * Waits for the next completion or prompt after sending.
- *
- * @param text The text to send
- * @param addNewline Whether to press Enter after (default: true)
- * @param timeout Max wait time in milliseconds (default: 30000)
- * @param name Terminal name (default: 'default')
- */
-export async function terminalInput(
-  text: string,
-  addNewline?: boolean,
-  timeout?: number,
-  name?: string,
-): Promise<TerminalRunResult> {
-  const result = await sendClientRequest(
-    'terminal.input',
-    {text, addNewline, timeout, name},
-    TERMINAL_TIMEOUT_MS,
-  );
-  assertResult<TerminalRunResult>(result, 'terminal.input');
-  return result;
-}
-
-/**
- * Get the current terminal state without modifying anything.
- *
- * @param name Terminal name (default: 'default')
- */
-export async function terminalGetState(name?: string): Promise<TerminalRunResult> {
-  const result = await sendClientRequest('terminal.state', {name});
-  assertResult<TerminalRunResult>(result, 'terminal.state');
-  return result;
-}
-
-/**
- * Send Ctrl+C to kill the running process in a terminal.
- *
- * @param name Terminal name (default: 'default')
- */
-export async function terminalKill(name?: string): Promise<TerminalRunResult> {
-  const result = await sendClientRequest('terminal.kill', {name});
-  assertResult<TerminalRunResult>(result, 'terminal.kill');
-  return result;
-}
-
 // ── Output Methods ───────────────────────────────────────
 
 /**

src/host-pipe.ts

@@ -239,6 +239,18 @@ export async function restartMcpServer(): Promise<void> {
   }
 }
 
+/**
+ * Send a notification message to the Host extension for display
+ * in the VS Code window. Fire-and-forget — failures are silently ignored.
+ */
+export async function showHostNotification(message: string): Promise<void> {
+  try {
+    await sendHostRequest('showNotification', {message}, 5_000);
+  } catch {
+    // Non-critical — notification may not be shown
+  }
+}
+
 /**
  * Check if the Host pipe is reachable via a system.ping.
  */

src/main.ts

@@ -12,7 +12,7 @@ import process from 'node:process';
 import {parseArguments} from './cli.js';
 import {loadConfig, type ResolvedConfig} from './config.js';
 import {hasBuildChangedSinceWindowStart, isBuildStale} from './extension-watcher.js';
-import {restartMcpServer} from './host-pipe.js';
+import {restartMcpServer, showHostNotification} from './host-pipe.js';
 import {loadIssueDescriptions} from './issue-descriptions.js';
 import {logger, saveLogsToFile} from './logger.js';
 import {McpResponse} from './McpResponse.js';
@@ -54,6 +54,10 @@ const mcpProcessStartTime = Date.now();
 let mcpServerHotReloadInfo: {builtAt: number} | null = null;
 let mcpServerRestartScheduled = false;
 
+// ── Extension Hot-Reload State ──────────────────────────
+
+let extensionHotReloadInfo: {builtAt: number} | null = null;
+
 /**
  * Run an incremental build for hot-reload: `tsc` + `post-build` WITHOUT
  * `rmSync('build')`. The full `pnpm run build` cleans the build dir first,
@@ -384,6 +388,7 @@ function registerTool(tool: ToolDefinition): void {
             // Check 1: Source files newer than build output → needs rebuild.
             if (!mcpServerRestartScheduled && hasMcpServerSourceChanged(mcpServerDir)) {
               logger(`[tool:${tool.name}] MCP server source changed — triggering self rebuild…`);
+              showHostNotification('🔨 MCP Server: Rebuilding…').catch(() => {});
 
               try {
                 await runMcpServerBuild();
@@ -482,6 +487,7 @@ function registerTool(tool: ToolDefinition): void {
             const reason = stale ? 'source stale' : 'manual build detected';
             logger(`[tool:${tool.name}] Extension needs hot-reload (${reason}) — reloading…`);
             await lifecycleService.handleHotReload();
+            extensionHotReloadInfo = {builtAt: Date.now()};
             logger(`[tool:${tool.name}] Hot-reload complete — reconnected`);
           }
         }
@@ -580,6 +586,16 @@ function registerTool(tool: ToolDefinition): void {
           mcpServerHotReloadInfo = null;
         }
 
+        // Inject extension hot-reload banner on first tool call after extension reload
+        if (extensionHotReloadInfo) {
+          const secondsAgo = Math.round((Date.now() - extensionHotReloadInfo.builtAt) / 1000);
+          result.content.unshift({
+            type: 'text',
+            text: `✅ **Extension was recently updated.** Latest build completed ${secondsAgo}s ago. The Extension Development Host is now running the newest code.`,
+          });
+          extensionHotReloadInfo = null;
+        }
+
         return result;
       } catch (err) {
         logger(`[tool:${tool.name}] ERROR: ${err && 'message' in err ? err.message : String(err)}`);

src/tools/reload-mcp-server.ts

@@ -6,7 +6,7 @@
 
 import process from 'node:process';
 
-import {restartMcpServer} from '../host-pipe.js';
+import {restartMcpServer, showHostNotification} from '../host-pipe.js';
 import {logger} from '../logger.js';
 import {
   getMcpServerRoot,
@@ -64,6 +64,7 @@ Examples:
     logger(`[reload_mcp_server] ${status}`);
 
     writeHotReloadMarker(mcpServerDir);
+    showHostNotification('🔄 MCP Server: Restarting…').catch(() => {});
 
     // Schedule restart after this response is flushed via stdio
     setTimeout(async () => {