afpnews-mcp-server 2.0.0 → 2.0.2

package/.serena/project.yml

@@ -3,21 +3,26 @@ project_name: "afpnews-mcp-server"
 
 
 # list of languages for which language servers are started; choose from:
-#   al                  bash                clojure             cpp                 csharp
-#   csharp_omnisharp    dart                elixir              elm                 erlang
-#   fortran             fsharp              go                  groovy              haskell
-#   java                julia               kotlin              lua                 markdown
-#   matlab              nix                 pascal              perl                php
-#   php_phpactor        powershell          python              python_jedi         r
-#   rego                ruby                ruby_solargraph     rust                scala
-#   swift               terraform           toml                typescript          typescript_vts
-#   vue                 yaml                zig
+#   al                  angular             ansible             bash                clojure
+#   cpp                 cpp_ccls            crystal             csharp              csharp_omnisharp
+#   dart                elixir              elm                 erlang              fortran
+#   fsharp              go                  groovy              haskell             haxe
+#   hlsl                html                java                json                julia
+#   kotlin              lean4               lua                 luau                markdown
+#   matlab              msl                 nix                 ocaml               pascal
+#   perl                php                 php_phpactor        powershell          python
+#   python_jedi         python_ty           r                   rego                ruby
+#   ruby_solargraph     rust                scala               scss                solidity
+#   swift               systemverilog       terraform           toml                typescript
+#   typescript_vts      vue                 yaml                zig
 #   (This list may be outdated. For the current list, see values of Language enum here:
 #   https://github.com/oraios/serena/blob/main/src/solidlsp/ls_config.py
 #   For some languages, there are alternative language servers, e.g. csharp_omnisharp, ruby_solargraph.)
 # Note:
 #   - For C, use cpp
 #   - For JavaScript, use typescript
+#   - For Angular projects, use angular (subsumes typescript+html; requires `npm install` in the project root)
+#   - For SCSS / Sass / plain CSS, use scss (some-sass-language-server handles all three)
 #   - For Free Pascal/Lazarus, use pascal
 # Special requirements:
 #   Some languages require additional setup/installations.
@@ -52,52 +57,19 @@ ignored_paths: []
 # Added on 2025-04-18
 read_only: false
 
-# list of tool names to exclude. We recommend not excluding any tools, see the readme for more details.
-# Below is the complete list of tools for convenience.
-# To make sure you have the latest list of tools, and to view their descriptions, 
-# execute `uv run scripts/print_tool_overview.py`.
-#
-#  * `activate_project`: Activates a project by name.
-#  * `check_onboarding_performed`: Checks whether project onboarding was already performed.
-#  * `create_text_file`: Creates/overwrites a file in the project directory.
-#  * `delete_lines`: Deletes a range of lines within a file.
-#  * `delete_memory`: Deletes a memory from Serena's project-specific memory store.
-#  * `execute_shell_command`: Executes a shell command.
-#  * `find_referencing_code_snippets`: Finds code snippets in which the symbol at the given location is referenced.
-#  * `find_referencing_symbols`: Finds symbols that reference the symbol at the given location (optionally filtered by type).
-#  * `find_symbol`: Performs a global (or local) search for symbols with/containing a given name/substring (optionally filtered by type).
-#  * `get_current_config`: Prints the current configuration of the agent, including the active and available projects, tools, contexts, and modes.
-#  * `get_symbols_overview`: Gets an overview of the top-level symbols defined in a given file.
-#  * `initial_instructions`: Gets the initial instructions for the current project.
-#     Should only be used in settings where the system prompt cannot be set,
-#     e.g. in clients you have no control over, like Claude Desktop.
-#  * `insert_after_symbol`: Inserts content after the end of the definition of a given symbol.
-#  * `insert_at_line`: Inserts content at a given line in a file.
-#  * `insert_before_symbol`: Inserts content before the beginning of the definition of a given symbol.
-#  * `list_dir`: Lists files and directories in the given directory (optionally with recursion).
-#  * `list_memories`: Lists memories in Serena's project-specific memory store.
-#  * `onboarding`: Performs onboarding (identifying the project structure and essential tasks, e.g. for testing or building).
-#  * `prepare_for_new_conversation`: Provides instructions for preparing for a new conversation (in order to continue with the necessary context).
-#  * `read_file`: Reads a file within the project directory.
-#  * `read_memory`: Reads the memory with the given name from Serena's project-specific memory store.
-#  * `remove_project`: Removes a project from the Serena configuration.
-#  * `replace_lines`: Replaces a range of lines within a file with new content.
-#  * `replace_symbol_body`: Replaces the full definition of a symbol.
-#  * `restart_language_server`: Restarts the language server, may be necessary when edits not through Serena happen.
-#  * `search_for_pattern`: Performs a search for a pattern in the project.
-#  * `summarize_changes`: Provides instructions for summarizing the changes made to the codebase.
-#  * `switch_modes`: Activates modes by providing a list of their names
-#  * `think_about_collected_information`: Thinking tool for pondering the completeness of collected information.
-#  * `think_about_task_adherence`: Thinking tool for determining whether the agent is still on track with the current task.
-#  * `think_about_whether_you_are_done`: Thinking tool for determining whether the task is truly completed.
-#  * `write_memory`: Writes a named memory (for future reference) to Serena's project-specific memory store.
+# list of tool names to exclude.
+# This extends the existing exclusions (e.g. from the global configuration)
+# Find the list of tools here: https://oraios.github.io/serena/01-about/035_tools.html
 excluded_tools: []
 
