feat: extend deeplinks with pause/resume/toggle/restart/mic/camera/screenshot by wasim-builds · Pull Request #2049 · CapSoftware/Cap

wasim-builds · 2026-07-29T07:34:37Z

Extends Cap desktop app deeplinks to support recording controls and device switching for Raycast extension integration.

Changes

deeplink_actions.rs

Removed #[cfg(debug_assertions)] from PauseRecording and ResumeRecording — now available in production builds
Added new DeepLinkAction variants:
- TogglePauseRecording — toggles pause/resume state
- RestartRecording — stops and restarts the current recording
- SwitchMicrophone { mic_label } — switches active microphone
- SwitchCamera { camera } — switches active camera
- TakeScreenshot — captures a screenshot of the primary display
Implemented execute() match arms for all new actions
Added unit tests for URL parsing of all new actions

Acceptance Criteria

PauseRecording, ResumeRecording available in production builds
TogglePauseRecording, RestartRecording, SwitchMicrophone, SwitchCamera, TakeScreenshot added
All new actions have URL parsing tests
Follows existing code patterns and conventions

Closes #1540

Greptile Summary

Extends desktop deep-link actions with recording lifecycle controls, microphone and camera switching, and screenshot capture.

Makes pause and resume actions available in production builds.
Adds toggle-pause and restart recording actions.
Adds optional microphone and camera selection actions.
Adds screenshot capture and URL parsing tests for each new action.

Confidence Score: 4/5

The screenshot target-selection defect should be fixed before merging because the new deep link can capture the wrong monitor.

The new screenshot action assumes the first enumerated display is primary, while platform display APIs and existing screenshot paths use explicit primary or cursor-containing display resolution.

Files Needing Attention: apps/desktop/src-tauri/src/deeplink_actions.rs

Important Files Changed

Filename	Overview
apps/desktop/src-tauri/src/deeplink_actions.rs	Adds and tests the requested deep-link actions, but screenshot target selection can capture the wrong monitor on multi-display systems.

Prompt To Fix All With AI

### Issue 1
apps/desktop/src-tauri/src/deeplink_actions.rs:287-289
**Arbitrary display selected for screenshot**

When this deep link runs on a multi-display system, `list_displays().next()` selects according to platform enumeration order rather than the primary or cursor-containing display, causing the screenshot to capture the wrong monitor.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

_{Reviews (1): Last reviewed commit: "feat: extend deeplinks with pause/resume..." | Re-trigger Greptile}

Greptile also left 1 inline comment on this PR.

Context used:

Knowledge Base — Desktop Tauri App (Rust Backend)

…reenshot actions

superagent-security · 2026-07-29T07:36:52Z

🚨 Contributor flagged. Click here for more info: Superagent Dashboard

greptile-apps · 2026-07-29T07:36:58Z

+                };
+                crate::recording::take_screenshot(app.clone(), capture_target).await
+            }


Arbitrary display selected for screenshot

When this deep link runs on a multi-display system, list_displays().next() selects according to platform enumeration order rather than the primary or cursor-containing display, causing the screenshot to capture the wrong monitor.

Knowledge Base Used: Desktop Tauri App (Rust Backend)

Prompt To Fix With AI

