-
Notifications
You must be signed in to change notification settings - Fork 1.7k
fix(cli): improve error message for custom mode not found #4861
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.
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
fix(cli): improve error message for custom mode not found #4861
Conversation
🦋 Changeset detectedLatest commit: 6583bcc The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@benzntech Appears to me that the PR's are stacked on top of each other? Would you be able to split the CLI from the PR? Or are they dependent? Thanks! |
- Changed error from 'Invalid mode' to 'Mode not found' with search path details - Added SearchedPath interface and getSearchedPaths() export for error reporting - Added debug logging to help diagnose custom modes loading issues - Shows exactly where CLI searched for custom modes (global and project paths) - Added 14 unit tests for customModes.ts Fixes Kilo-Org#4575, fixes Kilo-Org#4600
91381bc to
8fcf303
Compare
updated @marius-kilocode |
|
Looks like a good improvement for usability @benzntech . In the best case we reuse the same mechanism for both cases:
|
@marius-kilocode updated |
|
Recreated in: #4982 |

Summary
This PR improves the CLI error message when a custom mode is not found, addressing issues #4575 and #4600.
Changes
cli/src/config/customModes.tsSearchedPathinterface to track search locationsgetSearchedPaths()export for error reportingcli/src/index.tscli/src/config/__tests__/customModes.test.tsBefore
After
Testing
Fixes #4575
Fixes #4600