mcp-reddit-ads 1.0.4 → 1.0.6

package/README.md

@@ -20,7 +20,7 @@ npm install mcp-reddit-ads
 Or clone the repository:
 
 ```bash
-git clone https://github.com/drak-marketing/mcp-reddit-ads.git
+git clone https://github.com/mharnett/mcp-reddit-ads.git
 cd mcp-reddit-ads
 npm install
 npm run build
@@ -98,54 +98,54 @@ Environment variables take precedence over config file values.
 
 | Tool | Description |
 |---|---|
-| `get_client_context` | Get account info and verify API connectivity |
-| `get_accounts` | List all ad accounts accessible to the authenticated user |
+| `reddit_ads_get_client_context` | Get account info and verify API connectivity |
+| `reddit_ads_get_accounts` | List all ad accounts accessible to the authenticated user |
 
 ### Read
 
 | Tool | Description |
 |---|---|
-| `get_campaigns` | List campaigns with optional status filter |
-| `get_ad_groups` | List ad groups for a campaign |
-| `get_ads` | List ads for an ad group |
-| `get_performance_report` | Aggregated performance metrics for campaigns/ad groups/ads |
-| `get_daily_performance` | Day-by-day performance breakdown |
+| `reddit_ads_get_campaigns` | List campaigns with optional status filter |
+| `reddit_ads_get_ad_groups` | List ad groups for a campaign |
+| `reddit_ads_get_ads` | List ads for an ad group |
+| `reddit_ads_get_performance_report` | Aggregated performance metrics for campaigns/ad groups/ads |
+| `reddit_ads_get_daily_performance` | Day-by-day performance breakdown |
 
 ### Write: Campaigns
 
 | Tool | Description |
 |---|---|
-| `create_campaign` | Create a new campaign (PAUSED by default) |
-| `update_campaign` | Update campaign name, budget, objective, or status |
+| `reddit_ads_create_campaign` | Create a new campaign (PAUSED by default) |
+| `reddit_ads_update_campaign` | Update campaign name, budget, objective, or status |
 
 ### Write: Ad Groups
 
 | Tool | Description |
 |---|---|
-| `create_ad_group` | Create a new ad group with targeting (PAUSED by default) |
-| `update_ad_group` | Update ad group bid, targeting, or status |
+| `reddit_ads_create_ad_group` | Create a new ad group with targeting (PAUSED by default) |
+| `reddit_ads_update_ad_group` | Update ad group bid, targeting, or status |
 
 ### Write: Ads
 
 | Tool | Description |
 |---|---|
-| `create_ad` | Create a new ad with headline, body, URL, and media (PAUSED by default) |
-| `update_ad` | Update ad creative or status |
+| `reddit_ads_create_ad` | Create a new ad with headline, body, URL, and media (PAUSED by default) |
+| `reddit_ads_update_ad` | Update ad creative or status |
 
 ### Bulk Operations
 
 | Tool | Description |
 |---|---|
-| `pause_items` | Pause multiple campaigns, ad groups, or ads at once |
-| `enable_items` | Enable multiple campaigns, ad groups, or ads at once |
+| `reddit_ads_pause_items` | Pause multiple campaigns, ad groups, or ads at once |
+| `reddit_ads_enable_items` | Enable multiple campaigns, ad groups, or ads at once |
 
 ### Targeting
 
 | Tool | Description |
 |---|---|
-| `search_subreddits` | Search for subreddits by keyword for targeting |
-| `get_interest_categories` | List available interest categories for targeting |
-| `search_geo_targets` | Search for geographic targeting options (countries, regions, metros) |
+| `reddit_ads_search_subreddits` | Search for subreddits by keyword for targeting |
+| `reddit_ads_get_interest_categories` | List available interest categories for targeting |
+| `reddit_ads_search_geo_targets` | Search for geographic targeting options (countries, regions, metros) |
 
 ## Key Conventions
 

package/dist/build-info.json

@@ -1 +1 @@
-{"sha":"4534c7a","builtAt":"2026-04-09T19:37:19.842Z"}
+{"sha":"6453862","builtAt":"2026-04-09T21:28:34.019Z"}

package/dist/index.js

@@ -7,6 +7,8 @@ import { join, dirname } from "path";
 import { RedditAdsAuthError, RedditAdsRateLimitError, RedditAdsServiceError, classifyError, validateCredentials, } from "./errors.js";
 import { tools } from "./tools.js";
 import { withResilience, safeResponse, logger } from "./resilience.js";
+// CLI package info
+const __cliPkg = JSON.parse(readFileSync(join(dirname(new URL(import.meta.url).pathname), "..", "package.json"), "utf-8"));
 // Log build fingerprint at startup
 try {
     const __buildInfoDir = dirname(new URL(import.meta.url).pathname);
@@ -14,22 +16,21 @@ try {
     console.error(`[build] SHA: ${buildInfo.sha} (${buildInfo.builtAt})`);
 }
 catch {
-    // build-info.json not present (dev mode)
+    console.error(`[build] ${__cliPkg.name}@${__cliPkg.version} (dev mode)`);
 }
 // CLI flags
-const __cliPkg = JSON.parse(readFileSync(join(dirname(new URL(import.meta.url).pathname), "..", "package.json"), "utf-8"));
 if (process.argv.includes("--help") || process.argv.includes("-h")) {
-    console.log(`${__cliPkg.name} v${__cliPkg.version}\n`);
-    console.log(`Usage: ${__cliPkg.name} [options]\n`);
-    console.log("MCP server communicating via stdio. Configure in your .mcp.json.\n");
-    console.log("Options:");
-    console.log("  --help, -h       Show this help message");
-    console.log("  --version, -v    Show version number");
-    console.log(`\nDocumentation: https://github.com/mharnett/mcp-reddit-ads`);
+    console.error(`${__cliPkg.name} v${__cliPkg.version}\n`);
+    console.error(`Usage: ${__cliPkg.name} [options]\n`);
+    console.error("MCP server communicating via stdio. Configure in your .mcp.json.\n");
+    console.error("Options:");
+    console.error("  --help, -h       Show this help message");
+    console.error("  --version, -v    Show version number");
+    console.error(`\nDocumentation: https://github.com/mharnett/mcp-reddit-ads`);
     process.exit(0);
 }
 if (process.argv.includes("--version") || process.argv.includes("-v")) {
-    console.log(__cliPkg.version);
+    console.error(__cliPkg.version);
     process.exit(0);
 }
 function loadConfig() {
@@ -296,7 +297,7 @@ class RedditAdsManager {
 // ============================================
 const config = loadConfig();
 const adsManager = new RedditAdsManager(config);
-const server = new Server({ name: "mcp-reddit-ads", version: "1.0.0" }, { capabilities: { tools: {} } });
+const server = new Server({ name: __cliPkg.name, version: __cliPkg.version }, { capabilities: { tools: {} } });
 server.setRequestHandler(ListToolsRequestSchema, async () => ({ tools }));
 server.setRequestHandler(CallToolRequestSchema, async (request) => {
     const { name, arguments: args } = request.params;
@@ -533,4 +534,15 @@ async function main() {
     await server.connect(transport);
     console.error("[startup] MCP Reddit Ads server running");
 }
+process.on("SIGTERM", () => {
+    console.error("[shutdown] SIGTERM received, exiting");
+    process.exit(0);
+});
+process.on("SIGINT", () => {
+    console.error("[shutdown] SIGINT received, exiting");
+    process.exit(0);
+});
+process.on("SIGPIPE", () => {
+    // Client disconnected -- expected during shutdown
+});
 main().catch(console.error);

package/dist/resilience.js

@@ -1,4 +1,4 @@
-import { retry, circuitBreaker, wrap, handleAll, timeout, TimeoutStrategy, ExponentialBackoff, ConsecutiveBreaker, } from "cockatiel";
+import { retry, circuitBreaker, wrap, handleWhen, timeout, TimeoutStrategy, ExponentialBackoff, ConsecutiveBreaker, } from "cockatiel";
 import pino from "pino";
 // ============================================
 // LOGGER
@@ -12,34 +12,50 @@ export const logger = pino({
                 colorize: true,
                 singleLine: true,
                 translateTime: "SYS:standard",
+                destination: 2, // stderr -- stdout is reserved for MCP JSON-RPC
             },
         },
     }),
-});
+}, 
+// When no transport (test mode), write to stderr directly
+process.env.NODE_ENV === "test" ? pino.destination(2) : undefined);
 // ============================================
 // 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 (Array.isArray(data)) {
