LSP not recommending shortest subpath imports from a shared package in a monorepo

[Frederick888]

🔎 Search Terms

"package.json exports", "monorepo exports"

🕗 Version & Regression Information

• This is the behavior in every version I tried, and I reviewed the FAQ for entries about export maps

⏯ Playground Link

https://github.com/Frederick888/playground/tree/monorepo

💻 Code

I'm setting up a monorepo where there are a few 'executables' and a shared package. In the example repo, it's structured this way:

.
├── package.json
├── packages
│  ├── bar
│  │  ├── package.json
│  │  ├── spec
│  │  ├── src
│  │  │  └── index.ts                   <- playground
│  │  └── tsconfig.json
│  └── shared
│     ├── dist                          <- outDir, only *.js files listed here
│     │  ├── src
│     │  │  ├── index.js
│     │  │  └── utils
│     │  │     ├── bonjour.js
│     │  │     └── index.js
│     │  └── tsconfig.tsbuildinfo
│     ├── package.json
│     ├── spec                          <- tests
│     ├── src
│     │  ├── index.ts
│     │  └── utils
│     │     ├── bonjour.ts
│     │     └── index.ts                <- barrel index.ts to re-export things we want to expose from utils/
│     └── tsconfig.json
├── tsconfig.base.json
└── tsconfig.json

(Please clear the playground index.ts before testing these. tsserver actually recommends "@playground/shared/utils" if it's already there.)

Now in bar/, if I import something from shared/utils/index.ts, it'll recommend import { greetings } from "@playground/shared/utils/index". I guess this part is similar to #51418.

Another issue is that if I import something from shared/utils/bonjour.ts, it'll recommend import { bonjour } from "@playground/shared/utils/bonjour".

🙁 Actual behavior

// import { greetings } from "@playground/shared/utils/index" <- Nope
// import { bonjour } from "@playground/shared/utils/bonjour" <- Nope
import { bonjour, greetings } from "@playground/shared/utils"

console.log(greetings())
console.log(bonjour())

Both the two 'nope' import recommendations result in runtime import errors.

🙂 Expected behavior

It recommends the re-exports from @playground/shared/utils.

Additional information about the issue

Please feel free to close this if the underlying issue is the same as ##51418's.

Also I'm not sure whether #58657 is relevant.

[Frederick888]

Closing. should've used

    "./*": {
      "types": "./dist/src/*/index.d.ts",
      "default": "./dist/src/*/index.js"
    }

or just

    "./*": "./dist/src/*/index.js"

instead of

    "./*": {
      "types": "./dist/src/*.d.ts",
      "default": "./dist/src/*/index.js"
    }

in shared/'s package.json.