Skip to content

Harden browser recording controls and performance - #725

Merged
capy-ai[bot] merged 2 commits into
devfrom
capy/harden-browser-recording-control
Aug 10, 2026
Merged

Harden browser recording controls and performance#725
capy-ai[bot] merged 2 commits into
devfrom
capy/harden-browser-recording-control

Conversation

@zortos293

@zortos293 zortos293 commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

Adds browser-stream recording controls without pulling in the fork’s native GStreamer recording suite. This PR stacks on #724 and bounds local capture work so recording does not starve live decode/render.

  • Adds 720p/1080p/1440p and 30/60 FPS capture settings with safe 720p30 defaults and a 12 Mbps custom bitrate ceiling.
  • Captures only presented video frames into an aspect-preserving bounded canvas, then cancels callbacks, timers, tracks, and audio resources on every stop/error/unmount path.
  • Serializes MediaRecorder chunk writes before finalization so stop cannot race pending file writes.
  • Adds keyboard-accessible controls and explicit Starting, Recording, Finalizing, and error states in the normal Stream settings and Quick Menu Media page.
  • Keeps screenshots, auth, packaging, updater behavior, and generated translations unchanged.

Browser recording controls

Ported from work by Akamiya Chizui in Chizuui/OpenNOW-Modified.

Open in Capy

@zortos293 zortos293 added the capy Generated by capy.ai label Aug 9, 2026 — with Capy AI
@cursor

cursor Bot commented Aug 9, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d9bc15ed-a6d8-4d2d-abc6-5c76b872e66e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@capy-ai capy-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Added 1 comment

Comment thread opennow-stable/src/renderer/src/hooks/useStreamRecorder.ts Outdated
zortos293 and others added 2 commits August 9, 2026 23:28
Co-authored-by: Akamiya Chizui <127862333+Chizuui@users.noreply.github.com>

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>

Co-authored-by: Akamiya Chizui <127862333+Chizuui@users.noreply.github.com>
@zortos293
zortos293 force-pushed the capy/harden-browser-recording-control branch from c5d2c1b to cbc2c76 Compare August 9, 2026 23:29
@zortos293
zortos293 changed the base branch from capy/add-gfn-style-compact to dev August 9, 2026 23:29
@capy-ai
capy-ai Bot merged commit 4c79514 into dev Aug 10, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

capy Generated by capy.ai

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant