refactor(services): give each NATS service its own registerHandlers name

Five services each exported a function called registerHandlers. Every one is
imported only inside its own service, so nothing was broken — but a tree-wide
search for registerHandlers returned five unrelated definitions, and three of
them ranked in the codebase graph's top ten by connectivity, which made the
most-connected symbols in the system also the least searchable.

Each is now service-qualified: registerPromptStoreHandlers,
registerContentIngestHandlers, registerRecordResolverHandlers,
registerResponseStoreHandlers, registerRowStoreHandlers.

registerPersonHandlers in record-surrealdb-resolver is deliberately unchanged.
It was already unique, and renaming an unambiguous symbol is churn.

Corrects a count in the refactor backlog. That document said three, because
three was what the graph's top-ten showed; there are five. God-node rankings
report the top of a distribution, not a census — worth remembering the next
time one is read as a complete list. The backlog now records that, along with
the three other items that shipped without originally being scoped: the turbo
removal, the React eviction, and the shell typecheck fix.

Verified: all five services typecheck clean, 19 packages build, and every test
suite passes including the end-to-end group, which now has Docker available.

Files changed:
- services/{prompt-store,content-ingest,record-surrealdb-resolver,response-store,row-store}/src/{handlers,server}.ts
- context-v/refactors/Structural-Refactors-Surfaced-by-the-Codebase-Graph.md

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019a8tSPbFdvF1pKtADnWyDg

Author: mpstaton

context-v/refactors/Structural-Refactors-Surfaced-by-the-Codebase-Graph.md

@@ -291,7 +291,7 @@ errors, and every non-Docker test suite passing.
 | 3 — utility consolidation | ⬜ not started (and see the services caveat below) |
 | 4 — design system | ⬜ not started — the actual project |
 | 5 — `Actor` import cycle | ✅ shipped — moved to `services/workspace/src/types.ts` |
-| 5 — rename three `registerHandlers()` | ⬜ not started |
+| 5 — rename `registerHandlers()` | ✅ shipped — **five**, not three; one per service |
 
 **Why 1.2 was deferred rather than done.** "Hoist the identical deps to the
 root" is the wrong fix under pnpm. pnpm's strict resolution means a package
@@ -304,12 +304,34 @@ lockfile and touches all 17 manifests, so it wants its own commit and its own
 verification pass — and it collides with the root `package.json` change in the
 stranded design-system work (see below).
 
-**Two things shipped that this document did not originally list:** removing
-`turbo.json` and repointing the root scripts at `pnpm -r`, which made
-`pnpm build` work for the first time in the repo's history (reasoning in
-[[Why-This-Monorepo-Does-Not-Need-Turbo]]); and flagging the root
-`tsconfig.json` as vestigial — it sets `jsx: "react-jsx"` in a repo where React
-is prohibited and nothing extends it. Removing it is a separate decision.
+**Four things shipped that this document did not originally list:**
+
+1. **`turbo.json` removed**, root scripts repointed at `pnpm -r`, which made
+   `pnpm build` work for the first time in the repo's history. Reasoning in
+   [[Why-This-Monorepo-Does-Not-Need-Turbo]].
+2. **React evicted.** The root `tsconfig.json` set `jsx: "react-jsx"` — the
+   only React reference anywhere in the repo, against a hard prohibition, with
+   zero `.tsx` files and no `react` dependency to justify it. Now
+   `jsx: "preserve"`: TSX stays legal, no runtime is bound. Target also raised
+   ES2020 → ES2022 and three dead path aliases removed.
+3. **The shell's typecheck fixed** — it had *never* passed. A missing
+   `css.d.ts` shim, compounded by a tsconfig that omitted `src/**/*.d.ts` from
+   `include` so the shim would have been ignored anyway; plus a real type error
+   (`stageEl` typed `HTMLDivElement` while bound to a `<main>`). This also
+   revealed the shell as the **eighteenth** copy of the converged tsconfig.
+4. **`registerHandlers()` renamed** — see below.
+
+The `registerHandlers()` count in this document was **wrong**. It said three,
+because three appeared in the graph's top-ten by connectivity. There are
+**five**, one per NATS service. All are now service-qualified
+(`registerPromptStoreHandlers`, `registerContentIngestHandlers`,
+`registerRecordResolverHandlers`, `registerResponseStoreHandlers`,
+`registerRowStoreHandlers`), so a tree-wide search for any one of them is
+unambiguous. `registerPersonHandlers` in `record-surrealdb-resolver` was left
+alone — it was already unique, which was the entire point.
+
+A reminder that god-node rankings show the *top* of a distribution, not the
+whole of it. Read them as "look here," never as a census.
 
 **Conflict surface with the stranded design-system work.** Phase 0 adds
 `design:drift` / `design:contrast` to the root `package.json`, which this work

services/content-ingest/src/handlers.ts

@@ -85,7 +85,7 @@ type PreviewResult = {
   error?: string;
 };
 
