Skip to content

Conversation

@uinstinct
Copy link
Contributor

@uinstinct uinstinct commented Nov 13, 2025

Description

Filter out file lines with more than 1000 characters when ripgrep search.

resolves CON-4237

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • [] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Summary by cubic

Filter CLI ripgrep search results to exclude lines longer than 1000 characters so results are cleaner and more useful. Addresses CON-4237; if all matches are filtered, we now return "No matches found" and added a unit test for the new behavior.

Written for commit bde28fb. Summary will update automatically on new commits.

@uinstinct uinstinct requested a review from a team as a code owner November 13, 2025 12:49
@uinstinct uinstinct requested review from RomneyDa and removed request for a team November 13, 2025 12:49
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Nov 13, 2025
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Nov 17, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 17, 2025
@sestinj sestinj merged commit ae57291 into continuedev:main Nov 17, 2025
55 of 59 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Nov 17, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Nov 17, 2025
@uinstinct uinstinct deleted the cli-search-line-filter branch November 18, 2025 04:47
@sestinj
Copy link
Contributor

sestinj commented Nov 18, 2025

🎉 This PR is included in version 1.29.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Nov 19, 2025

🎉 This PR is included in version 1.34.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Nov 19, 2025

🎉 This PR is included in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Nov 20, 2025

🎉 This PR is included in version 1.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer released size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants