termify-agent 1.0.29 → 1.0.30

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "termify-agent",
-  "version": "1.0.29",
+  "version": "1.0.30",
   "description": "Termify Agent CLI - Connect your local terminal to Termify",
   "type": "module",
   "main": "dist/index.js",
@@ -33,13 +33,15 @@
     "chalk": "^5.3.0",
     "commander": "^12.0.0",
     "conf": "^12.0.0",
-    "node-pty": "^1.1.0",
     "open": "^10.0.0",
     "ora": "^8.0.1",
     "simple-git": "^3.30.0",
-    "ssh2": "^1.15.0",
     "ws": "^8.16.0"
   },
+  "optionalDependencies": {
+    "node-pty": "^1.1.0",
+    "ssh2": "^1.15.0"
+  },
   "devDependencies": {
     "@types/node": "^20.11.0",
     "@types/ssh2": "^1.15.0",

package/scripts/postinstall.js

@@ -31,30 +31,29 @@ const STATS_AGENT_PATH = join(TERMIFY_DIR, `stats-agent${EXE}`);
 const MCP_DIR = join(TERMIFY_DIR, 'termify-mcp');
 const MCP_BUNDLE_PATH = join(MCP_DIR, 'index.mjs');
 
+// Marker: skip node-pty test on repeated installs if it already works for this ABI
+const NODE_PTY_OK_MARKER = join(TERMIFY_DIR, `.node-pty-ok-${process.versions.modules}`);
+
 /**
- * Test if node-pty actually works by spawning a shell
+ * Test if node-pty actually works by spawning a shell in a child process.
+ * Uses child process to isolate potential SIGSEGV from bad prebuilds.
  */
 function testNodePty() {
+  // Fast path: marker from previous successful install
+  if (existsSync(NODE_PTY_OK_MARKER)) return true;
+
   try {
-    const testScript = `
-      const pty = require('node-pty');
-      const term = pty.spawn(process.platform === 'win32' ? 'cmd.exe' : '/bin/sh', [], {
-        name: 'xterm-256color',
-        cols: 80,
-        rows: 24,
-        cwd: process.env.HOME || process.env.USERPROFILE || '/',
-        env: process.env
-      });
-      term.kill();
-      process.exit(0);
-    `;
-    execSync(`node -e "${testScript.replace(/"/g, '\\"').replace(/\n/g, ' ')}"`, {
+    execSync('node -e "require(\'node-pty\').spawn(\'/bin/sh\',[],{cols:80,rows:24}).kill()"', {
       stdio: 'pipe',
-      timeout: 10000,
+      timeout: 5000,
       cwd: process.cwd(),
     });
+    // Cache result for next install
+    mkdirSync(TERMIFY_DIR, { recursive: true });
+    try { writeFileSync(NODE_PTY_OK_MARKER, Date.now().toString()); } catch {}
     return true;
-  } catch (err) {
+  } catch {
+    try { unlinkSync(NODE_PTY_OK_MARKER); } catch {}
     return false;
   }
 }
@@ -77,21 +76,20 @@ function resolveNodePtyDir() {
  * For Node.js v24+, prebuilds may not be compatible, so we compile from source
  */
 function rebuildNodePty() {
-  console.log('[termify-agent] Setting up node-pty for your platform...');
-  console.log(`[termify-agent] Node.js version: ${process.version}`);
+  // node-pty is an optional dependency — skip entirely if not installed
+  const nodePtyDir = resolveNodePtyDir();
+  if (!nodePtyDir) {
+    console.log('[termify-agent] node-pty not installed (optional), skipping PTY setup.');
+    return false;
+  }
 
-  // First, check if node-pty already works
+  // Fast path: already works (cached or live test)
   if (testNodePty()) {
-    console.log('[termify-agent] node-pty is already working.');
+    console.log('[termify-agent] node-pty is working.');
     return true;
   }
 
-  const nodePtyDir = resolveNodePtyDir();
-  if (!nodePtyDir) {
-    console.error('[termify-agent] CRITICAL: node-pty directory not found.');
-    console.error('[termify-agent] Manual fix: npm install node-pty --build-from-source');
-    return false;
-  }
+  console.log(`[termify-agent] Node.js version: ${process.version}`);
 
   console.log(`[termify-agent] node-pty resolved at: ${nodePtyDir}`);
   console.log('[termify-agent] node-pty prebuilt not working, attempting source rebuild...');
@@ -787,57 +785,95 @@ function ensureCodexInstructions() {
   }
 }
 
+// ---------------------------------------------------------------------------
+// Pretty CLI output (no external deps — runs during postinstall)
+// ---------------------------------------------------------------------------
+
+const CYAN = '\x1b[36m';
+const GREEN = '\x1b[32m';
+const YELLOW = '\x1b[33m';
+const DIM = '\x1b[2m';
+const BOLD = '\x1b[1m';
+const RESET = '\x1b[0m';
+const CHECK = `${GREEN}✔${RESET}`;
+const WARN = `${YELLOW}⚠${RESET}`;
+const SPINNER_FRAMES = ['⠋', '⠙', '⠹', '⠸', '⠼', '⠴', '⠦', '⠧', '⠇', '⠏'];
+
+function createSpinner(text) {
+  let i = 0;
+  const id = setInterval(() => {
+    process.stdout.write(`\r  ${CYAN}${SPINNER_FRAMES[i++ % SPINNER_FRAMES.length]}${RESET} ${text}  `);
+  }, 80);
+  return {
+    succeed(msg) { clearInterval(id); process.stdout.write(`\r  ${CHECK} ${msg || text}  \n`); },
+    warn(msg) { clearInterval(id); process.stdout.write(`\r  ${WARN} ${msg || text}  \n`); },
+    update(t) { text = t; },
+  };
+}
+
+// Suppress verbose internal logs — only spinner UI shows
+const _log = console.log;
+const _warn = console.warn;
+const _err = console.error;
+function muteConsole() { console.log = () => {}; console.warn = () => {}; console.error = () => {}; }
+function unmuteConsole() { console.log = _log; console.warn = _warn; console.error = _err; }
+
 /**
  * Main
  */
 async function main() {
-  console.log('[termify-agent] Running postinstall...');
-  console.log(`[termify-agent] Platform: ${platform()}-${arch()}`);
-
-  // Step 1: Rebuild node-pty (critical for terminal functionality)
-  const nodePtyOk = rebuildNodePty();
-  if (!nodePtyOk) {
-    console.error('[termify-agent] WARNING: Terminal features may not work until node-pty is fixed.');
-  }
-
-  // Step 2: Install termify-daemon (efficient process monitoring)
-  try {
-    await installDaemon();
-  } catch (err) {
-    console.warn(`[termify-agent] Warning: ${err.message}`);
-  }
-
-  // Step 3: Install stats-agent (system stats collection)
-  try {
-    await installStatsAgent();
-  } catch (err) {
-    console.warn(`[termify-agent] Warning: ${err.message}`);
-  }
-
-  // Step 4: Install termify-mcp, hooks, and configure all CLIs
+  console.log('');
+  console.log(`  ${BOLD}${CYAN}Termify Agent${RESET} ${DIM}postinstall${RESET}`);
+  console.log(`  ${DIM}${platform()}-${arch()} · Node ${process.version}${RESET}`);
+  console.log('');
+
+  muteConsole();
+
+  // Step 1: node-pty
+  const s1 = createSpinner('Checking node-pty...');
+  const ptyOk = rebuildNodePty();
+  if (ptyOk) s1.succeed('node-pty ready');
+  else s1.warn('node-pty skipped ${DIM}(optional)${RESET}');
+
+  // Step 2+3: Binaries in parallel
+  const s2 = createSpinner('Installing binaries...');
+  const [daemonOk, statsOk] = await Promise.all([
+    installDaemon().catch(() => false),
+    installStatsAgent().catch(() => false),
+  ]);
+  const parts = [];
+  if (daemonOk) parts.push('daemon');
+  if (statsOk) parts.push('stats-agent');
+  if (parts.length) s2.succeed(`Binaries ready ${DIM}(${parts.join(', ')})${RESET}`);
+  else s2.warn('Binaries skipped');
+
+  // Step 3: MCP + hooks
+  const s3 = createSpinner('Configuring hooks & MCP...');
   try {
     installTermifyMcp();
+    s3.succeed(`Hooks & MCP configured ${DIM}(Claude · Gemini · Codex)${RESET}`);
   } catch (err) {
-    console.warn(`[termify-agent] Warning: ${err.message}`);
+    s3.warn(`Hooks & MCP: ${err.message}`);
   }
 
-  // Step 5: Ensure AI tool instructions include termify_status usage
+  // Step 4: Instructions
+  const s4 = createSpinner('Updating AI instructions...');
   try {
     ensureClaudeInstructions();
-  } catch (err) {
-    console.warn(`[termify-agent] Warning: ${err.message}`);
-  }
-  try {
     ensureCodexInstructions();
+    s4.succeed('AI instructions updated');
   } catch (err) {
-    console.warn(`[termify-agent] Warning: ${err.message}`);
+    s4.warn(`Instructions: ${err.message}`);
   }
 
-  console.log('[termify-agent] Postinstall complete.');
+  unmuteConsole();
+
+  console.log('');
+  console.log(`  ${GREEN}${BOLD}Done!${RESET} ${DIM}Termify Agent is ready.${RESET}`);
+  console.log('');
 }
 
 main().catch((err) => {
-  console.warn(`[termify-agent] Postinstall warning: ${err.message}`);
-  // Never fail the install - postinstall errors should be warnings only
+  console.warn(`\n  ${WARN} Postinstall warning: ${err.message}\n`);
   process.exit(0);
 });