Shorten error spans for errors reported on constructor declarations (#58061)

Author: Andarist

src/compiler/utilities.ts

@@ -2309,6 +2309,17 @@ export function getErrorSpanForNode(sourceFile: SourceFile, node: Node): TextSpa
             const pos = skipTrivia(sourceFile.text, (node as JSDocSatisfiesTag).tagName.pos);
             return getSpanOfTokenAtPosition(sourceFile, pos);
         }
+        case SyntaxKind.Constructor: {
+            const constructorDeclaration = node as ConstructorDeclaration;
+            const start = skipTrivia(sourceFile.text, constructorDeclaration.pos);
+            const scanner = createScanner(sourceFile.languageVersion, /*skipTrivia*/ true, sourceFile.languageVariant, sourceFile.text, /*onError*/ undefined, start);
+            let token = scanner.scan();
+            while (token !== SyntaxKind.ConstructorKeyword && token !== SyntaxKind.EndOfFileToken) {
+                token = scanner.scan();
+            }
+            const end = scanner.getTokenEnd();
+            return createTextSpanFromBounds(start, end);
+        }
     }
 
     if (errorNode === undefined) {

tests/baselines/reference/ClassDeclaration10.errors.txt

@@ -5,7 +5,7 @@ ClassDeclaration10.ts(3,4): error TS2391: Function implementation is missing or
 ==== ClassDeclaration10.ts (2 errors) ====
     class C {
        constructor();
-       ~~~~~~~~~~~~~~
+       ~~~~~~~~~~~
 !!! error TS2390: Constructor implementation is missing.
        foo();
        ~~~

tests/baselines/reference/ClassDeclaration11.errors.txt

@@ -4,7 +4,7 @@ ClassDeclaration11.ts(2,4): error TS2390: Constructor implementation is missing.
 ==== ClassDeclaration11.ts (1 errors) ====
     class C {
        constructor();
-       ~~~~~~~~~~~~~~
+       ~~~~~~~~~~~
 !!! error TS2390: Constructor implementation is missing.
        foo() { }
     }

tests/baselines/reference/ClassDeclaration14.errors.txt

@@ -8,6 +8,6 @@ ClassDeclaration14.ts(3,4): error TS2390: Constructor implementation is missing.
        ~~~
 !!! error TS2391: Function implementation is missing or not immediately following the declaration.
        constructor();
-       ~~~~~~~~~~~~~~
+       ~~~~~~~~~~~
 !!! error TS2390: Constructor implementation is missing.
     }

tests/baselines/reference/ClassDeclaration8.errors.txt

@@ -4,6 +4,6 @@ ClassDeclaration8.ts(2,3): error TS2390: Constructor implementation is missing.
 ==== ClassDeclaration8.ts (1 errors) ====
     class C {
       constructor();
-      ~~~~~~~~~~~~~~
+      ~~~~~~~~~~~
 !!! error TS2390: Constructor implementation is missing.
     }

tests/baselines/reference/bases.errors.txt

@@ -35,9 +35,9 @@ bases.ts(18,9): error TS2339: Property 'y' does not exist on type 'C'.
 !!! error TS2420:   Property 'x' is missing in type 'C' but required in type 'I'.
 !!! related TS2728 bases.ts:2:5: 'x' is declared here.
         constructor() {
-        ~~~~~~~~~~~~~~~
+        ~~~~~~~~~~~
+!!! error TS2377: Constructors for derived classes must contain a 'super' call.
             this.x: any;
-    ~~~~~~~~~~~~~~~~~~~~
             ~~~~
 !!! error TS17009: 'super' must be called before accessing 'this' in the constructor of a derived class.
                  ~
@@ -47,8 +47,6 @@ bases.ts(18,9): error TS2339: Property 'y' does not exist on type 'C'.
                     ~~~
 !!! error TS2693: 'any' only refers to a type, but is being used as a value here.
         }
-    ~~~~~
-!!! error TS2377: Constructors for derived classes must contain a 'super' call.
     }
 
     new C().x;

