fix(60592): JSDoc implements space sensetive (#60640)

This commit is contained in:
Oleksandr T.
2025-01-17 22:12:12 +02:00
committed by GitHub
parent f99803d05f
commit 5170645f4e
11 changed files with 196 additions and 0 deletions
+1
View File
@@ -9536,6 +9536,7 @@ namespace Parser {
const node = factory.createExpressionWithTypeArguments(expression, typeArguments) as ExpressionWithTypeArguments & { expression: Identifier | PropertyAccessEntityNameExpression; };
const res = finishNode(node, pos);
if (usedBrace) {
skipWhitespace();
parseExpected(SyntaxKind.CloseBraceToken);
}
return res;
+19
View File
@@ -17,6 +17,16 @@ class B extends A {
super();
}
}
/**
* @extends { A }
* @constructor
*/
class C extends A {
constructor() {
super();
}
}
//// [foo.js]
@@ -35,3 +45,12 @@ class B extends A {
super();
}
}
/**
* @extends { A }
* @constructor
*/
class C extends A {
constructor() {
super();
}
}
@@ -24,3 +24,17 @@ class B extends A {
}
}
/**
* @extends { A }
* @constructor
*/
class C extends A {
>C : Symbol(C, Decl(foo.js, 15, 1))
>A : Symbol(A, Decl(foo.js, 0, 0))
constructor() {
super();
>super : Symbol(A, Decl(foo.js, 0, 0))
}
}
@@ -30,3 +30,22 @@ class B extends A {
}
}
/**
* @extends { A }
* @constructor
*/
class C extends A {
>C : C
> : ^
>A : A
> : ^
constructor() {
super();
>super() : void
> : ^^^^
>super : typeof A
> : ^^^^^^^^
}
}
@@ -0,0 +1,26 @@
//// [tests/cases/conformance/jsdoc/extendsTag6.ts] ////
=== foo.js ===
/**
* @constructor
*/
class A {
>A : Symbol(A, Decl(foo.js, 0, 0))
constructor() {}
}
/**
* @extends { A }
* @constructor
*/
class B extends A {
>B : Symbol(B, Decl(foo.js, 5, 1))
>A : Symbol(A, Decl(foo.js, 0, 0))
constructor() {
super();
>super : Symbol(A, Decl(foo.js, 0, 0))
}
}
@@ -0,0 +1,32 @@
//// [tests/cases/conformance/jsdoc/extendsTag6.ts] ////
=== foo.js ===
/**
* @constructor
*/
class A {
>A : A
> : ^
constructor() {}
}
/**
* @extends { A }
* @constructor
*/
class B extends A {
>B : B
> : ^
>A : A
> : ^
constructor() {
super();
>super() : void
> : ^^^^
>super : typeof A
> : ^^^^^^^^
}
}
@@ -0,0 +1,17 @@
//// [tests/cases/conformance/jsdoc/jsdocImplementsTag.ts] ////
=== /a.js ===
/**
* @typedef { { foo: string } } A
*/
/**
* @implements { A }
*/
class B {
>B : Symbol(B, Decl(a.js, 0, 0))
foo = ''
>foo : Symbol(B.foo, Decl(a.js, 7, 9))
}
@@ -0,0 +1,21 @@
//// [tests/cases/conformance/jsdoc/jsdocImplementsTag.ts] ////
=== /a.js ===
/**
* @typedef { { foo: string } } A
*/
/**
* @implements { A }
*/
class B {
>B : B
> : ^
foo = ''
>foo : string
> : ^^^^^^
>'' : ""
> : ^^
}
@@ -20,3 +20,13 @@ class B extends A {
super();
}
}
/**
* @extends { A }
* @constructor
*/
class C extends A {
constructor() {
super();
}
}
@@ -0,0 +1,22 @@
// @allowJs: true
// @checkJs: true
// @target: esnext
// @noEmit: true
// @Filename: foo.js
/**
* @constructor
*/
class A {
constructor() {}
}
/**
* @extends { A }
* @constructor
*/
class B extends A {
constructor() {
super();
}
}
@@ -0,0 +1,15 @@
// @allowJs: true
// @checkJs: true
// @noEmit: true
// @Filename: /a.js
/**
* @typedef { { foo: string } } A
*/
/**
* @implements { A }
*/
class B {
foo = ''
}