chore: remove dead Claude OTEL tab from dashboard#49
Merged
Conversation
ADR-059 replaced narrator tool calls with Monster Manual. The Claude tab tracked tool use/cost/tokens that no longer exist. Removed tab, JS functions, and state fields. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2 tasks
slabgorb
added a commit
that referenced
this pull request
Apr 25, 2026
* docs(spec): shared Room.snapshot for ADR-037 (Python port) Design for the proper architectural fix to per-session GameSnapshot divergence in multiplayer. Replaces the band-aid _merge_peer_state_into_snapshot helper with a single canonical snapshot held on SessionRoom and shared by every WS session bound to the slug. Constraint that simplifies scope: no saved MP games exist on disk (multiplayer has never worked end-to-end). No migration path needed. Band-aid + its 5 merge tests are deleted in the same change. Out of scope: ADR-028 LLM rewrites, per-recipient narration region filtering, PlayerState overlay struct. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * docs(plan): shared Room.snapshot implementation plan (ADR-037 Python port) Step-by-step plan implementing the spec at docs/superpowers/specs/2026-04-25-shared-room-snapshot-design.md. 9 tasks, 8 commits, ends at 739 passed / 2 skipped on the server + agents sweep. Sequential single-developer plan suitable for inline execution as the Bicycle Repair Man dev agent. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * docs(spec): OTEL dashboard restoration design (faithful ADR-031 port) Forensic audit found four root causes of the OTEL dashboard regression since the Rust→Python port: broken `just otel` recipe, ~80% dead SPAN_* constants, missing Layer-3 narrative validator, and impoverished translator. Design specifies a four-phase faithful port that restores full parity with the Rust contract. Approved interactively via /superpowers:brainstorming. Self-review pass fixed an emit-double on json_extraction_result (translator owns; not the validator) and reclassified SPAN_CONTENT_RESOLVE to FLAT_ONLY due to volume. Next: writing-plans for implementation plan. * fix(justfile): point otel recipe at playtest_dashboard.py - Update `just otel` to call `playtest_dashboard.py` via `uv run` instead of the deleted `playtest.py --dashboard-only` - Add `__main__` entry point with argparse to `playtest_dashboard.py` (it was a library-only module with no CLI entry point) - Add `websockets>=12.0` and `rich>=13.0` to orchestrator `pyproject.toml` (required by playtest_dashboard.py, previously missing from manifest) * ci: smoke-test the just otel recipe to catch script renames Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs(adr): ADR-090 OTEL Dashboard Restoration after Python Port Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs(adr): amend ADR-031 with Python-port section, regen indexes Add observability tag, replace stale Rust-only opening blockquote, and append Python-port note pointing at the canonical telemetry implementation files in sidequest-server. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs(plan): OTEL dashboard restoration implementation plan Pairs with the design spec at docs/superpowers/specs/2026-04-25-otel-dashboard-restoration-design.md. This is the 25-task plan (Phase 0–4) executed by the OTEL feat branches across orchestrator, sidequest-server, and sidequest-ui. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * docs(otel): Phase-2 port-audit follow-up after bundles #1 + #2 Discovered-state correction for the original Phase-2 emission-rollouts handoff: the Python port (ADR-082) replaced most Rust info_span!/ start_as_current_span calls with logger.info(...) strings or direct _watcher_publish calls. The SPAN_* constants in spans.py are mostly byte-identical-to-Rust name-string prefixes, not actual span openings. Records: - Which families are genuinely live in Python (the ones that ship now). - Which ~7 bundles in the original handoff are port-dead and require engine porting before routing. - The two real Phase-2 workstreams: migrate live _watcher_publish sites (Workstream A — what bundles #49 and #50 did), vs. port the missing engines (Workstream B — out of scope for telemetry restoration). - Suggested next bundle picks from Workstream A. - Open question on whether to bridge port gaps via log-string parsing (recommended: no — Illusionism violation per ADR-031). Closes the loop after PRs slabgorb/sidequest-server#49 and #50. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
🤖 Generated with Claude Code