feat(autocomplete): Add id field to cache for better visibility tracking

.changeset/fast-carrots-remember.md

@@ -0,0 +1,7 @@
+---
+"kilo-code": patch
+---
+
+Add id field to autocomplete cache for better visibility tracking
+
+Added an `id` property to `FillInAtCursorSuggestion` type and updated visibility tracking to use the id instead of the prefix/suffix/suggestion triplet for more reliable telemetry.

src/services/ghost/classic-auto-complete/AutocompleteTelemetry.ts

@@ -9,7 +9,7 @@ export type { AutocompleteContext, CacheMatchType, FillInAtCursorSuggestion }
  * This key is used to track whether the same suggestion is still being displayed.
  */
 export function getSuggestionKey(suggestion: FillInAtCursorSuggestion): string {
-	return `${suggestion.prefix}|${suggestion.suffix}|${suggestion.text}`
+	return suggestion.id
 }
 
 /**

src/services/ghost/classic-auto-complete/GhostInlineCompletionProvider.ts

@@ -322,8 +322,13 @@ export class GhostInlineCompletionProvider implements vscode.InlineCompletionIte
 			return
 		}
 
+		// Ensure the suggestion has an id for telemetry tracking
+		const suggestionWithId = fillInAtCursor.id
+			? fillInAtCursor
+			: { ...fillInAtCursor, id: crypto.randomUUID() }
+
 		// Add to the end of the array (most recent)
-		this.suggestionsHistory.push(fillInAtCursor)
+		this.suggestionsHistory.push(suggestionWithId)
 
 		// Remove oldest if we exceed the limit
 		if (this.suggestionsHistory.length > MAX_SUGGESTIONS_HISTORY) {
@@ -368,7 +373,7 @@ export class GhostInlineCompletionProvider implements vscode.InlineCompletionIte
 	): FillInAtCursorSuggestion {
 		if (!suggestionText) {
 			this.telemetry?.captureSuggestionFiltered("empty_response", telemetryContext)
-			return { text: "", prefix, suffix }
+			return { text: "", prefix, suffix, id: crypto.randomUUID() }
 		}
 
 		const processedText = postprocessGhostSuggestion({
@@ -379,11 +384,11 @@ export class GhostInlineCompletionProvider implements vscode.InlineCompletionIte
 		})
 
 		if (processedText) {
-			return { text: processedText, prefix, suffix }
+			return { text: processedText, prefix, suffix, id: crypto.randomUUID() }
 		}
 
 		this.telemetry?.captureSuggestionFiltered("filtered_by_postprocessing", telemetryContext)
-		return { text: "", prefix, suffix }
+		return { text: "", prefix, suffix, id: crypto.randomUUID() }
 	}
 
 	private async disposeIgnoreController(): Promise<void> {

src/services/ghost/classic-auto-complete/__tests__/AutocompleteTelemetry.test.ts

@@ -19,6 +19,7 @@ describe("AutocompleteTelemetry", () => {
 			text: `text-${index}`,
 			prefix: `prefix-${index}`,
 			suffix: `suffix-${index}`,
+			id: `suggestion-${index}`,
 		}
 	}
 

src/services/ghost/types.ts

@@ -96,6 +96,8 @@ export interface FillInAtCursorSuggestion {
 	text: string
 	prefix: string
 	suffix: string
+	/** Unique identifier for this suggestion, used for telemetry tracking */
+	id: string
 }
 
 export interface MatchingSuggestionResult {