mcp-bing-ads 1.0.7 → 1.0.9

package/dist/build-info.json

@@ -1 +1 @@
-{"sha":"26e5811","builtAt":"2026-04-09T21:28:13.723Z"}
+{"sha":"9c3433f","builtAt":"2026-04-09T21:47:25.221Z"}

package/dist/errors.js

@@ -12,7 +12,7 @@ export class BingAdsAuthError extends Error {
 export class BingAdsRateLimitError extends Error {
     retryAfterMs;
     constructor(retryAfterMs, cause) {
-        super(`Rate limited, retry after ${retryAfterMs}ms`);
+        super(`Bing Ads rate limited, retry after ${retryAfterMs}ms`);
         this.retryAfterMs = retryAfterMs;
         this.name = "BingAdsRateLimitError";
         this.cause = cause;
@@ -47,7 +47,7 @@ export function classifyError(error) {
         message.includes("OAuth token refresh failed") ||
         message.includes("AuthenticationTokenExpired") ||
         message.includes("InvalidCredentials")) {
-        return new BingAdsAuthError(`Auth failed: ${message}. Refresh token may be expired. Re-authenticate and update Keychain.`, error);
+        return new BingAdsAuthError(`Bing Ads auth failed: ${message}. Refresh token may be expired. Re-authenticate and update Keychain.`, error);
     }
     if (status === 429 || message.includes("RateLimit") || message.includes("CallRateExceeded")) {
         const retryMs = 60_000;

package/dist/index.js

@@ -33,6 +33,10 @@ if (process.argv.includes("--version") || process.argv.includes("-v")) {
     console.error(__cliPkg.version);
     process.exit(0);
 }
+// ============================================
+// ENV VAR TRIMMING
+// ============================================
+const envTrimmed = (key) => (process.env[key] || "").trim().replace(/^["']|["']$/g, "");
 function loadConfig() {
     const configPath = join(dirname(new URL(import.meta.url).pathname), "..", "config.json");
     if (!existsSync(configPath)) {
@@ -69,14 +73,14 @@ class BingAdsManager {
             throw new BingAdsAuthError(msg);
         }
         console.error("[startup] Credentials validated: all required env vars present");
-        this.developerToken = process.env.BING_ADS_DEVELOPER_TOKEN;
-        this.refreshToken = process.env.BING_ADS_REFRESH_TOKEN;
+        this.developerToken = envTrimmed("BING_ADS_DEVELOPER_TOKEN");
+        this.refreshToken = envTrimmed("BING_ADS_REFRESH_TOKEN");
         // Allow env vars to override config for OAuth
         if (process.env.BING_ADS_CLIENT_ID) {
-            this.config.oauth.client_id = process.env.BING_ADS_CLIENT_ID;
+            this.config.oauth.client_id = envTrimmed("BING_ADS_CLIENT_ID");
         }
         if (process.env.BING_ADS_CLIENT_SECRET) {
-            this.config.oauth.client_secret = process.env.BING_ADS_CLIENT_SECRET;
+            this.config.oauth.client_secret = envTrimmed("BING_ADS_CLIENT_SECRET");
         }
     }
     async getAccessToken() {
@@ -738,6 +742,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             error: true,
             error_type: error.name,
             message: error.message,
+            server: __cliPkg.name,
         };
         if (error instanceof BingAdsAuthError) {
             response.action_required = "Re-authenticate: refresh token may be expired. Update Keychain entry BING_ADS_REFRESH_TOKEN.";
@@ -753,11 +758,11 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             response.details = rawError.stack;
         }
         return {
+            isError: true,
             content: [{
                     type: "text",
                     text: JSON.stringify(response, null, 2),
                 }],
-            isError: true,
         };
     }
 });
@@ -788,4 +793,7 @@ process.on("SIGINT", () => {
 process.on("SIGPIPE", () => {
     // Client disconnected -- expected during shutdown
 });
+process.on("unhandledRejection", (reason) => {
+    console.error("[error] Unhandled promise rejection:", reason);
+});
 main().catch(console.error);

package/dist/resilience.js

@@ -5,7 +5,7 @@ import pino from "pino";
 // ============================================
 export const logger = pino({
     level: process.env.LOG_LEVEL || "info",
-    ...(process.env.NODE_ENV !== "test" && {
+    ...(process.env.NODE_ENV !== "test" && process.stderr.isTTY && {
         transport: {
             target: "pino-pretty",
             options: {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "mcp-bing-ads",
   "mcpName": "io.github.mharnett/bing-ads",
-  "version": "1.0.7",
+  "version": "1.0.9",
   "description": "MCP server for Microsoft Advertising (Bing Ads) API with campaign, ad group, keyword, and performance reporting. First comprehensive open-source Bing Ads MCP.",
   "main": "dist/index.js",
   "bin": {
@@ -48,7 +48,7 @@
   },
   "homepage": "https://github.com/mharnett/mcp-bing-ads#readme",
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=18.18.0"
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^0.5.0",