Skip to content
This repository was archived by the owner on Mar 11, 2026. It is now read-only.

feat(capture): crash-safe rolling chunk recording#124

Merged
Mikecranesync merged 1 commit intomainfrom
worktree-feat-open-brain
Mar 9, 2026
Merged

feat(capture): crash-safe rolling chunk recording#124
Mikecranesync merged 1 commit intomainfrom
worktree-feat-open-brain

Conversation

@Mikecranesync
Copy link
Copy Markdown
Owner

@Mikecranesync Mikecranesync commented Mar 5, 2026

Status: post-V1 — Will revisit after revenue bot is live.

Summary

  • Rewrites record_session() to write rolling 8-second MP4 chunks instead of one big file — each chunk is a complete, valid MP4
  • Adds stop-file mechanism (cookoff/clips/.stop_recording) so recording can be stopped from a separate process (no Ctrl+C needed)
  • Adds capture_fio.py stop CLI subcommand and stop_recording() helper
  • Updates visual_test_loop.py to consume rolling chunks directly (no post-hoc chunking for live sessions)

Test plan

  • Start recording in background, stop via capture_fio.py stop — exits cleanly
  • All 4 chunks verified as valid MP4s (32 frames each, correct resolution)
  • Stale stop file cleaned up at recording start
  • Full session mode with visual_test_loop.py --session

🤖 Generated with Claude Code

@Mikecranesync Mikecranesync added the post-v1 Deferred until after revenue bot ships label Mar 9, 2026
@Mikecranesync Mikecranesync force-pushed the worktree-feat-open-brain branch from 695faff to 98cdc00 Compare March 9, 2026 02:21
Rewrite record_session() to write 8s MP4 chunks on-the-fly instead of
one big file. Each chunk is released before the next starts, so a crash
only loses the current partial chunk. Stop mechanism uses a sentinel
file (.stop_recording) instead of Ctrl+C, which doesn't work reliably
from Claude Code's Bash tool.

- Add stop_recording() helper and `capture_fio.py stop` CLI subcommand
- Update visual_test_loop.py run_session() to use rolling chunks directly
- Include sim controllers and cookoff source files

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Mikecranesync Mikecranesync force-pushed the worktree-feat-open-brain branch from 98cdc00 to 98d0357 Compare March 9, 2026 02:23
@Mikecranesync Mikecranesync merged commit a8f1809 into main Mar 9, 2026
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

post-v1 Deferred until after revenue bot ships

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant