Skip to content

improvement(computer-use): allow every app by default, protect sensitive apps, and make per-app approval optional - #9980

Merged
waleedlatif1 merged 4 commits into
stagingfrom
feat/computer-use-app-access
Nov 13, 2026
Merged

waleedlatif1 merged 4 commits into
stagingfrom
feat/computer-use-app-access

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Computer Use no longer asks before every app: once it is on and Accessibility + Screen Recording are granted, Sim can use any app except Sim itself, System Settings, password managers (Keychain Access, Passwords, 1Password, Bitwarden, KeePassXC, LastPass, Enpass, Proton Pass, Strongbox) and terminals (Terminal, iTerm, Warp, Ghostty — shell access stays behind the separate Terminal control). The helper refuses those apps; bundle IDs match case-insensitively
  • Approving each app's first use is now an opt-in Ask before using a new app setting (computerUseAskForApps, new computer-use:set-ask-for-apps channel with the same gates as set-enabled). Sign-out/deployment reset clears it with the opt-in and approvals
  • Settings → Desktop → Computer Use is trimmed to the switch, one line on what is excluded, the two macOS permission rows (shown once it is on, with an Allow button), the ask toggle, and Approved apps only in ask mode
  • Docs and desktop README describe the new model
  • Compatibility: an older desktop shell reports no askForApps, so new web keeps its approval list and it keeps prompting as before. Existing users move to allow-all on upgrade (computer use was org-limited until v0.9.55). A self-hosted deployment running older web with this shell keeps the old "each app requires your approval" line until web updates

Type of Change

  • Improvement

Testing

  • apps/desktop: computer-use service + IPC suites (100 tests); new default-mode test fails without the change
  • apps/sim: settings desktop + computer-use component suites
  • lint, check:client-boundary, check:test-patterns, check:comment-hygiene, check:api-validation:strict; full gate in CI

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

… managers, make per-app approval optional, and trim the settings

Turning Computer Use on and granting Accessibility and Screen Recording now covers
every app except Sim, System Settings and password managers. Approving each app's
first use becomes an opt-in setting, and the settings section keeps only the switch,
the two macOS permissions and the optional approvals.
…-insensitively, and remember unlisted app names
@vercel

vercel Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Nov 13, 2026 4:45am UTC

Request Review

@cubic-dev-ai cubic-dev-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.

All reported issues were addressed across 12 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[High impact] The PR appears safe to merge; no new actionable issues were found.

Summary

Computer Use allows unprotected apps by default and makes first-use approval optional.

  • Computer Use opens ordinary apps by default and keeps sensitive apps blocked.
  • Computer Use settings show permissions and approvals only when they apply.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  Request[Authorized app request] --> Mode{Ask mode enabled?}
  Mode -->|No| Guard
  Mode -->|Yes| Approval{Saved or task approval?}
  Approval -->|Yes| Guard
  Approval -->|No| Prompt[Ask the user]
  Prompt -->|Allowed| Guard
  Prompt -->|Denied| Stop[Reject request]
  Guard{Protected app?} -->|Yes| Stop
  Guard -->|No| App[Observe or control app]
Loading

Reviews (3) · Last reviewed commit: "fix(computer-use): describe ask mode in ..." · Reviewed by Greptile

…rd, and keep the approval copy for older shells
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-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.

All reported issues were addressed across 13 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-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.

No issues found across 14 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Turn on auto-fix | Re-trigger cubic

This branch was previously deployed

1 inactive deployment
Preview — 963b06b3 Deployed Nov 13, 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