When attaching JSDoc diagnostics, make sure to only include diagnostics that actually belong with the JSDoc (#57271)

Author: abrahamguo

src/compiler/parser.ts

@@ -8760,7 +8760,7 @@ namespace Parser {
                 if (!jsDocDiagnostics) {
                     jsDocDiagnostics = [];
                 }
-                jsDocDiagnostics.push(...parseDiagnostics);
+                addRange(jsDocDiagnostics, parseDiagnostics, saveParseDiagnosticsLength);
             }
             currentToken = saveToken;
             parseDiagnostics.length = saveParseDiagnosticsLength;

tests/baselines/reference/errorIsolation.symbols

@@ -0,0 +1,16 @@
+//// [tests/cases/conformance/jsdoc/errorIsolation.ts] ////
+
+=== errorIsolation.js ===
+const async = { doSomething: _ => {} };
+>async : Symbol(async, Decl(errorIsolation.js, 0, 5))
+>doSomething : Symbol(doSomething, Decl(errorIsolation.js, 0, 15))
+>_ : Symbol(_, Decl(errorIsolation.js, 0, 28))
+
+async.doSomething(
+>async.doSomething : Symbol(doSomething, Decl(errorIsolation.js, 0, 15))
+>async : Symbol(async, Decl(errorIsolation.js, 0, 5))
+>doSomething : Symbol(doSomething, Decl(errorIsolation.js, 0, 15))
+
+	/***/
+	() => {}
+);

tests/baselines/reference/errorIsolation.types

@@ -0,0 +1,21 @@
+//// [tests/cases/conformance/jsdoc/errorIsolation.ts] ////
+
+=== errorIsolation.js ===
+const async = { doSomething: _ => {} };
+>async : { doSomething: (_: any) => void; }
+>{ doSomething: _ => {} } : { doSomething: (_: any) => void; }
+>doSomething : (_: any) => void
+>_ => {} : (_: any) => void
+>_ : any
+
+async.doSomething(
+>async.doSomething(	/***/	() => {}) : void
+>async.doSomething : (_: any) => void
+>async : { doSomething: (_: any) => void; }
+>doSomething : (_: any) => void
+
+	/***/
+	() => {}
+>() => {} : () => void
+
+);

tests/cases/conformance/jsdoc/errorIsolation.ts

@@ -0,0 +1,8 @@
+// @noEmit: true
+// @checkJs: true
+// @filename: errorIsolation.js
+const async = { doSomething: _ => {} };
+async.doSomething(
+	/***/
+	() => {}
+);