Repository navigation
Don't show full package list in "Add Dependency" completion immediately after separator - #6029
Conversation
DanTup
added
is enhancement
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #6029 +/- ##
==========================================
+ Coverage 68.23% 68.36% +0.12%
==========================================
Files 170 170
Lines 12925 12927 +2
Branches 2574 2575 +1
==========================================
+ Hits 8820 8837 +17
+ Misses 3642 3631 -11
+ Partials 463 459 -4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the AddDependencyCommand to improve the package search experience. It ensures that the full package list is not displayed when a user has entered a separator (like a space or comma) but has not yet started typing the next package name, addressing issue #5952. The logic now adds a blank entry in these cases, and the return condition was updated to check for any user input. New tests have been added to verify this behavior, and existing tests were reorganized for better structure. I have no feedback to provide.
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
…ly after separator Require at least one character to be typed. Fixes #5952
3577318 to
7a29179
Compare
7a29179 to
c9f8487
Compare
Require at least one character to be typed.
Fixes #5952