github-search-cli 9.4.2 → 9.4.3

package/src/history.ts

@@ -1,55 +1,69 @@
-/**
- * Persistent search history — append-only JSONL with rotation.
- */
-import type { HistoryEntry } from "./types";
-import { readJSONL, appendJSONL, writeJSON, writeRaw } from "./storage";
-
-const HISTORY_FILE = "history.jsonl";
-const MAX_ENTRIES = 500;
-
-/** Append one entry to the history log. */
-export function appendHistory(entry: HistoryEntry): void {
-  appendJSONL(HISTORY_FILE, entry);
-}
-
-/** Read all history entries, newest first, deduplicating consecutive identical queries. */
-export function readHistory(): HistoryEntry[] {
-  const entries = readJSONL<HistoryEntry>(HISTORY_FILE);
-  entries.reverse();
-  const deduped: HistoryEntry[] = [];
-  let lastQuery: string | undefined;
-  for (const e of entries) {
-    if (e.query !== lastQuery) {
-      deduped.push(e);
-      lastQuery = e.query;
-    }
-  }
-  return deduped;
-}
-
-/** Delete a single history entry by index (0 = newest). */
-export function deleteHistoryEntry(index: number): void {
-  const entries = readHistory();
-  if (index < 0 || index >= entries.length) return;
-  const removed = entries[index];
-  const remaining = entries.filter((e) => e.timestamp !== removed.timestamp || e.query !== removed.query);
-  writeRaw(HISTORY_FILE, remaining.reverse().map((e) => JSON.stringify(e)).join("\n") + "\n");
-}
-
-/** Clear all history. */
-export function clearHistory(): void {
-  writeRaw(HISTORY_FILE, "");
-}
-
-/** Rotate history to max entries (trim oldest). */
-export function rotateHistory(): void {
-  const entries = readHistory();
-  if (entries.length <= MAX_ENTRIES) return;
-  const trimmed = entries.slice(0, MAX_ENTRIES);
-  writeRaw(HISTORY_FILE, trimmed.reverse().map((e) => JSON.stringify(e)).join("\n") + "\n");
-}
-
-/** Count entries in history file. */
-export function historyCount(): number {
-  return readJSONL<HistoryEntry>(HISTORY_FILE).length;
-}
+/**
+ * Persistent search history — append-only JSONL with rotation.
+ */
+import type { HistoryEntry } from "./types";
+import { readJSONL, appendJSONL, writeJSON, writeRaw } from "./storage";
+
+const HISTORY_FILE = "history.jsonl";
+const MAX_ENTRIES = 500;
+
+/** Append one entry to the history log. */
+export function appendHistory(entry: HistoryEntry): void {
+  appendJSONL(HISTORY_FILE, entry);
+}
+
+/** Read all history entries, newest first, deduplicating consecutive identical queries. */
+export function readHistory(): HistoryEntry[] {
+  const entries = readJSONL<HistoryEntry>(HISTORY_FILE);
+  entries.reverse();
+  const deduped: HistoryEntry[] = [];
+  let lastQuery: string | undefined;
+  for (const e of entries) {
+    if (e.query !== lastQuery) {
+      deduped.push(e);
+      lastQuery = e.query;
+    }
+  }
+  return deduped;
+}
+
+/** Delete a single history entry by index (0 = newest). */
+export function deleteHistoryEntry(index: number): void {
+  const entries = readHistory();
+  if (index < 0 || index >= entries.length) return;
+  const removed = entries[index];
+  const remaining = entries.filter(
+    (e) => e.timestamp !== removed.timestamp || e.query !== removed.query,
+  );
+  writeRaw(
+    HISTORY_FILE,
+    remaining
+      .reverse()
+      .map((e) => JSON.stringify(e))
+      .join("\n") + "\n",
+  );
+}
+
+/** Clear all history. */
+export function clearHistory(): void {
+  writeRaw(HISTORY_FILE, "");
+}
+
+/** Rotate history to max entries (trim oldest). */
+export function rotateHistory(): void {
+  const entries = readHistory();
+  if (entries.length <= MAX_ENTRIES) return;
+  const trimmed = entries.slice(0, MAX_ENTRIES);
+  writeRaw(
+    HISTORY_FILE,
+    trimmed
+      .reverse()
+      .map((e) => JSON.stringify(e))
+      .join("\n") + "\n",
+  );
+}
+
+/** Count entries in history file. */
+export function historyCount(): number {
+  return readJSONL<HistoryEntry>(HISTORY_FILE).length;
+}

package/src/init.ts

@@ -1,55 +1,60 @@
-/**
- * `ghfind init` wizard — interactive one-time config setup.
- * 
- * ponytail: simple stdin prompts, no curses/readline UI.
- */
-import { createInterface } from "readline";
-import { saveConfig } from "./config";
-import { listThemes } from "./themes";
-import type { Config, SortStrategy } from "./types";
-
-function ask(query: string): Promise<string> {
-  const rl = createInterface({ input: process.stdin, output: process.stdout });
-  return new Promise((resolve) => {
-    rl.question(query, (answer) => {
-      rl.close();
-      resolve(answer.trim());
-    });
-  });
-}
-
-export async function runInitWizard(): Promise<void> {
-  console.log("");
-  console.log("  ghfind setup wizard");
-  console.log("  ──────────────────────");
-  console.log("");
-
-  const token = await ask("  GitHub token (optional, press Enter to skip): ");
-  if (token && !token.startsWith("ghp_") && !token.startsWith("github_pat_") && !token.startsWith("gho_")) {
-    console.log("  ⚠ Token doesn't look like a GitHub token (should start with ghp_, gho_, or github_pat_)");
-  }
-
-  const sortStr = await ask("  Default sort [best-match/stars/updated/forks] (best-match): ");
-  const limitStr = await ask("  Default result limit [10-100] (50): ");
-
-  const themes = listThemes();
-  const themeStr = await ask(`  Theme [${themes.join("/")}] (tokyo-night): `);
-
-  const config: Config = {
-    defaultSort: (["best-match", "stars", "updated", "forks"].includes(sortStr)
-      ? sortStr
-      : "best-match") as SortStrategy,
-    defaultLimit: Math.min(100, Math.max(10, parseInt(limitStr) || 50)),
-    theme: themes.includes(themeStr) ? themeStr : themes[0],
-    cacheTtlSeconds: 300,
-    defaultTab: "search",
-  };
-
-  if (token) config.githubToken = token;
-
-  saveConfig(config);
-  const { configPath } = await import("./config.ts");
-  console.log(`  ✓ Config saved to ${configPath()}`);
-  console.log("  Run 'ghfind' to start browsing!");
-  console.log("");
-}
+/**
+ * `ghfind init` wizard — interactive one-time config setup.
+ *
+ * ponytail: simple stdin prompts, no curses/readline UI.
+ */
+import { createInterface } from "readline";
+import { saveConfig } from "./config";
+import { isLikelyGithubToken } from "./login";
+import { listThemes } from "./themes";
+import type { Config, SortStrategy } from "./types";
+
+function ask(query: string): Promise<string> {
+  const rl = createInterface({ input: process.stdin, output: process.stdout });
+  return new Promise((resolve) => {
+    rl.question(query, (answer) => {
+      rl.close();
+      resolve(answer.trim());
+    });
+  });
+}
+
+export async function runInitWizard(): Promise<void> {
+  console.log("");
+  console.log("  ghfind setup wizard");
+  console.log("  ──────────────────────");
+  console.log("");
+
+  const token = await ask("  GitHub token (optional, press Enter to skip): ");
+  if (token && !isLikelyGithubToken(token)) {
+    console.log(
+      "  ⚠ Token doesn't look like a GitHub token (should start with ghp_, gho_, or github_pat_)",
+    );
+  }
+
+  const sortStr = await ask(
+    "  Default sort [best-match/stars/updated/forks] (best-match): ",
+  );
+  const limitStr = await ask("  Default result limit [10-100] (50): ");
+
+  const themes = listThemes();
+  const themeStr = await ask(`  Theme [${themes.join("/")}] (tokyo-night): `);
+
+  const config: Config = {
+    defaultSort: (["best-match", "stars", "updated", "forks"].includes(sortStr)
+      ? sortStr
+      : "best-match") as SortStrategy,
+    defaultLimit: Math.min(100, Math.max(10, parseInt(limitStr) || 50)),
+    theme: themes.includes(themeStr) ? themeStr : themes[0],
+    cacheTtlSeconds: 300,
+    defaultTab: "search",
+  };
+
+  if (token) config.githubToken = token;
+
+  saveConfig(config);
+  const { configPath } = await import("./config.ts");
+  console.log(`  ✓ Config saved to ${configPath()}`);
+  console.log("  Run 'ghfind' to start browsing!");
+  console.log("");
+}

package/src/landing.ts

