Skip to content

feat(lens): allow microphone and speaker access - #341

Merged
JongSikLim merged 1 commit into
mainfrom
feat/lens-audio-permissions
Aug 6, 2026
Merged

feat(lens): allow microphone and speaker access#341
JongSikLim merged 1 commit into
mainfrom
feat/lens-audio-permissions

Conversation

@JongSikLim

Copy link
Copy Markdown
Contributor

Summary

  • allow the active Lens page to play audio, capture microphone input, and select an audio output device while rejecting camera, unrelated, popup, and stale-view requests
  • add the macOS microphone usage description and audio-input entitlements
  • document the permission behavior and add focused policy and packaging tests

Verification

  • bun test tests/lens-audio-permissions.test.ts tests/lens-browser-manager.test.ts tests/lens-url.test.ts tests/scripts-smoke.test.ts (39 passed)
  • bun run typecheck
  • bun run build:desktop
  • bun run check:doc-paths
  • bun run check:max-lines-ratchet
  • bun run check:switch-exhaustiveness
  • plutil -lint build/entitlements.mac.plist build/entitlements.mac.inherit.plist
  • Electron 41 Lens runtime smoke: acquired one live microphone track, selected an output sink with setSinkId, and rejected camera capture with NotAllowedError

@JongSikLim
JongSikLim requested a review from a team as a code owner August 6, 2026 04:50
@JongSikLim
JongSikLim requested review from heath-s and schang124 and removed request for a team August 6, 2026 04:50
@JongSikLim
JongSikLim enabled auto-merge (squash) August 6, 2026 04:52

@heath-s heath-s left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved

@JongSikLim
JongSikLim merged commit 2faf080 into main Aug 6, 2026
4 checks passed
@JongSikLim
JongSikLim deleted the feat/lens-audio-permissions branch August 6, 2026 04:55
@JongSikLim JongSikLim mentioned this pull request Aug 6, 2026
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.

2 participants