Skip to content

T001-T004: Add session chain recording#20

Merged
grobomo merged 3 commits intomainfrom
001-T001-add-chain-recording
Apr 16, 2026
Merged

T001-T004: Add session chain recording#20
grobomo merged 3 commits intomainfrom
001-T001-add-chain-recording

Conversation

@grobomo
Copy link
Copy Markdown
Owner

@grobomo grobomo commented Apr 16, 2026

Summary

  • verify_claude_working() now returns the JSONL file path (truthy) instead of True, or None instead of False
  • New record_session_chain() writes old→new session transitions to session-chain.jsonl
  • Wired into main() so every successful context reset records the chain
  • 12 new tests for chain recording, 82 total passing

Test plan

  • All 82 tests pass (python scripts/test.py)
  • verify_claude_working tests updated for path return type
  • record_session_chain tests: JSONL format, append behavior, None handling, skip-when-both-None

grobomo added 3 commits April 15, 2026 22:51
- verify_claude_working() now returns JSONL path (truthy) or None (falsy)
- Add record_session_chain() to write old->new transitions to session-chain.jsonl
- Wire chain recording into main() after successful verify
Tests now check for JSONL path string instead of True, and None instead of False.
- 12 new tests for record_session_chain (JSONL format, append, None handling)
- Updated 3 verify_claude_working tests for path/None return type
- 82 tests total, all passing
@grobomo grobomo merged commit f40ec95 into main Apr 16, 2026
1 check passed
@grobomo grobomo deleted the 001-T001-add-chain-recording branch April 16, 2026 03:54
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