Fix: Ignore KCL changes when loading settings and fixed useEffect to use .current

[nadr0]

Issue

• I wrote wrong react logic for prev/curr useEffect checking. I shouldn't have used the getter I should have used the .current
• When editing any .kcl file in a project while in the model scene all of the settings reload from disk into memory

Implementation

• Removed my poor logic to use the .current from the getter
• Implemented a check to exit early when reading settings from disk. It should only update if one of two files match

Any time we edited a .kcl file from code mods or if the user is typing it would reload all of the project settings from disk into memory. That logic does not need to run. By writing the wrong react API logic I uncovered that the settings were constantly being reloaded from disk because my getter kept getting set again even though the .current value inside was not changing. I am glad I wrote the line of code wrong to find this.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
modeling-app	✅ Ready (Inspect)	Visit Preview	Jan 31, 2025 7:06pm

[qa-wolf]

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

[franknoirot]

Nice!

[franknoirot]

This is probably our WS BS that we're dealing with right now.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.94%. Comparing base (f8f4474) to head (4e72aa7).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5126   +/-   ##
=======================================
  Coverage   85.94%   85.94%           
=======================================
  Files          90       90           
  Lines       32718    32718           
=======================================
  Hits        28121    28121           
  Misses       4597     4597           

Flag	Coverage Δ
wasm-lib	85.94% <ø> (ø)

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

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