Skip to content

fix: refuse an unready recording - #62

Merged
abrichr merged 1 commit into
mainfrom
fix/refuse-unready-recording
Jul 28, 2026
Merged

fix: refuse an unready recording#62
abrichr merged 1 commit into
mainfrom
fix/refuse-unready-recording

Conversation

@abrichr

@abrichr abrichr commented Jul 28, 2026

Copy link
Copy Markdown
Member

Summary

  • return a visible nonzero failure when the recorder cannot become ready
  • never print Recorded or Saved for an unready zero-event session
  • preserve the successful recording path and recorder cleanup

Verification

  • 33 focused behavior and high-level API tests
  • Ruff on the production tree

Independent review is active. This PR does not change Flow or Desktop.

@abrichr
abrichr merged commit 3e6b933 into main Jul 28, 2026
12 checks passed
@abrichr
abrichr deleted the fix/refuse-unready-recording branch July 28, 2026 06:18
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