From 9d21138e623518970c549884d3e4f262e15f944c Mon Sep 17 00:00:00 2001
From: Meleki <meleki@gott.local>
Date: Fri, 3 Jul 2026 10:33:17 +0200
Subject: [PATCH] Fix MCP-Allowlist: exakte Server-Form mcp__<name>__* statt
 nie-matchendem Glob

Die Claude-CLI-Permission-Engine parst eine Allowlist-Regel via split("__")
zu {serverName, toolName} und vergleicht den Servernamen per strenger
Gleichheit (===); ein '*' wirkt ausschliesslich im Tool-Teil, nie im
Server-Teil. Die generalisierte Form "mcp__*<name>*" parste damit zu einem
Server namens "*<name>*" und matchte NIE -> jeder ueber die MCP-Liste
eingetragene Server wurde headless still abgelehnt (Regression aus 5f7dec6).

- ClaudeCliService.cs: Allowlist-Glob mcp__*<name>* -> mcp__<name>__*
  (exakter Server + Tool-Wildcard, dieselbe Form wie mcp__web-search-prime__*);
  Kommentar erklaert die strenge Server-Gleichheit.
- index.html: UI-Hinweis korrigiert - Name muss exakt dem Tool-Praefix
  entsprechen (z.B. UnityMCP, nicht Unity); Freigabe als mcp__<name>__*.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 Services/ClaudeCliService.cs | 11 ++++++++---
 wwwroot/index.html           |  2 +-
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/Services/ClaudeCliService.cs b/Services/ClaudeCliService.cs
index c1f8b44..922e5c9 100644
--- a/Services/ClaudeCliService.cs
+++ b/Services/ClaudeCliService.cs
@@ -410,14 +410,19 @@ public class ClaudeCliService
             tools.Add("mcp__gott-remote__memory_list");
         }
         // Benutzerdefinierte MCP-Server: alle Tools je Server erlauben.
-        // Glob mcp__*<name>* ist bewusst locker (fängt Casing-/Präfix-Varianten ab,
-        // z.B. Servername "unity" vs. tatsächlicher Tool-Präfix "UnityMCP").
+        // WICHTIG: Die CLI-Permission-Engine vergleicht den SERVER-Namen per strenger
+        // Gleichheit (Regel wird als mcp__<server>__<tool> geparst; ein '*' wirkt NUR im
+        // Tool-Teil, NICHT im Server-Teil). Ein Glob wie "mcp__*<name>*" würde zu einem
+        // Server namens "*<name>*" parsen und NIE matchen. Daher exakte Form
+        // "mcp__<name>__*" (exakter Server + alle Tools) – dieselbe Form wie web-search-prime.
+        // Konsequenz: Der eingetragene Name MUSS dem echten Tool-Präfix entsprechen
+        // (z.B. "UnityMCP", nicht "Unity").
         if (mcpServerNames != null)
         {
             foreach (var mcpName in mcpServerNames)
             {
                 if (!string.IsNullOrWhiteSpace(mcpName))
-                    tools.Add($"mcp__*{mcpName}*");
+                    tools.Add($"mcp__{mcpName}__*");
             }
         }
         args.Add("--allowed-tools");
diff --git a/wwwroot/index.html b/wwwroot/index.html
index de7e37e..c86aa30 100644
--- a/wwwroot/index.html
+++ b/wwwroot/index.html
@@ -175,7 +175,7 @@
             <label>MCP-Server <small>(optional, URL-basiert)</small></label>
             <div class="mcp-servers" id="mcpServersList"></div>
             <button type="button" id="addMcpServerBtn" class="add-cmd-btn">+ MCP-Server hinzufügen</button>
-            <small class="form-hint">Pro Eintrag: Name, URL (http/sse) und optionaler Token. Tools werden headless als <code>mcp__*&lt;name&gt;*</code> freigegeben.</small>
+            <small class="form-hint">Pro Eintrag: Name, URL (http/sse) und optionaler Token. Der Name muss <b>exakt</b> dem Tool-Präfix entsprechen (z.B. <code>UnityMCP</code>, nicht <code>Unity</code>). Tools werden headless als <code>mcp__&lt;name&gt;__*</code> freigegeben.</small>
             <label>Telegram Bot</label>
             <div id="telegramBotSelectCard" class="agent-select-card">
               <div class="agent-select-empty"><i class="fa fa-paper-plane"></i> Kein Bot zugewiesen</div>