feat(tests): enhance MCP tool validation with combined name length check

- Added a new constant for the Cursor IDE prefix and a test to ensure the combined length of the tool name and prefix does not exceed 60 characters.
- Updated the tool registration name for clarity and consistency.

Author: agoldis

mcp-server/src/server.test.ts

@@ -30,6 +30,10 @@ const TOOL_NAME_MAX_LENGTH = 64;
 const TOOL_NAME_PATTERN = /^[a-zA-Z0-9_\-./]+$/;
 // No hard spec limit; 1024 is the practical ceiling across major LLM providers
 const TOOL_DESCRIPTION_MAX_LENGTH = 1024;
+// Cursor IDE prefixes tool names with "extension-<server>:" when displaying them.
+// Combined length must stay ≤ 60 to avoid filtering warnings.
+const CURSOR_SERVER_PREFIX = "extension-currents:";
+const CURSOR_COMBINED_MAX_LENGTH = 60;
 
 describe("MCP tool best practices", () => {
   it("has at least one registered tool", () => {
@@ -59,6 +63,14 @@ describe("MCP tool best practices", () => {
       expect(name.length).toBeGreaterThan(0);
     });
 
+    it(`combined Cursor name length ≤ ${CURSOR_COMBINED_MAX_LENGTH}`, () => {
+      const combined = `${CURSOR_SERVER_PREFIX}${name}`;
+      expect(
+        combined.length,
+        `"${combined}" is ${combined.length} chars`
+      ).toBeLessThanOrEqual(CURSOR_COMBINED_MAX_LENGTH);
+    });
+
     // ── description constraints ─────────────────────────────────
     it("description is not empty", () => {
       expect(description.length).toBeGreaterThan(0);

mcp-server/src/server.ts

@@ -141,7 +141,7 @@ server.registerTool(
 );
 
 server.registerTool(
-  "currents-get-affected-test-executions-by-action",
+  "currents-get-affected-executions",
   {
     description:
       "List test executions where a specific action/rule was applied, within a date range. Uses cursor-based pagination. Requires actionId, date_start, and date_end.",