Microsoft/vscode
1
0
Fork 0
mirror of https://github.com/microsoft/vscode.git synced 2026-04-18 15:55:59 +01:00
Code Issues Packages Projects Releases Wiki Activity

Agent Debug Panel: Copilot CLI changes (#302552)

Browse Source 
* Copilot CLI changes

* feedbback changes
This commit is contained in:
Vijay Upadya
2026-03-17 14:52:55 -07:00
committed by GitHub
parent 56c8cb1e6a
commit 3e6435e12b
4 changed files with 60 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
src/vs/workbench/contrib/chat/test/common/chatDebugServiceImpl.test.ts
View File

@@ -21,7 +21,8 @@ suite('ChatDebugServiceImpl', () => {
const sessionA = LocalChatSessionUri.forSession('a');
const sessionB = LocalChatSessionUri.forSession('b');
const sessionGeneric = URI.parse('vscode-chat-session://local/session');
const nonLocalSession = URI.parse('vscode-chat-session://remote-provider/session-1');
const nonLocalSession = URI.parse('some-other-scheme://authority/session-1');
const copilotCliSession = URI.parse('copilotcli:/test-session-id');
setup(() => {
service = disposables.add(new ChatDebugServiceImpl());
@@ -148,7 +149,7 @@ suite('ChatDebugServiceImpl', () => {
assert.strictEqual(event.parentEventId, 'parent-1');
});
test('should not log events for non-local sessions', () => {
test('should not log events for ineligible session schemes', () => {
const firedEvents: IChatDebugEvent[] = [];
disposables.add(service.onDidAddEvent(e => firedEvents.push(e)));
@@ -157,6 +158,16 @@ suite('ChatDebugServiceImpl', () => {
assert.strictEqual(firedEvents.length, 0);
assert.strictEqual(service.getEvents(nonLocalSession).length, 0);
});
test('should log events for copilotcli sessions', () => {
const firedEvents: IChatDebugEvent[] = [];
disposables.add(service.onDidAddEvent(e => firedEvents.push(e)));
service.log(copilotCliSession, 'cli-event', 'details');
assert.strictEqual(firedEvents.length, 1);
assert.strictEqual(service.getEvents(copilotCliSession).length, 1);
});
});
suite('getSessionResources', () => {
@@ -435,7 +446,7 @@ suite('ChatDebugServiceImpl', () => {
assert.strictEqual(tokenA.isCancellationRequested, false, 'session-a token should not be cancelled');
});
test('should not invoke providers for non-local sessions', async () => {
test('should not invoke providers for ineligible session schemes', async () => {
let providerCalled = false;
const provider: IChatDebugLogProvider = {
@@ -458,6 +469,29 @@ suite('ChatDebugServiceImpl', () => {
assert.strictEqual(service.getEvents(nonLocalSession).length, 0);
});
test('should invoke providers for copilotcli sessions', async () => {
let providerCalled = false;
const provider: IChatDebugLogProvider = {
provideChatDebugLog: async () => {
providerCalled = true;
return [{
kind: 'generic',
sessionResource: copilotCliSession,
created: new Date(),
name: 'cli-provider-event',
level: ChatDebugLogLevel.Info,
}];
},
};
disposables.add(service.registerProvider(provider));
await service.invokeProviders(copilotCliSession);
assert.strictEqual(providerCalled, true);
assert.ok(service.getEvents(copilotCliSession).length > 0);
});
test('newly registered provider should be invoked for active sessions', async () => {
// Start an invocation before the provider is registered
const firstProvider: IChatDebugLogProvider = {