mcp-libraryofcongress 1.0.0 → 1.1.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## 1.1.0
+
+- An argument no tool declared was read and dropped, and the answer came back
+  computed on the defaults with nothing said about it. A caller who mistyped a
+  name, or wrote one a tool answering a neighbouring question takes, was
+  answered as confidently as one who had asked what they meant. Every tool now
+  refuses an argument it does not declare, under the `invalid_input` code,
+  naming the argument and offering the declared name when one is close:
+  `identifer` on `get_item` is answered with `identifier`, `limit_per_page` on
+  `search_newspapers` with `limit`.
+- The schema each tool publishes now carries `additionalProperties: false`, so a
+  client reads the rule before it calls rather than discovering it from a
+  refusal.
+
 ## 1.0.0
 
 First stable release. The tool names, the argument names and the shape of the

package/README.md

@@ -3,6 +3,7 @@
 [![npm](https://img.shields.io/npm/v/mcp-libraryofcongress.svg)](https://www.npmjs.com/package/mcp-libraryofcongress)
 [![CI](https://github.com/smeet666/mcp-libraryofcongress/actions/workflows/ci.yml/badge.svg)](https://github.com/smeet666/mcp-libraryofcongress/actions/workflows/ci.yml)
 [![license](https://img.shields.io/npm/l/mcp-libraryofcongress.svg)](LICENSE)
+[![MCP Registry](https://img.shields.io/badge/MCP_Registry-listed-6E56CF)](https://registry.modelcontextprotocol.io/v0/servers?search=io.github.smeet666/mcp-libraryofcongress)
 [![Glama](https://glama.ai/mcp/servers/smeet666/mcp-libraryofcongress/badges/score.svg)](https://glama.ai/mcp/servers/smeet666/mcp-libraryofcongress)
 
 An MCP server for the [Library of Congress](https://www.loc.gov). **Search the

package/dist/index.js

@@ -7,7 +7,7 @@ import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 
 // src/version.ts
-var PKG_VERSION = "1.0.0";
+var PKG_VERSION = "1.1.0";
 var REPO_URL = "https://github.com/smeet666/mcp-libraryofcongress";
 
 // src/config.ts
@@ -991,25 +991,77 @@ var LocClient = class {
 };
 
 // src/tools/getItem.ts
-import { z as z2 } from "zod";
+import { z as z3 } from "zod";
 
-// src/tools/shared.ts
+// src/tools/arguments.ts
 import { z } from "zod";
+var INVALID_INPUT = "invalid_input";
+function strictInput(shape) {
+  const declared = Object.keys(shape);
+  return z.strictObject(shape, {
+    error: (issue) => issue.code === "unrecognized_keys" ? unknownArgumentMessage(issue.keys, declared) : void 0
+  });
+}
+function unknownArgumentMessage(keys, declared) {
+  const named = keys.map((key) => {
+    const near = nearestArgument(key, declared);
+    return near ? `'${key}' (did you mean '${near}'?)` : `'${key}'`;
+  }).join(", ");
+  return `[${INVALID_INPUT}] Unknown ${keys.length > 1 ? "arguments" : "argument"} ${named}. This tool takes: ${declared.join(", ")}.`;
+}
+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;
+  const sameName = declared.find((name) => flatten(name) === flat);
+  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;
+  let closest;
+  let shortest = Number.POSITIVE_INFINITY;
+  for (const name of declared) {
+    const distance = editDistance(flat, flatten(name));
+    if (distance < shortest) {
+      shortest = distance;
+      closest = name;
+    }
+  }
+  return shortest <= Math.max(1, Math.floor(flat.length / 3)) ? closest : void 0;
+}
+function editDistance(left, right) {
+  let previous = Array.from({ length: right.length + 1 }, (_, index) => index);
+  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);
+    }
+    previous = current;
+  }
+  return previous[right.length];
+}
+
+// src/tools/shared.ts
+import { z as z2 } from "zod";
 var MAX_TEXT_CHARS = 2200;
 var ATTRIBUTION = "Source: Library of Congress";
-var recordSchema = z.object({
-  identifier: z.string().nullable().describe(
+var recordSchema = z2.object({
+  identifier: z2.string().nullable().describe(
     "Pass this to get_item to read the full record. Null when the row names no address a record can be read at."
   ),
-  title: z.string().nullable(),
-  creator: z.string().nullable().describe("Whoever the Library credits for the work."),
-  year: z.number().int().nullable().describe("Read off 'date' when that date names a year."),
-  date: z.string().nullable().describe("The date as published, which is often a range."),
-  format: z.string().nullable().describe("What the thing is: book, photo, map, newspaper."),
-  location: z.array(z.string()).describe("Places the record is catalogued under."),
-  subjects: z.array(z.string()),
-  online: z.boolean().describe("Whether a digitised copy can be read online."),
-  source_url: z.string().describe("Public page. Show this when citing the record.")
+  title: z2.string().nullable(),
+  creator: z2.string().nullable().describe("Whoever the Library credits for the work."),
+  year: z2.number().int().nullable().describe("Read off 'date' when that date names a year."),
+  date: z2.string().nullable().describe("The date as published, which is often a range."),
+  format: z2.string().nullable().describe("What the thing is: book, photo, map, newspaper."),
+  location: z2.array(z2.string()).describe("Places the record is catalogued under."),
+  subjects: z2.array(z2.string()),
+  online: z2.boolean().describe("Whether a digitised copy can be read online."),
+  source_url: z2.string().describe("Public page. Show this when citing the record.")
 });
 function indentMarkerLines(body) {
   return body.replace(/^(Note:|Source:)/gm, " $1");
@@ -1078,46 +1130,46 @@ var getItemDescription = [
   "'full_metadata' returns every field the Library publishes for the record, which is large and rarely needed.",
   "A long description paginates: when 'next_offset' is not null, call again with 'offset' set to it."
 ].join(" ");
-var getItemInput = z2.object({
-  identifier: z2.string().min(1).max(300).describe("Record identifier, such as '2017645459' or 'sn83045462/1929-02-03/ed-1'."),
-  sections: z2.array(z2.enum(SECTIONS)).default(["basic"]).describe("Which parts to return. Each one beyond 'basic' adds to the size of the answer."),
-  offset: z2.number().int().min(0).default(0).describe("Where to resume the description."),
-  max_description_chars: z2.number().int().min(200).max(2e4).default(2e3).describe("Characters of the description to return in one call.")
+var getItemInput = strictInput({
+  identifier: z3.string().min(1).max(300).describe("Record identifier, such as '2017645459' or 'sn83045462/1929-02-03/ed-1'."),
+  sections: z3.array(z3.enum(SECTIONS)).default(["basic"]).describe("Which parts to return. Each one beyond 'basic' adds to the size of the answer."),
+  offset: z3.number().int().min(0).default(0).describe("Where to resume the description."),
+  max_description_chars: z3.number().int().min(200).max(2e4).default(2e3).describe("Characters of the description to return in one call.")
 });
-var getItemOutput = z2.object({
-  item: z2.object({
-    identifier: z2.string(),
-    title: z2.string().nullable(),
-    creator: z2.string().nullable(),
-    year: z2.number().int().nullable(),
-    date: z2.string().nullable(),
-    format: z2.string().nullable(),
-    source_url: z2.string().describe("Public page. Show this when citing the record.")
+var getItemOutput = z3.object({
+  item: z3.object({
+    identifier: z3.string(),
+    title: z3.string().nullable(),
+    creator: z3.string().nullable(),
+    year: z3.number().int().nullable(),
+    date: z3.string().nullable(),
+    format: z3.string().nullable(),
+    source_url: z3.string().describe("Public page. Show this when citing the record.")
   }),
-  description: z2.string().nullable(),
-  offset: z2.number().int(),
-  next_offset: z2.number().int().nullable().describe("Pass as 'offset' to read the rest of the description. Null when it ends here."),
-  notes_on_record: z2.array(z2.string()).describe("Notes the Library published about the record itself."),
-  subjects: z2.array(z2.string()),
-  location: z2.array(z2.string()),
-  language: z2.array(z2.string()),
-  part_of: z2.array(z2.string()).describe("Collections and divisions the record sits in."),
-  repository: z2.string().nullable().describe("Where the original is held."),
-  call_number: z2.string().nullable(),
-  rights: z2.string().nullable().describe(
+  description: z3.string().nullable(),
+  offset: z3.number().int(),
+  next_offset: z3.number().int().nullable().describe("Pass as 'offset' to read the rest of the description. Null when it ends here."),
+  notes_on_record: z3.array(z3.string()).describe("Notes the Library published about the record itself."),
+  subjects: z3.array(z3.string()),
+  location: z3.array(z3.string()),
+  language: z3.array(z3.string()),
+  part_of: z3.array(z3.string()).describe("Collections and divisions the record sits in."),
+  repository: z3.string().nullable().describe("Where the original is held."),
+  call_number: z3.string().nullable(),
+  rights: z3.string().nullable().describe(
     "What the Library says about reuse. Null when it says nothing, which is not permission."
   ),
-  citations: z2.record(z2.string(), z2.string()).optional(),
-  resources: z2.array(
-    z2.object({
-      caption: z2.string().nullable(),
-      file_count: z2.number().int().nullable(),
-      url: z2.string().nullable(),
-      image_url: z2.string().nullable()
+  citations: z3.record(z3.string(), z3.string()).optional(),
+  resources: z3.array(
+    z3.object({
+      caption: z3.string().nullable(),
+      file_count: z3.number().int().nullable(),
+      url: z3.string().nullable(),
+      image_url: z3.string().nullable()
     })
   ).optional(),
-  full_metadata: z2.record(z2.string(), z2.unknown()).optional(),
-  notes: z2.array(z2.string())
+  full_metadata: z3.record(z3.string(), z3.unknown()).optional(),
+  notes: z3.array(z3.string())
 });
 async function runGetItem(client, args) {
   try {
@@ -1207,35 +1259,35 @@ async function runGetItem(client, args) {
 }
 
 // src/tools/listCollections.ts
-import { z as z3 } from "zod";
+import { z as z4 } from "zod";
 var listCollectionsDescription = [
   "List the digital collections of the Library of Congress: bodies of material a curator chose, described and published together.",
   "Use it to see what is there before searching, since a collection names a corpus that a query would have to guess at.",
   "'collection_filter' on each row is the wording search_items takes as its 'collection' argument.",
   "'item_count' is how many records the collection gathers, which is the size of the corpus rather than the number of rows here."
 ].join(" ");
-var listCollectionsInput = z3.object({
-  limit: z3.number().int().min(1).max(50).default(20).describe("Collections to return."),
-  page: z3.number().int().min(1).max(100).default(1).describe("Which page of collections, from 1."),
-  max_description_chars: z3.number().int().min(80).max(2e3).default(300).describe("Budget for one collection's description.")
+var listCollectionsInput = strictInput({
+  limit: z4.number().int().min(1).max(50).default(20).describe("Collections to return."),
+  page: z4.number().int().min(1).max(100).default(1).describe("Which page of collections, from 1."),
+  max_description_chars: z4.number().int().min(80).max(2e3).default(300).describe("Budget for one collection's description.")
 });
-var listCollectionsOutput = z3.object({
-  total: z3.number().int().describe("Collections the Library publishes, not the number returned."),
-  page: z3.number().int(),
-  collections: z3.array(
-    z3.object({
-      identifier: z3.string().nullable().describe("The slug the collection is addressed by."),
-      title: z3.string(),
-      collection_filter: z3.string().describe("Pass as 'collection' to search_items to keep results within this collection."),
-      description: z3.string().nullable(),
-      item_count: z3.number().int().nullable().describe("Records the collection gathers. Null when the Library states none."),
-      subjects: z3.array(z3.string()),
-      formats: z3.array(z3.string()).describe("Kinds of thing the collection holds."),
-      source_url: z3.string(),
-      items_url: z3.string().nullable().describe("Lists what the collection holds, page by page.")
+var listCollectionsOutput = z4.object({
+  total: z4.number().int().describe("Collections the Library publishes, not the number returned."),
+  page: z4.number().int(),
+  collections: z4.array(
+    z4.object({
+      identifier: z4.string().nullable().describe("The slug the collection is addressed by."),
+      title: z4.string(),
+      collection_filter: z4.string().describe("Pass as 'collection' to search_items to keep results within this collection."),
+      description: z4.string().nullable(),
+      item_count: z4.number().int().nullable().describe("Records the collection gathers. Null when the Library states none."),
+      subjects: z4.array(z4.string()),
+      formats: z4.array(z4.string()).describe("Kinds of thing the collection holds."),
+      source_url: z4.string(),
+      items_url: z4.string().nullable().describe("Lists what the collection holds, page by page.")
     })
   ),
-  notes: z3.array(z3.string())
+  notes: z4.array(z4.string())
 });
 async function runListCollections(client, args) {
   try {
@@ -1288,7 +1340,7 @@ async function runListCollections(client, args) {
 }
 
 // src/tools/searchItems.ts
-import { z as z4 } from "zod";
+import { z as z5 } from "zod";
 var searchItemsDescription = [
   "Search the Library of Congress catalogue: books, photographs, maps, recordings, films, manuscripts, sheet music and newspaper titles.",
   "'media_type' is required, because the Library keeps a separate catalogue for each kind of thing.",
@@ -1298,30 +1350,30 @@ var searchItemsDescription = [
   "Every row carries an 'identifier', which get_item takes."
 ].join(" ");
 var mediaTypes = FORMAT_ROUTES;
-var searchItemsInput = z4.object({
-  query: z4.string().min(1).max(300).describe("Words to look for in titles, creators and catalogue descriptions."),
-  media_type: z4.enum(mediaTypes).describe(
+var searchItemsInput = strictInput({
+  query: z5.string().min(1).max(300).describe("Words to look for in titles, creators and catalogue descriptions."),
+  media_type: z5.enum(mediaTypes).describe(
     "Which catalogue to search: books, photos, maps, audio, film-and-videos, manuscripts, notated-music or newspapers. 'newspapers' holds the papers themselves; a phrase printed on a page belongs in search_newspapers."
   ),
-  year_from: z4.number().int().min(1e3).max(9999).optional().describe("Earliest year, inclusive."),
-  year_to: z4.number().int().min(1e3).max(9999).optional().describe("Latest year, inclusive."),
-  subject: z4.string().max(120).optional().describe("A subject heading, such as 'crime'."),
-  location: z4.string().max(120).optional().describe("A place, such as 'oklahoma'."),
-  language: z4.string().max(120).optional().describe("A language, written in English: 'english'."),
-  collection: z4.string().max(160).optional().describe(
+  year_from: z5.number().int().min(1e3).max(9999).optional().describe("Earliest year, inclusive."),
+  year_to: z5.number().int().min(1e3).max(9999).optional().describe("Latest year, inclusive."),
+  subject: z5.string().max(120).optional().describe("A subject heading, such as 'crime'."),
+  location: z5.string().max(120).optional().describe("A place, such as 'oklahoma'."),
+  language: z5.string().max(120).optional().describe("A language, written in English: 'english'."),
+  collection: z5.string().max(160).optional().describe(
     "A collection, named exactly as list_collections reports it under 'collection_filter'."
   ),
-  online_only: z4.boolean().default(true).describe("Keep to material with a digitised copy. Set false to include shelf-only records."),
-  sort: z4.enum(["relevance", "newest", "oldest", "title"]).default("relevance").describe("Order of the results."),
-  limit: z4.number().int().min(1).max(50).default(10),
-  page: z4.number().int().min(1).max(100).default(1)
+  online_only: z5.boolean().default(true).describe("Keep to material with a digitised copy. Set false to include shelf-only records."),
+  sort: z5.enum(["relevance", "newest", "oldest", "title"]).default("relevance").describe("Order of the results."),
+  limit: z5.number().int().min(1).max(50).default(10),
+  page: z5.number().int().min(1).max(100).default(1)
 });
-var searchItemsOutput = z4.object({
-  query: z4.string(),
-  total: z4.number().int().describe("Records matching across this catalogue, not the number returned."),
-  page: z4.number().int(),
-  items: z4.array(recordSchema),
-  notes: z4.array(z4.string())
+var searchItemsOutput = z5.object({
+  query: z5.string(),
+  total: z5.number().int().describe("Records matching across this catalogue, not the number returned."),
+  page: z5.number().int(),
+  items: z5.array(recordSchema),
+  notes: z5.array(z5.string())
 });
 function describeNarrowing(args) {
   const written = [];
@@ -1433,7 +1485,7 @@ ${renderRecords(items)}`;
 }
 
 // src/tools/searchNewspapers.ts
-import { z as z5 } from "zod";
+import { z as z6 } from "zod";
 var searchNewspapersDescription = [
   "Search the text inside digitised American newspaper pages held by the Library of Congress.",
   "This reads what optical recognition took off the scanned pages, so it finds a phrase that appears nowhere in a title or a catalogue record.",
@@ -1444,49 +1496,49 @@ var searchNewspapersDescription = [
   "The Library returns the opening of a page's text with each row rather than the whole page, so the searched words are often further down than the excerpts reach: 'words_located' says which of the two happened for each match.",
   "Use search_items instead when looking for a work by its title, creator or subject."
 ].join(" ");
-var searchNewspapersInput = z5.object({
-  query: z5.string().min(2).max(300).describe(`Words or a quoted phrase, such as '"cure for influenza"'.`),
-  location: z5.string().max(120).optional().describe(
+var searchNewspapersInput = strictInput({
+  query: z6.string().min(2).max(300).describe(`Words or a quoted phrase, such as '"cure for influenza"'.`),
+  location: z6.string().max(120).optional().describe(
     "The state a paper was published in, written as the Library writes it: 'new york', 'district of columbia'. It is the value 'state' carries on the matches this tool returns."
   ),
-  publication: z5.string().max(200).optional().describe(
+  publication: z6.string().max(200).optional().describe(
     "One newspaper, named with its town and the years it ran: 'new-york tribune (new york [n.y.]) 1866-1924'. Take the wording from 'publication' on a match here, or from a title in search_items with media_type 'newspapers'."
   ),
-  year_from: z5.number().int().min(1e3).max(9999).optional().describe("Earliest year, inclusive."),
-  year_to: z5.number().int().min(1e3).max(9999).optional().describe("Latest year, inclusive."),
-  limit: z5.number().int().min(1).max(25).default(10).describe("Matches to return."),
-  page: z5.number().int().min(1).max(100).default(1).describe("Which page of matches, from 1."),
-  max_excerpt_chars: z5.number().int().min(80).max(1200).default(300).describe(
+  year_from: z6.number().int().min(1e3).max(9999).optional().describe("Earliest year, inclusive."),
+  year_to: z6.number().int().min(1e3).max(9999).optional().describe("Latest year, inclusive."),
+  limit: z6.number().int().min(1).max(25).default(10).describe("Matches to return."),
+  page: z6.number().int().min(1).max(100).default(1).describe("Which page of matches, from 1."),
+  max_excerpt_chars: z6.number().int().min(80).max(1200).default(300).describe(
     "Budget for one passage. Read it together with 'max_excerpts_per_match': the size of the answer is the product of the two and the number of matches."
   ),
-  max_excerpts_per_match: z5.number().int().min(1).max(10).default(3).describe(
+  max_excerpts_per_match: z6.number().int().min(1).max(10).default(3).describe(
     "Passages to keep per match. A page holding the words several times yields several, and the later ones rarely say anything the first did not."
   )
 });
-var searchNewspapersOutput = z5.object({
-  query: z5.string(),
-  total: z5.number().int().describe(
+var searchNewspapersOutput = z6.object({
+  query: z6.string(),
+  total: z6.number().int().describe(
     "Newspaper pages that match, not the number returned and not a count of occurrences. Raise 'page' to read further into it."
   ),
-  page: z5.number().int(),
-  hits: z5.array(
-    z5.object({
-      identifier: z5.string().nullable().describe("Pass to get_item for the record of the issue this leaf belongs to."),
-      title: z5.string().nullable().describe("How the Library names the leaf, paper and date included."),
-      creator: z5.string().nullable().describe("Whoever contributed the scan, which is a library."),
-      year: z5.number().int().nullable(),
-      page_number: z5.number().int().nullable().describe("The leaf within the issue."),
-      published_on: z5.string().nullable().describe("Date of the issue, as published."),
-      publication: z5.string().nullable().describe("The newspaper, with the years it ran."),
-      state: z5.string().nullable().describe("Where the paper was published."),
-      words_located: z5.boolean().describe(
+  page: z6.number().int(),
+  hits: z6.array(
+    z6.object({
+      identifier: z6.string().nullable().describe("Pass to get_item for the record of the issue this leaf belongs to."),
+      title: z6.string().nullable().describe("How the Library names the leaf, paper and date included."),
+      creator: z6.string().nullable().describe("Whoever contributed the scan, which is a library."),
+      year: z6.number().int().nullable(),
+      page_number: z6.number().int().nullable().describe("The leaf within the issue."),
+      published_on: z6.string().nullable().describe("Date of the issue, as published."),
+      publication: z6.string().nullable().describe("The newspaper, with the years it ran."),
+      state: z6.string().nullable().describe("Where the paper was published."),
+      words_located: z6.boolean().describe(
         "True when the searched words were found in the text returned with this row, in which case the excerpts are centred on them. False when they sit further down the page than that text reaches, in which case the excerpts are its opening."
       ),
-      excerpts: z5.array(z5.string()).describe("Passages as a machine read them off the page."),
-      source_url: z5.string().describe("Opens the leaf itself, with the query applied.")
+      excerpts: z6.array(z6.string()).describe("Passages as a machine read them off the page."),
+      source_url: z6.string().describe("Opens the leaf itself, with the query applied.")
     })
   ),
-  notes: z5.array(z5.string())
+  notes: z6.array(z6.string())
 });
 function describeNarrowing2(args) {
   const written = [];