8000 Switch to a pnpm-based monorepo with turbo by cte · Pull Request #2677 · RooVetGit/Roo-Code · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Switch to a pnpm-based monorepo with turbo #2677

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

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Switch to a pnpm-based monorepo with turbo #2677

wants to merge 10 commits into from

Conversation

cte
Copy link
Collaborator
@cte cte commented Apr 16, 2025

Context

This allows us to carve our code up into separate packages for easy re-use across a variety of applications (VSCode extension, JetBrains extension, cli, evals, etc).

Implementation

Screenshots

before after

How to Test

Get in Touch


Important

Switches project to a pnpm-based monorepo with turbo, updating package management, build processes, and configurations.

  • Monorepo Setup:
    • Converts project to a pnpm-based monorepo using turbo.
    • Adds pnpm-workspace.yaml to define package structure.
    • Updates package.json scripts to use pnpm and turbo.
  • Build and CI:
    • Replaces npm with pnpm in GitHub Actions workflows (changeset-release.yml, code-qa.yml).
    • Updates esbuild.js and generate-types.mts to align with pnpm.
  • Configuration Changes:
    • Removes .npmrc files and flake files.
    • Updates jest.config.js and turbo.json for new setup.
  • Localization:
    • Updates localization files in src/i18n/locales to reflect pnpm usage.
  • Miscellaneous:
    • Updates .gitignore in webview-ui to ignore pnpm-related files.
    • Adjusts index.html and index.tsx in webview-ui for new module paths.

This description was created by Ellipsis for 7c5d02d. It will automatically update as commits are pushed.

Copy link
changeset-bot bot commented Apr 16, 2025

🦋 Changeset detected

Latest commit: 7c5d02d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@roo-code/vscode Patch

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

@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. enhancement New feature or request labels Apr 16, 2025
Copy link
ellipsis-dev bot commented Apr 16, 2025

This pull request is quite large, with changes spanning across 101 files and involving over 21,000 lines added and 48,000 lines removed. While the migration to pnpm seems to be a cohesive task, the updates to localization files might be unrelated to the core migration changes.

To facilitate easier review and ensure that each set of changes receives the attention it deserves, I suggest splitting the localization updates into a separate pull request. This will help reviewers focus on the migration changes and the localization updates independently.

Please consider creating a separate pull request for the localization updates. Thank you!

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Apr 16, 2025
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Apr 16, 2025
@samhvw8
Copy link
Collaborator
samhvw8 commented Apr 16, 2025

Cool, with this we can share code easier and save node_module space :<

@shubham-root
Copy link

I have worked some more on #2644, command line tool now uses monorepo structure

@qdaxb
Copy link
qdaxb commented Apr 24, 2025

@cte Any updates? We can start extracting the core modules that are unrelated to VSCode if this PR gets merged.

@cte cte marked this pull request as draft May 5, 2025 17:28
SmartManoj pushed a commit to SmartManoj/Roo-Code that referenced this pull request May 6, 2025
…ooVetGit#2677)

* initial

* Menu

* small clarity change

* cleanup

* Tests

* Update .changeset/ninety-pots-rescue.md

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>

* Update src/test/TerminalCommandApprovalSettings.test.ts

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>

* indentation

* smooth transition

* cleanup

* Consolidated enabledActionsList for display, fixed unchecking issue

* Updated shouldAutoApproveTool to better manage sub-options for future auto approve changes

* removed test

* cleanup

* comments and extra bool check

* cleanup

* change to compare ids isntead of label strings

* test in progress

* test in progress

* no tests for now

* type safety change

* cleanup

* rename

* comments

* Copy

---------

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: Saoud Rizwan <7799382+saoudrizwan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
Status: PR [Pre Approval Review]
Development

Successfully merging this pull request may close these issues.

4 participants
0