Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Jan 11, 2026

https://agentclientprotocol.com/overview/introduction

To run locally edit your zed settings (making sure to replace PATH_TO_ROO_CODE_REPO):

"agent_servers": {
  "Roo Code CLI": {
    "type": "custom",
    "command": "node",
    "args": [
      "[PATH_TO_ROO_CODE_REPO]/apps/cli/dist/index.js",
      "acp",
      "--provider",
      "openrouter",
      "--model",
      "anthropic/claude-4.5-sonnet",
    ],
  },
}

This can be done by opening the GUI settings and then clicking the button on the top right:
Screenshot 2026-01-10 at 11 38 32 PM

Then, build the extension host and cli:

pnpm --filter roo-cline bundle && \
  pnpm --filter @roo-code/cli build

Then, add an OpenRouter API key to your shell's config (in my case I used ~/.zprofile):

export OPENROUTER_API_KEY=sk-or-v1-...

And finally, in zed, open up the right dock, click the plus sign and select Roo Code CLI:
Screenshot 2026-01-10 at 11 30 59 PM
I:

If you want some friendly debugging, open up the ACP logs in zed:
Screenshot 2026-01-10 at 11 33 32 PM

Then kick off a task:
Screenshot 2026-01-10 at 11 35 30 PM

@cte cte requested review from jr and mrubens as code owners January 11, 2026 07:26
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. Enhancement New feature or request labels Jan 11, 2026
@roomote
Copy link
Contributor

roomote bot commented Jan 11, 2026

Rooviewer Clock   See task on Roo Cloud

Reviewed commit 9c9220a (release v0.0.46). All previous issues remain resolved. No new issues found.

  • agent.ts:183 - Model default inconsistency: hardcoded fallback differs from DEFAULT_FLAGS.model
  • agent.ts:226 - Model ID mismatch: getModelState() defaults to DEFAULT_MODELS[0]!.modelId while session model defaults to DEFAULT_FLAGS.model
  • types.ts:63-64 - Mismatched model name: first entry in DEFAULT_MODELS now uses DEFAULT_FLAGS.model (opus) but name/description still say "Claude Sonnet 4.5"
Previous reviews

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jan 11, 2026
| --------------------------- | -------------------------------------------- | ----------------------------- |
| `-e, --extension <path>` | Path to the extension bundle directory | Auto-detected |
| `-p, --provider <provider>` | API provider (anthropic, openai, openrouter) | `openrouter` |
| `-m, --model <model>` | Model to use | `anthropic/claude-sonnet-4.5` |
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think I merged a change to default to opus fyi

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@roomote Please fix.

Copy link
Contributor

@roomote roomote bot Jan 12, 2026

Choose a reason for hiding this comment

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

Fixaroo Clock   See task on Roo Cloud

Fixed the reported issues. Changed default model to opus and fixed model ID mismatch in getModelState().

View commit | Revert commit

* Get the current model state.
*/
private async getModelState(): Promise<AcpModelState> {
const currentModelId = this.options.model || DEFAULT_MODELS[0]!.modelId
Copy link
Contributor

Choose a reason for hiding this comment

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

Model ID mismatch: this defaults to DEFAULT_MODELS[0]!.modelId ("anthropic/claude-sonnet-4.5") while the session model at line 179 defaults to DEFAULT_FLAGS.model ("anthropic/claude-4.5-sonnet"). These are different model IDs, causing the model picker to display a different model than what the session actually uses when no model is specified.

Suggested change
const currentModelId = this.options.model || DEFAULT_MODELS[0]!.modelId
const currentModelId = this.options.model || DEFAULT_FLAGS.model

Fix it with Roo Code or mention @roomote and request a fix.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@roomote Please fix. Let's make sure the source of truth for the default model is DEFAULT_FLAGS.model and DEFAULT_MODELS should probably read from that.

Copy link
Contributor

@roomote roomote bot Jan 12, 2026

Choose a reason for hiding this comment

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

Fixaroo Clock   See task on Roo Cloud

Fixed the model ID mismatch by using DEFAULT_FLAGS.model as the single source of truth. All local checks passed.

View commit | Revert commit

Comment on lines 63 to 64
name: "Claude Sonnet 4.5",
description: "Best balance of speed and capability",
Copy link
Contributor

Choose a reason for hiding this comment

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

The model name and description no longer match the actual model. Since DEFAULT_FLAGS.model is now "anthropic/claude-opus-4.5", the name should be "Claude Opus 4.5" and the description should reflect Opus capabilities. Additionally, this creates a duplicate entry since the second model in the array is also claude-opus-4.5 with the correct name.

Suggested change
name: "Claude Sonnet 4.5",
description: "Best balance of speed and capability",
name: "Claude Opus 4.5",
description: "Most capable for complex work",

Fix it with Roo Code or mention @roomote and request a fix.

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

Labels

Enhancement New feature or request Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

5 participants