Skip to content

Commit 7d83746

Browse files
authored
Merge pull request #11 from tjx666/tj/fix/more-dynamic-cases
fix: some dynamic import cases hover tooltip not work
2 parents 981d066 + 1b93301 commit 7d83746

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

src/hoverTooltips/modules.ts

+6-3
Original file line numberDiff line numberDiff line change
@@ -67,16 +67,19 @@ export class ModulesHoverProvider implements HoverProvider {
6767

6868
const stringBeforeLeftQuote = lineText.slice(0, leftQuotaIndex);
6969
const stringAfterRightQuote = lineText.slice(rightQuotaIndex + 1);
70-
const afterFnCallRegexp = /^\s*\)(?:;.*|\n?)$/;
70+
const afterFnCallRegexp = /^\s*\)(?:;.*|\n?)/;
7171
const afterStatementRegexp = /^\s*(?:;.*)?$/;
7272
const isModule = // named imports/exports: import { pick } from 'lodash' or export * from 'antd'
7373
(/(?:\b|\s+)from\s+$/.test(stringBeforeLeftQuote) &&
7474
afterStatementRegexp.test(stringAfterRightQuote)) ||
7575
// unnamed import
7676
(/(?:\b|\s+)import\s+$/.test(stringBeforeLeftQuote) &&
7777
afterStatementRegexp.test(stringAfterRightQuote)) ||
78-
// dynamic imports: import( 'lodash' )
79-
(/(?:\b|\s+)import\s*\(\s*$/.test(stringBeforeLeftQuote) &&
78+
// dynamic imports:
79+
// case1: import( 'lodash' )
80+
// case2: import( 'lodash' ).then()
81+
// case3: /** @type {import('next').NextConfig} */
82+
(/(?:\b|\{|\s+)import\s*\(\s*$/.test(stringBeforeLeftQuote) &&
8083
afterFnCallRegexp.test(stringAfterRightQuote)) ||
8184
// require: require('lodash' )
8285
(/require\s+\(\s*$/.test(stringBeforeLeftQuote) &&

0 commit comments

Comments
 (0)