-            const truncated = data.slice(0, Math.max(1, Math.floor(data.length * 0.5)));
-            return truncated;
+    let current = data;
+    for (let pass = 0; pass < 10; pass++) {
+        const jsonStr = JSON.stringify(current);
+        const sizeBytes = Buffer.byteLength(jsonStr, "utf-8");
+        if (sizeBytes <= MAX_RESPONSE_SIZE)
+            return current;
+        logger.warn({ sizeBytes, maxSize: MAX_RESPONSE_SIZE, context, pass }, "Response exceeds size limit, truncating");
+        if (Array.isArray(current)) {
+            current = current.slice(0, Math.max(1, Math.floor(current.length * 0.5)));
+            continue;
         }
-        if (typeof data === "object" && data !== null) {
-            const obj = data;
-            for (const key of ["items", "results", "data", "rows"]) {
-                if (Array.isArray(obj[key])) {
+        if (typeof current === "object" && current !== null) {
+            const obj = current;
+            let truncated = false;
+            for (const key of ["items", "results", "data", "rows", "tags", "triggers", "variables"]) {
+                if (Array.isArray(obj[key]) && obj[key].length > 1) {
                     obj[key] = obj[key].slice(0, Math.max(1, Math.floor(obj[key].length * 0.5)));
-                    return obj;
+                    if ("count" in obj)
+                        obj.count = obj[key].length;
+                    if ("row_count" in obj)
+                        obj.row_count = obj[key].length;
+                    obj.truncated = true;
+                    truncated = true;
+                    break;
                 }
             }
+            if (truncated)
+                continue;
         }
+        break;
     }
-    return data;
+    return current;
 }
 // ============================================
 // RETRY + CIRCUIT BREAKER + TIMEOUT
@@ -48,11 +64,24 @@ const backoff = new ExponentialBackoff({
     initialDelay: 100,
     maxDelay: 5_000,
 });
-const retryPolicy = retry(handleAll, {
+const isTransient = handleWhen((err) => {
+    const msg = (err?.message || "").toLowerCase();
+    const code = err?.code || err?.status;
+    if (code === 401 || code === 403 || code === 7 || code === 16)
+        return false;
+    if (msg.includes("unauthenticated") || msg.includes("permission_denied") || msg.includes("invalid_grant"))
+        return false;
+    if (code === 429 || msg.includes("rate"))
+        return true;
+    if (code >= 400 && code < 500)
+        return false;
+    return true;
+});
+const retryPolicy = retry(isTransient, {
     maxAttempts: 3,
     backoff,
 });
-const circuitBreakerPolicy = circuitBreaker(handleAll, {
+const circuitBreakerPolicy = circuitBreaker(isTransient, {
     halfOpenAfter: 60_000,
     breaker: new ConsecutiveBreaker(5),
 });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "mcp-reddit-ads",
   "mcpName": "io.github.mharnett/reddit-ads",
-  "version": "1.0.4",
+  "version": "1.0.6",
   "description": "MCP server for Reddit Ads API v3 with campaign, ad group, ad management, and performance reporting.",
   "main": "dist/index.js",
   "bin": {