import/no-unresolved unable to resolve @atlaskit/pragmatic-drag-and-drop’s nested imports after upgrading 3.8.7 → 4.2.4

[acusti]

i am using @atlassian/pragmatic-drag-and-drop, which exposes a number of nested import targets. when i upgrade eslint-import-resolver-typescript from 3.8.7 to 4.2.4, i get errors when trying to import from any of those targets:

Error:    1:25  error  Unable to resolve path to module '@atlaskit/pragmatic-drag-and-drop/combine'            import/no-unresolved
Error:    5:8   error  Unable to resolve path to module '@atlaskit/pragmatic-drag-and-drop/external/adapter'   import/no-unresolved
Error:    9:8   error  Unable to resolve path to module '@atlaskit/pragmatic-drag-and-drop/external/file'      import/no-unresolved
Error:   10:34  error  Unable to resolve path to module '@atlaskit/pragmatic-drag-and-drop/prevent-unhandled'  import/no-unresolved

here are the lines in question:

import { combine } from '@atlaskit/pragmatic-drag-and-drop/combine';
import {
    dropTargetForExternal,
    monitorForExternal,
} from '@atlaskit/pragmatic-drag-and-drop/external/adapter';
import {
    containsFiles,
    getFiles,
} from '@atlaskit/pragmatic-drag-and-drop/external/file';
import { preventUnhandled } from '@atlaskit/pragmatic-drag-and-drop/prevent-unhandled';

using v3.8.7, there are no linting errors. i am using v1.5.2 of @atlaskit/pragmatic-drag-and-drop. my eslint.config.js file configures the import/resolver like so:

            'import/resolver': { typescript: { alwaysTryTypes: true } },

and i am using eslint v9.23.0

let me know if there’s anything else that would be useful for me to provide, and thanks for this excellent project! i had a lot of struggles with eslint and typescript and the import/* eslint rules before finding it and getting it configured correctly.

[JounQin]

Please provide a GitHub repository reproduction.

[JounQin]

Just a quick look, it seems resolving package.json is not supported by oxc-resolver currently.

cc @Boshen

I'll try whether can fix it by myself.

[JounQin]

@acusti I can't reproduce it at #416, please provide reproduction.

[JounQin]

Closing due to lack of reproduction, feel free to provide a reproduction, then I'll take a look again.

[acusti]

@JounQin sorry for the delay. i finally put together a minimal reproducible example: https://github.com/acusti/eslint-import-resolver-typescript-repro

it’s using latest yarn (v4.9.1) and PnP, but i’m not sure if that’s required to make it reproducible. it might be only reproducible from a tsx file, because my initial attempt to make a reproduction that didn’t include react didn’t trigger an error.

[JounQin]

Thanks for providing the reproduction, I'll take a look when I'm free, or contributing will be much appreciated.

[JounQin]

OK, seems related to P'n'P support. If I changed to use nodeLinker: node-modules, there will be no issue.

[JounQin]

Close in favour of https://github.com/import-js/eslint-import-resolver-typescript/releases/tag/v4.3.4