offgrid-ai 0.8.1 → 0.8.3

package/install.sh

@@ -131,10 +131,10 @@ if command -v npm &>/dev/null; then
   # Try the modern way first, then legacy
   NPM_PREFIX="$(npm prefix -g 2>/dev/null || true)"
   if [[ -n "$NPM_PREFIX" ]]; then
-    NPM_BIN="${NPM_PREFIX}/bin"
+    NPM_BIN="${NPM_PREFIX%/}/bin"
   fi
   # Validate: the binary should exist there
-  if [[ ! -d "$NPM_BIN" ]]; then
+  if [[ ! -x "$NPM_BIN/offgrid-ai" ]]; then
     NPM_BIN=""
   fi
 fi
@@ -158,12 +158,14 @@ if [[ -n "$NPM_BIN" && -x "$NPM_BIN/offgrid-ai" ]]; then
 
     # Add to shell config for future sessions (pick first existing or .zshrc)
     ADDED_TO_RC=false
-    for RC_FILE in "$HOME/.zshrc" "$HOME/.bashrc" "$HOME/.bash_profile"; do
+    # Respect user's shell preference; default to .zshrc on macOS
+    [[ "$OSTYPE" == darwin* ]] && [[ -z "$DEFAULT_RC" ]] && DEFAULT_RC="$HOME/.zshrc"
+    RC_CANDIDATES=("${DEFAULT_RC:-}" "$HOME/.zshrc" "$HOME/.bashrc" "$HOME/.bash_profile")
+    for RC_FILE in "${RC_CANDIDATES[@]}"; do
+      [[ -z "$RC_FILE" ]] && continue
       if [[ -f "$RC_FILE" || "$RC_FILE" == "$HOME/.zshrc" ]]; then
         if ! grep -qF "$NPM_BIN" "$RC_FILE" 2>/dev/null; then
-          echo '' >> "$RC_FILE"
-          echo '# Added by offgrid-ai installer' >> "$RC_FILE"
-          echo "export PATH=\"$NPM_BIN:\$PATH\"" >> "$RC_FILE"
+          { echo ''; echo '# Added by offgrid-ai installer'; echo "export PATH=\"$NPM_BIN:\$PATH\""; } >> "$RC_FILE"
           ok "Added $NPM_BIN to $RC_FILE"
           ADDED_TO_RC=true
           # Only add to one rc file to avoid duplicates

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "offgrid-ai",
-  "version": "0.8.1",
+  "version": "0.8.3",
   "description": "Privacy-first CLI for running local LLMs — discover, configure, run, benchmark",
   "author": "Eeshan Srivastava (https://eeshans.com)",
   "type": "module",

package/src/postinstall.mjs

@@ -1,12 +1,20 @@
 #!/usr/bin/env node
 import { existsSync, readFileSync, appendFileSync } from "node:fs";
-import { join } from "node:path";
+import { mkdirSync } from "node:fs";
+import { dirname, join } from "node:path";
 
 if (process.env.CI || process.env.OFFGRID_SKIP_POSTINSTALL) process.exit(0);
-if (process.env.npm_config_global !== "true") process.exit(0);
+
+// npm v9+ sets npm_config_global="true" for global installs. Some contexts (Hermes,
+// older npm, or wrapped installs) may not set it, so also check the prefix.
+const isGlobalInstall = process.env.npm_config_global === "true" || isLikelyGlobalPrefix(process.env.npm_config_prefix, process.env.HOME);
+if (!isGlobalInstall) process.exit(0);
 
 const prefix = process.env.npm_config_prefix;
