fix(mcp): remove validate_address, add list_carrier_connections, clean up tests

Remove validate_address references (unsupported feature) from README, tests,
and client. Add list_carrier_connections tool to carriers module. Remove
console.log statements from integration tests.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: danh91

packages/mcp/README.md

@@ -6,7 +6,7 @@ Connect any AI agent (Claude, ChatGPT, Cursor, VS Code Copilot) to 50+ shipping
 
 ## Features
 
-- **11 shipping tools** -- rates, labels, tracking, address validation, pickups, manifests, orders, and more
+- **10 shipping tools** -- rates, labels, tracking, pickups, manifests, orders, and more
 - **MCP Resources** for a live carrier capability catalog -- no other shipping MCP does this
 - **stdio + Streamable HTTP** transport (remote-ready)
 - **Works with any Karrio instance** -- cloud or self-hosted
@@ -114,7 +114,6 @@ The server exposes:
 | `list_shipments` | List shipments with status, carrier, and date filters | No |
 | `cancel_shipment` | Cancel a shipment and void its label | Yes |
 | `track_package` | Track a package by tracking number with full event history | No |
-| `validate_address` | Validate and correct a shipping address | No |
 | `list_carriers` | List connected carriers and their capabilities | No |
 | `schedule_pickup` | Schedule a carrier pickup for one or more shipments | Yes |
 | `create_manifest` | Create an end-of-day manifest (SCAN form) | Yes |
@@ -128,8 +127,6 @@ The server exposes:
 
 **`track_package`** -- Real-time package tracking. Provide a tracking number and optionally a carrier name (auto-detection is supported). Returns status, estimated delivery, and a chronological event history with locations.
 
-**`validate_address`** -- Verify addresses before shipping. Returns the validated/corrected address with any corrections applied by the carrier.
-
 **`list_carriers`** -- Discover which carriers are connected to your Karrio instance and what capabilities each supports (tracking, rating, shipping, pickup).
 
 **`schedule_pickup`** -- Book a carrier pickup at a specified address with date, ready time, and closing time windows. Optionally include specific shipment IDs.
@@ -197,8 +194,7 @@ src/
     rates.ts            get_shipping_rates
     shipments.ts        create_shipment, get_shipment, list_shipments, cancel_shipment
     tracking.ts         track_package
-    addresses.ts        validate_address
-    carriers.ts         list_carriers
+    carriers.ts         list_carriers, list_carrier_connections
     pickups.ts          schedule_pickup
     manifests.ts        create_manifest
     orders.ts           list_orders

packages/mcp/src/client.ts

@@ -96,16 +96,16 @@ export class KarrioClient {
     return this.request("GET", "/v1/trackers", { params });
   }
 
-  // Address validation
-  async validateAddress(payload: Record<string, unknown>): Promise<any> {
-    return this.request("POST", "/v1/addresses/validate", { body: payload });
-  }
-
   // Carriers
   async listCarriers(params?: Record<string, string>): Promise<any> {
     return this.request("GET", "/v1/carriers", { params });
   }
 
+  // Carrier connections (configured accounts)
+  async listConnections(params?: Record<string, string>): Promise<any> {
+    return this.request("GET", "/v1/connections", { params });
+  }
+
   // Reference data
   async getReferences(): Promise<any> {
     return this.request("GET", "/v1/references");

packages/mcp/src/tools/carriers.ts

@@ -8,7 +8,7 @@ export function registerCarrierTools(
 ): void {
   server.tool(
     "list_carriers",
-    "List all carrier accounts connected to your Karrio instance with their capabilities (tracking, rating, shipping, pickup). Use this to discover available carriers and their service codes before creating shipments or fetching rates.",
+    "List all supported carrier integrations in the Karrio catalog with their capabilities. Use this to discover what carriers can be connected. For carriers already connected to your account, use list_carrier_connections instead.",
     {
       carrier_name: z
         .string()
@@ -47,10 +47,8 @@ export function registerCarrierTools(
           (carrier: any) => ({
             id: carrier.id,
             carrier_name: carrier.carrier_name,
-            carrier_id: carrier.carrier_id,
-            active: carrier.active ?? carrier.is_active ?? null,
+            display_name: carrier.display_name ?? null,
             capabilities: carrier.capabilities ?? [],
-            test_mode: carrier.test_mode ?? null,
           }),
         );
 
@@ -79,4 +77,79 @@ export function registerCarrierTools(
       }
     },
   );
+
+  server.tool(
+    "list_carrier_connections",
+    "List carrier accounts connected to your Karrio instance. These are the carriers configured with credentials that you can use for rating, shipping, and tracking. Returns carrier_id, display_name, capabilities, and connection status.",
+    {
+      carrier_name: z
+        .string()
+        .optional()
+        .describe("Filter by carrier name (e.g., 'fedex', 'ups')"),
+      limit: z
+        .number()
+        .int()
+        .default(20)
+        .describe("Maximum results"),
+      offset: z
+        .number()
+        .int()
+        .default(0)
+        .describe("Pagination offset"),
+    },
+    {
+      readOnlyHint: true,
+      destructiveHint: false,
+      idempotentHint: true,
+      openWorldHint: false,
+    },
+    async (params) => {
+      try {
+        const queryParams: Record<string, string> = {
+          limit: String(params.limit),
+          offset: String(params.offset),
+        };
+        if (params.carrier_name) {
+          queryParams.carrier_name = params.carrier_name;
+        }
+
+        const response = await client.listConnections(queryParams);
+
+        const connections = (response.results ?? []).map(
+          (conn: any) => ({
+            id: conn.id,
+            carrier_name: conn.carrier_name,
+            carrier_id: conn.carrier_id,
+            display_name: conn.display_name ?? null,
+            capabilities: conn.capabilities ?? [],
+            active: conn.active ?? conn.is_active ?? null,
+            test_mode: conn.test_mode ?? null,
+          }),
+        );
+
+        const result = {
+          connections,
+          count: response.count ?? connections.length,
+          limit: params.limit,
+          offset: params.offset,
+        };
+
+        return {
+          content: [
+            { type: "text", text: JSON.stringify(result, null, 2) },
+          ],
+        };
+      } catch (error: any) {
+        return {
+          content: [
+            {
+              type: "text",
+              text: `Error listing carrier connections: ${error.message}`,
+            },
+          ],
+          isError: true,
+        };
+      }
+    },
+  );
 }

packages/mcp/tests/integration/server.test.ts

@@ -8,8 +8,6 @@ describe("MCP Server", () => {
       apiKey: "test_key",
     });
 
-    // Server should be created without errors
-    console.log(server);
     expect(server).toBeDefined();
   });
 
@@ -19,7 +17,7 @@ describe("MCP Server", () => {
       apiKey: "custom_key",
     });
 
-    console.log(server);
+
     expect(server).toBeDefined();
   });
 
@@ -33,7 +31,7 @@ describe("MCP Server", () => {
       apiKey: "test_key",
     });
 
-    console.log(server);
+
     expect(server).toBeInstanceOf(McpServer);
   });
 
@@ -48,8 +46,6 @@ describe("MCP Server", () => {
       apiKey: "key_2",
     });
 
-    console.log(server1);
-    console.log(server2);
     expect(server1).not.toBe(server2);
   });
 });

packages/mcp/tests/unit/client.test.ts

@@ -263,24 +263,6 @@ describe("KarrioClient", () => {
     );
   });
 
-  it("calls correct URL for validateAddress", async () => {
-    global.fetch = vi.fn().mockResolvedValue({
-      ok: true,
-      json: () => Promise.resolve({ valid: true }),
-    });
-
-    const payload = { address: { postal_code: "10001", country_code: "US" } };
-    const result = await client.validateAddress(payload);
-
-
-    expect(fetch).toHaveBeenCalledWith(
-      "https://api.karrio.io/v1/addresses/validate",
-      expect.objectContaining({
-        method: "POST",
-        body: JSON.stringify(payload),
-      }),
-    );
-  });
 
   it("calls correct URL for getReferences", async () => {
     global.fetch = vi.fn().mockResolvedValue({