-export function registerHandlers(nc: NatsConnection): void {
+export function registerContentIngestHandlers(nc: NatsConnection): void {
   // corpus.domain.write_index — internal (resolver → here): write a domain's
   // folder + index.md definition file (<type-plural>/<slug>/index.md). Not a
   // browser capability; the resolver's domain.create handler requests it so the

services/content-ingest/src/server.ts

@@ -1,12 +1,12 @@
 import { connect } from '@nats-io/transport-node';
-import { registerHandlers } from './handlers';
+import { registerContentIngestHandlers } from './handlers';
 
 const NATS_URL = process.env.NATS_URL ?? 'nats://localhost:4222';
 
 async function main(): Promise<void> {
   const nc = await connect({ servers: NATS_URL, name: 'content-ingest-service' });
   console.log(JSON.stringify({ level: 'info', msg: 'nats connected', url: NATS_URL }));
-  registerHandlers(nc);
+  registerContentIngestHandlers(nc);
   console.log(JSON.stringify({ level: 'info', msg: 'content-ingest-service ready' }));
 }
 

services/prompt-store/src/handlers.ts

@@ -25,7 +25,7 @@ import {
   type RecordSetContext,
 } from './store';
 
-export function registerHandlers(nc: NatsConnection): void {
+export function registerPromptStoreHandlers(nc: NatsConnection): void {
   // prompt.list.requested
   (async () => {
     const sub = nc.subscribe('prompt.list.requested');

services/prompt-store/src/server.ts

@@ -1,6 +1,6 @@
 import { connect } from '@nats-io/transport-node';
 import { load } from './store';
-import { registerHandlers } from './handlers';
+import { registerPromptStoreHandlers } from './handlers';
 
 const NATS_URL = process.env.NATS_URL ?? 'nats://localhost:4222';
 const PROMPT_STORE_PATH = process.env.PROMPT_STORE_PATH ?? './data/prompts.json';
@@ -12,7 +12,7 @@ async function main(): Promise<void> {
   const nc = await connect({ servers: NATS_URL, name: 'prompt-store-service' });
   console.log(JSON.stringify({ level: 'info', msg: 'nats connected', url: NATS_URL }));
 
-  registerHandlers(nc);
+  registerPromptStoreHandlers(nc);
   console.log(JSON.stringify({ level: 'info', msg: 'prompt-store-service ready' }));
 }
 

services/record-surrealdb-resolver/src/handlers.ts

@@ -45,7 +45,7 @@ import {
   type OrgEntryRemoveInput,
 } from './resolver';
 
-export function registerHandlers(nc: NatsConnection): void {
+export function registerRecordResolverHandlers(nc: NatsConnection): void {
   // resolver.candidates
   (async () => {
     const sub = nc.subscribe('resolver.candidates.requested');

services/record-surrealdb-resolver/src/server.ts

@@ -1,5 +1,5 @@
 import { connect } from '@nats-io/transport-node';
-import { registerHandlers } from './handlers';
+import { registerRecordResolverHandlers } from './handlers';
 import { registerDomainHandlers } from './domains';
 import { registerPersonHandlers } from './person-handlers';
 import { registerOrgRelationHandlers } from './org-relations';
@@ -23,7 +23,7 @@ async function main(): Promise<void> {
   }
   const nc = await connect({ servers: NATS_URL, name: 'record-surrealdb-resolver-service' });
   console.log(JSON.stringify({ level: 'info', msg: 'nats connected', url: NATS_URL }));
-  registerHandlers(nc);
+  registerRecordResolverHandlers(nc);
   registerDomainHandlers(nc);
   registerPersonHandlers(nc);
   registerOrgRelationHandlers(nc);

services/response-store/src/handlers.ts

@@ -29,7 +29,7 @@ import {
   type ResponseFlag,
 } from './store';
 
-export function registerHandlers(nc: NatsConnection): void {
+export function registerResponseStoreHandlers(nc: NatsConnection): void {
   // response.create.requested — fire-and-forget from prompt-runner
   (async () => {
     const sub = nc.subscribe('response.create.requested');

services/response-store/src/server.ts

@@ -6,7 +6,7 @@
 
 import { connect } from '@nats-io/transport-node';
 import { load } from './store';
-import { registerHandlers } from './handlers';
+import { registerResponseStoreHandlers } from './handlers';
 
 const NATS_URL = process.env.NATS_URL ?? 'nats://localhost:4222';
 const RESPONSE_STORE_PATH = process.env.RESPONSE_STORE_PATH ?? './data/responses.json';
@@ -18,7 +18,7 @@ async function main(): Promise<void> {
   const nc = await connect({ servers: NATS_URL, name: 'response-store-service' });
   console.log(JSON.stringify({ level: 'info', msg: 'nats connected', url: NATS_URL }));
 
-  registerHandlers(nc);
+  registerResponseStoreHandlers(nc);
   console.log(JSON.stringify({ level: 'info', msg: 'response-store-service ready' }));
 }
 

services/row-store/src/handlers.ts

@@ -32,7 +32,7 @@ import {
   type RecordSet,
 } from './store';
 
-export function registerHandlers(nc: NatsConnection): void {
+export function registerRowStoreHandlers(nc: NatsConnection): void {
   // record_set.list.requested
   (async () => {
     const sub = nc.subscribe('record_set.list.requested');