8000 Add import tests for @roo-code/types, fix the build by cte · Pull Request #4060 · RooCodeInc/Roo-Code · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add import tests for @roo-code/types, fix the build #4060

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
merged 3 commits into from
May 28, 2025
Merged

Add import tests for @roo-code/types, fix the build #4060

merged 3 commits into from
May 28, 2025

Conversation

cte
Copy link
Collaborator
@cte cte commented May 28, 2025 •
edited by ellipsis-dev bot
Loading

Description

Important

Add import tests for CommonJS and ESM, update package configuration, and fix build process.

  • Tests:
    • Add cjs-import.test.ts for CommonJS import tests, verifying require syntax and export consistency.
    • Add esm-import.test.ts for ESM import tests, verifying dynamic and static import consistency.
    • Add package-exports.test.ts to test package exports, ensuring consistency between ESM and CJS builds and checking TypeScript definitions.
  • Configuration:
    • Update package.json to correct main, module, and types paths in publishConfig and exports.
    • Modify tsup.config.ts to set output extensions based on format.
    • Add vitest.config.ts to configure Vitest with global test variables.
  • Build:
    • Fix build process by ensuring correct output paths and formats in tsup.config.ts.

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

@cte cte requested a review from mrubens as a code owner May 28, 2025 03:31
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request labels May 28, 2025
ellipsis-dev[bot]
ellipsis-dev bot reviewed May 28, 2025
View reviewed changes
@cte cte merged commit 59f1d4c into main May 28, 2025
10 checks passed
@cte cte deleted the cte/fix-build branch May 28, 2025 03:44
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 28, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap May 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ellipsis-dev ellipsis-dev[bot] ellipsis-dev[bot] left review comments

@mrubens mrubens Awaiting requested review from mrubens mrubens is a code owner

Assignees
No one assigned
Labels
enhancement New feature or request size:L This PR changes 100-499 lines, ignoring generated files.
Projects
Roo Code Roadmap
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cte
0