Skip to content

Conversation

@jr
Copy link
Collaborator

@jr jr commented Jun 13, 2025

And do some cleanup & consolidation.


Important

Move marketplace types to @roo-code/types and update imports and tests accordingly.

  • Marketplace Types:
    • Move marketplace types to @roo-code/types.
    • Update imports in MarketplaceManager.ts, RemoteConfigLoader.ts, SimpleInstaller.ts, and 8 other files.
  • Schema and Types:
    • Add marketplace.ts with schemas and types for marketplace items.
    • Remove types.ts and schemas.ts from src/services/marketplace.
  • Tests:
    • Update tests in MarketplaceManager.test.ts, RemoteConfigLoader.test.ts, SimpleInstaller.test.ts, and 8 other test files to use new imports.
    • Ensure tests handle new type locations and schema validations.

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

And do some cleanup & consolidation.
@jr jr requested review from cte and mrubens as code owners June 13, 2025 22:28
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 13, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 13, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 13, 2025
@jr jr merged commit 7dd56d6 into main Jun 14, 2025
28 of 30 checks passed
@jr jr deleted the jr/marketplace-types branch June 14, 2025 00:02
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 14, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 14, 2025
cte pushed a commit that referenced this pull request Jun 24, 2025
And do some cleanup & consolidation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants