@chrischall/tripadvisor-mcp 0.2.0 → 0.2.1

package/.claude-plugin/marketplace.json

@@ -7,7 +7,7 @@
   },
   "metadata": {
     "description": "MCP server for the TripAdvisor Terra API — location search, details, photos, and reviews",
-    "version": "0.2.0"
+    "version": "0.2.1"
   },
   "plugins": [
     {
@@ -15,7 +15,7 @@
       "displayName": "TripAdvisor",
       "source": "./",
       "description": "TripAdvisor travel data via the Terra API — search hotels, restaurants, and attractions, with details, photos, and reviews",
-      "version": "0.2.0",
+      "version": "0.2.1",
       "author": {
         "name": "Chris Hall"
       },

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "tripadvisor-mcp",
   "displayName": "TripAdvisor",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "MCP server for the TripAdvisor Terra API — location search, details, photos, and reviews",
   "author": {
     "name": "Chris Hall",

package/dist/bundle.js

@@ -34718,8 +34718,12 @@ var API_KEY_RE = new RegExp([
   // webhook signing secret (Stripe-style)
 ].map((p) => `\\b${p}`).join("|"), "g");
 var QUERY_SECRET_RE = /([?&](?:access_token|refresh_token|client_secret|api_?key|signature|token|key|sig)=)[^&#\s"'<>`]+/gi;
+var AWS_SIGV4_RE = /([?&]X-Amz-(?:Signature|Security-Token|Credential)=)[^&#\s"'<>`]+/gi;
+var JSON_SECRET_KEYS = "access_token|refresh_token|client_secret|api_?key|password|passwd|secret|token";
+var JSON_SECRET_DQ_RE = new RegExp(`("(?:${JSON_SECRET_KEYS})"\\s*:\\s*")[^"]*(")`, "gi");
+var JSON_SECRET_SQ_RE = new RegExp(`('(?:${JSON_SECRET_KEYS})'\\s*:\\s*')[^']*(')`, "gi");
 function redactSecrets(text) {
-  return text.replace(BEARER_RE, "$1[REDACTED]").replace(BASIC_AUTH_RE, "$1[REDACTED]").replace(SET_COOKIE_RE, "$1$2=[REDACTED]").replace(COOKIE_HEADER_RE, (_m, prefix, pairs) => `${prefix}${pairs.replace(/=[^;,\s]*/g, "=[REDACTED]")}`).replace(API_KEY_RE, "[REDACTED]").replace(QUERY_SECRET_RE, "$1[REDACTED]").replace(JWT_RE, "[REDACTED]");
+  return text.replace(BEARER_RE, "$1[REDACTED]").replace(BASIC_AUTH_RE, "$1[REDACTED]").replace(SET_COOKIE_RE, "$1$2=[REDACTED]").replace(COOKIE_HEADER_RE, (_m, prefix, pairs) => `${prefix}${pairs.replace(/=[^;,\s]*/g, "=[REDACTED]")}`).replace(API_KEY_RE, "[REDACTED]").replace(QUERY_SECRET_RE, "$1[REDACTED]").replace(AWS_SIGV4_RE, "$1[REDACTED]").replace(JSON_SECRET_DQ_RE, "$1[REDACTED]$2").replace(JSON_SECRET_SQ_RE, "$1[REDACTED]$2").replace(JWT_RE, "[REDACTED]");
 }
 function truncateErrorMessage(text, max = DEFAULT_ERROR_MESSAGE_MAX) {
   const str2 = text === null || text === void 0 ? "" : String(text);
@@ -34795,6 +34799,100 @@ async function loadDotenvSafely(opts = {}) {
     return false;
   }
 }
+function readIntEnv(key, opts = {}) {
+  const raw = readEnvVar(key, opts.env ? { env: opts.env } : {});
+  if (raw === void 0)
+    return opts.default;
+  if (!/^-?\d+$/.test(raw))
+    return opts.default;
+  const n = Number(raw);
+  if (!Number.isSafeInteger(n))
+    return opts.default;
+  const min = opts.min ?? 0;
+  if (n < min)
+    return opts.default;
+  if (opts.max !== void 0 && n > opts.max)
+    return opts.default;
+  return n;
+}
+function readTtlMsEnv(key, defaultMs, opts = {}) {
+  const seconds = readIntEnv(key, { ...opts.env ? { env: opts.env } : {}, min: 0 });
+  return seconds === void 0 ? defaultMs : seconds * 1e3;
+}
+
+// node_modules/@chrischall/mcp-utils/dist/http/response-cache.js
+var RESPONSE_CACHE_MAX_ENTRIES = 256;
+function createResponseCache(opts) {
+  const now = opts.now ?? Date.now;
+  const maxEntries = opts.maxEntries ?? RESPONSE_CACHE_MAX_ENTRIES;
+  const store = /* @__PURE__ */ new Map();
+  const ttlFor = (tier) => opts.ttlMs[tier] ?? 0;
+  function evictForInsert() {
+    if (store.size < maxEntries)
+      return;
+    const t = now();
+    for (const [key, entry] of store) {
+      if (entry.expiresAt <= t)
+        store.delete(key);
+    }
+    while (store.size >= maxEntries) {
+      const oldest = store.keys().next();
+      if (oldest.done)
+        break;
+      store.delete(oldest.value);
+    }
+  }
+  function get(key) {
+    const entry = store.get(key);
+    if (!entry)
+      return void 0;
+    if (entry.expiresAt <= now()) {
+      store.delete(key);
+      return void 0;
+    }
+    return entry.value;
+  }
+  function set2(key, value, tier = "dynamic") {
+    const ttl = ttlFor(tier);
+    if (ttl <= 0)
+      return;
+    if (!store.has(key))
+      evictForInsert();
+    else
+      store.delete(key);
+    store.set(key, { expiresAt: now() + ttl, value });
+  }
+  return {
+    get,
+    set: set2,
+    async fetchThrough(key, load, tier = "dynamic") {
+      const hit = get(key);
+      if (hit !== void 0)
+        return hit;
+      const value = await load();
+      set2(key, value, tier);
+      return value;
+    },
+    clear() {
+      store.clear();
+    },
+    get size() {
+      return store.size;
+    }
+  };
+}
+
+// node_modules/@chrischall/mcp-utils/dist/http/net-atoms.js
+function parseRetryAfterMs(header, opts = {}) {
+  const defaultMs = opts.defaultMs ?? 2e3;
+  const capMs = opts.capMs ?? 3e4;
+  if (header == null)
+    return defaultMs;
+  const trimmed = header.trim();
+  if (!/^\d+$/.test(trimmed))
+    return defaultMs;
+  return Math.min(Number(trimmed) * 1e3, capMs);
+}
 
 // node_modules/@chrischall/mcp-utils/dist/http/index.js
 function buildQueryString(params) {
@@ -34841,7 +34939,7 @@ var pageSchema = {
 };
 
 // src/version.ts
-var VERSION = "0.2.0";
+var VERSION = "0.2.1";
 
 // src/client.ts
 import { dirname, join } from "node:path";
@@ -34853,33 +34951,24 @@ var SERVICE = "TripAdvisor Terra API";
 var REQUEST_TIMEOUT_MS = 3e4;
 var DEFAULT_CACHE_TTL_MS = 3e5;
 var DEFAULT_STATIC_CACHE_TTL_MS = 36e5;
-var CACHE_MAX_ENTRIES = 256;
 var MAX_RETRY_AFTER_MS = 1e4;
-function readCacheTtlMs(envVar, defaultMs) {
-  const raw = readEnvVar(envVar);
-  if (raw === void 0) return defaultMs;
-  const secs = Number(raw);
-  return Number.isFinite(secs) && secs >= 0 ? secs * 1e3 : defaultMs;
-}
 var TripAdvisorClient = class {
   apiKey;
   configError;
   fetchImpl;
   sleep;
-  cacheTtlMs;
-  staticCacheTtlMs;
-  now;
-  cache = /* @__PURE__ */ new Map();
+  cache;
   /**
    * Defer the config error so the server still boots (and answers the host's
    * install-time tools/list probe) when TRIPADVISOR_API_KEY isn't set yet. The
    * error is re-raised at request time via requireKey().
    */
   constructor(opts = {}) {
-    this.now = opts.now ?? Date.now;
+    const now = opts.now ?? Date.now;
     this.sleep = opts.sleep ?? ((ms) => new Promise((r) => setTimeout(r, ms)));
-    this.cacheTtlMs = opts.cacheTtlMs ?? readCacheTtlMs("TRIPADVISOR_CACHE_TTL", DEFAULT_CACHE_TTL_MS);
-    this.staticCacheTtlMs = opts.staticCacheTtlMs ?? readCacheTtlMs("TRIPADVISOR_STATIC_CACHE_TTL", DEFAULT_STATIC_CACHE_TTL_MS);
+    const cacheTtlMs = opts.cacheTtlMs ?? readTtlMsEnv("TRIPADVISOR_CACHE_TTL", DEFAULT_CACHE_TTL_MS);
+    const staticCacheTtlMs = opts.staticCacheTtlMs ?? readTtlMsEnv("TRIPADVISOR_STATIC_CACHE_TTL", DEFAULT_STATIC_CACHE_TTL_MS);
+    this.cache = createResponseCache({ ttlMs: { dynamic: cacheTtlMs, static: staticCacheTtlMs }, now });
     this.fetchImpl = opts.fetchImpl ?? fetch;
     const key = readEnvVar("TRIPADVISOR_API_KEY");
     if (!key) {
@@ -34905,25 +34994,8 @@ var TripAdvisorClient = class {
    * (TRIPADVISOR_CACHE_TTL) is for searches.
    */
   async get(path, opts = {}) {
-    const ttl = opts.cache === "static" ? this.staticCacheTtlMs : this.cacheTtlMs;
-    if (ttl > 0) {
-      const hit = this.cache.get(path);
-      if (hit && hit.expiresAt > this.now()) return hit.value;
-    }
-    const value = await this.request(path);
-    if (ttl > 0) {
-      if (this.cache.size >= CACHE_MAX_ENTRIES) {
-        const t = this.now();
-        for (const [k, v] of this.cache) if (v.expiresAt <= t) this.cache.delete(k);
-        while (this.cache.size >= CACHE_MAX_ENTRIES) {
-          const oldest = this.cache.keys().next().value;
-          if (oldest === void 0) break;
-          this.cache.delete(oldest);
-        }
-      }
-      this.cache.set(path, { expiresAt: this.now() + ttl, value });
-    }
-    return value;
+    const tier = opts.cache === "static" ? "static" : "dynamic";
+    return this.cache.fetchThrough(path, () => this.request(path), tier);
   }
   async request(path, isRetry = false) {
     const key = this.requireKey();
@@ -34932,8 +35004,7 @@ var TripAdvisorClient = class {
     if (res.ok) return await res.json();
     const text = await res.text();
     if (res.status === 429 && !isRetry) {
-      const retryAfter = Number(res.headers.get("retry-after"));
-      const delayMs = Number.isFinite(retryAfter) && retryAfter > 0 ? Math.min(retryAfter * 1e3, MAX_RETRY_AFTER_MS) : 1e3;
+      const delayMs = parseRetryAfterMs(res.headers.get("retry-after"), { defaultMs: 1e3, capMs: MAX_RETRY_AFTER_MS });
       await this.sleep(delayMs);
       return this.request(path, true);
     }
@@ -38511,7 +38582,7 @@ function healthcheckHint(args) {
   return `Unexpected error \u2014 see the error.message field for details.`;
 }
 function registerBridgeHealthcheckTool(args) {
-  const { server, prefix, probePath, hostLabel, transport, probeFn } = args;
+  const { server, prefix, probePath, hostLabel, transport, probeFn, classifyThrown, hints } = args;
   const probeUrl = `https://${hostLabel}${probePath}`;
   server.registerTool(`${prefix}_healthcheck`, {
     title: "Verify the fetchproxy bridge end-to-end",
@@ -38544,16 +38615,39 @@ function registerBridgeHealthcheckTool(args) {
     } : { url: probeUrl, elapsed_ms: probeResult.elapsed_ms };
     let error51;
     let bridgeHint;
+    let customHint;
+    let customDetail;
     if (probeResult.error) {
-      const kind = probeResult.error.kind === "other" ? "unknown" : probeResult.error.kind;
+      let kind = probeResult.error.kind === "other" ? "unknown" : probeResult.error.kind;
       bridgeHint = thrown instanceof FetchproxyBridgeDownError ? thrown.hint : void 0;
+      if (thrown !== void 0 && classifyThrown) {
+        const custom2 = classifyThrown(thrown);
+        if (custom2) {
+          kind = custom2.kind;
+          customHint = custom2.hint;
+          customDetail = custom2.detail;
+        }
+      }
       error51 = {
         kind,
         message: probeResult.error.message,
-        ...bridgeHint !== void 0 ? { bridge_hint: bridgeHint } : {}
+        ...bridgeHint !== void 0 ? { bridge_hint: bridgeHint } : {},
+        ...customDetail !== void 0 ? { detail: customDetail } : {}
       };
     }
     const lastExtensionMessageAt = transport.status().lastExtensionMessageAt;
+    const arm = ok ? "ok" : error51?.kind === "bridge_down" ? "bridge_down" : probeResult.bridge.role === null ? "no_role" : error51?.kind === "timeout" ? "timeout" : error51?.kind === "protocol" || error51?.kind === "http" ? "protocol" : "unknown";
+    const defaultHint = healthcheckHint({
+      ok,
+      role: probeResult.bridge.role,
+      // FIX: the real configured port from bridgeHealth(), not a literal 37149.
+      port: probeResult.bridge.port,
+      hostLabel,
+      prefix,
+      probePath,
+      errorKind: error51?.kind,
+      bridgeHint: error51?.kind === "bridge_down" ? bridgeHint : void 0
+    });
     const result = {
       ok,
       bridge: {
@@ -38562,17 +38656,8 @@ function registerBridgeHealthcheckTool(args) {
       },
       probe,
       ...error51 ? { error: error51 } : {},
-      hint: healthcheckHint({
-        ok,
-        role: probeResult.bridge.role,
-        // FIX: the real configured port from bridgeHealth(), not a literal 37149.
-        port: probeResult.bridge.port,
-        hostLabel,
-        prefix,
-        probePath,
-        errorKind: error51?.kind,
-        bridgeHint: error51?.kind === "bridge_down" ? bridgeHint : void 0
-      })
+      // Precedence: classifyThrown's hint > per-arm override > default ladder.
+      hint: customHint ?? hints?.[arm] ?? defaultHint
     };
     return {
       content: [{ type: "text", text: JSON.stringify(result, null, 2) }]

package/dist/client.js

@@ -1,6 +1,6 @@
 import { dirname, join } from 'node:path';
 import { fileURLToPath } from 'node:url';
-import { loadDotenvSafely, readEnvVar, formatApiError, truncateErrorMessage, McpToolError } from '@chrischall/mcp-utils';
+import { loadDotenvSafely, readEnvVar, readTtlMsEnv, createResponseCache, parseRetryAfterMs, formatApiError, truncateErrorMessage, McpToolError, } from '@chrischall/mcp-utils';
 // Load .env for local dev; silently skip if dotenv is unavailable (e.g. the
 // .mcpb bundle). loadDotenvSafely never lets .env override a host-provided value.
 const __dirname = dirname(fileURLToPath(import.meta.url));
@@ -15,40 +15,25 @@ const DEFAULT_CACHE_TTL_MS = 300_000;
 // Location details/photos/reviews change slowly — 1 hour by default.
 // Override with TRIPADVISOR_STATIC_CACHE_TTL (seconds; 0 = off).
 const DEFAULT_STATIC_CACHE_TTL_MS = 3_600_000;
-// Bound the cache so a long-lived server doesn't grow unbounded across many
-// distinct paths; oldest entries are evicted first.
-const CACHE_MAX_ENTRIES = 256;
 // Cap a server-supplied Retry-After so one bad header can't stall a tool call.
 const MAX_RETRY_AFTER_MS = 10_000;
-/** Resolve a cache TTL (ms) from an env var holding seconds. A blank or
- * non-numeric value falls back to `defaultMs`; a valid `0` disables caching. */
-function readCacheTtlMs(envVar, defaultMs) {
-    const raw = readEnvVar(envVar);
-    if (raw === undefined)
-        return defaultMs;
-    const secs = Number(raw);
-    return Number.isFinite(secs) && secs >= 0 ? secs * 1000 : defaultMs;
-}
 export class TripAdvisorClient {
     apiKey;
     configError;
     fetchImpl;
     sleep;
-    cacheTtlMs;
-    staticCacheTtlMs;
-    now;
-    cache = new Map();
+    cache;
     /**
      * Defer the config error so the server still boots (and answers the host's
      * install-time tools/list probe) when TRIPADVISOR_API_KEY isn't set yet. The
      * error is re-raised at request time via requireKey().
      */
     constructor(opts = {}) {
-        this.now = opts.now ?? Date.now;
+        const now = opts.now ?? Date.now;
         this.sleep = opts.sleep ?? ((ms) => new Promise((r) => setTimeout(r, ms)));
-        this.cacheTtlMs = opts.cacheTtlMs ?? readCacheTtlMs('TRIPADVISOR_CACHE_TTL', DEFAULT_CACHE_TTL_MS);
-        this.staticCacheTtlMs =
-            opts.staticCacheTtlMs ?? readCacheTtlMs('TRIPADVISOR_STATIC_CACHE_TTL', DEFAULT_STATIC_CACHE_TTL_MS);
+        const cacheTtlMs = opts.cacheTtlMs ?? readTtlMsEnv('TRIPADVISOR_CACHE_TTL', DEFAULT_CACHE_TTL_MS);
+        const staticCacheTtlMs = opts.staticCacheTtlMs ?? readTtlMsEnv('TRIPADVISOR_STATIC_CACHE_TTL', DEFAULT_STATIC_CACHE_TTL_MS);
+        this.cache = createResponseCache({ ttlMs: { dynamic: cacheTtlMs, static: staticCacheTtlMs }, now });
         this.fetchImpl = opts.fetchImpl ?? fetch;
         const key = readEnvVar('TRIPADVISOR_API_KEY');
         if (!key) {
@@ -76,31 +61,8 @@ export class TripAdvisorClient {
      * (TRIPADVISOR_CACHE_TTL) is for searches.
      */
     async get(path, opts = {}) {
-        const ttl = opts.cache === 'static' ? this.staticCacheTtlMs : this.cacheTtlMs;
-        if (ttl > 0) {
-            const hit = this.cache.get(path);
-            if (hit && hit.expiresAt > this.now())
-                return hit.value;
-        }
-        const value = await this.request(path);
-        if (ttl > 0) {
-            if (this.cache.size >= CACHE_MAX_ENTRIES) {
-                // Evict expired entries first; if still full, drop the oldest (Map
-                // preserves insertion order, so the first key is the oldest).
-                const t = this.now();
-                for (const [k, v] of this.cache)
-                    if (v.expiresAt <= t)
-                        this.cache.delete(k);
-                while (this.cache.size >= CACHE_MAX_ENTRIES) {
-                    const oldest = this.cache.keys().next().value;
-                    if (oldest === undefined)
-                        break;
-                    this.cache.delete(oldest);
-                }
-            }
-            this.cache.set(path, { expiresAt: this.now() + ttl, value });
-        }
-        return value;
+        const tier = opts.cache === 'static' ? 'static' : 'dynamic';
+        return this.cache.fetchThrough(path, () => this.request(path), tier);
     }
     async request(path, isRetry = false) {
         const key = this.requireKey();
@@ -112,8 +74,7 @@ export class TripAdvisorClient {
             return (await res.json());
         const text = await res.text();
         if (res.status === 429 && !isRetry) {
-            const retryAfter = Number(res.headers.get('retry-after'));
-            const delayMs = Number.isFinite(retryAfter) && retryAfter > 0 ? Math.min(retryAfter * 1000, MAX_RETRY_AFTER_MS) : 1_000;
+            const delayMs = parseRetryAfterMs(res.headers.get('retry-after'), { defaultMs: 1_000, capMs: MAX_RETRY_AFTER_MS });
             await this.sleep(delayMs);
             return this.request(path, true);
         }

package/dist/version.js

@@ -1,2 +1,2 @@
 /** Single source of the server version. release-please bumps the literal below. */
-export const VERSION = '0.2.0'; // x-release-please-version
+export const VERSION = '0.2.1'; // x-release-please-version

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrischall/tripadvisor-mcp",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "mcpName": "io.github.chrischall/tripadvisor-mcp",
   "description": "TripAdvisor Terra API MCP server for Claude — search locations, details, photos, and reviews. Developed and maintained by AI (Claude Code).",
   "author": "Claude Code (AI) <https://www.anthropic.com/claude>",
@@ -44,7 +44,7 @@
     "test:coverage": "vitest run --coverage"
   },
   "dependencies": {
-    "@chrischall/mcp-utils": "^0.10.0",
+    "@chrischall/mcp-utils": "^0.12.0",
     "@fetchproxy/server": "^1.3.4",
     "@modelcontextprotocol/sdk": "^1.29.0",
     "dotenv": "^17.4.0",

package/server.json

@@ -6,12 +6,12 @@
     "url": "https://github.com/chrischall/tripadvisor-mcp",
     "source": "github"
   },
-  "version": "0.2.0",
+  "version": "0.2.1",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "@chrischall/tripadvisor-mcp",
-      "version": "0.2.0",
+      "version": "0.2.1",
       "transport": {
         "type": "stdio"
       },