MediaMetz/gemini-cli
1
0
Fork 0
mirror of https://github.com/google-gemini/gemini-cli.git synced 2026-03-17 09:30:58 -07:00
Code Issues Packages Projects Releases Wiki Activity

feat: enable message bus integration by default (#14329)

Browse Source
This commit is contained in:
Allen Hutchison
2025-12-02 16:05:54 -08:00
committed by GitHub
parent 344f2f26e7
commit 533a3fb312
5 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
packages/cli/src/config/config.test.ts
View File

@@ -632,6 +632,14 @@ describe('loadCliConfig', () => {
DEFAULT_FILE_FILTERING_OPTIONS.respectGeminiIgnore,
);
});
it('should default enableMessageBusIntegration to true when unconfigured', async () => {
process.argv = ['node', 'script.js'];
const argv = await parseArguments({} as Settings);
const settings: Settings = {};
const config = await loadCliConfig(settings, 'test-session', argv);
expect(config['enableMessageBusIntegration']).toBe(true);
});
});
describe('Hierarchical Memory Loading (config.ts) - Placeholder Suite', () => {

2
packages/cli/src/config/config.ts
View File

@@ -513,7 +513,7 @@ export async function loadCliConfig(
);
const enableMessageBusIntegration =
settings.tools?.enableMessageBusIntegration ?? false;
settings.tools?.enableMessageBusIntegration ?? true;
const allowedTools = argv.allowedTools || settings.tools?.allowed || [];
const allowedToolsSet = new Set(allowedTools);

2
packages/cli/src/config/settingsSchema.ts
View File

@@ -1055,7 +1055,7 @@ const SETTINGS_SCHEMA = {
label: 'Enable Message Bus Integration',
category: 'Tools',
requiresRestart: true,
default: false,
default: true,
description: oneLine`
Enable policy-based tool confirmation via message bus integration.
When enabled, tools automatically respect policy engine decisions (ALLOW/DENY/ASK_USER) without requiring individual tool implementations.