Skip to content

macOS: defer microphone/computer-control permission asks to dashboard tasks#2881

Merged
awlevin merged 1 commit into
feature/dashboard-main-onboardingfrom
swarm/task-2
Feb 15, 2026
Merged

macOS: defer microphone/computer-control permission asks to dashboard tasks#2881
awlevin merged 1 commit into
feature/dashboard-main-onboardingfrom
swarm/task-2

Conversation

@awlevin

@awlevin awlevin commented Feb 15, 2026

Copy link
Copy Markdown
Contributor

Context

Part of #2864
Closes #2869

Changes

  • Removed microphone (SpeechPermissionStepView) and screen recording (ScreenPermissionStepView) permission steps from the default onboarding flow
  • Hatch + first conversation no longer includes privileged permission requests
  • Accessibility permission step retained since it may be needed for basic functionality
  • Permission views preserved for later reuse by dashboard tasks
  • Updated onboarding state machine: crack progress mapping, step clamping, and anyPermissionDenied to only check accessibility
  • Updated stage image mapping and egg scene animation triggers for the reduced 6-step flow
  • Updated progress dots default from 8 to 6 total steps
  • Fixed pre-existing merge conflict in DynamicPageSurfaceView.swift and argument ordering in SurfaceContainerView.swift

Flow change

Before (8 steps): WakeUp -> Naming -> APIKey -> FnKey -> Speech -> Accessibility -> Screen -> Alive -> Interview

After (6 steps): WakeUp -> Naming -> APIKey -> FnKey -> Accessibility -> Alive -> Interview

Validation

  • Swift build passes

Open with Devin

… tasks

Remove SpeechPermissionStepView (microphone) and ScreenPermissionStepView
(screen recording) from the default onboarding flow so hatch + first
conversation no longer include privileged permission requests. Accessibility
permission is retained as it may be needed for basic functionality.

The permission views are preserved for later reuse by dashboard tasks.

Flow reduced from 8 steps (0-7 + interview) to 6 steps (0-5 + interview):
  0: WakeUp  1: Naming  2: APIKey  3: FnKey  4: Accessibility  5: Alive

Also fixes pre-existing merge conflict in DynamicPageSurfaceView.swift and
argument ordering in SurfaceContainerView.swift.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@awlevin awlevin self-assigned this Feb 15, 2026
@awlevin awlevin merged commit f8b909b into feature/dashboard-main-onboarding Feb 15, 2026
@awlevin awlevin deleted the swarm/task-2 branch February 15, 2026 23:41

@devin-ai-integration devin-ai-integration 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 5 additional findings.

Open in Devin Review

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