afpnews-mcp-server 2.0.0 → 2.0.1

package/.serena/project.yml

@@ -133,3 +133,17 @@ read_only_memory_patterns: []
 # Possible values: unset (use global setting), "lf", "crlf", or "native" (platform default)
 # This does not affect Serena's own files (e.g. memories and configuration files), which always use native line endings.
 line_ending:
+
+# list of regex patterns for memories to completely ignore.
+# Matching memories will not appear in list_memories or activate_project output
+# and cannot be accessed via read_memory or write_memory.
+# To access ignored memory files, use the read_file tool on the raw file path.
+# Extends the list from the global configuration, merging the two lists.
+# Example: ["_archive/.*", "_episodes/.*"]
+ignored_memory_patterns: []
+
+# advanced configuration option allowing to configure language server-specific options.
+# Maps the language key to the options.
+# Have a look at the docstring of the constructors of the LS implementations within solidlsp (e.g., for C# or PHP) to see which options are available.
+# No documentation on options means no options are available.
+ls_specific_settings: {}

package/README.md

@@ -138,7 +138,11 @@ The `afp_search_articles` tool supports presets that apply predefined filters:
 
 ### Full text
 
-By default, `afp_search_articles` returns excerpts (first 4 paragraphs). Set `fullText: true` to get the complete article body. Presets default to full text.
+By default, `afp_search_articles` returns excerpts (first 2 paragraphs). Set `fullText: true` to get the complete article body. Presets default to full text.
+
+### Pagination
+
+Use `offset` to paginate through results (e.g. `offset: 10` to skip the first 10). For large chronological scans, prefer narrowing `dateFrom`/`dateTo` ranges in `facets` over high offsets. Keep `size` small (10–20) for best performance.
 
 ## Prompts
 

package/bun.lock

@@ -5,15 +5,19 @@
     "": {
       "name": "afpnews-mcp-server",
       "dependencies": {
-        "@modelcontextprotocol/sdk": "^1.27.1",
-        "afpnews-api": "^2.4.0",
+        "@modelcontextprotocol/sdk": "^1.29.0",
         "elysia": "^1.4.28",
         "elysia-rate-limit": "^4.5.1",
-        "jose": "^6.2.1",
-        "zod": "^4.3.6",
+        "jose": "^6.2.2",
       },
       "devDependencies": {
-        "@types/node": "^25.5.0",
+        "@types/node": "^25.5.2",
+        "afpnews-api": "^2.4.1",
+        "zod": "^4.3.6",
+      },
+      "peerDependencies": {
+        "afpnews-api": "^2.4.0",
+        "zod": "^4.3.6",
       },
     },
   },
@@ -24,7 +28,7 @@
 
     "@hono/node-server": ["@hono/node-server@1.19.9", "", { "peerDependencies": { "hono": "4.11.9" } }, "sha512-vHL6w3ecZsky+8P5MD+eFfaGTyCeOHUIFYMGpQGbrBTSmNNoxv0if69rEZ5giu36weC5saFuznL411gRX7bJDw=="],
 
