anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-07-23 02:36:57 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

[PR #1833] [CLOSED] fix: Grep tool Windows path parsing issue #10076

New Issue
Closed
opened 2026-02-16 18:14:40 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 18:14:40 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/anomalyco/opencode/pull/1833
Author: @ledmaster
Created: 8/11/2025
Status: ❌ Closed

Base: dev ← Head: fix/grep-windows-path-parsing

📝 Commits (7)

📊 Changes

2 files changed (+28 additions, -6 deletions)

View changed files

📝 .github/workflows/guidelines-check.yml (+1 -1)
📝 packages/opencode/src/tool/grep.ts (+27 -5)

📄 Description

Fixed an issue where the Grep tool failed to parse ripgrep output on Windows systems due to colons in drive letters (e.g., C:\path\to\file). The fix identifies the line number by its numeric nature rather than assuming a fixed position, making it compatible with both Windows and Unix-style paths.

🔄 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/1833 **Author:** [@ledmaster](https://github.com/ledmaster) **Created:** 8/11/2025 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `fix/grep-windows-path-parsing` --- ### 📝 Commits (7) - [`d3f26a9`](https://github.com/anomalyco/opencode/commit/d3f26a95cff53e7f633ea2c3d1e26b4ac22f4042) fix: Grep tool Windows path parsing issue - [`05517df`](https://github.com/anomalyco/opencode/commit/05517df3d2127225be42ef62602f1a437597f94d) Merge branch 'sst:dev' into fix/grep-windows-path-parsing - [`1fe81e5`](https://github.com/anomalyco/opencode/commit/1fe81e5521a79e7fd245dee5d32c85774f5d4bcb) Merge branch 'dev' of https://github.com/sst/opencode into dev - [`9aa0700`](https://github.com/anomalyco/opencode/commit/9aa0700dc42bf1c8040a6ca5edc6b4f294e6070c) ci: sync - [`9543114`](https://github.com/anomalyco/opencode/commit/9543114778b1f9806cb951ed31f1a7e6dadb14b0) Merge branch 'dev' into fix/grep-windows-path-parsing - [`8f7bae7`](https://github.com/anomalyco/opencode/commit/8f7bae7641a9c38d730b6afa518e195bfdcae1a2) fix: replace let with const in grep tool line number parsing - [`f4a96dc`](https://github.com/anomalyco/opencode/commit/f4a96dc546e1416833f1997224f71394dd5bb106) Merge branch 'fix/grep-windows-path-parsing' of https://github.com/ledmaster/opencode into fix/grep-windows-path-parsing ### 📊 Changes **2 files changed** (+28 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/guidelines-check.yml` (+1 -1) 📝 `packages/opencode/src/tool/grep.ts` (+27 -5) </details> ### 📄 Description Fixed an issue where the Grep tool failed to parse ripgrep output on Windows systems due to colons in drive letters (e.g., C:\\path\\to\\file). The fix identifies the line number by its numeric nature rather than assuming a fixed position, making it compatible with both Windows and Unix-style paths. --- <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:14:40 -05:00
yindo closed this issue 2026-02-16 18:14:40 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
acp
bad
bug
discussion
docs
documentation
enhancement
extension
good first issue
growth
help-wanted
mcp
model-problem
needs:compliance
nix
opentui
perf
pull-request
Mirrored from GitHub Pull Request
 tui
web
windows
zen
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#10076
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?