Skip to content

Limit glob tool output #6208

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 5 commits into from
Jun 21, 2025
Merged

Limit glob tool output #6208

merged 5 commits into from
Jun 21, 2025

Conversation

RomneyDa
Copy link
Collaborator

  • Adds a maxResults option to IDE getFileResults methods
  • Implements it for the Glob Tool - limits to 100 results for now, not configurable, warns model when limit reached.
  • Updates the docs accordingly

@RomneyDa RomneyDa requested a review from a team as a code owner June 20, 2025 02:11
@RomneyDa RomneyDa requested review from Patrick-Erichsen and removed request for a team June 20, 2025 02:11
@netlify Netlify
Copy link

netlify bot commented Jun 20, 2025 •
edited
Loading

✅ Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 3ae9cc0
🔍 Latest deploy log https://app.netlify.com/projects/continuedev/deploys/68561eeb1e942e0008fb3adf

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jun 20, 2025
@recurseml recurseML
Copy link

recurseml bot commented Jun 20, 2025

⚠️ Only 5 files will be analyzed due to processing limits.

recurseml[bot]
recurseml bot reviewed Jun 20, 2025
View reviewed changes
@recurseml recurseML
Copy link

recurseml bot commented Jun 20, 2025

😱 Found 1 issue. Time to roll up your sleeves! 😱

@RomneyDa RomneyDa requested review from sestinj and removed request for sestinj June 20, 2025 22:56
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 21, 2025
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Jun 21, 2025
@Patrick-Erichsen Patrick-Erichsen merged commit e6406de into main Jun 21, 2025
37 checks passed
@Patrick-Erichsen Patrick-Erichsen deleted the dallin/limit-glob-output branch June 21, 2025 04:47
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Jun 21, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jun 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@recurseml recurseml[bot] recurseml[bot] left review comments

@Patrick-Erichsen Patrick-Erichsen Patrick-Erichsen approved these changes

Assignees
No one assigned
Labels
lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
Issues and PRs
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@RomneyDa @Patrick-Erichsen