refactor: improve provider declaration and add tests for all of them #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

neilime wants to merge 1 commit into writingmate:main from neilime:refactor-and-tests

neilime commented Dec 22, 2025 •

edited

Loading

Summary

Refactored MCP tool registration to be provider-driven: server.ts now loops over a provider list and calls registerTool uniformly.
Added a shared Provider type to standardize provider metadata (tool name/description/schema) plus the generateImage implementation.
Updated each provider module to export only its provider object (single public export per provider file).

Testing

Made server.ts import-safe and added registerProviders() to enable unit testing without starting the server process.
Added unit tests using node:test (organized with describe/it).
Added provider-specific tests that stub external dependencies via dependency injection (no real network/API calls and no filesystem writes).
Updated npm test to run Node's test runner with TS support: node --import tsx --test.


          refactor: improve provider declaration and add tests for all of them

89619c5

Signed-off-by: Emilien Escalle <[email protected]>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet