From 3b12e585cae1b6c14915466be4ca3c1c648d5311 Mon Sep 17 00:00:00 2001
From: Chris Coutinho <chris@coutinho.io>
Date: Wed, 29 Oct 2025 11:56:56 +0100
Subject: [PATCH] fix(helm): Update helm chart with extraArgs

---
 charts/nextcloud-mcp-server/README.md         | 21 ++++++++++++++++++-
 .../templates/deployment.yaml                 |  5 +++--
 charts/nextcloud-mcp-server/values.yaml       |  2 --
 3 files changed, 23 insertions(+), 5 deletions(-)

diff --git a/charts/nextcloud-mcp-server/README.md b/charts/nextcloud-mcp-server/README.md
index addc52d..7843d86 100644
--- a/charts/nextcloud-mcp-server/README.md
+++ b/charts/nextcloud-mcp-server/README.md
@@ -114,6 +114,26 @@ ingress:
 | `auth.oauth.persistence.enabled` | Enable persistent storage for OAuth | `true` |
 | `auth.oauth.persistence.size` | Size of OAuth storage PVC | `100Mi` |
 
+#### MCP Server Configuration
+
+| Parameter | Description | Default |
+|-----------|-------------|---------|
+| `mcp.transport` | Transport mode | `streamable-http` |
+| `mcp.port` | Server port (used by both auth modes) | `8000` |
+| `mcp.extraArgs` | Additional command-line arguments | `[]` |
+
+The `extraArgs` parameter allows you to pass additional command-line arguments to the MCP server. This is useful for enabling debug logging, enabling specific apps, or other runtime configuration.
+
+**Example:**
+```yaml
+mcp:
+  extraArgs:
+    - "--log-level"
+    - "debug"
+    - "--enable-app"
+    - "notes"
+```
+
 #### Image Configuration
 
 | Parameter | Description | Default |
@@ -137,7 +157,6 @@ ingress:
 |-----------|-------------|---------|
 | `service.type` | Service type | `ClusterIP` |
 | `service.port` | Service port | `8000` |
-| `service.oauthPort` | OAuth service port | `8001` |
 
 #### Ingress
 
diff --git a/charts/nextcloud-mcp-server/templates/deployment.yaml b/charts/nextcloud-mcp-server/templates/deployment.yaml
index b43d550..eac0ac4 100644
--- a/charts/nextcloud-mcp-server/templates/deployment.yaml
+++ b/charts/nextcloud-mcp-server/templates/deployment.yaml
@@ -46,11 +46,12 @@ spec:
             - "{{ .Values.mcp.transport }}"
             {{- if eq .Values.auth.mode "oauth" }}
             - "--oauth"
-            - "--port"
-            - "{{ .Values.auth.oauth.port }}"
             - "--oauth-token-type"
             - "{{ .Values.auth.oauth.tokenType }}"
             {{- end }}
+            {{- with .Values.mcp.extraArgs }}
+            {{- toYaml . | nindent 12 }}
+            {{- end }}
           ports:
             - name: http
               containerPort: {{ include "nextcloud-mcp-server.port" . }}
diff --git a/charts/nextcloud-mcp-server/values.yaml b/charts/nextcloud-mcp-server/values.yaml
index db30f10..9d27763 100644
--- a/charts/nextcloud-mcp-server/values.yaml
+++ b/charts/nextcloud-mcp-server/values.yaml
@@ -172,8 +172,6 @@ securityContext:
 service:
   type: ClusterIP
   port: 8000
-  # For OAuth mode, you may want to expose both ports
-  oauthPort: 8001
   annotations: {}
 
 ingress: