Skip to content

Copilot using vscode's ripgrep#302323

Draft
roblourens wants to merge 1 commit intomainfrom
roblou/copilot-vscode-rg
Draft

Copilot using vscode's ripgrep#302323
roblourens wants to merge 1 commit intomainfrom
roblou/copilot-vscode-rg

Conversation

@roblourens
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings March 17, 2026 02:35
@roblourens roblourens self-assigned this Mar 17, 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

Updates the Copilot agent host integration so the Copilot CLI subprocess can use VS Code’s shipped ripgrep instead of relying on a Copilot-bundled ripgrep being copied into the SDK during packaging.

Changes:

  • Add @vscode/ripgrep’s binary directory to the Copilot CLI subprocess PATH (and set USE_BUILTIN_RIPGREP=0) in the agent host.
  • Stop copying ripgrep into @github/copilot/... during packaging; keep only node-pty prebuild copying.
  • Remove macOS universal-app and Mach-O verification special-casing for Copilot’s ripgrep directory.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/vs/platform/agentHost/node/copilot/copilotAgent.ts Injects VS Code’s ripgrep location into the Copilot CLI subprocess environment.
build/gulpfile.vscode.ts Removes packaging-time copying of ripgrep into the Copilot SDK layout.
build/darwin/verify-macho.ts Stops skipping Copilot ripgrep paths during Mach-O architecture validation.
build/darwin/create-universal-app.ts Removes cross-copying and arch-file patterns for Copilot ripgrep in universal app creation.

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