Wrong module resolution in tsconfig.json file

[guibulator]

Environment

• Operating System: Darwin
• Node Version: v20.13.1
• Nuxt Version: 3.12.4
• CLI Version: 3.12.0
• Nitro Version: 2.9.7
• Package Manager: yarn@1.22.22
• Builder: -
• User Config: compatibilityDate, devtools, modules
• Runtime Modules: @sidebase/nuxt-auth@0.8.1
• Build Modules: -

Reproduction

You have the issue with your default repro as well.
https://codesandbox.io/p/devbox/gifted-boyd-vmf834?file=%2F.nuxt%2Ftsconfig.json%3A145%2C5

Describe the bug

@danielroe
I opened an issue on @sidebase/nuxt-auth because PageMeta was not properly augmented from #app sidebase/nuxt-auth#807. I tested with a local module augmenting #app and it works as expected.

Turns out that the issue is that files under include section in the tsconfig.json are not resolved properly. You can check with even your own devtools module. Check the path in .nuxt/tsconfig.json for devtools: ../node_modules/@nuxt/devtools/runtime

But it should be ../node_modules/@nuxt/devtools/dist/runtime

Additional context

No response

Logs

No response

[guibulator]

Issue is here:

nuxt/packages/kit/src/template.ts

Line 159 in 228b9f6

include.add(join(relative, 'runtime'))

[HigherOrderLogic]

I want to work on this issue.

One question @danielroe, is it the convention (or required) for modules to place runtime inside dist like the starter template? Some like the UnoCss module has its runtime dir placed outside.

[danielroe]

We can support both. Local modules won't have a dist, for example.

[guibulator]

Hi @danielroe , I'm sorry to reopen this conversation but after re-testing the fix, there are still some issues with discoverability. Maybe my understanding is wrong and you can guide me a bit with this.

In the include section of the tsconfig.json file, we now have this, which is good

But we also have the node_modules excluded which causes the type augmentation to not be found.

If I remove the node_modules from the exlude, it works but I guess that's not the solution. Should the reference to the typings be explicitly added to .nuxt/types or should it work with the node_modules excluded as well?

I updated my repro with the fix: https://github.com/guibulator/nuxt-pagemeta-repro