Does not handle correctly absolute paths in baseUrl and paths

[andreww2012]

Nuxt 3.6.x generates a tsconfig file with absolute paths in baseUrl and paths fields. This causes eslint-import-resolver-typescript (which depends on get-tsconfig) to incorrectly report that imports with nuxt aliases cannot be resolved.

Reproduction

https://github.com/andreww2012/eslint-plugin-import-with-nuxt-3.6-false-reports

npm i
npm run repro
npm run repro:eslint

[privatenumber]

Can you update your reproduction to be minimal (e.g. only get-tsconfig as a dependency)? Currently, it's possible the bugs are in the get-tsconfig usages.

FWIW we have tests that prove absolute paths are handled:
https://github.com/search?q=repo%3Aprivatenumber%2Fget-tsconfig+absolute+path%3A%2F%5Etests%5C%2Fspecs%5C%2F%2F&type=code

[andreww2012]

@privatenumber npm run repro only uses get-tsconfig in the example. Could you verify whether its output correct or incorrect?

[privatenumber]

But your file loads a directory that isn't in the project:
https://github.com/andreww2012/eslint-plugin-import-with-nuxt-3.6-false-reports/blob/03a4501c53ded2dba533066fbbd66f3d1f71c320/get-tsconfig.js#L3

I'm closing this for now, but I'm happy to re-open once the repo is minimal (get-tsconfig should be the only dependency) and all required files are present.

[andreww2012]

You need to run npm i so nuxt generates its tsconfig. That's what the line you mentioned looks for. :)
I could not include the generated tsconfig in that repo as it contains private absolute paths.

[privatenumber]

I looked into it and I'm able to reproduce it, but this reproduction could have been a lot more minimal. If you use parseTsconfig, the absolute paths do not need to be real.

Will fix soon.

[andreww2012]

Thank you very much and sorry if the reproduction repo was too bloated. I tried to demonstrate my workflow and how I came to the need of absolute paths handling.

[andreww2012]

I just tried 4.6.1, and running npm run repro from my test repo now yields the following:

{
  baseUrl: { broken: '..', correct: '..' }, // fixed
  matcher: {
    broken: [
      'C:/Users/my-username/Documents/a/b/eslint-plugin-import-with-nuxt-3.6-false-reports/C:/Users/my-username/Documents/a/b/eslint-plugin-import-with-nuxt-3.6-false-reports/.nuxt/imports'
    ],
    correct: [ 'C:/Users/my-username/Documents/a/.nuxt/imports' ]
  }
}

I understood that the generated absolute paths do not need to be real, but the output still looks quite weird (plus eslint-import-resolver-typescript continues not to work).

Just for comparison, this is 4.6.0 output

{
  baseUrl: {
    broken: './C:/Users/my-username/Documents/a/b/eslint-plugin-import-with-nuxt-3.6-false-reports',
    correct: '..'
  },
  matcher: {
    broken: [
      'C:/Users/my-username/Documents/a/b/eslint-plugin-import-with-nuxt-3.6-false-reports/.nuxt/C:/Users/my-username/Documents/a/b/eslint-plugin-import-with-nuxt-3.6-false-reports/C:/Users/my-username/Documents/a/b/eslint-plugin-import-with-nuxt-3.6-false-reports/.nuxt/imports'
    ],
    correct: [ 'C:/Users/my-username/Documents/a/.nuxt/imports' ]
  }
}

[privatenumber]

🎉 This issue has been resolved in version 4.6.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[privatenumber]

Give this a try ^

[andreww2012]

Everything works now, thank you a lot for the quick fix 💯 🥇