ratatoskr/persistent-memory.md

Persistent memory — ratatoskr

Last updated: 2026-05-23

This file captures durable intent and supporting evidence (goals, decisions, foot-gun warnings, in-flight state) across context resets. Read it at session start; treat it as one input alongside CLAUDE.md and the auto-memory system, not as the single source of truth.

When durable state shifts enough to warrant capture, run /snapshot and commit alongside the next commit per the persistent-memory commit-along rule in CLAUDE.md.

---

Repo purpose

Ratatoskr is a dev-grade debug-observability TUI for Worldtree's Conversation API. The product IS the observability surface; chat is the input mechanism. Devs run Ratatoskr against a local Worldtree to watch a turn flow through every layer of the system, side-by-side, in one terminal: agent SSE stream, persona/Vili affect dispatch, tool calls, Bifrost handshake state, admin lifecycle events, optional raw server log.

Named after the squirrel that runs up and down Yggdrasil carrying messages between layers. On-the-nose Worldtree resonance (Yggdrasil = the World Tree).

Origin: althing ask from worldtree-dev (thread 01KS3R34XD3N6HMK91VXESHGW7, 2026-05-20) for the shape of a TUI Conversation API consumer. brokkr-smithy ran the shape pass; operator's reframe routed it as a new repo with a separate dev team rather than an in-tree Worldtree tool.

Current state / in-flight

As of 2026-05-23 (end of day, post-#12 implementation, pre-commit):

Status: issue #12 (presenter contract semantics amendment) TDD-complete, in working tree, awaiting commit. Seven core issues complete (sse_client #1, sessions #2, cli #3, tui #4, --end-user-id #5, TUI startup error visibility #6, presenter contract semantics amendment #12) + robustness fix #7 (MalformedSseData

• empty-skip). 208/208 tests GREEN; ruff clean. pyproject.toml bumped to v0.2.0; uv.lock refreshed. Working tree has 9 modified files + the new docs/contracts/issues/12.contract.md (untracked); commit not yet authored.

Last commits on main:

• 8282156 snapshot: persistent-memory Heimdall scope-model foot-gun (post-v0.1.0)
• 804c2df feat(sessions,cli,tui): issues #5 + #6 + worldtree-dev consumer-API follow-up (tagged v0.1.0)

