Skip to content

Conversation

@KJ7LNW
Copy link
Contributor

@KJ7LNW KJ7LNW commented Jun 3, 2025

Context

Add comprehensive documentation for monorepo structure and workflows to help models understand the repository structure better.

Implementation

This PR adds detailed documentation in .roo/rules/monorepo.md that covers:

  • Workspace organization and primary development areas
  • Package dependency management with pnpm
  • Turborepo configuration guidelines
  • Test execution strategies for both vitest and jest
  • Migration path from deprecated test formats to modern ones

This documentation will help prevent errors in package management and ensure tests are properly created and executed.

@cte Please review these guidelines to ensure they accurately reflect the monorepo structure and best practices.

Fixes: #4295


Important

Adds .roo/rules/monorepo.md for monorepo management, detailing workspace organization, dependency management, Turborepo configuration, and testing strategies.

  • Documentation:
    • Adds .roo/rules/monorepo.md for monorepo management.
    • Covers workspace organization, primary development areas, and package dependency management with pnpm.
    • Provides guidelines for Turborepo configuration.
    • Details test execution strategies for vitest and jest.
    • Describes migration from deprecated .test.ts to .spec.ts formats.
  • Testing:
    • Emphasizes using vitest for new tests and migrating failing jest tests to vitest.
    • Specifies file naming conventions for tests: .spec.ts for vitest, .test.ts for jest (deprecated).

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

Add comprehensive documentation for monorepo structure and workflows:
- Workspace organization and primary development areas
- Package dependency management with pnpm
- Turborepo configuration guidelines
- Test execution strategies for both vitest and jest
- Migration path from deprecated test formats to modern ones

Fixes: #4295
Signed-off-by: Eric Wheeler <[email protected]>
@KJ7LNW KJ7LNW requested review from cte and mrubens as code owners June 3, 2025 22:09
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. documentation Improvements or additions to documentation labels Jun 3, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 3, 2025
@cte cte merged commit 17b18e3 into RooCodeInc:main Jun 3, 2025
18 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 3, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 3, 2025
@adamhill
Copy link
Contributor

adamhill commented Jun 3, 2025

Any chance we could take a stab at: (since you have broached the subject)

  1. Putting the "How to add to Roo's Settings system" doc in here.
  2. An .editorconfig
  3. Some TS rules we all can live with for patterns and practices, directions we want to move to (like not using jest anymore). Anf maybe a high level description of the project.

@KJ7LNW
Copy link
Contributor Author

KJ7LNW commented Jun 3, 2025

Any chance we could take a stab at: (since you have broached the subject)

1. Putting the "How to add to Roo's Settings system" doc in here.

2. An .editorconfig

3. Some TS rules we all can live with for patterns and practices, directions we want to move to (like not using jest anymore). Anf maybe a high level description of the project.

good idea but I think it should be done as a roo-specific mode. with instructions as part of the role definition that specifically state "use this mode whenever creating new settings in this application". it must be in the role definition not in the custom instructions so that it is visible at all times to the model.

See #4130 for those instructions since they were removed when the docs directory was deleted

@KJ7LNW
Copy link
Contributor Author

KJ7LNW commented Jun 3, 2025

@adamhill is that a pull request you can put together?

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

Labels

documentation Improvements or additions to documentation lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Models struggle with monorepo structure and test management

3 participants