Skip to content

Fix/remove barrel imports - #220

Merged
smarcet merged 3 commits into
mainfrom
fix/remove-barrel-imports
Apr 21, 2026
Merged

Fix/remove barrel imports#220
smarcet merged 3 commits into
mainfrom
fix/remove-barrel-imports

Conversation

@santipalenque

@santipalenque santipalenque commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

https://app.clickup.com/t/86b9595px

Summary by CodeRabbit

  • Chores
    • Optimized module imports across multiple components for better bundling efficiency by switching to direct module path imports instead of barrel exports for utilities and UI icons.

@coderabbitai

coderabbitai Bot commented Apr 17, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c286179a-947e-47a0-859e-df3347caae33

📥 Commits

Reviewing files that changed from the base of the PR and between d9a897c and c4b18ee.

📒 Files selected for processing (1)
  • src/components/mui/table/mui-table.js

📝 Walkthrough

Walkthrough

This PR refactors import statements across multiple components and utilities, replacing Lodash named/default imports with module-specific paths (e.g., lodash/debounce, lodash/isEqual) and Material-UI icon barrel imports with individual module imports. No runtime logic or behavior is modified.

Changes

Cohort / File(s) Summary
Lodash Utility Imports Refactoring
src/components/inputs/registration-company-input.js, src/components/mui/search-input.js, src/components/mui/table/mui-table.js, src/utils/query-actions.js
Replaced Lodash named imports (e.g., { debounce }, { isEqual }) with module-specific imports (lodash/debounce, lodash/isEqual, lodash/isBoolean). All call sites remain unchanged; only import resolution is modified.
Material-UI Icon Imports Refactoring
src/components/inputs/upload-input-v3/index.js, src/components/mui/confirm-dialog.js
Replaced grouped Material-UI icon imports from @mui/icons-material barrel export with individual per-icon module imports (e.g., @mui/icons-material/UploadFile). Icon usage in JSX templates unchanged.
Unused Import Removal
src/components/table-sortable/SortableTable.js
Removed unused default Lodash import (import _ from 'lodash'). No other code changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 Hop, hop, a bundler's delight!
Granular imports made tight and light,
No logic changed, just cleaner ways—
Tree-shaking wins save bytes and days! 🌳✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix/remove barrel imports' directly aligns with the main objective of replacing barrel imports with direct module paths across multiple files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/remove-barrel-imports

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@smarcet
smarcet merged commit f42f3e9 into main Apr 21, 2026
4 of 5 checks passed
@santipalenque
santipalenque deleted the fix/remove-barrel-imports branch August 26, 2026 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants