[PR #700] [MERGED] fix(tool): fix ripgrep invocation on Windows #9688

Closed
opened 2026-02-16 18:13:59 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/anomalyco/opencode/pull/700
Author: @strager
Created: 7/4/2025
Status: Merged
Merged: 7/8/2025
Merged by: @adamdotdevin

Base: devHead: rg-windows


📝 Commits (2)

  • b195a29 fix(tool): fix ripgrep invocation on Windows
  • 6b7739b Merge branch 'dev' into rg-windows

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 packages/opencode/src/file/ripgrep.ts (+1 -1)

📄 Description

On Windows, the path to rg.exe will contain backslashes (e.g.
"C:\ProgramData\chocolatey\bin\rg.exe"). This breaks our invocation
because the \ characters are interpreted specially by Bun's shell. Fix
this problem by escaping the path.

This patch fixes the tool.glob > basic test on Windows.

This patch helps #143/#631.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/anomalyco/opencode/pull/700 **Author:** [@strager](https://github.com/strager) **Created:** 7/4/2025 **Status:** ✅ Merged **Merged:** 7/8/2025 **Merged by:** [@adamdotdevin](https://github.com/adamdotdevin) **Base:** `dev` ← **Head:** `rg-windows` --- ### 📝 Commits (2) - [`b195a29`](https://github.com/anomalyco/opencode/commit/b195a29ffec170ba544829df4d8b365125ae90c7) fix(tool): fix ripgrep invocation on Windows - [`6b7739b`](https://github.com/anomalyco/opencode/commit/6b7739bab994f91e8aa2e02b32770030ad88f5a5) Merge branch 'dev' into rg-windows ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `packages/opencode/src/file/ripgrep.ts` (+1 -1) </details> ### 📄 Description On Windows, the path to rg.exe will contain backslashes (e.g. "C:\ProgramData\chocolatey\bin\rg.exe"). This breaks our invocation because the \ characters are interpreted specially by Bun's shell. Fix this problem by escaping the path. This patch fixes the tool.glob > basic test on Windows. This patch helps #143/#631. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 18:13:59 -05:00
yindo closed this issue 2026-02-16 18:13:59 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#9688