-    "@modelcontextprotocol/sdk": ["@modelcontextprotocol/sdk@1.27.1", "", { "dependencies": { "@hono/node-server": "^1.19.9", "ajv": "^8.17.1", "ajv-formats": "^3.0.1", "content-type": "^1.0.5", "cors": "^2.8.5", "cross-spawn": "^7.0.5", "eventsource": "^3.0.2", "eventsource-parser": "^3.0.0", "express": "^5.2.1", "express-rate-limit": "^8.2.1", "hono": "^4.11.4", "jose": "^6.1.3", "json-schema-typed": "^8.0.2", "pkce-challenge": "^5.0.0", "raw-body": "^3.0.0", "zod": "^3.25 || ^4.0", "zod-to-json-schema": "^3.25.1" }, "peerDependencies": { "@cfworker/json-schema": "^4.1.1" }, "optionalPeers": ["@cfworker/json-schema"] }, "sha512-sr6GbP+4edBwFndLbM60gf07z0FQ79gaExpnsjMGePXqFcSSb7t6iscpjk9DhFhwd+mTEQrzNafGP8/iGGFYaA=="],
+    "@modelcontextprotocol/sdk": ["@modelcontextprotocol/sdk@1.29.0", "", { "dependencies": { "@hono/node-server": "^1.19.9", "ajv": "^8.17.1", "ajv-formats": "^3.0.1", "content-type": "^1.0.5", "cors": "^2.8.5", "cross-spawn": "^7.0.5", "eventsource": "^3.0.2", "eventsource-parser": "^3.0.0", "express": "^5.2.1", "express-rate-limit": "^8.2.1", "hono": "^4.11.4", "jose": "^6.1.3", "json-schema-typed": "^8.0.2", "pkce-challenge": "^5.0.0", "raw-body": "^3.0.0", "zod": "^3.25 || ^4.0", "zod-to-json-schema": "^3.25.1" }, "peerDependencies": { "@cfworker/json-schema": "^4.1.1" }, "optionalPeers": ["@cfworker/json-schema"] }, "sha512-zo37mZA9hJWpULgkRpowewez1y6ML5GsXJPY8FI0tBBCd77HEvza4jDqRKOXgHNn867PVGCyTdzqpz0izu5ZjQ=="],
 
     "@sinclair/typebox": ["@sinclair/typebox@0.34.48", "", {}, "sha512-kKJTNuK3AQOrgjjotVxMrCn1sUJwM76wMszfq1kdU4uYVJjvEWuFQ6HgvLt4Xz3fSmZlTOxJ/Ie13KnIcWQXFA=="],
 
@@ -38,13 +42,13 @@
 
     "@types/nearley": ["@types/nearley@2.11.5", "", {}, "sha512-dM7TrN0bVxGGXTYGx4YhGear8ysLO5SOuouAWM9oltjQ3m9oYa13qi8Z1DJp5zxVMPukvQdsrnZmgzpeuTSEQA=="],
 
-    "@types/node": ["@types/node@25.5.0", "", { "dependencies": { "undici-types": "~7.18.0" } }, "sha512-jp2P3tQMSxWugkCUKLRPVUpGaL5MVFwF8RDuSRztfwgN1wmqJeMSbKlnEtQqU8UrhTmzEmZdu2I6v2dpp7XIxw=="],
+    "@types/node": ["@types/node@25.5.2", "", { "dependencies": { "undici-types": "~7.18.0" } }, "sha512-tO4ZIRKNC+MDWV4qKVZe3Ql/woTnmHDr5JD8UI5hn2pwBrHEwOEMZK7WlNb5RKB6EoJ02gwmQS9OrjuFnZYdpg=="],
 
     "@types/statuses": ["@types/statuses@2.0.6", "", {}, "sha512-xMAgYwceFhRA2zY+XbEA7mxYbA093wdiW8Vu6gZPGWy9cmOyU9XesH1tNcEWsKFd5Vzrqx5T3D38PWx1FIIXkA=="],
 
     "accepts": ["accepts@2.0.0", "", { "dependencies": { "mime-types": "3.0.1", "negotiator": "1.0.0" } }, "sha512-5cvg6CtKwfgdmVqY1WIiXKc3Q1bkRqGLi+2W/6ao+6Y7gu/RCwRuAhGEzh5B4KlszSuTLgZYuqFqo5bImjNKng=="],
 
-    "afpnews-api": ["afpnews-api@2.4.0", "", { "dependencies": { "@types/btoa-lite": "1.0.2", "@types/moo": "0.5.10", "@types/nearley": "2.11.5", "@types/node": "25.2.3", "@types/statuses": "2.0.6", "btoa-lite": "1.0.0", "events": "3.3.0", "moo": "0.5.2", "nearley": "2.20.1", "statuses": "2.0.2", "zod": "4.3.6" } }, "sha512-VErqy0P5h2sjVjaSoIHF1uFsKx16PYKZUFBCCUmHsPdPM99lQhk0ClibPU7Wk9i34BtUiqLcQxrmTdCOj9ikpA=="],
+    "afpnews-api": ["afpnews-api@2.4.1", "", { "dependencies": { "@types/btoa-lite": "^1.0.2", "@types/moo": "^0.5.10", "@types/nearley": "^2.11.5", "@types/node": "^25.5.0", "@types/statuses": "^2.0.6", "btoa-lite": "^1.0.0", "events": "^3.3.0", "moo": "^0.5.3", "nearley": "^2.20.1", "statuses": "^2.0.2", "zod": "^4.3.6" } }, "sha512-A/1vZeq/qqLph11XYZbXqhs0qUSQ9zi/4YK2a+XVpn5iF6q1VXWABW19lL0X2mZM7RsdsEUW+rMIhDmHbdFsBw=="],
 
     "ajv": ["ajv@8.17.1", "", { "dependencies": { "fast-deep-equal": "3.1.3", "fast-uri": "3.1.0", "json-schema-traverse": "1.0.0", "require-from-string": "2.0.2" } }, "sha512-B/gBuNg5SiMTrPkC+A2+cW0RszwxYmn6VYxB/inlBStS5nx6xHIt/ehKRhIMhqusl7a8LjQoZnjCs5vhwxOQ1g=="],
 