tests/baselines/reference/classConstructorOverloadsAccessibility.errors.txt

@@ -6,10 +6,10 @@ classConstructorOverloadsAccessibility.ts(11,2): error TS2385: Overload signatur
 ==== classConstructorOverloadsAccessibility.ts (3 errors) ====
     class A {
     	public constructor(a: boolean) // error
-    	~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+    	~~~~~~~~~~~~~~~~~~
 !!! error TS2385: Overload signatures must all be public, private or protected.
     	protected constructor(a: number) // error
-    	~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+    	~~~~~~~~~~~~~~~~~~~~~
 !!! error TS2385: Overload signatures must all be public, private or protected.
     	private constructor(a: string)
     	private constructor() { 
@@ -19,7 +19,7 @@ classConstructorOverloadsAccessibility.ts(11,2): error TS2385: Overload signatur
 
     class B {
     	protected constructor(a: number) // error
-    	~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+    	~~~~~~~~~~~~~~~~~~~~~
 !!! error TS2385: Overload signatures must all be public, private or protected.
     	constructor(a: string)
     	constructor() { 

tests/baselines/reference/classUpdateTests.errors.txt

@@ -53,7 +53,7 @@ classUpdateTests.ts(113,1): error TS1128: Declaration or statement expected.
 
     class F extends E {
     	constructor() {} // ERROR - super call required
-    	~~~~~~~~~~~~~~~~
+    	~~~~~~~~~~~
 !!! error TS2377: Constructors for derived classes must contain a 'super' call.
     }
 

tests/baselines/reference/classWithTwoConstructorDefinitions.errors.txt

@@ -7,18 +7,18 @@ classWithTwoConstructorDefinitions.ts(8,5): error TS2392: Multiple constructor i
 ==== classWithTwoConstructorDefinitions.ts (4 errors) ====
     class C {
         constructor() { } // error
-        ~~~~~~~~~~~~~~~~~
+        ~~~~~~~~~~~
 !!! error TS2392: Multiple constructor implementations are not allowed.
         constructor(x) { } // error
-        ~~~~~~~~~~~~~~~~~~
+        ~~~~~~~~~~~
 !!! error TS2392: Multiple constructor implementations are not allowed.
     }
 
     class D<T> {
         constructor(x: T) { } // error
-        ~~~~~~~~~~~~~~~~~~~~~
+        ~~~~~~~~~~~
 !!! error TS2392: Multiple constructor implementations are not allowed.
         constructor(x: T, y: T) { } // error
-        ~~~~~~~~~~~~~~~~~~~~~~~~~~~
+        ~~~~~~~~~~~
 !!! error TS2392: Multiple constructor implementations are not allowed.
     }

tests/baselines/reference/constructorOverloads1.errors.txt

@@ -17,25 +17,21 @@ constructorOverloads1.ts(17,18): error TS2769: No overload matches this call.
 ==== constructorOverloads1.ts (6 errors) ====
     class Foo {
         constructor(s: string);
-        ~~~~~~~~~~~~~~~~~~~~~~~
+        ~~~~~~~~~~~
 !!! error TS2392: Multiple constructor implementations are not allowed.
         constructor(n: number);
-        ~~~~~~~~~~~~~~~~~~~~~~~
+        ~~~~~~~~~~~
 !!! error TS2392: Multiple constructor implementations are not allowed.
         constructor(x: any) {
-        ~~~~~~~~~~~~~~~~~~~~~
-    
+        ~~~~~~~~~~~
+!!! error TS2392: Multiple constructor implementations are not allowed.
 
         }
-    ~~~~~
-!!! error TS2392: Multiple constructor implementations are not allowed.
         constructor(x: any) {
-        ~~~~~~~~~~~~~~~~~~~~~
-    
+        ~~~~~~~~~~~
+!!! error TS2392: Multiple constructor implementations are not allowed.
 
         }
-    ~~~~~
-!!! error TS2392: Multiple constructor implementations are not allowed.
         bar1() {  /*WScript.Echo("bar1");*/ }
         bar2() {  /*WScript.Echo("bar1");*/ }
     }