Skip to content

Commit d1895da

Browse files
authored
Align "Configure Tool Sets" command (microsoft#250605)
* add to view menu * change wording to "Create new..." * ellipsis in title microsoft#250474
1 parent a4dc523 commit d1895da

File tree

1 file changed

+16
-11
lines changed

1 file changed

+16
-11
lines changed

src/vs/workbench/contrib/chat/browser/tools/toolSetsContribution.ts

Lines changed: 16 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import { ThemeIcon } from '../../../../../base/common/themables.js';
1414
import { assertType, isObject } from '../../../../../base/common/types.js';
1515
import { URI } from '../../../../../base/common/uri.js';
1616
import { localize, localize2 } from '../../../../../nls.js';
17-
import { Action2 } from '../../../../../platform/actions/common/actions.js';
17+
import { Action2, MenuId } from '../../../../../platform/actions/common/actions.js';
1818
import { IFileService } from '../../../../../platform/files/common/files.js';
1919
import { ServicesAccessor } from '../../../../../platform/instantiation/common/instantiation.js';
2020
import { ILogService } from '../../../../../platform/log/common/log.js';
@@ -34,6 +34,8 @@ import { parse } from '../../../../../base/common/jsonc.js';
3434
import { IJSONSchema } from '../../../../../base/common/jsonSchema.js';
3535
import * as JSONContributionRegistry from '../../../../../platform/jsonschemas/common/jsonContributionRegistry.js';
3636
import { Registry } from '../../../../../platform/registry/common/platform.js';
37+
import { ContextKeyExpr } from '../../../../../platform/contextkey/common/contextkey.js';
38+
import { ChatViewId } from '../chat.js';
3739

3840

3941
const toolEnumValues: string[] = [];
@@ -317,9 +319,15 @@ export class ConfigureToolSets extends Action2 {
317319
constructor() {
318320
super({
319321
id: ConfigureToolSets.ID,
320-
title: localize2('chat.configureToolSets', 'Configure Tool Sets...'),
322+
title: localize2('chat.configureToolSets', 'Configure Tool Sets'),
321323
category: CHAT_CATEGORY,
322324
f1: true,
325+
menu: {
326+
id: MenuId.ViewTitle,
327+
when: ContextKeyExpr.equals('view', ChatViewId),
328+
order: 11,
329+
group: '2_manage'
330+
},
323331
});
324332
}
325333

@@ -334,6 +342,12 @@ export class ConfigureToolSets extends Action2 {
334342

335343
const picks: ((IQuickPickItem & { toolset?: ToolSet }) | IQuickPickSeparator)[] = [];
336344

345+
picks.push({
346+
label: localize('chat.configureToolSets.add', 'Create new tool sets file...'),
347+
alwaysShow: true,
348+
iconClass: ThemeIcon.asClassName(Codicon.plus)
349+
});
350+
337351
for (const toolSet of toolsService.toolSets.get()) {
338352
if (toolSet.source.type !== 'user') {
339353
continue;
@@ -347,15 +361,6 @@ export class ConfigureToolSets extends Action2 {
347361
});
348362
}
349363

350-
if (picks.length !== 0) {
351-
picks.push({ type: 'separator' });
352-
}
353-
354-
picks.push({
355-
label: localize('chat.configureToolSets.add', 'Add Tool Sets File...'),
356-
alwaysShow: true,
357-
iconClass: ThemeIcon.asClassName(Codicon.tools)
358-
});
359364

360365

361366
const pick = await quickInputService.pick(picks, {

0 commit comments

Comments
 (0)