@@ -156,7 +160,7 @@
 
     "isexe": ["isexe@2.0.0", "", {}, "sha512-RHxMLp9lnKHGHRng9QFhRCMbYAcVpn69smSGcq3f36xjgVVWThj4qqLbTLlq7Ssj8B+fIQ1EuCEGI2lKsyQeIw=="],
 
-    "jose": ["jose@6.2.1", "", {}, "sha512-jUaKr1yrbfaImV7R2TN/b3IcZzsw38/chqMpo2XJ7i2F8AfM/lA4G1goC3JVEwg0H7UldTmSt3P68nt31W7/mw=="],
+    "jose": ["jose@6.2.2", "", {}, "sha512-d7kPDd34KO/YnzaDOlikGpOurfF0ByC2sEV4cANCtdqLlTfBlw2p14O/5d/zv40gJPbIQxfES3nSx1/oYNyuZQ=="],
 
     "json-schema-traverse": ["json-schema-traverse@1.0.0", "", {}, "sha512-NM8/P9n3XjXhIZn1lLhkFaACTOURQXjWhV4BA/RnOv8xvgqtqpAX9IO4mRQxSx1Rlo4tqzeqb0sOlruaOy3dug=="],
 
@@ -174,7 +178,7 @@
 
     "mime-types": ["mime-types@3.0.1", "", { "dependencies": { "mime-db": "1.54.0" } }, "sha512-xRc4oEhT6eaBpU1XF7AjpOFD+xQmXNB5OVKwp4tqCuBpHLS/ZbBDrc07mYTDqVMg6PfxUjjNp85O6Cd2Z/5HWA=="],
 
-    "moo": ["moo@0.5.2", "", {}, "sha512-iSAJLHYKnX41mKcJKjqvnAN9sf0LMDTXDEvFv+ffuRR9a1MIuXLjMNL6EsnDHSkKLTWNqQQ5uo61P4EbU4NU+Q=="],
+    "moo": ["moo@0.5.3", "", {}, "sha512-m2fmM2dDm7GZQsY7KK2cme8agi+AAljILjQnof7p1ZMDe6dQ4bdnSMx0cPppudoeNv5hEFQirN6u+O4fDE0IWA=="],
 
     "ms": ["ms@2.1.2", "", {}, "sha512-sGkPx+VjMtmA6MX27oA4FBFELFCZZ4S4XqeGOXCv68tT+jb3vk/RyaKWP0PTKyWtmLSM0b+adUTEvbs1PEaH2w=="],
 
@@ -270,7 +274,7 @@
 
     "@tokenizer/inflate/debug": ["debug@4.4.3", "", { "dependencies": { "ms": "2.1.3" } }, "sha512-RGwwWnwQvkVfavKVt22FGLw+xYSdzARwm0ru6DhTVA3umU5hZc28V3kO4stgYryrTlLpuvgI9GiijltAjNbcqA=="],
 
-    "afpnews-api/@types/node": ["@types/node@25.2.3", "", { "dependencies": { "undici-types": "7.16.0" } }, "sha512-m0jEgYlYz+mDJZ2+F4v8D1AyQb+QzsNqRuI7xg1VQX/KlKS0qT9r1Mo16yo5F/MtifXFgaofIFsdFMox2SxIbQ=="],
+    "afpnews-api/@types/node": ["@types/node@25.5.0", "", { "dependencies": { "undici-types": "~7.18.0" } }, "sha512-jp2P3tQMSxWugkCUKLRPVUpGaL5MVFwF8RDuSRztfwgN1wmqJeMSbKlnEtQqU8UrhTmzEmZdu2I6v2dpp7XIxw=="],
 
     "body-parser/debug": ["debug@4.4.3", "", { "dependencies": { "ms": "2.1.3" } }, "sha512-RGwwWnwQvkVfavKVt22FGLw+xYSdzARwm0ru6DhTVA3umU5hZc28V3kO4stgYryrTlLpuvgI9GiijltAjNbcqA=="],
 
