Need to fix mcp server endpoitn setup

since it's behind a tls terminating proxy, need to let it know how to handle nothing

Author: michaeljguarino

apps/core/lib/core/mcp/router.ex

@@ -8,6 +8,12 @@ defmodule Core.MCP.Router do
    pass: ["text/*"],
    json_decoder: Jason
 
+  plug Plug.SSL, rewrite_on: [
+    :x_forwarded_proto,
+    :x_forwarded_host,
+    :x_forwarded_port
+  ]
+
   plug :match
   plug :ensure_session_id
   plug :authorize

apps/core/lib/core/mcp/server.ex

@@ -18,30 +18,23 @@ defmodule Core.MCP.Server do
   end
 
   @impl true
-  def handle_initialize(request_id, params) do
-    case validate_protocol_version(params["protocolVersion"]) do
-      :ok ->
-        {:ok,
-         %{
-           jsonrpc: "2.0",
-           id: request_id,
-           result: %{
-             protocolVersion: @protocol_version,
-             capabilities: %{
-               tools: %{
-                 listChanged: true
-               }
-             },
-             serverInfo: %{
-               name: "Plural MCP Server",
-               version: "0.1.0"
-             }
-           }
-         }}
-
-      {:error, reason} ->
-        {:error, reason}
-    end
+  def handle_initialize(request_id, _params) do
+    {:ok, %{
+      jsonrpc: "2.0",
+      id: request_id,
+      result: %{
+        protocolVersion: @protocol_version,
+        capabilities: %{
+          tools: %{
+            listChanged: true
+          }
+        },
+        serverInfo: %{
+          name: "Plural MCP Server",
+          version: "0.1.0"
+        }
+      }
+    }}
   end
 
   @impl true

plural/helm/plural/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: plural
 description: A helm chart for installing plural
 appVersion: 0.11.8
-version: 0.10.102
+version: 0.10.103
 dependencies:
   - name: hydra
     version: 0.26.5

plural/helm/plural/templates/deployment.yaml

@@ -271,3 +271,80 @@ spec:
       topologySpreadConstraints:
         {{- toYaml . | nindent 8 }}
       {{- end }}
+---
+apiVersion: apps/v1
+kind: Deployment
+metadata:
+  name: plural-mcp
+  labels:
+{{ include "plural.labels" . | indent 4 }}
+spec:
+  replicas: 1
+  selector:
+    matchLabels:
+      app.kubernetes.io/name: plural-mcp
+      app.kubernetes.io/instance: {{ .Release.Name }}
+  template:
+    metadata:
+      labels:
+        app.kubernetes.io/name: plural-mcp
+        app.kubernetes.io/instance: {{ .Release.Name }}
+        plural.sh/monitoring: scrape
+      annotations:
+        prometheus.io/scrape: "true"
+        prometheus.io/path: "/metrics"
+        prometheus.io/port: {{ .Values.api.port | quote }}
+        checksum/config: {{ include (print $.Template.BasePath "/secrets.yaml") . | sha256sum }}
+    spec:
+      serviceAccountName: plural
+    {{- with .Values.imagePullSecrets }}
+      imagePullSecrets:
+        {{- toYaml . | nindent 6 }}
+    {{- end }}
+      initContainers:
+      {{ include "plural.wait-for-migration" . | nindent 6 }}
+      containers:
+      - name: api
+        image: "{{ .Values.global.registry }}/{{ .Values.image.repository }}:{{ include "plural.imageTag" . }}"
+        imagePullPolicy: {{ .Values.image.pullPolicy }}
+        envFrom:
+        - secretRef:
+            name: plural-env
+        env:
+        {{ include "plural.env" . | nindent 8 }}
+        ports:
+        - name: http
+          containerPort: {{ .Values.api.port }}
+          protocol: TCP
+        - name: epmd
+          containerPort: 4369
+          protocol: TCP
+        - name: mcp
+          containerPort: {{ .Values.mcp.port }}
+          protocol: TCP
+        livenessProbe:
+          httpGet:
+            path: /health
+            port: http
+        readinessProbe:
+          httpGet:
+            path: /__traffic
+            port: http
+        resources:
+          {{- toYaml .Values.api.resources | nindent 12 }}
+      {{- with .Values.nodeSelector }}
+      nodeSelector:
+        {{- toYaml . | nindent 8 }}
+      {{- end }}
+    {{- with .Values.affinity }}
+      affinity:
+        {{- toYaml . | nindent 8 }}
+    {{- end }}
+    {{- with .Values.tolerations }}
+      tolerations:
+        {{- toYaml . | nindent 8 }}
+    {{- end }}
+      {{- with .Values.api.topologySpreadConstraints }}
+      topologySpreadConstraints:
+        {{- toYaml . | nindent 8 }}
+      {{- end }}

plural/helm/plural/templates/service.yaml

@@ -38,7 +38,7 @@ spec:
     protocol: TCP
     name: mcp
   selector:
-    app.kubernetes.io/name: plural-api
+    app.kubernetes.io/name: plural-mcp
     app.kubernetes.io/instance: {{ .Release.Name }}
 ---
 apiVersion: v1