chore(cli): re-export cjs entry for defineConfig#336
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
This PR adds CommonJS support for the CLI package by creating a CJS entry point that re-exports functionality from both the core and test packages. The changes simplify export management by replacing individual named exports with wildcard exports.
- Replaced individual named exports with
export *from@voidzero-dev/vite-plus-test/configin the ESM entry point - Created a new CommonJS entry point (
index.cts) that spreads exports from both vite-plus-core and vite-plus-test - Added the
requirefield to package.json exports to point to the compiled CJS file
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| packages/cli/src/index.ts | Simplified exports by replacing individual named exports with wildcard export from test config |
| packages/cli/src/index.cts | New CommonJS entry point that combines exports from core and test packages |
| packages/cli/package.json | Added require field to exports configuration to support CommonJS consumers |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Merge activity
|
Brooooooklyn
deleted the
12-09-chore_cli_re-export_cjs_entry_for_defineconfig
branch
No description provided.