[nuxt-client] @hey-api/client-nuxt breaks production server

[lvaroqui]

Description

When using @hey-api/client-nuxt somewhere in the nuxt app, the production build is broken.

Reproducible example or configuration

This is easily reproducible with the example in this repo:

cd examples/openapi-ts-nuxt
pnpm install # I had to replace the `workspace:*` part in my dependencies with the actual version number
pnpm run build
node .output/server/index.mjs

Then try to load the page at http://localhost:3000, and the servers throws an internal error:

[nuxt] [request error] [unhandled] [500] Package import specifier "#build/nuxt.config.mjs" is not defined in package /tmp/openapi-ts/examples/openapi-ts-nuxt/.output/server/node_modules/nuxt/package.json imported from /tmp/openapi-ts/examples/openapi-ts-nuxt/.output/server/node_modules/nuxt/dist/app/nuxt.js

[nuxt] [request error] [unhandled] [500] Package import specifier "#build/nuxt.config.mjs" is not defined in package /tmp/openapi-ts/examples/openapi-ts-nuxt/.output/server/node_modules/nuxt/package.json imported from /tmp/openapi-ts/examples/openapi-ts-nuxt/.output/server/node_modules/nuxt/dist/app/nuxt.js

The error is unfortunately not very descriptive, but I think it is related to the way @hey-api/client-nuxt imports the nuxt/app module. I suspect '@hey-api/client-nuxt' should be a nuxt plugin or nuxt module

OpenAPI specification (optional)

No response

System information (optional)

No response

[mrlubos]

Hi @lvaroqui, does this fix it for you? #1660 (comment)

[lvaroqui]

Hi @mrlubos, it does indeed! I can't believe I didn't see the issue after looking around for 3 hours T_T

I think this should be addressed in the README and updated in the example. In the best case, user shouldn't be able to run into this issue!

[eszterczotter]

The mentioned solution works for me too, but I have a feeling it can only be a temporary one. I agree with @lvaroqui about what’s causing the issue. I don’t think this should be included in the readme or the docs.

[lvaroqui]

I spent 3 hours finding the root cause of the issue yesterday. So, until a proper fix is implemented, updating the README and doc seems pretty useful so others don't run into the same problem!

[eszterczotter]

If it won't become the final solution then sure, that would also help. I also spent half a day with this issue.

[mrlubos]

This will be fixed soon, thank you for your patience!