Address PR #243 review feedback

Author: data-douser

server/dist/codeql-development-mcp-server.js

@@ -193973,8 +193973,11 @@ function registerCLITool(server, definition) {
           const lastDumpDilFlag = [...rawAdditionalArgs].reverse().find(
             (arg) => arg === "--dump-dil" || arg === "--no-dump-dil"
           );
-          if (lastDumpDilFlag === "--dump-dil") options["dump-dil"] = true;
-          else if (lastDumpDilFlag === "--no-dump-dil") options["dump-dil"] = false;
+          if (lastDumpDilFlag === "--dump-dil") {
+            options["dump-dil"] = true;
+          } else if (lastDumpDilFlag === "--no-dump-dil") {
+            options["dump-dil"] = false;
+          }
           if (lastDumpDilFlag !== void 0) {
             rawAdditionalArgs = rawAdditionalArgs.filter(
               (arg) => arg !== "--dump-dil" && arg !== "--no-dump-dil"

server/dist/codeql-development-mcp-server.js.map

@@ -575,8 +575,11 @@ export function registerCLITool(server: McpServer, definition: CLIToolDefinition
           const lastDumpDilFlag = [...rawAdditionalArgs].reverse().find(
             (arg) => arg === '--dump-dil' || arg === '--no-dump-dil',
           );
-          if (lastDumpDilFlag === '--dump-dil') options['dump-dil'] = true;
-          else if (lastDumpDilFlag === '--no-dump-dil') options['dump-dil'] = false;
+          if (lastDumpDilFlag === '--dump-dil') {
+            options['dump-dil'] = true;
+          } else if (lastDumpDilFlag === '--no-dump-dil') {
+            options['dump-dil'] = false;
+          }
           if (lastDumpDilFlag !== undefined) {
             rawAdditionalArgs = rawAdditionalArgs.filter(
               (arg) => arg !== '--dump-dil' && arg !== '--no-dump-dil',

server/src/lib/cli-tool-registry.ts

@@ -955,7 +955,7 @@ describe('registerCLITool handler behavior', () => {
     const handler = (mockServer.registerTool as ReturnType<typeof vi.fn>).mock.calls[0][2];
 
     executeCodeQLCommand.mockResolvedValueOnce({
-      stdout: 'DIL output here',
+      stdout: '',
       stderr: '',
       success: true
     });