-if (!prefix) process.exit(0);
+if (!prefix) {
+  console.log("offgrid-ai postinstall: npm global prefix not detected; skipping PATH update.");
+  process.exit(0);
+}
 
 if (isHermesPrefix(prefix, process.env.HOME)) {
   console.log("offgrid-ai installed with a Hermes-managed npm prefix.");
@@ -19,35 +27,80 @@ const marker = "# Added by offgrid-ai installer";
 const pathLine = `export PATH="${npmBin}:$PATH"`;
 const currentPath = process.env.PATH ?? "";
 
-if (currentPath.split(":").includes(npmBin)) process.exit(0);
+if (currentPath.split(":").includes(npmBin)) {
+  console.log(`offgrid-ai is already on PATH (${npmBin})`);
+  process.exit(0);
+}
 
 const home = process.env.HOME;
-if (!home) process.exit(0);
+if (!home) {
+  console.log("offgrid-ai postinstall: HOME not set; cannot update shell PATH.");
+  process.exit(0);
+}
 
+// Detect target shell config file. Prefer the file matching the user's shell.
 const shell = process.env.SHELL ?? "";
-const rcCandidates = shell.endsWith("zsh")
-  ? [join(home, ".zshrc"), join(home, ".zprofile"), join(home, ".profile")]
-  : [join(home, ".bashrc"), join(home, ".bash_profile"), join(home, ".profile"), join(home, ".zshrc")];
+const isZsh = /\bzsh$/u.test(shell);
+const isBash = /\bbash$/u.test(shell);
+
+let rcCandidates;
+if (isZsh) {
+  rcCandidates = [".zshrc", ".zprofile", ".profile"];
+} else if (isBash) {
+  rcCandidates = [".bashrc", ".bash_profile", ".profile"];
+} else {
+  // Fallback: try common files, prefer existing ones
+  rcCandidates = [".bashrc", ".bash_profile", ".zshrc", ".zprofile", ".profile"];
+}
+
+const rcPaths = rcCandidates.map((name) => join(home, name));
+let rcFile = rcPaths.find((file) => existsSync(file));
+if (!rcFile) {
+  // No existing rc file: create the one matching the user's shell.
+  const defaultName = isZsh ? ".zshrc" : isBash ? ".bashrc" : ".bashrc";
+  rcFile = join(home, defaultName);
+}
 
-const rcFile = rcCandidates.find((file) => existsSync(file)) ?? rcCandidates[0];
 let content = "";
 try {
   content = existsSync(rcFile) ? readFileSync(rcFile, "utf8") : "";
-} catch {
+} catch (err) {
+  console.log(`offgrid-ai postinstall: could not read ${rcFile}: ${err.message}`);
   process.exit(0);
 }
 
 if (!content.includes(npmBin)) {
-  appendFileSync(rcFile, `${content.endsWith("\n") || content.length === 0 ? "" : "\n"}\n${marker}\n${pathLine}\n`, "utf8");
-  console.log(`offgrid-ai added ${npmBin} to ${rcFile}`);
-  console.log(`Open a new terminal, or run: source ${rcFile}`);
+  try {
+    mkdirSync(dirname(rcFile), { recursive: true });
+    appendFileSync(rcFile, `${content.endsWith("\n") || content.length === 0 ? "" : "\n"}\n${marker}\n${pathLine}\n`, "utf8");
+    console.log(`offgrid-ai added ${npmBin} to ${rcFile}`);
+    console.log(`Open a new terminal, or run: source ${rcFile}`);
+  } catch (err) {
+    console.log(`offgrid-ai postinstall: could not write to ${rcFile}: ${err.message}`);
+  }
 } else {
   console.log(`offgrid-ai is installed in ${npmBin}`);
   console.log(`Open a new terminal if the command is not found yet.`);
 }
 
+if (process.getuid?.() === 0) {
+  console.log("Warning: offgrid-ai was installed as root. PATH was added to root's shell config, not the regular user's.");
+  console.log("If you installed with sudo, run the installer as your normal user instead, or manually add the line above to your user's shell config.");
+}
+
 function isHermesPrefix(prefix, home) {
   const normalized = prefix.replace(/\\/gu, "/");
   if (normalized.includes("/.hermes/")) return true;
   return Boolean(home && normalized === `${home.replace(/\\/gu, "/")}/.hermes/node`);
 }
+
+function isLikelyGlobalPrefix(prefix, home) {
+  if (!prefix || !home) return false;
+  const normalized = prefix.replace(/\\/gu, "/");
+  const homeNormalized = home.replace(/\\/gu, "/");
+  // Common global prefixes: /usr/local, /opt/homebrew, nvm paths, ~/.npm-global, ~/.local
+  return normalized.startsWith("/usr/") ||
+    normalized.startsWith("/opt/") ||
+    normalized.startsWith(homeNormalized) ||
+    /\/versions\/node\/v?\d/u.test(normalized);
+}

package/src/updates.mjs

@@ -20,10 +20,7 @@ export async function checkForUpdate({ now = Date.now(), fetchImpl = globalThis.
     if (cached.currentVersion === currentVersion) {
       return updateResult(currentVersion, cached.latestVersion);
     }
-    // Cache is from a different version — if latest isn't newer than current, no update
-    if (cached.latestVersion && !isNewerVersion(cached.latestVersion, currentVersion)) {
-      return null;
-    }
+    // Cache is from a different installed version — invalidate and refetch
   }
 
   try {