Skip to content

fix: refuse an unready capture - #1070

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

fix: refuse an unready capture#1070
abrichr merged 1 commit into
mainfrom
fix/refuse-unready-capture

Conversation

@abrichr

@abrichr abrichr commented Jul 28, 2026

Copy link
Copy Markdown
Member

Summary

  • make the launcher direct recorder path fail when wait_for_ready returns false
  • suppress misleading Recording and Saved output
  • preserve cleanup and the successful capture path

Verification

  • 125 tests passed, 6 skipped
  • Ruff on launcher code and tests

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

@abrichr
abrichr force-pushed the fix/refuse-unready-capture branch from 730e2dc to 559cac3 Compare July 28, 2026 06:22
@abrichr
abrichr merged commit 2591825 into main Jul 28, 2026
11 checks passed
@abrichr
abrichr deleted the fix/refuse-unready-capture branch July 28, 2026 06:25
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