Changed all test tools to snake case, cleaned up console logging in test, fixed sandbox message on startup. Removed test:unit, made test target run prettier-check and all tests.

Author: BobDickinson

cli/__tests__/cli.test.ts

@@ -39,8 +39,8 @@ describe("CLI Tests", () => {
       // Validate expected tools from test-mcp-server
       const toolNames = json.tools.map((tool: any) => tool.name);
       expect(toolNames).toContain("echo");
-      expect(toolNames).toContain("get-sum");
-      expect(toolNames).toContain("get-annotated-message");
+      expect(toolNames).toContain("get_sum");
+      expect(toolNames).toContain("get_annotated_message");
     });
 
     it("should fail with nonexistent method", async () => {
@@ -315,7 +315,7 @@ describe("CLI Tests", () => {
         "--method",
         "prompts/get",
         "--prompt-name",
-        "simple-prompt",
+        "simple_prompt",
       ]);
 
       expectCliSuccess(result);
@@ -340,7 +340,7 @@ describe("CLI Tests", () => {
         "--method",
         "prompts/get",
         "--prompt-name",
-        "args-prompt",
+        "args_prompt",
         "--prompt-args",
         "city=New York",
         "state=NY",

cli/__tests__/tools.test.ts

@@ -31,8 +31,8 @@ describe("Tool Tests", () => {
       // Validate expected tools from test-mcp-server
       const toolNames = json.tools.map((tool: any) => tool.name);
       expect(toolNames).toContain("echo");
-      expect(toolNames).toContain("get-sum");
-      expect(toolNames).toContain("get-annotated-message");
+      expect(toolNames).toContain("get_sum");
+      expect(toolNames).toContain("get_annotated_message");
     });
   });
 
@@ -69,7 +69,7 @@ describe("Tool Tests", () => {
         "--method",
         "tools/call",
         "--tool-name",
-        "get-sum",
+        "get_sum",
         "--tool-arg",
         "a=42",
         "b=58",
@@ -95,7 +95,7 @@ describe("Tool Tests", () => {
         "--method",
         "tools/call",
         "--tool-name",
-        "get-sum",
+        "get_sum",
         "--tool-arg",
         "a=19.99",
         "b=20.01",
@@ -121,7 +121,7 @@ describe("Tool Tests", () => {
         "--method",
         "tools/call",
         "--tool-name",
-        "get-annotated-message",
+        "get_annotated_message",
         "--tool-arg",
         "messageType=success",
         "includeImage=true",
@@ -146,7 +146,7 @@ describe("Tool Tests", () => {
         "--method",
         "tools/call",
         "--tool-name",
-        "get-annotated-message",
+        "get_annotated_message",
         "--tool-arg",
         "messageType=error",
         "includeImage=false",
@@ -195,7 +195,7 @@ describe("Tool Tests", () => {
         "--method",
         "tools/call",
         "--tool-name",
-        "get-sum",
+        "get_sum",
         "--tool-arg",
         "a=42.5",
         "b=57.5",
@@ -421,7 +421,7 @@ describe("Tool Tests", () => {
         "--method",
         "prompts/get",
         "--prompt-name",
-        "args-prompt",
+        "args_prompt",
         "--prompt-args",
         "city=New York",
         "state=NY",
@@ -441,7 +441,7 @@ describe("Tool Tests", () => {
     });
 
     it("should handle prompt with simple arguments", async () => {
-      // Note: simple-prompt doesn't accept arguments, but the CLI should still
+      // Note: simple_prompt doesn't accept arguments, but the CLI should still
       // accept the command and the server should ignore the arguments
       const { command, args } = getTestMcpServerCommand();
       const result = await runCli([
@@ -451,7 +451,7 @@ describe("Tool Tests", () => {
         "--method",
         "prompts/get",
         "--prompt-name",
-        "simple-prompt",
+        "simple_prompt",
         "--prompt-args",
         "name=test",
         "count=5",
@@ -464,7 +464,7 @@ describe("Tool Tests", () => {
       expect(json.messages.length).toBeGreaterThan(0);
       expect(json.messages[0]).toHaveProperty("content");
       expect(json.messages[0].content).toHaveProperty("type", "text");
-      // test-mcp-server's simple-prompt returns standard message (ignoring args)
+      // test-mcp-server's simple_prompt returns standard message (ignoring args)
       expect(json.messages[0].content.text).toBe(
         "This is a simple prompt for testing purposes.",
       );
@@ -503,7 +503,7 @@ describe("Tool Tests", () => {
         "--method",
         "tools/call",
         "--tool-name",
-        "get-sum",
+        "get_sum",
         "--tool-arg",
         "a=10",
         "b=20",