Merge pull request #52 from doitintl/hotfix/talc/anomalies-filters-value

feat(mcp): remove anomaly filter

Author: taltultc

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@doitintl/doit-mcp-server",
-  "version": "0.1.29",
+  "version": "0.1.31",
   "description": "DoiT official MCP Server",
   "keywords": [
     "doit",

src/tools/__tests__/anomalies.test.ts

@@ -116,7 +116,7 @@ Top SKUs:
     });
 
     it("should call makeDoitRequest with correct parameters and return success response", async () => {
-      const mockArgs = { filter: "platform:gcp", pageToken: "next-page" };
+      const mockArgs = { pageToken: "next-page" };
       const mockApiResponse = {
         rowCount: 1,
         anomalies: [
@@ -144,14 +144,14 @@ Top SKUs:
       const response = await handleAnomaliesRequest(mockArgs, mockToken);
 
       expect(makeDoitRequest).toHaveBeenCalledWith(
-        "https://api.doit.com/anomalies/v1?filter=platform%3Agcp&pageToken=next-page&maxResults=24",
+        "https://api.doit.com/anomalies/v1?pageToken=next-page&maxResults=32",
         mockToken,
         {
           method: "GET",
         }
       );
       expect(createSuccessResponse).toHaveBeenCalledWith(
-        expect.stringContaining("Found 1 anomalies (filtered by: platform:gcp)")
+        expect.stringContaining("Found 1 anomalies")
       );
       expect(response).toEqual({
         content: [
@@ -161,7 +161,7 @@ Top SKUs:
     });
 
     it("should handle no anomalies found", async () => {
-      const mockArgs = { filter: "platform:aws" };
+      const mockArgs = {};
       const mockApiResponse = {
         rowCount: 0,
         anomalies: [],
@@ -172,7 +172,7 @@ Top SKUs:
       const response = await handleAnomaliesRequest(mockArgs, mockToken);
 
       expect(makeDoitRequest).toHaveBeenCalledWith(
-        "https://api.doit.com/anomalies/v1?filter=platform%3Aaws&maxResults=24",
+        "https://api.doit.com/anomalies/v1?maxResults=32",
         mockToken,
         {
           method: "GET",
@@ -191,7 +191,7 @@ Top SKUs:
       const response = await handleAnomaliesRequest(mockArgs, mockToken);
 
       expect(makeDoitRequest).toHaveBeenCalledWith(
-        "https://api.doit.com/anomalies/v1?maxResults=24",
+        "https://api.doit.com/anomalies/v1?maxResults=32",
         mockToken,
         { method: "GET" }
       );
@@ -203,22 +203,6 @@ Top SKUs:
       });
     });
 
-    it("should handle ZodError for invalid arguments", async () => {
-      const mockArgs = { filter: 123 }; // Invalid filter type
-      const response = await handleAnomaliesRequest(mockArgs, mockToken);
-
-      expect(formatZodError).toHaveBeenCalled();
-      expect(createErrorResponse).toHaveBeenCalled();
-      expect(response).toEqual({
-        content: [
-          {
-            type: "text",
-            text: expect.stringContaining("Formatted Zod Error:"),
-          },
-        ],
-      });
-    });
-
     it("should handle general errors", async () => {
       const mockArgs = {};
       (makeDoitRequest as vi.Mock).mockRejectedValue(
@@ -237,6 +221,23 @@ Top SKUs:
         ],
       });
     });
+
+    it("should handle ZodError for invalid arguments", async () => {
+      const mockArgs = { pageToken: 123 }; // Invalid argument type for Zod
+      const response = await handleAnomaliesRequest(mockArgs, mockToken);
+
+      expect(createErrorResponse).toHaveBeenCalledWith(
+        expect.stringContaining("Formatted Zod Error:")
+      );
+      expect(response).toEqual({
+        content: [
+          {
+            type: "text",
+            text: expect.stringContaining("Formatted Zod Error:"),
+          },
+        ],
+      });
+    });
   });
 
   describe("handleAnomalyRequest", () => {

src/tools/anomalies.ts

@@ -10,12 +10,6 @@ import {
 
 // Schema definitions
 export const AnomaliesArgumentsSchema = z.object({
-  filter: z
-    .string()
-    .optional()
-    .describe(
-      "Filter string in format 'key:value|key:value'. Multiple values for same key are treated as OR, different keys as AND."
-    ),
   pageToken: z
     .string()
     .optional()
@@ -65,11 +59,6 @@ export const anomaliesTool = {
   inputSchema: {
     type: "object",
     properties: {
-      filter: {
-        type: "string",
-        description:
-          "Filter string in format 'key:value|key:value'. Multiple values for same key are treated as OR, different keys as AND.",
-      },
       pageToken: {
         type: "string",
         description:
@@ -129,17 +118,15 @@ export function formatAnomaly(anomaly: Anomaly): string {
 // Handle anomalies request
 export async function handleAnomaliesRequest(args: any, token: string) {
   try {
-    const { filter, pageToken } = AnomaliesArgumentsSchema.parse(args);
+    const { pageToken } = AnomaliesArgumentsSchema.parse(args);
 
     // Create API URL with query parameters
     const params = new URLSearchParams();
-    if (filter && filter.length > 1) {
-      params.append("filter", filter);
-    }
+
     if (pageToken && pageToken.length > 1) {
       params.append("pageToken", pageToken);
     }
-    params.append("maxResults", "24");
+    params.append("maxResults", "32");
 
     let anomaliesUrl = `${DOIT_API_BASE}/anomalies/v1`;
 
@@ -191,9 +178,7 @@ export async function handleAnomaliesRequest(args: any, token: string) {
       }));
 
       let anomaliesText = `Found ${rowCount} anomalies`;
-      if (filter) {
-        anomaliesText += ` (filtered by: ${filter})`;
-      }
+
       anomaliesText += `:\n\n${formattedAnomalies
         .map((a) => JSON.stringify(a, null, 2))
         .join("\n")}\n\n${