@@ -1,104 +1,106 @@
-/** Landing screen pieces that are worth testing in isolation. */
-import { BoxRenderable, TextRenderable } from "@opentui/core";
-
-export interface LandingButtonColors {
-  accent: string;
-  border: string;
-  text: string;
-}
-
-/** Label for the landing-screen bookmarks button. */
-export function bookmarksButtonLabel(count: number): string {
-  if (count <= 0) return "\u{1F516}  Bookmarks  (none yet)";
-  if (count === 1) return "\u{1F516}  Bookmarks  (1 saved)";
-  return `\u{1F516}  Bookmarks  (${count} saved)`;
-}
-
-/**
- * Whether the landing screen should consume navigation keys.
- * False once another mode owns the screen (search, trending, packages,
- * or the bookmarks page) or an overlay (help, ...) is open on top.
- */
-export function landingKeysActive(mode: string, overlay: string): boolean {
-  return mode === "landing" && overlay === "none";
-}
-
-/** Keys the landing screen consumes; anything else falls through. */
-const LANDING_KEYS = new Set([
-  "up",
-  "k",
-  "down",
-  "j",
-  "enter",
-  "return",
-  "b",
-  "?",
-  "h",
-  "q",
-]);
-
-/**
- * Whether the landing screen consumes this key. Callers must stop propagation
- * for these, or the *same* event reaches the global handler next — e.g.
- * Enter opening the bookmarks page and then immediately being re-read as
- * "open the selected bookmark in a browser".
- */
-export function landingConsumesKey(name: string): boolean {
-  return LANDING_KEYS.has(name);
-}
-
-/** Clamped selection movement for the landing menu (no wrap-around). */
-export function moveSelection(
-  current: number,
-  delta: number,
-  total: number,
-): number {
-  if (total <= 0) return 0;
-  return Math.min(total - 1, Math.max(0, current + delta));
-}
-
-/**
- * Compact bordered button rendered under the landing mode cards.
- * Highlights like a card when selected; label carries the bookmark count.
- */
-export function createBookmarksButton(
-  renderer: ConstructorParameters<typeof BoxRenderable>[0],
-  colors: LandingButtonColors,
-) {
-  const box = new BoxRenderable(renderer, {
-    height: 3,
-    width: "60%",
-    marginTop: 1,
-    border: true,
-    borderColor: colors.border,
-    paddingX: 1,
-    alignItems: "center",
-    justifyContent: "center",
-    focusable: false,
-  });
-  const label = new TextRenderable(renderer, {
-    content: bookmarksButtonLabel(0),
-    color: colors.text,
-    height: 1,
-  });
-  box.add(label);
-
-  return {
-    box,
-    setCount(n: number): void {
-      label.content = bookmarksButtonLabel(n);
-    },
-    setSelected(selected: boolean): void {
-      box.borderColor = selected ? colors.accent : colors.border;
-      label.color = selected ? colors.accent : colors.text;
-    },
-    get labelText(): string {
-      return label.content as string;
-    },
-    get borderColor(): unknown {
-      return box.borderColor;
-    },
-  };
-}
-
-export type BookmarksButton = ReturnType<typeof createBookmarksButton>;
+/** Landing screen pieces that are worth testing in isolation. */
+import { BoxRenderable, TextRenderable } from "@opentui/core";
+
+export interface LandingButtonColors {
+  accent: string;
+  border: string;
+  text: string;
+}
+
+/** Label for the landing-screen bookmarks button. */
+export function bookmarksButtonLabel(count: number): string {
+  if (count <= 0) return "\u{1F516}  Bookmarks  (none yet)";
+  if (count === 1) return "\u{1F516}  Bookmarks  (1 saved)";
+  return `\u{1F516}  Bookmarks  (${count} saved)`;
+}
+
+/**
+ * Whether the landing screen should consume navigation keys.
+ * False once another mode owns the screen (search, trending, packages,
+ * or the bookmarks page) or an overlay (help, ...) is open on top.
+ */
+export function landingKeysActive(mode: string, overlay: string): boolean {
+  return mode === "landing" && overlay === "none";
+}
+
+/** Keys the landing screen consumes; anything else falls through. */
+const LANDING_KEYS = new Set([
+  "up",
+  "k",
+  "down",
+  "j",
+  "enter",
+  "return",
+  "b",
+  "?",
+  "h",
+  "q",
+]);
+
+/**
+ * Whether the landing screen consumes this key. Callers must stop propagation
+ * for these, or the *same* event reaches the global handler next — e.g.
+ * Enter opening the bookmarks page and then immediately being re-read as
+ * "open the selected bookmark in a browser".
+ */
+export function landingConsumesKey(name: string): boolean {
+  return LANDING_KEYS.has(name);
+}
+
+/** Clamped selection movement for the landing menu (no wrap-around). */
+export function moveSelection(
+  current: number,
+  delta: number,
+  total: number,
+): number {
+  if (total <= 0) return 0;
+  return Math.min(total - 1, Math.max(0, current + delta));
+}
+
+/**
+ * Compact bordered button rendered under the landing mode cards.
+ * Highlights like a card when selected; label carries the bookmark count.
+ */
+export function createBookmarksButton(
+  renderer: ConstructorParameters<typeof BoxRenderable>[0],
+  colors: LandingButtonColors,
+) {
+  const box = new BoxRenderable(renderer, {
+    height: 3,
+    width: "60%",
+    marginTop: 1,
+    border: true,
+    borderColor: colors.border,
+    paddingX: 1,
+    alignItems: "center",
+    justifyContent: "center",
+    focusable: false,
+  });
+  const label = new TextRenderable(renderer, {
+    content: bookmarksButtonLabel(0),
+    fg: colors.text,
+    height: 1,
+  });
+  box.add(label);
+  let currentLabel = bookmarksButtonLabel(0);
+
+  return {
+    box,
+    setCount(n: number): void {
+      currentLabel = bookmarksButtonLabel(n);
+      label.content = currentLabel;
+    },
+    setSelected(selected: boolean): void {
+      box.borderColor = selected ? colors.accent : colors.border;
+      label.fg = selected ? colors.accent : colors.text;
+    },
+    get labelText(): string {
+      return currentLabel;
+    },
+    get borderColor(): unknown {
+      return box.borderColor;
+    },
+  };
+}
+
+export type BookmarksButton = ReturnType<typeof createBookmarksButton>;