-# list of tools to include that would otherwise be disabled (particularly optional tools that are disabled by default)
+# list of tools to include that would otherwise be disabled (particularly optional tools that are disabled by default).
+# This extends the existing inclusions (e.g. from the global configuration).
+# Find the list of tools here: https://oraios.github.io/serena/01-about/035_tools.html
 included_optional_tools: []
 
 # fixed set of tools to use as the base tool set (if non-empty), replacing Serena's default set of tools.
 # This cannot be combined with non-empty excluded_tools or included_optional_tools.
+# Find the list of tools here: https://oraios.github.io/serena/01-about/035_tools.html
 fixed_tools: []
 
 # list of mode names to that are always to be included in the set of active modes
@@ -108,11 +80,14 @@ fixed_tools: []
 # Set this to a list of mode names to always include the respective modes for this project.
 base_modes:
 
-# list of mode names that are to be activated by default.
-# The full set of modes to be activated is base_modes + default_modes.
-# If the setting is undefined, the default_modes from the global configuration (serena_config.yml) apply.
+# list of mode names that are to be activated by default, overriding the setting in the global configuration.
+# The full set of modes to be activated is base_modes (from global config) + default_modes + added_modes.
+# If the setting is undefined/empty, the default_modes from the global configuration (serena_config.yml) apply.
 # Otherwise, this overrides the setting from the global configuration (serena_config.yml).
+# Therefore, you can set this to [] if you do not want the default modes defined in the global config to apply
+# for this project.
 # This setting can, in turn, be overridden by CLI parameters (--mode).
+# See https://oraios.github.io/serena/02-usage/050_configuration.html#modes
 default_modes:
 
 # initial prompt for the project. It will always be given to the LLM upon activating the project
@@ -133,3 +108,33 @@ 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: {}
+
+# list of mode names to be activated additionally for this project, e.g. ["query-projects"]
+# The full set of modes to be activated is base_modes (from global config) + default_modes + added_modes.
+# See https://oraios.github.io/serena/02-usage/050_configuration.html#modes
+added_modes:
+
+# list of additional workspace folder paths for cross-package reference support (e.g. in monorepos).
+# Paths can be absolute or relative to the project root.
+# Each folder is registered as an LSP workspace folder, enabling language servers to discover
+# symbols and references across package boundaries.
+# Currently supported for: TypeScript.
+# Example:
+#   additional_workspace_folders:
+#     - ../sibling-package
+#     - ../shared-lib
+additional_workspace_folders: []

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.2",
   "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';