chore: address copilot suggestions

Author: himanshusinghs

src/transports/dryModeRunner.ts

@@ -39,12 +39,13 @@ export class DryRunModeRunner extends TransportRunnerBase {
     }
 
     private dumpTools(): void {
-        const tools = this.server?.tools
-            .filter((tool) => tool.isEnabled())
-            .map((tool) => ({
-                name: tool.name,
-                category: tool.category,
-            }));
+        const tools =
+            this.server?.tools
+                .filter((tool) => tool.isEnabled())
+                .map((tool) => ({
+                    name: tool.name,
+                    category: tool.category,
+                })) ?? [];
         this.consoleLogger.log("Enabled tools:");
         this.consoleLogger.log(JSON.stringify(tools, null, 2));
     }

tests/e2e/cli.test.ts

@@ -1,27 +1,27 @@
 import path from "path";
-import { exec } from "child_process";
+import { execFile } from "child_process";
 import { promisify } from "util";
 import { describe, expect, it } from "vitest";
 import packageJson from "../../package.json" with { type: "json" };
 
-const execAsync = promisify(exec);
+const execFileAsync = promisify(execFile);
 const CLI_PATH = path.join(import.meta.dirname, "..", "..", "dist", "index.js");
 
 describe("CLI entrypoint", () => {
     it("should handle version request", async () => {
-        const { stdout, stderr } = await execAsync(`${process.execPath} ${CLI_PATH} --version`);
+        const { stdout, stderr } = await execFileAsync(process.execPath, [CLI_PATH, "--version"]);
         expect(stdout).toContain(packageJson.version);
         expect(stderr).toEqual("");
     });
 
     it("should handle help request", async () => {
-        const { stdout, stderr } = await execAsync(`${process.execPath} ${CLI_PATH} --help`);
+        const { stdout, stderr } = await execFileAsync(process.execPath, [CLI_PATH, "--help"]);
         expect(stdout).toContain("For usage information refer to the README.md");
         expect(stderr).toEqual("");
     });
 
     it("should handle dry run request", async () => {
-        const { stdout, stderr } = await execAsync(`${process.execPath} ${CLI_PATH} --dryRun`);
+        const { stdout, stderr } = await execFileAsync(process.execPath, [CLI_PATH, "--dryRun"]);
         expect(stdout).toContain("Configuration:");
         expect(stdout).toContain("Enabled tools:");
         expect(stderr).toEqual("");