Multiple Entries on Go to Definition

[karlhorky]

In the app/page.tsx file in the Next.js app dir example cmd-clicking on the styles.code property to trigger Go to Definition in VS Code shows an extra entry of the node_modules/next/types/global.d.ts file with the *.module.css type definition.

This makes it no longer a one-click Go to Definition interaction. It would be great to get rid of these generic *.module.css / *.module.scss / *.module.sass entries so that it's a single click again.

This is also a problem with other extensions:

• Intellisense priority issues breaks Go to definition clinyong/vscode-css-modules#63

Potential approach

It seems that @zardoy has found a potentially interesting approach in his extension TypeScript Essential Plugins (repo). Some interesting parts of the code:

    proxy.getDefinitionAndBoundSpan = (fileName, position) => {
      const prior = info.languageService.getDefinitionAndBoundSpan(fileName, position)
      // ...
      prior.definitions = prior.definitions.filter(({ fileName, containerName, containerKind, kind, name, ...rest }) => {
        // ...
        if (moduleDeclaration?.name.text === '*.module.css') return false
        return true
      })
      // ...
    })

• https://github.com/zardoy/typescript-vscode-plugins/blob/a102a76786a0292aa8a9a46fb6cc43595e8871a7/typescript/src/definitions.ts#L7-L8
• https://github.com/zardoy/typescript-vscode-plugins/blob/a102a76786a0292aa8a9a46fb6cc43595e8871a7/typescript/src/definitions.ts#L123
• https://github.com/zardoy/typescript-vscode-plugins/blob/a102a76786a0292aa8a9a46fb6cc43595e8871a7/typescript/src/definitions.ts#L141

If I'm understanding correctly, this is filtering out previous definitions if they have the name *.module.css, which could also be extended to include *.module.scss and *.module.sass.

[karlhorky]

I tried the changes from #70 out just now in v1.6.2, but I'm still getting the duplicate entries:

More details:

• Page: packages/fusion.upleveled.io/app/(auth)/login/page.tsx
• CSS Module File: packages/fusion.upleveled.io/app/(auth)/login/page.module.scss
• Next.js type definitions: node_modules/next/types/global.d.ts

[Viijay-Kr]

@karlhorky Can u also let me know the typescript version you are using in your project ?

One thing I tried in the example monorepo project is changing the typescript version to point to the node module version.

In this case it works. Will continue to debug further

Screen.Recording.2023-02-01.at.19.51.48.mov

[karlhorky]

I tried with the following, neither seem to work (tried reloading the editor as well between switching):

• VS Code's version: TypeScript 4.9.4
• Workspace version: TypeScript 4.9.5

In the video above, there are two things that may be different:

1. The Vite app is open
2. VS Code seems to be using examples/monorepo/packages/vite-app as the root (in many situations when working in a monorepo, the VS Code root would be examples/monorepo)

[Viijay-Kr]

I tried with the following, neither seem to work (tried reloading the editor as well between switching):

• VS Code's version: TypeScript 4.9.4
• Workspace version: TypeScript 4.9.5

In the video above, there are two things that may be different:

1. The Vite app is open
2. VS Code seems to be using examples/monorepo/packages/vite-app as the root (in many situations when working in a monorepo, the VS Code root would be examples/monorepo)

@karlhorky That was my bad . However it doesn't make any difference. Even if I open the entire monorepo project
I am able to make the feature work by swapping the typescript version.

I may have a potential fix which I would try . I will keep you posted

[karlhorky]

Seems like it is reproducible in the examples/next-js-app-dir-css-modules example (nothing to do with monorepos / workspace packages):

Change to this directory, install the dependencies and open VS Code using:

cd examples/next-js-app-dir-css-modules
npm install
code .

1. Open app/page.tsx
2. cmd/ctrl-click on the code in styles.code on line 13

[Viijay-Kr]

@karlhorky So I shipped 1.6.4 (latest) which fixes the problem.

As pointed out here Viijay-Kr/typescript-cleanup-definitions#1 (comment)

The extension was not shipped with node_modules which resulted in a error where plugin could not be loaded since the node_modules folder wasn't part of the installed extension.

[karlhorky]

Indeed, 1.6.4 seems to be fixed, both with the built-in VS Code TypeScript version (4.9.4) and the Workspace version (4.9.5).

Thanks for this, nice work!

Closing.

[karlhorky]

@Viijay-Kr Hmm... need to reopen this :( Looks like this Next.js entry removal is broken again in version 1.9.6 with *.module.scss imports (not sure which version was the first one with the broken behavior):

Here's the project repo: https://github.com/upleveled/next-js-example-winter-2023-vienna-austria

Here's the config I tried:

.vscode/settings.json

{
  "reactTsScss.autoComplete": true,
  "reactTsScss.baseDir": "src",
  "reactTsScss.cleanUpDefs": [
    "*.module.css",
    "*.module.scss",
    "*.module.sass",
    "*.module.less",
    "*.module.styl",
    "node_modules/next/types/global.d.ts"
  ],
  "reactTsScss.codelens": false,
  "reactTsScss.cssAutoComplete": true,
  "reactTsScss.cssDefinitions": true,
  "reactTsScss.cssSyntaxColor": true,
  "reactTsScss.definition": true,
  "reactTsScss.diagnostics": true,
  "reactTsScss.peekProperties": true,
  "reactTsScss.references": false,
  "reactTsScss.tsconfig": "./tsconfig.json",
  "typescript.tsdk": "./node_modules/typescript/lib",
  "typescript.enablePromptUseWorkspaceTsdk": true,
}