fix: allow sorting and mutating arrays due perf + bump lockfile

[evenstensberg]

Summary

Allows using Array.sort in eslint, could be replaced by array.toSorted, but it will impact perf. Updated lockfile too.

What kind of change does this PR introduce?

fix

Did you add tests for your changes?

N/A

Does this PR introduce a breaking change?

No

If relevant, what needs to be documented once your changes are merged or what have you already documented?

N/A

Use of AI

Claude helped with implementation.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.38%. Comparing base (03e9d08) to head (24d305f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #434   +/-   ##
=======================================
  Coverage   93.38%   93.38%           
=======================================
  Files           1        1           
  Lines         121      121           
  Branches       48       47    -1     
=======================================
  Hits          113      113           
  Misses          7        7           
  Partials        1        1           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 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.

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	serialize-javascript@​7.0.3 ⏵ 7.1.1		+2		+2

View full report

[alexander-akait]

We should not do it, we should fix it, eslint rules should be not suppressed, that they rules without the real problems

[evenstensberg]

We can use toSorted instead but it will impact performance, so that's why this is a lint change instead

[alexander-akait]

How it impacts performance?

[alexander-akait]

We use it in many places

[evenstensberg]

toSorted created a copy instead of modifying/mutating the array, so more memory will be used.

https://dev.to/sundarbadagala081/sort-vs-tosorted-3gmi

[alexander-akait]

it means we may have a potential problem/bug and code should be rewritten, never disable rules like because you want, each disabling should have a comment and why it was disabled, and where and how it should be resolve in future, it is very bad practice just disable