Disallow and suppress misused spread operator. (#23294)

This commit is contained in:
Christian Gunderman
2026-03-21 05:21:53 +00:00
committed by GitHub
parent d114f7feb8
commit 6ade35aace
25 changed files with 59 additions and 0 deletions
@@ -511,6 +511,7 @@ describe('McpClientManager', () => {
await manager.startExtension(extension);
mockedMcpClient.getServerConfig.mockReturnValue({
// eslint-disable-next-line @typescript-eslint/no-misused-spread
...extension.mcpServers!['test-server'],
extension,
});
@@ -215,6 +215,7 @@ export class McpClientManager {
await Promise.all(
Object.entries(extension.mcpServers ?? {}).map(([name, config]) =>
this.maybeDiscoverMcpServer(name, {
// eslint-disable-next-line @typescript-eslint/no-misused-spread
...config,
extension,
}),
@@ -331,7 +332,9 @@ export class McpClientManager {
const env = { ...(base.env ?? {}), ...(override.env ?? {}) };
return {
// eslint-disable-next-line @typescript-eslint/no-misused-spread
...base,
// eslint-disable-next-line @typescript-eslint/no-misused-spread
...override,
includeTools,
excludeTools: excludeTools.length > 0 ? excludeTools : undefined,
@@ -367,6 +367,7 @@ describe('WriteFileTool', () => {
const abortSignal = new AbortController().signal;
const mockGemini3Config = {
// eslint-disable-next-line @typescript-eslint/no-misused-spread
...mockConfig,
getActiveModel: () => 'gemini-3.0-pro',
} as unknown as Config;