Add test github action

Author: niechen

.github/workflows/ci.yml

@@ -0,0 +1,42 @@
+name: Python CI
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Set up Python
+      uses: actions/setup-python@v5
+      with:
+        python-version: '3.10'
+    - name: Install uv
+      uses: astral-sh/setup-uv@v5
+      with:
+        enable-cache: true
+    - name: Run tests
+      run: |
+        uv sync --group dev
+        uv run pytest
+
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Set up Python
+      uses: actions/setup-python@v5
+      with:
+        python-version: '3.10'
+    - name: Install uv
+      uses: astral-sh/setup-uv@v5
+      with:
+        enable-cache: true
+    - name: Check code style with ruff
+      run: |
+        uv sync --group dev
+        uv run ruff check src/ tests/

pyproject.toml

@@ -31,6 +31,7 @@ dependencies = [
     "openai>=1.72.0",
     "aiohttp>=3.11.16",
     "watchfiles>=1.0.4",
+    "duckdb>=1.2.2",
 ]
 
 [project.urls]
@@ -52,8 +53,14 @@ line-length = 120
 fix = true
 
 [tool.ruff.lint]
-select = ["F", "E", "W", "I"]
-fixable = ["I", "F401"]
+select = ["E", "F", "W", "Q", "I"]
+fixable = ["ALL"]
+ignore = ["E501"]
 
 [dependency-groups]
-dev = ["ipython>=8.34.0", "pytest>=8.3.5"]
+dev = [
+    "ipython>=8.34.0",
+    "pytest>=8.3.5",
+    "pytest-asyncio>=0.26.0",
+    "ruff>=0.11.4",
+]

src/mcpm/commands/inspector.py

@@ -2,11 +2,12 @@
 MCPM Inspector command for examining MCP servers through a user interface
 """
 
-import click
 import os
-import subprocess
 import shlex
+import subprocess
 import sys
+
+import click
 from rich.console import Console
 from rich.panel import Panel
 

src/mcpm/commands/list.py

@@ -46,8 +46,8 @@ def list():
         return
 
     # Count the configured servers
-    server_count = len(servers)
-    stashed_count = len(stashed_servers)
+    len(servers)
+    len(stashed_servers)
 
     # Print active servers
     if servers:

src/mcpm/monitor/duckdb.py

@@ -67,7 +67,7 @@ def _initialize_storage_impl(self) -> bool:
             # Create the events table if it doesn't exist using identity column for auto-incrementing
             self.connection.execute("""
                 CREATE SEQUENCE IF NOT EXISTS event_id_seq;
-                
+
                 CREATE TABLE IF NOT EXISTS monitor_events (
                     id INTEGER DEFAULT nextval('event_id_seq') PRIMARY KEY,
                     event_type VARCHAR,
@@ -88,7 +88,7 @@ def _initialize_storage_impl(self) -> bool:
 
             # Create a backward compatibility view
             self.connection.execute("""
-                CREATE VIEW IF NOT EXISTS access_events AS 
+                CREATE VIEW IF NOT EXISTS access_events AS
                 SELECT * FROM monitor_events
             """)
 

src/mcpm/utils/display.py

@@ -92,5 +92,5 @@ def print_client_error(client_name):
     Args:
         client_name: Name of the client that caused the error
     """
-    console.print(f"[bold red]Error:[/] Unsupported active client")
-    console.print(f"Please switch to a supported client using 'mcpm client <client-name>'")
+    console.print("[bold red]Error:[/] Unsupported active client")
+    console.print("Please switch to a supported client using 'mcpm client <client-name>'")

src/mcpm/utils/repository.py

@@ -6,8 +6,8 @@
 import logging
 import os
 from datetime import datetime
-from pathlib import Path
-from typing import Dict, List, Optional, Any
+from typing import Any, Dict, List, Optional
+
 import requests
 
 # Import ConfigManager to get the config directory

tests/test_access_monitor.py

@@ -61,8 +61,8 @@ async def test_track_event(temp_db_path):
 
     # Query the database directly to check if the event was recorded
     result = monitor.connection.execute("""
-        SELECT * FROM monitor_events 
-        WHERE server_id = 'test-server' 
+        SELECT * FROM monitor_events
+        WHERE server_id = 'test-server'
         AND resource_id = 'test-tool'
     """).fetchall()
 

tests/test_client.py

@@ -108,8 +108,8 @@ def test_client_set_command_unsupported(monkeypatch):
     assert "Error" in result.output
     assert "Unknown client: unsupported-client" in result.output
     # Verify supported clients are listed
-    for client in supported_clients:
-        assert client in result.output
+    for supported_client in supported_clients:
+        assert supported_client in result.output
 
 
 def test_client_set_command_failure(monkeypatch):