github-search-cli 9.8.2 → 9.8.3

package/CHANGELOG.md

@@ -6,6 +6,63 @@ Format follows [Keep a Changelog](https://keepachangelog.com/en/1.0.0/). Publish
 
 ---
 
+## [9.8.3] — 2026-10-01 (released)
+
+### Fixed
+
+- **The TUI always opens on the landing menu again.** A saved session used to
+  skip it on every launch, and one user's session had an MCP handshake request
+  saved as its query, so startup auto-ran a garbage search instead of showing
+  the menu. Sessions now restore preferences and pre-fill the last query only
+  when a mode is picked; they never auto-run a search. Both save and restore
+  sanitize the session: JSON-shaped or oversized queries are dropped, and
+  malformed fields fall back to defaults, so a corrupted file can't hijack
+  startup again.
+- **401/403 API errors now say what actually went wrong instead of "Network
+  error".** A rejected token (401) shows "Bad GitHub token (401)" with keys to
+  fix it; an authenticated-but-forbidden 403 (e.g. SAML/SSO enforcement) names
+  the SSO case; only genuine connectivity/server failures keep the generic
+  network message. A rate-limited search no longer swallows the error and
+  renders as a silent zero-result "success" — the real message reaches the
+  status bar, the MCP server reports it as structured backoff advice, and the
+  adapter no longer turns rate limits into empty results.
+- **Recover-from-errors without leaving the TUI.** While the matching error is
+  showing, `t` opens an inline token prompt (saves to config and re-runs the
+  failed query), `u` clears the token and retries unauthenticated, `c` swaps in
+  a different token after a rate limit, and `r` retries. Documented in the help
+  overlay and README.
+- **An image wrapped in a link now renders.** The badge-and-screenshot shape
+  `[![alt](shot.png)](url)` — used by `sst/opencode` (its only hero image was
+  lost) and `withastro/astro` (23 of 25 images lost) — was treated as prose
+  because the markdown token walk rejected `link` tokens instead of recursing
+  into them. The image now renders and keeps its link as an OSC 8 hyperlink on
+  the image cells, so clicking it in the terminal opens the same URL.
+- **A README full of small images no longer floods the screen.** Images whose
+  README declares an edge under 100 px (`<img width="48">` sponsor avatars,
+  badges) are filtered while parsing, before any request is made; images with
+  no declared size are measured from their header after the download
+  (`Bun.Image.metadata()`, header read only — no pixel decode) and rejected
+  before rasterising. On `openclaw/openclaw` (661 `<img>` tags, 648 of them
+  48×48 avatars) this takes the README from 649 downloads / 28.3 s to open to
+  1 download / ~0.5 s.
+- A pure-image block whose refs are all filtered out renders as nothing instead
+  of falling back to prose, which would have printed the raw markdown of the
+  filtered-out image.
+
+### Changed
+
+- **Images smaller than 100 px on any edge are no longer rendered, even when
+  the declared size understates the real one** (two images in
+  `openclaw/openclaw` declare `width="48"` but are 420×420). The README is
+  treated as the authority on size; the alternative — rendering the avatar at
+  whatever the terminal fits — is what made these READMEs unusable.
+- The TUI no longer skips the landing menu on startup: a saved session
+  restores sort/limit/trending tab and pre-fills the query when you pick Repo
+  Search, but the menu always comes first. See the session-sanitizing fix
+  above.
+
+---
+
 ## [9.8.2] — 2026-09-28 (released)
 
 ### Security

package/README.md

@@ -224,6 +224,15 @@ ghfind --completion fish | source
 | `Esc`                 | Close current overlay                          |
 | `q`                   | Quit ghfind                                    |
 
+**Error recovery** — active only while the matching error is in the status bar
+
+| Key | Action                                        |
+| --- | --------------------------------------------- |
+| `r` | Retry the failed search / trending load       |
+| `t` | Fix the GitHub token (401 bad token, 403 SSO) |
+| `u` | Search without a token (401)                  |
+| `c` | Change token (rate limit)                     |
+
 **Leader menu** — press `Space`, navigate with `↑`/`↓`, press `Enter` to dispatch
 
 Actions shown depend on current mode (search vs trending) and selection:
@@ -313,14 +322,14 @@ cd ghfind
 # With Bun (recommended for TUI)
 bun install
 bun start        # TUI
-bun test         # 482 tests
+bun test         # 516 tests
 bun run check    # format check + tests — the green gate
 bun run build    # build dist/
 
 # Or with npm
 npm install
 npm start        # TUI (requires Node 20+)
-npm test         # 414 tests
+npm test         # 516 tests
 npm run build    # build dist/
 
 > Want to contribute? See [CONTRIBUTING.md](CONTRIBUTING.md) for commands,
@@ -331,23 +340,22 @@ npm run build    # build dist/
 
 ## Known limitations
 
-**README image rendering is currently incomplete.** Images that do render look
-right, but several common cases silently show nothing:
-
-- **An image wrapped in a link does not render at all.** This is the common
-  badge-and-screenshot style: `[![alt](shot.png)](https://example.com)`.
-  The whole block is treated as prose and the image is dropped.
-- **An image smaller than 100px is skipped**, judged by the `width`/`height`
-  the README declares in its `<img>` tag. A README that understates an
-  image's size hides that image.
-- **SVG is not rendered** — the rasteriser cannot draw it.
-- Markdown `![](url)` images have no declared size, so they are downloaded
-  before the size check can reject them. A README full of them is slow to open.
-
-A README using none of the above — plain `![]()` images and a few large
-banners — renders fine and fast. See
+README image rendering has one deliberate blind spot and one hard one:
+
+- **Images smaller than 100 px on any edge are skipped.** For `<img>` tags the
+  declared `width`/`height` is trusted, so an image that understates its size
+  is hidden (a README that _overstates_ nothing is never affected). Markdown
+  `![]()` images declare no size, so they are measured from the image header
+  instead — still before any pixels are decoded.
+- **SVG is not rendered** — the rasteriser cannot draw it, so SVGs (and
+  badge-service images, which are tiny SVGs with text baked in) show an
+  `🖼 alt` caption instead.
+
+An image wrapped in a link — `[![alt](shot.png)](url)`, the badge-and-screenshot
+style — renders fine and keeps its link: the image cells carry an OSC 8
+hyperlink, so clicking it opens the target in your browser. See
 [.issues/tickets/readme-image-rendering.md](.issues/tickets/readme-image-rendering.md)
-for the measurements and the remaining work.
+for the architecture, measurements, and remaining work.
 
 ---
 
@@ -355,6 +363,15 @@ for the measurements and the remaining work.
 
 Recent releases (full history in [CHANGELOG.md](CHANGELOG.md)):
 
+### v9.8.3
+
+- **401/403 say what actually happened** — a rejected token and an authenticated-but-forbidden request (e.g. SAML/SSO enforcement) both used to report "Network error", naming a connectivity problem you didn't have; only genuine connectivity and server failures keep that message now
+- **A rate-limited search no longer looks like an empty one** — the adapter used to swallow the error and render a silent, successful "no repositories found"; the real message reaches the status bar, and the `ghfind_search_repos` MCP tool returns `rateLimited: true` with backoff advice instead of a tool failure
+- **Recover from errors without leaving the TUI** — while the matching error is in the status bar, `t` opens an inline token prompt (saves it and re-runs the failed query), `u` clears the token and retries unauthenticated, `c` swaps in a different token, `r` retries
+- **A corrupted session no longer hijacks startup** — a `session.json` holding an MCP handshake blob used to skip the landing menu and auto-run that garbage as a search on every launch; the menu always comes first now, and both save and restore sanitize the session
+- **Images wrapped in a link render now** — the badge-and-screenshot shape `[![alt](shot.png)](url)` rendered nothing at all, which cost `sst/opencode` its only image and `withastro/astro` 23 of its 25; the image cells carry an OSC 8 hyperlink, so clicking it opens the target
+- **A README full of small images no longer floods the screen** — images the README declares under 100px on any edge are skipped before a request is made, and undeclared `![]()` images are measured from the image header before any pixel decode; on `openclaw/openclaw` (648 sponsor avatars) that is 649 fetches / 28.3s → 1 fetch / ~0.5s
+
 ### v9.8.2
 
 - **Security: tokens no longer leak into crash reports** — an uncaught error auto-built a GitHub issue URL from raw `process.argv`, so a `--token` value ended up in a public URL; every field is redacted now, and ghfind prints the URL to copy instead of opening a browser when it finds a credential