Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented May 28, 2025

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
@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

Labels

enhancement New feature or request 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.

2 participants