github-search-cli 9.8.0 → 9.8.2

package/src/storage.ts

@@ -2,16 +2,15 @@
  * Storage adapter — single point of file I/O for state files.
  *
  * All domain modules (history, bookmarks, saved-searches, notifications,
- * session, config) delegate file I/O here. Swap for an in-memory
- * implementation in tests.
+ * session, config) delegate file I/O here.
  */
 import {
-  existsSync,
   mkdirSync,
   readFileSync,
   writeFileSync,
   appendFileSync,
   unlinkSync,
+  renameSync,
 } from "node:fs";
 import { join } from "node:path";
 import { stateDir } from "./config";
@@ -31,10 +30,27 @@ export function readJSON<T>(filename: string, fallback: T): T {
   }
 }
 
+/** Write a file atomically: temp file in the same dir, then rename. */
+function writeFileAtomic(filename: string, data: string): void {
+  ensureDir();
+  const target = join(stateDir(), filename);
+  const tmp = `${target}.tmp-${process.pid}`;
+  try {
+    writeFileSync(tmp, data, "utf-8");
+    renameSync(tmp, target);
+  } catch (e) {
+    try {
+      unlinkSync(tmp);
+    } catch {
+      // temp already gone
+    }
+    throw e;
+  }
+}
+
 /** Write data as JSON to a file. */
 export function writeJSON(filename: string, data: unknown): void {
-  ensureDir();
-  writeFileSync(join(stateDir(), filename), JSON.stringify(data, null, 2));
+  writeFileAtomic(filename, JSON.stringify(data, null, 2));
 }
 
 /** Append one JSON object as a line to a JSONL file. */
@@ -68,8 +84,7 @@ export function readJSONL<T>(filename: string): T[] {
 
 /** Write raw text to a file (bypasses JSON encoding). */
 export function writeRaw(filename: string, data: string): void {
-  ensureDir();
-  writeFileSync(join(stateDir(), filename), data);
+  writeFileAtomic(filename, data);
 }
 
 /** Delete a file if it exists. */
@@ -94,43 +109,3 @@ export function debugLog(msg: string): void {
     /* ignore */
   }
 }
-
-/** Check if a file exists in the state dir. */
-export function fileExists(filename: string): boolean {
-  return existsSync(join(stateDir(), filename));
-}
-
-// ─── In-memory implementation for tests ────────────────────────────────
-
-const memStore = new Map<string, string>();
-
-export const memoryStorage = {
-  readJSON<T>(filename: string, fallback: T): T {
-    const raw = memStore.get(filename);
-    if (!raw) return fallback;
-    try {
-      return JSON.parse(raw) as T;
-    } catch {
-      return fallback;
-    }
-  },
-  writeJSON(filename: string, data: unknown): void {
-    memStore.set(filename, JSON.stringify(data, null, 2));
-  },
-  appendJSONL(filename: string, obj: unknown): void {
-    const existing = memStore.get(filename) ?? "";
-    memStore.set(filename, `${existing + JSON.stringify(obj)}\n`);
-  },
-  readJSONL<T>(filename: string): T[] {
-    const raw = memStore.get(filename);
-    if (!raw) return [];
-    return raw
-      .trim()
-      .split("\n")
-      .filter(Boolean)
-      .map((l) => JSON.parse(l) as T);
-  },
-  clear(): void {
-    memStore.clear();
-  },
-};

package/src/tui.ts

@@ -75,7 +75,8 @@ import { saveSession, restoreSession } from "./session";
 import { fetchDeepDive, buildDeepDiveText } from "./deepdive";
 import { buildComparisonTable } from "./compare";
 import { createMarkdownView } from "./markdown-view";
-import { fetchImageBytes, readmeBaseUrl } from "./image-loader";
+import { fetchImageBytes } from "./image-loader";
+import { fetchReadme } from "./readme";
 import { fetchTopics } from "./explore";
 import { exportToFile, type ExportFormat } from "./output";
 import {
@@ -1888,28 +1889,14 @@ export async function launchBrowser(theme_override?: string): Promise<void> {
     readmeScroll.scrollTop = 0;
     showOverlay("readme");
     try {
-      const headers: Record<string, string> = { "User-Agent": "ghfind/1.0" };
-      if (githubToken) headers.Authorization = `Bearer ${githubToken}`;
-      let text = "";
-      let sourceUrl = "";
-      for (const [branch, file] of [
-        ["main", "README.md"],
-        ["master", "README.md"],
-        ["main", "README.rst"],
-      ] as const) {
-        // The README's own URL doubles as the base for relative image paths.
-        const url = readmeBaseUrl(repo.owner, repo.name, branch, file);
-        const r = await fetch(url, { headers });
-        if (r.ok) {
-          text = await r.text();
-          sourceUrl = url;
-          break;
-        }
-      }
-      if (!text) {
-        readmeView.setContent(`_(no README found for ${repo.fullName})_`);
+      const readme = await fetchReadme(repo.owner, repo.name, {
+        token: githubToken,
+        private: repo.private,
+      });
+      if (readme) {
+        readmeView.setContent(readme.text, readme.sourceUrl);
       } else {
-        readmeView.setContent(text, sourceUrl);
+        readmeView.setContent(`_(no README found for ${repo.fullName})_`);
       }
     } catch {
       readmeView.setContent("_Failed to load README._");
@@ -2440,10 +2427,13 @@ export async function launchBrowser(theme_override?: string): Promise<void> {
         }
       }
     } catch (err) {
-      currentRepos = [];
-      resultsSelect.options = [
-        { name: " (error)", description: "", value: null },
-      ];
+      // A failed "load more" page must not wipe the pages already on screen.
+      if (!append) {
+        currentRepos = [];
+        resultsSelect.options = [
+          { name: " (error)", description: "", value: null },
+        ];
+      }
       const msg =
         err instanceof SearchCliError
           ? err.userMessage
@@ -3085,18 +3075,12 @@ ${pack.description ?? ""}`;
 // ── Utilities ─────────────────────────────────────────────────────────
 
 function formatResultLine(repo: Repo): string {
-  const stars = formatStars(repo.stars);
+  const stars = `★ ${fmtStars(repo.stars)}`;
   const lang = (repo.language ?? "?").padEnd(12).slice(0, 12);
   const name = repo.fullName.padEnd(40).slice(0, 40);
   return `${name}  ${lang}  ${stars}`;
 }
 
-function formatStars(n: number): string {
-  if (n >= 1000000) return `★ ${(n / 1000000).toFixed(1).replace(/\.0$/, "")}M`;
-  if (n >= 1000) return `★ ${(n / 1000).toFixed(1).replace(/\.0$/, "")}k`;
-  return `★ ${n}`;
-}
-
 function formatToolbar(
   sort: SortStrategy | PackageSortMode,
   limit: number,

package/src/user.ts

@@ -212,20 +212,21 @@ export async function fetchUserProfile(
   if (userPayload.public_repos > 0) {
     const perPage = Math.min(limit, 100);
     const maxPages = Math.ceil(limit / perPage);
-    for (let page = 1; page <= maxPages; page++) {
-      const url =
-        `https://api.github.com/users/${encodeURIComponent(name)}/repos` +
-        `?per_page=${perPage}&sort=pushed&page=${page}`;
-      const res = await fetch(url, { headers: apiHeaders(token) });
-      if (!res.ok) {
-        // User metadata succeeded; degraded summary beats a hard failure.
-        break;
-      }
-      const items = (await res.json()) as GitHubRepoPayload[];
-      if (!Array.isArray(items)) break; // unexpected shape — degrade gracefully
-      repos.push(...items);
-      if (repos.length >= limit || items.length < perPage) break;
-    }
+    const pages = await Promise.all(
+      Array.from({ length: maxPages }, (_, i) => i + 1).map(async (page) => {
+        const url =
+          `https://api.github.com/users/${encodeURIComponent(name)}/repos` +
+          `?per_page=${perPage}&sort=pushed&page=${page}`;
+        const res = await fetch(url, { headers: apiHeaders(token) });
+        if (!res.ok) {
+          // User metadata succeeded; degraded summary beats a hard failure.
+          return [];
+        }
+        const items = (await res.json()) as GitHubRepoPayload[];
+        return Array.isArray(items) ? items : []; // unexpected shape — skip
+      }),
+    );
+    repos.push(...pages.flat());
   }
 
   return normalizeUser(userPayload, repos.slice(0, limit));

package/dist/chunk-GYIL4S5X.js

@@ -1,59 +0,0 @@
-// src/config.ts
-import { homedir } from "os";
-import { join } from "path";
-import { mkdirSync, readFileSync, writeFileSync } from "fs";
-function stateDir() {
-  const xdg = process.env.XDG_STATE_HOME;
-  const base = xdg ? join(xdg, "ghfind") : join(homedir(), ".local", "share", "ghfind");
-  return base;
-}
-function cacheDir() {
-  const xdg = process.env.XDG_CACHE_HOME;
-  const base = xdg ? join(xdg, "ghfind") : join(homedir(), ".cache", "ghfind");
-  return base;
-}
-var DEFAULTS = {
-  defaultSort: "best-match",
-  defaultLimit: 50,
-  theme: "tokyo-night",
-  cacheTtlSeconds: 300,
-  defaultTab: "search"
-};
-function configPath() {
-  const envPath = process.env.GHFIND_CONFIG;
-  if (envPath) return envPath;
-  const xdg = process.env.XDG_CONFIG_HOME;
-  const base = xdg ? join(xdg, "ghfind") : join(homedir(), ".config", "ghfind");
-  return join(base, "config.json");
-}
-function loadConfig() {
-  const path = configPath();
-  try {
-    const raw = readFileSync(path, "utf-8");
-    const parsed = JSON.parse(raw);
-    return mergeWithEnv({ ...DEFAULTS, ...parsed });
-  } catch {
-    return mergeWithEnv({ ...DEFAULTS });
-  }
-}
-function saveConfig(config) {
-  const path = configPath();
-  const dir = path.slice(
-    0,
-    Math.max(path.lastIndexOf("/"), path.lastIndexOf("\\"))
-  );
-  mkdirSync(dir, { recursive: true });
-  writeFileSync(path, JSON.stringify(config, null, 2));
-}
-function mergeWithEnv(config) {
-  if (process.env.GITHUB_TOKEN) config.githubToken = process.env.GITHUB_TOKEN;
-  return config;
-}
-
-export {
-  stateDir,
-  cacheDir,
-  configPath,
-  loadConfig,
-  saveConfig
-};

package/dist/config-OCIPF2IF.js

@@ -1,14 +0,0 @@
-import {
-  cacheDir,
-  configPath,
-  loadConfig,
-  saveConfig,
-  stateDir
-} from "./chunk-GYIL4S5X.js";
-export {
-  cacheDir,
-  configPath,
-  loadConfig,
-  saveConfig,
-  stateDir
-};