search

nuxt-modules / sanity

Sanity integration for Nuxt
https://sanity.nuxtjs.org
MIT License
211 stars 35 forks source link

fix: `parse-headers`, `debug` and `async-cache-dedupe` no longer need to be optimized #992

Closed stipsan closed 2 months ago

stipsan commented 2 months ago

In get-it v8.4.24 we started inlining parse-headers and debug as it were causing issues with the latest version of hydrogen. The same is true for @sanity/core-loader v1.6.7 and async-cache-dedupe.

What's the reason for inlining @sanity/client btw? It should be strictly ESM compatible in its latest version 🤔

netlify[bot] commented 2 months ago

Deploy Preview for nuxt-sanity-module canceled.

Name Link
Latest commit f0eebc58c840d019114fa588aebbe011797c4e57
Latest deploy log https://app.netlify.com/sites/nuxt-sanity-module/deploys/662a65da837fc3000874c4fe
danielroe commented 2 months ago

(I don't think we are inlining @sanity/client - where do you see that?)

stipsan commented 2 months ago

@danielroe oops, I didn't mean inline, but optimize: https://github.com/nuxt-modules/sanity/blob/193fa68ca82edd5806b24e285f614e7ba2f8c27e/src/module.ts#L323