mstro-app 0.1.58 → 0.3.0

package/bin/mstro.js

@@ -8,20 +8,19 @@
  * Main entry point for the Mstro AI assistant.
  *
  * Usage:
- *   mstro                     # Start Mstro (auto-finds available port)
- *   mstro login               # Authenticate this device
+ *   mstro                     # Start Mstro (logs in automatically if needed)
+ *   mstro login               # Re-authenticate this device
  *   mstro logout              # Sign out
  *   mstro whoami              # Show current user
  *   mstro status              # Show connection status
- *   mstro setup-terminal      # Enable web terminal
  *   mstro -p 4105             # Start on specific port (overrides auto port)
  *   mstro configure-hooks     # Configure Claude Code security hooks
  *   mstro --help              # Show help
  */
 
-import { execSync, spawn } from 'node:child_process';
+import { spawn } from 'node:child_process';
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
-import { homedir, platform as osPlatform } from 'node:os';
+import { homedir } from 'node:os';
 import { dirname, join, resolve } from 'node:path';
 import { createInterface } from 'node:readline';
 import { fileURLToPath } from 'node:url';
@@ -36,10 +35,27 @@ const CLIENT_ROOT = resolve(__dirname, '..');
 const pkg = JSON.parse(readFileSync(join(CLIENT_ROOT, 'package.json'), 'utf-8'));
 
 // Check for updates (runs async in background, notifies on next run)
+// update-notifier initializes lastUpdateCheck to Date.now(), which means the
+// first check won't happen until 24h after install. We detect first-run by
+// checking if the configstore has never stored an update result, and if so
+// reset the timestamp to force an immediate background check.
 const notifier = updateNotifier({
   pkg,
   updateCheckInterval: 1000 * 60 * 60 * 24  // Check daily
 });
+try {
+  if (notifier.config && !notifier.config.has('update') && !notifier.update) {
+    const lastCheck = notifier.config.get('lastUpdateCheck');
+    // If lastUpdateCheck was just set (within the last 30s), this is a fresh
+    // configstore — reset it to 0 so the library spawns a check immediately.
+    if (lastCheck && (Date.now() - lastCheck) < 30_000) {
+      notifier.config.set('lastUpdateCheck', 0);
+      notifier.check();
+    }
+  }
+} catch {
+  // Non-critical — don't let update check logic crash the CLI
+}
 
 // Capture the user's original working directory before any cwd changes
 const USER_CWD = process.cwd();
@@ -47,10 +63,10 @@ const USER_CWD = process.cwd();
 // First-run detection paths
 const MSTRO_CONFIG_DIR = join(homedir(), '.mstro');
 const MSTRO_FIRST_RUN_FLAG = join(MSTRO_CONFIG_DIR, '.configured');
-const MSTRO_TERMINAL_CHECKED_FLAG = join(MSTRO_CONFIG_DIR, '.terminal-checked');
 const CLAUDE_SETTINGS_FILE = join(homedir(), '.claude', 'settings.json');
 const CLAUDE_HOOKS_DIR = join(homedir(), '.claude', 'hooks');
 const BOUNCER_HOOK_FILE = join(CLAUDE_HOOKS_DIR, 'bouncer.sh');
