@grimoire-rs/indexer 0.5.2 → 0.5.4

package/dist/renderer/astro/layouts/Base.astro

@@ -62,16 +62,18 @@ const { title, description, image = config.logo ?? config.favicon } = Astro.prop
 
       // Also before first paint: a deep link (a keyword chip on a package
       // page) or a stored preference means the catalog the server rendered —
-      // every package, empty search box, name order, deprecated hidden — is
-      // the wrong one. Flag it so CSS holds the catalog back rather than
-      // showing the full list and rearranging it a moment later. The island
-      // drops the flag once it has applied the view — which is the render
-      // after hydration, never the hydrating one itself; the timeout is the
-      // failsafe for an island that never hydrates, where a flash beats a
-      // blank page.
+      // every package, empty search box, name order, deprecated hidden, as
+      // cards — is the wrong one. Flag it so CSS holds the catalog back
+      // rather than showing the full list and rearranging it a moment
+      // later. The island drops the flag once it has applied the view —
+      // which is the render after hydration, never the hydrating one
+      // itself; the timeout is the failsafe for an island that never
+      // hydrates, where a flash beats a blank page.
       const view = new URLSearchParams(location.search);
-      const arranged = ["sort", "dir", "deprecated"].some((k) => pref("grim.catalog." + k));
-      if (view.get("q") || view.get("kind") || arranged) {
+      const arranged = ["sort", "dir", "deprecated", "view"].some((k) =>
+        pref("grim.catalog." + k),
+      );
+      if (view.get("q") || view.get("kind") || view.get("kw") || arranged) {
         document.documentElement.dataset.query = "";
         setTimeout(() => delete document.documentElement.dataset.query, 3000);
       }
@@ -102,6 +104,12 @@ const { title, description, image = config.logo ?? config.favicon } = Astro.prop
         for (const glyph of bar.querySelectorAll("[data-os-glyph]")) {
           glyph.hidden = glyph.dataset.osGlyph !== pick.dataset.osName;
         }
+        // The action segment follows the choice where it has one to follow:
+        // the registry bar's VS Code link carries the scope the command names,
+        // so picking Global cannot leave the button writing project config.
+        // A choice with no href leaves the link alone rather than blanking it.
+        const link = bar.querySelector("[data-os-link]");
+        if (link && pick.dataset.osHref) link.href = pick.dataset.osHref;
       };
 
       // The platform a bar should start on, or null for one this visitor's
@@ -853,12 +861,12 @@ const { title, description, image = config.logo ?? config.favicon } = Astro.prop
            that is not yet in use, so it withdraws the moment the field is
            focused or holds a query — at which point it would only be
            sitting on top of what is being typed. */
