fix: Remove confirmation from browser connect and disconnect tools (no-changelog) (#30096)

This commit is contained in:
Dimitri Lavrenük 2026-05-08 16:43:06 +02:00 committed by GitHub
parent 1e8f89bd5a
commit bda1782de6
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 16 additions and 4 deletions

View File

@ -50,6 +50,13 @@ describe('createSessionTools', () => {
});
});
describe('getAffectedResources', () => {
it('returns empty array (no confirmation required)', () => {
const result = tool.getAffectedResources({}, TOOL_CONTEXT);
expect(result).toEqual([]);
});
});
describe('execute', () => {
let freshConnection: ReturnType<typeof createMockConnection>;
@ -128,6 +135,13 @@ describe('createSessionTools', () => {
});
});
describe('getAffectedResources', () => {
it('returns empty array (no confirmation required)', () => {
const result = tool.getAffectedResources({}, TOOL_CONTEXT);
expect(result).toEqual([]);
});
});
describe('execute', () => {
it('calls connection.disconnect and returns disconnected true', async () => {
const fresh = createMockConnection();

View File

@ -61,7 +61,7 @@ function browserConnect(
}
},
getAffectedResources(_args, _context: ToolContext): AffectedResource[] {
return [{ toolGroup: 'browser', resource: 'browser', description: 'Connect to browser' }];
return [];
},
};
}
@ -96,9 +96,7 @@ function browserDisconnect(
}
},
getAffectedResources(_args, _context: ToolContext): AffectedResource[] {
return [
{ toolGroup: 'browser', resource: 'browser', description: 'Disconnect from browser' },
];
return [];
},
};
}