Prevent panic and duplicate diagnostics for malformed tsconfig properties

internal/execute/tsctests/tsc_test.go

@@ -51,6 +51,14 @@ func TestTscCommandline(t *testing.T) {
 			subScenario:     "when build not first argument",
 			commandLineArgs: []string{"--verbose", "--build"},
 		},
+		{
+			subScenario: "malformed tsconfig property without value",
+			files: FileMap{
+				"/home/src/workspaces/project/tsconfig.json": `{"" }`,
+				"/home/src/workspaces/project/index.ts":      "",
+			},
+			commandLineArgs: nil,
+		},
 		{
 			subScenario:     "Initialized TSConfig with files options",
 			commandLineArgs: []string{"--init", "file0.st", "file1.ts", "file2.ts"},

internal/parser/parser.go

@@ -228,7 +228,7 @@ func (p *Parser) parseJSONText() *ast.SourceFile {
 
 func getErrorSpanForNode(sourceText string, node *ast.Node) core.TextRange {
 	pos := scanner.SkipTrivia(sourceText, node.Pos())
-	return core.NewTextRange(pos, node.End())
+	return core.NewTextRange(min(pos, node.End()), max(pos, node.End()))
 }
 
 func (p *Parser) validateJsonValue(sourceFile *ast.SourceFile, valueExpression *ast.Expression) {

internal/tsoptions/tsconfigparsing.go

@@ -261,7 +261,13 @@ func parseOwnConfigOfJsonSourceFile(
 			onPropertySet,
 		},
 	)
-	errors = append(errors, err...)
+	for _, diagnostic := range err {
+		if !slices.ContainsFunc(sourceFile.Diagnostics(), func(parseDiagnostic *ast.Diagnostic) bool {
+			return parseDiagnostic.Code() == diagnostic.Code() && parseDiagnostic.Pos() == diagnostic.Pos()
+		}) {
+			errors = append(errors, diagnostic)
+		}
+	}
 	// if len(rootCompilerOptions) != 0  && json != nil && json.CompilerOptions != nil {
 	//    errors = append(errors, ast.NewDiagnostic(sourceFile, rootCompilerOptions[0], diagnostics.X_0_should_be_set_inside_the_compilerOptions_object_of_the_config_json_file))
 	// }

testdata/baselines/reference/tsc/commandLine/malformed-tsconfig-property-without-value.js

@@ -0,0 +1,51 @@
+currentDirectory::/home/src/workspaces/project
+useCaseSensitiveFileNames::true
+Input::
+//// [/home/src/workspaces/project/index.ts] *new* 
+
+//// [/home/src/workspaces/project/tsconfig.json] *new* 
+{"" }
+
+tsgo 
+ExitStatus:: DiagnosticsPresent_OutputsGenerated
+Output::
+tsconfig.json:1:4 - error TS1328: Property value can only be string literal, numeric literal, 'true', 'false', 'null', object literal or array literal.
+
+1 {"" }
+     ~
+
+tsconfig.json:1:5 - error TS1005: ':' expected.
+
+1 {"" }
+      ~
+
+
+Found 2 errors in the same file, starting at: tsconfig.json:1
+
+//// [/home/src/tslibs/TS/Lib/lib.es2025.full.d.ts] *Lib*
+/// <reference no-default-lib="true"/>
+interface Boolean {}
+interface Function {}
+interface CallableFunction {}
+interface NewableFunction {}
+interface IArguments {}
+interface Number { toExponential: any; }
+interface Object {}
+interface RegExp {}
+interface String { charAt: any; }
+interface Array<T> { length: number; [n: number]: T; }
+interface ReadonlyArray<T> {}
+interface SymbolConstructor {
+    (desc?: string | number): symbol;
+    for(name: string): symbol;
+    readonly toStringTag: symbol;
+}
+declare var Symbol: SymbolConstructor;
+interface Symbol {
+    readonly [Symbol.toStringTag]: string;
+}
+declare const console: { log(msg: any): void; };
+//// [/home/src/workspaces/project/index.js] *new* 
+"use strict";
+
+