Fix nil pointer dereference in snapshotFS.GetFile for non-existent files

Author: dyxushuai

internal/project/snapshot_test.go

@@ -101,4 +101,28 @@ func TestSnapshot(t *testing.T) {
 		assert.Check(t, snapshotAfter.fs.diskFiles["/home/projects/ts/p1/a.ts"] == nil)
 		assert.Check(t, snapshotAfter.fs.diskFiles["/home/projects/ts/p2/b.ts"] != nil)
 	})
+
+	t.Run("GetFile returns nil for non-existent files", func(t *testing.T) {
+		t.Parallel()
+		files := map[string]any{
+			"/home/projects/TS/p1/tsconfig.json": "{}",
+			"/home/projects/TS/p1/index.ts":      "console.log('Hello, world!');",
+		}
+		session := setup(files)
+		session.DidOpenFile(context.Background(), "file:///home/projects/TS/p1/index.ts", 1, files["/home/projects/TS/p1/index.ts"].(string), lsproto.LanguageKindTypeScript)
+		snapshot, release := session.Snapshot()
+		defer release()
+
+		// Test that GetFile returns nil for non-existent file (first call)
+		handle := snapshot.GetFile("/home/projects/TS/p1/nonexistent.ts")
+		assert.Check(t, handle == nil, "GetFile should return nil for non-existent file on first call")
+
+		// Test that GetFile returns nil for non-existent file (second call - triggers the cached path)
+		handle2 := snapshot.GetFile("/home/projects/TS/p1/nonexistent.ts")
+		assert.Check(t, handle2 == nil, "GetFile should return nil for non-existent file on second call")
+
+		// Test that ReadFile returns false for non-existent file
+		_, ok := snapshot.ReadFile("/home/projects/TS/p1/nonexistent.ts")
+		assert.Check(t, !ok, "ReadFile should return false for non-existent file")
+	})
 }

internal/project/snapshotfs.go

@@ -49,10 +49,12 @@ func (s *snapshotFS) GetFile(fileName string) FileHandle {
 		}
 		return nil
 	}))
-	if entry, ok := s.readFiles.LoadOrStore(s.toPath(fileName), newEntry); ok {
-		return entry()
+	entry, _ := s.readFiles.LoadOrStore(s.toPath(fileName), newEntry)
+	file := entry()
+	if file == nil {
+		return nil
 	}
-	return nil
+	return file
 }
 
 type snapshotFSBuilder struct {