mcp-metacritic 1.2.0 → 2.0.0

package/dist/index.js

@@ -1,10 +1,14 @@
 #!/usr/bin/env node
 
 // src/index.ts
+import process2 from "process";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 
+// src/config.ts
+import process from "process";
+
 // src/version.ts
-var PKG_VERSION = "1.2.0";
+var PKG_VERSION = "2.0.0";
 var REPO_URL = "https://github.com/smeet666/mcp-metacritic";
 
 // src/config.ts
@@ -28,7 +32,9 @@ var MAX_ALLOWED_INTERVAL_MS = 6e4;
 var LOG_LEVELS = ["silent", "error", "info", "debug"];
 function readNumber(name, env, range) {
   const raw = env[name];
-  if (raw === void 0 || raw.trim() === "") return range.fallback;
+  if (raw === void 0 || raw.trim() === "") {
+    return range.fallback;
+  }
   const parsed = Number(raw);
   if (!Number.isFinite(parsed)) {
     warn(`${name}="${raw}" is not a number, using ${range.fallback}`);
@@ -49,7 +55,9 @@ function warn(message) {
 }
 function readInterval(env) {
   const raw = env.MC_MIN_INTERVAL_MS;
-  if (raw === void 0 || raw.trim() === "") return DEFAULTS.minIntervalMs;
+  if (raw === void 0 || raw.trim() === "") {
+    return DEFAULTS.minIntervalMs;
+  }
   const parsed = Number(raw);
   if (!Number.isFinite(parsed)) {
     warn(`MC_MIN_INTERVAL_MS="${raw}" is not a number, using ${DEFAULTS.minIntervalMs}ms`);
@@ -142,7 +150,9 @@ var TtlLruCache = class {
   store = /* @__PURE__ */ new Map();
   get(key) {
     const entry = this.store.get(key);
-    if (!entry) return void 0;
+    if (!entry) {
+      return void 0;
+    }
     if (Date.now() >= entry.expiresAt) {
       this.store.delete(key);
       return void 0;
@@ -152,12 +162,16 @@ var TtlLruCache = class {
     return entry.value;
   }
   set(key, value) {
-    if (this.maxEntries <= 0 || this.ttlMs <= 0) return;
+    if (this.maxEntries <= 0 || this.ttlMs <= 0) {
+      return;
+    }
     this.store.delete(key);
     this.store.set(key, { value, expiresAt: Date.now() + this.ttlMs });
     while (this.store.size > this.maxEntries) {
       const oldest = this.store.keys().next();
-      if (oldest.done) break;
+      if (oldest.done) {
+        break;
+      }
       this.store.delete(oldest.value);
     }
   }
@@ -172,7 +186,7 @@ var TtlLruCache = class {
 // src/errors.ts
 var McError = class extends Error {
   constructor(code, message, details = {}) {
-    super(message);
+    super(message, details.cause === void 0 ? void 0 : { cause: details.cause });
     this.code = code;
     this.details = details;
     this.name = "McError";
@@ -202,11 +216,11 @@ function rateLimited(url, retryAfterMs) {
     }
   );
 }
-function parseFailure(url, what) {
+function parseFailure(url, what, cause) {
   return new McError(
     "parse_failure",
     `Metacritic answered, but the response did not have the expected shape (${what}).`,
-    { url, hint: `Please report this, with the request you made, at ${ISSUES_URL}` }
+    { url, hint: `Please report this, with the request you made, at ${ISSUES_URL}`, cause }
   );
 }
 function upstreamError(url, status) {
@@ -274,10 +288,14 @@ var RateLimiter = class {
     this.intervalMs = Math.max(this.baseIntervalMs, Math.floor(this.intervalMs * 0.75));
   }
   async waitForSlot() {
-    if (this.intervalMs === 0 || this.lastStart === 0) return;
+    if (this.intervalMs === 0 || this.lastStart === 0) {
+      return;
+    }
     const elapsed = Date.now() - this.lastStart;
     const remaining = Math.min(this.intervalMs, this.intervalMs - elapsed);
-    if (remaining > 0) await sleep(remaining);
+    if (remaining > 0) {
+      await sleep(remaining);
+    }
   }
 };
 function sleep(ms) {
@@ -292,10 +310,57 @@ function backoffDelay(attempt, random = Math.random) {
   const capped = Math.min(BACKOFF_MAX_MS, BACKOFF_BASE_MS * BACKOFF_FACTOR ** attempt);
   return Math.round(capped * (0.5 + random() * 0.5));
 }
+function readAnswer(url, status, body, retryAfterMs, ownGuessMs) {
+  if (status === 429 || status === 503 || status === 403) {
+    return {
+      kind: "again",
+      error: rateLimited(url, retryAfterMs ?? ownGuessMs),
+      // A server that says when to come back knows better than our own guess.
+      waitMs: retryAfterMs,
+      penalise: true,
+      because: `refused with ${status}`
+    };
+  }
+  if (status >= 500) {
+    return {
+      kind: "again",
+      error: upstreamError(url, status),
+      waitMs: null,
+      penalise: false,
+      because: `status ${status}`
+    };
+  }
+  if (status === 404) {
+    return { kind: "refused", error: notFound(url, "that request") };
+  }
+  if (status >= 400) {
+    return { kind: "refused", error: upstreamError(url, status) };
+  }
+  const trimmed = body.trim();
+  if (trimmed === "") {
+    return {
+      kind: "again",
+      error: rateLimited(url, ownGuessMs),
+      waitMs: null,
+      penalise: true,
+      because: "empty body, treated as a refusal"
+    };
+  }
+  if (!(trimmed.startsWith("{") || trimmed.startsWith("["))) {
+    return {
+      kind: "again",
+      error: parseFailure(url, "the body is not JSON"),
+      waitMs: retryAfterMs,
+      penalise: true,
+      because: "non-JSON body"
+    };
+  }
+  return { kind: "usable" };
+}
 async function fetchText(url, deps) {
   const { config, limiter, logger } = deps;
   const doFetch = deps.fetchImpl ?? fetch;
-  return limiter.schedule(async () => {
+  return await limiter.schedule(async () => {
     let lastError;
     let askedWaitMs = null;
     for (let attempt = 0; attempt <= config.maxRetries; attempt += 1) {
@@ -326,33 +391,17 @@ async function fetchText(url, deps) {
         logger.debug(`${lastError.code} for ${url}: ${lastError.message}`);
         continue;
       }
-      if (status === 429 || status === 503 || status === 403) {
-        limiter.penalize();
-        askedWaitMs = retryAfterMs;
-        lastError = rateLimited(url, retryAfterMs ?? backoffDelay(attempt));
-        logger.info(
-          `refused on ${url} with ${status}, interval now ${limiter.currentIntervalMs}ms`
-        );
-        continue;
-      }
-      if (status >= 500) {
-        lastError = upstreamError(url, status);
-        continue;
+      const verdict = readAnswer(url, status, body, retryAfterMs, backoffDelay(attempt));
+      if (verdict.kind === "refused") {
+        throw verdict.error;
       }
-      if (status === 404) throw notFound(url, "that request");
-      if (status >= 400) throw upstreamError(url, status);
-      const trimmed = body.trim();
-      if (trimmed === "") {
-        limiter.penalize();
-        lastError = rateLimited(url, backoffDelay(attempt));
-        logger.info(`empty body on ${url}, treating as a refusal`);
-        continue;
-      }
-      if (!trimmed.startsWith("{") && !trimmed.startsWith("[")) {
-        limiter.penalize();
-        lastError = parseFailure(url, "the body is not JSON");
-        askedWaitMs = retryAfterMs;
-        logger.info(`non-JSON body on ${url}, retrying`);
+      if (verdict.kind === "again") {
+        if (verdict.penalise) {
+          limiter.penalize();
+          logger.info(`${verdict.because} on ${url}, interval now ${limiter.currentIntervalMs}ms`);
+        }
+        askedWaitMs = verdict.waitMs;
+        lastError = verdict.error;
         continue;
       }
       limiter.relax();
@@ -362,15 +411,23 @@ async function fetchText(url, deps) {
   });
 }
 function parseRetryAfter(raw) {
-  if (!raw) return null;
+  if (!raw) {
+    return null;
+  }
   const seconds = Number(raw.trim());
-  if (Number.isFinite(seconds) && seconds >= 0) return Math.round(seconds * 1e3);
+  if (Number.isFinite(seconds) && seconds >= 0) {
+    return Math.round(seconds * 1e3);
+  }
   const when = Date.parse(raw);
-  if (Number.isNaN(when)) return null;
+  if (Number.isNaN(when)) {
+    return null;
+  }
   return Math.max(0, when - Date.now());
 }
 function asTransportError(error, url) {
-  if (error instanceof McError) return error;
+  if (error instanceof McError) {
+    return error;
+  }
   const name = error instanceof Error ? error.name : "";
   if (name === "TimeoutError" || name === "AbortError") {
     return new McError("timeout", "Metacritic did not answer in time.", {
@@ -428,10 +485,14 @@ var FIELD = {
 };
 
 // src/mc/urls.ts
+var SLUG = /^[a-z0-9][a-z0-9-]*$/i;
+var IMDB_ID = /^tt\d+$/;
 function build(path, params) {
   const query = new URLSearchParams();
   for (const [key, value] of Object.entries(params)) {
-    if (value !== void 0 && value !== "") query.set(key, String(value));
+    if (value !== void 0 && value !== "") {
+      query.set(key, String(value));
+    }
   }
   const suffix = query.toString();
   return suffix ? `${API_BASE}${path}?${suffix}` : `${API_BASE}${path}`;
@@ -441,7 +502,7 @@ function checkSlug(slug) {
   if (trimmed === "") {
     throw invalidInput("The slug is empty.", "Take it from a search_titles result.");
   }
-  if (!/^[a-z0-9][a-z0-9-]*$/i.test(trimmed)) {
+  if (!SLUG.test(trimmed)) {
     throw invalidInput(
       `"${slug}" does not look like a Metacritic slug.`,
       "Slugs are lowercase words joined by hyphens, such as the-matrix. Use search_titles to get one."
@@ -484,7 +545,7 @@ function reviewsUrl(options) {
 }
 function watchUrl(imdbId, kind) {
   const trimmed = imdbId.trim();
-  if (!/^tt\d+$/.test(trimmed)) {
+  if (!IMDB_ID.test(trimmed)) {
     throw invalidInput(
       `"${imdbId}" is not an IMDb id.`,
       "IMDb ids look like tt0133093. get_title returns one as imdb_id."
@@ -503,38 +564,51 @@ function bucketFor(sentiment) {
 }
 
 // src/mc/parse.ts
+var ABSOLUTE_URL = /^https?:\/\//i;
 var isObject = (value) => typeof value === "object" && value !== null && !Array.isArray(value);
 function envelope(raw, url, what) {
   let body;
   try {
     body = JSON.parse(raw);
-  } catch {
-    throw parseFailure(url, "the body is not valid JSON");
+  } catch (cause) {
+    throw parseFailure(url, "the body is not valid JSON", cause);
+  }
+  if (!isObject(body)) {
+    throw parseFailure(url, "the body is not an object");
   }
-  if (!isObject(body)) throw parseFailure(url, "the body is not an object");
   const errors = body[FIELD.errors];
   if (Array.isArray(errors) && errors.length > 0) {
     const first = isObject(errors[0]) ? errors[0] : {};
-    if (first.code === 404) throw notFound(url, what);
+    if (first.code === 404) {
+      throw notFound(url, what);
+    }
     throw parseFailure(
       url,
       `the site reported ${String(first.reason ?? first.code ?? "an error")}`
     );
   }
   const data = body[FIELD.data];
-  if (!isObject(data)) throw parseFailure(url, "the response carries no data object");
+  if (!isObject(data)) {
+    throw parseFailure(url, "the response carries no data object");
+  }
   return data;
 }
 var str = (value) => {
-  if (typeof value !== "string") return null;
+  if (typeof value !== "string") {
+    return null;
+  }
   const trimmed = value.trim();
   return trimmed === "" ? null : trimmed;
 };
 var num = (value) => {
-  if (typeof value === "number" && Number.isFinite(value)) return value;
+  if (typeof value === "number" && Number.isFinite(value)) {
+    return value;
+  }
   if (typeof value === "string") {
     const parsed = Number(value);
-    if (Number.isFinite(parsed)) return parsed;
+    if (Number.isFinite(parsed)) {
+      return parsed;
+    }
   }
   return null;
 };
@@ -548,18 +622,28 @@ var aggregate = (value) => {
 };
 function kindOf(value) {
   const type = str(value);
-  if (type === SEARCH_TYPE.movie) return "movie";
-  if (type === SEARCH_TYPE.show) return "show";
-  if (type === SEARCH_TYPE.game) return "game";
+  if (type === SEARCH_TYPE.movie) {
+    return "movie";
+  }
+  if (type === SEARCH_TYPE.show) {
+    return "show";
+  }
+  if (type === SEARCH_TYPE.game) {
+    return "game";
+  }
   return null;
 }
 function toSummary(node) {
-  if (!isObject(node)) return null;
+  if (!isObject(node)) {
+    return null;
+  }
   const id = intOf(node.id);
   const slug = str(node.slug);
   const title = str(node.title);
   const kind = kindOf(node.type);
-  if (id === null || !slug || !title || !kind) return null;
+  if (id === null || !slug || !title || !kind) {
+    return null;
+  }
   const critic = isObject(node.criticScoreSummary) ? node.criticScoreSummary : {};
   const userSummary = isObject(node.userScoreSummary) ? node.userScoreSummary : {};
   const userObject = isObject(node.userScore) ? node.userScore : {};
@@ -580,11 +664,15 @@ function toSummary(node) {
 function parseTitlePage(raw, url, what) {
   const data = envelope(raw, url, what);
   const items = data[FIELD.items];
-  if (!Array.isArray(items)) throw parseFailure(url, "the response carries no items array");
+  if (!Array.isArray(items)) {
+    throw parseFailure(url, "the response carries no items array");
+  }
   const titles = [];
   for (const node of items) {
     const row = toSummary(node);
-    if (row) titles.push(row);
+    if (row) {
+      titles.push(row);
+    }
   }
   if (items.length > 0 && titles.length === 0) {
     throw parseFailure(url, `${items.length} rows but none could be read`);
@@ -597,35 +685,51 @@ function parseTitlePage(raw, url, what) {
 }
 function toCompanies(value) {
   const source = isObject(value) ? value.companies : value;
-  if (!Array.isArray(source)) return [];
+  if (!Array.isArray(source)) {
+    return [];
+  }
   const seen = /* @__PURE__ */ new Set();
   const out = [];
   for (const node of source) {
     const name = isObject(node) ? str(node.name) : str(node);
-    if (!name) continue;
+    if (!name) {
+      continue;
+    }
     const key = name.toLowerCase();
-    if (seen.has(key)) continue;
+    if (seen.has(key)) {
+      continue;
+    }
     seen.add(key);
     out.push({ name, id: isObject(node) ? intOf(node.id) : null });
   }
   return out;
 }
 function toStringList(value) {
-  if (!Array.isArray(value)) return [];
+  if (!Array.isArray(value)) {
+    return [];
+  }
   const out = [];
   for (const node of value) {
     const name = isObject(node) ? str(node.name) : str(node);
-    if (name) out.push(name);
+    if (name) {
+      out.push(name);
+    }
   }
   return out;
 }
 function toAwards(value) {
-  if (!Array.isArray(value)) return [];
+  if (!Array.isArray(value)) {
+    return [];
+  }
   const out = [];
   for (const node of value) {
-    if (!isObject(node)) continue;
+    if (!isObject(node)) {
+      continue;
+    }
     const ceremony = str(node.awardEvent);
-    if (!ceremony) continue;
+    if (!ceremony) {
+      continue;
+    }
     out.push({
       ceremony,
       wins: intOf(node.wins),
@@ -637,10 +741,14 @@ function toAwards(value) {
 function parseDetail(raw, url, kind, slug) {
   const data = envelope(raw, url, `${kind} "${slug}"`);
   const item = data[FIELD.item];
-  if (!isObject(item)) throw parseFailure(url, "the response carries no item object");
+  if (!isObject(item)) {
+    throw parseFailure(url, "the response carries no item object");
+  }
   const id = intOf(item.id);
   const title = str(item.title);
-  if (id === null || !title) throw parseFailure(url, "the entry has no id or no title");
+  if (id === null || !title) {
+    throw parseFailure(url, "the entry has no id or no title");
+  }
   const critic = isObject(item.criticScoreSummary) ? item.criticScoreSummary : {};
   return {
     id,
@@ -668,9 +776,13 @@ function parseDetail(raw, url, kind, slug) {
 function parseScore(raw, url, what) {
   const data = envelope(raw, url, what);
   const item = data[FIELD.item];
-  if (!isObject(item)) throw parseFailure(url, "the response carries no score object");
+  if (!isObject(item)) {
+    throw parseFailure(url, "the response carries no score object");
+  }
   const max = num(item.max);
-  if (max === null) throw parseFailure(url, "the score carries no scale");
+  if (max === null) {
+    throw parseFailure(url, "the score carries no scale");
+  }
   return {
     score: aggregate(item.score),
     max,
@@ -683,7 +795,9 @@ function parseScore(raw, url, what) {
 }
 function reviewBucket(data, url, sentiment) {
   const item = data[FIELD.item];
-  if (!isObject(item)) throw parseFailure(url, "the response carries no reviews object");
+  if (!isObject(item)) {
+    throw parseFailure(url, "the response carries no reviews object");
+  }
   const bucket = item[bucketFor(sentiment)];
   if (!Array.isArray(bucket)) {
     throw parseFailure(url, `there is no "${bucketFor(sentiment)}" group of reviews`);
@@ -695,10 +809,14 @@ function parseCriticReviews(raw, url, what, sentiment) {
   const bucket = reviewBucket(data, url, sentiment);
   const reviews = [];
   for (const node of bucket) {
-    if (!isObject(node)) continue;
+    if (!isObject(node)) {
+      continue;
+    }
     const quote = str(node.quote);
     const publication = str(node.publicationName);
-    if (!quote || !publication) continue;
+    if (!(quote && publication)) {
+      continue;
+    }
     reviews.push({
       quote,
       score: num(node.score),
@@ -722,9 +840,13 @@ function parseUserReviews(raw, url, what, sentiment) {
   const bucket = reviewBucket(data, url, sentiment);
   const reviews = [];
   for (const node of bucket) {
-    if (!isObject(node)) continue;
+    if (!isObject(node)) {
+      continue;
+    }
     const quote = str(node.quote);
-    if (!quote) continue;
+    if (!quote) {
+      continue;
+    }
     reviews.push({ quote, score: num(node.score), date: str(node.date) });
   }
   if (bucket.length > 0 && reviews.length === 0) {
@@ -737,10 +859,14 @@ function parseUserReviews(raw, url, what, sentiment) {
   };
 }
 function unwrapLink(link) {
-  if (!link) return null;
+  if (!link) {
+    return null;
+  }
   try {
     const target = new URL(link).searchParams.get("r");
-    if (target && /^https?:\/\//i.test(target)) return target;
+    if (target && ABSOLUTE_URL.test(target)) {
+      return target;
+    }
   } catch {
   }
   return link;
@@ -748,14 +874,22 @@ function unwrapLink(link) {
 function parseOffers(raw, url, what) {
   const data = envelope(raw, url, what);
   const item = data[FIELD.item];
-  if (!isObject(item)) throw parseFailure(url, "the response carries no offers object");
+  if (!isObject(item)) {
+    throw parseFailure(url, "the response carries no offers object");
+  }
   const offers = [];
   for (const [group, value] of Object.entries(item)) {
-    if (!Array.isArray(value)) continue;
+    if (!Array.isArray(value)) {
+      continue;
+    }
     for (const node of value) {
-      if (!isObject(node)) continue;
+      if (!isObject(node)) {
+        continue;
+      }
       const provider = str(node.providerName);
-      if (!provider) continue;
+      if (!provider) {
+        continue;
+      }
       offers.push({ provider, kind: group, url: unwrapLink(str(node.link)) });
     }
   }
@@ -763,8 +897,9 @@ function parseOffers(raw, url, what) {
 }
 
 // src/mc/client.ts
+var BROWSER_IDENTITY = /mozilla\/|applewebkit|chrome\/|safari\/|gecko/i;
 function withGuarantees(config) {
-  const userAgent = /mozilla\/|applewebkit|chrome\/|safari\/|gecko/i.test(config.userAgent) ? `${config.userAgent} ${DEFAULT_USER_AGENT}` : config.userAgent;
+  const userAgent = BROWSER_IDENTITY.test(config.userAgent) ? `${config.userAgent} ${DEFAULT_USER_AGENT}` : config.userAgent;
   return {
     ...config,
     userAgent,
@@ -806,7 +941,7 @@ var McClient = class {
    */
   async search(query, limit, offset) {
     const url = searchUrl(query, limit, offset);
-    return this.fetchParsed(
+    return await this.fetchParsed(
       url,
       this.catalogueCache,
       (body) => parseTitlePage(body, url, `the query "${query}"`)
@@ -814,7 +949,7 @@ var McClient = class {
   }
   async browse(options) {
     const url = browseUrl(options);
-    return this.fetchParsed(
+    return await this.fetchParsed(
       url,
       this.catalogueCache,
       (body) => parseTitlePage(body, url, `that ${options.kind} listing`)
@@ -823,7 +958,7 @@ var McClient = class {
   async getDetail(kind, slug) {
     const url = detailUrl(kind, slug);
     const clean = slug.trim();
-    return this.fetchParsed(
+    return await this.fetchParsed(
       url,
       this.catalogueCache,
       (body) => parseDetail(body, url, kind, clean)
@@ -831,7 +966,7 @@ var McClient = class {
   }
   async getScore(kind, slug, source) {
     const url = scoreUrl(kind, slug, source);
-    return this.fetchParsed(
+    return await this.fetchParsed(
       url,
       this.scoresCache,
       (body) => parseScore(body, url, `the ${source} score of ${kind} "${slug}"`)
@@ -839,7 +974,7 @@ var McClient = class {
   }
   async getCriticReviews(options) {
     const url = reviewsUrl({ ...options, source: "critic" });
-    return this.fetchParsed(
+    return await this.fetchParsed(
       url,
       this.scoresCache,
       (body) => parseCriticReviews(
@@ -853,7 +988,7 @@ var McClient = class {
   }
   async getUserReviews(options) {
     const url = reviewsUrl({ ...options, source: "user" });
-    return this.fetchParsed(
+    return await this.fetchParsed(
       url,
       this.scoresCache,
       (body) => parseUserReviews(
@@ -867,7 +1002,7 @@ var McClient = class {
   }
   async getWatchOffers(imdbId, kind) {
     const url = watchUrl(imdbId, kind);
-    return this.fetchParsed(
+    return await this.fetchParsed(
       url,
       this.scoresCache,
       (body) => parseOffers(body, url, `streaming offers for ${imdbId}`)
@@ -903,10 +1038,24 @@ import { z as z3 } from "zod";
 // src/tools/arguments.ts
 import { z } from "zod";
 var INVALID_INPUT = "invalid_input";
+var defaultMessage = z.core.locales.en().localeError;
 function strictInput(shape) {
   const declared = Object.keys(shape);
-  const errorMap = (issue, ctx) => issue.code === z.ZodIssueCode.unrecognized_keys ? { message: unknownArgumentMessage(issue.keys, declared) } : { message: ctx.defaultError };
-  return z.strictObject(shape, { errorMap });
+  const refuse = (issue) => issue.code === "unrecognized_keys" ? unknownArgumentMessage(issue.keys, declared) : `[${INVALID_INPUT}] ${defaultMessage(issue)}`;
+  for (const argument of Object.values(shape)) {
+    carryTheCode(argument, refuse);
+  }
+  return z.strictObject(shape, { error: refuse });
+}
+function carryTheCode(schema, refuse) {
+  const definition = schema._zod.def;
+  definition.error = refuse;
+  for (const check of definition.checks ?? []) {
+    check._zod.def.error = refuse;
+  }
+  if (definition.innerType) {
+    carryTheCode(definition.innerType, refuse);
+  }
 }
 function unknownArgumentMessage(keys, declared) {
   const named = keys.map((key) => {
@@ -918,15 +1067,21 @@ function unknownArgumentMessage(keys, declared) {
 function nearestArgument(key, declared) {
   const flatten = (name) => name.toLowerCase().replace(/[^a-z0-9]/g, "");
   const flat = flatten(key);
-  if (flat.length === 0) return void 0;
+  if (flat.length === 0) {
+    return void 0;
+  }
   const sameName = declared.find((name) => flatten(name) === flat);
-  if (sameName) return sameName;
+  if (sameName) {
+    return sameName;
+  }
   const overlapping = declared.find((name) => {
     const other = flatten(name);
     const [shorter, longer] = other.length < flat.length ? [other, flat] : [flat, other];
     return shorter.length >= 3 && (longer.startsWith(shorter) || longer.endsWith(shorter));
   });
-  if (overlapping) return overlapping;
+  if (overlapping) {
+    return overlapping;
+  }
   let closest;
   let shortest = Number.POSITIVE_INFINITY;
   for (const name of declared) {
@@ -940,15 +1095,25 @@ function nearestArgument(key, declared) {
 }
 function editDistance(left, right) {
   let previous = Array.from({ length: right.length + 1 }, (_, index) => index);
+  let distance = right.length;
   for (let row = 1; row <= left.length; row += 1) {
     const current = [row];
-    for (let column = 1; column <= right.length; column += 1) {
-      const substitution = previous[column - 1] + (left[row - 1] === right[column - 1] ? 0 : 1);
-      current[column] = Math.min(substitution, previous[column] + 1, current[column - 1] + 1);
+    let diagonal = row - 1;
+    let leftward = row;
+    for (const [column, above] of previous.entries()) {
+      if (column === 0) {
+        continue;
+      }
+      const substitution = diagonal + (left[row - 1] === right[column - 1] ? 0 : 1);
+      const cell = Math.min(substitution, above + 1, leftward + 1);
+      current.push(cell);
+      diagonal = above;
+      leftward = cell;
     }
     previous = current;
+    distance = leftward;
   }
-  return previous[right.length];
+  return distance;
 }
 
 // src/tools/shared.ts
@@ -1034,21 +1199,29 @@ ${trailer}`;
 function toToolError(error) {
   const known = error instanceof McError ? error : new McError("network_error", error instanceof Error ? error.message : String(error));
   const lines = [`[${known.code}] ${known.message}`];
-  if (known.details.hint) lines.push(`Hint: ${known.details.hint}`);
+  if (known.details.hint) {
+    lines.push(`Hint: ${known.details.hint}`);
+  }
   return { content: [{ type: "text", text: lines.join("\n") }], isError: true };
 }
 function truncate(text, maxChars) {
-  if (text.length <= maxChars) return text;
+  if (text.length <= maxChars) {
+    return text;
+  }
   return `${text.slice(0, maxChars - 1).trimEnd()}\u2026`;
 }
 function sliceAtLineBoundary(text, offset, maxChars) {
   const rest = text.slice(offset);
-  if (rest.length <= maxChars) return { slice: rest, nextOffset: null };
+  if (rest.length <= maxChars) {
+    return { slice: rest, nextOffset: null };
+  }
   const window = rest.slice(0, maxChars);
   const lastBreak = window.lastIndexOf("\n");
   let cut = lastBreak > 0 ? lastBreak : maxChars;
   const code = rest.charCodeAt(cut - 1);
-  if (code >= 55296 && code <= 56319) cut -= 1;
+  if (code >= 55296 && code <= 56319) {
+    cut -= 1;
+  }
   return { slice: rest.slice(0, cut), nextOffset: offset + cut };
 }
 function renderTitleList(titles) {
@@ -1070,6 +1243,15 @@ function renderTitleList(titles) {
 }
 
 // src/tools/browseTitles.ts
+function orderedBy(sort) {
+  if (sort === "score") {
+    return " by critic score";
+  }
+  if (sort === "recent") {
+    return " by release date";
+  }
+  return " by current popularity";
+}
 var browseTitlesDescription = [
   "List films, shows or games by score, by release date or by current popularity.",
   "Use this when there is no specific title to look up: best rated, newest, what people are looking at now.",
@@ -1110,7 +1292,9 @@ async function runBrowseTitles(client, args) {
     });
     const results = data.titles.map(toTitleSummaryOut);
     const notes = [];
-    if (cached) notes.push("Served from this server's short-lived in-memory cache.");
+    if (cached) {
+      notes.push("Served from this server's short-lived in-memory cache.");
+    }
     if (results.length === 0) {
       notes.push(
         args.genre ? `No ${args.kind} matched the genre "${args.genre}" at this offset. Check the spelling, or drop the genre.` : "The listing returned no rows at this offset."
@@ -1125,7 +1309,7 @@ async function runBrowseTitles(client, args) {
     const heading = [
       args.genre ? `${args.genre} ` : "",
       `${args.kind}s`,
-      args.sort === "score" ? " by critic score" : args.sort === "recent" ? " by release date" : " by current popularity"
+      orderedBy(args.sort)
     ].join("");
     const summary = results.length === 0 ? `No rows for ${heading}.` : `${results.length} of ${data.totalResults} ${heading}:
 ${renderTitleList(results)}`;
@@ -1211,7 +1395,9 @@ async function runGetReviews(client, args) {
       date: review.date
     }));
     const notes = [];
-    if (cached) notes.push("Served from this server's short-lived in-memory cache.");
+    if (cached) {
+      notes.push("Served from this server's short-lived in-memory cache.");
+    }
     if (reviews.length === 0) {
       if (args.offset > 0 && args.offset >= data.reviews.length) {
         notes.push(
@@ -1271,6 +1457,46 @@ ${listing}`;
 
 // src/tools/getTitle.ts
 import { z as z5 } from "zod";
+function notesOnWhatWasReadAndAsked(read) {
+  const { basic, wanted, slice, nextOffset, fullLength, offset, maxChars } = read;
+  const notes = [];
+  if (basic && nextOffset !== null) {
+    notes.push(
+      `The description is longer than ${maxChars} characters. Call again with offset=${nextOffset} for the rest.`
+    );
+  }
+  if (basic && slice === "" && offset > 0 && fullLength > 0) {
+    notes.push(
+      `offset=${offset} is past the end of a description of ${fullLength} characters. Call again with offset=0 to read it from the start.`
+    );
+  }
+  if (!wanted.has("scores")) {
+    notes.push(
+      "Scores were not requested, so critic_score and user_score are null here regardless of what Metacritic publishes. Add 'scores' to sections to read them."
+    );
+  }
+  return notes;
+}
+var PRODUCTION_CAP = 25;
+function attachAskedSections(structured, item, wanted, notes) {
+  if (wanted.has("awards")) {
+    structured.awards = item.awards;
+  }
+  if (wanted.has("networks")) {
+    structured.networks = item.networks;
+  }
+  if (!wanted.has("production")) {
+    return [];
+  }
+  const production = item.production.slice(0, PRODUCTION_CAP);
+  structured.production = production;
+  if (item.production.length > PRODUCTION_CAP) {
+    notes.push(
+      `${item.production.length} companies are credited and the first ${PRODUCTION_CAP} are shown. The list mixes producers with distributors and home-video labels across every territory.`
+    );
+  }
+  return production;
+}
 var SECTIONS = ["basic", "scores", "awards", "production", "networks", "where_to_watch"];
 var getTitleDescription = [
   "Read one Metacritic entry by slug and kind, both from search_titles.",
@@ -1318,7 +1544,9 @@ async function runGetTitle(client, args) {
     const wanted = new Set(args.sections);
     const notes = [];
     const detail = await client.getDetail(args.kind, args.slug);
-    if (detail.cached) notes.push("Served from this server's short-lived in-memory cache.");
+    if (detail.cached) {
+      notes.push("Served from this server's short-lived in-memory cache.");
+    }
     const item = detail.data;
     let criticScore = null;
     let userScore = null;
@@ -1331,21 +1559,17 @@ async function runGetTitle(client, args) {
     const basic = wanted.has("basic");
     const full = basic ? item.description ?? "" : "";
     const { slice, nextOffset } = sliceAtLineBoundary(full, args.offset, args.max_chars);
-    if (basic && nextOffset !== null) {
-      notes.push(
-        `The description is longer than ${args.max_chars} characters. Call again with offset=${nextOffset} for the rest.`
-      );
-    }
-    if (basic && slice === "" && args.offset > 0 && full.length > 0) {
-      notes.push(
-        `offset=${args.offset} is past the end of a description of ${full.length} characters. Call again with offset=0 to read it from the start.`
-      );
-    }
-    if (!wanted.has("scores")) {
-      notes.push(
-        "Scores were not requested, so critic_score and user_score are null here regardless of what Metacritic publishes. Add 'scores' to sections to read them."
-      );
-    }
+    notes.push(
+      ...notesOnWhatWasReadAndAsked({
+        basic,
+        wanted,
+        slice,
+        nextOffset,
+        fullLength: full.length,
+        offset: args.offset,
+        maxChars: args.max_chars
+      })
+    );
     const structured = {
       title: {
         ...toTitleSummaryOut(item),
@@ -1366,24 +1590,24 @@ async function runGetTitle(client, args) {
       user_score: userScore ? toScoreOut(userScore) : null,
       notes
     };
-    if (wanted.has("awards")) structured.awards = item.awards;
-    const PRODUCTION_CAP = 25;
-    const production = wanted.has("production") ? item.production.slice(0, PRODUCTION_CAP) : [];
-    if (wanted.has("production")) {
-      structured.production = production;
-      if (item.production.length > PRODUCTION_CAP) {
-        notes.push(
-          `${item.production.length} companies are credited and the first ${PRODUCTION_CAP} are shown. The list mixes producers with distributors and home-video labels across every territory.`
-        );
-      }
-    }
-    if (wanted.has("networks")) structured.networks = item.networks;
+    const production = attachAskedSections(structured, item, wanted, notes);
     const offers = wanted.has("where_to_watch") ? await watchOffers(client, item, args.kind, notes) : [];
-    if (wanted.has("where_to_watch")) structured.where_to_watch = offers;
-    return ok(structured, render(item, slice, criticScore, userScore, wanted, offers, production), {
-      notes,
-      sourceUrl: item.sourceUrl
-    });
+    if (wanted.has("where_to_watch")) {
+      structured.where_to_watch = offers;
+    }
+    return ok(
+      structured,
+      render(item, slice, wanted, {
+        critic: criticScore,
+        user: userScore,
+        offers,
+        production
+      }),
+      {
+        notes,
+        sourceUrl: item.sourceUrl
+      }
+    );
   } catch (error) {
     return toToolError(error);
   }
@@ -1421,35 +1645,25 @@ async function watchOffers(client, item, kind, notes) {
     return [];
   }
 }
-function render(item, description, critic, user, wanted, offers, production) {
-  const yearShown = item.year !== null && !item.title.includes(`(${item.year})`);
-  const header = [item.title, yearShown ? `(${item.year})` : "", `\xB7 ${item.kind}`].filter(Boolean).join(" ");
-  const lines = [header];
-  if (wanted.has("basic")) {
-    if (item.genres.length > 0) lines.push(`Genres: ${item.genres.join(", ")}`);
-    if (item.duration !== null) lines.push(`Runtime: ${item.duration} min`);
-  }
-  if (critic?.score !== null && critic !== null) {
-    lines.push(
-      `Critics: ${critic.score}/${critic.max} from ${critic.reviewCount ?? "?"} reviews${critic.sentiment ? ` (${critic.sentiment})` : ""}`
-    );
-  }
-  if (user?.score !== null && user !== null) {
-    lines.push(
-      `Users: ${user.score}/${user.max} from ${user.reviewCount ?? "?"} ratings${user.sentiment ? ` (${user.sentiment})` : ""}`
-    );
+function renderAwards(item, wanted) {
+  if (!wanted.has("awards") || item.awards.length === 0) {
+    return [];
   }
-  if (description) lines.push("", description);
-  if (wanted.has("awards") && item.awards.length > 0) {
-    lines.push("", "Awards:");
-    for (const a of item.awards) {
+  return [
+    "",
+    "Awards:",
+    ...item.awards.map((a) => {
       const tally = [
         a.wins === null ? "" : `${a.wins} win${a.wins === 1 ? "" : "s"}`,
         a.nominations === null ? "" : `${a.nominations} nomination${a.nominations === 1 ? "" : "s"}`
       ].filter(Boolean);
-      lines.push(`  ${a.ceremony}${tally.length > 0 ? `: ${tally.join(", ")}` : ""}`);
-    }
-  }
+      return `  ${a.ceremony}${tally.length > 0 ? `: ${tally.join(", ")}` : ""}`;
+    })
+  ];
+}
+function renderAskedSections(item, wanted, offers, production) {
+  const lines = [];
+  lines.push(...renderAwards(item, wanted));
   if (wanted.has("networks") && item.networks.length > 0) {
     lines.push("", `Networks: ${item.networks.join(", ")}`);
   }
@@ -1461,13 +1675,45 @@ function render(item, description, critic, user, wanted, offers, production) {
   }
   if (wanted.has("where_to_watch")) {
     lines.push("", offers.length === 0 ? "Where to watch: nothing listed." : "Where to watch:");
-    for (const o of offers) lines.push(`  ${o.provider} (${o.kind})${o.url ? ` \u2014 ${o.url}` : ""}`);
+    for (const o of offers) {
+      lines.push(`  ${o.provider} (${o.kind})${o.url ? ` \u2014 ${o.url}` : ""}`);
+    }
+  }
+  return lines;
+}
+function render(item, description, wanted, said) {
+  const { critic, user, offers, production } = said;
+  const yearShown = item.year !== null && !item.title.includes(`(${item.year})`);
+  const header = [item.title, yearShown ? `(${item.year})` : "", `\xB7 ${item.kind}`].filter(Boolean).join(" ");
+  const lines = [header];
+  if (wanted.has("basic")) {
+    if (item.genres.length > 0) {
+      lines.push(`Genres: ${item.genres.join(", ")}`);
+    }
+    if (item.duration !== null) {
+      lines.push(`Runtime: ${item.duration} min`);
+    }
+  }
+  if (critic?.score !== null && critic !== null) {
+    lines.push(
+      `Critics: ${critic.score}/${critic.max} from ${critic.reviewCount ?? "?"} reviews${critic.sentiment ? ` (${critic.sentiment})` : ""}`
+    );
   }
+  if (user?.score !== null && user !== null) {
+    lines.push(
+      `Users: ${user.score}/${user.max} from ${user.reviewCount ?? "?"} ratings${user.sentiment ? ` (${user.sentiment})` : ""}`
+    );
+  }
+  if (description) {
+    lines.push("", description);
+  }
+  lines.push(...renderAskedSections(item, wanted, offers, production));
   return lines.join("\n");
 }
 
 // src/tools/searchTitles.ts
 import { z as z6 } from "zod";
+var WHITESPACE = /\s/;
 var searchTitlesDescription = [
   "Search Metacritic for films, shows and games by title.",
   "Returns one compact row per match, carrying the critic Metascore, so a question about the critical verdict needs no second call.",
@@ -1497,8 +1743,10 @@ async function runSearchTitles(client, args) {
     const results = matching.slice(0, args.limit).map(toTitleSummaryOut);
     const windowExhausted = args.kind !== "any" && matching.length === results.length;
     const notes = [];
-    if (cached) notes.push("Served from this server's short-lived in-memory cache.");
-    const looseCount = /\s/.test(args.query.trim());
+    if (cached) {
+      notes.push("Served from this server's short-lived in-memory cache.");
+    }
+    const looseCount = WHITESPACE.test(args.query.trim());
     if (looseCount) {
       notes.push(
         "Metacritic counts a multi-word query loosely, so total_available is far larger than the number of real matches. The rows themselves are ordered by relevance."
@@ -1606,16 +1854,16 @@ async function main() {
   const transport = new StdioServerTransport();
   await server.connect(transport);
   const shutdown = () => {
-    void server.close().finally(() => process.exit(0));
+    server.close().catch(() => void 0).finally(() => process2.exit(0));
   };
-  process.on("SIGINT", shutdown);
-  process.on("SIGTERM", shutdown);
+  process2.on("SIGINT", shutdown);
+  process2.on("SIGTERM", shutdown);
 }
 main().catch((error) => {
-  process.stderr.write(
+  process2.stderr.write(
     `[mcp-metacritic] fatal: ${error instanceof Error ? error.stack ?? error.message : String(error)}
 `
   );
-  process.exit(1);
+  process2.exit(1);
 });
 //# sourceMappingURL=index.js.map