Add AI request ID to feedback telemetry events

Includes a unique AI request identifier in feedback telemetry to enable improved traceability and correlation of feedback to specific AI requests. Updates documentation and relevant interfaces to reflect the new field.

Author: sergeibbb

docs/telemetry-events.md

@@ -145,16 +145,11 @@
 
 ```typescript
 {
-  // Request duration if available
-  'duration': number,
+  'ai.request.id': string,
   // The AI feature that feedback was submitted for
-  'feature': 'explain' | 'generateCommit' | 'generateRebase' | 'generateStash' | 'generateDraft' | 'generateChangelog' | 'generateCreatePullRequest',
+  'feature': string,
   // Type of feedback provided
   'feedbackType': 'preset' | 'writeIn' | 'both',
-  // Whether write-in feedback was provided
-  'hasWriteInFeedback': boolean,
-  // Input/output lengths if available
-  'input.length': number,
   // Model information
   'model.id': string,
   'model.provider.id': 'anthropic' | 'azure' | 'deepseek' | 'gemini' | 'github' | 'gitkraken' | 'huggingface' | 'mistral' | 'ollama' | 'openai' | 'openaicompatible' | 'openrouter' | 'vscode' | 'xai',
@@ -165,11 +160,15 @@
   // The original rating that led to this feedback
   'rating': 'positive' | 'negative',
   'usage.completionTokens': number,
+  'usage.limits.limit': number,
+  'usage.limits.resetsOn': string,
+  'usage.limits.used': number,
   // Token usage information if available
   'usage.promptTokens': number,
   'usage.totalTokens': number,
   // Length of write-in feedback if provided
-  'writeInFeedback.length': number
+  'writeInFeedback.length': number,
+  'writeInFeedback.text': string
 }
 ```
 

src/commands/explainBase.ts

@@ -70,6 +70,7 @@ export abstract class ExplainCommandBase extends GlCommandBase {
 				providerName: result.model.provider.name,
 			},
 			usage: result.usage,
+			aiRequestId: result.id,
 			outputLength: result.content?.length,
 		};
 

src/commands/generateRebase.ts

@@ -359,6 +359,7 @@ export async function generateRebase(
 				providerName: result.model.provider.name,
 			},
 			usage: result.usage,
+			aiRequestId: result.id,
 			outputLength: result.content?.length,
 		};
 		const telemetryEnabled = container.telemetry.enabled;

src/constants.telemetry.ts

@@ -425,6 +425,7 @@ export interface AIFeedbackEvent {
 	'usage.limits.used'?: number;
 	'usage.limits.limit'?: number;
 	'usage.limits.resetsOn'?: string;
+	'ai.request.id'?: string;
 
 	'output.length'?: number;
 }

src/plus/ai/aiFeedbackService.ts

@@ -19,6 +19,7 @@ export interface AIFeedbackContext {
 			resetsOn: Date;
 		};
 	};
+	aiRequestId: string | undefined;
 	outputLength: number;
 }
 
@@ -66,6 +67,7 @@ export class AIFeedbackService implements Disposable {
 			'usage.limits.used': context.usage?.limits?.used,
 			'usage.limits.limit': context.usage?.limits?.limit,
 			'usage.limits.resetsOn': context.usage?.limits?.resetsOn?.toISOString(),
+			'ai.request.id': context.aiRequestId,
 			'output.length': context.outputLength,
 		};