-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Simplify Roo Code's quick actions #3024
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
Conversation
|
|
Q: Are the Commands still able to be keybound? |
It's only the "Quick Fix" options when language server diagnostics are available that seems overbearing; I ended up keeping "Improve" / "Explain" / etc in the default case. |
| import { sleep, waitFor, waitUntilCompleted } from "./utils" | ||
|
|
||
| suite("Roo Code Subtasks", () => { | ||
| suite.skip("Roo Code Subtasks", () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Avoid leaving tests permanently skipped unless strictly necessary. If skipping is temporary, please add a comment explaining why and a TODO to re-enable the test.
|
This is a good improvement! Is there an option to hide the remaining two actions? |
* Fix: Increase Ollama provider timeout from 30s to 120s * Add changeset for Ollama provider timeout fix
Context
I think there are too many actions, and the input prompt seems unnecessary; the VSCode diagnostics already tell the model exactly what's wrong so it knows what to do.
Before:
After:
When language server diagnostics are present:

Otherwise:

Important
Simplifies Roo Code's quick actions by removing unnecessary prompts and commands, updating command handling, and adjusting related tests and UI components.
registerCodeActions.tsandregisterTerminalActions.ts.registerCodeActionPairandregisterTerminalActionPairfunctions.handleCodeActionandhandleTerminalActioninClineProvider.tsto handle simplified actions.roo-cline.terminalFixCommandInCurrentTaskandroo-cline.terminalExplainCommandInCurrentTaskfrompackage.jsonand related localization files.registerCommands.tsto reflect removed commands.Roo Code Subtaskstest suite insubtasks.test.ts.CodeActionProvider.test.tsto reflect changes in available actions.CodeActionProvider.tsto removecreateActionPairand adjustprovideCodeActionslogic.ChatView.tsxto handle UI changes related to quick actions.This description was created by
for 1454e35. You can customize this summary. It will automatically update as commits are pushed.