Skip to content

Conversation

@Cristhianzl
Copy link
Member

This pull request updates the src/frontend/package-lock.json file to explicitly mark many dependencies as peer dependencies by adding the "peer": true property to their entries. Additionally, it updates the version of lodash-es. These changes help clarify dependency relationships and can improve compatibility and package management in complex projects.

Dependency metadata improvements:

  • Added "peer": true to a wide range of dependencies, including but not limited to @babel/core, @chakra-ui/styled-system, @chakra-ui/system, @codemirror/language, @codemirror/state, @codemirror/view, @swc/core, @types/react, @types/sortablejs, acorn, chokidar, d3-selection, esbuild, express, jest, jiti, jsdom, jsep, nanoid, react, react-dom, react-hook-form, rollup, sortablejs, storybook, tailwindcss, typescript, vite, and zod. This marks these packages as peer dependencies, which can help prevent version conflicts and clarify their usage expectations. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18] [19] [20] [21] [22] [23] [24] [25] [26] [27] [28] [29] [30] [31] [32] [33] [34]

Dependency version update:

  • Updated lodash-es from version 4.17.22 to 4.17.23 to ensure the latest fixes and improvements are included.

@Cristhianzl Cristhianzl self-assigned this Jan 26, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 26, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • src/frontend/package-lock.json is excluded by !**/package-lock.json

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch cz/update-lodash-audit-fix

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.

@github-actions
Copy link
Contributor

Frontend Unit Test Coverage Report

Coverage Summary

Lines Statements Branches Functions
Coverage: 17%
17.53% (5026/28667) 10.9% (2407/22075) 11.63% (731/6281)

Unit Test Results

Tests Skipped Failures Errors Time
2006 0 💤 0 ❌ 0 🔥 25.971s ⏱️

@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 34.80%. Comparing base (3503877) to head (97cb849).
⚠️ Report is 7 commits behind head on main.

❌ Your project status has failed because the head coverage (41.61%) is below the target coverage (60.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #11441   +/-   ##
=======================================
  Coverage   34.80%   34.80%           
=======================================
  Files        1417     1417           
  Lines       67838    67838           
  Branches     9941     9941           
=======================================
  Hits        23610    23610           
  Misses      43003    43003           
  Partials     1225     1225           
Flag Coverage Δ
frontend 16.04% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions bot added the lgtm This PR has been approved by a maintainer label Jan 26, 2026
@Cristhianzl Cristhianzl added this pull request to the merge queue Jan 26, 2026
Merged via the queue into main with commit f0dec81 Jan 26, 2026
29 of 30 checks passed
@Cristhianzl Cristhianzl deleted the cz/update-lodash-audit-fix branch January 26, 2026 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants