chore(daemon): collapse the HTTP router onto one deep request adapter + route table

[sethvoltz]

What & why

The daemon's handle() was a 2545-line shell routing via a hand-written if/else cascade of 83 branches, each re-implementing the same mechanics: read body → validate → call subsystem → shape an ad-hoc { error } envelope → pick a status. Apply the deletion test and the behavior doesn't vanish — the tell of a shallow shell carrying no logic of its own.

This introduces one deep request adapter (router.ts: handleRequest) plus a route table of declarative data rows (method, match) → { auth?, schema?, handler }. The adapter owns the cross-cutting mechanics — auth gate → body parse → schema validation → handler → error envelope — so handle() shrinks to a 3-line dispatch and each of the 83 branches becomes one row. Modeled on the repo's own blessed exemplar (inbox/route-registry.ts): the schema field is the producer-agnostic { safeParse } structural contract, so the router never imports zod's full surface.

Faithfulness — structural, not a contract change

Verified by 3 independent adversarial passes diffing the new table against the original cascade (git show HEAD):

• 83 original branches → 83 table rows, 1:1 — none dropped, none added.
• Auth split preserved exactly: the 21 inline-authorizeSameHost routes carry auth: true; the other 62 rely on the 127.0.0.1 bind as before. No gated→public regression.
• Declaration order preserves every specific-before-broad case (/api/memory/search before /<id>; habits checkin/archive before bare /<id>; schedules action-paths before bare; etc.).
• The 5 streaming/binary/seam routes (events SSE, uploads POST/GET, evolve scan, commands dispatch) are raw rows that own req/res — no body parse, no serialize. The ADR-036 evolve-scan daemonEffects object (config-gated triage/builder spawn IO + the un-forgeable evolveEscalation carve-out) moves byte-for-byte, as do the helper tail + createAgent.

Three intentional improvements (each test-pinned)

1. POST /api/memory's inline title/content check becomes a zod schema run by the adapter (the one demonstration of the schema path) — 400 message text changes and non-array tags now 400s instead of being silently mis-stored.
2. A malformed JSON body becomes one clean 400 instead of an uncaught readJson throw (previously an unhandled rejection / hung connection).
3. An uncaught handler throw becomes one centralized 500 instead of a hang.

Tests

• router.test.ts unit-tests the adapter's 4 cross-cutting behaviors + the matcher once each (14 tests, fake req/res, real zod schema so validation genuinely runs).
• Added round-trip endpoint tests for habits / memory / uploads — families that had no daemon HTTP coverage and would otherwise have migrated without a net.
• pnpm --filter @friday/daemon exec vitest run src/api → 14 files / 83 tests green; daemon tsc --noEmit clean; prettier clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01PGvF96oXJeppyc7PvhMPen