Skip to content

Conversation

@dileepyavan
Copy link
Member

fixes:#291022
For linux, bubblewrap, socat and ripgrep should be installed. Updated the setting UI to show this

image

Copilot AI review requested due to automatic review settings January 29, 2026 17:12
@dileepyavan dileepyavan enabled auto-merge (squash) January 29, 2026 17:13
@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 29, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the terminal sandbox settings descriptions to surface required tooling so users understand the prerequisites for enabling the sandbox, especially on Linux. The changes are limited to configuration metadata used by the Settings UI.

Changes:

  • Updated the Linux terminal sandbox file system setting description to note that bubblewrap, socat, and ripgrep must be installed.
  • Updated the macOS terminal sandbox file system setting description to note that ripgrep must be installed.
Comments suppressed due to low confidence (1)

src/vs/workbench/contrib/terminalContrib/chatAgentTools/common/terminalChatAgentToolsConfiguration.ts:577

  • The macOS sandbox description also has spacing issues: there should be a space after "macOS." and a space before the parenthesis in "patterns (ex: …)". Cleaning this up will make the user-facing text read more naturally and match the Linux description style.
		markdownDescription: localize('terminalSandbox.macFileSystemSetting', "Note: this setting is applicable only when {0} is enabled. Controls file system access in the terminal sandbox on macOS.Paths also support git-style glob patterns(ex: *.ts, ./src, ./src/**/*.ts, file?.txt). **ripgrep** should be installed for this setting to work.", `\`#${TerminalChatAgentToolsSettingId.TerminalSandboxEnabled}#\``),

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@dileepyavan dileepyavan merged commit d6d5aec into main Jan 29, 2026
22 checks passed
@dileepyavan dileepyavan deleted the DileepY/sandbox_prereqs1 branch January 29, 2026 18:52
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.

3 participants