Skip to content

Conversation

sriramveeraghanta
Copy link
Member

@sriramveeraghanta sriramveeraghanta commented Sep 25, 2025

Description

  • Replace lodash pacakge with lodash-es
  • Replace all the implementation with lodash-es as barrel imports.

Type of Change

  • Improvement (change that would cause existing functionality to not work as expected)

Copy link
Contributor

coderabbitai bot commented Sep 25, 2025

Important

Review skipped

More than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review.

91 files out of 198 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-lodash-package

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.

@sriramveeraghanta sriramveeraghanta marked this pull request as ready for review September 25, 2025 15:34
@Copilot Copilot AI review requested due to automatic review settings September 25, 2025 15:34
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request replaces the lodash package with lodash-es across the entire monorepo to improve bundle performance through better tree-shaking support. The change enables ES modules to be imported individually, reducing final bundle sizes.

Changes:

  • Replace all lodash/module imports with lodash-es/module imports
  • Update package dependencies from lodash to lodash-es in all package.json files
  • Convert some individual imports to barrel imports where multiple functions are used

Reviewed Changes

Copilot reviewed 198 out of 199 changed files in this pull request and generated 6 comments.

File Description
pnpm-workspace.yaml Update catalog dependencies from lodash to lodash-es
packages/*/package.json Replace lodash dependency with lodash-es in all packages
Multiple .ts/.tsx files Convert imports from lodash/module to lodash-es/module format
apps/*/package.json Update lodash dependencies to lodash-es in all applications
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

cursor[bot]

This comment was marked as outdated.

@sriramveeraghanta sriramveeraghanta changed the title fix: replace lodash package with lodash-es [WEB-5027] fix: replace lodash package with lodash-es Sep 26, 2025
Copy link

makeplane bot commented Sep 26, 2025

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

@sriramveeraghanta sriramveeraghanta merged commit bedc1fa into preview Sep 26, 2025
7 of 8 checks passed
@sriramveeraghanta sriramveeraghanta deleted the fix-lodash-package branch September 26, 2025 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants