diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 4243e03..51cae39 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -16,14 +16,8 @@ jobs: uses: hoverkraft-tech/compose-action@8be2d741e891ac9b8ac20825e6f3904149599925 # v2.2.0 with: compose-file: "./docker-compose.yml" - - - name: Install dependencies - run: | - sudo apt update -y && sudo apt install -y pipx - pipx install uv - uv sync --locked - env: - DEBIAN_FRONTEND: "noninteractive" + - name: Install the latest version of uv + uses: astral-sh/setup-uv@v6 - name: Wait for service to be ready run: | @@ -48,4 +42,4 @@ jobs: NEXTCLOUD_USERNAME: "admin" NEXTCLOUD_PASSWORD: "admin" run: | - uv run python -m pytest + uv run --frozen python -m pytest diff --git a/Dockerfile b/Dockerfile index b5e3fc8..161b277 100644 --- a/Dockerfile +++ b/Dockerfile @@ -6,8 +6,4 @@ COPY . . RUN uv sync --locked -ENV VIRTUAL_ENV=/app/.venv -ENV PATH="$VIRTUAL_ENV/bin:$PATH" -ENV FASTMCP_LOG_LEVEL=DEBUG - -CMD ["mcp", "run", "--transport", "sse", "nextcloud_mcp_server/server.py:mcp"] +CMD ["uv", "run", "--locked", "mcp", "run", "--transport", "sse", "nextcloud_mcp_server/server.py:mcp"] diff --git a/nextcloud_mcp_server/client.py b/nextcloud_mcp_server/client.py index d891a31..916ac2f 100644 --- a/nextcloud_mcp_server/client.py +++ b/nextcloud_mcp_server/client.py @@ -1,12 +1,9 @@ import os -import time # Import time for sleep import mimetypes -from io import BytesIO from httpx import ( Client, Auth, BasicAuth, - Headers, Request, Response, HTTPStatusError, diff --git a/nextcloud_mcp_server/config.py b/nextcloud_mcp_server/config.py index 44e5ce1..5f5e978 100644 --- a/nextcloud_mcp_server/config.py +++ b/nextcloud_mcp_server/config.py @@ -4,11 +4,8 @@ LOGGING_CONFIG = { "version": 1, "handlers": { "default": { - "class": "logging.FileHandler", + "class": "logging.StreamHandler", "formatter": "http", - # "stream": "ext://sys.stderr" - "filename": "/tmp/nextcloud-mcp-server.log", - "mode": "a", } }, "formatters": { diff --git a/nextcloud_mcp_server/server.py b/nextcloud_mcp_server/server.py index 1f0457c..abbc4ae 100644 --- a/nextcloud_mcp_server/server.py +++ b/nextcloud_mcp_server/server.py @@ -11,8 +11,6 @@ import asyncio # Import asyncio setup_logging() -logger = logging.getLogger(__name__) - @dataclass class AppContext: @@ -23,11 +21,9 @@ class AppContext: async def app_lifespan(server: FastMCP) -> AsyncIterator[AppContext]: """Manage application lifecycle with type-safe context""" # Initialize on startup - logger.info("Creating Nextcloud client") + logging.info("Creating Nextcloud client") client = NextcloudClient.from_env() - # Add a small delay to allow client initialization to complete - logger.info("Waiting 2 seconds for client initialization...") - logger.info("Client initialization wait complete.") + logging.info("Client initialization wait complete.") try: yield AppContext(client=client) finally: @@ -38,6 +34,8 @@ async def app_lifespan(server: FastMCP) -> AsyncIterator[AppContext]: # Create an MCP server mcp = FastMCP("Nextcloud MCP", lifespan=app_lifespan) +logger = logging.getLogger(__name__) + @mcp.resource("nc://capabilities") def nc_get_capabilities():