Address PR feedback: remove unused markers and fix nil pointer crash

- Remove unused markers (/*1*/, /*2*/, /*3*/) from formatDocumentInPlace_test.go
- Apply fix from PR #1993 to prevent nil pointer crash in indent.go when firstListChild is nil
- Add comment about known trailing space issue (#1997)

The indent.go fix prevents crashes when formatting ranges where the list has no first child.

Co-authored-by: DanielRosenwasser <972891+DanielRosenwasser@users.noreply.github.com>

Author: Copilot

internal/format/indent.go

@@ -66,8 +66,11 @@ func getIndentationForNodeWorker(
 			// }, {                  itself contributes nothing.
 			//   prop: 1        L3 - The indentation of the second object literal is best understood by
 			// })                    looking at the relationship between the list and *first* list item.
-			listLine, _ := getStartLineAndCharacterForNode(firstListChild, sourceFile)
-			listIndentsChild := firstListChild != nil && listLine > containingListOrParentStartLine
+			var listIndentsChild bool
+			if firstListChild != nil {
+				listLine, _ := getStartLineAndCharacterForNode(firstListChild, sourceFile)
+				listIndentsChild = listLine > containingListOrParentStartLine
+			}
 			actualIndentation := getActualIndentationForListItem(current, sourceFile, options, listIndentsChild)
 			if actualIndentation != -1 {
 				return actualIndentation + indentationDelta

internal/fourslash/tests/formatDocumentInPlace_test.go

@@ -10,12 +10,13 @@ import (
 func TestFormatDocumentInPlace(t *testing.T) {
 	t.Parallel()
 	defer testutil.RecoverAndFail(t, "Panic on fourslash test")
-	const content = `/*1*/for (;;) { }
-/*2*/for (var x;x<0;x++) { }
-/*3*/for (var x ;x<0 ;x++) { }`
+	const content = `for (;;) { }
+for (var x;x<0;x++) { }
+for (var x ;x<0 ;x++) { }`
 	f := fourslash.NewFourslash(t, nil /*capabilities*/, content)
 	f.FormatDocument(t)
 	// After formatting, verify the formatted content
+	// Note: trailing space on last line is a known issue (see https://github.com/microsoft/typescript-go/issues/1997)
 	f.CurrentFileContentIs(t, `for (; ;) { }
 for (var x; x < 0; x++) { }
 for (var x; x < 0; x++) { } `)