mcp-reddit-ads 1.1.1 → 1.1.2

package/dist/build-info.json

@@ -1 +1 @@
-{"sha":"58320cf","builtAt":"2026-04-18T18:09:07.962Z"}
+{"sha":"19917ed","builtAt":"2026-07-09T14:42:15.009Z"}

package/dist/index.js

@@ -4,17 +4,18 @@ import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"
 import { CallToolRequestSchema, ListToolsRequestSchema, } from "@modelcontextprotocol/sdk/types.js";
 import { readFileSync, existsSync } from "fs";
 import { join, dirname } from "path";
+import { fileURLToPath } from "url";
 import { RedditAdsAuthError, RedditAdsRateLimitError, RedditAdsServiceError, classifyError, validateCredentials, } from "./errors.js";
 import { tools } from "./tools.js";
 import { withResilience, safeResponse, logger } from "./resilience.js";
 import { filterTools, assertWriteAllowed, isWriteEnabled } from "./writeGate.js";
-import { checkForUpdate } from "./updateNotifier.js";
+import { checkForUpdate } from "mcp-updatenotifier";
 import v8 from "v8";
 // CLI package info
-const __cliPkg = JSON.parse(readFileSync(join(dirname(new URL(import.meta.url).pathname), "..", "package.json"), "utf-8"));
+const __cliPkg = JSON.parse(readFileSync(join(dirname(fileURLToPath(import.meta.url)), "..", "package.json"), "utf-8"));
 // Log build fingerprint at startup
 try {
-    const __buildInfoDir = dirname(new URL(import.meta.url).pathname);
+    const __buildInfoDir = dirname(fileURLToPath(import.meta.url));
     const buildInfo = JSON.parse(readFileSync(join(__buildInfoDir, "build-info.json"), "utf-8"));
     console.error(`[build] SHA: ${buildInfo.sha} (${buildInfo.builtAt})`);
 }
@@ -64,7 +65,7 @@ if (heapLimit < 256 * 1024 * 1024) {
 const envTrimmed = (key) => (process.env[key] || "").trim().replace(/^["']|["']$/g, "");
 function loadConfig() {
     // Try config.json first
-    const configPath = join(dirname(new URL(import.meta.url).pathname), "..", "config.json");
+    const configPath = join(dirname(fileURLToPath(import.meta.url)), "..", "config.json");
     let config;
     if (existsSync(configPath)) {
         config = JSON.parse(readFileSync(configPath, "utf-8"));

package/dist/writeGate.js

@@ -1,3 +1,4 @@
+import { createWriteGate } from "mcp-write-gate";
 /**
  * Tools that mutate Reddit Ads state. These are hidden from the tool list
  * and refused at call time unless REDDIT_ADS_MCP_WRITE=true.
@@ -15,17 +16,18 @@ export const WRITE_TOOLS = new Set([
     "reddit_ads_pause_items",
     "reddit_ads_enable_items",
 ]);
+const gate = createWriteGate({
+    writeTools: WRITE_TOOLS,
+    envPrefix: "REDDIT_ADS",
+});
 export function isWriteTool(name) {
-    return WRITE_TOOLS.has(name);
+    return gate.isWriteTool(name);
 }
 export function isWriteEnabled(env = process.env) {
-    const v = (env.REDDIT_ADS_MCP_WRITE || "").trim().toLowerCase();
-    return v === "true" || v === "1" || v === "yes";
+    return gate.isWriteEnabled(env);
 }
 export function filterTools(allTools, env = process.env) {
-    if (isWriteEnabled(env))
-        return [...allTools];
-    return allTools.filter((t) => !WRITE_TOOLS.has(t.name));
+    return gate.filterTools(allTools, env);
 }
 export const WRITE_DISABLED_MESSAGE = "Write operations are disabled. Set REDDIT_ADS_MCP_WRITE=true in the MCP server environment to enable mutating tools (create/update/pause/enable).";
 /**
@@ -33,9 +35,10 @@ export const WRITE_DISABLED_MESSAGE = "Write operations are disabled. Set REDDIT
  * Throws a clear Error if the tool mutates state and writes are disabled.
  */
 export function assertWriteAllowed(toolName, env = process.env) {
-    if (!isWriteTool(toolName))
-        return;
-    if (isWriteEnabled(env))
-        return;
-    throw new Error(`Tool "${toolName}" is a write operation. ${WRITE_DISABLED_MESSAGE}`);
+    try {
+        gate.assertWriteAllowed(toolName, env);
+    }
+    catch (e) {
+        throw new Error(`Tool "${toolName}" is a write operation. ${WRITE_DISABLED_MESSAGE}`);
+    }
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "mcp-reddit-ads",
   "mcpName": "io.github.mharnett/reddit-ads",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "MCP server for Reddit Ads API v3 with campaign, ad group, ad management, and performance reporting.",
   "main": "dist/index.js",
   "bin": {
@@ -25,7 +25,8 @@
     "build": "tsc && node -e \"fs=require('fs');cp=require('child_process');sha=cp.execSync('git rev-parse --short HEAD 2>/dev/null||echo unknown').toString().trim();fs.writeFileSync('dist/build-info.json',JSON.stringify({sha,builtAt:new Date().toISOString()}))\"",
     "start": "node dist/index.js",
     "dev": "tsx src/index.ts",
-    "test": "vitest run"
+    "test": "vitest run",
+    "smoke": "scripts/healthcheck.sh"
   },
   "author": "drak-marketing",
   "license": "MIT",
@@ -35,6 +36,8 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^0.5.0",
     "cockatiel": "^3.2.1",
+    "mcp-updatenotifier": "^1.0.0",
+    "mcp-write-gate": "^1.0.0",
     "pino": "^8.21.0",
     "pino-pretty": "^13.1.3"
   },
@@ -44,5 +47,9 @@
     "tsx": "^4.7.0",
     "typescript": "^5.3.0",
     "vitest": "^4.0.18"
+  },
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/mharnett/mcp-reddit-ads.git"
   }
 }

package/dist/updateNotifier.d.ts

@@ -1,7 +0,0 @@
-export type FetchLatestVersion = () => Promise<string>;
-export interface UpdateNotifierDeps {
-    fetchLatestVersion?: FetchLatestVersion;
-    log?: (msg: string) => void;
-    env?: NodeJS.ProcessEnv;
-}
-export declare function checkForUpdate(pkgName: string, currentVersion: string, deps?: UpdateNotifierDeps): Promise<void>;

package/dist/updateNotifier.js

@@ -1,59 +0,0 @@
-// Fire-and-forget npm registry check at startup. Logs to stderr when a
-// newer version is available. stdout is reserved for MCP JSON-RPC, so the
-// message never goes there. Silent on network error, timeout, or when the
-// installed version is equal to or ahead of the registry latest.
-//
-// Opt out by setting MCP_DISABLE_UPDATE_CHECK=1 (CI, offline, air-gapped).
-// Also skipped when NODE_ENV=test to keep vitest runs silent.
-export async function checkForUpdate(pkgName, currentVersion, deps = {}) {
-    const env = deps.env ?? process.env;
-    if (env.MCP_DISABLE_UPDATE_CHECK === "1" || env.NODE_ENV === "test") {
-        return;
-    }
-    const log = deps.log ?? ((msg) => process.stderr.write(msg + "\n"));
-    const fetcher = deps.fetchLatestVersion ?? (() => defaultFetch(pkgName));
-    let latest;
-    try {
-        latest = await fetcher();
-    }
-    catch {
-        return;
-    }
-    if (!latest || !semverLt(currentVersion, latest)) {
-        return;
-    }
-    log(`[update] ${pkgName}@${latest} is available (running ${currentVersion}). ` +
-        `Upgrade: npx -y ${pkgName}@latest (and relaunch Claude Desktop).`);
-}
-async function defaultFetch(pkgName) {
-    const controller = new AbortController();
-    const timer = setTimeout(() => controller.abort(), 2_000);
-    try {
-        const res = await fetch(`https://registry.npmjs.org/${pkgName}/latest`, {
-            signal: controller.signal,
-            headers: { Accept: "application/json" },
-        });
-        if (!res.ok) {
-            throw new Error(`HTTP ${res.status}`);
-        }
-        const body = (await res.json());
-        if (typeof body.version !== "string") {
-            throw new Error("registry response missing version field");
-        }
-        return body.version;
-    }
-    finally {
-        clearTimeout(timer);
-    }
-}
-function semverLt(a, b) {
-    const pa = a.split(".").map((x) => parseInt(x, 10) || 0);
-    const pb = b.split(".").map((x) => parseInt(x, 10) || 0);
-    for (let i = 0; i < 3; i++) {
-        if ((pa[i] ?? 0) < (pb[i] ?? 0))
-            return true;
-        if ((pa[i] ?? 0) > (pb[i] ?? 0))
-            return false;
-    }
-    return false;
-}