feat(types): type $fetch to match json serialization output

[mahdiboomeri]

🔗 Linked issue

resolves #804, resolves #1000

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 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)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This PR transforms the output type of $Fetch to better match what JSON.stringify returns (using trpc and remix implementation).

This is potentially a breaking change.

📝 Checklist

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

[codecov]

Codecov Report

Merging #1002 (e405cbb) into main (d83f2b6) will decrease coverage by 0.13%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1002      +/-   ##
==========================================
- Coverage   67.79%   67.66%   -0.13%     
==========================================
  Files          60       60              
  Lines        6133     6161      +28     
  Branches      692      692              
==========================================
+ Hits         4158     4169      +11     
- Misses       1966     1982      +16     
- Partials        9       10       +1     

Impacted Files	Coverage Δ
src/types/fetch.ts	100.00% <100.00%> (ø)
src/build.ts	44.62% <0.00%> (-1.45%)	⬇️
src/rollup/plugins/esbuild.ts	54.41% <0.00%> (+1.07%)	⬆️
src/presets/lagon.ts	35.71% <0.00%> (+4.76%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[pi0]

💯

[danielroe]

This is really great.

I only have one suggestion, so we can avoid the super-permissive object type. Otherwise 👌

[mahdiboomeri]

@danielroe Thanks 💚, I've also updated the tests. It should be ready to go.

[danielroe]

@mahdiboomeri Just to let you know we had to revert part of this as a hotfix owing to increased type complexity: see #1050. If you have any ideas, a PR would be most welcome 🙏

[mahdiboomeri]

@danielroe I think moving Serialize (and Simplify) to nitro-routes.d.ts can help. I did a quick experiment and it seems like it improves type checking duration.

Before:

After:

Is there any reproduction on this?

[danielroe]

Previously, it reproduced in the Nuxt test:types check.