mcp-linkedin-ads 1.0.0 → 1.0.2

package/dist/build-info.json

@@ -1 +1 @@
-{"sha":"c1e5873","builtAt":"2026-03-13T17:44:17.111Z"}
+{"sha":"574aa73","builtAt":"2026-03-13T18:23:01.642Z"}

package/dist/index.js

@@ -7,11 +7,12 @@ import { join, dirname } from "path";
 import { execSync } from "child_process";
 import { LinkedInAdsAuthError, classifyError, validateCredentials, } from "./errors.js";
 import { tools } from "./tools.js";
+import { withResilience, safeResponse, logger } from "./resilience.js";
 // Log build fingerprint at startup
 try {
     const __buildInfoDir = dirname(new URL(import.meta.url).pathname);
     const buildInfo = JSON.parse(readFileSync(join(__buildInfoDir, "build-info.json"), "utf-8"));
-    console.error(`[build] SHA: ${buildInfo.sha} (${buildInfo.builtAt})`);
+    logger.info({ sha: buildInfo.sha, builtAt: buildInfo.builtAt }, "Build fingerprint");
 }
 catch {
     // build-info.json not present (dev mode)
@@ -44,11 +45,11 @@ class LinkedInAdsManager {
         // Validate credentials at startup — fail fast
         const creds = validateCredentials();
         if (!creds.valid) {
-            const msg = `[STARTUP ERROR] Missing required credentials: ${creds.missing.join(", ")}. Check run-mcp.sh and Keychain entries.`;
-            console.error(msg);
+            const msg = `Missing required credentials: ${creds.missing.join(", ")}. Check run-mcp.sh and Keychain entries.`;
+            logger.error({ missing: creds.missing }, msg);
             throw new LinkedInAdsAuthError(msg);
         }
-        console.error("[startup] Credentials validated: token env vars present");
+        logger.info("Credentials validated: token env vars present");
         this.refreshToken = process.env.LINKEDIN_ADS_REFRESH_TOKEN || "";
         if (process.env.LINKEDIN_ADS_CLIENT_ID) {
             this.config.oauth.client_id = process.env.LINKEDIN_ADS_CLIENT_ID;
@@ -99,10 +100,10 @@ class LinkedInAdsManager {
             try {
                 execSync(`security delete-generic-password -a linkedin-ads-mcp -s LINKEDIN_ADS_REFRESH_TOKEN 2>/dev/null; ` +
                     `security add-generic-password -a linkedin-ads-mcp -s LINKEDIN_ADS_REFRESH_TOKEN -w "${data.refresh_token}"`);
-                console.error("[token] Rotated refresh token persisted to Keychain");
+                logger.info("Rotated refresh token persisted to Keychain");
             }
             catch (err) {
-                console.error("[token] WARNING: Failed to persist rotated refresh token to Keychain:", err);
+                logger.warn({ err }, "Failed to persist rotated refresh token to Keychain");
             }
         }
         return this.accessToken;
@@ -114,38 +115,44 @@ class LinkedInAdsManager {
             const qs = new URLSearchParams(params).toString();
             url += (url.includes("?") ? "&" : "?") + qs;
         }
-        const resp = await fetch(url, {
-            method: "GET",
-            headers: {
-                "Authorization": `Bearer ${token}`,
-                "LinkedIn-Version": this.config.api.version,
-                "X-Restli-Protocol-Version": "2.0.0",
-            },
-        });
-        if (!resp.ok) {
-            const text = await resp.text();
-            const error = Object.assign(new Error(`LinkedIn API error: ${resp.status} ${text}`), { status: resp.status });
-            throw classifyError(error);
-        }
-        return await resp.json();
+        return withResilience(async () => {
+            const resp = await fetch(url, {
+                method: "GET",
+                headers: {
+                    "Authorization": `Bearer ${token}`,
+                    "LinkedIn-Version": this.config.api.version,
+                    "X-Restli-Protocol-Version": "2.0.0",
+                },
+                signal: AbortSignal.timeout(30_000),
+            });
+            if (!resp.ok) {
+                const text = await resp.text();
+                const error = Object.assign(new Error(`LinkedIn API error: ${resp.status} ${text}`), { status: resp.status });
+                throw classifyError(error);
+            }
+            return await resp.json();
+        }, `apiGet:${path}`);
     }
     // Raw GET with pre-built URL (for complex Rest.li query params)
     async apiGetRaw(fullUrl) {
         const token = await this.getAccessToken();
-        const resp = await fetch(fullUrl, {
-            method: "GET",
-            headers: {
-                "Authorization": `Bearer ${token}`,
-                "LinkedIn-Version": this.config.api.version,
-                "X-Restli-Protocol-Version": "2.0.0",
-            },
-        });
-        if (!resp.ok) {
-            const text = await resp.text();
-            const error = Object.assign(new Error(`LinkedIn API error: ${resp.status} ${text}`), { status: resp.status });
-            throw classifyError(error);
-        }
-        return await resp.json();
+        return withResilience(async () => {
+            const resp = await fetch(fullUrl, {
+                method: "GET",
+                headers: {
+                    "Authorization": `Bearer ${token}`,
+                    "LinkedIn-Version": this.config.api.version,
+                    "X-Restli-Protocol-Version": "2.0.0",
+                },
+                signal: AbortSignal.timeout(30_000),
+            });
+            if (!resp.ok) {
+                const text = await resp.text();
+                const error = Object.assign(new Error(`LinkedIn API error: ${resp.status} ${text}`), { status: resp.status });
+                throw classifyError(error);
+            }
+            return await resp.json();
+        }, `apiGetRaw:${fullUrl.split("?")[0]}`);
     }
     // ============================================
     // ACCOUNT MANAGEMENT
@@ -295,14 +302,14 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             case "linkedin_ads_list_accounts": {
                 const result = await adsManager.listAccounts();
                 return {
-                    content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
+                    content: [{ type: "text", text: JSON.stringify(safeResponse(result, "list_accounts"), null, 2) }],
                 };
             }
             case "linkedin_ads_list_campaign_groups": {
                 const accountId = resolveAccountId(args?.account_id);
                 const result = await adsManager.listCampaignGroups(accountId);
                 return {
-                    content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
+                    content: [{ type: "text", text: JSON.stringify(safeResponse(result, "list_campaign_groups"), null, 2) }],
                 };
             }
             case "linkedin_ads_list_campaigns": {
@@ -312,7 +319,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                     campaignGroupId: args?.campaign_group_id,
                 });
                 return {
-                    content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
+                    content: [{ type: "text", text: JSON.stringify(safeResponse(result, "list_campaigns"), null, 2) }],
                 };
             }
             case "linkedin_ads_campaign_performance": {
@@ -323,7 +330,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                     timeGranularity: args?.time_granularity,
                 });
                 return {
-                    content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
+                    content: [{ type: "text", text: JSON.stringify(safeResponse(result, "campaign_performance"), null, 2) }],
                 };
             }
             case "linkedin_ads_account_performance": {
@@ -334,7 +341,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                     timeGranularity: args?.time_granularity,
                 });
                 return {
-                    content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
+                    content: [{ type: "text", text: JSON.stringify(safeResponse(result, "account_performance"), null, 2) }],
                 };
             }
             case "linkedin_ads_analytics": {
@@ -350,7 +357,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                     campaignGroupIds: args?.campaign_group_ids,
                 });
                 return {
-                    content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
+                    content: [{ type: "text", text: JSON.stringify(safeResponse(result, "analytics"), null, 2) }],
                 };
             }
             default:
@@ -381,6 +388,6 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
 async function main() {
     const transport = new StdioServerTransport();
     await server.connect(transport);
-    console.error("MCP LinkedIn Ads server running");
+    logger.info("MCP LinkedIn Ads server running");
 }
-main().catch(console.error);
+main().catch((err) => logger.error({ err }, "Server failed to start"));

package/dist/resilience.d.ts

@@ -0,0 +1,3 @@
+export declare const logger: import("pino").Logger<never>;
+export declare function safeResponse<T>(data: T, context: string): T;
+export declare function withResilience<T>(fn: () => Promise<T>, operationName: string): Promise<T>;

package/dist/resilience.js

@@ -0,0 +1,80 @@
+import { retry, circuitBreaker, wrap, handleAll, timeout, TimeoutStrategy, ExponentialBackoff, ConsecutiveBreaker, } from "cockatiel";
+import pino from "pino";
+// ============================================
+// LOGGER
+// ============================================
+export const logger = pino({
+    level: process.env.LOG_LEVEL || "info",
+    ...(process.env.NODE_ENV !== "test" && {
+        transport: {
+            target: "pino-pretty",
+            options: {
+                colorize: true,
+                singleLine: true,
+                translateTime: "SYS:standard",
+            },
+        },
+    }),
+});
+// ============================================
+// SAFE RESPONSE (Response Size Limiting)
+// ============================================
+const MAX_RESPONSE_SIZE = 200_000; // 200KB
+export function safeResponse(data, context) {
+    const jsonStr = JSON.stringify(data);
+    const sizeBytes = Buffer.byteLength(jsonStr, "utf-8");
+    if (sizeBytes > MAX_RESPONSE_SIZE) {
+        logger.warn({ sizeBytes, maxSize: MAX_RESPONSE_SIZE, context }, `Response exceeds size limit, truncating`);
+        // If it's an array, truncate
+        if (Array.isArray(data)) {
+            const truncated = data.slice(0, Math.max(1, Math.floor(data.length * 0.5)));
+            return truncated;
+        }
+        // If it's an object with items/results, truncate those
+        if (typeof data === "object" && data !== null) {
+            const obj = data;
+            for (const key of ["items", "results", "data", "rows", "elements"]) {
+                if (Array.isArray(obj[key])) {
+                    obj[key] = obj[key].slice(0, Math.max(1, Math.floor(obj[key].length * 0.5)));
+                    return obj;
+                }
+            }
+        }
+    }
+    return data;
+}
+// ============================================
+// RETRY + CIRCUIT BREAKER + TIMEOUT
+// ============================================
+const backoff = new ExponentialBackoff({
+    initialDelay: 100,
+    maxDelay: 5_000,
+});
+// Individual policies
+const retryPolicy = retry(handleAll, {
+    maxAttempts: 3,
+    backoff,
+});
+const circuitBreakerPolicy = circuitBreaker(handleAll, {
+    halfOpenAfter: 60_000, // 60s to attempt recovery
+    breaker: new ConsecutiveBreaker(5), // Open after 5 consecutive failures
+});
+const timeoutPolicy = timeout(30_000, TimeoutStrategy.Cooperative);
+// Combine policies: timeout -> circuit breaker -> retry
+const policy = wrap(timeoutPolicy, circuitBreakerPolicy, retryPolicy);
+// ============================================
+// WRAPPED API CALL WITH LOGGING
+// ============================================
+export async function withResilience(fn, operationName) {
+    try {
+        logger.debug({ operation: operationName }, "Starting API call");
+        const result = await policy.execute(() => fn());
+        logger.debug({ operation: operationName }, "API call succeeded");
+        return result;
+    }
+    catch (err) {
+        const error = err instanceof Error ? err : new Error(String(err));
+        logger.error({ operation: operationName, error: error.message, stack: error.stack }, "API call failed after retries");
+        throw error;
+    }
+}

package/package.json

@@ -1,6 +1,7 @@
 {
   "name": "mcp-linkedin-ads",
-  "version": "1.0.0",
+  "mcpName": "io.github.mharnett/linkedin-ads",
+  "version": "1.0.2",
   "description": "MCP server for LinkedIn Campaign Manager API with full campaign, ad group, creative, and targeting support. Production-proven with 65+ campaigns under active management.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -37,17 +38,19 @@
   "license": "MIT",
   "repository": {
     "type": "git",
-    "url": "https://github.com/drak-marketing/mcp-linkedin-ads"
+    "url": "https://github.com/mharnett/mcp-linkedin-ads"
   },
   "bugs": {
-    "url": "https://github.com/drak-marketing/mcp-linkedin-ads/issues"
+    "url": "https://github.com/mharnett/mcp-linkedin-ads/issues"
   },
-  "homepage": "https://github.com/drak-marketing/mcp-linkedin-ads#readme",
+  "homepage": "https://github.com/mharnett/mcp-linkedin-ads#readme",
   "engines": {
     "node": ">=18.0.0"
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^0.5.0",
+    "cockatiel": "^3.2.1",
+    "pino": "^8.21.0",
     "zod": "^3.22.4"
   },
   "devDependencies": {