enable cli_help agent by default (#16100)

This commit is contained in:
Tommaso Sciortino
2026-01-07 15:43:12 -08:00
committed by GitHub
parent 51d3f44d51
commit 1aa35c8796
7 changed files with 12 additions and 10 deletions
+1 -1
View File
@@ -1467,7 +1467,7 @@ const SETTINGS_SCHEMA = {
label: 'Enable CLI Help Agent',
category: 'Experimental',
requiresRestart: true,
default: false,
default: true,
description: 'Enable the CLI Help Agent.',
showInDialog: true,
},
+3 -4
View File
@@ -209,6 +209,7 @@ describe('AgentRegistry', () => {
const disabledConfig = makeFakeConfig({
enableAgents: false,
codebaseInvestigatorSettings: { enabled: false },
cliHelpAgentSettings: { enabled: false },
});
const disabledRegistry = new TestableAgentRegistry(disabledConfig);
@@ -220,10 +221,8 @@ describe('AgentRegistry', () => {
).not.toHaveBeenCalled();
});
it('should register CLI help agent if enabled', async () => {
const config = makeFakeConfig({
cliHelpAgentSettings: { enabled: true },
});
it('should register CLI help agent by default', async () => {
const config = makeFakeConfig();
const registry = new TestableAgentRegistry(config);
await registry.initialize();
+1
View File
@@ -941,6 +941,7 @@ describe('Server Config (config.ts)', () => {
const params: ConfigParameters = {
...baseParams,
codebaseInvestigatorSettings: { enabled: false },
cliHelpAgentSettings: { enabled: false },
};
const config = new Config(params);
+3 -2
View File
@@ -622,7 +622,7 @@ export class Config {
model: params.codebaseInvestigatorSettings?.model,
};
this.cliHelpAgentSettings = {
enabled: params.cliHelpAgentSettings?.enabled ?? false,
enabled: params.cliHelpAgentSettings?.enabled ?? true,
};
this.continueOnFailedApiCall = params.continueOnFailedApiCall ?? true;
this.enableShellOutputEfficiency =
@@ -1754,7 +1754,8 @@ export class Config {
// Register DelegateToAgentTool if agents are enabled
if (
this.isAgentsEnabled() ||
this.getCodebaseInvestigatorSettings().enabled
this.getCodebaseInvestigatorSettings().enabled ||
this.getCliHelpAgentSettings().enabled
) {
// Check if the delegate tool itself is allowed (if allowedTools is set)
const allowedTools = this.getAllowedTools();