Skip to content

fix(cli): start Instant recordings behind a start gate - #2654

Merged
richiemcilroy merged 1 commit into
mainfrom
fix/cli-instant-start-gate
Oct 17, 2026
Merged

richiemcilroy merged 1 commit into
mainfrom
fix/cli-instant-start-gate

Conversation

@richiemcilroy

@richiemcilroy richiemcilroy commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

cap record --mode instant started recordings without a start gate. Its audio was then anchored at the first audio sample while the screen started at its first encoded frame, so a microphone that became ready late played early by its start-up delay. The suite's real-capture recording/instant/late-mic case measures about 1 s on both 0.6.0 and main.

// apps/cli/src/record.rs
let start_gate = cap_recording::RecordingStartGate::new();
builder = builder.with_start_gate(start_gate.clone());
let actor = builder.build(/* ... */).await?;
start_gate.arm();
  • The desktop app already starts every recording behind a gate (session.start()); this gives the CLI the same start, armed as soon as the recording is built since the CLI has no start cue.
  • Behind the gate both tracks start at the arm time, and Instant audio is anchored there (fix(recording): anchor gated Instant audio at the recording start on every platform #2650).
  • A 4 s CLI Instant recording on macOS: video 3.93 s, audio 4.01 s, both starting at 0; on main the audio ran 4.18 s because it included capture before the screen started.

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge; no actionable issues were found.

Summary

The CLI now builds Instant recordings behind a shared start gate and arms it immediately after the actor is ready.

  • CLI Instant recordings start screen and audio from one shared gate.

Reviews (1) · Last reviewed commit: "fix(cli): start Instant recordings behin..." · Reviewed by Greptile

The CLI built Instant recordings without a start gate, so its audio was
anchored at the first sample while the screen started at its first
encoded frame: a microphone that became ready late played early by its
start-up delay (about 1 s in the suite's late-mic recording, as in 0.6).
Like the desktop app, the CLI now passes a RecordingStartGate and arms it
once the recording is built.
@vercel

vercel Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
cap-status Ready Ready Preview Oct 16, 2026 7:16pm UTC

@capy-ai capy-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Capy found no issues.

View 1 other finding in Capy.

Open in Capy

@richiemcilroy
richiemcilroy merged commit 2c87e6b into main Oct 17, 2026
37 checks passed

This branch was successfully deployed

2 active deployments
Preview – cap-web — 9acfe8d8 Deployed Oct 16, 2026 by vercel[bot]
Preview – cap-status — 9acfe8d8 Deployed Oct 16, 2026 by vercel[bot]
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