Skip to content

chat - core provided chat submenus in components #250851

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 6, 2025
Merged

chat - core provided chat submenus in components #250851

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 30 additions & 0 deletions src/vs/workbench/contrib/chat/browser/actions/chatActions.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1036,3 +1036,33 @@ export function shouldShowClearEditingSessionConfirmation(editingSession: IChatE

return false;
}

// --- Chat Submenus in various Components

const menuContext = ContextKeyExpr.and(
ChatContextKeys.Setup.hidden.negate(),
ChatContextKeys.Setup.disabled.negate()
);

const title = localize('copilot', "Copilot");

MenuRegistry.appendMenuItem(MenuId.EditorContext, {
submenu: MenuId.ChatTextEditorMenu,
group: '1_copilot',
title,
when: menuContext
});

MenuRegistry.appendMenuItem(MenuId.ExplorerContext, {
submenu: MenuId.ChatExplorerMenu,
group: '5_copilot',
title,
when: menuContext
});

MenuRegistry.appendMenuItem(MenuId.TerminalInstanceContext, {
submenu: MenuId.ChatTerminalMenu,
group: '2_copilot',
title,
when: menuContext
});
35 changes: 1 addition & 34 deletions src/vs/workbench/contrib/chat/browser/chatSetup.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ import { URI } from '../../../../base/common/uri.js';
import { ServicesAccessor } from '../../../../editor/browser/editorExtensions.js';
import { MarkdownRenderer } from '../../../../editor/browser/widget/markdownRenderer/browser/markdownRenderer.js';
import { localize, localize2 } from '../../../../nls.js';
import { Action2, MenuId, MenuRegistry, registerAction2 } from '../../../../platform/actions/common/actions.js';
import { Action2, MenuId, registerAction2 } from '../../../../platform/actions/common/actions.js';
import { ICommandService } from '../../../../platform/commands/common/commands.js';
import { ConfigurationTarget, IConfigurationService } from '../../../../platform/configuration/common/configuration.js';
import { Extensions as ConfigurationExtensions, IConfigurationRegistry } from '../../../../platform/configuration/common/configurationRegistry.js';
Expand Down Expand Up @@ -988,7 +988,6 @@ export class ChatSetupContribution extends Disposable implements IWorkbenchContr

this.registerSetupAgents(context, controller);
this.registerActions(context, requests, controller);
this.registerMenus();
this.registerUrlLinkHandler();
}

Expand Down Expand Up @@ -1294,38 +1293,6 @@ export class ChatSetupContribution extends Disposable implements IWorkbenchContr
registerAction2(EnableOveragesAction);
}

private registerMenus(): void {
const menuContext = ContextKeyExpr.and(
// TODO@bpasero this needs to be revisited when the Copilot
// extension contributes this OOTB where this menu is also
// defined.
ChatContextKeys.Setup.installed.negate(),
ChatContextKeys.Setup.hidden.negate(),
ChatContextKeys.Setup.disabled.negate()
);

MenuRegistry.appendMenuItem(MenuId.ExplorerContext, {
submenu: MenuId.ChatExplorerMenu,
group: '5_copilot',
title: localize('title4', "Copilot"),
when: menuContext
});

MenuRegistry.appendMenuItem(MenuId.EditorContext, {
submenu: MenuId.ChatTextEditorMenu,
group: '1_copilot',
title: localize('title4', "Copilot"),
when: menuContext
});

MenuRegistry.appendMenuItem(MenuId.TerminalInstanceContext, {
submenu: MenuId.ChatTerminalMenu,
group: '2_copilot',
title: localize('title4', "Copilot"),
when: menuContext
});
}

private registerUrlLinkHandler(): void {
this._register(ExtensionUrlHandlerOverrideRegistry.registerHandler({
canHandleURL: url => {
Expand Down
21 changes: 21 additions & 0 deletions src/vs/workbench/services/actions/common/menusExtensionPoint.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -450,6 +450,27 @@ const apiMenus: IAPIMenu[] = [
description: localize('menus.chatModelPicker', "The chat model picker dropdown menu"),
supportsSubmenus: false,
proposed: 'chatParticipantPrivate'
},
{
key: 'explorer/context/chat',
id: MenuId.ChatExplorerMenu,
description: localize('menus.chatExplorer', "The Chat submenu in the explorer context menu."),
supportsSubmenus: false,
proposed: 'chatParticipantPrivate'
},
{
key: 'editor/context/chat',
id: MenuId.ChatTextEditorMenu,
description: localize('menus.chatTextEditor', "The Chat submenu in the text editor context menu."),
supportsSubmenus: false,
proposed: 'chatParticipantPrivate'
},
{
key: 'terminal/context/chat',
id: MenuId.ChatTerminalMenu,
description: localize('menus.chatTerminal', "The Chat submenu in the terminal context menu."),
supportsSubmenus: false,
proposed: 'chatParticipantPrivate'
}
];

Expand Down
Loading