Fix: barrel imports - #295
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe package version is updated to ChangesComponent release update
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Updates the library’s component “barrel” export strategy to reduce unintended dependency requirements from importing lib/components, and documents the preferred per-module import approach for consumers.
Changes:
- Removed
UploadInputV3from thesrc/components/index.jsbarrel export and listed it under the “3rd party deps” commented-out section. - Added README guidance recommending direct per-component import paths and explaining why some components are excluded from the barrel.
- Bumped the package version to
5.0.44-beta.0.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/components/index.js | Removes UploadInputV3 from the barrel and documents it as excluded due to external deps. |
| readme.md | Adds guidance on importing from individual component paths and rationale for excluding some exports from the barrel. |
| package.json | Updates package version to 5.0.44-beta.0. |
Suppressed comments (1)
readme.md:31
- This new "Importing Components" section makes the following "Enviroment Variables" heading look like a sub-section of it. If this is meant to be a top-level section, bump it back to
##and fix the spelling to "Environment".
will not resolve them.
### Enviroment Variables
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| // export {default as TextEditorV2} from './inputs/editor-input-v2' | ||
| // export {default as TextEditorV3} from './inputs/editor-input-v3' | ||
| // export {default as CompanyInputV2} from './inputs/company-input-v2.js' | ||
| // export {default as UploadInputV3} from './inputs/upload-input-v3' // @mui/material, @mui/icons-material, @mui/x-date-pickers, spark-md5, dropzone, react-dropzone |
| Components that pull in heavy 3rd-party dependencies (MUI, Stripe, | ||
| react-beautiful-dnd, etc.) are intentionally commented out of |
https://app.clickup.com/t/9014802374/86baz0zt6
Summary by CodeRabbit
5.0.44-beta.0.