build(vite-plus): sync exports in test package#349
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
![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 implements automatic synchronization of test package exports from @voidzero-dev/vite-plus-test to @voidzero-dev/vite-plus under the ./test/* subpath. This enables users to import test utilities from a single unified package instead of requiring separate package imports.
Key changes:
- Added build script logic to generate shim files that re-export test package modules
- Updated package.json with 60+ new test-related exports mapped under
./test/* - Added comprehensive documentation explaining the build architecture and export synchronization
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 9 comments.
| File | Description |
|---|---|
| packages/cli/package.json | Added dist/test to files array and 60+ new exports under ./test/* namespace that mirror the test package structure |
| packages/cli/build.ts | Implemented syncTestPackageExports() function with helpers to read test package exports, generate shim files, and update package.json exports automatically |
| packages/cli/BUNDLING.md | Added comprehensive documentation covering the 3-step build process, export mapping strategy, shim file generation, and technical reference |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Brooooooklyn
force-pushed
the
12-16-build_vite-plus_sync_exports_in_test_package
branch
from
2ec1a7c to
93080c4
Compare
Brooooooklyn
force-pushed
the
12-16-docs_add_vitest_deps_bundle_doc
branch
from
a005cf6 to
70f96ae
Compare
Brooooooklyn
force-pushed
the
12-16-build_vite-plus_sync_exports_in_test_package
branch
from
93080c4 to
0058fb8
Compare
Brooooooklyn
force-pushed
the
12-16-build_vite-plus_sync_exports_in_test_package
branch
from
0058fb8 to
b27abdd
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Merge activity
|
Brooooooklyn
deleted the
12-16-build_vite-plus_sync_exports_in_test_package
branch
No description provided.