--send validated end-to-end against personal Worldtree at v0.1.0 (http://10.250.50.152:8081, mimir on qwen3.6-35-a3b, 2026-05-23 smoke returned [done] turn_id=116 duration_ms=5467). Lofn smoke is auth-unblocked as of 2026-05-23 — worldtree-dev confirmed our key (c990f0be) already covers Tier 1 agents via the agent.call:* baseline policy; the initial "scope-add needed" diagnosis was a phantom ask (see Tried-and-abandoned). The actual lofn fix shipped as issue #5 (--end-user-id flag).

Async cross-frontier activity in flight:

• Issue #12 code-review consult posted to volva 2026-05-23 (althing thread 01KSBH8GYH4G3H03T767X613W7). Reply pending in inbox.

Outstanding operator-side todos:

• Commit issue #12 work + tag v0.2.0 + push. 9 modified files + new 12.contract.md ready.
• Post-v0.2.0 mimir smoke (the visual one) — source env.sh && uv run ratatoskr --new --agent mimir --send "test" to eyeball the new rendering (. thinking: ... coalesce, . worker_phase: ... demotion, duration=5.5s formatting, usage 6756 in -> 126 out (...) shape). The v0.1.0 mimir smoke confirmed wire-level backwards compat but did NOT exercise the v0.2.0 rendering.
• Post-v0.2.0 lofn smoke — source env.sh && uv run ratatoskr --new --agent lofn --send "hello" (env.sh ships RATATOSKR_END_USER_ID="ratatoskr-tui"). Now unblocked on auth.

Pending issues filed but not started:

• Issue #8 (startup agent picker) — filed but unscaffolded. GET /agents is free to call (worldtree-dev confirmed); auth side is unblocked. Depends on #5 composably (both thread through ParsedArgs → _resolve_then_run).
• Issue #9 (spec-pin refresh v0.19.0 → v0.22.1) — filed 2026-05-23. Documentation debt. None of the v0.20.0/v0.21.0/v0.22.0 changes break ratatoskr's existing surface; the pin lies about what we've committed to.
• Issue #10 (subject:{type,id} migration) — filed 2026-05-23 to track Worldtree #196's LOCKED-but-not-shipped breaking change. Don't pre-implement per worldtree-dev's explicit guidance.
• Issue #11 (AdminEvents pane auth prerequisite) — filed 2026-05-23. Future side-pane requires admin.events.read scope.

Branch: main (dirty with #12 work pending commit). Remote: origin → git@gitea.phasefinal.com:vh/ratatoskr.git.

Next natural moves:

1. Triage volva's #12 code-review when the reply lands in the inbox; apply tactical fixes inline, surface architectural calls.
2. Commit + tag v0.2.0 + push.
3. Post-v0.2.0 smokes — mimir (visual), lofn (newly unblocked).
4. Issue #8 (startup agent picker) — scaffold + contract, then TDD. Composes with the forward end_user_id direction (see Recent decisions).
5. Side-pane issues — Persona pane first (file-tail, cheap).
6. Issue #9 (spec-pin refresh) — defer unless we need a v0.20.0+ capability (e.g., memory_context for Phase 2.1).

Recent decisions

Chronological log of decisions with [YYYY-MM-DD] prefix. One line per decision. Captures rationale that won't be obvious from code alone.

• [2026-05-20] Project name Ratatoskr (squirrel on Yggdrasil — runs up and down carrying messages). Earlier candidate Andvari demoted on the cursed-ring association.
• [2026-05-20] Separate repo, separate dev team. Operator's call; the in-tree-at-Worldtree/tools/ alternative was considered and rejected to dogfood the API boundary.
• [2026-05-20] No Worldtree-source imports. Spec-only dependency. Triple version-skew mitigation: spec-pin in pyproject.toml + recorded-SSE snapshot tests + conformance smoke. Initial pin: 55101e909abcd2219833266b6f905c5bc956e0f0 (Worldtree v0.19.0). See docs/SPEC-PIN.md.
• [2026-05-20] Textual (not rich+prompt_toolkit). Driver: debug observability is the primary purpose, and a multi-pane dashboard with persistent side panes + independent scrollback is structurally application-shell-shaped. Volva consulted via cross-frontier second-opinion and converged on the same call.
• [2026-05-20] httpx-sse for SSE consumption. The server emits composite {turn_id}:{seq} id: lines (Worldtree INV-014) load-bearing for SSE-resume; hand-rolled data:-only parsing (the skaldsong pattern) silently drops these. Ratatoskr becomes the reference Python SSE-resume implementation.
• [2026-05-20] Persona-pane PII posture: label-don't-refuse. persona.log is process-wide; pane title flips between [Persona — PROCESS-WIDE] and [Persona — session <id>…] based on whether log lines carry session_id. Refuse-against-non-local was considered and rejected as paternalistic.
• [2026-05-20] Server-stdout pane: opt-in via --server-log <path>. No auto-detection of well-known paths.
• [2026-05-20] Two-stage Ctrl-C. First cancels in-flight turn server-side; second exits app. Ctrl-D bound to immediate exit.
• [2026-05-20] Single-session-per-launch + startup picker. No in-app /switch. CLI flags --session <id> and --new for scripted use. Session identity always visible in Textual footer.
• [2026-05-20] Markdown rendering default-on; --raw opt-out. Don't pre-design --no-stream-formatting (Volva: add only if streaming-markdown rendering is empirically ugly).
• [2026-05-20] Non-interactive --send mode. Single SSE consumer module, two presenters (TUI + stdout). Keeps Ratatoskr honest as an API consumer; useful for CI / scripted probes.
• [2026-05-20] First contract: ratatoskr.sse_client. Bundles stream_turn + reconnect_turn + cancel_turn + private _parse_sse_id into one module — the SSE-resume flow is coupled (cancel needs turn_id from the SSE wire id:, reconnect re-uses the same parsed SseId), so they share a contract. Hard invariant INV-002 makes the composite {turn_id}:{seq} id: parsing load-bearing — closes the foot-gun the design-brief §3 names (hand-rolled data:-only parsing silently drops the id:).
• [2026-05-21] Contract converted to issue-scoped (issue #1). Frontmatter shape switched from module-scoped (module:/purpose:) to issue-scoped (target_module:/scope:/prd:) per CONTRACT-FORMAT §2.1.I. prd: block pins to issue body hash. Known parser stale-ness: contract_parser.py --validate ERRORs on issue-scoped frontmatter — CONTRACT-FORMAT §2.1.L H10, a documented Brokkr-side follow-up. Parser is a canonical sync, so we do NOT patch it locally. Treat parser ERROR-on-issue-scoped as expected until canonical bumps.
• [2026-05-21] Default issue-tracker labels seeded (17 total). Sleipnir gating, triage, type, resolution, Ratatoskr-specific area labels (sse-client, tui, cli, observability).
• [2026-05-21] Volva paraphrase + code-review across all 4 issues — calibration consistent. Paraphrase rounds flag 3-5 contract ambiguities per issue; code-review rounds flag 3-8 code-vs-contract drifts after TDD-passing implementation. Hit rates: #1 paraphrase 3-of-5 amended / code-review 4 findings; #2 3-of-5 / 3 findings; #3 5-of-5 / 5 findings; #4 5-of-5 / 8 findings. The post-TDD code-review consistently catches three classes of gap the test-author's hypotheses don't cover: PRE-assertion boundary drift, exception-payload truncation / never-rendered-to-user observability misses, and "tested the state but not whether the user can see it" gaps (issue #4's primary finding: TUI footer state stored but never rendered to a visible widget — same-model TDD would systematically miss this).
• [2026-05-21] Manual smoke is load-bearing — found a real defect tests couldn't. First wire-level smoke against personal Worldtree (post-TDD, post-Volva-code-review on #4) revealed httpx's default 5s read timeout killed the SSE connection mid-stream during mimir's thinking phase (~30s LLM latency >> 5s read timeout). The unit/contract test infrastructure (respx-mocked SSE wire) doesn't model real LLM latency, so the gap was invisible at the test layer. Fix: caller-owned httpx.AsyncClient constructed with timeout=httpx.Timeout(connect=10.0, read=None, write=10.0, pool=10.0); defense in depth: sse_client.stream_turn ERROR_ROUTING catches httpx.ReadTimeout → SseConnectionDropped. Three contracts amended in-place to document the timeout policy. Lesson: keep manual-smoke step in the per-issue cadence; mock-only validation is insufficient for streaming-against-real-server code. Re-smoke succeeded: [done] turn_id=88 model=qwen3.6-35-a3b duration_ms=2351. Wire-compat envelope (personal v0.16.2 vs ratatoskr's v0.19.0 pin) confirmed end-to-end.
• [2026-05-22] Issues #5/#6/#7 filed: per-user-agent support + TUI-startup-visibility + mid-stream-robustness. Discovered during 2026-05-22 mimir TUI conversation: long completion (turn 93, 1077 events consumed) crashed with JSONDecodeError("Expecting value: line 1 column 1 (char 0)") from json.loads('') on an empty-data: SSE frame. Diagnosis surfaced #7 (the crash). Earlier same day, ratatoskr --new --agent lofn failed with 422 end_user_id_required — surfacing #5 (--end-user-id flag needed for per-user agents). #6 (TUI alt-screen masks the diagnostic before user can read it) was a corollary observation. All three filed; user reordered to #7 first (highest-impact for daily TUI use).
• [2026-05-22] Issue #8 (startup agent picker) filed. GET /agents exists in the vendored spec (spec line 832); returns agent_id, name, description + optional version, capabilities, ui_hints. --agent becomes conditionally optional: still required for --send --new (non-interactive); optional for TUI --new. When omitted in TUI mode, a new AgentPickerScreen fetches the agent list and presents a ListView. Depends on list_agents() function in ratatoskr.sessions. Composes naturally with issue #5 (both thread through ParsedArgs → on_mount / _resolve_then_run). Out of scope: search/sort, ui_hints rendering, --send mode picker.
• [2026-05-23] Issue #6 (TUI startup error visibility) contract drafted + Volva paraphrase complete. Restructures run_tui lifecycle: session resolution moves OUT of on_mount (alt-screen) into a new _resolve_then_run async helper (pre-App.run()). AsyncClient ownership also moves to run_tui's async with; RatatoskrApp.__init__ takes pre-resolved session_id/agent_id/client; on_mount shrinks to identity-widget population. Pre-alt-screen errors → real stderr (same labels/codes as --send). Mid-session errors → RichLog (unchanged, per issue #4 INV-008). Volva paraphrase triage applied the new 5-category framework (Genuine add / Sharpening / Restatement / Out-of-place / Wrong-grounding + ignorance-of-context check). 2 of 5 flagged items amended: F1 (Category 1 — internal contract contradiction: assumptions block said "two sequential event loops" while normative STEPS said await app.run_async() — corrected to describe one async flow); F3 (Category 2 — sharpening: informal <truncated> prose aligned to normative {exc.body!r} shape already in STEPS). 3 accepted: F2 (Category 5 — httpx exception hierarchy mis-inference without httpx source access), F4 (Category 3 — restatement of settled architectural guardrail), F5 (Category 2 — sharpening confirming test is the load-bearing spec element).
• [2026-05-22] Issue #7 (MalformedSseData + empty-skip) implemented via TDD + Volva-code-reviewed + smoked. Contract → Volva paraphrase (4 ambiguities, all amended; INV-001 wording tightened around exact sse.data == '' rule, ordering-before-id-parse made explicit, test-description bug fixed) → TDD (6 tests, full vertical-slice ordering) → Volva code-review (3 findings — F1 test-gap probing internal last_sse_id non-advancement via post-skip drop, F2 contract precision around log-vs-propagate responsibility, F3 cli test tightening for raw='X' shape + truncation coverage; all amended) → smoke (3193-token completion against personal Worldtree confirmed clean termination; original crash unreproducible). Calibration milestone: issue #7 is the first issue with zero drift findings from Volva code-review — TDD caught all runtime behavior cleanly. The 3 findings were assertion-precision and architectural-correctness-of-wording, not behavioral. Hypothesis: the tighter the contract spec + the smaller the code surface, the more Volva's role shifts from "catch behavioral drift" to "tighten observability + wording". Calibration table now: #1 (4 findings, 3 drift + 1 test-gap), #2 (3, 1+1+1 precision), #3 (5, 3+1+1), #4 (8, 5+2+1), #7 (3, 0 drift + 2 test-gap + 1 precision).
• [2026-05-23] Issue #6 (TUI startup error visibility) implemented via TDD + Volva-code-review (two rounds). Lifecycle restructure: run_tui becomes a thin sync wrapper around asyncio.run(_resolve_then_run(args)); the new _resolve_then_run opens the httpx.AsyncClient via async with, does pre-flight session resolution, routes AgentNotFound/SessionApiFailed/network errors to real sys.stderr (verbatim same labels as cli._amain), THEN constructs RatatoskrApp with pre-resolved state and calls await app.run_async(). RatatoskrApp.__init__ signature widens to (args, *, session_id, agent_id, client) — all three required. on_mount narrows to identity-widget population; on_unmount becomes a no-op. The alt-screen never opens on resolution errors (INV-001). Two Volva code-review rounds: round 1 returned 6 findings (1 drift + 5 test-gaps), all Category 1 fixed (F1 added the missing PRE-001 assertion at _resolve_then_run entry; F2-F6 tightened test precision — Rule separator assertions on markdown render, RichLog-write spy on empty submit, input-cleared + no-new-worker on cancelling busy, worker.cancel observation on force-exit paths). Round 2 returned 2 NEW test-gaps (F7 client_lifetime_owned_by_run_tui patched run_async so on_unmount wasn't actually exercised — added a sibling test_on_unmount_does_not_close_client; F8 no happy-path --new resolve test — added test_happy_new_session_resolve asserting POST count + identity propagation). Calibration confirmed multi-round-Volva value: round 2 found things round 1's amendments didn't anticipate, but they were strictly test-precision, no behavioral drift.
• [2026-05-23] Issue #5 (--end-user-id) implemented via TDD. Small surface change across three modules (sessions, cli, tui): create_session(client, agent_id, *, end_user_id=None) widens with optional kwarg; body conditionally adds the field when non-None (INV-002: omitting != sending empty); PRE-003 asserts non-empty. ParsedArgs.end_user_id: str | None = None field; --end-user-id CLI flag with non-empty validation (mirrors --send check). _amain and _resolve_then_run thread end_user_id=args.end_user_id to their create_session calls. Post-#6 adjustment: the contract originally named on_mount as the TUI threading site, but #6 had moved session resolution to _resolve_then_run — same shape, different function. 7 new tests across the 3 modules.
• [2026-05-23] Worldtree-dev consult landed authoritative consumer-API guidance (althing thread 01KSBARG2B8M8C82H6AJGJWX1B). Key takeaways shaped follow-on work: (1) end_user_id is a free-form partition key for long-term memory + persona/valence state; same value → same partition, different values → fully isolated. For Vuong-debugging-Worldtree the recommended posture is a project-stable default with --end-user-id override. (2) No programmatic requires_end_user_id discovery on GET /agents — "try and react to 422" remains the pattern. (3) Breaking-change #196 LOCKED but not shipped: subject:{type,id} replaces end_user_id at future v0.22.x or v0.23.0; don't pre-implement. (4) Spec pin (v0.19.0) is 3 minor versions stale (current v0.22.1); none of v0.20.0/v0.21.0/v0.22.0 break ratatoskr's surface but the pin lies about what we're committed to. (5) User-Agent header: send one (ratatoskr/<version> (vh@phasefinal.com)). (6) agents.call:lofn scope needed for lofn smoke. (7) GET /agents requires no special scope; issue #8 unblocked on auth.
• [2026-05-23] Follow-up acted on: User-Agent header added to both _amain and _resolve_then_run httpx.AsyncClient constructions (with importlib.metadata version lookup + fallback to 0.0.0); RATATOSKR_END_USER_ID env-var fallback added to _parse_args (resolution: flag > env > None); env.sh ships RATATOSKR_END_USER_ID="ratatoskr-tui" as project-stable default. Original issue #5 posture rejected env-var fallback as "papering over isolation"; revised after worldtree-dev's guidance that the realistic single-operator use case wants partition continuity. Issue #5 + #3 contracts amended in-place to document the env-var fallback. Infra-ops pinged via althing for agents.call:lofn scope (broker pattern; they forwarded to worldtree-dev). Three Gitea issues filed: #9 (spec-pin refresh), #10 (subject:{type,id} migration tracking), #11 (AdminEvents pane auth prereq).
• [2026-05-23] Issue #12 (presenter contract semantics amendment) implemented via TDD. Headline: thinking deltas render as ONE coalesced growing line (CLI) / one closed RichLog entry per run + live Static(id="thinking-current") widget per-delta (TUI), not 50 lines per turn. Introduced stateful per-turn presenters: CliPresenterState (cli.py) and TuiPresenterState (tui.py), both @dataclass(slots=True) with thinking_buffer + thinking_open (+ text_written_since_newline for CLI). Editorial promotion line settled: load-bearing = Text/Done/Error/Cancelled (no prefix); demoted telemetry = WorkerPhase/Thinking/TextBoundary/ToolStart/ToolResult (CLI . ASCII prefix; TUI · Unicode dim prefix). CLI stdout/stderr newline-boundary INV-005: when text was streamed mid-line, flush a \n to stdout before writing terminal labels to stderr; text_written_since_newline = not event.content.endswith("\n") per Volva F4 fix. Helpers _format_duration_ms (347ms / 5.5s / 1.2m autoscale) and _format_usage (6756 in -> 126 out (6882 total, 0 cached) with arrow="->" CLI or "→" TUI). Per Vor (eitri-smithy-dev cross-frontier consult, althing 01KSBE52YZR5) + Volva paraphrase (5 contract-text ambiguities all fixed in #12.contract.md). [create_session] lifecycle line demoted to . create_session: (written directly by _amain, bypasses state.render). Old _render_event / _render_event_to_log functions and their TestRenderEvent/TestRenderEventToLog classes removed (no-backwards-compat rule). Contracts amended: #3 (CliPresenterState block + _run_turn thread state + _amain create_session demotion + _format_* helper blocks), #4 (TuiPresenterState block + _stream_turn_worker state construction + compose Static widget addition). 39 new tests; 19 obsolete tests removed; net 208 GREEN. v0.1.0 → v0.2.0 (minor; pre-amendment output shape broken intentionally — scripts grepping [thinking] ' no longer work; that's the intended cleanup). Cross-frontier design pass with eitri-smithy-dev returned 16-of-16 confirmed decisions + 4 material divergences applied (ASCII · factual fix, RichLog-one-entry-per-run vs inline-mirror, presenter-state object vs stateless, "contract semantics amendment" framing not "polish"). Calibration note: eitri-smithy-dev's value here was architectural (state-object pattern + chronological-vs-live decoupling) not just tactical; the framing rename alone justified the consult. Volva paraphrase round added 5 prose-precision fixes (INV-001 "growing display" semantics, TUI hide mechanism unification, render_error security/readability tension, newline-tracking corner case, [create_session] integration path).
• [2026-05-23] Forward direction: Ratatoskr will require end_user_id for EVERY access before too long. Operator's call. Reasoning: even Tier 1 foundational agents (mimir, all Asgardians) that don't require end_user_id server-side currently fall back to a _no_end_user sentinel substrate partition — effectively pollution from a single-operator-debug-tool's perspective. The right shape is "every conversation has an explicit partition key." RATATOSKR_END_USER_ID="ratatoskr-tui" env-default in env.sh is the first step toward that posture; once we've validated the partition-isolation experience, the next move is making end_user_id mandatory (probably remove the None-default in _parse_args, fail-closed with a UsageError if neither flag nor env provides it). Consequence for cross-project asks: declined worldtree-dev's offer to ship requires_end_user_id: bool on AgentInfoResponse because we'd treat every value as true regardless; the try-and-react-to-422 pattern goes away from our side because we never send a request without the field. File a ratatoskr issue when scheduling the change — touches _parse_args validation + _resolve_then_run + _amain + tests + contract amendments to #3 / #5. Treat as a v0.2.0 minor (breaking: existing --new --agent mimir without env or flag would start failing). Cross-frontier alignment (worldtree-dev ack 2026-05-23, althing 01KSBD9FPMCWJMBXNNS4B3MYBS): the platform side agrees with this framing — _no_end_user is a substrate accommodation for identity-less transports, NOT a consumer model. The fallback's _is_fallback=True trap door (#185 INV-185-5/8) "could become operator-controlled later" per worldtree-dev, meaning Worldtree itself may tighten the substrate-fallback path. Ratatoskr's forward posture pre-empts that tightening — moving from "we send end_user_id when set" to "we never send a request without end_user_id" stays consumer-correct regardless of what Worldtree does with the fallback knob.

For per-issue TDD implementation notes, Volva findings, and contract amendments, see the git log (commits 9703eb2..61c3941 carry the full per-issue trail with structured commit messages).

Tried and abandoned

Log of approaches that were tried and rejected, with rationale. Future-self defense against re-attempting the same cul-de-sac.

• [2026-05-20] rich + prompt_toolkit framework choice. Considered first (during initial shape draft). Volva flagged that §1 and §5 pulled in opposite directions: a real side-panel observability surface would silently become a widget framework reimplementation. Operator's debug-observability reframe sealed the flip to Textual. Don't re-attempt rich+pt unless the scope shrinks to transcript-first REPL (which would also flip back §5 to inline-log-presenter).
• [2026-05-20] In-tree at Worldtree/tools/ratatoskr/. Earlier draft committed to in-tree-with-import-direction-smoke-test. Rejected at operator-routing — separate dev team forces separate repo.
• [2026-05-20] New /persona/log SSE endpoint on Worldtree. Considered as alternative to file-tailing persona.log. Rejected — contract amendment + Vor round + AFK dispatch loop is weeks of consumer-side spec work for a debug feature file-tail handles in a day. Documented follow-up trigger in docs/design-brief.md §5: if a Worldtree-on-server / TUI-on-laptop debug case appears, the contract cost becomes worth paying.
• [2026-05-20] Cross-process Last-Event-ID resume. Considered — would require persisting per-session Last-Event-ID to ~/.config/ratatoskr/. Deferred to v2 if/when it turns out to matter; v1 ships "reconnect, not resume-across-process."
• [2026-05-21] RichLog widget with markup=True. Default impulse, but Rich interprets [xxx] spans as style markup and silently strips them. Every labeled stderr-style line — [cancel_failed], [done], [error], [busy], [worker_phase] — would render as just the content after the bracketed label, breaking the user-visible observability surface. Fix: markup=False. The post-Done Markdown rendering still works because rich.markdown.Markdown is a Renderable that ignores widget-level markup setting. Don't flip back to markup=True without first renaming every labeled-line format away from [bracket] notation.
• [2026-05-21] Querying self.query_one("#transcript", RichLog) from inside a Textual run_worker coroutine. Failed initially with NoMatches because the worker fires before the test's pilot.pause() allows the Input.Submitted handler to fully dispatch (and thus the widget tree to settle). Initial reactive fix: widen worker signature to take log as a parameter (passed from the handler). Volva code-review flagged this as contract drift (signature didn't match spec). Reverted to single-param signature. The real fix was test-side: add await pilot.pause() between inp.action_submit() and the polling loop in _submit_and_wait so the handler finishes dispatching before the worker reads the widget tree. Don't widen worker signatures to dodge test timing.
• [2026-05-21] TUI session-identity rendering via self.sub_title + self.hint plain attributes. Stored state but never rendered to a visible widget. The contract's "session-identity-always-visible" invariant was satisfied at the state-attribute level but not the user-visible-widget level. Tests asserted the attributes (which passed); Volva code-review flagged the gap. Fix: dedicated Static(id="identity") + Static(id="hint") widgets in compose; _set_hint() helper mirrors state → widget. Calibration evidence for the "TDD catches state, code-review catches whether the user can see it" pattern.
• [2026-05-23] Using the cross-model review agent's name directly in composed prose. The peer review agent's name (the althing handle starting with "V-o-l-v-a") is one letter from a body-part term. Anthropic's content classifier does fuzzy matching and intermittently blocks responses mid-stream when the name appears in composed prose sentences (especially in meta-commentary about the agent's work). Direct-quoted tool output (e.g., the althing-cli thread body) passes through fine. Mitigation: use role descriptions ("the cross-model reviewer," "the paraphrase peer") in prose rather than the name; quote content via tool output. Confirmed by switching to Sonnet 4.6 for a test read — same raw content read cleanly when fetched via Bash rather than composed into an LLM response. This is a persistent environmental constraint, not a one-off.
• [2026-05-22] json.loads(sse.data) unguarded against empty data. _iter_events unconditionally called json.loads on every dispatched ServerSentEvent. When httpx_sse surfaced a frame with id: present but data: empty (a known library-vs-spec divergence — RFC says don't dispatch; httpx_sse is permissive), json.loads('') raised JSONDecodeError → propagated through Textual's worker → app crash. Crashed mimir conversation at turn 93/seq 1078 after 1077 successful events. Fix: if sse.data == '': continue BEFORE _parse_sse_id (empty-data event with a malformed id is still a keepalive — don't reorder). Non-empty malformed data raises new MalformedSseData(raw[:200]). Don't reintroduce unconditional json.loads(sse.data); always pre-check for the empty case.
• [2026-05-23] Phantom "per-Tier-1-agent scope add" pattern. Issue #5's lofn 422 was initially diagnosed (with worldtree-dev's first reply) as needing agents.call:lofn added to ratatoskr's existing key. Routed through infra-ops via althing per the credential-brokerage rule; infra-ops discovered no public scope-mutation endpoint on personal Worldtree, brokered to worldtree-dev for the actual mechanism. Worldtree-dev came back with a correction: their first answer conflated two distinct Heimdall scope namespaces. Tier 1 foundational agents (mimir, lofn, soong, all Asgardians) are covered by a blanket agent.call:* (singular) baseline rule in config/policies.yaml > tiers.<tier>.scopes for ALL authenticated tiers including user. There is no per-agent grant for Tier 1 — the baseline rule covers it. Tier 3 consumer-defined agents (IDs containing :, like vh:custom-bot) use the plural agents.call:<owner>:<agent> shape granted implicitly via owning a consumer_agents DB row, registered through POST /agents/define. The two notations differ by one letter and that was the source of the confusion. The actual lofn fix was issue #5's --end-user-id flag — it was always a request-body validation, not an auth-scope gate. Don't ping infra-ops for "per-Tier-1-agent scope adds" again; the pattern is a phantom ask. Real future infra-ops asks: admin-tier key for the AdminEvents pane (admin.events.read scope, different tier), and Tier 3 custom-agent registration (different flow entirely, requires POST /agents/define).