@openparachute/hub 0.7.7-rc.8 → 0.7.7

package/src/hub.ts

@@ -541,43 +541,76 @@ const DISCOVERY_SCRIPT = `<script>
   /**
    * Render the "Get started" section (hub#342) above the Services grid.
    *
-   * One hardcoded target, conditional on its prerequisite being installed:
-   *   - "Open Notes" → /surface/notes/  (requires parachute-surface installed;
-   *     Surface auto-bootstraps Notes-as-UI per parachute-surface §17, so the
-   *     mere presence of Surface means /surface/notes/ is live)
+   * Driven by the uis[] sub-unit arrays the well-known doc already carries
+   * (W2-12 — retires the hardcoded "Open Notes" → /surface/notes/ tile):
+   * one tile per ACTIVE uis[] entry across every services[] row, each
+   * carrying its own displayName + tagline + mount path. Under the app's
+   * notes→parachute surface-identity rename this stays correct whichever
+   * mount an install serves — an existing notes-ui install renders its
+   * "notes" entry at /surface/notes, a post-rename install renders its
+   * "parachute" entry at /surface/parachute, no hub-side name knowledge
+   * required. Entries named "parachute" / "notes" sort first (they're the
+   * end-user "browse my content" CTA — the audience that needs the
+   * strongest above-the-fold prompt); the rest follow alphabetically.
    *
-   * The earlier "Browse Vault" tile retired in workstream C (2026-05-25)
-   * once vault declared uiUrl in its module.json (per patterns#96). With
-   * the multi-instance prefix lifted into buildWellKnown, every vault
-   * instance now renders its own tile in the Services grid below — one
-   * per instance, with the actual instance name in the discovery doc
-   * rather than a hub-side guess at "first vault."
-   *
-   * Notes stays here because Notes is the end-user CTA (the audience
-   * that needs the strongest above-the-fold prompt); the Services grid
-   * positions it equally with admin tiles, which underweights the
-   * "browse my content" path for a returning operator.
+   * Legacy fallback: a surface-host row that predates uis{}
+   * self-registration carries no uis[] even while it serves notes-ui at
+   * /surface/notes (the hub proxies /surface/* blind — mounts are
+   * surface-host's on-disk state, not the manifest's). When no uis[]
+   * entries exist anywhere but parachute-surface IS installed, keep
+   * rendering the pre-W2-12 "Open Notes" tile so those installs' discovery
+   * page is untouched. The fallback dies naturally the first time the
+   * surface-host re-registers with uis{}.
    *
-   * If the prerequisite is not met (fresh install pre-wizard, no app)
-   * the section stays hidden. The hardcoded shape mirrors the wizard's
-   * own done-screen "Start using your vault" tile — same architectural
-   * shape (single obvious entry point) at a different surface.
+   * The earlier "Browse Vault" tile retired in workstream C (2026-05-25)
+   * once vault declared uiUrl in its module.json (per patterns#96) — vault
+   * instances render in the Services grid below.
    *
-   * Not driven by /api/modules because discovery is unauth — the
-   * services list from /.well-known/parachute.json is sufficient
-   * (it carries the same install-detection signal we need, no
-   * Bearer required).
+   * If nothing qualifies (fresh install pre-wizard, no app) the section
+   * stays hidden. Not driven by /api/modules because discovery is unauth —
+   * the services list from /.well-known/parachute.json is sufficient (it
+   * carries the uis[] fan-out, no Bearer required).
    */
   function renderGetStarted(services) {
     if (!getStartedGrid || !getStartedSection) return;
-    const tiles = [];
-    const hasSurface = services.some((s) => s && s.name === 'parachute-surface');
-    if (hasSurface) {
-      tiles.push({
-        title: 'Open Notes',
-        desc: 'Browse + capture in the Notes app — reads from your vault.',
-        href: '/surface/notes/',
-      });
+    const uiEntries = [];
+    const seenPaths = new Set();
+    for (const svc of services) {
+      if (!svc || !Array.isArray(svc.uis)) continue;
+      for (const ui of svc.uis) {
+        if (!ui || typeof ui.path !== 'string' || ui.path === '') continue;
+        // Absent status means active (services-manifest.ts UiSubUnitStatus).
+        if (ui.status && ui.status !== 'active') continue;
+        // Multi-path parent rows repeat the same uis[] on every path —
+        // de-duplicate by mount (well-known.ts's documented consumer duty).
+        if (seenPaths.has(ui.path)) continue;
+        seenPaths.add(ui.path);
+        uiEntries.push(ui);
+      }
+    }
+    const rank = (ui) => (ui.name === 'parachute' ? 0 : ui.name === 'notes' ? 1 : 2);
+    uiEntries.sort(
+      (a, b) =>
+        rank(a) - rank(b) ||
+        (a.displayName || a.name || '').localeCompare(b.displayName || b.name || ''),
+    );
+    const tiles = uiEntries.map((ui) => ({
+      title: ui.displayName || ui.name || ui.path,
+      desc: ui.tagline || '',
+      // Trailing slash keeps relative asset URLs resolving under the mount
+      // (same reason the retired hardcode pointed at /surface/notes/).
+      href: ui.path.endsWith('/') ? ui.path : ui.path + '/',
+    }));
+    if (tiles.length === 0) {
+      // Legacy fallback (see docstring): pre-uis surface-host installs.
+      const hasSurface = services.some((s) => s && s.name === 'parachute-surface');
+      if (hasSurface) {
+        tiles.push({
+          title: 'Open Notes',
+          desc: 'Browse + capture in the Notes app — reads from your vault.',
+          href: '/surface/notes/',
+        });
+      }
     }
     if (tiles.length === 0) {
       getStartedSection.setAttribute('hidden', '');

package/src/module-ops-client.ts

@@ -376,7 +376,8 @@ export async function fetchModuleStates(deps: DriveModuleOpDeps): Promise<Module
   try {
     res = await doFetch(`${baseUrl}/api/modules`, {
       method: "GET",
-      // `/api/modules` parses the scheme-cased `Bearer ` prefix; match it exactly.
+      // `/api/modules` parses the Bearer scheme case-insensitively (RFC 7235,
+      // V1.4/C1.3 parity); we still emit the canonical `Bearer ` prefix.
       headers: { authorization: `Bearer ${bearer}` },
       // Bound the read so a wedged hub handler degrades `status` rather than
       // hanging it. AbortSignal.timeout fires `AbortError` once the ceiling

package/src/notes-serve.ts

@@ -1,15 +1,19 @@
 #!/usr/bin/env bun
 
 /**
- * Tiny static-file server for the @openparachute/notes PWA bundle.
+ * Tiny static-file server for a Parachute PWA bundle — originally written
+ * for @openparachute/notes, generalized in hub-parity P5 (2026-07-11) to
+ * also serve @openparachute/parachute-app (the super-surface front door,
+ * port 1944, mount `/app`) via the same shim. Any package matching this
+ * shape (a prebuilt SPA `dist/` with no server of its own) can reuse it.
  *
- * Notes is a SPA — no backend of its own. `parachute start notes` invokes
- * this shim with the installed `dist/` path so the PWA is served at a
- * known port and can be reverse-proxied by `parachute expose` alongside
- * the other services.
+ * A served bundle is a SPA — no backend of its own. `parachute start
+ * <svc>` invokes this shim with the installed `dist/` path so the PWA is
+ * served at a known port and can be reverse-proxied by `parachute expose`
+ * alongside the other services.
  *
  * Invoked as:
- *   bun <this-file> --port <n> [--dist <path>] [--mount <prefix>]
+ *   bun <this-file> --port <n> [--dist <path>] [--mount <prefix>] [--package <npmName>]
  *
  * `--mount` (default `/notes`) is the path prefix the reverse proxy hands
  * us. We strip it before resolving against `dist/` so a request for
@@ -17,27 +21,43 @@
  * `{dist}/notes/sw.js`. Without the strip, the SW + .webmanifest both
  * SPA-fall-back to index.html with content-type text/html, and the PWA
  * install prompt never fires. Pass `--mount ""` (or `--mount /`) when the
- * bundle is served at the origin root.
+ * bundle is served at the origin root. THIS IS LOAD-BEARING for every
+ * package served by this shim, not just notes — keep it exactly as-is.
  *
- * If --dist is omitted, we resolve @openparachute/notes's dist directory
- * via Bun.resolveSync. If that fails (package not installed globally, or
+ * `--package` (default `@openparachute/notes`, back-compat) names the npm
+ * package whose `dist/` we resolve when `--dist` is omitted. Passed by
+ * FIRST_PARTY_FALLBACKS entries whose startCmd composes this shim for a
+ * package other than notes (e.g. `app`'s `--package @openparachute/parachute-app`).
+ *
+ * If --dist is omitted, we resolve the package's dist directory via
+ * Bun.resolveSync. If that fails (package not installed globally, or
  * package doesn't ship dist/), exit 1 with a clear error.
+ *
+ * `/health` (post-mount-strip) always answers 2xx — the doctor/status
+ * probe (`probeModuleHealth`) only cares about the status code, but we
+ * answer explicitly rather than relying on the SPA-shell catch-all so a
+ * missing/corrupt `dist/index.html` can't take the health check down with it.
  */
 
 import { existsSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, join, resolve } from "node:path";
 
+/** Back-compat default — the shim's original (and still primary) consumer. */
+const DEFAULT_PACKAGE = "@openparachute/notes";
+
 interface Args {
   port: number;
   dist?: string;
   mount: string;
+  pkg: string;
 }
 
 function parseArgs(argv: string[]): Args {
   let port = 5173;
   let dist: string | undefined;
   let mount = "/notes";
+  let pkg = DEFAULT_PACKAGE;
   for (let i = 0; i < argv.length; i++) {
     const a = argv[i];
     if (a === "--port") {
@@ -56,11 +76,15 @@ function parseArgs(argv: string[]): Args {
       const v = argv[++i];
       if (v === undefined) throw new Error("--mount requires a value");
       mount = normalizeMount(v);
+    } else if (a === "--package") {
+      const v = argv[++i];
+      if (!v) throw new Error("--package requires a value");
+      pkg = v;
     } else {
       throw new Error(`unknown argument: ${a}`);
     }
   }
-  return { port, dist, mount };
+  return { port, dist, mount, pkg };
 }
 
 export function normalizeMount(raw: string): string {
@@ -70,22 +94,22 @@ export function normalizeMount(raw: string): string {
 
 /**
  * Candidate base directories that `Bun.resolveSync` walks from when looking
- * for `@openparachute/notes/package.json`. Order matters:
+ * for `<package>/package.json`. Order matters:
  *
- *   1. `process.cwd()` — works when notes-serve is invoked from inside the
- *      notes checkout (e.g. via `installDir` cwd in lifecycle.ts) or from
- *      any project that depends on `@openparachute/notes`.
+ *   1. `process.cwd()` — works when the shim is invoked from inside the
+ *      package's own checkout (e.g. via `installDir` cwd in lifecycle.ts) or
+ *      from any project that depends on the package.
  *   2. `~/.bun/install/global/node_modules` — modern Bun's global-install
- *      layout. This is where `bun add -g @openparachute/notes` lands the
- *      package, and where `bun link @openparachute/notes` symlinks it.
+ *      layout. This is where `bun add -g <package>` lands the package, and
+ *      where `bun link <package>` symlinks it.
  *   3. `~/.bun/install/global` — defensive fallback for older Bun layouts.
  *
- * Hub itself does NOT depend on `@openparachute/notes`, so when
- * `parachute start notes` is run from the hub repo dir, the cwd-relative
- * resolve walks ancestral node_modules and finds nothing. Bun does not
- * auto-consult the global install dir, so bun-linked installs fail to
- * resolve without (2)/(3). hub#194: Aaron hit silent 502 on tailnet
- * `/notes/` because of this — fixed by trying the global install dirs.
+ * Hub itself does NOT depend on the served package, so when `parachute
+ * start <svc>` is run from the hub repo dir, the cwd-relative resolve walks
+ * ancestral node_modules and finds nothing. Bun does not auto-consult the
+ * global install dir, so bun-linked installs fail to resolve without
+ * (2)/(3). hub#194: Aaron hit silent 502 on tailnet `/notes/` because of
+ * this — fixed by trying the global install dirs.
  *
  * Exported (and parameterized via `cwd`/`home`) so tests can drive the
  * resolution order against a real fixture install without monkey-patching
@@ -98,6 +122,8 @@ export function notesDistCandidates(cwd: string, home: string): string[] {
 export interface ResolveNotesDistDeps {
   cwd?: string;
   home?: string;
+  /** npm package name to resolve. Defaults to `@openparachute/notes` (back-compat). */
+  pkg?: string;
   /** Override `Bun.resolveSync` for tests. */
   resolveSync?: (specifier: string, base: string) => string;
   existsSync?: (path: string) => boolean;
@@ -106,6 +132,7 @@ export interface ResolveNotesDistDeps {
 export function resolveNotesDistFrom(deps: ResolveNotesDistDeps = {}): string {
   const cwd = deps.cwd ?? process.cwd();
   const home = deps.home ?? homedir();
+  const pkg = deps.pkg ?? DEFAULT_PACKAGE;
   const resolveSync = deps.resolveSync ?? Bun.resolveSync;
   const exists = deps.existsSync ?? existsSync;
   const candidates = notesDistCandidates(cwd, home);
@@ -113,7 +140,7 @@ export function resolveNotesDistFrom(deps: ResolveNotesDistDeps = {}): string {
   for (const base of candidates) {
     let pkgPath: string;
     try {
-      pkgPath = resolveSync("@openparachute/notes/package.json", base);
+      pkgPath = resolveSync(`${pkg}/package.json`, base);
     } catch (err) {
       resolveErrors.push(`  - ${base}: ${err instanceof Error ? err.message : String(err)}`);
       continue;
@@ -126,18 +153,18 @@ export function resolveNotesDistFrom(deps: ResolveNotesDistDeps = {}): string {
       // other candidates — they'd resolve to the same package and report
       // the same problem.
       throw new Error(
-        `@openparachute/notes resolved at ${root} has no dist/ directory at ${dist}. The package may not ship a prebuilt bundle — ask the notes maintainer to add a prepublishOnly build step.`,
+        `${pkg} resolved at ${root} has no dist/ directory at ${dist}. The package may not ship a prebuilt bundle — ask the ${pkg} maintainer to add a prepublishOnly build step.`,
       );
     }
     return dist;
   }
   throw new Error(
-    `Could not resolve @openparachute/notes from any of:\n${resolveErrors.join("\n")}\nIs the package installed? Try \`bun add -g @openparachute/notes\` or \`parachute install notes\`.`,
+    `Could not resolve ${pkg} from any of:\n${resolveErrors.join("\n")}\nIs the package installed? Try \`bun add -g ${pkg}\` or the matching \`parachute install <short>\`.`,
   );
 }
 
-function resolveNotesDist(): string {
-  return resolveNotesDistFrom();
+function resolveNotesDist(pkg: string): string {
+  return resolveNotesDistFrom({ pkg });
 }
 
 function mimeFor(path: string): string | undefined {
@@ -160,6 +187,17 @@ export function notesFetch(dist: string, mount: string): (req: Request) => Respo
     if (mount && (pathname === mount || pathname.startsWith(`${mount}/`))) {
       pathname = pathname.slice(mount.length) || "/";
     }
+    if (pathname === "/health") {
+      // Explicit rather than falling through to the SPA shell: the doctor /
+      // status probe (`probeModuleHealth`) only checks for a 2xx status, but
+      // answering directly means a missing/corrupt `dist/index.html` can't
+      // take the health check down with it. Every FIRST_PARTY_FALLBACKS
+      // entry served by this shim declares `health` under its own mount
+      // (e.g. `/notes/health`, `/app/health`) — this answers all of them.
+      return new Response(JSON.stringify({ status: "ok" }), {
+        headers: { "content-type": "application/json" },
+      });
+    }
     if (pathname === "/" || pathname.endsWith("/")) {
       return spaShell();
     }
@@ -200,17 +238,21 @@ export function notesServeOptions(
 }
 
 if (import.meta.main) {
-  const { port, dist: distArg, mount } = parseArgs(process.argv.slice(2));
+  const { port, dist: distArg, mount, pkg } = parseArgs(process.argv.slice(2));
 
   let dist: string;
   try {
-    dist = distArg ?? resolveNotesDist();
+    dist = distArg ?? resolveNotesDist(pkg);
   } catch (err) {
-    console.error(`parachute-notes-serve: ${err instanceof Error ? err.message : String(err)}`);
+    console.error(
+      `parachute-static-serve (${pkg}): ${err instanceof Error ? err.message : String(err)}`,
+    );
     process.exit(1);
   }
 
   Bun.serve(notesServeOptions(port, dist, mount));
 
-  console.log(`notes static-serve listening on :${port} (dist=${dist}, mount=${mount || "/"})`);
+  console.log(
+    `static-serve listening on :${port} (pkg=${pkg}, dist=${dist}, mount=${mount || "/"})`,
+  );
 }

package/src/oauth-handlers.ts

@@ -418,13 +418,9 @@ function oauthErrorRedirect(
  * Keyed on the SHORT name (resolved from each services.json row via
  * `shortNameForManifest`) rather than the raw manifest name so a row written
  * under a legacy manifest name still counts as installed — e.g. an un-upgraded
- * `parachute-channel` row aliases to short `agent` (channel→agent rename
- * 2026-06-17), so `agent:send` is still correctly advertised until the daemon
- * re-registers under `parachute-agent`.
  */
 const OPTIONAL_MODULE_SCOPES: ReadonlyArray<readonly [prefix: string, short: string]> = [
   ["scribe:", "scribe"],
-  ["agent:", "agent"],
   ["surface:", "surface"],
 ];
 
@@ -1903,13 +1899,7 @@ export async function handleApproveClientPost(
     // Headers logged are non-sensitive (Origin/Referer/Host are public);
     // the bound set is hub's own configuration. Body content not logged.
     console.warn(
-      `[oauth] approve POST same-origin check failed. headers: ` +
-        `origin=${JSON.stringify(req.headers.get("origin"))} ` +
-        `referer=${JSON.stringify(req.headers.get("referer"))} ` +
-        `host=${JSON.stringify(req.headers.get("host"))} ` +
-        `xff-host=${JSON.stringify(req.headers.get("x-forwarded-host"))} ` +
-        `xff-proto=${JSON.stringify(req.headers.get("x-forwarded-proto"))}. ` +
-        `bound origins: ${JSON.stringify(bound)}`,
+      `[oauth] approve POST same-origin check failed. headers: origin=${JSON.stringify(req.headers.get("origin"))} referer=${JSON.stringify(req.headers.get("referer"))} host=${JSON.stringify(req.headers.get("host"))} xff-host=${JSON.stringify(req.headers.get("x-forwarded-host"))} xff-proto=${JSON.stringify(req.headers.get("x-forwarded-proto"))}. bound origins: ${JSON.stringify(bound)}`,
     );
     return htmlError(
       "Cross-origin request rejected",

package/src/operator-token.ts

@@ -104,7 +104,6 @@ export const OPERATOR_TOKEN_SCOPE_SETS: Readonly<Record<OperatorScopeSet, readon
     "parachute:host:vault",
     "vault:admin",
     "scribe:admin",
-    "agent:send",
   ],
 };
 

package/src/origin-check.ts

@@ -198,8 +198,8 @@ const MUTATION_METHODS = new Set(["POST", "PUT", "PATCH", "DELETE"]);
  *
  * NOT belted, and why:
  *   - GET/HEAD/OPTIONS — read-shaped; the mint GETs
- *     (`/admin/host-admin-token`, `/admin/agent-token`,
- *     `/admin/module-token/<short>`, `/admin/vault-admin-token/<name>`)
+ *     (`/admin/host-admin-token`, `/admin/module-token/<short>`,
+ *     `/admin/vault-admin-token/<name>`)
  *     enforce GET-only with a 405 and their response bodies are unreadable
  *     cross-origin (no CORS on these routes).
  *   - Bearer-authed requests — a cross-site page cannot attach an

package/src/root-serve.ts

@@ -0,0 +1,156 @@
+#!/usr/bin/env bun
+
+/**
+ * Serving the Parachute app AT the hub's origin root (`root_mode = serve-app`).
+ *
+ * When an operator flips the root mode to `serve-app` (or a fresh hub installs
+ * the app during setup), the hub answers its own origin root with the installed
+ * `@openparachute/parachute-app` bundle instead of 302-ing to `/admin`. This is
+ * the self-hosted mirror of the hosted door: hit your box's URL, land in the
+ * app — no redirect hop.
+ *
+ * Why this is asset-correct with no rebuild: the app's Vite build is ROOT-based
+ * (absolute `/assets/*`, PWA scope `/`, OAuth redirect URIs at the origin root).
+ * The `/app` service mount serves the SAME dist through the `notes-serve.ts`
+ * shim with a `/app` prefix-strip; here we serve that identical dist verbatim at
+ * `/`, which is exactly the origin the build assumes. So `serveAppAtRoot` and
+ * the `/app` mount hand back byte-identical bundle files — the only difference
+ * is the URL prefix.
+ *
+ * Precedence is the caller's job (hub-server.ts): the fresh-hub setup wizard
+ * funnel and the pre-admin 503 lockout run BEFORE the root `/` handler, and
+ * every hub-owned route (/admin, /oauth, /.well-known, /vault, /git, /api,
+ * service mounts) dispatches BEFORE the SPA-fallback tail. This module only
+ * decides "given an already-unclaimed GET, does the app dist answer it?".
+ *
+ * NO chrome injection ever rides a root-served response — the app owns its whole
+ * page (same posture as the public/surface chrome opt-out). We build the
+ * responses here directly rather than routing through `decorateWithChrome`.
+ */
+
+import { existsSync, statSync } from "node:fs";
+import { join } from "node:path";
+import { resolveNotesDistFrom } from "./notes-serve.ts";
+
+/** The npm package whose built `dist/` is the app front door. */
+export const APP_PACKAGE = "@openparachute/parachute-app";
+
+/**
+ * Namespaces that keep the hub's branded 404 even in serve-app mode. These are
+ * hub / protocol surfaces, never app client-side routes, so an SPA shell there
+ * would be wrong — and would mask a genuine 404 for an API/OAuth/well-known
+ * typo. (Real dist files never live under these prefixes anyway; this makes the
+ * "don't shell it" decision explicit rather than incidental.)
+ */
+export const ROOT_SERVE_RESERVED_PREFIXES: readonly string[] = [
+  "/api/",
+  "/oauth/",
+  "/.well-known/",
+];
+
+/**
+ * Build a resolver for the installed app's `dist/` directory.
+ *
+ * Resolution goes through the SAME `resolveNotesDistFrom` machinery the `/app`
+ * mount's static-serve shim uses (so root-serve and `/app` resolve to one dist).
+ * A SUCCESSFUL resolution is memoized for the process lifetime — a resolved dist
+ * path doesn't move while the hub runs, and re-walking `Bun.resolveSync` on every
+ * asset request would be needless work. A FAILURE is NOT cached: an operator who
+ * set `serve-app` before installing the app, then runs `parachute install app`,
+ * is picked up on the next request without a hub restart (dynamic recovery).
+ *
+ * `resolve` is the test seam (defaults to the real package resolution). Return
+ * value is the dist dir, or `null` when the app can't be resolved (not installed
+ * / ships no `dist/`).
+ */
+export function makeAppDistResolver(
+  resolve: () => string = () => resolveNotesDistFrom({ pkg: APP_PACKAGE }),
+): () => string | null {
+  let cached: string | null = null;
+  return () => {
+    if (cached !== null) return cached;
+    try {
+      cached = resolve();
+      return cached;
+    } catch {
+      return null;
+    }
+  };
+}
+
+/** MIME overrides Bun.file doesn't infer (mirrors notes-serve.ts). */
+function mimeFor(path: string): string | undefined {
+  // Without this the PWA install prompt sees text/html for the manifest + bails.
+  if (path.endsWith(".webmanifest")) return "application/manifest+json";
+  return undefined;
+}
+
+/**
+ * Answer an already-unclaimed request from the app's root-based `dist/`, or
+ * return `null` to tell the caller to fall through to its normal handling
+ * (the `/` 302 fallback, or the branded 404 tail).
+ *
+ * Contract:
+ *   - non-GET                                     → null (fall through)
+ *   - a reserved hub/protocol prefix              → null (keep the branded 404)
+ *   - GET + an existing file under dist           → that file (asset-correct MIME)
+ *   - GET `/` or a trailing-slash "dir" request   → dist/index.html (SPA shell)
+ *   - GET + no file + Accept: text/html           → dist/index.html (SPA deep link)
+ *   - GET + no file + non-HTML (API probe, etc.)  → null (branded 404)
+ *
+ * `dist` is a resolved directory (from `makeAppDistResolver`); if its
+ * `index.html` has since vanished (app uninstalled after a successful resolve)
+ * we return `null` rather than serving a broken shell.
+ */
+export function serveAppAtRoot(dist: string, req: Request, pathname: string): Response | null {
+  // Only GET is served (HEAD/POST/etc. keep the caller's default — a non-GET to
+  // an unclaimed path stays a 404, and a non-GET `/` keeps its 302).
+  if (req.method !== "GET") return null;
+
+  // Hub/protocol namespaces are never the app's — leave their 404 branded.
+  for (const prefix of ROOT_SERVE_RESERVED_PREFIXES) {
+    if (pathname.startsWith(prefix)) return null;
+  }
+
+  const indexHtml = join(dist, "index.html");
+  // The resolved bundle lost its shell (uninstalled mid-flight) → fall through.
+  if (!existsSync(indexHtml)) return null;
+
+  const spaShell = () =>
+    new Response(Bun.file(indexHtml), {
+      headers: { "content-type": "text/html; charset=utf-8" },
+    });
+
+  // Bare root or a trailing-slash "directory" request → the SPA shell.
+  if (pathname === "/" || pathname.endsWith("/")) return spaShell();
+
+  // An existing dist file: /assets/*, /icon.svg, /manifest.webmanifest, /sw.js …
+  // Malformed percent-encoding (e.g. `/foo%`, `/assets/%ZZ`) makes
+  // `decodeURIComponent` throw a URIError. Catch it and fall through to the
+  // branded 404 (redirect-mode parity) — WITHOUT the guard the throw escapes to
+  // the dispatch outer catch, which classifies a non-DB error as "other" and
+  // re-throws → a bare 500. This path is reachable pre-auth on a public expose,
+  // so a garbage-encoded asset URL must 404, never 500.
+  let decodedPath: string;
+  try {
+    decodedPath = decodeURIComponent(pathname);
+  } catch {
+    return null;
+  }
+  // Path-traversal guard: the joined path must stay strictly under dist/.
+  const filePath = join(dist, decodedPath);
+  if (filePath.startsWith(`${dist}/`) && existsSync(filePath) && statSync(filePath).isFile()) {
+    const mime = mimeFor(filePath);
+    return new Response(
+      Bun.file(filePath),
+      mime ? { headers: { "content-type": mime } } : undefined,
+    );
+  }
+
+  // No matching file. Serve the SPA shell ONLY for HTML navigations (a deep link
+  // like /some-note the app routes client-side). A browser navigation sends
+  // `Accept: text/html`; a non-HTML unclaimed request (API probe, a missing
+  // asset fetched with `Accept: */*`) falls through to the branded 404.
+  const wantsHtml = (req.headers.get("accept") ?? "").includes("text/html");
+  return wantsHtml ? spaShell() : null;
+}

package/src/scope-explanations.ts

@@ -46,7 +46,7 @@ export const SCOPE_EXPLANATIONS: Record<string, ScopeExplanation> = {
       "Read and write everything, plus admin: config & settings, triggers & automation, GitHub backup, and minting access tokens.",
     level: "admin",
   },
-  // Optional-module scopes (scribe / agent / surface). These are in
+  // Optional-module scopes (scribe / surface). These are in
   // FIRST_PARTY_SCOPES (= Object.keys(this map)) but the modules may not be
   // installed — so they're GATED in `OPTIONAL_MODULE_SCOPES` (oauth-handlers.ts)
   // and only advertised in `scopes_supported` when the service is in
@@ -61,10 +61,7 @@ export const SCOPE_EXPLANATIONS: Record<string, ScopeExplanation> = {
     label: "Manage Scribe configuration (provider keys, models, quotas).",
     level: "admin",
   },
-  "agent:send": {
-    label: "Post messages to your Agent.",
-    level: "send",
-  },
+
   // Surface Git Transport scopes (surface-host). `surface:read` = clone/fetch a
   // surface's hub-hosted git repo; `surface:write` = push to it. Named forms
   // (`surface:<name>:<verb>`) collapse to these via the 3→2-segment rule in