Skip to content

chore(lodash) - remove deprecated packages#917

Merged
gabrielseco merged 2 commits intomainfrom
remove-depreacated-packages
Apr 14, 2026
Merged

chore(lodash) - remove deprecated packages#917
gabrielseco merged 2 commits intomainfrom
remove-depreacated-packages

Conversation

@gabrielseco
Copy link
Copy Markdown
Collaborator

@gabrielseco gabrielseco commented Apr 14, 2026

Note

Medium Risk
Replaces widely-used helper behavior (lodash.get/lodash.isequal) in form value extraction and schema diffing, which could subtly change equality and nested-path resolution in edge cases.

Overview
Removes deprecated lodash.get and lodash.isequal dependencies and their type packages, adding fast-deep-equal instead.

Updates JSON-schema field diffing to use fast-deep-equal for deep comparisons, and replaces lodash.get usage in form utilities with a new internal getNestedValue helper.

Adds getNestedValue implementation to src/lib/utils.ts plus unit tests covering nested paths, missing keys, null handling, and default values.

Reviewed by Cursor Bugbot for commit 5e8d03b. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 14, 2026

📦 Bundle Size Report

Metric Current Previous Change Status
Total (gzip) 207.89 kB 208.69 kB -797 B (-0.4%) 🟢
Total (raw) 557.98 kB 560.5 kB -2.51 kB (-0.4%) 🟢
CSS (gzip) 20.09 kB 20.09 kB 0 B (0%) 🟢
CSS (raw) 103.08 kB 103.08 kB 0 B (0%) 🟢

Size Limits

  • ✅ Total gzipped: 207.89 kB / 250 kB (83.2%)
  • ✅ Total raw: 557.98 kB / 600 kB (93.0%)
  • ✅ CSS gzipped: 20.09 kB / 25 kB (80.3%)

Largest Files (Top 5)

  1. chunk-QM7RI4AA.js - 13.83 kB (new)
  2. index.css - 10.04 kB (0 B (0%))
  3. styles.css - 10.04 kB (0 B (0%))
  4. index.js - 6.05 kB (-3 B (-0.0%))
  5. chunk-3U3IP5TS.js - 5.93 kB (new)
View All Files (310 total)
File Size (gzip) Change
chunk-QM7RI4AA.js 13.83 kB new
index.css 10.04 kB 0 B (0%)
styles.css 10.04 kB 0 B (0%)
index.js 6.05 kB -3 B (-0.0%)
chunk-3U3IP5TS.js 5.93 kB new
chunk-R47Q7R6C.js 5.82 kB 0 B (0%)
chunk-IZRFNO5W.js 4.41 kB new
chunk-G3O2LLJG.js 4.23 kB new
chunk-6ZPQEUEH.js 4.06 kB new
chunk-WBJP5DD2.js 3.36 kB new

✅ Bundle size check passed

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 14, 2026

Deploy preview for remote-flows-example-app ready!

✅ Preview
https://remote-flows-example-cf3o1ug9k-remotecom.vercel.app

Built with commit 5e8d03b.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 14, 2026

Deploy preview for remote-flows ready!

✅ Preview
https://remote-flows-9ghrd39dt-remotecom.vercel.app

Built with commit 5e8d03b.
This pull request is being automatically deployed with vercel-action

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 703b266. Configure here.

Comment thread src/lib/utils.ts Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 14, 2026

📊 Coverage Report

✅ Coverage increased! 🎉

Metric Current Previous Change Status
Lines 89.78% 89.76% +0.02% 🟢
Statements 89.43% 89.40% +0.02% 🟢
Functions 87.17% 87.15% +0.01% 🟢
Branches 79.68% 79.60% +0.07% 🟢

Detailed Breakdown

Lines Coverage
  • Covered: 3566 / 3972
  • Coverage: 89.78%
  • Change: +0.02% (9 lines)
Statements Coverage
  • Covered: 3629 / 4058
  • Coverage: 89.43%
  • Change: +0.02% (9 statements)
Functions Coverage
  • Covered: 978 / 1122
  • Coverage: 87.17%
  • Change: +0.01% (1 functions)
Branches Coverage
  • Covered: 2215 / 2780
  • Coverage: 79.68%
  • Change: +0.07% (10 branches)

✅ Coverage check passed

@gabrielseco gabrielseco self-assigned this Apr 14, 2026
@gabrielseco gabrielseco merged commit 9715667 into main Apr 14, 2026
7 checks passed
@gabrielseco gabrielseco deleted the remove-depreacated-packages branch April 14, 2026 09:26
@gabrielseco gabrielseco mentioned this pull request Apr 15, 2026
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