droid 0.72.0 → 0.74.0

package/README.md

@@ -8,6 +8,30 @@
 npm install -g droid
 ```
 
+If the `droid` command is not found after installing, your npm global bin directory may not be in your PATH. Run the following to fix it:
+
+```bash
+# For zsh (macOS default)
+echo 'export PATH="$(npm prefix -g)/bin:$PATH"' >> ~/.zshrc && source ~/.zshrc
+
+# For bash (Linux)
+echo 'export PATH="$(npm prefix -g)/bin:$PATH"' >> ~/.bashrc && source ~/.bashrc
+
+# For bash (macOS)
+echo 'export PATH="$(npm prefix -g)/bin:$PATH"' >> ~/.bash_profile && source ~/.bash_profile
+
+# For fish
+echo 'fish_add_path "$(npm prefix -g)/bin"' >> ~/.config/fish/config.fish && source ~/.config/fish/config.fish
+```
+
+On Windows, run in PowerShell:
+
+```powershell
+[Environment]::SetEnvironmentVariable('Path', $env:PATH + ';' + (npm prefix -g), 'User')
+```
+
+Then restart your terminal.
+
 Then navigate to your project and start the droid CLI:
 
 ```bash

package/install.js

@@ -221,6 +221,122 @@ function extractFileFromTarball(tarballBuffer, filepath) {
   return null;
 }
 
+/**
+ * Check if this is a global npm install by looking at the npm_config_global
+ * environment variable that npm sets during installation.
+ */
+function isGlobalInstall() {
+  return (
+    process.env.npm_config_global === 'true' ||
+    process.env.npm_config_global === ''
+  );
+}
+
+/**
+ * Detect the npm global bin directory.
+ * Uses `npm prefix -g` and appends /bin on POSIX (Windows puts binaries
+ * directly in the prefix directory).
+ */
+function getNpmGlobalBinDir() {
+  try {
+    const prefix = child_process
+      .execSync('npm prefix -g', { encoding: 'utf8', timeout: 10000 })
+      .trim();
+    if (!prefix) return null;
+    return process.platform === 'win32' ? prefix : path.join(prefix, 'bin');
+  } catch (e) {
+    return null;
+  }
+}
+
+/**
+ * Check if a directory is already present in the current PATH.
+ * Uses path.resolve() to normalize slashes and handles case-insensitive
+ * comparison on Windows.
+ */
+function isDirInPath(dir) {
+  const envPath = process.env.PATH || '';
+  const sep = process.platform === 'win32' ? ';' : ':';
+  const normalize = (d) => path.resolve(d);
+  const normalizedDir = normalize(dir);
+  const dirs = envPath.split(sep).map(normalize);
+  if (process.platform === 'win32') {
+    return dirs.some((d) => d.toLowerCase() === normalizedDir.toLowerCase());
+  }
+  return dirs.includes(normalizedDir);
+}
+
+/**
+ * Detect the user's shell and suggest the appropriate profile file.
+ */
+function getShellProfileHint() {
+  const shell = process.env.SHELL || '';
+  const shellName = path.basename(shell);
+  if (shellName === 'zsh') return '~/.zshrc';
+  if (shellName === 'bash') {
+    return process.platform === 'darwin' ? '~/.bash_profile' : '~/.bashrc';
+  }
+  if (shellName === 'fish') return '~/.config/fish/config.fish';
+  return null;
+}
+
+/**
+ * Print a warning if the npm global bin directory is not in the user's PATH.
+ * Only runs for global installs. Does NOT modify any files.
+ */
+function warnIfNotInPath() {
+  try {
+    if (process.env.CI || process.env.CONTINUOUS_INTEGRATION) return;
+    if (!isGlobalInstall()) return;
+
+    const binDir = getNpmGlobalBinDir();
+    if (!binDir) return;
+    if (isDirInPath(binDir)) return;
+
+    const profileHint = getShellProfileHint();
+
+    if (process.platform === 'win32') {
+      console.log(
+        '\n' +
+          'Note: The "droid" command may not be available in your terminal.\n' +
+          `The npm global bin directory is not in your PATH:\n` +
+          `  ${binDir}\n\n` +
+          'To fix for this session, run:\n' +
+          `  set PATH=%PATH%;${binDir}\n\n` +
+          'To fix permanently (PowerShell):\n' +
+          `  [Environment]::SetEnvironmentVariable('Path', $env:PATH + ';${binDir}', 'User')\n\n` +
+          'Then restart your terminal.\n'
+      );
+    } else if (profileHint) {
+      const exportLine =
+        profileHint === '~/.config/fish/config.fish'
+          ? `fish_add_path "${binDir}"`
+          : `export PATH="${binDir}:$PATH"`;
+
+      console.log(
+        '\n' +
+          'Note: The "droid" command may not be available in your terminal.\n' +
+          `The npm global bin directory is not in your PATH:\n` +
+          `  ${binDir}\n\n` +
+          `To fix, run:\n` +
+          `  echo '${exportLine}' >> ${profileHint} && source ${profileHint}\n`
+      );
+    } else {
+      console.log(
+        '\n' +
+          'Note: The "droid" command may not be available in your terminal.\n' +
+          `The npm global bin directory is not in your PATH:\n` +
+          `  ${binDir}\n\n` +
+          'To fix, add the following to your shell profile (~/.bashrc, ~/.zshrc, etc.):\n' +
+          `  export PATH="${binDir}:$PATH"\n\n` +
+          'Then restart your terminal.\n'
+      );
+    }
+  } catch (e) {
+    // PATH check is best-effort; never fail the install
+  }
+}
+
 async function main() {
   const shimPath = path.join(__dirname, 'bin', 'droid');
   let binaryPath = getBinaryPath();
@@ -308,7 +424,11 @@ async function main() {
   }
 }
 
-main().catch((e) => {
-  console.error('postinstall error:', e.message);
-  // Don't fail the install - the JS shim will still work
-});
+main()
+  .then(() => {
+    warnIfNotInPath();
+  })
+  .catch((e) => {
+    console.error('postinstall error:', e.message);
+    // Don't fail the install - the JS shim will still work
+  });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "droid",
-  "version": "0.72.0",
+  "version": "0.74.0",
   "description": "Factory Droid CLI - AI-powered software engineering agent",
   "bin": {
     "droid": "bin/droid"
@@ -16,15 +16,15 @@
     "postinstall": "node install.js"
   },
   "optionalDependencies": {
-    "@factory/cli-darwin-arm64": "0.72.0",
-    "@factory/cli-darwin-x64": "0.72.0",
-    "@factory/cli-darwin-x64-baseline": "0.72.0",
-    "@factory/cli-linux-arm64": "0.72.0",
-    "@factory/cli-linux-x64": "0.72.0",
-    "@factory/cli-linux-x64-baseline": "0.72.0",
-    "@factory/cli-win32-x64": "0.72.0",
-    "@factory/cli-win32-x64-baseline": "0.72.0",
-    "@factory/cli-win32-arm64": "0.72.0"
+    "@factory/cli-darwin-arm64": "0.74.0",
+    "@factory/cli-darwin-x64": "0.74.0",
+    "@factory/cli-darwin-x64-baseline": "0.74.0",
+    "@factory/cli-linux-arm64": "0.74.0",
+    "@factory/cli-linux-x64": "0.74.0",
+    "@factory/cli-linux-x64-baseline": "0.74.0",
+    "@factory/cli-win32-x64": "0.74.0",
+    "@factory/cli-win32-x64-baseline": "0.74.0",
+    "@factory/cli-win32-arm64": "0.74.0"
   },
   "engines": {
     "node": ">=20.0.0"