Avoid using ? with get_item to handle unhashable inputs properly

[sydney-runkle]

Change Summary

Avoid using ? to unpack the result of get_item in the case where unhashable inputs would produce a TypeError which bubbles all of the way up. We would want to treat this instead just like a ValidationError.

Related issue number

Fix pydantic/pydantic#8183

Checklist

• Unit tests for the changes exist
• Documentation reflects the changes where applicable
• Pydantic tests pass with this pydantic-core (except for expected changes)
• My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

[codecov]

Codecov Report

Merging #1089 (60c2c35) into main (3d3f406) will increase coverage by 0.01%.
Report is 2 commits behind head on main.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1089      +/-   ##
==========================================
+ Coverage   89.62%   89.64%   +0.01%     
==========================================
  Files         106      106              
  Lines       16363    16363              
  Branches       35       35              
==========================================
+ Hits        14665    14668       +3     
+ Misses       1691     1688       -3     
  Partials        7        7              

Files	Coverage Δ
src/lookup_key.rs	92.69% <100.00%> (ø)
src/serializers/type_serializers/dataclass.rs	91.89% <100.00%> (ø)
src/validators/arguments.rs	99.57% <100.00%> (ø)
src/validators/dataclass.rs	97.72% <100.00%> (ø)
src/validators/literal.rs	95.13% <100.00%> (+2.08%)	⬆️
src/validators/model.rs	97.83% <100.00%> (ø)
src/validators/model_fields.rs	98.25% <100.00%> (ø)
src/validators/typed_dict.rs	96.93% <100.00%> (ø)

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a7739ec...60c2c35. Read the comment docs.

[codspeed-hq]

CodSpeed Performance Report

Merging #1089 will not alter performance

_{Comparing fix-for-hash-bubble-error (60c2c35) with main (a7739ec)}

Summary

✅ 140 untouched benchmarks

[samuelcolvin]

This looks great, well done.