fix: resolve IDE parameter mismatches and ensure memory tool security

Author: sontoriyama

packages/core/src/tools/memoryTool.ts

@@ -310,14 +310,13 @@ export class MemoryTool
   static readonly Name = MEMORY_TOOL_NAME;
 
   constructor(messageBus: MessageBus) {
-    // Standard initialization for MemoryTool with messageBus for confirmation
+    // Initialize the tool with its specific properties
     super(
       MemoryTool.Name,
       'SaveMemory',
       memoryToolDescription,
       Kind.Think,
       memoryToolSchemaData.parametersJsonSchema as Record<string, unknown>,
-      // @ts-expect-error - The IDE incorrectly shifts parameter positions in the constructor hierarchy, but runtime/tsc are correct
       messageBus,
       true,
       false,

packages/core/src/utils/memoryDiscovery.test.ts

@@ -20,7 +20,6 @@ import {
   DEFAULT_CONTEXT_FILENAME,
 } from '../tools/memoryTool.js';
 import { FileDiscoveryService } from '../services/fileDiscoveryService.js';
-// @ts-expect-error - The IDE incorrectly identifies this as missing, but it exists and tests pass
 import { GEMINI_DIR, homedir as pathsHomedir } from './paths.js';
 import { Config, type GeminiCLIExtension } from '../config/config.js';
 import { Storage } from '../config/storage.js';