Skip to content

feat(tape): session cards command + recording analysis#226

Merged
delebedev merged 2 commits intomainfrom
feat/session-cards
Mar 23, 2026
Merged

feat(tape): session cards command + recording analysis#226
delebedev merged 2 commits intomainfrom
feat/session-cards

Conversation

@delebedev
Copy link
Owner

Summary

  • New just tape session cards <session> command — extracts card manifest (names, types, oracle text, abilities) from md-frames.jsonl via Arena card DB. ~2s per session vs ~60s with sequential just card calls.
  • Updated recording-inspect skill with annotation-based activity analysis guidance and seat detection notes.
  • Enriched docs/catalog.yaml SBA_UnattachedAura wire spec from recording evidence.

Context

Built during a 4-session recording analysis pass (2026-03-22). The card manifest is the first layer of a progressive zoom workflow: cards.json (what's in the game) → annotation activity (what happened) → deep trace (protocol details).

Test plan

  • just tape session cards tested on 4 sessions
  • Resolves names, types, subtypes, abilities from Arena sqlite DB
  • Strips HTML tags from card names/ability text
  • Groups output by seat with ability summaries

🤖 Generated with Claude Code

delebedev and others added 2 commits March 22, 2026 23:48
Extracts all cards from md-frames.jsonl, resolves names, types, subtypes,
and ability text from the Arena card DB (sqlite), writes cards.json per
session. Much faster than sequential `just card` lookups (~2s vs ~60s).

Also updates recording-inspect skill with annotation-based activity
analysis guidance and seat detection notes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…dence

Session 2026-03-22_23-20-47 (River's Favor) — first recorded instance.
Documents: LayeredEffectDestroyed fires first, then ObjectIdChanged +
ZoneTransfer(SBA_UnattachedAura). affectorId absent. Fires in same diff
as the SBA_Damage that killed the enchanted creature.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link

Test Results

  236 files  ±0  236 suites  ±0   1m 56s ⏱️ ±0s
  998 tests ±0  972 ✅ ±0   26 💤 ±0  0 ❌ ±0 
1 779 runs  ±0  987 ✅ ±0  792 💤 ±0  0 ❌ ±0 

Results for commit cea1408. ± Comparison against base commit 8b2f850.

@github-actions
Copy link

CI Report — Gate

Tests: 819/819 passed (188 skipped)

Coverage: 10.9% (637/5857 lines)

Module Coverage Lines
tooling 🔴 11% 637/5857
Slow tests (>3s): 1
  • InstanceIdReallocTest.Limbo grows across multiple plays (5.4s)

@delebedev delebedev merged commit aaa1bae into main Mar 23, 2026
4 checks passed
@delebedev delebedev deleted the feat/session-cards branch March 23, 2026 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant