github-search-cli 9.7.0 → 9.7.1

package/dist/cli.js

@@ -62,14 +62,14 @@ function flattenHtml(html) {
 }
 function parseRepoLink(block) {
   const anchor = block.match(
-    /<a[^>]*?href="\/([A-Za-z0-9._-]+)\/([^\"?#]+)"[^>]*class="Link"/
+    /<a[^>]*?href="\/([A-Za-z0-9._-]+)\/([^"?#]+)"[^>]*class="Link"/
   );
   if (!anchor) return void 0;
   return { owner: anchor[1], name: anchor[2] };
 }
 function parseStars(block) {
   const starsMatch = block.match(
-    /href="\/[^\/\"]+\/[^\/\"]+\/stargazers"[^>]*>.*?<\/svg>\s*(\d[\d,]*)/
+    /href="\/[^/"]+\/[^/"]+\/stargazers"[^>]*>.*?<\/svg>\s*(\d[\d,]*)/
   );
   return starsMatch ? parseInt(starsMatch[1].replace(/,/g, ""), 10) : 0;
 }
@@ -91,9 +91,8 @@ function parseTrendingHtml(html) {
   const flat = flattenHtml(html);
   const repos = [];
   const articleRe = /<article class="Box-row">(.*?)<\/article>/gi;
-  let match;
   let rank = 0;
-  while ((match = articleRe.exec(flat)) !== null) {
+  for (const match of flat.matchAll(articleRe)) {
     rank++;
     const block = match[1];
     const link = parseRepoLink(block);
@@ -240,7 +239,7 @@ function suggestFor(input) {
     return QUALIFIER_SUGGESTIONS[trimmed];
   }
   const matches = KNOWN_QUALIFIERS.filter((k) => k.startsWith(trimmed));
-  return matches.map((k) => k + ":");
+  return matches.map((k) => `${k}:`);
 }
 function tokenize(input) {
   const tokens = [];
@@ -353,7 +352,6 @@ function githubSortParam(sort) {
       return { sort: "forks", order: "desc" };
     case "updated":
       return { sort: "updated", order: "desc" };
-    case "best-match":
     default:
       return {};
   }
@@ -378,7 +376,6 @@ function rankRepos(repos, strategy) {
     case "updated":
       copy.sort(byKey((r) => Date.parse(r.updatedAt)));
       break;
-    case "best-match":
     default:
       break;
   }
@@ -585,6 +582,7 @@ function trendingRepoToRepo(r) {
 }
 var TrendingAdapter = class {
   name = "trending";
+  // biome-ignore lint/correctness/noUnusedPrivateClassMembers: part of the adapter constructor contract — callers pass a shared logger (search.ts:757) even though this adapter currently logs nothing itself.
   logger;
   constructor(logger2 = noopLogger) {
     this.logger = logger2;
@@ -740,12 +738,17 @@ import {
 } from "@opentui/core";
 
 // src/open-url.ts
+import { spawn } from "child_process";
 function openUrl(url) {
   try {
     const p = process.platform;
-    if (p === "win32") Bun.spawn(["cmd", "/c", "start", "", url]);
-    else if (p === "darwin") Bun.spawn(["open", url]);
-    else Bun.spawn(["xdg-open", url]);
+    const cmd = p === "win32" ? "explorer" : p === "darwin" ? "open" : "xdg-open";
+    if (typeof Bun !== "undefined") {
+      Bun.spawn([cmd, url]);
+      return;
+    }
+    const child = spawn(cmd, [url], { detached: true, stdio: "ignore" });
+    child.unref();
   } catch {
   }
 }
@@ -935,7 +938,7 @@ function formatPackageLine(pack) {
   const nameStr = `${pack.name}@${pack.version}`.padEnd(28).slice(0, 28);
   const dlStr = `\u2193 ${fmtCount(pack.downloads)}`.padEnd(20).slice(0, 20);
   const scoreStr = `score ${pack.score.toFixed(2)}`.padEnd(12).slice(0, 12);
-  const desc = (pack.description ?? "").length > 60 ? (pack.description ?? "").slice(0, 57) + "..." : pack.description ?? "";
+  const desc = (pack.description ?? "").length > 60 ? `${(pack.description ?? "").slice(0, 57)}...` : pack.description ?? "";
   const line1 = t`${bold(fg(C.nameText)(nameStr))}${fg(C.cyan)(dlStr)}${fg(C.gold)(scoreStr)}`;
   const line2 = t`${dim(fg(C.descText)(`     ${desc}`))}`;
   const newline = { text: "\n", __isChunk: true };
@@ -1198,9 +1201,13 @@ var results = [];
 function add(status, label, detail) {
   results.push({ status, label, detail });
 }
-function run(cmd, args) {
+function run(cmd, args, opts = {}) {
   try {
-    const res = spawnSync(cmd, args, { encoding: "utf8", timeout: 15e3 });
+    const res = spawnSync(cmd, args, {
+      encoding: "utf8",
+      timeout: 15e3,
+      ...opts
+    });
     return {
       status: res.status,
       stdout: res.stdout ?? "",
@@ -1375,6 +1382,97 @@ function checkDirs() {
     }
   }
 }
+var PKG_NAME = "github-search-cli";
+var BIN_NAME = "ghfind";
+function runningVersion() {
+  try {
+    const pkg = JSON.parse(
+      readFileSync(join(PKG_DIR, "package.json"), "utf8")
+    );
+    return pkg.version ?? "";
+  } catch {
+    return "";
+  }
+}
+function installVersionAt(shimPath) {
+  const dir = dirname(shimPath);
+  for (const ancestor of [dir, dirname(dir)]) {
+    for (const sub of ["node_modules", join("lib", "node_modules")]) {
+      const pkgJson = join(ancestor, sub, PKG_NAME, "package.json");
+      if (existsSync(pkgJson)) {
+        try {
+          const pkg = JSON.parse(readFileSync(pkgJson, "utf8"));
+          return { version: pkg.version ?? "", dir: dirname(pkgJson) };
+        } catch {
+        }
+      }
+    }
+  }
+  const res = run(shimPath, ["--version"], {
+    shell: platform() === "win32",
+    timeout: 1e4
+  });
+  const m = /\d+\.\d+\.\d+/.exec(res.stdout);
+  return m ? { version: m[0], dir } : null;
+}
+function checkInstallShadowing() {
+  const isWin = platform() === "win32";
+  const res = isWin ? run("where", [BIN_NAME]) : run("which", ["-a", BIN_NAME]);
+  const shims = [
+    ...new Set(
+      res.stdout.split(/\r?\n/).map((l) => l.trim()).filter((l) => l.length > 0)
+    )
+  ];
+  if (shims.length === 0) {
+    add("WARN", "Global installs", `${BIN_NAME} not found on PATH.`);
+    return;
+  }
+  const running = runningVersion();
+  const norm = (p) => isWin ? p.replace(/\\/g, "/").toLowerCase() : p;
+  const seenDirs = /* @__PURE__ */ new Set();
+  const stale = [];
+  let knownCount = 0;
+  let staleVersion = "";
+  let staleDir = "";
+  for (const shim of shims) {
+    const info = installVersionAt(shim);
+    const dirKey = norm(dirname(shim));
+    if (seenDirs.has(dirKey)) continue;
+    seenDirs.add(dirKey);
+    knownCount++;
+    const isRunningInstall = info !== null && norm(info.dir) === norm(PKG_DIR);
+    if (!isRunningInstall && info !== null && running && info.version !== running) {
+      stale.push(`${dirname(shim)} (v${info.version} \u2260 running v${running})`);
+      staleVersion = info.version;
+      staleDir = dirname(shim);
+    }
+  }
+  if (stale.length > 0 && knownCount > 1) {
+    add(
+      "WARN",
+      "Global installs",
+      `${knownCount} install(s) on PATH; stale copies shadowing updates \u2192 ${stale.join("; ")}. Remove with: npm uninstall -g ${PKG_NAME} --prefix <dir>`
+    );
+  } else if (stale.length > 0) {
+    add(
+      "PASS",
+      "Global installs",
+      `v${staleVersion} at ${staleDir} (running v${running || "?"} from source/build) \u2014 single install, nothing shadowed.`
+    );
+  } else if (knownCount > 1) {
+    add(
+      "PASS",
+      "Global installs",
+      `${knownCount} install(s) on PATH, all v${running || "?"}.`
+    );
+  } else {
+    add(
+      "PASS",
+      "Global installs",
+      `v${running || "?"} at ${dirname(shims[0])}.`
+    );
+  }
+}
 function printReport() {
   const noColor = process.env.NO_COLOR !== void 0 || !process.stdout.isTTY;
   const colors2 = (s, n) => {
@@ -1417,6 +1515,7 @@ async function runDoctor() {
   checkConfig();
   checkGithubToken();
   checkDirs();
+  checkInstallShadowing();
   printReport();
   const hasFail = results.some((r) => r.status === "FAIL");
   process.exitCode = hasFail ? 1 : 0;
@@ -1429,7 +1528,7 @@ import {
   version as nodeVersion,
   versions as nodeVersions
 } from "process";
-import { spawn } from "child_process";
+import { spawn as spawn2 } from "child_process";
 
 // src/version.ts
 import { readFileSync as readFileSync2 } from "fs";
@@ -1439,7 +1538,7 @@ var cached = null;
 function getVersion() {
   if (cached) return cached;
   if (true) {
-    cached = "9.7.0";
+    cached = "9.7.1";
     return cached;
   }
   try {
@@ -1515,14 +1614,15 @@ function tryOpenUrl(url) {
   try {
     const p = osPlatform;
     if (typeof Bun !== "undefined") {
-      if (p === "win32") Bun.spawn(["cmd", "/c", "start", "", url]);
-      else if (p === "darwin") Bun.spawn(["open", url]);
+      if (p === "win32") {
+        Bun.spawn(["explorer", url]);
+      } else if (p === "darwin") Bun.spawn(["open", url]);
       else Bun.spawn(["xdg-open", url]);
       return true;
     }
-    const cmd = p === "win32" ? "cmd" : p === "darwin" ? "open" : "xdg-open";
-    const args = p === "win32" ? ["/c", "start", "", url] : [url];
-    const child = spawn(cmd, args, { detached: true, stdio: "ignore" });
+    const cmd = p === "win32" ? "explorer" : p === "darwin" ? "open" : "xdg-open";
+    const args = [url];
+    const child = spawn2(cmd, args, { detached: true, stdio: "ignore" });
     child.unref();
     return true;
   } catch {
@@ -1543,11 +1643,11 @@ function reportError(err, context = {
   console.error("");
   const opened = tryOpenUrl(url);
   if (opened) {
-    console.error("  \u2192 Opened: " + url);
+    console.error(`  \u2192 Opened: ${url}`);
   } else {
     console.error("  Copy this URL to file a bug report:");
     console.error("");
-    console.error("    " + url);
+    console.error(`    ${url}`);
   }
   console.error("");
   return url;
@@ -2148,7 +2248,8 @@ function appendJSONL(filename, obj) {
   try {
     appendFileSync(
       join3(stateDir(), filename),
-      JSON.stringify(obj) + "\n",
+      `${JSON.stringify(obj)}
+`,
       "utf-8"
     );
   } catch {
@@ -2299,7 +2400,7 @@ async function fetchReleases(fullName, etag, token) {
     Accept: "application/vnd.github+json"
   };
   if (etag) headers["If-None-Match"] = etag;
-  if (token) headers["Authorization"] = `Bearer ${token}`;
+  if (token) headers.Authorization = `Bearer ${token}`;
   const res = await fetch(
     `https://api.github.com/repos/${fullName}/releases?per_page=${PER_REPO}`,
     { headers }
@@ -2536,7 +2637,7 @@ async function runInitWizard() {
   const themeStr = await ask(`  Theme [${themes.join("/")}] (tokyo-night): `);
   const config = {
     defaultSort: ["best-match", "stars", "updated", "forks"].includes(sortStr) ? sortStr : "best-match",
-    defaultLimit: Math.min(100, Math.max(10, parseInt(limitStr) || 50)),
+    defaultLimit: Math.min(100, Math.max(10, parseInt(limitStr, 10) || 50)),
     theme: themes.includes(themeStr) ? themeStr : themes[0],
     cacheTtlSeconds: 300,
     defaultTab: "search"
@@ -2585,18 +2686,18 @@ function buildComparisonTable(repos) {
   const header = ["".padEnd(14), ...names.map((n) => n.padEnd(colW))];
   const sep = ["".padEnd(14, "\u2500"), ...names.map(() => "".padEnd(colW, "\u2500"))];
   const lines = [
-    "\u250C" + sep.map((s) => "\u2500" + s + "\u2500").join("\u252C") + "\u2510",
-    "\u2502" + header.map((h) => ` ${h} `).join("\u2502") + "\u2502",
-    "\u251C" + sep.map((s) => "\u2500" + s + "\u2500").join("\u253C") + "\u2524"
+    `\u250C${sep.map((s) => `\u2500${s}\u2500`).join("\u252C")}\u2510`,
+    `\u2502${header.map((h) => ` ${h} `).join("\u2502")}\u2502`,
+    `\u251C${sep.map((s) => `\u2500${s}\u2500`).join("\u253C")}\u2524`
   ];
   for (const row of rows) {
     const cells = [
       row.label.padEnd(14),
       ...row.values.map((v) => v.padEnd(colW))
     ];
-    lines.push("\u2502" + cells.map((c) => ` ${c} `).join("\u2502") + "\u2502");
+    lines.push(`\u2502${cells.map((c) => ` ${c} `).join("\u2502")}\u2502`);
   }
-  lines.push("\u2514" + sep.map((s) => "\u2500" + s + "\u2500").join("\u2534") + "\u2518");
+  lines.push(`\u2514${sep.map((s) => `\u2500${s}\u2500`).join("\u2534")}\u2518`);
   return lines.join("\n");
 }
 function comparisonRepo(r) {
@@ -3074,7 +3175,8 @@ function deleteHistoryEntry(index) {
   );
   writeRaw(
     HISTORY_FILE,
-    remaining.reverse().map((e) => JSON.stringify(e)).join("\n") + "\n"
+    `${remaining.reverse().map((e) => JSON.stringify(e)).join("\n")}
+`
   );
 }
 function clearHistory() {
@@ -3086,7 +3188,8 @@ function rotateHistory() {
   const trimmed = entries.slice(0, MAX_ENTRIES);
   writeRaw(
     HISTORY_FILE,
-    trimmed.reverse().map((e) => JSON.stringify(e)).join("\n") + "\n"
+    `${trimmed.reverse().map((e) => JSON.stringify(e)).join("\n")}
+`
   );
 }
 
@@ -3171,12 +3274,12 @@ function renderMarkdown(md, opts = {}) {
     if (codeLines.length === 0) return;
     const maxLen = Math.max(...codeLines.map((l) => l.length), codeLang.length);
     const barW = Math.min(maxLen + 4, width - 2);
-    out.push("\u250C" + "\u2500".repeat(barW) + "\u2510");
-    if (codeLang) out.push("\u2502 " + codeLang.padEnd(barW - 1) + "\u2502");
+    out.push(`\u250C${"\u2500".repeat(barW)}\u2510`);
+    if (codeLang) out.push(`\u2502 ${codeLang.padEnd(barW - 1)}\u2502`);
     for (const cl of codeLines) {
-      out.push("\u2502 " + cl.padEnd(barW - 1) + "\u2502");
+      out.push(`\u2502 ${cl.padEnd(barW - 1)}\u2502`);
     }
-    out.push("\u2514" + "\u2500".repeat(barW) + "\u2518");
+    out.push(`\u2514${"\u2500".repeat(barW)}\u2518`);
     out.push("");
     codeLines = [];
     codeLang = "";
@@ -3198,16 +3301,16 @@ function renderMarkdown(md, opts = {}) {
     const totalW = colW.reduce((a, b) => a + b, 0) + colCount * 3 - 1;
     if (totalW > width) {
       for (const row of dataRows) {
-        out.push("  " + row.map((c) => c.trim()).join(" \u2502 "));
+        out.push(`  ${row.map((c) => c.trim()).join(" \u2502 ")}`);
       }
     } else {
-      const sep = "\u250C" + colW.map((w) => "\u2500".repeat(w + 2)).join("\u252C") + "\u2510";
-      const mid = "\u251C" + colW.map((w) => "\u2500".repeat(w + 2)).join("\u253C") + "\u2524";
-      const end = "\u2514" + colW.map((w) => "\u2500".repeat(w + 2)).join("\u2534") + "\u2518";
+      const sep = `\u250C${colW.map((w) => "\u2500".repeat(w + 2)).join("\u252C")}\u2510`;
+      const mid = `\u251C${colW.map((w) => "\u2500".repeat(w + 2)).join("\u253C")}\u2524`;
+      const end = `\u2514${colW.map((w) => "\u2500".repeat(w + 2)).join("\u2534")}\u2518`;
       out.push(sep);
       dataRows.forEach((row, i) => {
         const cells = colW.map((w, ci) => (row[ci] ?? "").trim().padEnd(w));
-        out.push("\u2502 " + cells.map((c) => ` ${c} `).join("\u2502") + "\u2502");
+        out.push(`\u2502 ${cells.map((c) => ` ${c} `).join("\u2502")}\u2502`);
         if (i === 0) out.push(mid);
       });
       out.push(end);
@@ -3238,7 +3341,7 @@ function renderMarkdown(md, opts = {}) {
     return parts;
   }
   for (let i = 0; i < lines.length; i++) {
-    let line = lines[i];
+    const line = lines[i];
     if (/^```/.test(line)) {
       if (inCodeBlock) {
         flushCode();
@@ -3283,7 +3386,7 @@ function renderMarkdown(md, opts = {}) {
     if (bqMatch) {
       const inner = renderInline(bqMatch[1]);
       for (const wl of wrapLine(inner, width - 4)) {
-        out.push("\u2502 " + wl);
+        out.push(`\u2502 ${wl}`);
       }
       continue;
     }
@@ -3291,7 +3394,7 @@ function renderMarkdown(md, opts = {}) {
     if (ulMatch) {
       const indent = ulMatch[1].length;
       const depth = Math.floor(indent / 2);
-      const prefix = "  ".repeat(depth) + "\u2022 ";
+      const prefix = `${"  ".repeat(depth)}\u2022 `;
       const inner = renderInline(ulMatch[2]);
       for (const wl of wrapLine(inner, width - prefix.length)) {
         out.push(prefix + wl);
@@ -3302,7 +3405,7 @@ function renderMarkdown(md, opts = {}) {
     if (olMatch) {
       const indent = olMatch[1].length;
       const depth = Math.floor(indent / 2);
-      const prefix = "  ".repeat(depth) + "\u25E6 ";
+      const prefix = `${"  ".repeat(depth)}\u25E6 `;
       const inner = renderInline(olMatch[2]);
       for (const wl of wrapLine(inner, width - prefix.length)) {
         out.push(prefix + wl);
@@ -3392,7 +3495,7 @@ var StatusManager = class {
 };
 
 // src/share.ts
-import { spawn as spawn2 } from "child_process";
+import { spawn as spawn3 } from "child_process";
 function formatShare(repo, format2) {
   switch (format2) {
     case "markdown":
@@ -3410,7 +3513,7 @@ async function copyToClipboard(text) {
     const p = process.platform;
     const cmd = p === "win32" ? ["clip"] : p === "darwin" ? ["pbcopy"] : await findLinuxClipboard();
     if (!cmd) return false;
-    const proc = spawn2(cmd[0], cmd.slice(1), {
+    const proc = spawn3(cmd[0], cmd.slice(1), {
       stdio: ["pipe", "ignore", "ignore"]
     });
     proc.stdin.write(text);
@@ -3473,9 +3576,10 @@ function nextTip() {
 
 // src/update-check.ts
 import { readFileSync as readFileSync5 } from "fs";
-import { join as join5, dirname as dirname3 } from "path";
+import { join as join5, dirname as dirname3, posix } from "path";
 import { fileURLToPath as fileURLToPath3 } from "url";
 var REGISTRY_URL = "https://registry.npmjs.org/github-search-cli/latest";
+var PKG_NAME2 = "github-search-cli";
 var CHECK_INTERVAL_MS = 24 * 60 * 60 * 1e3;
 var REQUEST_TIMEOUT_MS = 5e3;
 var __dirname2 = dirname3(fileURLToPath3(import.meta.url));
@@ -3560,30 +3664,55 @@ function suppressUpdateNotices() {
     suppressed: true
   });
 }
-async function performUpdate() {
-  const pkg = "github-search-cli";
-  const isBun = typeof process.versions !== "undefined" && "bun" in process.versions;
-  const cmd = isBun ? ["bun", "install", "-g", pkg] : ["npm", "install", "-g", pkg];
+function detectInstallLocation(pkgDir) {
+  const norm = pkgDir.split("\\").join("/");
+  const nodeModules = posix.dirname(norm);
+  if (posix.basename(nodeModules) !== "node_modules") return null;
+  const owner = posix.dirname(nodeModules);
+  if (/(^|\/)\.bun(\/|$)/.test(owner)) return { channel: "bun" };
+  const prefix = posix.basename(owner) === "lib" ? posix.dirname(owner) : owner;
+  return { channel: "npm", prefix };
+}
+function buildUpdateCommand(loc) {
+  if (loc.channel === "bun") return ["bun", "install", "-g", PKG_NAME2];
+  const args = ["npm", "install", "-g"];
+  if (loc.prefix) args.push("--prefix", loc.prefix);
+  args.push(PKG_NAME2);
+  return args;
+}
+function quoteWin32(arg) {
+  return /\s/.test(arg) ? `"${arg}"` : arg;
+}
+async function performUpdate(location = detectInstallLocation(PKG_DIR2)) {
+  if (!location) {
+    debugLog(
+      "Update skipped: the running install is not under a node_modules tree (compiled binary or dev checkout). Update it manually."
+    );
+    return false;
+  }
+  const argv = buildUpdateCommand(location);
   try {
-    if (isBun) {
-      const proc = Bun.spawn(cmd, {
-        stdout: "pipe",
-        stderr: "pipe"
-      });
-      await proc.exited;
-      if (proc.exitCode !== 0) {
-        const err = await proc.stderr.text();
-        debugLog(`Install failed (code ${proc.exitCode}): ${err}`);
-      }
-      return proc.exitCode === 0;
-    }
-    const { spawn: spawn3 } = await import("child_process");
-    return await new Promise((resolve) => {
-      const proc = spawn3(cmd[0], cmd.slice(1), { stdio: "ignore" });
-      proc.on("close", (code) => resolve(code === 0));
-      proc.on("error", () => resolve(false));
+    const { spawn: spawn4 } = await import("child_process");
+    const useShell = process.platform === "win32";
+    const proc = spawn4(
+      argv[0],
+      useShell ? argv.slice(1).map(quoteWin32) : argv.slice(1),
+      { stdio: ["ignore", "ignore", "pipe"], shell: useShell }
+    );
+    let stderr = "";
+    proc.stderr?.on("data", (chunk) => {
+      stderr += String(chunk);
     });
-  } catch {
+    const code = await new Promise((resolve) => {
+      proc.on("close", (c) => resolve(c));
+      proc.on("error", () => resolve(-1));
+    });
+    if (code !== 0) {
+      debugLog(`Install failed (code ${code}): ${stderr.trim()}`);
+    }
+    return code === 0;
+  } catch (e) {
+    debugLog(`Update failed: ${e instanceof Error ? e.message : String(e)}`);
     return false;
   }
 }
@@ -3603,7 +3732,7 @@ function fetchReleaseNotes(version) {
   try {
     const changelogPath = join5(PKG_DIR2, "CHANGELOG.md");
     const changelog = readFileSync5(changelogPath, "utf-8");
-    const header = "## [" + version + "]";
+    const header = `## [${version}]`;
     const start = changelog.indexOf(header);
     if (start === -1) return null;
     const bodyStart = start + header.length;
@@ -3747,8 +3876,8 @@ async function launchBrowser(theme_override) {
   let currentPage = 1;
   let totalCount = 0;
   let deepDiveActive = false;
-  let compareList = [];
-  let quitArmed = false;
+  const compareList = [];
+  const _quitArmed = false;
   const header = new TextRenderable4(renderer, {
     content: " ghfind \u2014 GitHub repo browser   [/]search  [Space]menu  [\u2192]open  [?]help  [q]uit",
     bg: colors.bg,
@@ -4611,6 +4740,7 @@ async function launchBrowser(theme_override) {
     if (menuStack.length <= 1) return false;
     const prev = menuStack[menuStack.length - 2];
     const current = menuStack.pop();
+    if (!current) return false;
     leaderSelect.options = prev.entries.map((e) => ({
       name: e.type === "category" ? `  ${e.icon} ${e.name} \u2192` : `  ${e.name}`,
       description: e.description,
@@ -4927,17 +5057,17 @@ async function launchBrowser(theme_override) {
     if (!cachedVersion) {
       cachedVersion = getVersion();
     }
-    const currentVersion2 = cachedVersion;
+    const currentVersion = cachedVersion;
     if (!state.lastInstalledVersion) return;
-    if (state.lastInstalledVersion === currentVersion2) return;
-    const notes = fetchReleaseNotes(currentVersion2);
+    if (state.lastInstalledVersion === currentVersion) return;
+    const notes = fetchReleaseNotes(currentVersion);
     const notesWidth = Math.max(
       30,
       Math.floor((process.stdout.columns || 80) * 0.55) - 8
     );
     const notesBlock = notes ? renderMarkdown(notes, { width: notesWidth }) : "  No release notes found for this version.";
     updateSelectedOption = 0;
-    updateText.content = `  Updated ghfind ${state.lastInstalledVersion} \u2192 ${currentVersion2}
+    updateText.content = `  Updated ghfind ${state.lastInstalledVersion} \u2192 ${currentVersion}
 
 ${notesBlock}`;
     renderUpdateOptions();
@@ -4948,8 +5078,8 @@ ${notesBlock}`;
     if (cachedVersion === null) {
       cachedVersion = getVersion();
     }
-    const currentVersion2 = cachedVersion;
-    const latest = process.env.DEBUG_FORCE_UPDATE || await checkForUpdate(currentVersion2);
+    const currentVersion = cachedVersion;
+    const latest = process.env.DEBUG_FORCE_UPDATE || await checkForUpdate(currentVersion);
     markUpdateChecked(latest ?? void 0);
     if (latest) {
       updateSelectedOption = 0;
@@ -4960,20 +5090,20 @@ ${notesBlock}`;
       );
       const notesBlock = notes ? renderMarkdown(notes, { width: notesWidth }) : `  A new version is available
 
-  Current: ${currentVersion2}
+  Current: ${currentVersion}
   Latest:   ${latest}
 
   Run "npm install -g ghfind"`;
       updateText.content = `  A new version is available
 
-  Current: ${currentVersion2}
+  Current: ${currentVersion}
   Latest:   ${latest}
 
 ${notesBlock}`;
       renderUpdateOptions();
       showOverlay("update");
     } else {
-      debugLog(`No update available (current: ${currentVersion2})`);
+      debugLog(`No update available (current: ${currentVersion})`);
     }
   }
   function openUrlIfSelected() {
@@ -5018,7 +5148,7 @@ ${notesBlock}`;
   }
   function saveCurrentSearch() {
     if (currentQueryInput) {
-      const name = currentQueryInput.length > 40 ? currentQueryInput.slice(0, 37) + "..." : currentQueryInput;
+      const name = currentQueryInput.length > 40 ? `${currentQueryInput.slice(0, 37)}...` : currentQueryInput;
       saveSearch(
         name,
         currentQueryInput,
@@ -5235,13 +5365,11 @@ ${notesBlock}`;
     statusBar.content = ` ${text}`;
     renderer.requestRender();
   });
-  if (config.theme !== void 0 || true) {
-    setTimeout(() => {
-      if (currentOverlay === "none") {
-        setStatus(nextTip());
-      }
-    }, 2e3);
-  }
+  setTimeout(() => {
+    if (currentOverlay === "none") {
+      setStatus(nextTip());
+    }
+  }, 2e3);
   function setStatus(msg) {
     statusMgr.set("idle", msg);
   }
@@ -5361,7 +5489,7 @@ ${notesBlock}`;
       if (idx >= 0) {
         const val = labelVals[idx];
         label = val >= 1e3 ? `${(val / 1e3).toFixed(1).replace(/\.0$/, "")}k` : String(val);
-        label = label.padStart(5) + " ";
+        label = `${label.padStart(5)} `;
       }
       lines.push(`${label}\u2506${chars.join("")}`);
     }
@@ -5795,7 +5923,8 @@ ${pack.description ?? ""}`;
         if (key.name === "d") {
           const sel = historySelect.getSelectedOption();
           if (sel?.value) {
-            deleteHistoryEntry(sel.value.index);
+            const idx = sel.value.index;
+            if (idx !== void 0) deleteHistoryEntry(idx);
             refreshHistory();
             setStatus("Deleted history entry");
           }
@@ -5814,6 +5943,7 @@ ${pack.description ?? ""}`;
           if (sel?.value) {
             const entry = sel.value.entry;
             showOverlay("none");
+            if (!entry) return;
             if (entry.mode === "trending") {
               if (entry.tab)
                 trendingTab = entry.tab;
@@ -5832,7 +5962,9 @@ ${pack.description ?? ""}`;
         if (key.name === "d") {
           const sel = bookmarksSelect.getSelectedOption();
           if (sel?.value) {
-            removeBookmark(sel.value.repo.fullName);
+            removeBookmark(
+              sel.value.repo?.fullName ?? ""
+            );
             refreshBookmarks();
             setStatus("Bookmark removed");
           }
@@ -5841,7 +5973,8 @@ ${pack.description ?? ""}`;
         }
         if (key.name === "enter" || key.name === "return") {
           const sel = bookmarksSelect.getSelectedOption();
-          if (sel?.value) openUrl(sel.value.repo.url);
+          if (sel?.value)
+            openUrl(sel.value.repo?.url ?? "");
           return;
         }
         return;
@@ -5850,7 +5983,7 @@ ${pack.description ?? ""}`;
         if (key.name === "d") {
           const sel = savedSelect.getSelectedOption();
           if (sel?.value) {
-            deleteSavedSearch(sel.value.name);
+            deleteSavedSearch(sel.value.name ?? "");
             refreshSavedSearches();
             setStatus("Saved search deleted");
           }
@@ -5885,7 +6018,7 @@ ${pack.description ?? ""}`;
           if (sel?.value) {
             const topic = sel.value;
             showOverlay("none");
-            searchInput.value = `topic:${topic.name}`;
+            searchInput.value = `topic:${topic.name ?? ""}`;
             showSearchMode();
             doSearch(`topic:${topic.name}`);
           }
@@ -6076,7 +6209,7 @@ ${pack.description ?? ""}`;
     }
     if (/^[1-5]$/.test(key.name)) {
       if (currentMode === "trending") {
-        const idx = parseInt(key.name) - 1;
+        const idx = parseInt(key.name, 10) - 1;
         if (idx >= 0 && idx < TAB_NAMES.length) {
           trendingTab = TAB_NAMES[idx];
           loadTrending();
@@ -6120,14 +6253,14 @@ ${pack.description ?? ""}`;
       return;
     }
   });
-  const currentVersion = getVersion();
-  if (!session || !session.mode) {
+  const _currentVersion = getVersion();
+  if (!session?.mode) {
     showLanding(true);
   }
   checkForUpdateAndShow();
   checkPostUpgradePanel();
   renderer.start();
-  if (session && session.mode) {
+  if (session?.mode) {
     if (currentMode === "trending") {
       loadTrending();
     } else {
@@ -6801,7 +6934,8 @@ async function processServerLine(line, state, config) {
   }
 }
 function writeStdout(line) {
-  process.stdout.write(line + "\n");
+  process.stdout.write(`${line}
+`);
 }
 async function runMcpServer(options = {}) {
   const state = {
@@ -6820,8 +6954,9 @@ async function runMcpServer(options = {}) {
   let buffer = "";
   process.stdin.on("data", (chunk) => {
     buffer += chunk;
-    let idx;
-    while ((idx = buffer.indexOf("\n")) >= 0) {
+    for (; ; ) {
+      const idx = buffer.indexOf("\n");
+      if (idx < 0) break;
       const line = buffer.slice(0, idx).trim();
       buffer = buffer.slice(idx + 1);
       if (!line) continue;
@@ -6879,7 +7014,7 @@ function parseArgs(args) {
     doctor: false,
     mcp: false
   };
-  let queryParts = [];
+  const queryParts = [];
   for (let i = 0; i < args.length; i++) {
     const arg = args[i];
     switch (arg) {
@@ -6933,8 +7068,8 @@ function parseArgs(args) {
         flags.login = true;
         break;
       case "--limit": {
-        const parsed = parseInt(args[++i]);
-        if (isNaN(parsed) || parsed < 1 || parsed > 100) {
+        const parsed = parseInt(args[++i], 10);
+        if (Number.isNaN(parsed) || parsed < 1 || parsed > 100) {
           console.error(`Invalid limit: must be 1-100. Using default 50.`);
           flags.limit = 50;
         } else {
@@ -7174,7 +7309,12 @@ function printAgentSkill(name) {
 }
 async function runUserProfile(flags, outputFormat) {
   try {
-    const profile = await fetchUserProfile(flags.user, {
+    const user = flags.user ?? "";
+    if (!user) {
+      console.error("Usage: ghfind user <name> [--json] [--token <t>]");
+      process.exit(1);
+    }
+    const profile = await fetchUserProfile(user, {
       token: flags.token,
       limit: flags.limit
     });
@@ -7196,7 +7336,12 @@ async function runUserProfile(flags, outputFormat) {
 }
 async function runOrgProfile(flags, outputFormat) {
   try {
-    const profile = await fetchOrgProfile(flags.org, {
+    const org = flags.org ?? "";
+    if (!org) {
+      console.error("Usage: ghfind org <name> [--json] [--token <t>]");
+      process.exit(1);
+    }
+    const profile = await fetchOrgProfile(org, {
       token: flags.token,
       limit: flags.limit
     });
@@ -7327,7 +7472,7 @@ async function runReleases(flags, outputFormat) {
 }
 async function runNonInteractive(flags, outputFormat) {
   const handlers = {
-    trending: async (ctx2) => {
+    trending: async (_ctx) => {
       const since = flags.since === "weekly" ? "weekly" : flags.since === "monthly" ? "monthly" : "daily";
       const trendingSearch = createTrendingSearch();
       let language;
@@ -7398,7 +7543,7 @@ async function runNonInteractive(flags, outputFormat) {
         token: flags.token
       });
       const repos = rankRepos(response.repos, flags.sort);
-      await pipeExec(repos, flags.pipe);
+      if (flags.pipe) await pipeExec(repos, flags.pipe);
     },
     format: async (ctx2) => {
       const response = await ctx2.provider.search(ctx2.parsed, {