Missing renames

Author: osortega

src/vs/workbench/api/browser/mainThreadChatSessions.ts

@@ -7,7 +7,7 @@ import { CancellationToken } from '../../../base/common/cancellation.js';
 import { Disposable, DisposableMap } from '../../../base/common/lifecycle.js';
 import { URI, UriComponents } from '../../../base/common/uri.js';
 import { ILogService } from '../../../platform/log/common/log.js';
-import { IChatSessionDefinition, IChatSessionDefinitionProvider, IChatSessionsService } from '../../contrib/chat/common/chatSessionsService.js';
+import { IChatSessionItem, IChatSessionItemProvider, IChatSessionsService } from '../../contrib/chat/common/chatSessionsService.js';
 import { extHostNamedCustomer, IExtHostContext } from '../../services/extensions/common/extHostCustomers.js';
 import { ExtHostContext, MainContext, MainThreadChatSessionsShape } from '../common/extHost.protocol.js';
 
@@ -25,14 +25,14 @@ export class MainThreadChatSessions extends Disposable implements MainThreadChat
 
 	$registerChatSessionItemProvider(handle: number, chatSessionType: string): void {
 		// Register the provider handle - this tracks that a provider exists
-		const provider: IChatSessionDefinitionProvider = {
+		const provider: IChatSessionItemProvider = {
 			chatSessionType,
-			provideChatSessionDefinitions: (token) => this._provideChatSessionsInformation(handle, token)
+			provideChatSessionItems: (token) => this._provideChatSessionItems(handle, token)
 		};
-		this._registrations.set(handle, this._chatSessionsService.registerChatSessionDefinitionProvider(handle, provider));
+		this._registrations.set(handle, this._chatSessionsService.registerChatSessionItemProvider(handle, provider));
 	}
 
-	private async _provideChatSessionsInformation(handle: number, token: CancellationToken): Promise<IChatSessionDefinition[]> {
+	private async _provideChatSessionItems(handle: number, token: CancellationToken): Promise<IChatSessionItem[]> {
 		const proxy = this._extHostContext.getProxy(ExtHostContext.ExtHostChatSessions);
 
 		try {
@@ -56,7 +56,7 @@ export class MainThreadChatSessions extends Disposable implements MainThreadChat
 
 	private _reviveIconPath(
 		iconPath: UriComponents | { light: UriComponents; dark: UriComponents } | { id: string; color?: { id: string } | undefined })
-		: IChatSessionDefinition['iconPath'] {
+		: IChatSessionItem['iconPath'] {
 		if (!iconPath) {
 			return undefined;
 		}

src/vs/workbench/api/common/extHost.protocol.ts

@@ -58,7 +58,7 @@ import { ICodeMapperRequest, ICodeMapperResult } from '../../contrib/chat/common
 import { IChatRelatedFile, IChatRelatedFileProviderMetadata as IChatRelatedFilesProviderMetadata, IChatRequestDraft } from '../../contrib/chat/common/chatEditingService.js';
 import { IChatProgressHistoryResponseContent } from '../../contrib/chat/common/chatModel.js';
 import { IChatContentInlineReference, IChatFollowup, IChatNotebookEdit, IChatProgress, IChatTask, IChatTaskDto, IChatUserActionEvent, IChatVoteAction } from '../../contrib/chat/common/chatService.js';
-import { IChatSessionDefinition as IChatSessionDefinition } from '../../contrib/chat/common/chatSessionsService.js';
+import { IChatSessionItem } from '../../contrib/chat/common/chatSessionsService.js';
 import { IChatRequestVariableValue } from '../../contrib/chat/common/chatVariables.js';
 import { ChatAgentLocation } from '../../contrib/chat/common/constants.js';
 import { IChatMessage, IChatResponseFragment, ILanguageModelChatMetadataAndIdentifier, ILanguageModelChatSelector } from '../../contrib/chat/common/languageModels.js';
@@ -3107,7 +3107,7 @@ export interface MainThreadChatSessionsShape extends IDisposable {
 }
 
 export interface ExtHostChatSessionsShape {
-	$provideChatSessionItems(handle: number, token: CancellationToken): Promise<IChatSessionDefinition[]>;
+	$provideChatSessionItems(handle: number, token: CancellationToken): Promise<IChatSessionItem[]>;
 }
 
 // --- proxy identifiers

src/vs/workbench/api/common/extHostChatSessions.ts

@@ -11,7 +11,7 @@ import { Proxied } from '../../services/extensions/common/proxyIdentifier.js';
 import { ExtHostChatSessionsShape, MainContext, MainThreadChatSessionsShape } from './extHost.protocol.js';
 import { ExtHostCommands } from './extHostCommands.js';
 import { IExtHostRpcService } from './extHostRpcService.js';
-import { IChatSessionDefinition as IChatSessionItem } from '../../contrib/chat/common/chatSessionsService.js';
+import { IChatSessionItem } from '../../contrib/chat/common/chatSessionsService.js';
 
 export class ExtHostChatSessions extends Disposable implements ExtHostChatSessionsShape {
 

src/vs/workbench/contrib/chat/common/chatSessionsService.ts

@@ -11,7 +11,7 @@ import { InstantiationType, registerSingleton } from '../../../../platform/insta
 import { URI } from '../../../../base/common/uri.js';
 import { ThemeIcon } from '../../../../base/common/themables.js';
 
-export interface IChatSessionDefinition {
+export interface IChatSessionItem {
 	id: string;
 	label: string;
 	iconPath?: URI | {
@@ -20,38 +20,38 @@ export interface IChatSessionDefinition {
 	} | ThemeIcon;
 }
 
-export interface IChatSessionDefinitionProvider {
+export interface IChatSessionItemProvider {
 	readonly chatSessionType: string;
-	provideChatSessionDefinitions(token: CancellationToken): Promise<IChatSessionDefinition[]>;
+	provideChatSessionItems(token: CancellationToken): Promise<IChatSessionItem[]>;
 }
 
 export interface IChatSessionsService {
 	readonly _serviceBrand: undefined;
-	registerChatSessionDefinitionProvider(handle: number, provider: IChatSessionDefinitionProvider): IDisposable;
-	hasChatSessionDefinitionProviders: boolean;
-	provideChatSessionDefinitions(token: CancellationToken): Promise<{ provider: IChatSessionDefinitionProvider; session: IChatSessionDefinition }[]>;
+	registerChatSessionItemProvider(handle: number, provider: IChatSessionItemProvider): IDisposable;
+	hasChatSessionItemProviders: boolean;
+	provideChatSessionItems(token: CancellationToken): Promise<{ provider: IChatSessionItemProvider; session: IChatSessionItem }[]>;
 }
 
 export const IChatSessionsService = createDecorator<IChatSessionsService>('chatSessionsService');
 
 export class ChatSessionsService extends Disposable implements IChatSessionsService {
 	readonly _serviceBrand: undefined;
-	private _providers: Map<number, IChatSessionDefinitionProvider> = new Map();
+	private _providers: Map<number, IChatSessionItemProvider> = new Map();
 
 	constructor(
 		@ILogService private readonly _logService: ILogService,
 	) {
 		super();
 	}
 
-	public async provideChatSessionDefinitions(token: CancellationToken): Promise<{ provider: IChatSessionDefinitionProvider; session: IChatSessionDefinition }[]> {
-		const results: { provider: IChatSessionDefinitionProvider; session: IChatSessionDefinition }[] = [];
+	public async provideChatSessionItems(token: CancellationToken): Promise<{ provider: IChatSessionItemProvider; session: IChatSessionItem }[]> {
+		const results: { provider: IChatSessionItemProvider; session: IChatSessionItem }[] = [];
 
 		// Iterate through all registered providers and collect their results
 		for (const [handle, provider] of this._providers) {
 			try {
-				if (provider.provideChatSessionDefinitions) {
-					const sessions = await provider.provideChatSessionDefinitions(token);
+				if (provider.provideChatSessionItems) {
+					const sessions = await provider.provideChatSessionItems(token);
 					results.push(...sessions.map(session => ({ provider, session })));
 				}
 			} catch (error) {
@@ -65,7 +65,7 @@ export class ChatSessionsService extends Disposable implements IChatSessionsServ
 		return results;
 	}
 
-	public registerChatSessionDefinitionProvider(handle: number, provider: IChatSessionDefinitionProvider): IDisposable {
+	public registerChatSessionItemProvider(handle: number, provider: IChatSessionItemProvider): IDisposable {
 		this._providers.set(handle, provider);
 		return {
 			dispose: () => {
@@ -74,7 +74,7 @@ export class ChatSessionsService extends Disposable implements IChatSessionsServ
 		};
 	}
 
-	public get hasChatSessionDefinitionProviders(): boolean {
+	public get hasChatSessionItemProviders(): boolean {
 		return this._providers.size > 0;
 	}
 }