refactor(interopDefault): simplify implementation

[SukkaW]

🔗 Linked issue

N/A

❓ Type of change

• 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

While crafting antfu/unconfig#27, I stumbled upon the implementation of interopDefault. Upon closer examination, I found out that the implementation could be simplified and redundant code could be merged.

All tests passed on my machine locally.

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[pi0]

There is a small chance of behavior regression here.

In the current implementation, we were using the (cached) value of sourceModule.default via defaultValue now we dynamically access it.

[SukkaW]

Unless the supplied module comes with interop code (which already defines default as a getter), I don't anticipate any changes in behavior here.

Furthermore, the appropriate behavior is either to cache all keys (getter) or not cache any at all, instead of caching default only.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.17%. Comparing base (226a47b) to head (f7ae48d).
Report is 29 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #243      +/-   ##
==========================================
- Coverage   88.04%   87.17%   -0.87%     
==========================================
  Files           8        8              
  Lines        1062     1084      +22     
  Branches      188      196       +8     
==========================================
+ Hits          935      945      +10     
- Misses        127      139      +12     

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