diff --git a/extensions/copilot/src/extension/intents/node/toolCallingLoop.ts b/extensions/copilot/src/extension/intents/node/toolCallingLoop.ts index ea4e04fc0a0..cc40ed2c6e3 100644 --- a/extensions/copilot/src/extension/intents/node/toolCallingLoop.ts +++ b/extensions/copilot/src/extension/intents/node/toolCallingLoop.ts @@ -39,7 +39,7 @@ import { Mutable } from '../../../util/vs/base/common/types'; import { URI } from '../../../util/vs/base/common/uri'; import { generateUuid } from '../../../util/vs/base/common/uuid'; import { IInstantiationService } from '../../../util/vs/platform/instantiation/common/instantiation'; -import { ChatResponsePullRequestPart, LanguageModelDataPart2, LanguageModelPartAudience, LanguageModelTextPart, LanguageModelToolResult2, MarkdownString } from '../../../vscodeTypes'; +import { ChatResponsePullRequestPart, LanguageModelDataPart2, LanguageModelPartAudience, LanguageModelToolResult2, MarkdownString } from '../../../vscodeTypes'; import { InteractionOutcomeComputer } from '../../inlineChat/node/promptCraftingTypes'; import { ChatVariablesCollection } from '../../prompt/common/chatVariablesCollection'; import { AnthropicTokenUsageMetadata, Conversation, IResultMetadata, ResponseStreamParticipant, TurnStatus } from '../../prompt/common/conversation'; @@ -1318,14 +1318,6 @@ export abstract class ToolCallingLoop = Event.None; } diff --git a/extensions/copilot/src/platform/requestLogger/node/requestLogger.ts b/extensions/copilot/src/platform/requestLogger/node/requestLogger.ts index 31e4da24b73..616273ccabd 100644 --- a/extensions/copilot/src/platform/requestLogger/node/requestLogger.ts +++ b/extensions/copilot/src/platform/requestLogger/node/requestLogger.ts @@ -77,7 +77,6 @@ export abstract class AbstractRequestLogger extends Disposable implements IReque public abstract logModelListCall(id: string, requestMetadata: RequestMetadata, models: IModelAPIResponse[]): void; public abstract logToolCall(id: string, name: string | undefined, args: unknown, response: LanguageModelToolResult2): void; - public abstract logServerToolCall(id: string, name: string, args: unknown, result: LanguageModelToolResult2): void; public logContentExclusionRules(_repos: string[], _rules: { patterns: string[]; ifAnyMatch: string[]; ifNoneMatch: string[] }[], _durationMs: number): void { // no-op by default; concrete implementations can override diff --git a/extensions/copilot/src/platform/requestLogger/test/node/testRequestLogger.ts b/extensions/copilot/src/platform/requestLogger/test/node/testRequestLogger.ts index 9fc97242663..686b696002c 100644 --- a/extensions/copilot/src/platform/requestLogger/test/node/testRequestLogger.ts +++ b/extensions/copilot/src/platform/requestLogger/test/node/testRequestLogger.ts @@ -59,11 +59,6 @@ export class TestRequestLogger extends AbstractRequestLogger { this._onDidChangeRequests.fire(); } - public override logServerToolCall(id: string, name: string, args: unknown, result: LanguageModelToolResult2): void { - this._entries.push(new TestLoggedToolCall(id, name, args, result, this.currentRequest, Date.now())); - this._onDidChangeRequests.fire(); - } - /** * Clear all logged entries (useful between tests). */