@gettrace/cli 2.0.7 → 2.0.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gettrace/cli",
-  "version": "2.0.7",
+  "version": "2.0.8",
   "description": "Trace IDE Bridge for connecting local filesystem to Trace browser extensions.",
   "main": "dist/index.js",
   "type": "module",

package/scripts/postinstall.js

@@ -34,8 +34,10 @@ const __dirname = dirname(fileURLToPath(import.meta.url));
 
 // The extension IDs that are allowed to use this native host.
 const ALLOWED_EXTENSION_IDS = [
-    // Development / unpacked extension ID
+    // Development / unpacked extension ID (macOS)
     'jefgjgdcelekglimbmgplkaaabdfpgfm',
+    // Development / unpacked extension ID (Windows)
+    'fppbikhdjehmiidhjholkiaeeieoilfb',
     // Add the published CWS extension ID here when you have it:
     // 'PRODUCTION_EXTENSION_ID',
 ];
@@ -49,11 +51,18 @@ if (process.env.TRACE_EXTENSION_IDS) {
 
 const HOST_NAME = 'dev.gettrace.host';
 
-// Install native host files into ~/.local/share/trace/native-host/ so that
-// Chrome (and other browsers) can always read them. Files on ~/Desktop,
-// ~/Documents, ~/Downloads require explicit macOS TCC (Privacy) permission
-// which Chrome does not have by default. ~/.local/share/ is always accessible.
-const INSTALL_DIR    = join(homedir(), '.local', 'share', 'trace', 'native-host');
+// Install native host files into platform-specific directories
+// macOS/Linux: ~/.local/share/trace/native-host/ (no TCC permission issues)
+// Windows: %LOCALAPPDATA%\trace\native-host\ (standard Windows app data location)
+function getInstallDir() {
+    if (platform() === 'win32') {
+        const localAppData = process.env.LOCALAPPDATA || join(homedir(), 'AppData', 'Local');
+        return join(localAppData, 'trace', 'native-host');
+    }
+    return join(homedir(), '.local', 'share', 'trace', 'native-host');
+}
+
+const INSTALL_DIR    = getInstallDir();
 const HOST_ENTRY     = join(INSTALL_DIR, 'host-entry');
 const HOST_CJS_DEST  = join(INSTALL_DIR, 'host.cjs');
 
@@ -96,6 +105,84 @@ function generateHostEntry() {
         writeFileSync(join(INSTALL_DIR, 'package.json'), readFileSync(pkgSrc), { mode: 0o644 });
     }
 
+    // On Windows, create a .bat launcher that Chrome can execute
+    if (platform() === 'win32') {
+        generateWindowsHostEntry(nodePath, tracePath);
+    } else {
+        generateUnixHostEntry(nodePath, tracePath);
+    }
+}
+
+function generateWindowsHostEntry(nodePath, tracePath) {
+    const HOST_BAT = join(INSTALL_DIR, 'host-entry.bat');
+    const HOST_JS = join(INSTALL_DIR, 'host-entry.js');
+    
+    // Create the JavaScript loader file
+    const jsContent = [
+        `// Auto-generated by @gettrace/cli postinstall — do not edit manually`,
+        `// Generated: ${new Date().toISOString()}`,
+        `// Node:  ${nodePath}`,
+        tracePath ? `// Trace: ${tracePath}` : `// Trace: (resolves from PATH)`,
+        `'use strict';`,
+        ``,
+        `// Fix PATH before anything else — Chrome launches with a minimal environment`,
+        tracePath
+            ? `process.env.PATH = '${dirname(tracePath).replace(/\\/g, '\\\\')};${dirname(nodePath).replace(/\\/g, '\\\\')};' + (process.env.PATH || 'C:\\\\Windows\\\\System32');`
+            : `process.env.PATH = '${dirname(nodePath).replace(/\\/g, '\\\\')};;' + (process.env.PATH || 'C:\\\\Windows\\\\System32');`,
+        ``,
+        ...(() => {
+            const monorepoAgent = resolve(
+                __dirname, '..', '..', 'packages', 'trace-agent', 'dist', 'node', 'index.js'
+            );
+            const localCli = resolve(__dirname, '..', 'dist', 'index.js');
+            if (existsSync(monorepoAgent) && existsSync(localCli)) {
+                return [
+                    `// Dev mode: use local CLI build instead of the global npm package`,
+                    `process.env.TRACE_DEV_MODE = '1';`,
+                    `process.env.TRACE_CLI_PATH = '${localCli.replace(/\\/g, '\\\\')}';`,
+                    ``,
+                ];
+            }
+            return [];
+        })(),
+        `// Diagnostic logging — written synchronously so we catch crashes before async`,
+        `const fs = require('fs');`,
+        `const path = require('path');`,
+        `const logPath = path.join('${homedir().replace(/\\/g, '\\\\')}', 'trace-native-host.log');`,
+        `function log(msg) {`,
+        `    const line = new Date().toISOString() + ' ' + msg + '\\n';`,
+        `    try { fs.appendFileSync(logPath, line); } catch(_) {}`,
+        `}`,
+        ``,
+        `log('[host-entry] started, node=' + process.version + ' pid=' + process.pid);`,
+        ``,
+        `try {`,
+        `    require('${HOST_CJS_DEST.replace(/\\/g, '\\\\')}');`,
+        `    log('[host-entry] host.cjs loaded OK');`,
+        `} catch (err) {`,
+        `    log('[host-entry] FATAL: ' + (err.stack || err));`,
+        `    process.exit(1);`,
+        `}`,
+    ].join('\n');
+    
+    writeFileSync(HOST_JS, jsContent, { encoding: 'utf8' });
+    
+    // Create the .bat launcher that Chrome will execute
+    const batContent = [
+        `@echo off`,
+        `REM Auto-generated by @gettrace/cli postinstall`,
+        `REM Node: ${nodePath}`,
+        `"${nodePath}" "${HOST_JS}" %*`,
+    ].join('\r\n');
+    
+    writeFileSync(HOST_BAT, batContent, { encoding: 'utf8' });
+    
+    console.log(`✓ Generated Windows native host: ${HOST_BAT}`);
+    console.log(`  Node binary: ${nodePath}`);
+    if (tracePath) console.log(`  Trace binary: ${tracePath}`);
+}
+
+function generateUnixHostEntry(nodePath, tracePath) {
     // Log paths — write to home dir only (always writable by Chrome)
     const logPath1 = `${homedir()}/trace-native-host.log`;
 
@@ -112,16 +199,6 @@ function generateHostEntry() {
             ? `process.env.PATH = '${dirname(tracePath)}:${dirname(nodePath)}:' + (process.env.PATH || '/usr/bin:/bin');`
             : `process.env.PATH = '${dirname(nodePath)}:/opt/homebrew/bin:/usr/local/bin:' + (process.env.PATH || '/usr/bin:/bin');`,
         ``,
-        // Inject dev mode env vars ONLY when running from the trace monorepo
-        // (the dev workflow). Detect that by the presence of the local agent
-        // build that dev-mode resolution actually targets:
-        // <repo>/packages/trace-agent/dist/node/index.js. In a published npm
-        // install this path does not exist (the package lives under
-        // node_modules/@gettrace/cli), so production NEVER enables dev mode.
-        //
-        // NOTE: do NOT key this off `<pkg>/dist/index.js` — the published CLI
-        // ships that file too, which previously made every install spuriously
-        // enable dev mode and fail to locate the agent.
         ...(() => {
             const monorepoAgent = resolve(
                 __dirname, '..', '..', 'packages', 'trace-agent', 'dist', 'node', 'index.js'
@@ -179,10 +256,15 @@ function getNativeHostDir() {
 }
 
 function buildManifest() {
+    // On Windows, Chrome needs to execute a .bat file; Unix uses the shebang script
+    const executablePath = platform() === 'win32' 
+        ? join(INSTALL_DIR, 'host-entry.bat')
+        : HOST_ENTRY;
+        
     return JSON.stringify({
         name: HOST_NAME,
         description: 'Trace native host — spawns trace dev in the user\'s project directory',
-        path: HOST_ENTRY,  // Points to the generated entry file with hardcoded node path
+        path: executablePath,
         type: 'stdio',
         allowed_origins: ALLOWED_EXTENSION_IDS.map(id => `chrome-extension://${id}/`),
     }, null, 2);
@@ -190,15 +272,23 @@ function buildManifest() {
 
 function registerOnWindows() {
     const manifest = buildManifest();
-    const manifestPath = join(homedir(), 'AppData', 'Local', 'Trace', `${HOST_NAME}.json`);
+    // Store manifest in the same directory as the host files for consistency
+    const manifestPath = join(INSTALL_DIR, `${HOST_NAME}.json`);
     mkdirSync(dirname(manifestPath), { recursive: true });
     writeFileSync(manifestPath, manifest, 'utf8');
+    console.log(`✓ Native host manifest: ${manifestPath}`);
+    
     const regKey = `HKCU\\Software\\Google\\Chrome\\NativeMessagingHosts\\${HOST_NAME}`;
     try {
-        execSync(`reg add "${regKey}" /ve /t REG_SZ /d "${manifestPath}" /f`, { stdio: 'pipe' });
-        console.log(`✓ Native host registered (Windows registry)`);
+        // Use double backslashes for Windows paths in registry
+        const manifestPathWin = manifestPath.replace(/\//g, '\\');
+        execSync(`reg add "${regKey}" /ve /t REG_SZ /d "${manifestPathWin}" /f`, { stdio: 'pipe' });
+        console.log(`✓ Native host registered in Windows registry`);
+        console.log(`  Registry key: ${regKey}`);
+        console.log(`  Manifest path: ${manifestPathWin}`);
     } catch (e) {
         console.warn('⚠ Could not register native host in Windows registry:', e.message);
+        console.warn('  Try running as Administrator or manually import the registry key');
     }
 }