Fix nano version detection on macOS (pico)

- macOS: detect as "pico" instantly (skip --version which hangs)
- Linux: use --version for GNU nano version number
- Fallback: report "installed" if binary exists but version unknown
- Applied to both tool-versions and system-info endpoints

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: davindicode

app/routes/api/system-info.ts

@@ -37,7 +37,9 @@ export async function loader({ request }: Route.LoaderArgs) {
     shell: run("echo $SHELL") || run("echo %COMSPEC%") || null,
     node: process.version,
     tmux: run("tmux -V")?.replace(/^tmux\s+/i, "") || null,
-    nano: run("nano -V").match(/nano\s+([\d.]+)/i)?.[1] || null,
+    nano: os === "darwin"
+      ? (run("which nano") ? "pico" : null)
+      : (run("nano --version")?.match(/nano\s+([\d.]+)/i)?.[1] || (run("which nano") ? "installed" : null)),
     vim: run("vim --version").match(/Vi IMproved\s+([\d.]+)/)?.[1] || null,
     git: run("git --version")?.replace(/^git version\s+/i, "") || null,
     python: run("python3 --version")?.replace(/^Python\s+/i, "") || run("python --version")?.replace(/^Python\s+/i, "") || null,

app/routes/api/tool-versions.ts

@@ -1,11 +1,13 @@
 import { execSync } from "child_process";
+import { platform } from "os";
 import type { Route } from "./+types/tool-versions";
 
 function run(cmd: string): string | null {
   try {
     return execSync(cmd, { encoding: "utf-8", timeout: 2000, stdio: ["ignore", "pipe", "pipe"] }).trim();
-  } catch {
-    return null;
+  } catch (err: any) {
+    // Some commands write to stderr or exit non-zero but still have output
+    return err?.stdout?.trim() || err?.stderr?.trim() || null;
   }
 }
 
@@ -14,12 +16,17 @@ function getTmuxVersion(): string | null {
 }
 
 function getNanoVersion(): string | null {
-  // GNU nano supports --version; macOS pico doesn't (hangs), so use -V
-  // Try to extract from "GNU nano x.y" or similar
-  const out = run("nano -V");
-  if (!out) return null;
-  const match = out.match(/nano\s+([\d.]+)/i);
-  return match?.[1] || null;
+  // macOS ships pico instead of GNU nano — skip version check to avoid 2s hang
+  if (platform() === "darwin") {
+    return run("which nano") ? "pico" : null;
+  }
+  // GNU nano on Linux: --version prints to stdout and exits
+  const out = run("nano --version");
+  if (out) {
+    const match = out.match(/nano\s+([\d.]+)/i);
+    if (match) return match[1];
+  }
+  return run("which nano") ? "installed" : null;
 }
 
 function getVimVersion(): string | null {