@@ -280,6 +284,8 @@
 
     "finalhandler/debug": ["debug@4.4.3", "", { "dependencies": { "ms": "2.1.3" } }, "sha512-RGwwWnwQvkVfavKVt22FGLw+xYSdzARwm0ru6DhTVA3umU5hZc28V3kO4stgYryrTlLpuvgI9GiijltAjNbcqA=="],
 
+    "nearley/moo": ["moo@0.5.2", "", {}, "sha512-iSAJLHYKnX41mKcJKjqvnAN9sf0LMDTXDEvFv+ffuRR9a1MIuXLjMNL6EsnDHSkKLTWNqQQ5uo61P4EbU4NU+Q=="],
+
     "router/debug": ["debug@4.4.3", "", { "dependencies": { "ms": "2.1.3" } }, "sha512-RGwwWnwQvkVfavKVt22FGLw+xYSdzARwm0ru6DhTVA3umU5hZc28V3kO4stgYryrTlLpuvgI9GiijltAjNbcqA=="],
 
     "send/debug": ["debug@4.4.3", "", { "dependencies": { "ms": "2.1.3" } }, "sha512-RGwwWnwQvkVfavKVt22FGLw+xYSdzARwm0ru6DhTVA3umU5hZc28V3kO4stgYryrTlLpuvgI9GiijltAjNbcqA=="],
@@ -288,8 +294,6 @@
 
     "@tokenizer/inflate/debug/ms": ["ms@2.1.3", "", {}, "sha512-6FlzubTLZG3J2a/NVCAleEhjzq5oxgHyaCU9yYXvcLsvoVaHJq/s5xXI6/XXP6tz7R9xAOtHnSO/tXtF3WRTlA=="],
 
-    "afpnews-api/@types/node/undici-types": ["undici-types@7.16.0", "", {}, "sha512-Zz+aZWSj8LE6zoxD+xrjh4VfkIG8Ya6LvYkZqtUQGJPZjYl53ypCaUwWqo7eI0x66KBGeRo+mlBEkMSeSZ38Nw=="],
-
     "body-parser/debug/ms": ["ms@2.1.3", "", {}, "sha512-6FlzubTLZG3J2a/NVCAleEhjzq5oxgHyaCU9yYXvcLsvoVaHJq/s5xXI6/XXP6tz7R9xAOtHnSO/tXtF3WRTlA=="],
 
     "express/debug/ms": ["ms@2.1.3", "", {}, "sha512-6FlzubTLZG3J2a/NVCAleEhjzq5oxgHyaCU9yYXvcLsvoVaHJq/s5xXI6/XXP6tz7R9xAOtHnSO/tXtF3WRTlA=="],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "afpnews-mcp-server",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "description": "",
   "main": "src/index.ts",
   "type": "module",
@@ -12,25 +12,28 @@
     "dev": "bun --watch src/index.ts",
     "start": "bun src/index.ts",
     "typecheck": "tsc --noEmit",
-    "test": "bun test src/__tests__/format.test.ts src/__tests__/format-media.test.ts src/__tests__/types-media.test.ts src/__tests__/index-auth.test.ts src/__tests__/server.test.ts src/__tests__/get-media.test.ts && bun test src/__tests__/create-server.test.ts"
+    "typecheck:test": "tsc --noEmit -p tsconfig.test.json",
+    "test": "bun test src/__tests__/",
+    "test:coverage": "bun test --coverage --coverage-reporter=text src/__tests__/",
+    "test:coverage:lcov": "bun test --coverage --coverage-reporter=text --coverage-reporter=lcov --coverage-dir coverage src/__tests__/"
   },
   "keywords": [],
   "author": "",
   "license": "ISC",
-  "packageManager": "bun@1.3.10",
+  "packageManager": "bun@1.3.11",
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.27.1",
+    "@modelcontextprotocol/sdk": "^1.29.0",
     "elysia": "^1.4.28",
     "elysia-rate-limit": "^4.5.1",
