-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Include changes from Roo v3.25.8 #1827
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
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
Co-authored-by: Roo Code <roomote@roocode.com> Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
…ance (#5867) Co-authored-by: Roo Code <roomote@roocode.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> Co-authored-by: Daniel <57051444+daniel-lxs@users.noreply.github.com> Co-authored-by: Daniel Riccio <ricciodaniel98@gmail.com>
Co-authored-by: Daniel Riccio <ricciodaniel98@gmail.com>
…erruption (#6296)
* Cloud: use events in SettingsService * Cloud: simplify AuthService events * Cloud: convert CloudService to an EventEmitter * Apply suggestions from code review Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com> --------- Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
Co-authored-by: Daniel Riccio <ricciodaniel98@gmail.com>
* Clean up the auto-approve UI * Update webview-ui/src/components/settings/AutoApproveSettings.tsx Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com> * Fix * Translations --------- Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
Co-authored-by: Roo Code <roomote@roocode.com>
Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
Co-authored-by: Roo Code <roomote@roocode.com>
Co-authored-by: Roo Code <roomote@roocode.com>
Co-authored-by: Roo Code <roomote@roocode.com>
Co-authored-by: mrubens <2600+mrubens@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
Co-authored-by: Roo Code <roomote@roocode.com>
…g (#6411) * feat: conditionally include reminder section based on todo list configuration * feat: add tests for REMINDERS section based on todoListEnabled configuration
* feat: Before requesting, ensure checkpoint is initialized * Generate a checkpoint before modifying the code * refactor: streamline checkpoint handling and enhance getCheckpoints method * Blocked waiting for checkpoint initialization timing to change * cancel checkpoint restore limit * fix: ensure checkpoint service is undefined on initialization error and improve checkpoint diff handling * refactor: simplify checkpoint service initialization and cleanup unused variables in CheckpointMenu * fix: prevent race condition in checkpoint service initialization - Only assign service to cline.checkpointService after successful initialization - Add proper cleanup on initialization failure - Prevents service from being in inconsistent state if Git check fails * fix: remove checkpoint save from presentAssistantMessage for update_todo_list case --------- Co-authored-by: Daniel Riccio <ricciodaniel98@gmail.com>
|
| "Z.ai's new lightweight flagship model delivers SOTA performance with exceptional cost-effectiveness!", | ||
| "GLM-4.5-Air is the lightweight version of GLM-4.5. It balances performance and cost-effectiveness, and can flexibly switch to hybrid thinking models.", | ||
| }, | ||
| "glm-4.5-flash": { |
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.
Can you re-add this one? It's free so it's handy for testing.
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.
Just add them to both arrays, we'll see what happens.
|
You probably want to pull in this one too RooCodeInc/Roo-Code#6808 |
…evert rest to Roo (#1833) * Split out a Kilo-specific version of the TaskHeader, KiloTaskHeader Revert `TaskHeader` and `TaskHeader.spec.tsx` to match the RU implementation exactly, and then split out all Kilo-specific logic into the `KiloTaskHeader`. Create stories for both so we can watch for changes. * refactor(storybook): update story titles for consistency * style(ui): replace VSCodeBadge with span for total cost display --------- Co-authored-by: Chris Hasson <noreply@example.com>
|
Thanks @mrubens! |
| "openAiApiKey", | ||
| "geminiApiKey", | ||
| "openAiNativeApiKey", | ||
| "cerebrasApiKey", |
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.
There's zaiApiKey inside //killcode_change below, is that correct?
| "cerebras", | ||
| "sambanova", | ||
| "zai", | ||
| "fireworks", |
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.
These 3 are now duplicated
| "anthropic/claude-3.7-sonnet:thinking", | ||
| "anthropic/claude-sonnet-4", | ||
| "anthropic/claude-opus-4", | ||
| "anthropic/claude-opus-4.1", |
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.
👍
| mode?: string | ||
| } | ||
|
|
||
| export interface TaskProviderLike { |
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.
Lol, can we use this for #1821?
or are they also working on it?
| super({ | ||
| ...options, | ||
| providerName: "Z AI", | ||
| baseURL: isChina ? "https://open.bigmodel.cn/api/paas/v4" : "https://api.z.ai/api/paas/v4", |
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.
BigModel is China? Does that mean we can delete that provider?
(we can figure this out later ofc)
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.
I think it does but lets figure it out later indeed
|
|
||
| switch (block.name) { | ||
| case "write_to_file": | ||
| await checkpointSaveAndMark(cline) |
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.
🕺 we should resolve all the checkpoint issues after releasing this.
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.
agreed, or at least comment and ask if it resolves the issue
| "unlimited": "Unlimited" | ||
| }, | ||
| "apiCostLimit": { | ||
| "title": "Max Cost", |
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.
Were we planning on making this?
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.
We were, so I'll find the ticket and close it once we merge this
|
|
Attention points: