@digital-science-dsl/dimensions-analytics-mcp 1.4.2 → 1.4.4

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 1.4.4
+
+### Patch Changes
+
+- Allow `search_*` tools to omit `query` for filters-only searches. The shared input schema previously required a free-text `query` string even though the description already said it could be omitted and the DSL query builder already supported filter-only searches, causing tool calls with only `filters` to fail validation. Also clarified the `filters` field description with the required `field`/`operator`/`value` shape and supported operators.
+
+## 1.4.3
+
+### Patch Changes
+
+- Package the existing local server as a Claude Desktop extension and publish it with each release.
+
 ## 1.4.2
 
 ### Patch Changes

package/dist/mcp/tools/search-input.d.ts

@@ -36,7 +36,7 @@ export declare const SHARED_SEARCH_INPUT: {
     readonly confirmLargeFetch: z.ZodDefault<z.ZodOptional<z.ZodBoolean>>;
     readonly skip: z.ZodOptional<z.ZodNumber>;
     readonly page: z.ZodOptional<z.ZodNumber>;
-    readonly query: z.ZodString;
+    readonly query: z.ZodOptional<z.ZodString>;
     readonly limit: z.ZodDefault<z.ZodNumber>;
 };
 /** Output schema fields for paginated search responses. */

package/dist/mcp/tools/search-input.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"search-input.d.ts","sourceRoot":"","sources":["../../../src/mcp/tools/search-input.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAGxB,uEAAuE;AACvE,eAAO,MAAM,uBAAuB;;;CAmB1B,CAAC;AAEX,uDAAuD;AACvD,eAAO,MAAM,mBAAmB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA6BtB,CAAC;AAEX,2DAA2D;AAC3D,eAAO,MAAM,wBAAwB;;;;;;;;;;;;;;CAuB3B,CAAC"}
+{"version":3,"file":"search-input.d.ts","sourceRoot":"","sources":["../../../src/mcp/tools/search-input.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAGxB,uEAAuE;AACvE,eAAO,MAAM,uBAAuB;;;CAmB1B,CAAC;AAEX,uDAAuD;AACvD,eAAO,MAAM,mBAAmB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CAoCtB,CAAC;AAEX,2DAA2D;AAC3D,eAAO,MAAM,wBAAwB;;;;;;;;;;;;;;CAuB3B,CAAC"}

package/dist/mcp/tools/search-input.js

@@ -25,7 +25,8 @@ export const PAGINATION_INPUT_SCHEMA = {
 export const SHARED_SEARCH_INPUT = {
     query: z
         .string()
-        .describe("Search query terms (omit with filters-only search if the API allows)"),
+        .optional()
+        .describe("Free-text search terms. Optional — omit for a filters-only search that relies solely on `filters` and/or the entity-specific convenience filters (e.g. yearFrom/yearTo)."),
     limit: z
         .number()
         .int()
@@ -41,7 +42,9 @@ export const SHARED_SEARCH_INPUT = {
     filters: z
         .array(ExtendedWhereFilterSchema)
         .optional()
-        .describe("Additional where-clause filters"),
+        .describe("Additional where-clause filters. Each item needs field, operator, and (except for is_empty/is_not_empty) value, " +
+        'e.g. {"field": "researchers.id", "operator": "=", "value": "ur.01343614074.49"}. ' +
+        "Supported operators: =, !=, >, <, >=, <=, in, is_empty, is_not_empty."),
     confirmLargeFetch: z
         .boolean()
         .optional()

package/dist/mcp/tools/search-input.js.map

@@ -1 +1 @@
-{"version":3,"file":"search-input.js","sourceRoot":"","sources":["../../../src/mcp/tools/search-input.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EAAE,yBAAyB,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAE9E,uEAAuE;AACvE,MAAM,CAAC,MAAM,uBAAuB,GAAG;IACrC,IAAI,EAAE,CAAC;SACJ,MAAM,EAAE;SACR,GAAG,EAAE;SACL,GAAG,CAAC,CAAC,CAAC;SACN,GAAG,CAAC,aAAa,CAAC,SAAS,GAAG,CAAC,CAAC;SAChC,QAAQ,EAAE;SACV,QAAQ,CACP,wEAAwE,aAAa,CAAC,SAAS,iBAAiB,CACjH;IACH,IAAI,EAAE,CAAC;SACJ,MAAM,EAAE;SACR,GAAG,EAAE;SACL,GAAG,CAAC,CAAC,CAAC;SACN,GAAG,CAAC,aAAa,CAAC,QAAQ,GAAG,CAAC,CAAC;SAC/B,QAAQ,EAAE;SACV,QAAQ,CACP,qEAAqE,aAAa,CAAC,QAAQ,GAAG,CAAC,GAAG,CACnG;CACK,CAAC;AAEX,uDAAuD;AACvD,MAAM,CAAC,MAAM,mBAAmB,GAAG;IACjC,KAAK,EAAE,CAAC;SACL,MAAM,EAAE;SACR,QAAQ,CAAC,sEAAsE,CAAC;IACnF,KAAK,EAAE,CAAC;SACL,MAAM,EAAE;SACR,GAAG,EAAE;SACL,GAAG,CAAC,CAAC,CAAC;SACN,GAAG,CAAC,aAAa,CAAC,QAAQ,CAAC;SAC3B,OAAO,CAAC,GAAG,CAAC;SACZ,QAAQ,CAAC,kCAAkC,aAAa,CAAC,QAAQ,gBAAgB,CAAC;IACrF,GAAG,uBAAuB;IAC1B,MAAM,EAAE,CAAC;SACN,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC;SACjB,QAAQ,EAAE;SACV,QAAQ,CACP,kKAAkK,CACnK;IACH,OAAO,EAAE,CAAC;SACP,KAAK,CAAC,yBAAyB,CAAC;SAChC,QAAQ,EAAE;SACV,QAAQ,CAAC,iCAAiC,CAAC;IAC9C,iBAAiB,EAAE,CAAC;SACjB,OAAO,EAAE;SACT,QAAQ,EAAE;SACV,OAAO,CAAC,KAAK,CAAC;SACd,QAAQ,CACP,yIAAyI,CAC1I;CACK,CAAC;AAEX,2DAA2D;AAC3D,MAAM,CAAC,MAAM,wBAAwB,GAAG;IACtC,UAAU,EAAE,CAAC;SACV,MAAM,CAAC;QACN,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE;QACpB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;QAC/B,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE;QACvB,eAAe,EAAE,CAAC,CAAC,MAAM,EAAE;QAC3B,iBAAiB,EAAE,CAAC,CAAC,MAAM,EAAE;QAC7B,2BAA2B,EAAE,CAAC,CAAC,MAAM,EAAE;QACvC,0BAA0B,EAAE,CAAC,CAAC,MAAM,EAAE;KACvC,CAAC;SACD,QAAQ,EAAE;SACV,QAAQ,CAAC,sDAAsD,CAAC;IACnE,kBAAkB,EAAE,CAAC;SAClB,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CAAC,oEAAoE,CAAC;IACjF,YAAY,EAAE,CAAC;SACZ,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CAAC,kEAAkE,CAAC;CACvE,CAAC"}
+{"version":3,"file":"search-input.js","sourceRoot":"","sources":["../../../src/mcp/tools/search-input.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EAAE,yBAAyB,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAE9E,uEAAuE;AACvE,MAAM,CAAC,MAAM,uBAAuB,GAAG;IACrC,IAAI,EAAE,CAAC;SACJ,MAAM,EAAE;SACR,GAAG,EAAE;SACL,GAAG,CAAC,CAAC,CAAC;SACN,GAAG,CAAC,aAAa,CAAC,SAAS,GAAG,CAAC,CAAC;SAChC,QAAQ,EAAE;SACV,QAAQ,CACP,wEAAwE,aAAa,CAAC,SAAS,iBAAiB,CACjH;IACH,IAAI,EAAE,CAAC;SACJ,MAAM,EAAE;SACR,GAAG,EAAE;SACL,GAAG,CAAC,CAAC,CAAC;SACN,GAAG,CAAC,aAAa,CAAC,QAAQ,GAAG,CAAC,CAAC;SAC/B,QAAQ,EAAE;SACV,QAAQ,CACP,qEAAqE,aAAa,CAAC,QAAQ,GAAG,CAAC,GAAG,CACnG;CACK,CAAC;AAEX,uDAAuD;AACvD,MAAM,CAAC,MAAM,mBAAmB,GAAG;IACjC,KAAK,EAAE,CAAC;SACL,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CACP,0KAA0K,CAC3K;IACH,KAAK,EAAE,CAAC;SACL,MAAM,EAAE;SACR,GAAG,EAAE;SACL,GAAG,CAAC,CAAC,CAAC;SACN,GAAG,CAAC,aAAa,CAAC,QAAQ,CAAC;SAC3B,OAAO,CAAC,GAAG,CAAC;SACZ,QAAQ,CAAC,kCAAkC,aAAa,CAAC,QAAQ,gBAAgB,CAAC;IACrF,GAAG,uBAAuB;IAC1B,MAAM,EAAE,CAAC;SACN,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC;SACjB,QAAQ,EAAE;SACV,QAAQ,CACP,kKAAkK,CACnK;IACH,OAAO,EAAE,CAAC;SACP,KAAK,CAAC,yBAAyB,CAAC;SAChC,QAAQ,EAAE;SACV,QAAQ,CACP,kHAAkH;QAChH,mFAAmF;QACnF,uEAAuE,CAC1E;IACH,iBAAiB,EAAE,CAAC;SACjB,OAAO,EAAE;SACT,QAAQ,EAAE;SACV,OAAO,CAAC,KAAK,CAAC;SACd,QAAQ,CACP,yIAAyI,CAC1I;CACK,CAAC;AAEX,2DAA2D;AAC3D,MAAM,CAAC,MAAM,wBAAwB,GAAG;IACtC,UAAU,EAAE,CAAC;SACV,MAAM,CAAC;QACN,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE;QACpB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;QAC/B,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE;QACvB,eAAe,EAAE,CAAC,CAAC,MAAM,EAAE;QAC3B,iBAAiB,EAAE,CAAC,CAAC,MAAM,EAAE;QAC7B,2BAA2B,EAAE,CAAC,CAAC,MAAM,EAAE;QACvC,0BAA0B,EAAE,CAAC,CAAC,MAAM,EAAE;KACvC,CAAC;SACD,QAAQ,EAAE;SACV,QAAQ,CAAC,sDAAsD,CAAC;IACnE,kBAAkB,EAAE,CAAC;SAClB,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CAAC,oEAAoE,CAAC;IACjF,YAAY,EAAE,CAAC;SACZ,MAAM,EAAE;SACR,QAAQ,EAAE;SACV,QAAQ,CAAC,kEAAkE,CAAC;CACvE,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@digital-science-dsl/dimensions-analytics-mcp",
-  "version": "1.4.2",
+  "version": "1.4.4",
   "mcpName": "ai.dimensions/analytics-mcp",
   "description": "Dimensions Analytics MCP server for Dimensions DSL querying",
   "keywords": [

package/scripts/smoke-mcpb.mjs

@@ -0,0 +1,57 @@
+#!/usr/bin/env node
+import { mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join, resolve } from "node:path";
+import { fileURLToPath } from "node:url";
+import { unpackExtension } from "@anthropic-ai/mcpb";
+import { Client } from "@modelcontextprotocol/client";
+import { StdioClientTransport } from "@modelcontextprotocol/client/stdio";
+
+const root = resolve(fileURLToPath(new URL("../../..", import.meta.url)));
+const output = join(root, "dist/dimensions-analytics-mcp.mcpb");
+const temp = mkdtempSync(join(tmpdir(), "dimensions-mcpb-smoke-"));
+const client = new Client({ name: "mcpb-smoke", version: "1.0.0" });
+
+try {
+  if (!(await unpackExtension({ mcpbPath: output, outputDir: temp, silent: true }))) {
+    throw new Error("Could not unpack the MCPB");
+  }
+  const manifest = JSON.parse(readFileSync(join(temp, "manifest.json"), "utf8"));
+  const pkg = JSON.parse(readFileSync(join(root, "apps/mcp/package.json"), "utf8"));
+  if (manifest.version !== pkg.version) throw new Error("Bundle version differs from package");
+  const fixture = JSON.parse(
+    readFileSync(join(root, "apps/mcp/test/fixtures/describe-schema.json"), "utf8"),
+  );
+  const cache = join(temp, "schema.json");
+  writeFileSync(cache, JSON.stringify({ cachedAt: new Date().toISOString(), schema: fixture }));
+
+  const transport = new StdioClientTransport({
+    command: process.execPath,
+    args: [join(temp, manifest.server.entry_point)],
+    env: {
+      ...process.env,
+      DIMENSIONS_API_KEY: "smoke-test-key",
+      DIMENSIONS_MCP_UPDATE_CHECK: "0",
+      DIMENSIONS_MCP_AUTO_UPDATE: "0",
+      SCHEMA_CACHE_PATH: cache,
+    },
+  });
+  const timeout = setTimeout(() => {
+    console.error("MCPB smoke test timed out");
+    process.exitCode = 1;
+    void transport.close();
+  }, 20_000);
+  try {
+    await client.connect(transport);
+    const tools = await client.listTools();
+    if (!tools.tools.some((tool) => tool.name === "search_publications")) {
+      throw new Error("Bundled server did not expose search_publications");
+    }
+    console.log(`MCPB smoke test passed (${tools.tools.length} tools)`);
+  } finally {
+    clearTimeout(timeout);
+    await client.close();
+  }
+} finally {
+  rmSync(temp, { recursive: true, force: true });
+}

package/src/mcp/tools/search-input.ts

@@ -32,7 +32,10 @@ export const PAGINATION_INPUT_SCHEMA = {
 export const SHARED_SEARCH_INPUT = {
   query: z
     .string()
-    .describe("Search query terms (omit with filters-only search if the API allows)"),
+    .optional()
+    .describe(
+      "Free-text search terms. Optional — omit for a filters-only search that relies solely on `filters` and/or the entity-specific convenience filters (e.g. yearFrom/yearTo).",
+    ),
   limit: z
     .number()
     .int()
@@ -50,7 +53,11 @@ export const SHARED_SEARCH_INPUT = {
   filters: z
     .array(ExtendedWhereFilterSchema)
     .optional()
-    .describe("Additional where-clause filters"),
+    .describe(
+      "Additional where-clause filters. Each item needs field, operator, and (except for is_empty/is_not_empty) value, " +
+        'e.g. {"field": "researchers.id", "operator": "=", "value": "ur.01343614074.49"}. ' +
+        "Supported operators: =, !=, >, <, >=, <=, in, is_empty, is_not_empty.",
+    ),
   confirmLargeFetch: z
     .boolean()
     .optional()

package/test/tools/search.test.ts

@@ -407,6 +407,25 @@ describe("search tools", () => {
       const dsl = client.rawQuery.mock.calls[0][0] as string;
       expect(dsl).toContain("is empty");
     });
+
+    it("supports a filters-only search with no query", async () => {
+      client.rawQuery.mockResolvedValue(
+        apiRows("publications", [{ id: "pub1", title: "Filters-only Paper" }], 1),
+      );
+
+      const result = await callTool(handlers, "search_publications", {
+        filters: [{ field: "researchers.id", operator: "=", value: "ur.014403456555.45" }],
+        sortBy: "year",
+        limit: 100,
+      });
+
+      expect(result.isError).toBeFalsy();
+      const dsl = client.rawQuery.mock.calls[0][0] as string;
+      expect(dsl).toContain("search publications");
+      expect(dsl).not.toContain(" for ");
+      expect(dsl).toContain("researchers.id");
+      expect(parseToolResult(result).publications).toHaveLength(1);
+    });
   });
 
   describe("pagination", () => {