Skip to content

VSCode shim + basic cli #10452

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.

Sign up for GitHub

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

Jump to bottom
Merged
cte merged 20 commits into from
Jan 5, 2026
Merged

VSCode shim + basic cli #10452

cte merged 20 commits into from
Jan 5, 2026

Conversation

@cte
Copy link
Collaborator

@cte cte commented Jan 4, 2026 •
edited
Loading

Big shout-out to Kilo Code for the idea and execution on the VSCode compatibility layer. Very clever solution.

Important

Adds a CLI for Roo Code with a VSCode API shim, enabling command line execution and extensive testing.

  • CLI Implementation:
    • Adds @roo-code/cli for running Roo Code agent from the command line.
    • Supports interactive and non-interactive modes with options for API provider, model, and workspace.
    • Implements command line parsing in index.ts and task execution in extension-host.ts.
  • VSCode Shim:
    • Introduces @roo-code/vscode-shim to provide a VSCode API compatibility layer.
    • Includes classes like Uri, Position, Range, and APIs like WindowAPI, WorkspaceAPI.
    • Supports extension activation and message handling in a Node.js environment.
  • Testing:
    • Extensive unit and integration tests for CLI and VSCode shim in __tests__ directories.
    • Tests cover command execution, file operations, and API interactions.
  • Miscellaneous:
    • Updates package.json and adds eslint.config.mjs for linting.
    • Adds tsup.config.ts and vitest.config.ts for build and test configurations.

This description was created by Ellipsis for e74bbd3. You can customize this summary. It will automatically update as commits are pushed.

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

roomote bot commented Jan 4, 2026 •
edited
Loading

Rooviewer Clock   See task on Roo Cloud

All previously flagged issues have been addressed. No new issues found in the latest commit (f8d14c8).

  • Debug log file created unconditionally in workspace (extension-host.ts:221)
  • Unused ora import and dead spinner code (extension-host.ts:16)
  • Package documentation comment says @roo-code/cli instead of @roo-code/vscode-shim (index.ts:1-4)
Previous reviews

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

roomote[bot]
roomote bot reviewed Jan 4, 2026
View reviewed changes
apps/cli/src/extension-host.ts Outdated
this.setupQuietMode()

// Initialize debug log for streaming analysis
this.initDebugLog()
Copy link
Contributor

@roomote roomote bot Jan 4, 2026

Choose a reason for hiding this comment

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

This unconditionally creates a cli-stream-debug.log file in the workspace directory every time the CLI is activated. This could clutter user workspaces with debug artifacts. Consider gating this behind the verbose option or a dedicated debug flag.

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

Copy link
Collaborator Author

@cte cte Jan 4, 2026

Choose a reason for hiding this comment

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

@roomote We don't need this log file anymore - please remove it.

Copy link
Contributor

@roomote roomote bot Jan 4, 2026 •
edited
Loading

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 issue. All local checks passed.

View commit | Revert commit

roomote[bot]
roomote bot reviewed Jan 4, 2026
View reviewed changes
apps/cli/src/extension-host.ts Outdated
import path from "path"
import { fileURLToPath } from "url"
import fs from "fs"
import ora, { type Ora } from "ora"
Copy link
Contributor

@roomote roomote bot Jan 4, 2026

Choose a reason for hiding this comment

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

The ora package is imported but all spinner methods (startSpinner, updateSpinner, stopSpinner) are disabled no-ops, and the spinner property (line 63) is never used. Consider removing the import and related dead code, or implementing the spinner functionality properly.

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

Copy link
Collaborator Author

@cte cte Jan 4, 2026

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 4, 2026 •
edited
Loading

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 issue. Removed the unused ora import, spinner property, and disabled spinner methods from the CLI extension host. All local checks passed.

View commit | Revert commit

roomote[bot]
roomote bot reviewed Jan 4, 2026
View reviewed changes
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jan 4, 2026
@ellipsis-dev
Copy link
Contributor

ellipsis-dev bot commented Jan 5, 2026

⚠️ This PR is too big for Ellipsis, but support for larger PRs is coming soon. If you want us to prioritize this feature, let us know at [email protected]

Generated with ❤️ by ellipsis.dev

@ellipsis-dev
Copy link
Contributor

ellipsis-dev bot commented Jan 5, 2026

⚠️ This PR is too big for Ellipsis, but support for larger PRs is coming soon. If you want us to prioritize this feature, let us know at [email protected]

Generated with ❤️ by ellipsis.dev

@ellipsis-dev
Copy link
Contributor

ellipsis-dev bot commented Jan 5, 2026

⚠️ This PR is too big for Ellipsis, but support for larger PRs is coming soon. If you want us to prioritize this feature, let us know at [email protected]

Generated with ❤️ by ellipsis.dev

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 5, 2026
@cte cte merged commit b11d53a into main Jan 5, 2026
13 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jan 5, 2026
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jan 5, 2026
@cte cte deleted the cte/vscode-shim branch January 5, 2026 18:24
mini2s added a commit to zgsm-ai/costrict that referenced this pull request Jan 6, 2026
@mini2s @github-actions
@cte @hannesrudolph @roomote @roomote @mrubens @ellipsis-dev @jr @daniel-lxs @brunobergher @pdecat @sebastiand-cerebras @SannidhyaSah