fix(language-core): limit the range of parseDiagnostics checks

close #5823

Author: johnsoncodehk

packages/language-core/lib/codegen/utils/index.ts

@@ -45,9 +45,11 @@ export function* generateSfcBlockSection(
 
 	// #3632
 	if ('parseDiagnostics' in block.ast) {
-		const emptyEndLength = text.length - text.trimEnd().length;
+		const textEnd = text.trimEnd().length;
 		for (const diag of block.ast.parseDiagnostics as ts.DiagnosticWithLocation[]) {
-			if (diag.start >= end - emptyEndLength) {
+			const diagStart = diag.start;
+			const diagEnd = diag.start + diag.length;
+			if (diagStart >= textEnd && diagEnd <= end) {
 				yield `;`;
 				yield ['', block.name, end, codeFeatures.verification];
 				yield newLine;

packages/tsc/tests/typecheck.spec.ts

@@ -15,6 +15,7 @@ describe(`vue-tsc`, () => {
 			  "test-workspace/tsc/failureFixtures/#4569/main.vue(1,41): error TS4025: Exported variable '__VLS_export' has or is using private name 'Props'.",
 			  "test-workspace/tsc/failureFixtures/#5071/withScript.vue(1,19): error TS1005: ';' expected.",
 			  "test-workspace/tsc/failureFixtures/#5071/withoutScript.vue(2,26): error TS1005: ';' expected.",
+			  "test-workspace/tsc/failureFixtures/#5823/main.vue(6,13): error TS1109: Expression expected.",
 			  "test-workspace/tsc/failureFixtures/directives/main.vue(14,6): error TS2339: Property 'notExist' does not exist on type '{ exist: {}; Comp: () => void; $: ComponentInternalInstance; $data: {}; $props: {}; $attrs: Data; $refs: Data; $slots: Readonly<InternalSlots>; ... 8 more ...; $watch<T extends string | ((...args: any) => any)>(source: T, cb: T extends (...args: any) => infer R ? (args_0: R, args_1: R, args_2: OnCleanup) => any : (a...'.",
 			  "test-workspace/tsc/failureFixtures/directives/main.vue(17,2): error TS2578: Unused '@ts-expect-error' directive.",
 			  "test-workspace/tsc/failureFixtures/directives/main.vue(20,2): error TS2578: Unused '@ts-expect-error' directive.",

test-workspace/tsc/failureFixtures/#5823/main.vue

@@ -0,0 +1,7 @@
+<script lang="ts" generic="T" setup>
+const { info } = defineProps<{
+    info?: T;
+}>();
+
+const foo = ;
+</script>

test-workspace/tsc/failureFixtures/#5823/tsconfig.json

@@ -0,0 +1,4 @@
+{
+	"extends": "../../../tsconfig.base.json",
+	"include": ["**/*"]
+}

test-workspace/tsc/tsconfig.json

@@ -4,6 +4,7 @@
 		{ "path": "./failureFixtures/#3632" },
 		{ "path": "./failureFixtures/#4569" },
 		{ "path": "./failureFixtures/#5071" },
+		{ "path": "./failureFixtures/#5823" },
 		{ "path": "./failureFixtures/directives" },
 
 		{ "path": "./passedFixtures/#1886" },