Skip to content

Conversation

@Markolissimo
Copy link

Summary

This PR adds Python test coverage to CI and improves the MCP developer experience with Windows compatibility, version reporting, and better tooling.

Changes

1. CI: Python Test Job

  • Added python-test job to .github/workflows/ci.yml
  • Installs and tests both core and tools packages
  • Runs in parallel with Node tests
  • Blocks merge if Python tests fail

Fixes: #116

2. MCP Scripts: Windows Compatibility

  • Enhanced core/setup_mcp.py and core/verify_mcp.py for Windows
  • Conditional ANSI color support (graceful degradation without colorama)
  • Platform-aware command suggestions (PowerShell vs bash)
  • Better error messages with stdout/stderr capture

Fixes: #117, #118

3. MCP Test Script: CLI Flags

  • Created scripts/test-mcp.ts with CLI argument parsing
  • Supports --list-tools, --tool <name>, --category <cat>, --help
  • Backward compatible (no flags = default test suite)

Fixes: #120

4. MCP Smoke Test

  • Added core/tests/test_mcp_server.py with import/structure tests
  • Skips if MCP dependencies unavailable
  • Runs in CI via new Python test job

Fixes: #121

Testing

  • All Python tests pass locally (pytest core/tests tools/tests)
  • MCP scripts tested on Windows 11 PowerShell
  • MCP test script tested with all CLI flags
  • CI workflow syntax validated

Checklist

  • Follows Conventional Commits
  • Added tests for new functionality
  • Documentation updated (inline comments, help text)
  • No breaking changes

@RichardTang-Aden
Copy link
Collaborator

We did a major refactor today. Can you merge from main and see if the tests can work again?

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