fix(#2035): Prevent infinite loops caused by cyclic references on typedValue

[omarsy]

Closes #2035.

Contributors' checklist...

• Added new tests, or not needed, or not feasible
• Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory
• Updated the official documentation or not needed
• No breaking changes were made, or a BREAKING CHANGE: xxx message was included in the description
• Added references to related issues and PRs
• Provided any useful hints for running manual tests
• Added new benchmarks to generated graphs, if any. More info here.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.74%. Comparing base (555357e) to head (4044fe0).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2036      +/-   ##
==========================================
- Coverage   46.74%   46.74%   -0.01%     
==========================================
  Files         492      492              
  Lines       69616    69614       -2     
==========================================
- Hits        32542    32539       -3     
- Misses      34363    34364       +1     
  Partials     2711     2711              

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

[deelawn]

Thanks for identifying and solving this bug. Could make the same change for the map key as well and add a file test to demonstrate the issue is fixed?

[omarsy]

Thanks for identifying and solving this bug. Could make the same change for the map key as well and add a file test to demonstrate the issue is fixed?

Thank you for your review. ^^
I Add file test + apply same change for the map key on 4635def

[ltzmaxwell]

Thanks for identifying and solving this bug. Could make the same change for the map key as well and add a file test to demonstrate the issue is fixed?

Thank you for your review. ^^ I Add file test + apply same change for the map key on 4635def

I guess what @deelawn suggests is that you can have another test file show case using a struct as key of a map to prove you modification thoroughly. But actually I found another bug from this direction, see #2041. cc @deelawn .