mcp-databnf 1.1.0 → 2.0.0

package/dist/index.js

@@ -1,13 +1,17 @@
 #!/usr/bin/env node
 
 // src/index.ts
+import process2 from "process";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 
 // src/server.ts
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 
+// src/config.ts
+import process from "process";
+
 // src/version.ts
-var PKG_VERSION = "1.1.0";
+var PKG_VERSION = "2.0.0";
 var REPO_URL = "https://github.com/smeet666/mcp-databnf";
 
 // src/config.ts
@@ -20,7 +24,9 @@ var DEFAULT_USER_AGENT = `mcp-databnf/${PKG_VERSION} (+${REPO_URL})`;
 function createLogger(level) {
   const rank = LOG_LEVELS.indexOf(level);
   const write = (at, message) => {
-    if (rank === 0 || LOG_LEVELS.indexOf(at) > rank) return;
+    if (rank === 0 || LOG_LEVELS.indexOf(at) > rank) {
+      return;
+    }
     process.stderr.write(`[mcp-databnf] ${at}: ${message}
 `);
   };
@@ -35,9 +41,11 @@ function createLogger(level) {
 }
 function readInteger(env, name, fallback, min, max) {
   const raw = env[name];
-  if (raw === void 0 || raw.trim() === "") return fallback;
+  if (raw === void 0 || raw.trim() === "") {
+    return fallback;
+  }
   const value = Number(raw);
-  if (!Number.isFinite(value) || !Number.isInteger(value)) {
+  if (!(Number.isFinite(value) && Number.isInteger(value))) {
     process.stderr.write(
       `[mcp-databnf] error: ${name}="${raw}" is not a whole number; using ${fallback}.
 `
@@ -88,7 +96,7 @@ var BnfError = class extends Error {
   code;
   details;
   constructor(code, message, details = {}) {
-    super(message);
+    super(message, details.cause === void 0 ? void 0 : { cause: details.cause });
     this.name = "BnfError";
     this.code = code;
     this.details = details;
@@ -118,7 +126,9 @@ var Cache = class {
   entries = /* @__PURE__ */ new Map();
   get(key) {
     const entry = this.entries.get(key);
-    if (!entry) return void 0;
+    if (!entry) {
+      return void 0;
+    }
     if (entry.expiresAt <= Date.now()) {
       this.entries.delete(key);
       return void 0;
@@ -128,12 +138,16 @@ var Cache = class {
     return { value: entry.value, retrievedAt: entry.retrievedAt };
   }
   set(key, value, retrievedAt) {
-    if (this.ttlMs <= 0) return;
+    if (this.ttlMs <= 0) {
+      return;
+    }
     this.entries.delete(key);
     this.entries.set(key, { value, expiresAt: Date.now() + this.ttlMs, retrievedAt });
     while (this.entries.size > this.maxEntries) {
       const oldest = this.entries.keys().next();
-      if (oldest.done) break;
+      if (oldest.done) {
+        break;
+      }
       this.entries.delete(oldest.value);
     }
   }
@@ -143,6 +157,9 @@ var Cache = class {
 };
 
 // src/bnf/endpoint.ts
+var TRAILING_DOT = /\.$/;
+var FRAGMENT = /#.*$/;
+var HTTP_SCHEME = /^http:/;
 var SPARQL_ENDPOINT = "https://data.bnf.fr/sparql";
 var ALLOWED_HOST = "data.bnf.fr";
 var GALLICA_HOST = "gallica.bnf.fr";
@@ -150,22 +167,23 @@ function assertRequestable(url) {
   let parsed;
   try {
     parsed = new URL(url);
-  } catch {
-    throw refusal(`mcp-databnf refuses to request "${url}": it is not an address.`);
+  } catch (cause) {
+    throw refusal(`mcp-databnf refuses to request "${url}": it is not an address.`, cause);
   }
-  const host = parsed.hostname.toLowerCase().replace(/\.$/, "");
+  const host = parsed.hostname.toLowerCase().replace(TRAILING_DOT, "");
   if (host !== ALLOWED_HOST) {
     throw refusal(
       `mcp-databnf requests ${ALLOWED_HOST} and nothing else; "${host}" was refused. Gallica in particular is described from the catalogue and never read.`
     );
   }
 }
-var refusal = (message) => new BnfError("network_error", message, {
-  hint: "This server reads the BnF catalogue at data.bnf.fr. It holds no licence to read the digitised contents, so it does not follow an address that leaves that host."
+var refusal = (message, cause) => new BnfError("network_error", message, {
+  hint: "This server reads the BnF catalogue at data.bnf.fr. It holds no licence to read the digitised contents, so it does not follow an address that leaves that host.",
+  cause
 });
 function isGallicaAddress(address) {
   try {
-    const host = new URL(address).hostname.toLowerCase().replace(/\.$/, "");
+    const host = new URL(address).hostname.toLowerCase().replace(TRAILING_DOT, "");
     return host === GALLICA_HOST || host.endsWith(`.${GALLICA_HOST}`);
   } catch {
     return false;
@@ -192,21 +210,28 @@ var PREFIXES = {
   rdafrbr: "http://rdvocab.info/uri/schema/FRBRentitiesRDA/"
 };
 var PREFIX_BLOCK = Object.entries(PREFIXES).map(([name, iri]) => `PREFIX ${name}: <${iri}>`).join("\n");
-var publicPageFor = (entityIri) => entityIri.replace(/#.*$/, "").replace(/^http:/, "https:");
+var publicPageFor = (entityIri) => entityIri.replace(FRAGMENT, "").replace(HTTP_SCHEME, "https:");
 
 // src/bnf/http.ts
+var VIRTUOSO_ERROR = /^Virtuoso\s+(\w+)\s+Error\s+([\s\S]*)$/;
 var RETRYABLE = /* @__PURE__ */ new Set([408, 425, 429, 500, 502, 503, 504]);
 var PUSH_BACK = /* @__PURE__ */ new Set([429, 503]);
 var LONGEST_WAIT_MS = 3e4;
 var RETRIES_AFTER_SILENCE = 1;
 var MAX_REDIRECTS = 2;
 function parseRetryAfter(value, now = Date.now()) {
-  if (!value) return null;
+  if (!value) {
+    return null;
+  }
   const trimmed = value.trim();
   const seconds = Number(trimmed);
-  if (Number.isFinite(seconds) && seconds >= 0) return Math.round(seconds * 1e3);
+  if (Number.isFinite(seconds) && seconds >= 0) {
+    return Math.round(seconds * 1e3);
+  }
   const at = Date.parse(trimmed);
-  if (Number.isNaN(at)) return null;
+  if (Number.isNaN(at)) {
+    return null;
+  }
   return Math.max(0, at - now);
 }
 function backoffMs(attempt) {
@@ -214,8 +239,10 @@ function backoffMs(attempt) {
   return base + Math.floor(Math.random() * 500);
 }
 function readEngineError(body) {
-  const match = /^Virtuoso\s+(\w+)\s+Error\s+([\s\S]*)$/.exec(body.trim());
-  if (!match) return null;
+  const match = VIRTUOSO_ERROR.exec(body.trim());
+  if (!match) {
+    return null;
+  }
   const state = match[1] ?? "";
   const text2 = (match[2] ?? "").split("\n")[0]?.trim() ?? "";
   return { kind: state === "37000" ? "compile" : "runtime", text: text2 };
@@ -257,6 +284,78 @@ async function postOnce(options, signal) {
     url: SPARQL_ENDPOINT
   });
 }
+function readRefusal(response, attempt, maxRetries) {
+  const pushBack = RETRYABLE.has(response.status);
+  if (PUSH_BACK.has(response.status)) {
+    const asked = parseRetryAfter(response.headers.get("retry-after"));
+    if (asked !== null && asked > LONGEST_WAIT_MS) {
+      return {
+        kind: "refused",
+        pushBack,
+        error: rateLimited(
+          `data.bnf.fr asked this client to wait ${Math.round(asked / 1e3)} seconds (HTTP ${response.status}).`,
+          { url: SPARQL_ENDPOINT, status: response.status }
+        )
+      };
+    }
+    if (attempt >= maxRetries) {
+      return {
+        kind: "refused",
+        pushBack,
+        error: rateLimited(
+          `data.bnf.fr asked this client to slow down (HTTP ${response.status}).`,
+          {
+            url: SPARQL_ENDPOINT,
+            status: response.status
+          }
+        )
+      };
+    }
+    return { kind: "again", pushBack, waitMs: asked ?? backoffMs(attempt) };
+  }
+  if (RETRYABLE.has(response.status) && attempt < maxRetries) {
+    return { kind: "again", pushBack, waitMs: backoffMs(attempt) };
+  }
+  if (response.status === 400 || response.status === 422) {
+    const engine = readEngineError(response.body);
+    return {
+      kind: "refused",
+      pushBack,
+      error: invalidInput(
+        `data.bnf.fr would not run this query${engine?.text ? `: ${engine.text}` : "."}`,
+        "Try fewer or plainer words. If this keeps happening with ordinary input, please report it."
+      )
+    };
+  }
+  return {
+    kind: "refused",
+    pushBack,
+    error: networkError(`data.bnf.fr answered HTTP ${response.status}.`, {
+      url: SPARQL_ENDPOINT,
+      status: response.status
+    })
+  };
+}
+function readFailure(error, attempts) {
+  const { attempt, maxRetries, timeoutMs } = attempts;
+  if (error instanceof Error && error.name === "BnfError") {
+    throw error;
+  }
+  if (error instanceof Error && error.name === "AbortError") {
+    if (attempt >= Math.min(maxRetries, RETRIES_AFTER_SILENCE)) {
+      throw timeout(
+        `No answer from data.bnf.fr within ${timeoutMs}ms. A query spanning the whole catalogue can take longer than the service will spend on it.`,
+        { url: SPARQL_ENDPOINT }
+      );
+    }
+    return error;
+  }
+  const failure = error instanceof Error ? error : new Error(String(error));
+  if (attempt >= maxRetries) {
+    throw networkError(`Could not reach data.bnf.fr: ${failure.message}`, { url: SPARQL_ENDPOINT });
+  }
+  return failure;
+}
 async function runQuery(options) {
   const { timeoutMs, maxRetries, limiter, logger } = options;
   let lastError = null;
@@ -264,7 +363,8 @@ async function runQuery(options) {
   for (let attempt = 0; attempt <= maxRetries; attempt += 1) {
     if (askedWaitMs > 0) {
       logger.debug(`waiting ${askedWaitMs}ms, as asked`);
-      await new Promise((resolve) => setTimeout(resolve, askedWaitMs));
+      const asked = askedWaitMs;
+      await new Promise((resolve) => setTimeout(resolve, asked));
       askedWaitMs = 0;
     }
     await limiter.beforeRequest();
@@ -284,66 +384,19 @@ async function runQuery(options) {
         limiter.succeeded();
         return parsed;
       }
-      if (RETRYABLE.has(response.status)) limiter.pushBack();
-      if (PUSH_BACK.has(response.status)) {
-        const asked = parseRetryAfter(response.headers.get("retry-after"));
-        if (asked !== null && asked > LONGEST_WAIT_MS) {
-          throw rateLimited(
-            `data.bnf.fr asked this client to wait ${Math.round(asked / 1e3)} seconds (HTTP ${response.status}).`,
-            { url: SPARQL_ENDPOINT, status: response.status }
-          );
-        }
-        if (attempt >= maxRetries) {
-          throw rateLimited(
-            `data.bnf.fr asked this client to slow down (HTTP ${response.status}).`,
-            {
-              url: SPARQL_ENDPOINT,
-              status: response.status
-            }
-          );
-        }
-        askedWaitMs = asked ?? backoffMs(attempt);
-        lastError = new Error(`HTTP ${response.status}`);
-        continue;
-      }
-      if (RETRYABLE.has(response.status) && attempt < maxRetries) {
-        lastError = new Error(`HTTP ${response.status}`);
-        askedWaitMs = backoffMs(attempt);
-        continue;
+      const verdict = readRefusal(response, attempt, maxRetries);
+      if (verdict.pushBack) {
+        limiter.pushBack();
       }
-      if (response.status === 400 || response.status === 422) {
-        const engine = readEngineError(response.body);
-        throw invalidInput(
-          `data.bnf.fr would not run this query${engine?.text ? `: ${engine.text}` : "."}`,
-          "Try fewer or plainer words. If this keeps happening with ordinary input, please report it."
-        );
+      if (verdict.kind === "refused") {
+        throw verdict.error;
       }
-      throw networkError(`data.bnf.fr answered HTTP ${response.status}.`, {
-        url: SPARQL_ENDPOINT,
-        status: response.status
-      });
+      askedWaitMs = verdict.waitMs;
+      lastError = new Error(`HTTP ${response.status}`);
     } catch (error) {
       clearTimeout(deadline);
-      if (error instanceof Error && error.name === "BnfError") throw error;
-      if (error instanceof Error && error.name === "AbortError") {
-        lastError = error;
-        if (attempt >= Math.min(maxRetries, RETRIES_AFTER_SILENCE)) {
-          throw timeout(
-            `No answer from data.bnf.fr within ${timeoutMs}ms. A query spanning the whole catalogue can take longer than the service will spend on it.`,
-            { url: SPARQL_ENDPOINT }
-          );
-        }
-        askedWaitMs = backoffMs(attempt);
-        continue;
-      }
-      lastError = error instanceof Error ? error : new Error(String(error));
-      if (attempt >= maxRetries) {
-        throw networkError(`Could not reach data.bnf.fr: ${lastError.message}`, {
-          url: SPARQL_ENDPOINT
-        });
-      }
+      lastError = readFailure(error, { attempt, maxRetries, timeoutMs });
       askedWaitMs = backoffMs(attempt);
-      continue;
     } finally {
       clearTimeout(deadline);
     }
@@ -377,9 +430,10 @@ function parseResults(body) {
   let payload;
   try {
     payload = JSON.parse(body);
-  } catch {
+  } catch (cause) {
     throw parseFailure("data.bnf.fr answered with something that is not JSON.", {
-      url: SPARQL_ENDPOINT
+      url: SPARQL_ENDPOINT,
+      cause
     });
   }
   if (typeof payload !== "object" || payload === null || !("results" in payload) || typeof payload.results !== "object" || !Array.isArray(payload.results?.bindings)) {
@@ -391,6 +445,16 @@ function parseResults(body) {
 }
 
 // src/bnf/sparql.ts
+var KEPT_CHARACTER = /[\p{L}\p{N}'\-\s]/u;
+var TYPOGRAPHIC_APOSTROPHE = /[‘’ʼ]/u;
+var LEADING_JOINERS = /^['-]+/u;
+var TRAILING_JOINERS = /['-]+$/u;
+var JOINERS = /['-]+/u;
+var ARK_IN_INPUT = /(?:^|\/)ark:\/12148\/(c[bc][0-9a-z]+)/i;
+var LEADING_HASH = /^#/;
+var FRAGMENT_OR_QUERY = /[#?]/;
+var TEMP_WORK_IN_INPUT = /temp-work\/([0-9a-f]{32})/i;
+var GALLICA_ARK = /ark:\/12148\/([0-9a-z]+)/i;
 function escapeLiteral(value) {
   return value.replace(/\\/g, "\\\\").replace(/"/g, '\\"').replace(/\n/g, "\\n").replace(/\r/g, "\\r").replace(/\t/g, "\\t").replace(/[\u0000-\u0008\u000b\u000c\u000e-\u001f\u007f]/gu, "");
 }
@@ -402,15 +466,21 @@ function withoutInvisibles(input) {
 function invisibleCharacters(input) {
   const found = [];
   for (const character of input.match(INVISIBLE) ?? []) {
-    if (!found.includes(character)) found.push(character);
+    if (!found.includes(character)) {
+      found.push(character);
+    }
   }
   return found;
 }
 function charactersSetAside(input) {
   const found = [];
   for (const character of withoutInvisibles(input).normalize("NFC")) {
-    if (/[\p{L}\p{N}'\-\s]/u.test(character) || /[‘’ʼ]/u.test(character)) continue;
-    if (!found.includes(character)) found.push(character);
+    if (KEPT_CHARACTER.test(character) || TYPOGRAPHIC_APOSTROPHE.test(character)) {
+      continue;
+    }
+    if (!found.includes(character)) {
+      found.push(character);
+    }
   }
   return found;
 }
@@ -427,18 +497,24 @@ function toSearchWords(input) {
   const cleaned = withoutInvisibles(input).normalize("NFC").replace(/[‘’ʼ]/gu, "'").replace(/[^\p{L}\p{N}'-]+/gu, " ");
   const words = [];
   for (const raw of cleaned.split(" ")) {
-    const word = raw.replace(/^['-]+/u, "").replace(/['-]+$/u, "");
-    if (word === "") continue;
+    const word = raw.replace(LEADING_JOINERS, "").replace(TRAILING_JOINERS, "");
+    if (word === "") {
+      continue;
+    }
     words.push(word.length > 60 ? word.slice(0, 60) : word);
-    if (words.length >= 12) break;
+    if (words.length >= 12) {
+      break;
+    }
   }
   return words;
 }
 function toIndexTerms(words) {
   const terms = [];
   for (const word of words) {
-    for (const piece of word.split(/['-]+/u)) {
-      if (piece !== "" && !terms.includes(piece)) terms.push(piece);
+    for (const piece of word.split(JOINERS)) {
+      if (piece !== "" && !terms.includes(piece)) {
+        terms.push(piece);
+      }
     }
   }
   return terms;
@@ -460,8 +536,8 @@ function parseEntityId(input) {
   if (trimmed === "") {
     throw invalidInput("An identifier is required.");
   }
-  const arkMatch = /(?:^|\/)ark:\/12148\/(c[bc][0-9a-z]+)/i.exec(trimmed);
-  const bare = arkMatch ? arkMatch[1] : trimmed.replace(/^#/, "").split(/[#?]/)[0];
+  const arkMatch = ARK_IN_INPUT.exec(trimmed);
+  const bare = arkMatch ? arkMatch[1] : trimmed.replace(LEADING_HASH, "").split(FRAGMENT_OR_QUERY)[0];
   const candidate = (bare ?? "").toLowerCase();
   if (ARK_ID.test(candidate)) {
     return {
@@ -471,7 +547,7 @@ function parseEntityId(input) {
       pageUrl: `https://data.bnf.fr/ark:/12148/${candidate}`
     };
   }
-  const tempMatch = /temp-work\/([0-9a-f]{32})/i.exec(trimmed);
+  const tempMatch = TEMP_WORK_IN_INPUT.exec(trimmed);
   const digest = (tempMatch?.[1] ?? candidate).toLowerCase();
   if (TEMP_WORK_ID.test(digest)) {
     return {
@@ -492,7 +568,7 @@ function hintFor(input) {
     const name = (page2[2] ?? "").replace(/_/g, " ");
     return `That is the readable address of a data.bnf.fr page, and ${page2[1]} in it is the record's FRBNF number rather than the identifier these tools take. Search for "${name}" with search_authors or search_works, and use the identifier the row carries.`;
   }
-  const gallicaArk = /ark:\/12148\/([0-9a-z]+)/i.exec(input)?.[1] ?? input;
+  const gallicaArk = GALLICA_ARK.exec(input)?.[1] ?? input;
   if (GALLICA_DOCUMENT.test(gallicaArk)) {
     return "That is a Gallica identifier: it names a digitised document rather than a record in the catalogue, and these tools read the catalogue. Search for the work with search_works, then list_editions will give you the Gallica link alongside the edition it belongs to.";
   }
@@ -500,7 +576,7 @@ function hintFor(input) {
 }
 var iriOf = (id) => `<${id.iri}>`;
 function boundedInteger(value, min, max, name) {
-  if (!Number.isFinite(value) || !Number.isInteger(value)) {
+  if (!(Number.isFinite(value) && Number.isInteger(value))) {
     throw invalidInput(`'${name}' has to be a whole number.`);
   }
   if (value < min || value > max) {
@@ -721,6 +797,14 @@ LIMIT 20`);
 }
 
 // src/bnf/parse.ts
+var ARK_IN_IRI = /ark:\/12148\/(c[bc][0-9a-z]+)/i;
+var TEMP_WORK_IN_IRI = /temp-work\/([0-9a-f]{32})/i;
+var TRAILING_SEPARATOR = /[/#]$/;
+var PATH_SEPARATOR = /[/#]/;
+var ARK_SEGMENT = /ark:\/12148\/([^/?#]+)/;
+var QUERY_START = /[?#]/;
+var LEADING_DOTS = /^[./]+/;
+var TRAILING_SLASHES = /\/+$/;
 var P = {
   type: "http://www.w3.org/1999/02/22-rdf-syntax-ns#type",
   label: "http://www.w3.org/2000/01/rdf-schema#label",
@@ -776,36 +860,50 @@ var ALIGNMENT_HOSTS = [
   { host: "catalogue.bnf.fr", name: "catalogue" }
 ];
 var text = (term) => {
-  if (!term) return null;
+  if (!term) {
+    return null;
+  }
   const value = term.value.trim();
   return value === "" ? null : value;
 };
 var integer = (term) => {
   const raw = text(term);
-  if (raw === null) return null;
+  if (raw === null) {
+    return null;
+  }
   const value = Number(raw);
   return Number.isInteger(value) ? value : null;
 };
 function idFromIri(iri) {
-  const ark = /ark:\/12148\/(c[bc][0-9a-z]+)/i.exec(iri);
-  if (ark?.[1]) return ark[1].toLowerCase();
-  const temp = /temp-work\/([0-9a-f]{32})/i.exec(iri);
-  if (temp?.[1]) return `temp-work/${temp[1].toLowerCase()}`;
+  const ark = ARK_IN_IRI.exec(iri);
+  if (ark?.[1]) {
+    return ark[1].toLowerCase();
+  }
+  const temp = TEMP_WORK_IN_IRI.exec(iri);
+  if (temp?.[1]) {
+    return `temp-work/${temp[1].toLowerCase()}`;
+  }
   return null;
 }
-var vocabularyTerm = (iri) => decodeURIComponent(iri.replace(/[/#]$/, "").split(/[/#]/).pop() ?? iri);
+var vocabularyTerm = (iri) => decodeURIComponent(iri.replace(TRAILING_SEPARATOR, "").split(PATH_SEPARATOR).pop() ?? iri);
 function toDigitisedLink(address, role, fromId, fromTitle) {
-  if (!address || !isGallicaAddress(address)) return null;
-  const segment = /ark:\/12148\/([^/?#]+)/.exec(address)?.[1] ?? null;
-  if (!segment) return null;
+  if (!(address && isGallicaAddress(address))) {
+    return null;
+  }
+  const segment = ARK_SEGMENT.exec(address)?.[1] ?? null;
+  if (!segment) {
+    return null;
+  }
   const ark = segment.split(".")[0] ?? segment;
-  if (ark === "") return null;
+  if (ark === "") {
+    return null;
+  }
   return { ark, url: address, rendering: renderingOf(address, ark), role, fromId, fromTitle };
 }
 function renderingOf(address, ark) {
-  const path = address.split(/[?#]/)[0] ?? address;
+  const path = address.split(QUERY_START)[0] ?? address;
   const after = path.slice(path.indexOf(`ark:/12148/${ark}`) + `ark:/12148/${ark}`.length);
-  const rendering = after.replace(/^[./]+/, "").replace(/\/+$/, "");
+  const rendering = after.replace(LEADING_DOTS, "").replace(TRAILING_SLASHES, "");
   return rendering === "" ? null : rendering;
 }
 function page(rows, limit, skipped) {
@@ -815,9 +913,13 @@ function page(rows, limit, skipped) {
 function readWindowOccupancy(results) {
   for (const row of results.results.bindings) {
     const raw = text(row.windowRows);
-    if (raw === null) continue;
+    if (raw === null) {
+      continue;
+    }
     const count = Number(raw);
-    if (Number.isInteger(count)) return count >= TEXT_WINDOW;
+    if (Number.isInteger(count)) {
+      return count >= TEXT_WINDOW;
+    }
   }
   return null;
 }
@@ -826,10 +928,15 @@ function byPredicate(results) {
   const index = /* @__PURE__ */ new Map();
   for (const row of results.results.bindings) {
     const predicate = text(row.p);
-    if (predicate === null) continue;
+    if (predicate === null) {
+      continue;
+    }
     const bucket = index.get(predicate);
-    if (bucket) bucket.push(row);
-    else index.set(predicate, [row]);
+    if (bucket) {
+      bucket.push(row);
+    } else {
+      index.set(predicate, [row]);
+    }
   }
   return index;
 }
@@ -838,7 +945,9 @@ var allValues = (index, predicate) => {
   const seen = /* @__PURE__ */ new Set();
   for (const row of index.get(predicate) ?? []) {
     const value = text(row.o);
-    if (value !== null) seen.add(value);
+    if (value !== null) {
+      seen.add(value);
+    }
   }
   return [...seen];
 };
@@ -849,8 +958,12 @@ function alignments(index, selfId) {
   const found = {};
   for (const predicate of [P.sameAs, P.exactMatch, P.closeMatch, P.seeAlso]) {
     for (const address of allValues(index, predicate)) {
-      if (!address.startsWith("http")) continue;
-      if (selfId && address.includes(selfId)) continue;
+      if (!address.startsWith("http")) {
+        continue;
+      }
+      if (selfId && address.includes(selfId)) {
+        continue;
+      }
       let host;
       try {
         host = new URL(address).hostname.toLowerCase();
@@ -860,8 +973,12 @@ function alignments(index, selfId) {
       const known = ALIGNMENT_HOSTS.find(
         (entry) => host === entry.host || host.endsWith(`.${entry.host}`)
       );
-      if (!known) continue;
-      (found[known.name] ??= /* @__PURE__ */ new Set()).add(address);
+      if (!known) {
+        continue;
+      }
+      const addresses = found[known.name] ?? /* @__PURE__ */ new Set();
+      found[known.name] = addresses;
+      addresses.add(address);
     }
   }
   return Object.fromEntries(Object.entries(found).map(([name, set]) => [name, [...set]]));
@@ -876,14 +993,18 @@ function toAuthorSummaries(results, limit) {
   const seen = /* @__PURE__ */ new Set();
   let skipped = 0;
   for (const row of results.results.bindings) {
-    if (isOccupancyRow(row, "person")) continue;
+    if (isOccupancyRow(row, "person")) {
+      continue;
+    }
     const iri = text(row.person);
     const id = iri === null ? null : idFromIri(iri);
     if (iri === null || id === null) {
       skipped += 1;
       continue;
     }
-    if (seen.has(id)) continue;
+    if (seen.has(id)) {
+      continue;
+    }
     seen.add(id);
     rows.push({
       id,
@@ -904,18 +1025,24 @@ function toAuthorDetail(results, id, pageUrl) {
   const seenLabels = /* @__PURE__ */ new Set();
   for (const row of index.get(P.altLabel) ?? []) {
     const label = text(row.o);
-    if (label === null || seenLabels.has(label)) continue;
+    if (label === null || seenLabels.has(label)) {
+      continue;
+    }
     seenLabels.add(label);
     otherNames.push({ label, language: text(row.lang) });
   }
   const sameAs = alignments(index, id);
   const isni = isniAddresses(index);
-  if (isni.length > 0) sameAs.isni = [.../* @__PURE__ */ new Set([...sameAs.isni ?? [], ...isni])];
+  if (isni.length > 0) {
+    sameAs.isni = [.../* @__PURE__ */ new Set([...sameAs.isni ?? [], ...isni])];
+  }
   const catalogueUrl = allValues(index, P.seeAlso).find((address) => address.includes("catalogue.bnf.fr")) ?? null;
   const depictions = [];
   for (const address of allValues(index, P.depiction)) {
     const link = toDigitisedLink(address, "depiction", id, null);
-    if (link) depictions.push(link);
+    if (link) {
+      depictions.push(link);
+    }
   }
   return {
     id,
@@ -949,8 +1076,12 @@ function toAuthorDetail(results, id, pageUrl) {
 function readStatus(statement, iri) {
   if (statement !== null) {
     const said = statement.toLowerCase();
-    if (said.includes("provisional")) return "provisional";
-    if (said.includes("established")) return "established";
+    if (said.includes("provisional")) {
+      return "provisional";
+    }
+    if (said.includes("established")) {
+      return "established";
+    }
   }
   return iri.includes("/temp-work/") ? "provisional" : "established";
 }
@@ -959,7 +1090,9 @@ function toWorkSummaries(results, limit) {
   const byId = /* @__PURE__ */ new Map();
   let skipped = 0;
   for (const row of results.results.bindings) {
-    if (isOccupancyRow(row, "work")) continue;
+    if (isOccupancyRow(row, "work")) {
+      continue;
+    }
     const iri = text(row.work);
     const id = iri === null ? null : idFromIri(iri);
     if (iri === null || id === null) {
@@ -987,7 +1120,7 @@ function toWorkSummaries(results, limit) {
   }
   return {
     ...page(
-      order.map((id) => byId.get(id)),
+      order.map((id) => byId.get(id)).filter((one) => one !== void 0),
       limit,
       skipped
     ),
@@ -1020,13 +1153,15 @@ function toAuthoredWorks(results, limit) {
       order.push(id);
     }
     const form = text(row.form);
-    if (form !== null && form.startsWith("http")) {
+    if (form?.startsWith("http")) {
       const term = vocabularyTerm(form);
-      if (!work.forms.includes(term)) work.forms.push(term);
+      if (!work.forms.includes(term)) {
+        work.forms.push(term);
+      }
     }
   }
   return page(
-    order.map((id) => byId.get(id)),
+    order.map((id) => byId.get(id)).filter((one) => one !== void 0),
     limit,
     skipped
   );
@@ -1039,13 +1174,17 @@ function toWorkDetail(results, id, iri, pageUrl) {
   for (const row of index.get(P.creator) ?? []) {
     const creatorIri = text(row.o);
     const creatorId = creatorIri ? idFromIri(creatorIri) : null;
-    if (!creatorId || creators.some((c) => c.id === creatorId)) continue;
+    if (!creatorId || creators.some((c) => c.id === creatorId)) {
+      continue;
+    }
     creators.push({ id: creatorId, name: text(row.name) });
   }
   const depictions = [];
   for (const address of allValues(index, P.depiction)) {
     const link = toDigitisedLink(address, "depiction", id, null);
-    if (link) depictions.push(link);
+    if (link) {
+      depictions.push(link);
+    }
   }
   const statusStatement = firstValue(index, P.statusOfIdentification);
   const catalogueUrl = allValues(index, P.seeAlso).find((address) => address.includes("catalogue.bnf.fr")) ?? null;
@@ -1115,7 +1254,7 @@ function toEditions(results, limit) {
     }
   }
   return page(
-    order.map((id) => byId.get(id)),
+    order.map((id) => byId.get(id)).filter((one) => one !== void 0),
     limit,
     skipped
   );
@@ -1134,7 +1273,9 @@ function toDigitisedLinks(results, limit) {
       ["depiction", "depiction"]
     ]) {
       const link = toDigitisedLink(text(row[column]), role, fromId, title);
-      if (!link || seen.has(link.url)) continue;
+      if (!link || seen.has(link.url)) {
+        continue;
+      }
       seen.add(link.url);
       links.push(link);
     }
@@ -1145,7 +1286,9 @@ function toTypes(results) {
   const types = /* @__PURE__ */ new Set();
   for (const row of results.results.bindings) {
     const iri = text(row.type);
-    if (iri !== null) types.add(iri);
+    if (iri !== null) {
+      types.add(iri);
+    }
   }
   return [...types];
 }
@@ -1186,7 +1329,9 @@ var RateLimiter = class {
   async beforeRequest() {
     const elapsed = Date.now() - this.lastStartedAt;
     const wait = Math.min(this.intervalMs, this.intervalMs - elapsed);
-    if (wait > 0) await sleep(wait);
+    if (wait > 0) {
+      await sleep(wait);
+    }
     this.lastStartedAt = Date.now();
   }
   /** Called when the service asks for room: double the gap, up to the ceiling. */
@@ -1199,9 +1344,13 @@ var RateLimiter = class {
    * response after a rough patch does not undo the caution that earned it.
    */
   succeeded() {
-    if (this.intervalMs === this.baseIntervalMs) return;
+    if (this.intervalMs === this.baseIntervalMs) {
+      return;
+    }
     this.calmStreak += 1;
-    if (this.calmStreak < 3) return;
+    if (this.calmStreak < 3) {
+      return;
+    }
     this.calmStreak = 0;
     this.intervalMs = Math.max(this.baseIntervalMs, Math.round(this.intervalMs / 2));
   }
@@ -1220,7 +1369,9 @@ function withGuarantees(config) {
   const defaults = loadConfig({});
   const bounded = (value, fallback, min, max) => {
     const parsed = Number(value);
-    if (!Number.isFinite(parsed)) return fallback;
+    if (!Number.isFinite(parsed)) {
+      return fallback;
+    }
     return Math.min(max, Math.max(min, parsed));
   };
   const claimed = typeof config.userAgent === "string" ? config.userAgent.trim() : "";
@@ -1271,9 +1422,6 @@ var BnfClient = class {
     return this.config.timeoutMs;
   }
   /** The one address this client sends anything to. */
-  get endpoint() {
-    return SPARQL_ENDPOINT;
-  }
   /**
    * `cacheKey` is stated rather than taken from the query, because two reads of
    * one query can produce different values: a list cut to five rows is not the
@@ -1317,32 +1465,7 @@ var BnfClient = class {
     return { data, cached: false, retrievedAt: new Date(retrievedAt).toISOString() };
   }
   /** Read an identifier a caller wrote, refusing anything this dataset cannot address. */
-  identify(input) {
-    return parseEntityId(input);
-  }
   /** The words a search will actually look for, which a tool reports back. */
-  words(input) {
-    return toSearchWords(input);
-  }
-  /**
-   * The terms the index requires, which is what a row's presence rests on.
-   *
-   * The index splits inside a word at an apostrophe and at a hyphen, so a word
-   * a caller wrote as one can reach it as two, each required on its own.
-   */
-  indexTerms(words) {
-    return toIndexTerms(words);
-  }
-  /**
-   * How a piece of caller text became the terms the search was run with.
-   *
-   * A tool reports the reading back, so the same reading has to be the one the
-   * query was built from: the words, the terms the index makes of them, and
-   * the characters that reached neither.
-   */
-  searchReading(input) {
-    return readSearchText(input);
-  }
   searchAuthors(name, limit, offset) {
     const words = toSearchWords(name);
     const query = searchAuthorsQuery(words, limit, offset);
@@ -1457,11 +1580,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);
-  return z.strictObject(shape, {
-    error: (issue) => issue.code === "unrecognized_keys" ? unknownArgumentMessage(issue.keys, declared) : void 0
-  });
+  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) => {
@@ -1473,15 +1609,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) {
@@ -1495,24 +1637,37 @@ 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
 import { z as z2 } from "zod";
+var ISO_DAY = /^\d{4}-\d{2}-\d{2}$/;
+var BRACKETED_TAIL = /\(([^()]*)\)\s*$/;
+var YEAR = /^\d{4}$/;
 var MAX_TEXT_CHARS = 2400;
 var SOURCE_NAME = "data.bnf.fr (Biblioth\xE8que nationale de France)";
 function retrievalDay(retrievedAt) {
   const day = retrievedAt.slice(0, 10);
-  return /^\d{4}-\d{2}-\d{2}$/.test(day) ? day : retrievedAt;
+  return ISO_DAY.test(day) ? day : retrievedAt;
 }
 function attribution(retrievedAt, sourceUrl) {
   const credit = `Source: ${SOURCE_NAME}, retrieved ${retrievalDay(retrievedAt)}`;
@@ -1544,7 +1699,9 @@ function onOneLine(value) {
 function ok(structured, body, options) {
   const credit = attribution(options.retrievedAt, options.sourceUrl);
   const noteLines = (options.notes ?? []).map((note) => `Note: ${onOneLine(note)}`);
-  while (noteLines.length > 0 && noteLines.join("\n").length > MAX_TEXT_CHARS / 2) noteLines.pop();
+  while (noteLines.length > 0 && noteLines.join("\n").length > MAX_TEXT_CHARS / 2) {
+    noteLines.pop();
+  }
   const trailer = [...noteLines, credit].join("\n");
   const cut = "\n\n[shortened; the full result is in the structured output]";
   const budget = MAX_TEXT_CHARS - `
@@ -1561,11 +1718,15 @@ ${trailer}`;
 function toToolError(error) {
   const known = error instanceof BnfError ? error : new BnfError("network_error", error instanceof Error ? error.message : String(error));
   const lines = [`[${known.code}] ${onOneLine(known.message)}`];
-  if (known.details.hint) lines.push(`Hint: ${onOneLine(known.details.hint)}`);
+  if (known.details.hint) {
+    lines.push(`Hint: ${onOneLine(known.details.hint)}`);
+  }
   return { content: [{ type: "text", text: lines.join("\n") }], isError: true };
 }
 function truncate(text2, maxChars) {
-  if (text2.length <= maxChars) return text2;
+  if (text2.length <= maxChars) {
+    return text2;
+  }
   return `${text2.slice(0, Math.max(0, maxChars - 1)).trimEnd()}\u2026`;
 }
 var WINDOW_FULL_CAVEAT = "The window this search reads off the index came back full, so records carrying these words sit past what was read and this list is a part of the matches. Which records fill a full window is settled by the index as it answers, so another reading of the same search can bring back other rows. Requiring one more word narrows the reading and reaches what a full window leaves out.";
@@ -1596,7 +1757,9 @@ function readingNotes(reading, field) {
 }
 var NO_RANKING = "The full-text index answers whether a record matches, and it does not score how well. Rows are ordered by the address of the record, which measures nothing, so the row a person would call the obvious answer can sit anywhere in the list, or on a later page. That order is the one the pages are cut along, so reading page after page reaches every match once.";
 async function classifyEmptyPage(page2, firstPageHasRow) {
-  if (page2 === 1) return "absent";
+  if (page2 === 1) {
+    return "absent";
+  }
   try {
     return await firstPageHasRow() ? "past_the_end" : "absent";
   } catch {
@@ -1609,18 +1772,24 @@ var WORK_TYPES = [
   "http://rdaregistry.info/Elements/c/#C10001"
 ];
 function recordKindOf(types) {
-  if (types.includes(PERSON_TYPE)) return "person";
-  if (types.some((type) => WORK_TYPES.includes(type))) return "work";
+  if (types.includes(PERSON_TYPE)) {
+    return "person";
+  }
+  if (types.some((type) => WORK_TYPES.includes(type))) {
+    return "work";
+  }
   return null;
 }
 function headingYears(label) {
-  const bracketed = /\(([^()]*)\)\s*$/.exec(label);
+  const bracketed = BRACKETED_TAIL.exec(label);
   const [birth, death] = (bracketed?.[1] ?? "").split("-");
-  const year = (part) => part !== void 0 && /^\d{4}$/.test(part.trim()) ? Number(part.trim()) : null;
+  const year = (part) => part !== void 0 && YEAR.test(part.trim()) ? Number(part.trim()) : null;
   return { birth: year(birth), death: year(death) };
 }
 function headingYearConflicts(label, birthYear, deathYear) {
-  if (label === null) return [];
+  if (label === null) {
+    return [];
+  }
   const stated = headingYears(label);
   const conflicts = [];
   if (stated.birth !== null && birthYear !== null && stated.birth !== birthYear) {
@@ -1636,6 +1805,37 @@ var ALIGNMENTS_CAVEAT = "The addresses under 'same_as' are the BnF's own alignme
 var PROVISIONAL_CAVEAT = "A provisional record is one the BnF created to hold a title it has catalogued and not yet established as a work of its own. Its identifier can change, and it can be merged into an established record.";
 
 // src/tools/findDigitised.ts
+async function whichKindTheCatalogueDescribes(client, id) {
+  if (id.kind === "temp-work") {
+    return { kind: "work" };
+  }
+  const types = await client.types(id);
+  if (types.data.length === 0) {
+    return {
+      error: toToolError(
+        notFound(`data.bnf.fr describes nothing at "${id.id}".`, {
+          hint: "Find the identifier with search_authors or search_works rather than writing one.",
+          url: id.pageUrl
+        })
+      )
+    };
+  }
+  const found = recordKindOf(types.data);
+  if (found === null) {
+    return {
+      error: toToolError(
+        notFound(
+          `"${id.id}" is described by data.bnf.fr, and it is neither a person nor a work: it is typed ${types.data.join(", ")}.`,
+          {
+            hint: "This tool follows a person or a work. For one edition, list_editions carries the digitised copies alongside the imprint.",
+            url: id.pageUrl
+          }
+        )
+      )
+    };
+  }
+  return { kind: found };
+}
 var findDigitisedDescription = [
   "Gather the digitised documents the Biblioth\xE8que nationale de France catalogue attaches to one person or one work, and return them as links.",
   "Give the identifier of either; the tool asks the catalogue what kind of record it is and follows the right path. For a work it walks the editions; for a person it takes the images on the record and the digitised editions of the works they are credited with. The 'kind' returned is what the catalogue types the record as.",
@@ -1669,35 +1869,13 @@ var findDigitisedOutput = z3.object({
 });
 async function runFindDigitised(client, args) {
   try {
-    const id = client.identify(args.id);
+    const id = parseEntityId(args.id);
     const notes = [];
-    let kind;
-    if (id.kind === "temp-work") {
-      kind = "work";
-    } else {
-      const types = await client.types(id);
-      if (types.data.length === 0) {
-        return toToolError(
-          notFound(`data.bnf.fr describes nothing at "${id.id}".`, {
-            hint: "Find the identifier with search_authors or search_works rather than writing one.",
-            url: id.pageUrl
-          })
-        );
-      }
-      const found = recordKindOf(types.data);
-      if (found === null) {
-        return toToolError(
-          notFound(
-            `"${id.id}" is described by data.bnf.fr, and it is neither a person nor a work: it is typed ${types.data.join(", ")}.`,
-            {
-              hint: "This tool follows a person or a work. For one edition, list_editions carries the digitised copies alongside the imprint.",
-              url: id.pageUrl
-            }
-          )
-        );
-      }
-      kind = found;
+    const settled = await whichKindTheCatalogueDescribes(client, id);
+    if ("error" in settled) {
+      return settled.error;
     }
+    const kind = settled.kind;
     if (args.kind !== "auto" && args.kind !== kind) {
       return toToolError(
         invalidInput(
@@ -1707,7 +1885,9 @@ async function runFindDigitised(client, args) {
       );
     }
     const { data, cached, retrievedAt } = kind === "person" ? await client.digitisedForPerson(id, args.limit) : await client.digitisedForWork(id, args.limit);
-    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.");
+    }
     const links = data.rows.map((link) => ({
       ark: link.ark,
       url: link.url,
@@ -1722,7 +1902,9 @@ async function runFindDigitised(client, args) {
       depiction: countRole(data.rows, "depiction")
     };
     const rendered = links.filter((link) => link.rendering !== null);
-    if (links.length > 0) notes.push(GALLICA_CAVEAT);
+    if (links.length > 0) {
+      notes.push(GALLICA_CAVEAT);
+    }
     if (counts.depiction > 0) {
       notes.push(
         `${counts.depiction === 1 ? "One of these is an image" : `${counts.depiction} of these are images`} the catalogue uses to illustrate the record. Such an image can be a portrait, a title page, or a newspaper page that mentions the subject in passing, and the catalogue does not say which.`
@@ -1777,6 +1959,45 @@ var countRole = (links, role) => links.filter((link) => link.role === role).leng
 
 // src/tools/getAuthor.ts
 import { z as z4 } from "zod";
+function notesOnWhatTheRecordStates(data, args) {
+  const notes = [];
+  if (data.biographicalInformation !== null) {
+    notes.push(
+      `The BnF states "${data.biographicalInformation}" as this person's biographical information. That field holds an occupation rather than a life, and this is all the catalogue says.`
+    );
+  }
+  if (data.biographicalInformation !== null && data.biographicalInformation === data.occupation) {
+    notes.push(
+      "This record states the same text under 'biographical_information' and 'occupation', character for character. It is one statement written in two fields rather than two the record makes separately."
+    );
+  }
+  const headings = [data.label, ...data.otherNames.map((other) => other.label)];
+  const conflicts = [
+    ...new Set(
+      headings.flatMap((heading) => headingYearConflicts(heading, data.birthYear, data.deathYear))
+    )
+  ];
+  if (conflicts.length > 0) {
+    notes.push(
+      `This record states its dates twice and the two disagree: the heading says ${conflicts.join(", and ")}. Both are passed on as the BnF publishes them, and the record says nothing about which is right, so read source_url before citing either.`
+    );
+  }
+  if (data.deathDate === null && data.deathYear === null) {
+    const dated = data.birthDate !== null || data.birthYear !== null;
+    notes.push(
+      dated ? "The record states no date of death. That can mean the person is living, or that the BnF has not recorded one; the catalogue does not distinguish the two." : `This record states no date of birth and no date of death, so it dates the person nowhere and says nothing about whether they are living. ${SEVERAL_RECORDS_CAVEAT}`
+    );
+  }
+  if (Object.keys(data.sameAs).length > 0) {
+    notes.push(
+      `${ALIGNMENTS_CAVEAT} Following one is how to reach a biography, which this catalogue does not hold.`
+    );
+  }
+  if (args.include_depictions && data.depictions.length > 0) {
+    notes.push(GALLICA_CAVEAT);
+  }
+  return notes;
+}
 var getAuthorDescription = [
   "Read one person's record in the Biblioth\xE8que nationale de France authority file, by the identifier search_authors returns.",
   "It carries the dates and places of birth and death, the occupation the record names, the language and country it associates with the person, the field of activity with its Dewey class, and the addresses for the same person in VIAF, IdRef, DBpedia, Wikidata and ISNI.",
@@ -1831,45 +2052,13 @@ var getAuthorOutput = z4.object({
 });
 async function runGetAuthor(client, args) {
   try {
-    const id = client.identify(args.author_id);
+    const id = parseEntityId(args.author_id);
     const { data, cached, retrievedAt } = await client.getAuthor(id);
     const notes = [];
-    if (cached) notes.push("Served from this server's short-lived in-memory cache.");
-    if (data.biographicalInformation !== null) {
-      notes.push(
-        `The BnF states "${data.biographicalInformation}" as this person's biographical information. That field holds an occupation rather than a life, and this is all the catalogue says.`
-      );
+    if (cached) {
+      notes.push("Served from this server's short-lived in-memory cache.");
     }
-    if (data.biographicalInformation !== null && data.biographicalInformation === data.occupation) {
-      notes.push(
-        "This record states the same text under 'biographical_information' and 'occupation', character for character. It is one statement written in two fields rather than two the record makes separately."
-      );
-    }
-    const headings = [data.label, ...data.otherNames.map((other) => other.label)];
-    const conflicts = [
-      ...new Set(
-        headings.flatMap(
-          (heading) => headingYearConflicts(heading, data.birthYear, data.deathYear)
-        )
-      )
-    ];
-    if (conflicts.length > 0) {
-      notes.push(
-        `This record states its dates twice and the two disagree: the heading says ${conflicts.join(", and ")}. Both are passed on as the BnF publishes them, and the record says nothing about which is right, so read source_url before citing either.`
-      );
-    }
-    if (data.deathDate === null && data.deathYear === null) {
-      const dated = data.birthDate !== null || data.birthYear !== null;
-      notes.push(
-        dated ? "The record states no date of death. That can mean the person is living, or that the BnF has not recorded one; the catalogue does not distinguish the two." : `This record states no date of birth and no date of death, so it dates the person nowhere and says nothing about whether they are living. ${SEVERAL_RECORDS_CAVEAT}`
-      );
-    }
-    if (Object.keys(data.sameAs).length > 0) {
-      notes.push(
-        `${ALIGNMENTS_CAVEAT} Following one is how to reach a biography, which this catalogue does not hold.`
-      );
-    }
-    if (args.include_depictions && data.depictions.length > 0) notes.push(GALLICA_CAVEAT);
+    notes.push(...notesOnWhatTheRecordStates(data, args));
     const structured = {
       author: {
         id: data.id,
@@ -1911,11 +2100,11 @@ async function runGetAuthor(client, args) {
       }));
     }
     const born = [
-      data.birthDate ?? (data.birthYear !== null ? String(data.birthYear) : null),
+      data.birthDate ?? (data.birthYear === null ? null : String(data.birthYear)),
       data.birthPlace
     ].filter(Boolean).join(", ");
     const died = [
-      data.deathDate ?? (data.deathYear !== null ? String(data.deathYear) : null),
+      data.deathDate ?? (data.deathYear === null ? null : String(data.deathYear)),
       data.deathPlace
     ].filter(Boolean).join(", ");
     const lines = [
@@ -1939,6 +2128,43 @@ async function runGetAuthor(client, args) {
 
 // src/tools/getWork.ts
 import { z as z5 } from "zod";
+function notesOnWhatTheRecordStates2(data, args) {
+  const notes = [];
+  if (data.status === "provisional") {
+    notes.push(
+      `This is a provisional record, addressed under 'temp-work' rather than by an ARK${data.statusStatement ? ` and stated as "${data.statusStatement}"` : ""}. ${PROVISIONAL_CAVEAT}`
+    );
+  } else {
+    notes.push(
+      `This is an established work record, addressed by the ARK '${data.id}'${data.statusStatement ? ` and stated as "${data.statusStatement}"` : ""}. That identifier is the one to cite.`
+    );
+  }
+  if (data.truncated) {
+    notes.push(
+      "This record is longer than one query reads, so parts of it are missing from this answer. A field shown as empty here may be filled on the record itself: open source_url before concluding that the BnF states nothing."
+    );
+  }
+  if (data.expressionCount !== null && data.expressionCount > 0) {
+    notes.push(
+      `The record links ${data.expressionCount} expression(s): translations, adaptations and recordings of this work. Published editions are a separate count, and list_editions is what reads them.`
+    );
+  }
+  if (data.forms.length > 0) {
+    notes.push(FORMS_CAVEAT);
+  }
+  if (data.creators.length === 0) {
+    notes.push(
+      "The record credits nobody with this work. That is normal for anonymous and traditional works, and it also happens on records a cataloguer has not finished."
+    );
+  }
+  if (Object.keys(data.sameAs).length > 0) {
+    notes.push(ALIGNMENTS_CAVEAT);
+  }
+  if (args.include_depictions && data.depictions.length > 0) {
+    notes.push(GALLICA_CAVEAT);
+  }
+  return notes;
+}
 var getWorkDescription = [
   "Read one work's record in the Biblioth\xE8que nationale de France catalogue, by the identifier search_works returns.",
   "It carries the title, everyone the record credits, the date the BnF gives the work, its language, its form and its subject with the Dewey class.",
@@ -1988,37 +2214,13 @@ var getWorkOutput = z5.object({
 });
 async function runGetWork(client, args) {
   try {
-    const id = client.identify(args.work_id);
+    const id = parseEntityId(args.work_id);
     const { data, cached, retrievedAt } = await client.getWork(id);
     const notes = [];
-    if (cached) notes.push("Served from this server's short-lived in-memory cache.");
-    if (data.status === "provisional") {
-      notes.push(
-        `This is a provisional record, addressed under 'temp-work' rather than by an ARK${data.statusStatement ? ` and stated as "${data.statusStatement}"` : ""}. ${PROVISIONAL_CAVEAT}`
-      );
-    } else {
-      notes.push(
-        `This is an established work record, addressed by the ARK '${data.id}'${data.statusStatement ? ` and stated as "${data.statusStatement}"` : ""}. That identifier is the one to cite.`
-      );
-    }
-    if (data.truncated) {
-      notes.push(
-        "This record is longer than one query reads, so parts of it are missing from this answer. A field shown as empty here may be filled on the record itself: open source_url before concluding that the BnF states nothing."
-      );
-    }
-    if (data.expressionCount !== null && data.expressionCount > 0) {
-      notes.push(
-        `The record links ${data.expressionCount} expression(s): translations, adaptations and recordings of this work. Published editions are a separate count, and list_editions is what reads them.`
-      );
+    if (cached) {
+      notes.push("Served from this server's short-lived in-memory cache.");
     }
-    if (data.forms.length > 0) notes.push(FORMS_CAVEAT);
-    if (data.creators.length === 0) {
-      notes.push(
-        "The record credits nobody with this work. That is normal for anonymous and traditional works, and it also happens on records a cataloguer has not finished."
-      );
-    }
-    if (Object.keys(data.sameAs).length > 0) notes.push(ALIGNMENTS_CAVEAT);
-    if (args.include_depictions && data.depictions.length > 0) notes.push(GALLICA_CAVEAT);
+    notes.push(...notesOnWhatTheRecordStates2(data, args));
     const structured = {
       work: {
         id: data.id,
@@ -2069,6 +2271,12 @@ async function runGetWork(client, args) {
 
 // src/tools/listEditions.ts
 import { z as z6 } from "zod";
+function nothingOnThisPage(page2, work) {
+  if (page2 > 1) {
+    return `Page ${page2} of the editions of "${work}" holds no row.`;
+  }
+  return `The BnF catalogue links no edition to the work "${work}".`;
+}
 var listEditionsDescription = [
   "List the published editions of one work in the Biblioth\xE8que nationale de France catalogue, by the identifier search_works or get_work returns.",
   "Each row carries the publisher, the place, the year, the edition statement, the extent in the words of the record, the ISBN when there is one, and the record in the BnF general catalogue.",
@@ -2112,11 +2320,13 @@ var listEditionsOutput = z6.object({
 });
 async function runListEditions(client, args) {
   try {
-    const id = client.identify(args.work_id);
+    const id = parseEntityId(args.work_id);
     const offset = (args.page - 1) * args.limit;
     const { data, cached, retrievedAt } = await client.listEditions(id, args.limit, offset);
     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.");
+    }
     const editions = data.rows.map((row) => ({
       id: row.id,
       title: row.title,
@@ -2140,7 +2350,9 @@ async function runListEditions(client, args) {
       source_url: row.sourceUrl
     }));
     const digitisedCount = editions.filter((edition) => edition.digitised.length > 0).length;
-    if (digitisedCount > 0) notes.push(GALLICA_CAVEAT);
+    if (digitisedCount > 0) {
+      notes.push(GALLICA_CAVEAT);
+    }
     const bracketed = editions.some(
       (edition) => (edition.publisher?.includes("[") ?? false) || (edition.place?.includes("[") ?? false)
     );
@@ -2167,7 +2379,7 @@ async function runListEditions(client, args) {
         "Rows are ordered by the address the catalogue gives each edition. It is neither chronological nor an order of importance, so a first edition can appear anywhere in the list. That order is the one the pages are cut along, so reading page after page reaches every edition once."
       );
     }
-    const body = editions.length === 0 ? args.page > 1 ? `Page ${args.page} of the editions of "${id.id}" holds no row.` : `The BnF catalogue links no edition to the work "${id.id}".` : `${editions.length} edition(s) of "${id.id}":
+    const body = editions.length === 0 ? nothingOnThisPage(args.page, id.id) : `${editions.length} edition(s) of "${id.id}":
 ${editions.map((edition, index) => {
       const imprint = [edition.place, edition.publisher, edition.date].filter(Boolean).join(" : ");
       const extra = [
@@ -2199,7 +2411,9 @@ ${editions.map((edition, index) => {
   }
 }
 async function assertWork(client, id) {
-  if (id.kind === "temp-work") return;
+  if (id.kind === "temp-work") {
+    return;
+  }
   const types = await client.types(id);
   if (types.data.length === 0) {
     throw notFound(`data.bnf.fr describes nothing at "${id.id}".`, {
@@ -2281,11 +2495,13 @@ function emptyPageBody(emptiness, page2, authorId) {
 }
 async function runListWorks(client, args) {
   try {
-    const id = client.identify(args.author_id);
+    const id = parseEntityId(args.author_id);
     const offset = (args.page - 1) * args.limit;
     const { data, cached, retrievedAt } = await client.worksByAuthor(id, args.limit, offset);
     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.");
+    }
     const works = data.rows.map((row) => ({
       id: row.id,
       title: row.title,
@@ -2307,7 +2523,9 @@ async function runListWorks(client, args) {
         const first = await client.worksByAuthor(id, 1, 0);
         return first.data.rows.length > 0;
       });
-      if (emptiness === "absent") await assertPerson(client, id);
+      if (emptiness === "absent") {
+        await assertPerson(client, id);
+      }
       notes.push(emptyPageNote(emptiness, args.page, id.id));
     } else {
       if (data.hasMore) {
@@ -2316,7 +2534,9 @@ async function runListWorks(client, args) {
         );
       }
       notes.push(`${FORMS_CAVEAT} ${FORMS_FILTER_CAVEAT}`);
-      if (works.some((work) => work.status === "provisional")) notes.push(PROVISIONAL_CAVEAT);
+      if (works.some((work) => work.status === "provisional")) {
+        notes.push(PROVISIONAL_CAVEAT);
+      }
       notes.push(ORDER_CAVEAT, DATE_CAVEAT);
     }
     const body = works.length === 0 ? emptyPageBody(emptiness, args.page, id.id) : `${works.length} work(s) the catalogue links to "${id.id}" as their creator:
@@ -2364,6 +2584,7 @@ async function assertPerson(client, id) {
 
 // src/tools/searchAuthors.ts
 import { z as z8 } from "zod";
+var JOINER = /['-]/u;
 var searchAuthorsDescription = [
   "Find a person in the Biblioth\xE8que nationale de France authority file by name, and get the identifier the other tools take.",
   "This matches the name the BnF records, so it takes a surname, a full name or both names in either order. It does not read biographies, so it cannot find a person from what they wrote or what they did.",
@@ -2407,9 +2628,15 @@ var searchAuthorsOutput = z8.object({
 });
 var SPELLING_CAVEAT = "The index matches the words letter for letter, so a name written with other accents, or under another transliteration, is a different search reaching different records. A short answer is no evidence that the file holds nobody else of that name: ask again with the accents dropped, with them restored, and under the transliterations a library uses.";
 function lifespan(birthYear, deathYear) {
-  if (birthYear !== null && deathYear !== null) return ` (${birthYear}\u2013${deathYear})`;
-  if (birthYear !== null) return ` (born ${birthYear})`;
-  if (deathYear !== null) return ` (died ${deathYear})`;
+  if (birthYear !== null && deathYear !== null) {
+    return ` (${birthYear}\u2013${deathYear})`;
+  }
+  if (birthYear !== null) {
+    return ` (born ${birthYear})`;
+  }
+  if (deathYear !== null) {
+    return ` (died ${deathYear})`;
+  }
   return "";
 }
 function emptyPageNote2(emptiness, page2, words, windowFull) {
@@ -2439,7 +2666,7 @@ function emptyPageBody2(emptiness, page2, name, windowFull) {
 }
 async function runSearchAuthors(client, args) {
   try {
-    const reading = client.searchReading(args.name);
+    const reading = readSearchText(args.name);
     const words = reading.words;
     if (words.length === 0) {
       return toToolError(
@@ -2462,12 +2689,16 @@ async function runSearchAuthors(client, args) {
     const terms = reading.terms;
     const windowFull = data.indexWindowFull ?? null;
     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 (windowFull) {
       notes.push(WINDOW_FULL_CAVEAT);
-      if (!data.hasMore) notes.push(WINDOW_FULL_AND_NO_MORE);
+      if (!data.hasMore) {
+        notes.push(WINDOW_FULL_AND_NO_MORE);
+      }
     }
-    const split = words.filter((word) => /['-]/u.test(word));
+    const split = words.filter((word) => JOINER.test(word));
     if (split.length > 0) {
       notes.push(
         `${split.map((word) => `"${word}"`).join(", ")} reached the index as separate terms: it cuts a word at an apostrophe and at a hyphen, and requires each piece to appear in the name rather than in that arrangement, so a record writing the pieces apart is a match.`
@@ -2483,7 +2714,9 @@ async function runSearchAuthors(client, args) {
       role: row.role,
       source_url: row.sourceUrl
     }));
-    if (authors.length > 0) notes.push(NO_RANKING);
+    if (authors.length > 0) {
+      notes.push(NO_RANKING);
+    }
     notes.push(SPELLING_CAVEAT);
     const disputed = authors.map((author) => ({
       label: author.label ?? author.name,
@@ -2497,7 +2730,9 @@ async function runSearchAuthors(client, args) {
     const repeated = /* @__PURE__ */ new Map();
     for (const author of authors) {
       const key = (author.name ?? "").toLocaleLowerCase("fr");
-      if (key !== "") repeated.set(key, (repeated.get(key) ?? 0) + 1);
+      if (key !== "") {
+        repeated.set(key, (repeated.get(key) ?? 0) + 1);
+      }
     }
     const shared = [...repeated.entries()].filter(([, count]) => count > 1);
     if (shared.length > 0) {
@@ -2546,6 +2781,7 @@ ${authors.map((author, index) => {
 
 // src/tools/searchWorks.ts
 import { z as z9 } from "zod";
+var JOINER2 = /['-]/u;
 var searchWorksDescription = [
   "Find a work in the Biblioth\xE8que nationale de France catalogue by words in its title, and get the identifier get_work and list_editions take.",
   "A row matches when its title carries every word given. The index returns no measure of how well a row matches, so the rows are ordered by the address of the record and the work a person would call the obvious answer can sit anywhere in the list or on a later page.",
@@ -2612,7 +2848,7 @@ function emptyPageBody3(emptiness, page2, title, windowFull) {
 }
 async function runSearchWorks(client, args) {
   try {
-    const reading = client.searchReading(args.title);
+    const reading = readSearchText(args.title);
     const words = reading.words;
     if (words.length === 0) {
       return toToolError(
@@ -2635,12 +2871,16 @@ async function runSearchWorks(client, args) {
     const terms = reading.terms;
     const windowFull = data.indexWindowFull ?? null;
     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 (windowFull) {
       notes.push(WINDOW_FULL_CAVEAT);
-      if (!data.hasMore) notes.push(WINDOW_FULL_AND_NO_MORE);
+      if (!data.hasMore) {
+        notes.push(WINDOW_FULL_AND_NO_MORE);
+      }
     }
-    const split = words.filter((word) => /['-]/u.test(word));
+    const split = words.filter((word) => JOINER2.test(word));
     if (split.length > 0) {
       notes.push(
         `${split.map((word) => `"${word}"`).join(", ")} reached the index as separate terms: it cuts a word at an apostrophe and at a hyphen, and requires each piece to appear in the title rather than in that arrangement, so a record writing the pieces apart is a match.`
@@ -2655,8 +2895,12 @@ async function runSearchWorks(client, args) {
       status: row.status,
       source_url: row.sourceUrl
     }));
-    if (works.length > 0) notes.push(NO_RANKING);
-    if (works.some((work) => work.status === "provisional")) notes.push(PROVISIONAL_CAVEAT);
+    if (works.length > 0) {
+      notes.push(NO_RANKING);
+    }
+    if (works.some((work) => work.status === "provisional")) {
+      notes.push(PROVISIONAL_CAVEAT);
+    }
     if (terms.length > 2) {
       notes.push(
         `All ${terms.length} terms were required: ${terms.map((term) => `"${term}"`).join(", ")}. A record catalogued under a shorter form of the title carries fewer of them and falls out of the answer, so dropping a word widens the search.`
@@ -2828,15 +3072,15 @@ async function main() {
   await server.connect(transport);
   const shutdown = async () => {
     await server.close().catch(() => void 0);
-    process.exit(0);
+    process2.exit(0);
   };
-  process.on("SIGINT", shutdown);
-  process.on("SIGTERM", shutdown);
+  process2.on("SIGINT", shutdown);
+  process2.on("SIGTERM", shutdown);
 }
 main().catch((error) => {
   const message = error instanceof Error ? error.message : String(error);
-  process.stderr.write(`[mcp-databnf] fatal: ${message}
+  process2.stderr.write(`[mcp-databnf] fatal: ${message}
 `);
-  process.exit(1);
+  process2.exit(1);
 });
 //# sourceMappingURL=index.js.map