From 529daf2b4897171dc18d1691778f421bb3d316b3 Mon Sep 17 00:00:00 2001
From: Chris Coutinho <chris@coutinho.io>
Date: Sun, 16 Nov 2025 07:03:18 +0100
Subject: [PATCH] ci: temp disable sse in ci

---
 docker-compose.yml                                   |  8 +++++---
 nextcloud_mcp_server/client/notes.py                 |  2 +-
 nextcloud_mcp_server/observability/logging_config.py |  7 ++++++-
 tests/conftest.py                                    | 11 +++++++++--
 4 files changed, 21 insertions(+), 7 deletions(-)

diff --git a/docker-compose.yml b/docker-compose.yml
index f877271..4669033 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -70,11 +70,13 @@ services:
   mcp:
     build: .
     restart: always
+    command: ["--transport", "streamable-http"]
     depends_on:
       app:
         condition: service_healthy
     ports:
       - 127.0.0.1:8000:8000
+      - 127.0.0.1:9090:9090
     volumes:
       - mcp-data:/app/data
     environment:
@@ -85,7 +87,7 @@ services:
 
       # Vector sync configuration (ADR-007)
       - VECTOR_SYNC_ENABLED=true
-      - VECTOR_SYNC_SCAN_INTERVAL=300
+      - VECTOR_SYNC_SCAN_INTERVAL=60
       - VECTOR_SYNC_PROCESSOR_WORKERS=1
 
       #- LOG_FORMAT=json
@@ -193,8 +195,8 @@ services:
       # Provider auto-detected from OIDC_DISCOVERY_URL issuer
       # Using internal Docker hostname for discovery to get consistent issuer
       - OIDC_DISCOVERY_URL=http://keycloak:8080/realms/nextcloud-mcp/.well-known/openid-configuration
-      - OIDC_CLIENT_ID=nextcloud-mcp-server
-      - OIDC_CLIENT_SECRET=mcp-secret-change-in-production
+      - NEXTCLOUD_OIDC_CLIENT_ID=nextcloud-mcp-server
+      - NEXTCLOUD_OIDC_CLIENT_SECRET=mcp-secret-change-in-production
       - OIDC_JWKS_URI=http://keycloak:8080/realms/nextcloud-mcp/protocol/openid-connect/certs
 
       # Nextcloud API endpoint (for accessing APIs with validated token)
diff --git a/nextcloud_mcp_server/client/notes.py b/nextcloud_mcp_server/client/notes.py
index 601e94f..e845a2d 100644
--- a/nextcloud_mcp_server/client/notes.py
+++ b/nextcloud_mcp_server/client/notes.py
@@ -40,7 +40,7 @@ class NotesClient(BaseNextcloudClient):
         seen_ids: set[int] = set()
 
         while True:
-            params: Dict[str, Any] = {"chunkSize": 10}
+            params: Dict[str, Any] = {"chunkSize": 100}
             if cursor:
                 params["chunkCursor"] = cursor
             if prune_before is not None:
diff --git a/nextcloud_mcp_server/observability/logging_config.py b/nextcloud_mcp_server/observability/logging_config.py
index d64ddd4..15e43fb 100644
--- a/nextcloud_mcp_server/observability/logging_config.py
+++ b/nextcloud_mcp_server/observability/logging_config.py
@@ -39,7 +39,12 @@ class HealthCheckFilter(logging.Filter):
         message = record.getMessage()
         return not any(
             endpoint in message
-            for endpoint in ["/health/live", "/health/ready", "/metrics"]
+            for endpoint in [
+                "/health/live",
+                "/health/ready",
+                "/metrics",
+                "/app/vector-sync/status",
+            ]
         )
 
 
diff --git a/tests/conftest.py b/tests/conftest.py
index 98bb2e4..0eaf115 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -255,8 +255,15 @@ async def nc_mcp_client(anyio_backend) -> AsyncGenerator[ClientSession, Any]:
 
     Note: SSE transport is being deprecated. This fixture uses SSE for compatibility testing.
     """
-    async for session in create_mcp_client_session_sse(
-        url="http://localhost:8000/sse", client_name="Basic MCP (SSE)"
+
+    # async for session in create_mcp_client_session_sse(
+    # url="http://localhost:8000/sse", client_name="Basic MCP (SSE)"
+    # ):
+    # yield session
+
+    async for session in create_mcp_client_session(
+        url="http://localhost:8000/mcp",
+        client_name="Basic MCP (HTTP)",
     ):
         yield session