anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-07-22 10:15:31 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

Model tends to misuse Grep tool #1975

New Issue
Closed
opened 2026-02-16 17:33:31 -05:00 by yindo · 12 comments
yindo commented 2026-02-16 17:33:31 -05:00
Owner
Copy Link
Copy Source

Originally created by @agladysh on GitHub (Oct 6, 2025).

Originally assigned to: @thdxr on GitHub.

Grep *.ts (path=packages/minimal-cli/src/main.ts, pattern=lint)

No files found

The path argument seems to be for directories only, or something (the file is there, and contains the pattern).

Update:

Strangely enough, not specifying the path argument often fails as well:

Grep packages/minimal-cli/src/main.ts (pattern=reviewHowto)

No files found

Grep packages/minimal-cli/src/main.ts (pattern=review-howto)

No files found

Grep reviewHowto

Found 10 matches
/Users/<redacted>/packages/minimal-cli/src/main.ts:
  Line 715:   reviewHowto?: string;
  Line 761:       result.reviewHowto = args[++i];
  Line 864:   reviewHowto: { title: string; value: string },
  Line 878:   contexts.push(`<context># ${reviewHowto.title}\n\n${reviewHowto.value}</context>`);
  Line 891:   reviewHowto: { title: string; value: string },
  Line 898:   const contexts = buildContexts(guideline, howto, reviewHowto, projectRoot, allFiles, readFile);
  Line 1062:     const reviewHowtoPath = cliArgs.reviewHowto || 'review/howto-review-common.yaml';
  Line 1063:     const reviewHowto = loadHowtoYaml(reviewHowtoPath);
Originally created by @agladysh on GitHub (Oct 6, 2025). Originally assigned to: @thdxr on GitHub. ``` Grep *.ts (path=packages/minimal-cli/src/main.ts, pattern=lint) No files found ``` The `path` argument seems to be for directories only, or something (the file is there, and contains the pattern). Update: Strangely enough, not specifying the path argument often fails as well: ``` Grep packages/minimal-cli/src/main.ts (pattern=reviewHowto) No files found Grep packages/minimal-cli/src/main.ts (pattern=review-howto) No files found Grep reviewHowto Found 10 matches /Users/<redacted>/packages/minimal-cli/src/main.ts: Line 715: reviewHowto?: string; Line 761: result.reviewHowto = args[++i]; Line 864: reviewHowto: { title: string; value: string }, Line 878: contexts.push(`<context># ${reviewHowto.title}\n\n${reviewHowto.value}</context>`); Line 891: reviewHowto: { title: string; value: string }, Line 898: const contexts = buildContexts(guideline, howto, reviewHowto, projectRoot, allFiles, readFile); Line 1062: const reviewHowtoPath = cliArgs.reviewHowto || 'review/howto-review-common.yaml'; Line 1063: const reviewHowto = loadHowtoYaml(reviewHowtoPath); ```
yindo closed this issue 2026-02-16 17:33:31 -05:00
yindo commented 2026-02-16 17:33:31 -05:00
Author
Owner
Copy Link
Copy Source

@github-actions[bot] commented on GitHub (Oct 6, 2025):

This issue might be a duplicate of existing issues. Please check:

  • #2375: Glob/grep Tool Max Lines (Limit output of all tools?) - discusses safety issues and potential problems with grep/glob tool functionality

Feel free to ignore if none of these address your specific case.

@github-actions[bot] commented on GitHub (Oct 6, 2025): This issue might be a duplicate of existing issues. Please check: - #2375: Glob/grep Tool Max Lines (Limit output of all tools?) - discusses safety issues and potential problems with grep/glob tool functionality Feel free to ignore if none of these address your specific case.
yindo commented 2026-02-16 17:33:31 -05:00
Author
Owner
Copy Link
Copy Source

@rekram1-node commented on GitHub (Oct 6, 2025):

hm will look into it

@rekram1-node commented on GitHub (Oct 6, 2025): hm will look into it
yindo commented 2026-02-16 17:33:31 -05:00
Author
Owner
Copy Link
Copy Source

