github-search-cli 9.7.0 → 9.7.1

package/src/markdown-render.ts

@@ -26,12 +26,12 @@ export function renderMarkdown(md: string, opts: RenderOptions = {}): string {
     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("┌" + "─".repeat(barW) + "┐");
-    if (codeLang) out.push("│ " + codeLang.padEnd(barW - 1) + "│");
+    out.push(`┌${"─".repeat(barW)}┐`);
+    if (codeLang) out.push(`│ ${codeLang.padEnd(barW - 1)}│`);
     for (const cl of codeLines) {
-      out.push("│ " + cl.padEnd(barW - 1) + "│");
+      out.push(`│ ${cl.padEnd(barW - 1)}│`);
     }
-    out.push("└" + "─".repeat(barW) + "┘");
+    out.push(`└${"─".repeat(barW)}┘`);
     out.push("");
     codeLines = [];
     codeLang = "";
@@ -54,16 +54,16 @@ export function renderMarkdown(md: string, opts: RenderOptions = {}): string {
     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(" │ "));
+        out.push(`  ${row.map((c) => c.trim()).join(" │ ")}`);
       }
     } else {
-      const sep = "┌" + colW.map((w) => "─".repeat(w + 2)).join("┬") + "┐";
-      const mid = "├" + colW.map((w) => "─".repeat(w + 2)).join("┼") + "┤";
-      const end = "└" + colW.map((w) => "─".repeat(w + 2)).join("┴") + "┘";
+      const sep = `┌${colW.map((w) => "─".repeat(w + 2)).join("┬")}┐`;
+      const mid = `├${colW.map((w) => "─".repeat(w + 2)).join("┼")}┤`;
+      const end = `└${colW.map((w) => "─".repeat(w + 2)).join("┴")}┘`;
       out.push(sep);
       dataRows.forEach((row, i) => {
         const cells = colW.map((w, ci) => (row[ci] ?? "").trim().padEnd(w));
-        out.push("│ " + cells.map((c) => ` ${c} `).join("│") + "│");
+        out.push(`│ ${cells.map((c) => ` ${c} `).join("│")}│`);
         if (i === 0) out.push(mid);
       });
       out.push(end);
@@ -102,7 +102,7 @@ export function renderMarkdown(md: string, opts: RenderOptions = {}): string {
   }
 
   for (let i = 0; i < lines.length; i++) {
-    let line = lines[i];
+    const line = lines[i];
 
     // Code block boundaries
     if (/^```/.test(line)) {
@@ -160,7 +160,7 @@ export function renderMarkdown(md: string, opts: RenderOptions = {}): string {
     if (bqMatch) {
       const inner = renderInline(bqMatch[1]);
       for (const wl of wrapLine(inner, width - 4)) {
-        out.push("│ " + wl);
+        out.push(`│ ${wl}`);
       }
       continue;
     }
@@ -170,7 +170,7 @@ export function renderMarkdown(md: string, opts: RenderOptions = {}): string {
     if (ulMatch) {
       const indent = ulMatch[1].length;
       const depth = Math.floor(indent / 2);
-      const prefix = "  ".repeat(depth) + "• ";
+      const prefix = `${"  ".repeat(depth)}• `;
       const inner = renderInline(ulMatch[2]);
       for (const wl of wrapLine(inner, width - prefix.length)) {
         out.push(prefix + wl);
@@ -183,7 +183,7 @@ export function renderMarkdown(md: string, opts: RenderOptions = {}): string {
     if (olMatch) {
       const indent = olMatch[1].length;
       const depth = Math.floor(indent / 2);
-      const prefix = "  ".repeat(depth) + "◦ ";
+      const prefix = `${"  ".repeat(depth)}◦ `;
       const inner = renderInline(olMatch[2]);
       for (const wl of wrapLine(inner, width - prefix.length)) {
         out.push(prefix + wl);

package/src/mcp-server.ts

@@ -801,7 +801,7 @@ export async function processServerLine(
 // ─── stdio transport ───────────────────────────────────────────────────
 
 function writeStdout(line: string): void {
-  process.stdout.write(line + "\n");
+  process.stdout.write(`${line}\n`);
 }
 
 /**
@@ -831,8 +831,9 @@ export async function runMcpServer(
   let buffer = "";
   process.stdin.on("data", (chunk: string) => {
     buffer += chunk;
-    let idx: number;
-    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;

package/src/open-url.ts

@@ -1,15 +1,26 @@
 /**
- * Open a URL in the user's default browser — platform-aware.
+ * Open a URL in the user's default browser — platform- and runtime-aware.
  *
- * only if the open/xdg-open pattern fails somewhere.
+ * Windows note: `explorer.exe` is used instead of the classic
+ * `cmd /c start "" <url>` idiom. cmd splits the URL at the first unquoted
+ * `&` (a query-string separator) and executes the rest as a command, which
+ * truncated every URL with query params. explorer.exe receives the URL as a
+ * single CreateProcess argument, so `&` is never interpreted.
  */
+import { spawn } from "node:child_process";
 
 export function openUrl(url: string): void {
   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;
+    }
+    // Node fallback — detached so the child survives our exit.
+    const child = spawn(cmd, [url], { detached: true, stdio: "ignore" });
+    child.unref();
   } catch {
     // non-critical — user can copy the URL manually
   }

package/src/output.ts

@@ -5,7 +5,7 @@
  * clone-commands, ids) register in a single registry. One `format()` function
  * dispatches. `exportToFile` and `pipeExec` are methods on the same module.
  */
-import { writeFileSync } from "fs";
+import { writeFileSync } from "node:fs";
 import type { Repo } from "./types";
 import { openUrl } from "./open-url";
 

package/src/package-browser.ts

@@ -49,7 +49,7 @@ function formatPackageLine(pack: Package): StyledText {
   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 ?? "").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}`))}`;

package/src/releases.ts

@@ -1,16 +1,3 @@
-/**
- * Release tracker — checks bookmarked repos for new GitHub releases.
- *
- * Decisions (map 003):
- *   - Signal: new GitHub releases only (`GET /repos/{o}/{r}/releases?per_page=5`).
- *   - Seen state: `lastSeenAt` per bookmark; "new" = published after it.
- *   - Rate budget: etag conditional requests + token when present.
- *   - Prereleases shown (tagged `[pre]`), drafts never.
- *
- * ponytail: sequential fetches — fine for ≤100 bookmarks at launch;
- * parallelize with p-limit if users report slow startup.
- */
-import type { Bookmark } from "./types";
 import { readJSON, writeJSON } from "./storage";
 import { getBookmarks, getLastSeen, markSeen } from "./bookmarks";
 import { addNotification } from "./notifications";
@@ -68,7 +55,7 @@ async function fetchReleases(
     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 },

package/src/search-state.ts

@@ -4,7 +4,7 @@
  * The TUI view layer reads from SearchState and dispatches actions.
  * This module is testable without the OpenTUI renderer.
  */
-import type { Repo, SearchOptions, SortStrategy, SessionState } from "./types";
+import type { Repo, SortStrategy, SessionState } from "./types";
 import type { TabName } from "./trending";
 import { tabSince } from "./trending";
 import {
@@ -24,7 +24,6 @@ import {
   touchSavedSearch,
 } from "./saved-searches";
 import { fetchDeepDive, buildDeepDiveText } from "./deepdive";
-import { fetchTopics } from "./explore";
 import { loadConfig } from "./config";
 
 export type Overlay =
@@ -131,8 +130,6 @@ export class SearchStateManager {
         timestamp: Date.now(),
         resultCount: response.repos.length,
       });
-    } catch (err) {
-      throw err;
     } finally {
       this.state.isLoading = false;
     }

package/src/search.ts

@@ -181,7 +181,7 @@ export function suggestFor(input: string): string[] {
     return QUALIFIER_SUGGESTIONS[trimmed];
   }
   const matches = KNOWN_QUALIFIERS.filter((k) => k.startsWith(trimmed));
-  return matches.map((k) => k + ":");
+  return matches.map((k) => `${k}:`);
 }
 
 export function tokenize(input: string): string[] {
@@ -329,7 +329,6 @@ export function githubSortParam(sort: SortStrategy): {
       return { sort: "forks", order: "desc" };
     case "updated":
       return { sort: "updated", order: "desc" };
-    case "best-match":
     default:
       return {};
   }
@@ -359,7 +358,6 @@ export function rankRepos(repos: Repo[], strategy: SortStrategy): Repo[] {
     case "updated":
       copy.sort(byKey((r) => Date.parse(r.updatedAt)));
       break;
-    case "best-match":
     default:
       break;
   }
@@ -473,7 +471,7 @@ export class GitHubSearchAdapter implements SearchAdapter {
           if (
             !parsed ||
             typeof parsed !== "object" ||
-            !Array.isArray((parsed as any).items)
+            !Array.isArray((parsed as { items?: unknown }).items)
           ) {
             this.logger.error(
               `[github] unexpected response shape: ${JSON.stringify(parsed).slice(0, 200)}`,
@@ -630,6 +628,7 @@ function trendingRepoToRepo(r: RawTrendingRepo): Repo {
  */
 export class TrendingAdapter implements SearchAdapter {
   readonly 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.
   private readonly logger: Logger;
 
   constructor(logger: Logger = noopLogger) {

package/src/share.ts

@@ -19,7 +19,7 @@ export function formatShare(repo: Repo, format: ShareFormat): string {
 }
 
 /** Copy text to clipboard via platform command. */
-import { spawn } from "child_process";
+import { spawn } from "node:child_process";
 
 export async function copyToClipboard(text: string): Promise<boolean> {
   try {
@@ -46,7 +46,7 @@ export async function copyToClipboard(text: string): Promise<boolean> {
 }
 
 async function findLinuxClipboard(): Promise<string[] | null> {
-  const { execFileSync } = await import("child_process");
+  const { execFileSync } = await import("node:child_process");
   try {
     execFileSync("xclip", ["-version"], { stdio: "ignore" });
     return ["xclip", "-selection", "clipboard"];

package/src/storage.ts

@@ -12,8 +12,8 @@ import {
   writeFileSync,
   appendFileSync,
   unlinkSync,
-} from "fs";
-import { join } from "path";
+} from "node:fs";
+import { join } from "node:path";
 import { stateDir } from "./config";
 
 /** Ensure the state directory exists. */
@@ -43,7 +43,7 @@ export function appendJSONL(filename: string, obj: unknown): void {
   try {
     appendFileSync(
       join(stateDir(), filename),
-      JSON.stringify(obj) + "\n",
+      `${JSON.stringify(obj)}\n`,
       "utf-8",
     );
   } catch {
@@ -119,7 +119,7 @@ export const memoryStorage = {
   },
   appendJSONL(filename: string, obj: unknown): void {
     const existing = memStore.get(filename) ?? "";
-    memStore.set(filename, existing + JSON.stringify(obj) + "\n");
+    memStore.set(filename, `${existing + JSON.stringify(obj)}\n`);
   },
   readJSONL<T>(filename: string): T[] {
     const raw = memStore.get(filename);

package/src/themes.ts

@@ -242,6 +242,7 @@ export async function detectTerminalBackground(
     // Reply: ESC ] 11 ; <spec> (BEL | ESC \)
     const onData = (d: Buffer) => {
       buffer += d.toString("latin1");
+      // biome-ignore lint/suspicious/noControlCharactersInRegex: matches raw ANSI OSC-11 escape bytes, which are control characters by definition
       const m = buffer.match(/\x1b\]11;([^\x07\x1b]*)(?:\x07|\x1b\\)/);
       if (m) finish(parseOscColor(m[1]));
     };

package/src/trending-parser.ts

@@ -35,7 +35,7 @@ function parseRepoLink(block: string): RepoLink | undefined {
   // `/login?...&return_to=...` href fails the bare-path requirement and the
   // stargazers/forks links don't carry the heading's class="Link".
   const anchor = block.match(
-    /<a[^>]*?href="\/([A-Za-z0-9._-]+)\/([^\"?#]+)"[^>]*class="Link"/,
+    /<a[^>]*?href="\/([A-Za-z0-9._-]+)\/([^"?#]+)"[^>]*class="Link"/,
   );
   if (!anchor) return undefined;
   return { owner: anchor[1], name: anchor[2] };
@@ -43,7 +43,7 @@ function parseRepoLink(block: string): RepoLink | undefined {
 
 function parseStars(block: string): number {
   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;
 }
@@ -69,10 +69,9 @@ export function parseTrendingHtml(html: string): RawTrendingRepo[] {
   const flat = flattenHtml(html);
   const repos: RawTrendingRepo[] = [];
   const articleRe = /<article class="Box-row">(.*?)<\/article>/gi;
-  let match: RegExpExecArray | null;
   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);

package/src/trending.ts

@@ -118,13 +118,13 @@ export function formatRepoLine(repo: Repo, rank: number): StyledText {
   const rankStr = String(rank).padStart(2, "0");
   const full = `${repo.owner}/${repo.name}`;
   const lc = langColor(repo.language ?? "");
-  const rc = rankColor(rank);
+  const _rc = rankColor(rank);
   const star = "★";
   const arrow = repo.score > 0 ? "▲" : repo.score < 0 ? "▼" : "—";
   const growth = `${arrow} ${fmtSigned(repo.score)} today`;
   const desc =
     (repo.description ?? "").length > 60
-      ? (repo.description ?? "").slice(0, 57) + "..."
+      ? `${(repo.description ?? "").slice(0, 57)}...`
       : (repo.description ?? "");
   const line1 = t`${dim(fg(C.rankBg)(`[${rankStr}]`))} ${bold(fg("#c0caf5")(full))}  ${fg(lc)(`● ${repo.language}`)}  ${fg(C.gold)(`${star} ${fmtStars(repo.stars)}`)}  ${bold(fg(C.green)(growth))}`;
   const line2 = t`  ${dim(fg(C.muted)(desc))}`;
@@ -182,7 +182,7 @@ export async function launchTrending(): Promise<void> {
   let currentPeriod = "this week";
   let repos: Repo[] = [];
   let selectedRepoIdx = 0;
-  let isLoading = false;
+  let _isLoading = false;
   // ── Fetch from github.com/trending ──
   async function loadTab(index: number) {
     if (index < 0 || index >= TAB_NAMES.length) return;
@@ -194,7 +194,7 @@ export async function launchTrending(): Promise<void> {
         : since === "weekly"
           ? "this week"
           : "this month";
-    isLoading = true;
+    _isLoading = true;
     for (const rb of rowBoxes) scrollBox.remove(rb);
     rowBoxes.length = 0;
     rowTexts.length = 0;
@@ -233,7 +233,7 @@ export async function launchTrending(): Promise<void> {
       });
       scrollBox.add(errText);
     }
-    isLoading = false;
+    _isLoading = false;
     renderTabBar();
     renderer.requestRender();
   }
@@ -359,7 +359,7 @@ export async function launchTrending(): Promise<void> {
       const lc = langColor(r.language ?? "");
       const desc =
         (r.description ?? "").length > 55
-          ? (r.description ?? "").slice(0, 52) + "..."
+          ? `${(r.description ?? "").slice(0, 52)}...`
           : (r.description ?? "");
       const arrow = r.score > 0 ? "▲" : r.score < 0 ? "▼" : "—";
       const growthStr = `${arrow} ${fmtStars(Math.abs(r.score))} ${currentPeriod}`;
@@ -373,7 +373,7 @@ export async function launchTrending(): Promise<void> {
         fg: rc,
         height: 1,
       });
-      const descStr = desc.length > 60 ? desc.slice(0, 57) + "..." : desc;
+      const descStr = desc.length > 60 ? `${desc.slice(0, 57)}...` : desc;
       const pad = "     ";
       const line1 = t`${bold(fg("#c0caf5")(nameStr))}${fg(lc)(langStr)}${fg(C.muted)(starsStr)}${bold(fg(C.green)(growthStr))}`;
       const line2 = t`${fg(C.descText)(`${pad}${descStr}`)}`;

package/src/tui.ts

@@ -38,6 +38,8 @@ import type {
   SortStrategy,
   Package,
   SessionState,
+  HistoryEntry,
+  SavedSearch,
 } from "./types";
 import { fetchOrgProfile, type OrgProfile } from "./org";
 import {
@@ -54,13 +56,7 @@ import {
 import { tabSince, TAB_NAMES, fmtStars } from "./trending";
 import { loadConfig, saveConfig } from "./config";
 import { buildHelpSections, HELP_KEYS_COLUMN } from "./help";
-import {
-  SearchCliError,
-  NetworkError,
-  RateLimitError,
-  BadQueryError,
-  NoResultsError,
-} from "./errors";
+import { SearchCliError, NoResultsError } from "./errors";
 import {
   appendHistory,
   readHistory,
@@ -68,12 +64,7 @@ import {
   clearHistory,
   rotateHistory,
 } from "./history";
-import {
-  getBookmarks,
-  isBookmarked,
-  toggleBookmark,
-  removeBookmark,
-} from "./bookmarks";
+import { getBookmarks, toggleBookmark, removeBookmark } from "./bookmarks";
 import {
   getSavedSearches,
   saveSearch,
@@ -84,7 +75,7 @@ import { saveSession, restoreSession } from "./session";
 import { fetchDeepDive, buildDeepDiveText } from "./deepdive";
 import { buildComparisonTable } from "./compare";
 import { renderMarkdown } from "./markdown-render";
-import { fetchTopics, type TopicItem } from "./explore";
+import { fetchTopics } from "./explore";
 import { exportToFile, type ExportFormat } from "./output";
 import {
   listThemes,
@@ -269,8 +260,8 @@ export async function launchBrowser(theme_override?: string): Promise<void> {
   let currentPage = 1;
   let totalCount = 0;
   let deepDiveActive = false;
-  let compareList: Repo[] = [];
-  let quitArmed = false;
+  const compareList: Repo[] = [];
+  const _quitArmed = false;
 
   const header = new TextRenderable(renderer, {
     content:
@@ -1259,7 +1250,8 @@ export async function launchBrowser(theme_override?: string): Promise<void> {
   function popMenuLevel(): boolean {
     if (menuStack.length <= 1) return false;
     const prev = menuStack[menuStack.length - 2];
-    const current = menuStack.pop()!;
+    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}`,
@@ -1283,7 +1275,10 @@ export async function launchBrowser(theme_override?: string): Promise<void> {
   function buildMenuHierarchy(): MenuCategory[] {
     const hasRepo = (() => {
       const opt = resultsSelect.getSelectedOption();
-      return opt?.value && typeof (opt.value as any)?.fullName === "string";
+      return (
+        opt?.value &&
+        typeof (opt.value as { fullName?: unknown })?.fullName === "string"
+      );
     })();
 
     const groups: MenuCategory[] = [];
@@ -1694,7 +1689,7 @@ export async function launchBrowser(theme_override?: string): Promise<void> {
     if (currentQueryInput) {
       const name =
         currentQueryInput.length > 40
-          ? currentQueryInput.slice(0, 37) + "..."
+          ? `${currentQueryInput.slice(0, 37)}...`
           : currentQueryInput;
       saveSearch(
         name,
@@ -1925,16 +1920,11 @@ export async function launchBrowser(theme_override?: string): Promise<void> {
     statusBar.content = ` ${text}`;
     renderer.requestRender();
   });
-
-  // ── Startup tip ──
-  if (config.theme !== undefined || true) {
-    // Show a tip after 2s idle
-    setTimeout(() => {
-      if (currentOverlay === "none") {
-        setStatus(nextTip());
-      }
-    }, 2000);
-  }
+  setTimeout(() => {
+    if (currentOverlay === "none") {
+      setStatus(nextTip());
+    }
+  }, 2000);
 
   // ── Helper functions ────────────────────────────────────────────────
 
@@ -2097,7 +2087,7 @@ export async function launchBrowser(theme_override?: string): Promise<void> {
           val >= 1000
             ? `${(val / 1000).toFixed(1).replace(/\.0$/, "")}k`
             : String(val);
-        label = label.padStart(5) + " ";
+        label = `${label.padStart(5)} `;
       }
       lines.push(`${label}┆${chars.join("")}`);
     }
@@ -2624,7 +2614,8 @@ ${pack.description ?? ""}`;
         if (key.name === "d") {
           const sel = historySelect.getSelectedOption();
           if (sel?.value) {
-            deleteHistoryEntry((sel.value as any).index);
+            const idx = (sel.value as { index?: number }).index;
+            if (idx !== undefined) deleteHistoryEntry(idx);
             refreshHistory();
             setStatus("Deleted history entry");
           }
@@ -2641,8 +2632,9 @@ ${pack.description ?? ""}`;
         if (key.name === "enter" || key.name === "return") {
           const sel = historySelect.getSelectedOption();
           if (sel?.value) {
-            const entry = (sel.value as any).entry;
+            const entry = (sel.value as { entry?: HistoryEntry }).entry;
             showOverlay("none");
+            if (!entry) return;
             if (entry.mode === "trending") {
               if (entry.tab)
                 trendingTab = entry.tab as (typeof TAB_NAMES)[number];
@@ -2663,7 +2655,10 @@ ${pack.description ?? ""}`;
         if (key.name === "d") {
           const sel = bookmarksSelect.getSelectedOption();
           if (sel?.value) {
-            removeBookmark((sel.value as any).repo.fullName);
+            removeBookmark(
+              (sel.value as { repo?: { fullName: string } }).repo?.fullName ??
+                "",
+            );
             refreshBookmarks();
             setStatus("Bookmark removed");
           }
@@ -2672,7 +2667,8 @@ ${pack.description ?? ""}`;
         }
         if (key.name === "enter" || key.name === "return") {
           const sel = bookmarksSelect.getSelectedOption();
-          if (sel?.value) openUrl((sel.value as any).repo.url);
+          if (sel?.value)
+            openUrl((sel.value as { repo?: { url?: string } }).repo?.url ?? "");
           return;
         }
         return;
@@ -2683,7 +2679,7 @@ ${pack.description ?? ""}`;
         if (key.name === "d") {
           const sel = savedSelect.getSelectedOption();
           if (sel?.value) {
-            deleteSavedSearch((sel.value as any).name);
+            deleteSavedSearch((sel.value as { name?: string }).name ?? "");
             refreshSavedSearches();
             setStatus("Saved search deleted");
           }
@@ -2693,7 +2689,7 @@ ${pack.description ?? ""}`;
         if (key.name === "enter" || key.name === "return") {
           const sel = savedSelect.getSelectedOption();
           if (sel?.value) {
-            const s = sel.value as any;
+            const s = sel.value as SavedSearch;
             showOverlay("none");
             touchSavedSearch(s.name);
             if (s.mode === "trending") {
@@ -2718,9 +2714,9 @@ ${pack.description ?? ""}`;
         if (key.name === "enter" || key.name === "return") {
           const sel = topicsSelect.getSelectedOption();
           if (sel?.value) {
-            const topic = sel.value as any;
+            const topic = sel.value as { name?: string };
             showOverlay("none");
-            searchInput.value = `topic:${topic.name}`;
+            searchInput.value = `topic:${topic.name ?? ""}`;
             showSearchMode();
             doSearch(`topic:${topic.name}`);
           }
@@ -2973,7 +2969,7 @@ ${pack.description ?? ""}`;
     // Number keys 1-5 switch trending tabs (only in trending mode)
     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();
@@ -3025,16 +3021,16 @@ ${pack.description ?? ""}`;
   });
 
   // ── Update check (before start) ──────────────────────────────────────
-  const currentVersion = getVersion();
+  const _currentVersion = getVersion();
   // ── Start ──────────────────────────────────────────────────────────
   // Check for updates (non-blocking — doesn't delay TUI startup)
-  if (!session || !session.mode) {
+  if (!session?.mode) {
     showLanding(true);
   }
   checkForUpdateAndShow();
   checkPostUpgradePanel();
   renderer.start();
-  if (session && session.mode) {
+  if (session?.mode) {
     if (currentMode === "trending") {
       loadTrending();
     } else {