-        .search-hint {
+        .search-hint,
+        .search-clear {
           position: absolute;
           right: 0.5rem;
           top: 50%;
           translate: 0 -50%;
-          pointer-events: none;
           border: var(--grim-border-width) solid var(--grim-color-border);
           border-bottom-width: 2px;
           border-radius: var(--grim-radius-inset);
@@ -869,10 +877,33 @@ const { title, description, image = config.logo ?? config.favicon } = Astro.prop
           line-height: 1;
           padding: var(--grim-space-2) var(--grim-space-3);
         }
+        .search-hint {
+          pointer-events: none;
+        }
+        /* The two never coexist — the hint withdraws exactly when a query
+           appears, which is when the clear button is rendered — so they share
+           one corner rather than sitting beside each other. */
         .search-field:focus-within .search-hint,
         .search-field:has(input:not(:placeholder-shown)) .search-hint {
           display: none;
         }
+        /* Sized to the glyph rather than to a text line, so the box matches
+           the hint's height instead of growing around an SVG on a baseline. */
+        .search-clear {
+          display: grid;
+          place-items: center;
+          cursor: pointer;
+        }
+        .search-clear:hover {
+          color: var(--grim-color-fg);
+        }
+        /* The UA's own clear button, gone: it renders in this same corner in
+           Chromium and Safari, in a style that answers to no token here, and
+           it cannot be restyled beyond hiding. Ours replaces it. */
+        .controls input[type="search"]::-webkit-search-cancel-button {
+          appearance: none;
+          display: none;
+        }
         /* One line, always. Wrapping was the defect: at a narrow width the
            row dropped `deprecated` onto a second line, because no chip can
            shrink below its own longest word and the rail's chip count is a
@@ -1240,6 +1271,18 @@ const { title, description, image = config.logo ?? config.favicon } = Astro.prop
           outline-offset: 0;
           border-color: var(--grim-color-accent);
         }
+        /* The sort combo, while a POINTER is driving it. Chromium hands a
+           `<select>` `:focus-visible` on an ordinary click, so this control
+           alone would wear the keyboard ring for a mouse interaction — around
+           a closed box sitting behind its own open dropdown, which already
+           says everything the ring would. `data-pointer` is set by the
+           element's own `pointerdown` and cleared by a keypress or a blur, so
+           the ring is only ever suppressed for the interaction that did not
+           need it; every keyboard focus keeps it. */
+        .sort-field[data-pointer]:focus-visible {
+          outline: none;
+          border-color: var(--grim-color-accent);
+        }
 
         .grid {
           list-style: none;
@@ -1504,13 +1547,20 @@ const { title, description, image = config.logo ?? config.favicon } = Astro.prop
            the container rather than cutting square across it. */
         .table {
           display: grid;
-          /* logo | kind | name | description | updated. The two fixed-width
-             marks lead, so the names start on one line down the whole list
-             and the eye runs the column it actually reads. The name is
-             capped rather than `max-content`: an unbounded name ate the
-             description's space, and past the cap both clip instead. The
-             description is the only track that grows. */
-          grid-template-columns: auto auto minmax(8rem, 14rem) 1fr auto;
+          /* logo | kind | name | description | updated. The two marks lead,
+             so the names start on one line down the whole list and the eye
+             runs the column it actually reads.
+
+             Every track is sized to give the description whatever is left.
+             The marks are `min-content` — a 20px tile and a 16px glyph, and
+             nothing about them should reserve more than they draw. The name
+             is `fit-content(14rem)`: as wide as the longest name in the list
+             and no wider, capped, past which it clips. It was
+             `minmax(8rem, 14rem)`, and that 8rem floor was a hole — a list of
+             short names paid a fixed 8rem for them, so the description read
+             as pushed away from a column of empty space. */
+          grid-template-columns:
+            min-content min-content fit-content(14rem) 1fr auto;
           background: var(--grim-color-card);
           border: var(--grim-border-width) solid var(--grim-color-border);
           /* `xl`, the content-surface step — the same one the card wears.
@@ -1524,20 +1574,45 @@ const { title, description, image = config.logo ?? config.favicon } = Astro.prop
         /* One more track when the index publishes ratings. Decided once per
            index, never per filter — see `PackageTable`. */
         .table.rated {
-          grid-template-columns: auto auto minmax(8rem, 14rem) 1fr auto auto;
+          grid-template-columns:
+            min-content min-content fit-content(14rem) 1fr auto auto;
         }
         .table .row {
           grid-column: 1 / -1;
           display: grid;
           grid-template-columns: subgrid;
           align-items: center;
-          gap: var(--grim-space-5);
+          /* One step tighter than the row's own padding. A 12px gutter either
+             side of a 16px glyph reads as a hole rather than as spacing. */
+          gap: var(--grim-space-4);
           padding: var(--grim-space-3) var(--grim-space-5);
           color: inherit;
           text-decoration: none;
           font-size: var(--grim-text-sm);
           transition: background-color var(--grim-duration-base) ease;
         }
+        /* NO `content-visibility` on a row, and the reason is load-bearing.
+           `content-visibility: auto` applies SIZE CONTAINMENT, and a
+           size-contained element cannot be a subgrid — the `subgrid` above
+           degrades to `none`, silently. Measured in Chromium 152 against a
+           real build: with it, `getComputedStyle(row).gridTemplateColumns`
+           reads `none` and every cell is one sixth of the table (181px each
+           at a 1280px viewport); without it the parent's tracks apply and the
+           description gets the 832px the `1fr` exists to give it.
+
+           It shipped in 2a77c15 as a paint bound, and the check made at the
+           time — first and last rows' cells at identical offsets, before and
+           after scrolling — could not have caught this: when EVERY row is the
+           same independently-computed grid, the offsets match perfectly and
+           the alignment is fake. Any check here must read the row's own
+           `gridTemplateColumns`, or a cell's width against its content, never
+           two rows against each other. `style_contract.test.ts` holds the
+           text form of the invariant.
+
+           What that commit bought is not lost. The logo fetches it was aimed
+           at are held off by `loading="lazy"` on the image itself, and the
+           list's real cost — building every row — is bounded by the window in
+           `Catalog.tsx`, which is that same commit's other half. */
         /* The rule belongs between rows, not under the last one — which the
            container's own border already draws. */
         .table .row + .row {
@@ -1683,10 +1758,11 @@ const { title, description, image = config.logo ?? config.favicon } = Astro.prop
         @media (max-width: 40rem) {
           .table,
           .table.rated {
-            grid-template-columns: auto auto minmax(6rem, 1fr) auto;
+            grid-template-columns: min-content min-content minmax(6rem, 1fr) auto;
           }
           .table.rated {
-            grid-template-columns: auto auto minmax(6rem, 1fr) auto auto;
+            grid-template-columns:
+              min-content min-content minmax(6rem, 1fr) auto auto;
           }
           .t-desc {
             display: none;

package/dist/renderer/astro/lib/catalog.d.ts

@@ -1,5 +1,35 @@
 import type { CatalogPackage } from "../../types.js";
 export type { CatalogPackage } from "../../types.js";
+/**
+ * The fields the CATALOG ISLAND may read — and, because the island's props
+ * are serialized into the page, the only fields that are worth shipping.
+ *
+ * `index.astro` projects every package through `cardPackage()` before handing
+ * the array to `<Catalog>`. Astro serializes island props into an HTML
+ * attribute on `<astro-island>`, so anything left in a record is paid for
+ * twice on the landing page: once in the server-rendered card markup, and
+ * again as escaped JSON the browser parses at hydration. At a corporate-sized
+ * catalog the untrimmed record set was 323 KB of that attribute, ~18% of the
+ * page, most of it fields no card or row ever looks at — `license`,
+ * `authors`, `vendor`, `documentation`, `compatibility`, `revision`,
+ * `support`, `repository`, and every unknown enrichment key riding
+ * `IndexRecord`'s index signature.
+ *
+ * This is a TYPE, not just a filter, and that is the point: a component that
+ * reaches for a field not listed here fails to compile, rather than silently
+ * rendering `undefined` in the browser while the server render — which reads
+ * the full record — looks correct. The detail pages are unaffected; they read
+ * `data` directly and hydrate nothing.
+ */
+export type CardPackage = Pick<CatalogPackage, "name" | "kind" | "ref" | "namespace" | "description" | "summary" | "version" | "keywords" | "created" | "updated" | "deprecated" | "replacedBy" | "logo" | "rating">;
+/**
+ * One package, trimmed to what the island renders.
+ *
+ * An absent key is omitted rather than set to `undefined`: `JSON.stringify`
+ * drops an undefined value anyway, and omitting it keeps the projection's
+ * output identical to what a hand-written object literal would produce.
+ */
+export declare function cardPackage(p: CatalogPackage): CardPackage;
 /**
  * When a package last moved, for sorting and for the "updated" stamp.
  *
@@ -9,7 +39,7 @@ export type { CatalogPackage } from "../../types.js";
  * existed: those records still date correctly instead of dropping into the
  * unknown bucket on the first build after an upgrade.
  */
-export declare function lastUpdated(p: CatalogPackage): string | undefined;
+export declare function lastUpdated(p: Pick<CatalogPackage, "updated" | "created">): string | undefined;
 /**
  * The deprecation sentence, or `null` for a package that is not deprecated.
  *
@@ -57,6 +87,13 @@ export declare function vscodeVoteUrl(extension: string | null, ref: string): st
  * `vscode://<publisher.extension>/add-registry?index=<url>&alias=<name>` — the
  * one-click counterpart of the `grim config registry add` line beside it.
  *
+ * `scope` carries the Scope picker sitting in the same bar, so the button
+ * writes where the copyable command says it would. The extension clamps it
+ * rather than trusting it — with no folder open the write is global whatever
+ * the link asked for — and words its confirmation modal off the RESULT, so a
+ * page steering a machine-wide write is what the user reads before agreeing.
+ * Omitting it leaves the extension on its own host-derived default.
+ *
  * Null unless the link would actually work: the handler takes https only (an
  * index locator is fetched with whatever credentials the user configures for
  * it), refuses embedded credentials, and caps the URL at 2048 characters.
@@ -66,7 +103,7 @@ export declare function vscodeVoteUrl(extension: string | null, ref: string): st
 export declare function addRegistryUrl(extension: string | null, registry: {
     alias: string;
     index: string;
-} | null): string | null;
+} | null, scope?: "global" | "project"): string | null;
 /**
  * A bundle member's `id` resolved to the reference the index stores.
  *

package/dist/renderer/astro/lib/catalog.js

@@ -1,5 +1,37 @@
 // SPDX-License-Identifier: Apache-2.0
 // Copyright 2026 The Grimoire Authors
+/** The keys of `CardPackage`, as one list the projection actually iterates. */
+const CARD_FIELDS = [
+    "name",
+    "kind",
+    "ref",
+    "namespace",
+    "description",
+    "summary",
+    "version",
+    "keywords",
+    "created",
+    "updated",
+    "deprecated",
+    "replacedBy",
+    "logo",
+    "rating",
+];
+/**
+ * One package, trimmed to what the island renders.
+ *
+ * An absent key is omitted rather than set to `undefined`: `JSON.stringify`
+ * drops an undefined value anyway, and omitting it keeps the projection's
+ * output identical to what a hand-written object literal would produce.
+ */
+export function cardPackage(p) {
+    const out = {};
+    for (const key of CARD_FIELDS) {
+        if (p[key] !== undefined)
+            out[key] = p[key];
+    }
+    return out;
+}
 /**
  * When a package last moved, for sorting and for the "updated" stamp.
  *
@@ -151,13 +183,20 @@ const REGISTRY_ALIAS = /^[A-Za-z0-9][A-Za-z0-9_-]{0,31}$/;
  * `vscode://<publisher.extension>/add-registry?index=<url>&alias=<name>` — the
  * one-click counterpart of the `grim config registry add` line beside it.
  *
+ * `scope` carries the Scope picker sitting in the same bar, so the button
+ * writes where the copyable command says it would. The extension clamps it
+ * rather than trusting it — with no folder open the write is global whatever
+ * the link asked for — and words its confirmation modal off the RESULT, so a
+ * page steering a machine-wide write is what the user reads before agreeing.
+ * Omitting it leaves the extension on its own host-derived default.
+ *
  * Null unless the link would actually work: the handler takes https only (an
  * index locator is fetched with whatever credentials the user configures for
  * it), refuses embedded credentials, and caps the URL at 2048 characters.
  * Checking here keeps a broken button off the page rather than putting the
  * failure in the user's hands.
  */
-export function addRegistryUrl(extension, registry) {
+export function addRegistryUrl(extension, registry, scope) {
     if (!extension || !registry || !REGISTRY_ALIAS.test(registry.alias))
         return null;
     let url;
@@ -172,6 +211,8 @@ export function addRegistryUrl(extension, registry) {
     if (url.href.length > 2048)
         return null;
     const query = new URLSearchParams({ index: url.href, alias: registry.alias });
+    if (scope)
+        query.set("scope", scope);
     return `vscode://${extension}/add-registry?${query.toString()}`;
 }
 /**

package/dist/renderer/astro/lib/catalog.ts

@@ -9,6 +9,78 @@ import type { CatalogPackage } from "../../types.js";
 
 export type { CatalogPackage } from "../../types.js";
 
+/**
+ * The fields the CATALOG ISLAND may read — and, because the island's props
+ * are serialized into the page, the only fields that are worth shipping.
+ *
+ * `index.astro` projects every package through `cardPackage()` before handing
+ * the array to `<Catalog>`. Astro serializes island props into an HTML
+ * attribute on `<astro-island>`, so anything left in a record is paid for
+ * twice on the landing page: once in the server-rendered card markup, and
+ * again as escaped JSON the browser parses at hydration. At a corporate-sized
+ * catalog the untrimmed record set was 323 KB of that attribute, ~18% of the
+ * page, most of it fields no card or row ever looks at — `license`,
+ * `authors`, `vendor`, `documentation`, `compatibility`, `revision`,
+ * `support`, `repository`, and every unknown enrichment key riding
+ * `IndexRecord`'s index signature.
+ *
+ * This is a TYPE, not just a filter, and that is the point: a component that
+ * reaches for a field not listed here fails to compile, rather than silently
+ * rendering `undefined` in the browser while the server render — which reads
+ * the full record — looks correct. The detail pages are unaffected; they read
+ * `data` directly and hydrate nothing.
+ */
+export type CardPackage = Pick<
+  CatalogPackage,
+  | "name"
+  | "kind"
+  | "ref"
+  | "namespace"
+  | "description"
+  | "summary"
+  | "version"
+  | "keywords"
+  | "created"
+  | "updated"
+  | "deprecated"
+  | "replacedBy"
+  | "logo"
+  | "rating"
+>;
+
+/** The keys of `CardPackage`, as one list the projection actually iterates. */
+const CARD_FIELDS = [
+  "name",
+  "kind",
+  "ref",
+  "namespace",
+  "description",
+  "summary",
+  "version",
+  "keywords",
+  "created",
+  "updated",
+  "deprecated",
+  "replacedBy",
+  "logo",
+  "rating",
+] as const satisfies readonly (keyof CardPackage)[];
+
+/**
+ * One package, trimmed to what the island renders.
+ *
+ * An absent key is omitted rather than set to `undefined`: `JSON.stringify`
+ * drops an undefined value anyway, and omitting it keeps the projection's
+ * output identical to what a hand-written object literal would produce.
+ */
+export function cardPackage(p: CatalogPackage): CardPackage {
+  const out: Record<string, unknown> = {};
+  for (const key of CARD_FIELDS) {
+    if (p[key] !== undefined) out[key] = p[key];
+  }
+  return out as CardPackage;
+}
+
 /**
  * When a package last moved, for sorting and for the "updated" stamp.
  *
@@ -18,7 +90,7 @@ export type { CatalogPackage } from "../../types.js";
  * existed: those records still date correctly instead of dropping into the
  * unknown bucket on the first build after an upgrade.
  */
-export function lastUpdated(p: CatalogPackage): string | undefined {
+export function lastUpdated(p: Pick<CatalogPackage, "updated" | "created">): string | undefined {
   return p.updated ?? p.created;
 }
 
@@ -166,6 +238,13 @@ const REGISTRY_ALIAS = /^[A-Za-z0-9][A-Za-z0-9_-]{0,31}$/;
  * `vscode://<publisher.extension>/add-registry?index=<url>&alias=<name>` — the
  * one-click counterpart of the `grim config registry add` line beside it.
  *
+ * `scope` carries the Scope picker sitting in the same bar, so the button
+ * writes where the copyable command says it would. The extension clamps it
+ * rather than trusting it — with no folder open the write is global whatever
+ * the link asked for — and words its confirmation modal off the RESULT, so a
+ * page steering a machine-wide write is what the user reads before agreeing.
+ * Omitting it leaves the extension on its own host-derived default.
+ *
  * Null unless the link would actually work: the handler takes https only (an
  * index locator is fetched with whatever credentials the user configures for
  * it), refuses embedded credentials, and caps the URL at 2048 characters.
@@ -175,6 +254,7 @@ const REGISTRY_ALIAS = /^[A-Za-z0-9][A-Za-z0-9_-]{0,31}$/;
 export function addRegistryUrl(
   extension: string | null,
   registry: { alias: string; index: string } | null,
+  scope?: "global" | "project",
 ): string | null {
   if (!extension || !registry || !REGISTRY_ALIAS.test(registry.alias)) return null;
   let url: URL;
@@ -186,6 +266,7 @@ export function addRegistryUrl(
   if (url.protocol !== "https:" || url.username !== "" || url.password !== "") return null;
   if (url.href.length > 2048) return null;
   const query = new URLSearchParams({ index: url.href, alias: registry.alias });
+  if (scope) query.set("scope", scope);
   return `vscode://${extension}/add-registry?${query.toString()}`;
 }
 

package/dist/renderer/astro/lib/commands.d.ts

@@ -20,6 +20,13 @@ export interface Choice {
     Icon?: (props: {
         size?: number;
     }) => unknown;
+    /**
+     * Where the bar's action segment should point once this choice is picked.
+     * Only the registry bar has one — a deep link that has to agree with the
+     * scope the command beside it names — and a bar whose choices carry none
+     * leaves its action link exactly as rendered.
+     */
+    href?: string;
 }
 /**
  * The installer choices, one per platform rather than one per config row.

package/dist/renderer/astro/lib/commands.js

@@ -14,6 +14,7 @@
 //   <CommandBar choices={installChoices(data.config)} noun="install command" … />
 import { FolderRoot, Globe } from "lucide-preact";
 import { mdiApple, mdiConsole, mdiMicrosoftWindows, mdiPenguin, } from "@mdi/js";
+import { addRegistryUrl } from "./catalog.js";
 /**
  * `mdiPenguin`, not `mdiLinux`: Tux's own silhouette is a solid mass that
  * reads as an indistinct blob at 16px, where the plainer penguin keeps its
@@ -74,9 +75,15 @@ export function registryScopeChoices(config, registry = config.registry) {
     const add = registryAddCommand(config, registry);
     if (!add)
         return [];
+    const link = (scope) => addRegistryUrl(config.vscodeExtension, registry, scope) ?? undefined;
     return [
-        { name: "Global", command: `grim --global ${add.slice("grim ".length)}`, Icon: Globe },
-        { name: "Project", command: add, Icon: FolderRoot },
+        {
+            name: "Global",
+            command: `grim --global ${add.slice("grim ".length)}`,
+            Icon: Globe,
+            href: link("global"),
+        },
+        { name: "Project", command: add, Icon: FolderRoot, href: link("project") },
     ];
 }
 /**

package/dist/renderer/astro/lib/commands.ts

@@ -21,6 +21,7 @@ import {
   mdiPenguin,
 } from "@mdi/js";
 import type { RegistryHint, ResolvedSiteConfig } from "../../../config.js";
+import { addRegistryUrl } from "./catalog.js";
 
 // Re-exported so an index's own page can name the shape it passes to the two
 // functions below without reaching past `@grim/lib/*` into the package's
@@ -45,6 +46,13 @@ export interface Choice {
   path?: string;
   /** Lucide component, taking a `size` prop. */
   Icon?: (props: { size?: number }) => unknown;
+  /**
+   * Where the bar's action segment should point once this choice is picked.
+   * Only the registry bar has one — a deep link that has to agree with the
+   * scope the command beside it names — and a bar whose choices carry none
+   * leaves its action link exactly as rendered.
+   */
+  href?: string;
 }
 
 /**
@@ -115,9 +123,16 @@ export function registryScopeChoices(
 ): Choice[] {
   const add = registryAddCommand(config, registry);
   if (!add) return [];
+  const link = (scope: "global" | "project") =>
+    addRegistryUrl(config.vscodeExtension, registry, scope) ?? undefined;
   return [
-    { name: "Global", command: `grim --global ${add.slice("grim ".length)}`, Icon: Globe },
-    { name: "Project", command: add, Icon: FolderRoot },
+    {
+      name: "Global",
+      command: `grim --global ${add.slice("grim ".length)}`,
+      Icon: Globe,
+      href: link("global"),
+    },
+    { name: "Project", command: add, Icon: FolderRoot, href: link("project") },
   ];
 }
 

package/dist/renderer/astro/lib/search.d.ts

@@ -0,0 +1,17 @@
+/** Every package that matched, by `ref`, with its 0–1 score. */
+export type Scores = ReadonlyMap<string, number>;
+export interface SearchIndex {
+    /** Score one query against the whole catalog. Empty query, empty map. */
+    search(query: string): Scores;
+}
+/**
+ * Fetch the full catalog and build the matcher over it.
+ *
+ * Rejects rather than degrading quietly — the caller decides what a failure
+ * means, and for the island it means "keep the substring filter". Records
+ * without a string `ref` are dropped instead of failing the whole load: `ref`
+ * is how a hit is joined back onto the card the island already holds, so a
+ * record without one could never be shown even if it matched.
+ */
+export declare function loadSearchIndex(url: string): Promise<SearchIndex>;
+//# sourceMappingURL=search.d.ts.map