+const PTY_SETUP_DISMISSED_FLAG = join(MSTRO_CONFIG_DIR, '.pty-setup-dismissed');
 
 /**
  * Mark Mstro as configured by writing the first-run flag file
@@ -195,19 +211,163 @@ async function promptBouncerSetup() {
   }
 }
 
+/**
+ * Check if node-pty native module is loadable
+ */
+async function isNodePtyAvailable() {
+  try {
+    await import('node-pty');
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+/**
+ * Check if user has dismissed the pty setup prompt
+ */
+function hasUserDismissedPtySetup() {
+  return existsSync(PTY_SETUP_DISMISSED_FLAG);
+}
+
+/**
+ * Mark pty setup prompt as dismissed
+ */
+function markPtySetupDismissed() {
+  if (!existsSync(MSTRO_CONFIG_DIR)) {
+    mkdirSync(MSTRO_CONFIG_DIR, { recursive: true, mode: 0o700 });
+  }
+  writeFileSync(PTY_SETUP_DISMISSED_FLAG, new Date().toISOString());
+}
+
+/**
+ * Show a one-line warning that node-pty is not available
+ */
+function showPtyWarning() {
+  log('  Terminal support not available. Run: mstro setup-terminal', colors.dim);
+}
+
+/**
+ * Get platform-specific build tool instructions
+ */
+function getPtyBuildInstructions() {
+  const os = process.platform;
+  if (os === 'darwin') {
+    return '    Install Xcode Command Line Tools: xcode-select --install';
+  }
+  if (os === 'win32') {
+    return '    Install Windows Build Tools: npm install -g windows-build-tools';
+  }
+  return '    Debian/Ubuntu: sudo apt install build-essential python3\n' +
+         '    Fedora/RHEL:   sudo dnf install gcc-c++ make python3\n' +
+         '    Arch:          sudo pacman -S base-devel python';
+}
+
+/**
+ * Attempt to rebuild/install node-pty from CLIENT_ROOT
+ * Returns true if npm command succeeded, false otherwise
+ */
+function attemptPtyRebuild() {
+  return new Promise((resolve) => {
+    const nodePtyDir = join(CLIENT_ROOT, 'node_modules', 'node-pty');
+    const npmCmd = process.platform === 'win32' ? 'npm.cmd' : 'npm';
+    const command = existsSync(nodePtyDir) ? 'rebuild' : 'install';
+    const args = command === 'rebuild'
+      ? ['rebuild', 'node-pty']
+      : ['install', 'node-pty', '--no-save'];
+
+    log(`\n  ${command === 'rebuild' ? 'Rebuilding' : 'Installing'} node-pty...`, colors.dim);
+
+    const child = spawn(npmCmd, args, {
+      cwd: CLIENT_ROOT,
+      stdio: 'inherit',
+    });
+
+    child.on('error', (err) => {
+      log(`  Error: ${err.message}`, colors.red);
+      resolve(false);
+    });
+
+    child.on('exit', (code) => {
+      resolve(code === 0);
+    });
+  });
+}
+
+/**
+ * Prompt user to set up node-pty for terminal support
+ * Returns: 'configure' | 'skip' | 'never'
+ */
+async function promptPtySetup() {
+  log('\n  Terminal Support\n', colors.bold + colors.cyan);
+  log('  Mstro includes a web terminal that lets you open a shell', colors.dim);
+  log('  directly in your browser. This requires compiling a native module (node-pty).\n', colors.dim);
+
+  const isInteractive = process.stdin.isTTY;
+
+  if (!isInteractive) {
+    log('  Non-interactive mode: skipping terminal setup.', colors.yellow);
+    log('  Run "mstro setup-terminal" to enable terminal support.\n', colors.dim);
+    return 'skip';
+  }
+
+  log('  Set up terminal support now?', colors.bold);
+  log('    [Y] Yes, compile now (requires build tools)', colors.dim);
+  log('    [n] Not now (ask again next time)', colors.dim);
+  log('    [d] Don\'t show this again\n', colors.dim);
+
+  const answer = await prompt('  Your choice [Y/n/d]: ');
+  const choice = answer.toLowerCase();
+
+  if (choice === '' || choice === 'y' || choice === 'yes') {
+    return 'configure';
+  }
+  if (choice === 'd' || choice === 'dont' || choice === "don't") {
+    log('\n  Got it! You can set up later with: mstro setup-terminal\n', colors.dim);
+    markPtySetupDismissed();
+    return 'never';
+  }
+  log('\n  Skipping for now. Will ask again next time.', colors.yellow);
+  log('  You can also set up with: mstro setup-terminal\n', colors.dim);
+  return 'skip';
+}
+
+/**
+ * Run the pty rebuild and show results
+ */
+async function runPtySetup() {
+  const success = await attemptPtyRebuild();
+
+  if (success) {
+    const available = await isNodePtyAvailable();
+    if (available) {
+      log('\n  Terminal support enabled successfully!\n', colors.bold + colors.green);
+      return true;
+    }
+    log('\n  node-pty installed but failed to load.', colors.red);
+  }
+
+  log('\n  Could not compile node-pty automatically.\n', colors.yellow);
+  log('  You may need to install build tools first:\n', colors.bold);
+  log(getPtyBuildInstructions(), colors.dim);
+  log('');
+  log('  After installing build tools, run: mstro setup-terminal\n', colors.dim);
+  return false;
+}
+
 function showHelp() {
-  log('\n  Mstro - No-code AI Workspace\n', colors.bold + colors.cyan);
-  log('  Run Claude Code workflows from your laptop, cloud VM, or any machine.\n', colors.dim);
+  log('\n  Mstro - Run Claude Code from any browser\n', colors.bold + colors.cyan);
+  log('  Streams live Claude Code sessions from your machine to mstro.app.\n', colors.dim);
   log('  Usage:', colors.bold);
-  log('    mstro                       Start Mstro (auto-finds available port)', colors.dim);
-  log('    mstro login                 Authenticate this device with mstro.app', colors.dim);
+  log('    mstro                       Start Mstro (logs in automatically if needed)', colors.dim);
+  log('    mstro login                 Re-authenticate this device with mstro.app', colors.dim);
   log('    mstro logout                Sign out of mstro.app', colors.dim);
   log('    mstro whoami                Show current user and device info', colors.dim);
   log('    mstro status                Show connection and auth status', colors.dim);
   log('    mstro telemetry [on|off]    Enable/disable anonymous telemetry', colors.dim);
-  log('    mstro setup-terminal        Enable web terminal (compiles native module)', colors.dim);
   log('    mstro -p 4105               Start on specific port (overrides auto port)', colors.dim);
   log('    mstro configure-hooks       Configure Claude Code security hooks', colors.dim);
+  log('    mstro setup-terminal        Enable web terminal (compiles native module)', colors.dim);
   log('    mstro --version             Show version number', colors.dim);
   log('    mstro --help                Show this help message', colors.dim);
   log('');
@@ -217,8 +377,8 @@ function showHelp() {
   log('    --verbose, -v               Enable verbose output', colors.dim);
   log('');
   log('  Authentication:', colors.bold);
-  log('    Run "mstro login" to connect this device to your mstro.app account.', colors.dim);
-  log('    Once logged in, orchestras sync automatically with your web dashboard.', colors.dim);
+  log('    Running "mstro" will prompt you to log in automatically if needed.', colors.dim);
+  log('    Once logged in, machines sync automatically with your web dashboard.', colors.dim);
   log('');
   log('  Security:', colors.bold);
   log('    Mstro includes a Security Bouncer that automatically manages', colors.dim);
@@ -229,7 +389,7 @@ function showHelp() {
 
 function runNpmScript(script, args = [], envOverrides = {}) {
   const npmCmd = process.platform === 'win32' ? 'npm.cmd' : 'npm';
-  const child = spawn(npmCmd, ['run', script, ...args], {
+  const child = spawn(npmCmd, ['run', '--silent', script, ...args], {
     cwd: CLIENT_ROOT,
     stdio: 'inherit',
     env: { ...process.env, MSTRO_WORKING_DIR: USER_CWD, ...envOverrides },
@@ -291,7 +451,6 @@ function runConfigureHooks(andThenStart = false) {
       markConfigured();
       if (andThenStart) {
         // After configuring, start the server
-        log('\nStarting Mstro client...', colors.bold + colors.cyan);
         const requestedPort = parsePort(process.argv.slice(2));
         const envOverrides = requestedPort ? { PORT: String(requestedPort) } : {};
         runNpmScript('start', [], envOverrides);
@@ -325,14 +484,18 @@ function parsePort(args) {
  * Show update notification if available
  */
 function showUpdateNotification() {
-  if (notifier.update && semverGt(notifier.update.latest, notifier.update.current)) {
-    const { current, latest, type } = notifier.update;
-    const updateCmd = 'npm i -g mstro@latest';
-
-    log('');
-    log(`  ${colors.yellow}Update available:${colors.reset} ${colors.dim}${current}${colors.reset} → ${colors.green}${latest}${colors.reset} ${colors.dim}(${type})${colors.reset}`);
-    log(`  Run: ${colors.cyan}${updateCmd}${colors.reset}`);
-    log('');
+  try {
+    if (notifier.update && semverGt(notifier.update.latest, notifier.update.current)) {
+      const { current, latest, type } = notifier.update;
+      const updateCmd = `npm i -g ${pkg.name}`;
+
+      log('');
+      log(`  ${colors.yellow}Update available:${colors.reset} ${colors.dim}${current}${colors.reset} → ${colors.green}${latest}${colors.reset} ${colors.dim}(${type})${colors.reset}`);
+      log(`  Run: ${colors.cyan}${updateCmd}${colors.reset}`);
+      log('');
+    }
+  } catch {
+    // Don't let a corrupted cache or invalid semver crash the CLI
   }
 }
 
@@ -353,136 +516,63 @@ function isLoggedIn() {
 }
 
 /**
- * Show login required message
+ * Auto-login if not authenticated. Exits on failure.
  */
-function showLoginRequired() {
-  log('\n  Authentication required', colors.bold + colors.yellow);
-  log('');
-  log('  You must be logged in to use mstro.', colors.dim);
-  log('  Run "mstro login" to authenticate this device.', colors.dim);
-  log('');
-}
-
-/**
- * Check if node-pty is loadable (native module compiled correctly)
- */
-async function isNodePtyAvailable() {
+async function ensureLoggedIn() {
+  if (isLoggedIn()) return;
+  log('\n  Not logged in — starting authentication...\n', colors.bold + colors.cyan);
   try {
-    const pty = await import('node-pty');
-    // Verify the native module actually works, not just that it imports
-    const test = pty.spawn('/bin/echo', ['test'], { name: 'xterm', cols: 80, rows: 24 });
-    test.kill();
-    return true;
-  } catch {
-    return false;
+    const { login } = await import('./commands/login.js');
+    await login(args, { inline: true });
+  } catch (err) {
+    log(`\n  Login failed: ${err.message}`, colors.red);
+    log('  Run "mstro login" to try again.\n', colors.dim);
+    process.exit(1);
   }
 }
 
 /**
- * Try to rebuild node-pty silently. Returns true on success.
+ * Prompt for bouncer setup if not configured
+ * Returns true if runConfigureHooks was called (async exit path)
  */
-function tryRebuildNodePty() {
-  try {
-    execSync('npm rebuild node-pty', { cwd: CLIENT_ROOT, stdio: 'pipe' });
-    return true;
-  } catch {
+async function ensureBouncerSetup() {
+  if (isBouncerConfigured()) return false;
+  if (hasUserDismissedSetup()) {
+    showBouncerWarning();
     return false;
   }
-}
-
-/**
- * Get platform-specific build tool install instructions
- */
-function getBuildToolInstructions() {
-  const os = osPlatform();
-  if (os === 'darwin') {
-    return '    xcode-select --install';
-  } else if (os === 'win32') {
-    return '    npm install -g windows-build-tools';
-  } else {
-    return '    sudo apt install build-essential python3    # Debian/Ubuntu\n    sudo dnf install gcc-c++ make python3       # Fedora/RHEL';
+  const choice = await promptBouncerSetup();
+  if (choice === 'configure') {
+    runConfigureHooks(true);
+    return true;
   }
+  return false;
 }
 
 /**
- * First-run terminal setup check (runs after bouncer setup).
- * Tries to rebuild node-pty automatically. If that fails, shows instructions.
+ * Prompt for node-pty setup if not available
  */
-async function checkTerminalSetup() {
-  if (await isNodePtyAvailable()) {
-    return; // Already working
-  }
-
-  log('\n  Web Terminal', colors.bold + colors.cyan);
-  log('  mstro includes a browser-based terminal (optional).\n', colors.dim);
-  log('  Attempting to compile native module...', colors.dim);
-
-  if (tryRebuildNodePty()) {
-    log('  Terminal support enabled!\n', colors.green);
+async function ensurePtySetup() {
+  const ptyAvailable = await isNodePtyAvailable();
+  if (ptyAvailable) return;
+  if (hasUserDismissedPtySetup()) {
+    showPtyWarning();
     return;
   }
-
-  log('  Could not compile terminal module.\n', colors.yellow);
-  log('  To enable the web terminal later:', colors.dim);
-  log('  1. Install build tools:', colors.dim);
-  log(getBuildToolInstructions(), colors.dim);
-  log('  2. Run:', colors.dim);
-  log('    mstro setup-terminal\n', colors.dim);
-}
-
-/**
- * Explicit setup-terminal command
- */
-async function setupTerminal() {
-  log('\n  Setting up terminal support...\n', colors.bold + colors.cyan);
-
-  if (await isNodePtyAvailable()) {
-    log('  Terminal support is already enabled.\n', colors.green);
-    return;
+  const choice = await promptPtySetup();
+  if (choice === 'configure') {
+    await runPtySetup();
   }
-
-  log('  Rebuilding node-pty native module...', colors.dim);
-
-  if (tryRebuildNodePty()) {
-    log('\n  Terminal support enabled! Restart mstro to use it.\n', colors.green + colors.bold);
-    return;
-  }
-
-  log('\n  Failed to build node-pty.\n', colors.red);
-  log('  Install build tools first:', colors.dim);
-  log(getBuildToolInstructions(), colors.dim);
-  log('\n  Then re-run: mstro setup-terminal\n', colors.dim);
-  process.exit(1);
 }
 
 async function startServer(envOverrides) {
-  if (!isLoggedIn()) {
-    showLoginRequired();
-    process.exit(1);
-  }
+  await ensureLoggedIn();
 
-  if (!isBouncerConfigured()) {
-    if (hasUserDismissedSetup()) {
-      showBouncerWarning();
-    } else {
-      const choice = await promptBouncerSetup();
-      if (choice === 'configure') {
-        runConfigureHooks(true);
-        return;
-      }
-    }
-  }
+  if (await ensureBouncerSetup()) return;
 
-  if (!existsSync(MSTRO_TERMINAL_CHECKED_FLAG)) {
-    await checkTerminalSetup();
-    if (!existsSync(MSTRO_CONFIG_DIR)) {
-      mkdirSync(MSTRO_CONFIG_DIR, { recursive: true, mode: 0o700 });
-    }
-    writeFileSync(MSTRO_TERMINAL_CHECKED_FLAG, new Date().toISOString());
-  }
+  await ensurePtySetup();
 
   showUpdateNotification();
-  log('\nStarting Mstro client...', colors.bold + colors.cyan);
   runNpmScript('start', [], envOverrides);
 }
 
@@ -504,7 +594,7 @@ async function main() {
     }],
     ['logout', async () => {
       const { logout } = await import('./commands/logout.js');
-      await logout();
+      await logout(args.slice(args.indexOf('logout') + 1));
     }],
     ['whoami', async () => {
       const { whoami } = await import('./commands/whoami.js');
@@ -518,8 +608,17 @@ async function main() {
       const { telemetry } = await import('./commands/config.js');
       await telemetry(args.slice(args.indexOf('telemetry') + 1));
     }],
-    ['setup-terminal', () => setupTerminal()],
     ['configure-hooks', () => runConfigureHooks(false)],
+    ['setup-terminal', async () => {
+      log('\n  Mstro Terminal Setup\n', colors.bold + colors.cyan);
+      const alreadyAvailable = await isNodePtyAvailable();
+      if (alreadyAvailable) {
+        log('  node-pty is already available. Terminal support is enabled!\n', colors.green);
+        return;
+      }
+      const success = await runPtySetup();
+      process.exit(success ? 0 : 1);
+    }],
   ]);
 
   // Flag-based commands
@@ -544,6 +643,7 @@ async function main() {
   const handler = subcommand ? commands.get(subcommand) : undefined;
   if (handler) {
     await handler();
+    showUpdateNotification();
     return;
   }
 

package/dist/server/cli/headless/claude-invoker.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"claude-invoker.d.ts","sourceRoot":"","sources":["../../../../server/cli/headless/claude-invoker.ts"],"names":[],"mappings":"AAGA;;;;GAIG;AAEH,OAAO,EAAE,KAAK,YAAY,EAAS,MAAM,oBAAoB,CAAC;AAK9D,OAAO,KAAK,EACV,eAAe,EACf,sBAAsB,EAEvB,MAAM,YAAY,CAAC;AAEpB,MAAM,WAAW,oBAAoB;IACnC,MAAM,EAAE,sBAAsB,CAAC;IAC/B,gBAAgB,EAAE,GAAG,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC;CAC7C;AA6WD;;;GAGG;AACH,wBAAsB,oBAAoB,CACxC,MAAM,EAAE,MAAM,EACd,WAAW,EAAE,MAAM,EACnB,cAAc,EAAE,MAAM,EACtB,OAAO,EAAE,oBAAoB,GAC5B,OAAO,CAAC,eAAe,CAAC,CAgN1B"}
+{"version":3,"file":"claude-invoker.d.ts","sourceRoot":"","sources":["../../../../server/cli/headless/claude-invoker.ts"],"names":[],"mappings":"AAGA;;;;GAIG;AAEH,OAAO,EAAE,KAAK,YAAY,EAAS,MAAM,oBAAoB,CAAC;AAO9D,OAAO,KAAK,EACV,eAAe,EACf,sBAAsB,EAGvB,MAAM,YAAY,CAAC;AAEpB,MAAM,WAAW,oBAAoB;IACnC,MAAM,EAAE,sBAAsB,CAAC;IAC/B,gBAAgB,EAAE,GAAG,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC;CAC7C;AAmgCD;;;GAGG;AACH,wBAAsB,oBAAoB,CACxC,MAAM,EAAE,MAAM,EACd,WAAW,EAAE,MAAM,EACnB,cAAc,EAAE,MAAM,EACtB,OAAO,EAAE,oBAAoB,GAC5B,OAAO,CAAC,eAAe,CAAC,CA0H1B"}