-    "jose": "^6.2.1"
+    "jose": "^6.2.2"
   },
   "peerDependencies": {
     "afpnews-api": "^2.4.0",
     "zod": "^4.3.6"
   },
   "devDependencies": {
-    "@types/node": "^25.5.0",
-    "afpnews-api": "^2.4.0",
+    "@types/node": "^25.5.2",
+    "afpnews-api": "^2.4.1",
     "zod": "^4.3.6"
   }
 }

package/src/__tests__/create-server.test.ts

@@ -1,74 +1,72 @@
-import { afterAll, beforeEach, describe, expect, it, mock } from 'bun:test';
-
-// Define shared mock functions at module scope
-const authenticateMock = mock();
-const apiCoreInstances: Array<{ token?: unknown; config?: unknown }> = [];
-const registerToolsMock = mock();
-const registerResourcesMock = mock();
-const registerPromptsMock = mock();
-
-// mock.module() is hoisted by bun:test before imports are resolved
-mock.module('afpnews-api', () => {
-  class MockApiCore {
-    token?: unknown;
-    config?: unknown;
-
-    constructor(config?: unknown) {
-      this.config = config;
-      apiCoreInstances.push(this);
-    }
-
-    authenticate = authenticateMock;
-  }
-
-  return { ApiCore: MockApiCore };
-});
-
-mock.module('../tools/index.js', () => ({ registerTools: registerToolsMock }));
-mock.module('../resources/index.js', () => ({ registerResources: registerResourcesMock }));
-mock.module('../prompts/index.js', () => ({ registerPrompts: registerPromptsMock }));
-
-// Restore module mocks after all tests in this file
-afterAll(() => mock.restore());
+import { describe, expect, it, mock } from 'bun:test';
 
 describe('createServer', () => {
-  beforeEach(() => {
-    authenticateMock.mockReset();
-    authenticateMock.mockResolvedValue(undefined);
-    registerToolsMock.mockReset();
-    registerResourcesMock.mockReset();
-    registerPromptsMock.mockReset();
-    apiCoreInstances.length = 0;
-  });
-
   it('authenticates with provided credentials', async () => {
-    // Dynamic import ensures mock.module() is active when the module is loaded
+    const authenticateMock = mock().mockResolvedValue(undefined);
+
+    mock.module('afpnews-api', () => ({
+      ApiCore: class {
+        token?: unknown;
+        config?: unknown;
+        constructor(config?: unknown) { this.config = config; }
+        authenticate = authenticateMock;
+      },
+    }));
+
     const { createServer } = await import('../mcp-server.js');
     await createServer({ apiKey: 'api-key', username: 'user', password: 'pass' });
 
-    expect(apiCoreInstances).toHaveLength(1);
-    expect(apiCoreInstances[0].config).toEqual({ apiKey: 'api-key' });
     expect(authenticateMock).toHaveBeenCalledWith({ username: 'user', password: 'pass' });
+    mock.restore();
   });
 
   it('passes baseUrl to ApiCore when provided', async () => {
+    let capturedConfig: unknown;
+    mock.module('afpnews-api', () => ({
+      ApiCore: class {
+        token?: unknown;
+        constructor(config?: unknown) { capturedConfig = config; }
+        authenticate = mock().mockResolvedValue(undefined);
+      },
+    }));
+
     const { createServer } = await import('../mcp-server.js');
     await createServer({ apiKey: 'api-key', username: 'user', password: 'pass', baseUrl: 'https://custom.api.com' });
 
-    expect(apiCoreInstances[0].config).toEqual({ apiKey: 'api-key', baseUrl: 'https://custom.api.com' });
+    expect(capturedConfig).toEqual({ apiKey: 'api-key', baseUrl: 'https://custom.api.com' });
+    mock.restore();
   });
 
   it('does not set baseUrl on ApiCore when omitted', async () => {
+    let capturedConfig: unknown;
+    mock.module('afpnews-api', () => ({
+      ApiCore: class {
+        token?: unknown;
+        constructor(config?: unknown) { capturedConfig = config; }
+        authenticate = mock().mockResolvedValue(undefined);
+      },
+    }));
+
     const { createServer } = await import('../mcp-server.js');
     await createServer({ apiKey: 'api-key', username: 'user', password: 'pass' });
 
-    expect(apiCoreInstances[0].config).not.toHaveProperty('baseUrl');
+    expect(capturedConfig).not.toHaveProperty('baseUrl');
+    mock.restore();
   });
 
   it('throws on missing credentials', async () => {
+    mock.module('afpnews-api', () => ({
+      ApiCore: class {
+        token?: unknown;
+        constructor() {}
+        authenticate = mock().mockResolvedValue(undefined);
+      },
+    }));
+
     const { createServer } = await import('../mcp-server.js');
     await expect(
       createServer({ apiKey: 'api-key', username: 'user', password: '' }),
     ).rejects.toThrow('Missing authentication');
+    mock.restore();
   });
 });

package/src/__tests__/format-media.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect } from 'vitest';
+import { describe, it, expect } from 'bun:test';
 import { extractRenditions, formatMediaDocument, formatMediaDocumentsAsJson, formatMediaDocumentsAsCsv, MEDIA_RENDITION_ROLE_MAP } from '../utils/format-media.js';
 
 // Fixture bagItem reprenant la structure réelle AFP
@@ -148,7 +148,7 @@ describe('formatMediaDocumentsAsCsv', () => {
       class: 'picture',
       renditions: { thumbnail: { href: 'https://example.com/t.jpg', width: 320, height: 213 } },
     }];
-    const r = formatMediaDocumentsAsCsv(docs as any);
+    const r = formatMediaDocumentsAsCsv(docs);
     expect(r.content.text).toContain('newsml.test');
     expect(r.content.text).toContain('My Photo');
     expect(r.content.text).toContain('JANE / AFP');
@@ -161,13 +161,13 @@ describe('formatMediaDocumentsAsCsv', () => {
       caption: 'A caption, with "quotes"',
       renditions: {},
     }];
-    const r = formatMediaDocumentsAsCsv(docs as any);
+    const r = formatMediaDocumentsAsCsv(docs);
     expect(r.content.text).toContain('"A caption, with ""quotes"""');
   });
 
   it('handles missing thumbnail href gracefully', () => {
     const docs = [{ uno: 'newsml.test', renditions: {} }];
-    const r = formatMediaDocumentsAsCsv(docs as any);
+    const r = formatMediaDocumentsAsCsv(docs);
     expect(r.content.text).toContain('newsml.test');
   });
 });
@@ -179,13 +179,13 @@ describe('formatMediaDocumentsAsJson', () => {
   ];
 
   it('returns { content: TextContent, truncated: boolean }', () => {
-    const r = formatMediaDocumentsAsJson(docs as any);
+    const r = formatMediaDocumentsAsJson(docs);
     expect(r.content.type).toBe('text');
     expect(typeof r.truncated).toBe('boolean');
   });
 
   it('output JSON contains total metadata and documents array', () => {
-    const r = formatMediaDocumentsAsJson(docs as any, { total: 100, offset: 0 });
+    const r = formatMediaDocumentsAsJson(docs, { total: 100, offset: 0 });
     const parsed = JSON.parse(r.content.text);
     expect(parsed.total).toBe(100);
     expect(parsed.offset).toBe(0);
@@ -196,7 +196,7 @@ describe('formatMediaDocumentsAsJson', () => {
   });
 
   it('truncated flag is false when content is small', () => {
-    const r = formatMediaDocumentsAsJson(docs as any, { total: 100, offset: 0 });
+    const r = formatMediaDocumentsAsJson(docs, { total: 100, offset: 0 });
     expect(r.truncated).toBe(false);
   });
 
@@ -208,7 +208,7 @@ describe('formatMediaDocumentsAsJson', () => {
       caption: 'A very long caption that also takes up considerable space in the JSON output here',
       renditions: { thumbnail: { href: `https://example.com/thumb-${i}.jpg`, width: 320, height: 213 } },
     }));
-    const r = formatMediaDocumentsAsJson(largeDocs as any, { total: 500, offset: 0 });
+    const r = formatMediaDocumentsAsJson(largeDocs, { total: 500, offset: 0 });
     expect(r.truncated).toBe(true);
     const parsed = JSON.parse(r.content.text);
     expect(parsed.shown).toBeLessThan(500);

package/src/__tests__/format.test.ts

@@ -38,10 +38,10 @@ describe('formatDocument', () => {
     expect(result.text).not.toContain('Advisory:');
   });
 
-  it('truncates to 4 paragraphs when fullText=false', () => {
+  it('truncates to 2 paragraphs when fullText=false', () => {
     const result = formatDocument(FIXTURE_DOC, false);
-    expect(result.text).toContain('Fourth paragraph wraps up.');
-    expect(result.text).not.toContain('Fifth paragraph is extra content.');
+    expect(result.text).toContain('Second paragraph with more details.');
+    expect(result.text).not.toContain('Third paragraph continues.');
   });
 
   it('includes all paragraphs when fullText=true', () => {

package/src/__tests__/get-media-handler.test.ts

@@ -0,0 +1,120 @@
+import { beforeEach, describe, expect, it, mock } from 'bun:test';
+import { afpGetMediaTool } from '../tools/get-media.js';
+
+const ORIGINAL_FETCH = globalThis.fetch;
+
+function getText(result: any, index = 0): string {
+  return result.content[index]?.text ?? '';
+}
+
+describe('afpGetMediaTool.handler', () => {
+  beforeEach(() => {
+    globalThis.fetch = ORIGINAL_FETCH;
+  });
+
+  it('returns error when media document is missing', async () => {
+    const apicore = { get: mock().mockResolvedValue(undefined) };
+
+    const result = await afpGetMediaTool.handler(apicore, { uno: 'MISSING' });
+
+    expect('isError' in result && result.isError).toBe(true);
+    expect(getText(result)).toContain('Media document not found');
+  });
+
+  it('returns metadata only when embed=false', async () => {
+    const apicore = {
+      get: mock().mockResolvedValue({
+        uno: 'MEDIA-001',
+        title: 'Photo',
+        class: 'picture',
+        creditLine: 'AFP',
+        bagItem: [{ medias: [{ role: 'Thumbnail', href: 'https://example.com/thumb.jpg', width: 320, height: 200, type: 'Photo' }] }],
+      }),
+    };
+
+    const result = await afpGetMediaTool.handler(apicore, { uno: 'MEDIA-001' });
+
+    expect(result.content).toHaveLength(1);
+    expect(getText(result)).toContain('## Photo');
+    expect(getText(result)).toContain('thumbnail: https://example.com/thumb.jpg');
+  });
+
+  it('warns when trying to embed an SVG graphic', async () => {
+    const apicore = {
+      get: mock().mockResolvedValue({
+        uno: 'GRAPHIC-001',
+        title: 'Graphic',
+        class: 'graphic',
+        bagItem: [{ medias: [{ role: 'Preview', href: 'https://example.com/graphic.svg', width: 1200, height: 800, type: 'Graphic' }] }],
+      }),
+    };
+
+    const result = await afpGetMediaTool.handler(apicore, { uno: 'GRAPHIC-001', embed: true });
+
+    expect(result.content).toHaveLength(2);
+    expect(getText(result, 1)).toContain('SVG graphics cannot be embedded');
+  });
+
+  it('warns when no rendition is available for embedding', async () => {
+    const apicore = {
+      get: mock().mockResolvedValue({
+        uno: 'MEDIA-EMPTY',
+        title: 'Empty',
+        class: 'picture',
+        bagItem: [],
+      }),
+    };
+
+    const result = await afpGetMediaTool.handler(apicore, { uno: 'MEDIA-EMPTY', embed: true });
+
+    expect(result.content).toHaveLength(2);
+    expect(getText(result, 1)).toContain('no rendition available');
+  });
+
+  it('embeds a video thumbnail and adds the poster-frame note', async () => {
+    globalThis.fetch = mock().mockResolvedValue({
+      ok: true,
+      arrayBuffer: async () => new Uint8Array([1, 2, 3]).buffer,
+      headers: new Headers({ 'content-type': 'image/jpeg' }),
+    } as Response);
+
+    const apicore = {
+      get: mock().mockResolvedValue({
+        uno: 'VIDEO-001',
+        title: 'Video',
+        class: 'video',
+        bagItem: [{ medias: [{ role: 'Thumbnail', href: 'https://example.com/poster.jpg', width: 320, height: 180, type: 'Photo' }] }],
+      }),
+    };
+
+    const result = await afpGetMediaTool.handler(apicore, { uno: 'VIDEO-001', embed: true, rendition: 'highdef' });
+
+    expect(result.content).toHaveLength(2);
+    expect(getText(result)).toContain('poster frame only');
+    expect(result.content[1]).toEqual(expect.objectContaining({ type: 'image', mimeType: 'image/jpeg' }));
+  });
+
+  it('returns a warning when image fetch fails', async () => {
+    globalThis.fetch = mock().mockResolvedValue({
+      ok: false,
+      status: 403,
+      statusText: 'Forbidden',
+      headers: new Headers(),
+    } as Response);
+
+    const apicore = {
+      get: mock().mockResolvedValue({
+        uno: 'MEDIA-403',
+        title: 'Photo',
+        class: 'picture',
+        bagItem: [{ medias: [{ role: 'Preview', href: 'https://example.com/prev.jpg', width: 1200, height: 800, type: 'Photo' }] }],
+      }),
+    };
+
+    const result = await afpGetMediaTool.handler(apicore, { uno: 'MEDIA-403', embed: true });
+
+    expect(result.content).toHaveLength(2);
+    expect(getText(result, 1)).toContain('image embed failed');
+    expect(getText(result, 1)).toContain('HTTP 403');
+  });
+});

package/src/__tests__/get-media.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect } from 'vitest';
+import { describe, it, expect } from 'bun:test';
 import { inferMimeType, selectRenditionForEmbed } from '../tools/get-media.js';
 import type { MediaRenditions } from '../utils/types.js';
 

package/src/__tests__/list-facets.test.ts

@@ -0,0 +1,52 @@
+import { describe, it, expect, mock } from 'bun:test';
+import { afpListFacetsTool } from '../tools/list-facets.js';
+
+describe('afpListFacetsTool', () => {
+  it('returns no-results message when list is empty', async () => {
+    const apicore = { list: mock().mockResolvedValue({ keywords: [] }) };
+
+    const result = await afpListFacetsTool.handler(apicore, { facet: 'slug' });
+
+    expect(result.content).toHaveLength(1);
+    expect(result.content[0]!.text).toContain('No facet values found');
+  });
+
+  it('applies AFP text defaults with explicit lang outside preset', async () => {
+    const apicore = { list: mock().mockResolvedValue({ keywords: [{ name: 'economy', count: 42 }] }) };
+
+    await afpListFacetsTool.handler(apicore, { facet: 'slug', lang: 'en' });
+
+    const [, params] = apicore.list.mock.calls[0]!;
+    expect(params).toEqual({ class: ['text'], provider: ['afp'], langs: ['en'], size: 10 });
+  });
+
+  it('adds truncation hint in json output when facet list is too large', async () => {
+    const keywords = Array.from({ length: 3000 }, (_, i) => ({ name: `topic-${i}-${'x'.repeat(30)}`, count: i + 1 }));
+    const apicore = { list: mock().mockResolvedValue({ keywords }) };
+
+    const result = await afpListFacetsTool.handler(apicore, { facet: 'slug', format: 'json' });
+
+    expect(result.content.length).toBeGreaterThan(1);
+    expect(result.content[1]!.text).toContain('Response truncated');
+  });
+
+  it('adds truncation hint in csv output when facet list is too large', async () => {
+    const keywords = Array.from({ length: 3000 }, (_, i) => ({ name: `topic-${i}-${'x'.repeat(30)}`, count: i + 1 }));
+    const apicore = { list: mock().mockResolvedValue({ keywords }) };
+
+    const result = await afpListFacetsTool.handler(apicore, { facet: 'slug', format: 'csv' });
+
+    expect(result.content.length).toBeGreaterThan(1);
+    expect(result.content[1]!.text).toContain('Response truncated');
+  });
+
+  it('adds truncation hint in markdown output when facet list is too large', async () => {
+    const keywords = Array.from({ length: 3000 }, (_, i) => ({ name: `topic-${i}-${'x'.repeat(30)}`, count: i + 1 }));
+    const apicore = { list: mock().mockResolvedValue({ keywords }) };
+
+    const result = await afpListFacetsTool.handler(apicore, { facet: 'slug' });
+
+    expect(result.content.length).toBeGreaterThan(1);
+    expect(result.content[1]!.text).toContain('Response truncated');
+  });
+});