@rekram1-node commented on GitHub (Oct 6, 2025):

@agladysh path is for directories only:

    path: z.string().optional().describe("The directory to search in. Defaults to the current working directory."),
@rekram1-node commented on GitHub (Oct 6, 2025): @agladysh `path` is for directories only: ``` path: z.string().optional().describe("The directory to search in. Defaults to the current working directory."), ```
yindo commented 2026-02-16 17:33:31 -05:00
Author
Owner
Copy Link
Copy Source

@rekram1-node commented on GitHub (Oct 7, 2025):

I think there is a simple fix here tho I have something working

@rekram1-node commented on GitHub (Oct 7, 2025): I think there is a simple fix here tho I have something working
yindo commented 2026-02-16 17:33:31 -05:00
Author
Owner
Copy Link
Copy Source

@rekram1-node commented on GitHub (Oct 7, 2025):

should be fixed in next release

@rekram1-node commented on GitHub (Oct 7, 2025): should be fixed in next release
yindo commented 2026-02-16 17:33:31 -05:00
Author
Owner
Copy Link
Copy Source

@agladysh commented on GitHub (Oct 8, 2025):

Seems much better, thank you!

@agladysh commented on GitHub (Oct 8, 2025): Seems much better, thank you!
yindo commented 2026-02-16 17:33:32 -05:00
Author
Owner
Copy Link
Copy Source

@lockie commented on GitHub (Jan 11, 2026):

Error: Found multiple matches for oldString. Provide more surrounding lines in oldString to identify the correct match

Noticing this all the time

@lockie commented on GitHub (Jan 11, 2026): > Error: Found multiple matches for oldString. Provide more surrounding lines in oldString to identify the correct match Noticing this all the time
yindo commented 2026-02-16 17:33:32 -05:00
Author
Owner
Copy Link
Copy Source

@DavMelchi commented on GitHub (Jan 16, 2026):

Same for me all time

Image
@DavMelchi commented on GitHub (Jan 16, 2026): Same for me all time <img width="682" height="271" alt="Image" src="https://github.com/user-attachments/assets/31dc4068-0732-4425-a78e-69187c9be40a" />
yindo commented 2026-02-16 17:33:32 -05:00
Author
Owner
Copy Link
Copy Source

@eparreno commented on GitHub (Jan 26, 2026):

Same here. Is quite annoying :(

@eparreno commented on GitHub (Jan 26, 2026): Same here. Is quite annoying :(
yindo commented 2026-02-16 17:33:32 -05:00
Author
Owner
Copy Link
Copy Source

@turningpointusa300-crypto commented on GitHub (Jan 31, 2026):

did you fix this Found multiple matches for oldString. Provide more surrounding lines in oldString to identify the correct match.
Found multiple matches for oldString. Provide more surrounding lines in oldString to identify the correct match.

@turningpointusa300-crypto commented on GitHub (Jan 31, 2026): did you fix this Found multiple matches for oldString. Provide more surrounding lines in oldString to identify the correct match. Found multiple matches for oldString. Provide more surrounding lines in oldString to identify the correct match.
yindo commented 2026-02-16 17:33:32 -05:00
Author
Owner
Copy Link
Copy Source

@rekram1-node commented on GitHub (Jan 31, 2026):

I think you all are referring to edit tool instead? Probably best mentioned in a fresh issue w/ the model and provider you are using.

Some models are much better than others

@rekram1-node commented on GitHub (Jan 31, 2026): I think you all are referring to edit tool instead? Probably best mentioned in a fresh issue w/ the model and provider you are using. Some models are much better than others
yindo commented 2026-02-16 17:33:32 -05:00
Author
Owner
Copy Link
Copy Source

@turningpointusa300-crypto commented on GitHub (Feb 1, 2026):

yes that why i am tell you so yall can fix it or do i need to run gethub

@turningpointusa300-crypto commented on GitHub (Feb 1, 2026): yes that why i am tell you so yall can fix it or do i need to run gethub
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
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#1975
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?