package/src/login.ts

@@ -0,0 +1,186 @@
+/**
+ * `ghfind login` — store a GitHub token for higher rate limits.
+ *
+ * Detects a token already managed by the GitHub CLI (`gh`) and offers to
+ * import it; otherwise accepts a pasted token. Writes to the same config file
+ * used by `ghfind init` (~/.config/ghfind/config.json). `GITHUB_TOKEN` in the
+ * environment always takes precedence at runtime (see src/config.ts).
+ */
+import { createInterface } from "readline";
+import { homedir } from "os";
+import { join } from "path";
+import { existsSync, readFileSync } from "fs";
+import { loadConfig, saveConfig, configPath } from "./config";
+import type { Config } from "./types";
+
+/** Prefixes that identify a GitHub-issued credential. */
+export const TOKEN_PREFIXES = ["ghp_", "gho_", "github_pat_"] as const;
+
+/** True when the string looks like a GitHub-issued token. */
+export function isLikelyGithubToken(token: string): boolean {
+  return TOKEN_PREFIXES.some((prefix) => token.startsWith(prefix));
+}
+
+/** Mask a token for display: `ghp_ab…wxyz`. */
+export function maskToken(token: string): string {
+  if (token.length <= 11) return `${token.slice(0, 4)}…`;
+  return `${token.slice(0, 7)}…${token.slice(-4)}`;
+}
+
+/** Directory where the `gh` CLI stores credentials. */
+export function ghCliConfigDir(): string {
+  return process.env.GH_CONFIG_DIR || join(homedir(), ".config", "gh");
+}
+
+/**
+ * Extract the `oauth_token` for a given host from `hosts.yml` (the gh CLI
+ * credential file). Top-level host keys are scanned line-by-line so tokens
+ * belonging to other hosts are never picked up.
+ */
+export function parseHostsYmlToken(
+  text: string,
+  host = "github.com",
+): string | undefined {
+  let inHost = false;
+  for (const line of text.split(/\r?\n/)) {
+    const trimmed = line.trim();
+    // Top-level key, e.g. `github.com:` or `github.com:\n` at column 0.
+    if (/^[A-Za-z0-9.-]+:\s*$/.test(trimmed)) {
+      inHost = trimmed.slice(0, -1).trim() === host;
+      continue;
+    }
+    if (inHost) {
+      const match = trimmed.match(/^oauth_token:\s*["']?([^"'\s]+)/);
+      if (match) return match[1];
+    }
+  }
+  return undefined;
+}
+
+/** Read the gh CLI token for `host` (default `github.com`) if one exists. */
+export function readGhCliToken(
+  ghDir: string = ghCliConfigDir(),
+  host = process.env.GH_HOST ?? "github.com",
+): string | undefined {
+  const hostsFile = join(ghDir, "hosts.yml");
+  if (!existsSync(hostsFile)) return undefined;
+  try {
+    return parseHostsYmlToken(readFileSync(hostsFile, "utf-8"), host);
+  } catch {
+    return undefined;
+  }
+}
+
+/** Write the token into the loaded config and persist it. */
+export function persistToken(
+  token: string,
+  deps: Pick<LoginDeps, "readConfig" | "writeConfig">,
+): void {
+  const config = deps.readConfig();
+  config.githubToken = token;
+  deps.writeConfig(config);
+}
+
+/** Interactive pieces of the wizard, injectable for testing. */
+export interface LoginDeps {
+  ask: (query: string) => Promise<string>;
+  readGhCliToken: () => string | undefined;
+  readConfig: () => Config;
+  writeConfig: (config: Config) => void;
+}
+
+/** Command-line yes/no + free-text prompt helper. */
+export function createAsk(): (query: string) => Promise<string> {
+  const rl = createInterface({ input: process.stdin, output: process.stdout });
+  return (query) =>
+    new Promise((resolve) => {
+      rl.question(query, (answer) => {
+        rl.close();
+        resolve(answer.trim());
+      });
+    });
+}
+
+/** Production defaults for the wizard dependencies. */
+export function defaultLoginDeps(): LoginDeps {
+  return {
+    ask: createAsk(),
+    readGhCliToken: () => readGhCliToken(),
+    readConfig: loadConfig,
+    writeConfig: (config) => saveConfig(config),
+  };
+}
+
+function successNote(): string[] {
+  return [
+    "  Authentication raises the search rate limit from 60 to",
+    "  5,000 requests/hour. Remove the token anytime by editing",
+    `  ${configPath()} — delete the "githubToken" line and save.`,
+  ];
+}
+
+/** `ghfind login` wizard: import from gh CLI or accept a pasted token. */
+export async function runLoginWizard(
+  deps: LoginDeps = defaultLoginDeps(),
+): Promise<void> {
+  const { ask } = deps;
+  console.log("");
+  console.log("  ghfind login");
+  console.log("  ────────────");
+  console.log("");
+
+  if (process.env.GITHUB_TOKEN) {
+    console.log(
+      "  ℹ GITHUB_TOKEN is set in the environment and already takes precedence.",
+    );
+    console.log("");
+  }
+
+  const current = deps.readConfig();
+  if (current.githubToken) {
+    console.log(
+      `  ✓ A token is already saved: ${maskToken(current.githubToken)}`,
+    );
+  }
+
+  const ghToken = deps.readGhCliToken();
+  if (ghToken) {
+    console.log("  ✓ Found a token managed by the GitHub CLI (gh).");
+    const answer = await ask("  Import it into ghfind? [Y/n]: ");
+    if (answer.toLowerCase() !== "n") {
+      persistToken(ghToken, deps);
+      finishLogin(ghToken);
+      return;
+    }
+  }
+
+  const pasted = await ask("  Paste a GitHub token (Enter to skip): ");
+  if (!pasted) {
+    console.log("  Skipped — no token saved.");
+    console.log("");
+    return;
+  }
+
+  if (!isLikelyGithubToken(pasted)) {
+    console.log(
+      `  ⚠ That doesn't look like a GitHub token (expected one of: ${TOKEN_PREFIXES.join(", ")}).`,
+    );
+    const confirm = await ask("  Save it anyway? [y/N]: ");
+    if (confirm.toLowerCase() !== "y") {
+      console.log("  Cancelled — nothing saved.");
+      console.log("");
+      return;
+    }
+  }
+
+  persistToken(pasted, deps);
+  finishLogin(pasted);
+}
+
+function finishLogin(token: string): void {
+  console.log(`  ✓ Token saved to ${configPath()}`);
+  console.log(`    (masked: ${maskToken(token)})`);
+  console.log("");
+  for (const line of successNote()) console.log(line);
+  console.log("");
+}

package/src/markdown-render.ts

@@ -49,12 +49,7 @@ export function renderMarkdown(md: string, opts: RenderOptions = {}): string {
     const colCount = Math.max(...dataRows.map((r) => r.length));
     const colW: number[] = [];
     for (let ci = 0; ci < colCount; ci++) {
-      colW.push(
-        Math.max(
-          3,
-          ...dataRows.map((r) => (r[ci] ?? "").length),
-        ),
-      );
+      colW.push(Math.max(3, ...dataRows.map((r) => (r[ci] ?? "").length)));
     }
     const totalW = colW.reduce((a, b) => a + b, 0) + colCount * 3 - 1;
     if (totalW > width) {