we only need to consider if the editor document is the event document

Author: markijbema

src/services/ghost/GhostProvider.ts

@@ -222,15 +222,17 @@ export class GhostProvider {
 
 		// Heuristic to filter out changes far from cursor (likely external or LLM edits)
 		const editor = vscode.window.activeTextEditor
-		if (editor && editor.document === event.document) {
-			const cursorPos = editor.selection.active
-			const isNearCursor = event.contentChanges.some((change) => {
-				const distance = Math.abs(cursorPos.line - change.range.start.line)
-				return distance <= 2
-			})
-			if (!isNearCursor) {
-				return
-			}
+		if (!editor || editor.document !== event.document) {
+			return
+		}
+
+		const cursorPos = editor.selection.active
+		const isNearCursor = event.contentChanges.some((change) => {
+			const distance = Math.abs(cursorPos.line - change.range.start.line)
+			return distance <= 2
+		})
+		if (!isNearCursor) {
+			return
 		}
 
 		await this.documentStore.storeDocument({ document: event.document })

src/services/ghost/__tests__/GhostProvider.spec.ts

@@ -434,6 +434,43 @@ console.log('test');]]></replace></change>`
 			expect(isNearCursor).toBe(true)
 		})
 
+		it("should filter out changes to non-active documents", async () => {
+			const initialContent = `console.log('test');`
+			const { mockDocument } = await setupTestDocument("inactive.js", initialContent)
+
+			// Create a different document for the active editor
+			const activeContent = `console.log('active');`
+			const activeUri = vscode.Uri.parse("file://active.js")
+			mockWorkspace.addDocument(activeUri, activeContent)
+			const activeDocument = await mockWorkspace.openTextDocument(activeUri)
+
+			// Mock active editor with a different document
+			;(vscode.window as any).activeTextEditor = {
+				document: activeDocument,
+				selection: {
+					active: new vscode.Position(0, 10),
+				},
+			}
+
+			// Simulate change to the non-active document
+			const event = {
+				document: mockDocument,
+				contentChanges: [
+					{
+						range: new vscode.Range(new vscode.Position(0, 10), new vscode.Position(0, 10)),
+						rangeLength: 0,
+						text: "a",
+					},
+				],
+				reason: undefined,
+			}
+
+			// Should be filtered out - document doesn't match active editor
+			const editor = (vscode.window as any).activeTextEditor
+			const shouldProcess = editor && editor.document === event.document
+			expect(shouldProcess).toBe(false)
+		})
+
 		it("should allow small paste operations near cursor", async () => {
 			const initialContent = `console.log('test');`
 			const { mockDocument } = await setupTestDocument("paste.js", initialContent)