Add DeleteReviewResult type and validate review exists before clearing state

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>

Author: Copilot

src/@types/vscode.proposed.chatParticipantAdditions.d.ts

@@ -66,10 +66,10 @@ declare module 'vscode' {
 
 	export class ChatResponseConfirmationPart {
 		title: string;
-		message: string | MarkdownString;
+		message: string;
 		data: any;
 		buttons?: string[];
-		constructor(title: string, message: string | MarkdownString, data: any, buttons?: string[]);
+		constructor(title: string, message: string, data: any, buttons?: string[]);
 	}
 
 	export class ChatResponseCodeCitationPart {
@@ -84,83 +84,8 @@ declare module 'vscode' {
 		constructor(toolName: string);
 	}
 
-	export interface ChatTerminalToolInvocationData {
-		commandLine: {
-			original: string;
-			userEdited?: string;
-			toolEdited?: string;
-		};
-		language: string;
-	}
-
-	export class ChatToolInvocationPart {
-		toolName: string;
-		toolCallId: string;
-		isError?: boolean;
-		invocationMessage?: string | MarkdownString;
-		originMessage?: string | MarkdownString;
-		pastTenseMessage?: string | MarkdownString;
-		isConfirmed?: boolean;
-		isComplete?: boolean;
-		toolSpecificData?: ChatTerminalToolInvocationData;
-		fromSubAgent?: boolean;
-
-		constructor(toolName: string, toolCallId: string, isError?: boolean);
-	}
-
-	/**
-	 * Represents a single file diff entry in a multi diff view.
-	 */
-	export interface ChatResponseDiffEntry {
-		/**
-		 * The original file URI (undefined for new files).
-		 */
-		originalUri?: Uri;
-
-		/**
-		 * The modified file URI (undefined for deleted files).
-		 */
-		modifiedUri?: Uri;
-
-		/**
-		 * Optional URI to navigate to when clicking on the file.
-		 */
-		goToFileUri?: Uri;
-
-		/**
-		 * Added data (e.g. line numbers) to show in the UI
-		 */
-		added?: number;
-
-		/**
-		 * Removed data (e.g. line numbers) to show in the UI
-		 */
-		removed?: number;
-	}
-
-	/**
-	 * Represents a part of a chat response that shows multiple file diffs.
-	 */
-	export class ChatResponseMultiDiffPart {
-		/**
-		 * Array of file diff entries to display.
-		 */
-		value: ChatResponseDiffEntry[];
-
-		/**
-		 * The title for the multi diff editor.
-		 */
-		title: string;
+	export type ExtendedChatResponsePart = ChatResponsePart | ChatResponseTextEditPart | ChatResponseNotebookEditPart | ChatResponseConfirmationPart | ChatResponseCodeCitationPart | ChatResponseReferencePart2 | ChatResponseMovePart | ChatResponseExtensionsPart | ChatPrepareToolInvocationPart;
 
-		/**
-		 * Create a new ChatResponseMultiDiffPart.
-		 * @param value Array of file diff entries.
-		 * @param title The title for the multi diff editor.
-		 */
-		constructor(value: ChatResponseDiffEntry[], title: string);
-	}
-
-	export type ExtendedChatResponsePart = ChatResponsePart | ChatResponseTextEditPart | ChatResponseNotebookEditPart | ChatResponseConfirmationPart | ChatResponseCodeCitationPart | ChatResponseReferencePart2 | ChatResponseMovePart | ChatResponseExtensionsPart | ChatResponsePullRequestPart | ChatPrepareToolInvocationPart | ChatToolInvocationPart | ChatResponseMultiDiffPart | ChatResponseThinkingProgressPart;
 	export class ChatResponseWarningPart {
 		value: MarkdownString;
 		constructor(value: string | MarkdownString);
@@ -172,23 +97,6 @@ declare module 'vscode' {
 		constructor(value: string, task?: (progress: Progress<ChatResponseWarningPart | ChatResponseReferencePart>) => Thenable<string | void>);
 	}
 
-	/**
-	 * A specialized progress part for displaying thinking/reasoning steps.
-	 */
-	export class ChatResponseThinkingProgressPart {
-		value: string | string[];
-		id?: string;
-		metadata?: { readonly [key: string]: any };
-		task?: (progress: Progress<LanguageModelThinkingPart>) => Thenable<string | void>;
-
-		/**
-		 * Creates a new thinking progress part.
-		 * @param value An initial progress message
-		 * @param task A task that will emit thinking parts during its execution
-		 */
-		constructor(value: string | string[], id?: string, metadata?: { readonly [key: string]: any }, task?: (progress: Progress<LanguageModelThinkingPart>) => Thenable<string | void>);
-	}
-
 	export class ChatResponseReferencePart2 {
 		/**
 		 * The reference target.
@@ -263,15 +171,6 @@ declare module 'vscode' {
 		constructor(extensions: string[]);
 	}
 
-	export class ChatResponsePullRequestPart {
-		readonly uri: Uri;
-		readonly linkTag: string;
-		readonly title: string;
-		readonly description: string;
-		readonly author: string;
-		constructor(uri: Uri, title: string, description: string, author: string, linkTag: string);
-	}
-
 	export interface ChatResponseStream {
 
 		/**
@@ -284,8 +183,6 @@ declare module 'vscode' {
 		*/
 		progress(value: string, task?: (progress: Progress<ChatResponseWarningPart | ChatResponseReferencePart>) => Thenable<string | void>): void;
 
-		thinkingProgress(thinkingDelta: ThinkingDelta): void;
-
 		textEdit(target: Uri, edits: TextEdit | TextEdit[]): void;
 
 		textEdit(target: Uri, isDone: true): void;
@@ -308,7 +205,7 @@ declare module 'vscode' {
 		 * TODO@API should this be MarkdownString?
 		 * TODO@API should actually be a more generic function that takes an array of buttons
 		 */
-		confirmation(title: string, message: string | MarkdownString, data: any, buttons?: string[]): void;
+		confirmation(title: string, message: string, data: any, buttons?: string[]): void;
 
 		/**
 		 * Push a warning to this stream. Short-hand for
@@ -328,8 +225,6 @@ declare module 'vscode' {
 		prepareToolInvocation(toolName: string): void;
 
 		push(part: ExtendedChatResponsePart): void;
-
-		clearToPreviousToolInvocation(reason: ChatResponseClearToPreviousToolInvocationReason): void;
 	}
 
 	export enum ChatResponseReferencePartStatusKind {
@@ -338,26 +233,6 @@ declare module 'vscode' {
 		Omitted = 3
 	}
 
-	export type ThinkingDelta = {
-		text?: string | string[];
-		id: string;
-		metadata?: { readonly [key: string]: any };
-	} | {
-		text?: string | string[];
-		id?: string;
-		metadata: { readonly [key: string]: any };
-	} |
-	{
-		text: string | string[];
-		id?: string;
-		metadata?: { readonly [key: string]: any };
-	};
-
-	export enum ChatResponseClearToPreviousToolInvocationReason {
-		NoReason = 0,
-		FilteredContentRetry = 1,
-		CopyrightContentRetry = 2,
-	}
 
 	/**
 	 * Does this piggy-back on the existing ChatRequest, or is it a different type of request entirely?
@@ -390,43 +265,6 @@ declare module 'vscode' {
 		export const onDidChangeChatRequestTools: Event<ChatRequest>;
 	}
 
-	export class LanguageModelToolExtensionSource {
-		/**
-		 * ID of the extension that published the tool.
-		 */
-		readonly id: string;
-
-		/**
-		 * Label of the extension that published the tool.
-		 */
-		readonly label: string;
-
-		private constructor(id: string, label: string);
-	}
-
-	export class LanguageModelToolMCPSource {
-		/**
-		 * Editor-configured label of the MCP server that published the tool.
-		 */
-		readonly label: string;
-
-		/**
-		 * Server-defined name of the MCP server.
-		 */
-		readonly name: string;
-
-		/**
-		 * Server-defined instructions for MCP tool use.
-		 */
-		readonly instructions?: string;
-
-		private constructor(label: string, name: string, instructions?: string);
-	}
-
-	export interface LanguageModelToolInformation {
-		source: LanguageModelToolExtensionSource | LanguageModelToolMCPSource | undefined;
-	}
-
 	// TODO@API fit this into the stream
 	export interface ChatUsedContext {
 		documents: ChatDocumentContext[];
@@ -476,10 +314,6 @@ declare module 'vscode' {
 			participant?: string;
 			command?: string;
 		};
-		/**
-		 * An optional detail string that will be rendered at the end of the response in certain UI contexts.
-		 */
-		details?: string;
 	}
 
 	export namespace chat {
@@ -574,17 +408,6 @@ declare module 'vscode' {
 		outcome: ChatEditingSessionActionOutcome;
 	}
 
-	export interface ChatEditingHunkAction {
-		// eslint-disable-next-line local/vscode-dts-string-type-literals
-		kind: 'chatEditingHunkAction';
-		uri: Uri;
-		lineCount: number;
-		linesAdded: number;
-		linesRemoved: number;
-		outcome: ChatEditingSessionActionOutcome;
-		hasRemainingEdits: boolean;
-	}
-
 	export enum ChatEditingSessionActionOutcome {
 		Accepted = 1,
 		Rejected = 2,
@@ -593,19 +416,14 @@ declare module 'vscode' {
 
 	export interface ChatUserActionEvent {
 		readonly result: ChatResult;
-		readonly action: ChatCopyAction | ChatInsertAction | ChatApplyAction | ChatTerminalAction | ChatCommandAction | ChatFollowupAction | ChatBugReportAction | ChatEditorAction | ChatEditingSessionAction | ChatEditingHunkAction;
+		readonly action: ChatCopyAction | ChatInsertAction | ChatApplyAction | ChatTerminalAction | ChatCommandAction | ChatFollowupAction | ChatBugReportAction | ChatEditorAction | ChatEditingSessionAction;
 	}
 
 	export interface ChatPromptReference {
 		/**
 		 * TODO Needed for now to drive the variableName-type reference, but probably both of these should go away in the future.
 		 */
 		readonly name: string;
-
-		/**
-		 * The list of tools were referenced in the value of the reference
-		 */
-		readonly toolReferences?: readonly ChatLanguageModelToolReference[];
 	}
 
 	export interface ChatResultFeedback {
@@ -647,13 +465,6 @@ declare module 'vscode' {
 	}
 
 	export interface ChatRequest {
-		readonly modeInstructions?: string;
-		readonly modeInstructions2?: ChatRequestModeInstructions;
-	}
-
-	export interface ChatRequestModeInstructions {
-		readonly content: string;
-		readonly toolReferences?: readonly ChatLanguageModelToolReference[];
-		readonly metadata?: Record<string, boolean | string | number>;
+		modeInstructions?: string;
 	}
 }