Merge pull request #34 from cbcoutinho/feature/logging

fix: Configure logging

.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 the latest version of uv
-      - name: Install dependencies
+        uses: astral-sh/setup-uv@v6
-        run: |
-          sudo apt update -y && sudo apt install -y pipx
-          pipx install uv
-          uv sync --locked
-        env:
-          DEBIAN_FRONTEND: "noninteractive"
 
       - 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

Dockerfile

@@ -6,8 +6,4 @@ COPY . .
 
 RUN uv sync --locked
 
-ENV VIRTUAL_ENV=/app/.venv
+CMD ["uv", "run", "--locked", "mcp", "run", "--transport", "sse", "nextcloud_mcp_server/server.py:mcp"]
-ENV PATH="$VIRTUAL_ENV/bin:$PATH"
-ENV FASTMCP_LOG_LEVEL=DEBUG
-
-CMD ["mcp", "run", "--transport", "sse", "nextcloud_mcp_server/server.py:mcp"]

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,

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": {

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
+    logging.info("Client initialization wait complete.")
-    logger.info("Waiting 2 seconds for client initialization...")
-    logger.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():