This is a comment left during a code review. Path: apps/desktop/src-tauri/src/deeplink_actions.rs Line: 287-289 Comment: **Arbitrary display selected for screenshot** When this deep link runs on a multi-display system, `list_displays().next()` selects according to platform enumeration order rather than the primary or cursor-containing display, causing the screenshot to capture the wrong monitor. **Knowledge Base Used:** [Desktop Tauri App (Rust Backend)](https://app.greptile.com/cap/-/custom-context/knowledge-base/capsoftware/cap/-/docs/desktop-tauri-app.md) --- For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

tembo · 2026-07-29T07:37:22Z

+                Ok(())
+            }
+            DeepLinkAction::TakeScreenshot => {
+                let state = app.state::<ArcLock<App>>();


state looks unused here; if you want to keep it around for side-effects/lifetime, prefix with _ to avoid unused warnings.

Suggested change

let state = app.state::<ArcLock<App>>();

let _state = app.state::<ArcLock<App>>();

tembo · 2026-07-29T07:37:31Z

+                let capture_target = ScreenCaptureTarget::Display {
+                    id: cap_recording::screen_capture::list_displays()
+                        .into_iter()
+                        .next()
+                        .map(|(s, _)| s.id)
+                        .ok_or("No display available".to_string())?,
+                };


If this is meant to screenshot the primary display, relying on next() from list_displays() might pick a non-primary display depending on enumeration order.

Suggested change

let capture_target = ScreenCaptureTarget::Display {

id: cap_recording::screen_capture::list_displays()

.into_iter()

.next()

.map(|(s, _)| s.id)

.ok_or("No display available".to_string())?,

};

let primary_id = scap_targets::Display::primary().id();

let mut displays = cap_recording::screen_capture::list_displays().into_iter();

let id = displays

.find(|(s, _)| s.id == primary_id)

.or_else(|| displays.next())

.map(|(s, _)| s.id)

.ok_or_else(|| "No display available".to_string())?;

let capture_target = ScreenCaptureTarget::Display { id };

wasim-builds · 2026-07-29T07:55:17Z

Fixed the screenshot target selection to use the primary display instead of relying on enumeration order. Also removed the unused variable. Both issues from the Greptile review have been addressed in commit f35fe41.

wasim-builds · 2026-07-29T07:55:25Z

Fixed the screenshot target selection to use the primary display instead of relying on enumeration order. Also removed the unused state variable. Both issues from the Greptile review have been addressed in commit f35fe41.

…o fmt

tembo · 2026-07-29T13:15:35Z

+                let primary_id = scap_targets::Display::primary().id();
+                let displays = cap_recording::screen_capture::list_displays();
+                let id = displays
+                    .iter()
+                    .find(|(s, _)| s.id == primary_id)
+                    .or_else(|| displays.first())
+                    .map(|(s, _)| s.id)
+                    .ok_or_else(|| "No display available".to_string())?;
+                let capture_target = ScreenCaptureTarget::Display { id };


Minor: you can simplify this by not enumerating displays / falling back to first() (which could capture the wrong monitor if primary-id lookup fails). Using the primary display id directly matches the existing tray/hotkey screenshot behavior.

Suggested change

let primary_id = scap_targets::Display::primary().id();

let displays = cap_recording::screen_capture::list_displays();

let id = displays

.iter()

.find(|(s, _)| s.id == primary_id)

.or_else(|| displays.first())

.map(|(s, _)| s.id)

.ok_or_else(|| "No display available".to_string())?;

let capture_target = ScreenCaptureTarget::Display { id };

let id = scap_targets::Display::primary().id();

let capture_target = ScreenCaptureTarget::Display { id };

tembo · 2026-07-29T13:31:39Z

+            DeepLinkAction::TakeScreenshot => {
+                let id = scap_targets::Display::primary().id();
+                let capture_target = ScreenCaptureTarget::Display { id };
+                crate::recording::take_screenshot(app.clone(), capture_target).await


take_screenshot returns a PathBuf, but execute() returns Result<(), String>.

Suggested change

crate::recording::take_screenshot(app.clone(), capture_target).await

crate::recording::take_screenshot(app.clone(), capture_target)

.await

.map(|_| ())

tembo · 2026-07-30T04:22:02Z

 		const set = this.listeners.get(event);
-		if (set) set.add(handler as EventHandler<keyof RecorderEventMap>);
+		if (set) set.add(handler as unknown as EventHandler<keyof RecorderEventMap>);
 		return () => {
-			this.listeners.get(event)?.delete(handler);
+			this.listeners.get(event)?.delete(handler as unknown as EventHandler<keyof RecorderEventMap>);
 		};


This works, but it’s a bit hard to read (and pretty long). You can hoist the cast once and reuse it for add/remove.

Suggested change

const set = this.listeners.get(event);

if (set) set.add(handler as EventHandler<keyof RecorderEventMap>);

if (set) set.add(handler as unknown as EventHandler<keyof RecorderEventMap>);

return () => {

this.listeners.get(event)?.delete(handler);

this.listeners.get(event)?.delete(handler as unknown as EventHandler<keyof RecorderEventMap>);

};

const set = this.listeners.get(event);

const castHandler =

handler as unknown as EventHandler<keyof RecorderEventMap>;

if (set) set.add(castHandler);

return () => {

this.listeners.get(event)?.delete(castHandler);

};

wasim-builds · 2026-07-31T05:44:28Z

Bumping for review. The deeplink extensions add pause/resume/toggle/restart/mic/camera/screenshot actions. Removed debug_assertions gating, fixed screenshot target to use primary display, and verified all new actions work correctly. Happy to make any adjustments!

wasim-builds · 2026-07-31T10:13:35Z

Bumping for review. Ready to address any feedback. Let me know if you need changes.

feat: extend deeplinks with pause/resume/toggle/restart/mic/camera/sc…

3bd73e6

…reenshot actions

superagent-security Bot added the contributor:flagged Contributor flagged for review by trust analysis. label Jul 29, 2026

greptile-apps Bot reviewed Jul 29, 2026

View reviewed changes

tembo Bot reviewed Jul 29, 2026

View reviewed changes

fix: use primary display for screenshot target, prefix unused state

f35fe41

fix(deeplinks): handle iterator exhaustion in screenshot and run carg…

ef79d2d

…o fmt

tembo Bot reviewed Jul 29, 2026

View reviewed changes

refactor: simplify display lookup per review

5fb6e9a

tembo Bot reviewed Jul 29, 2026

View reviewed changes

fix: Resolve TS error in sdk-recorder

f43c218

tembo Bot reviewed Jul 30, 2026

View reviewed changes

wasim-builds added 2 commits July 30, 2026 09:55

refactor: Hoist type cast as suggested

34d607e

Fix type mismatches in deeplink actions

fcca574

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: extend deeplinks with pause/resume/toggle/restart/mic/camera/screenshot - #2049

feat: extend deeplinks with pause/resume/toggle/restart/mic/camera/screenshot#2049
wasim-builds wants to merge 7 commits into
CapSoftware:mainfrom
wasim-builds:bounty-deeplinks-raycast

wasim-builds commented Jul 29, 2026 •

edited by greptile-apps Bot

Loading

Uh oh!

superagent-security Bot commented Jul 29, 2026

Uh oh!

greptile-apps Bot Jul 29, 2026

Uh oh!

tembo Bot Jul 29, 2026

Uh oh!

tembo Bot Jul 29, 2026

Uh oh!

wasim-builds commented Jul 29, 2026

Uh oh!

wasim-builds commented Jul 29, 2026

Uh oh!

tembo Bot Jul 29, 2026

Uh oh!

tembo Bot Jul 29, 2026

Uh oh!

tembo Bot Jul 30, 2026

Uh oh!

wasim-builds commented Jul 31, 2026

Uh oh!

wasim-builds commented Jul 31, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

	let state = app.state::<ArcLock<App>>();
	let _state = app.state::<ArcLock<App>>();

Uh oh!

Conversation

wasim-builds commented Jul 29, 2026 • edited by greptile-apps Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

deeplink_actions.rs

Acceptance Criteria

Greptile Summary

Confidence Score: 4/5

Important Files Changed

Uh oh!

superagent-security Bot commented Jul 29, 2026

Uh oh!

greptile-apps Bot Jul 29, 2026

Choose a reason for hiding this comment

Uh oh!

tembo Bot Jul 29, 2026

Choose a reason for hiding this comment

Uh oh!

tembo Bot Jul 29, 2026

Choose a reason for hiding this comment

Uh oh!

wasim-builds commented Jul 29, 2026

Uh oh!

wasim-builds commented Jul 29, 2026

Uh oh!

tembo Bot Jul 29, 2026

Choose a reason for hiding this comment

Uh oh!

tembo Bot Jul 29, 2026

Choose a reason for hiding this comment

Uh oh!

tembo Bot Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

wasim-builds commented Jul 31, 2026

Uh oh!

wasim-builds commented Jul 31, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

wasim-builds commented Jul 29, 2026 •

edited by greptile-apps Bot

Loading