@prontosite/mcp 0.10.0 → 0.10.3

package/CHANGELOG.md

@@ -3,6 +3,38 @@
 Alle nennenswerten Änderungen am MCP-Server für ProntoSite. Der Server ist ein dünner Client um die
 ProntoSite Deploy-API; die eigentliche Logik lebt serverseitig.
 
+## 0.10.3 – 2026-10-14
+
+- **`site_deploy`:** neuer Parameter `name` (Erst-Import) – setzt den Website-Namen explizit, falls die
+  Ableitung aus der Startseite danebenliegt (z. B. „Physio Phoenix – Bad Honnef Süd" → `name="Physio Phoenix"`).
+  Beschreibung ergänzt um die **Reihenfolge-Empfehlung** „Konto → Firmendaten → Import".
+- **`site_company_set`:** Beschreibung klärt, dass ein Firmendaten-Set NACH einem Import die Rechts-Automatik
+  AUTOMATISCH neu laufen lässt (Entwurf; `legal.refreshed` in der Antwort) – ein zweiter Import ist nicht mehr
+  nötig. `apiDeploy` meldet das im `companyNote` entsprechend.
+
+## 0.10.2 – 2026-10-14
+
+- **Verständliche Fehlermeldungen** statt roher Server-/HTTP-Fehler – zentral in `api.mjs`
+  (`friendlyApiError`, greift über `asJson` für JEDES Tool: `site_deploy`, `site_status`, `sites_list`, …):
+  - **403** mit „API-Zugang nicht im Tarif" (bzw. alt „Starter-Plan") → *„Dein ProntoSite-Tarif enthält keinen
+    API-Zugang. Wähle im Dashboard → Zahlung einen Tarif oder bitte deinen Betreuer um eine API-Freigabe."*
+  - **401** (Token ungültig/abgelaufen) → Hinweis, das Token im Dashboard → API-Zugang neu zu erstellen und in
+    `~/.prontosite.json` einzutragen.
+  - **429** → Hinweis auf das Rate-Limit (kurz warten und erneut versuchen).
+  Andere 403 (z. B. „Kein Zugriff auf dieses Konto") bleiben unverändert. Tests mit gemockten Antworten
+  (401/403/429 über mehrere Tools) in `tests/mcp-prontosite.test.js`.
+
+## 0.10.1 – 2026-10-03
+
+- **Fix:** In der `account_create`-Beschreibung beendeten gerade Anführungszeichen (`"`) innerhalb der
+  deutschen Quotes `„…"` den JS-String vorzeitig → `SyntaxError` beim Start (0.10.0 war nicht ladbar).
+  Alle Tool-Beschreibungen auf typografische `„…“` umgestellt.
+- **Schutz gegen Wiederholung:** `prepublishOnly` (= `npm run check` + `npm run smoke`) prüft vor jedem
+  Publish `node --check index.mjs api.mjs` **und** einen echten Import-Smoke (`smoke.mjs` lädt `index.mjs`
+  komplett – alle `server.tool`-Registrierungen laufen – ohne den stdio-Server zu starten, Dummy-Env).
+  Derselbe Smoke läuft in `npm test` des Hauptprojekts. `index.mjs` startet den Server jetzt nur noch bei
+  Direktausführung (Main-Guard), damit der Import nicht blockiert.
+
 ## 0.10.0 – 2026-10-03
 
 - **Neu: `account_create`** (schreibend, nur mit Betreuer-Token: admin/partner/agentur) – legt ein neues

package/README.md

@@ -122,6 +122,20 @@ genau diese Website, und Verwechslungen sind ausgeschlossen. `sites_list` zeigt
 Chat einfach sagen „deploye mas-monti", ohne Slugs auswendig zu kennen. (Alternativ ein Chat für alles und die
 Website je Aufruf über `site` wählen – die getrennten Chats sind aber übersichtlicher.)
 
+## Fehlermeldungen
+
+Statt roher HTTP-/Server-Fehler geben **alle** Tools bei den drei häufigen Betriebsfällen eine verständliche
+Meldung zurück (zentral in `api.mjs`):
+
+- **Kein API-Zugang im Tarif (403):** „Dein ProntoSite-Tarif enthält keinen API-Zugang. Wähle im
+  Dashboard → Zahlung einen Tarif oder bitte deinen Betreuer um eine API-Freigabe."
+- **Token ungültig/abgelaufen (401):** „Dein API-Token ist ungültig oder abgelaufen. Token im
+  Dashboard → API-Zugang neu erstellen und in `~/.prontosite.json` eintragen."
+- **Zu viele Anfragen (429):** Hinweis auf das Rate-Limit – einen Moment warten und erneut versuchen.
+
+Andere Fehler (z. B. „Kein Zugriff auf dieses Konto", „Website nicht gefunden") werden unverändert mit der
+Server-Meldung durchgereicht.
+
 ## Sicherheit
 
 - Tokens liegen nur lokal (Config/Env), werden nie geloggt und nie in Antworten zurückgegeben.

package/api.mjs

@@ -94,11 +94,31 @@ export function resolveTarget(cfg, site) {
 const baseUrl = cfg => `https://${cfg.host}/api`;
 const authHeaders = token => ({ authorization: `Bearer ${token}` });
 
+/**
+ * Server-Fehler → VERSTÄNDLICHE Meldung für den Chat. Zentral für ALLE Tools (über asJson). Übersetzt die
+ * drei häufigen Betriebsfälle; alles andere behält die Server-Meldung (bzw. „HTTP <status>"). Token werden
+ * nie geloggt/zurückgegeben – hier sind ohnehin nur Status + Server-Text im Spiel.
+ *   401  Token ungültig/abgelaufen → neues Token erstellen + in ~/.prontosite.json eintragen.
+ *   403  API-Gate (Text „API-Zugang nicht im Tarif" bzw. „Starter-Plan") → Tarif ohne API-Zugang.
+ *   429  Rate-Limit → kurz warten.
+ * Andere 403 (z. B. „Kein Zugriff auf dieses Konto") bleiben unverändert – nur das API-Zugangs-Gate wird gemappt.
+ */
+export function friendlyApiError(status, serverError = "") {
+  const raw = String(serverError || "").trim();
+  if (status === 401)
+    return "Dein API-Token ist ungültig oder abgelaufen. Token im Dashboard → API-Zugang neu erstellen und in ~/.prontosite.json eintragen.";
+  if (status === 403 && /API-Zugang nicht im Tarif|Starter-Plan/i.test(raw))
+    return "Dein ProntoSite-Tarif enthält keinen API-Zugang. Wähle im Dashboard → Zahlung einen Tarif oder bitte deinen Betreuer um eine API-Freigabe.";
+  if (status === 429)
+    return "Zu viele Anfragen in kurzer Zeit (Rate-Limit). Bitte einen Moment warten und erneut versuchen.";
+  return raw || `HTTP ${status}`;
+}
+
 /** Antwort als JSON lesen; bei Fehlerstatus eine sprechende Exception (ohne Token). */
 async function asJson(res) {
   let body = {};
   try { body = await res.json(); } catch { /* kein JSON */ }
-  if (!res.ok) throw new Error(body.error || `HTTP ${res.status}`);
+  if (!res.ok) throw new Error(friendlyApiError(res.status, body.error));
   return body;
 }
 
@@ -125,7 +145,7 @@ export async function apiExport(cfg, { site, which = "published", outDir = "." }
 }
 
 /** POST /api/deploy/<site> – ZIP übergeben (+ optional publish/resetOverlay/legal). site="new" = anlegen. */
-export async function apiDeploy(cfg, { site, customer = null, slug: wishSlug = null, zipPath, publish = false, resetOverlay = false, legal = null, company = null, siteOverride = false } = {}, fetchImpl = fetch) {
+export async function apiDeploy(cfg, { site, customer = null, slug: wishSlug = null, name = null, zipPath, publish = false, resetOverlay = false, legal = null, company = null, siteOverride = false } = {}, fetchImpl = fetch) {
   if (!zipPath || !fs.existsSync(zipPath)) throw new Error(`ZIP nicht gefunden: ${zipPath}`);
   const target = resolveTarget(cfg, site);
   const { token, slug } = target;
@@ -142,6 +162,7 @@ export async function apiDeploy(cfg, { site, customer = null, slug: wishSlug = n
   if (legal === false) q.set("legal", "0");
   if (effCustomer) q.set("customer", effCustomer);   // Betreuer-Token + „new": Ziel-Konto für den Erst-Import
   if (wishSlug) q.set("slug", String(wishSlug));   // Erst-Import: Wunsch-Adresse (sonst wird sie abgeleitet)
+  if (isNew && name) q.set("name", String(name));   // Erst-Import: Site-Name explizit setzen (sonst aus der Startseite abgeleitet)
   const qs = q.toString() ? `?${q}` : "";
   const res = await fetchImpl(`${baseUrl(cfg)}/deploy/${encodeURIComponent(slug)}${qs}`, {
     method: "POST", headers: { ...authHeaders(token), "content-type": "application/zip" }, body: fs.readFileSync(zipPath) });
@@ -151,9 +172,11 @@ export async function apiDeploy(cfg, { site, customer = null, slug: wishSlug = n
   // Publish übernimmt sie. Das melden wir als companyApplied/companyNote statt still zu lassen.
   if (hasCompany && isNew && out && (out.slug || out.site)) {
     try {
-      await apiCompanySet(cfg, { site: out.slug || out.site, company, siteOverride }, fetchImpl);
+      const compRes = await apiCompanySet(cfg, { site: out.slug || out.site, company, siteOverride }, fetchImpl);
       out.companyApplied = true;
-      if (out.legal && !out.legal.disabled) out.companyNote = "Firmendaten wurden NACH dem Erst-Import gesetzt – die automatisch erzeugten Rechtstexte übernehmen sie erst beim nächsten Rechts-Lauf/Publish.";
+      // Firmendaten-Set lässt serverseitig die Rechts-Automatik neu laufen (Entwurf) – kein zweiter Import nötig.
+      if (compRes && compRes.legal && compRes.legal.refreshed && compRes.legal.refreshed.length)
+        out.companyNote = `Firmendaten gesetzt – Rechtstexte (${compRes.legal.refreshed.join(", ")}) automatisch neu erzeugt (Entwurf). Zum Live-Schalten veröffentlichen.`;
     } catch (e) { out.companyApplied = false; out.companyError = e.message; }
   }
   return out;

package/index.mjs

@@ -8,6 +8,8 @@
  * Multi-Site (Betreuer): trägt die Config eine `sites`-Map, adressiert jedes Tool per `site`-Parameter die
  * jeweilige Website (eigenes Token je Site). Arbeitsweise: EIN Chat je Kunde. `sites_list` zeigt alle Sites.
  */
+import { fileURLToPath } from "node:url";
+import { realpathSync } from "node:fs";
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { z } from "zod";
@@ -17,7 +19,7 @@ const cfg = loadConfig();
 // Deutliche Startwarnung bei ambiger Konfiguration (Top-Level-Token NEBEN sites-Map). Nach stderr, damit
 // es im MCP-Log/Terminal auffällt; die endgültige Auflösung (Betreuer-Token-Vorrang vs. Map) klärt sites_list.
 for (const w of (cfg.warnings || [])) process.stderr.write("[prontosite-mcp] ⚠ " + w + "\n");
-const server = new McpServer({ name: "prontosite", version: "0.10.0" });
+const server = new McpServer({ name: "prontosite", version: "0.10.3" });
 const asText = obj => ({ content: [{ type: "text", text: JSON.stringify(obj, null, 2) }] });
 const asError = e => ({ isError: true, content: [{ type: "text", text: "Fehler: " + (e?.message || String(e)) }] });
 const SITE_DESC = "Website-Slug (z. B. 'mas-monti'). Ohne Angabe die Standard-Site der Config. Bei Betreuer-"
@@ -57,9 +59,9 @@ server.tool(
 server.tool(
   "account_create",
   "SCHREIBEND (nur mit Betreuer-Token: admin/partner/agentur): Legt ein NEUES Kundenkonto an – der Dashboard-"
-    + "Weg „Kunde anlegen". Nutze dieses Tool, BEVOR du eine Website importierst, wenn es für den Kunden noch "
+    + "Weg „Kunde anlegen“. Nutze dieses Tool, BEVOR du eine Website importierst, wenn es für den Kunden noch "
     + "KEIN Konto gibt (z. B. sites_list zeigt den Kunden nicht, oder der Nutzer sagt „für Kunde X gibt es noch "
-    + "kein Konto"). Erst Konto anlegen, dann mit der zurückgegebenen Konto-ID (k-…) per site_deploy und "
+    + "kein Konto“). Erst Konto anlegen, dann mit der zurückgegebenen Konto-ID (k-…) per site_deploy und "
     + "site=\"<k-id>:new\" die erste Website importieren. Richte KEIN site=new/Erst-Import ein, solange die "
     + "Konto-ID unbekannt ist – dann zuerst dieses Tool. Mit email wird ein Login-Konto angelegt und eine "
     + "Einladungs-Mail verschickt; ohne email ein verwaltetes Konto (ohne Login, Betrieb über den Betreuer). "
@@ -104,6 +106,11 @@ server.tool(
     + "(„Unter welcher Adresse soll die Website laufen?“, z. B. 'wuerttemberg' → wuerttemberg.prontosite.io) "
     + "und übergib sie als slug – ohne slug wird die Adresse nur aus Konto-/Seitenname ABGELEITET (die Antwort "
     + "nennt dann die abgeleitete Adresse; sie ist später änderbar). Bitte VOR dem Aufruf bestätigen lassen. "
+    + "REIHENFOLGE beim Erst-Import: zuerst Konto (account_create) → Firmendaten (company hier mitgeben ODER "
+    + "site_company_set) → Import; so stehen Impressum/Datenschutz sofort korrekt (ein Firmendaten-Set nach dem "
+    + "Import lässt die Rechts-Automatik automatisch neu laufen – kein zweiter Import nötig). "
+    + "WEBSITE-NAME: wird aus der Startseite abgeleitet (og:site_name → Titel-Präfix vor dem ersten Trenner → "
+    + "Header-H1/Logo-alt); mit name=… explizit setzen (Betreuer), falls die Ableitung danebenliegt. "
     + "MARKER (wenn du das Template lokal baust): markiere im HTML mit dem Attribut data-ps, wo Bausteine "
     + "erscheinen sollen – data-ps=\"formular:kontakt\" (verwaltetes Kontaktformular), data-ps=\"impressum\" bzw. "
     + "data-ps=\"datenschutz\" (Rechtstext-Seite) und data-ps=\"link-impressum\" / data-ps=\"link-datenschutz\" "
@@ -118,6 +125,7 @@ server.tool(
     + "Syntax-Referenz: https://prontosite.io/docs/marker-shortcodes",
   { site: z.string().optional().describe(SITE_DESC + " Erst-Import: 'new' (Einzel) bzw. '<label>:new' (Multi-Site). Betreuer-Token: '<Konto-ID>:new' (k-…) legt im betreuten Konto an – die Konto-ID im Label ersetzt den customer-Parameter. sites_list zeigt betreute Konten OHNE Website (needsFirstImport) samt ihrer k-ID."),
     slug: z.string().optional().describe("Erst-Import: gewünschte Web-Adresse (Subdomain-Label, z. B. 'wuerttemberg' → wuerttemberg.prontosite.io). Beim Erst-Import den Nutzer danach FRAGEN; wird sie weggelassen, leitet ProntoSite eine Adresse ab. Bei Updates ignoriert."),
+    name: z.string().optional().describe("Erst-Import: expliziter Website-Name (überschreibt die Ableitung aus der Startseite). Nützlich, wenn der Titel einen Ortszusatz trägt (z. B. 'Physio Phoenix – Bad Honnef Süd' → name='Physio Phoenix'). Bei Updates ignoriert."),
     customer: z.string().optional().describe("Betreuer-Token + Erst-Import: Ziel-Konto-ID (k-…) für die neue Website. Alternativ direkt im site-Argument als '<Konto-ID>:new'."),
     zipPath: z.string().describe("Pfad zur ZIP-Datei der Website (lokal)."),
     publish: z.boolean().default(false).describe("true = nach dem Import direkt veröffentlichen (LIVE)."),
@@ -125,8 +133,8 @@ server.tool(
     legal: z.boolean().optional().describe("Rechts-Automatik (Impressum/Datenschutz): true erzwingt, false schaltet ab."),
     company: companyObject.optional().describe("Optional: Firmendaten VOR dem Deploy setzen (ein Schritt) – Impressum/Datenschutz nutzen sie dann direkt. Teil-Update."),
     siteOverride: z.boolean().default(false).describe("Nur mit company: true = Firmendaten NUR für diese Website (statt konto-weit).") },
-  async ({ site, slug, customer, zipPath, publish, resetOverlay, legal, company, siteOverride }) => {
-    try { return asText(await apiDeploy(cfg, { site, slug, customer, zipPath, publish, resetOverlay, legal, company, siteOverride })); } catch (e) { return asError(e); }
+  async ({ site, slug, name, customer, zipPath, publish, resetOverlay, legal, company, siteOverride }) => {
+    try { return asText(await apiDeploy(cfg, { site, slug, name, customer, zipPath, publish, resetOverlay, legal, company, siteOverride })); } catch (e) { return asError(e); }
   }
 );
 
@@ -154,8 +162,11 @@ server.tool(
     + "USt-IdNr., Register, Kammer/Beruf, Aufsicht, § 18 MStV, Social). Diese Daten füllen beim nächsten Deploy "
     + "automatisch Impressum & Datenschutz. TEIL-UPDATE: nur die übergebenen Felder ändern sich. Du darfst die "
     + "Angaben aus jeder Vorlage extrahieren (Impressum-Text, Visitenkarte, Freitext) und strukturiert übergeben. "
-    + "Default konto-weit; siteOverride=true nur, wenn DIESE eine Website abweichende Angaben braucht. Bitte VOR "
-    + "dem Speichern die erkannten Felder kurz zusammenfassen und bestätigen lassen.",
+    + "Default konto-weit; siteOverride=true nur, wenn DIESE eine Website abweichende Angaben braucht. NACH EINEM "
+    + "IMPORT lässt dieser Aufruf die Rechts-Automatik AUTOMATISCH neu laufen (Entwurf) – Impressum/Datenschutz "
+    + "übernehmen die Firmendaten sofort, ein zweiter Import ist NICHT nötig (die Antwort nennt die neu erzeugten "
+    + "Rechtstexte unter legal.refreshed); danach site_publish zum Live-Schalten. Bitte VOR dem Speichern die "
+    + "erkannten Felder kurz zusammenfassen und bestätigen lassen.",
   { site: z.string().optional().describe(SITE_DESC),
     company: companyObject,
     siteOverride: z.boolean().default(false).describe("true = Firmendaten NUR für diese Website (statt konto-weit).") },
@@ -271,5 +282,13 @@ server.tool(
   }
 );
 
-const transport = new StdioServerTransport();
-await server.connect(transport);
+// Den stdio-Server NUR starten, wenn diese Datei direkt ausgeführt wird (als MCP-Server / bin).
+// Beim bloßen Importieren (Smoke-Test / Tests) NICHT verbinden – sonst würde der Import auf stdin blockieren.
+// So lädt der Import die Datei komplett (inkl. aller server.tool-Registrierungen → fängt Syntax-/Beschreibungs-
+// fehler) ohne den Server zu starten.
+let isMain = false;
+try { isMain = !!process.argv[1] && realpathSync(process.argv[1]) === fileURLToPath(import.meta.url); } catch { /* argv[1] fehlt (node -e) → kein Main */ }
+if (isMain) {
+  const transport = new StdioServerTransport();
+  await server.connect(transport);
+}

package/package.json

@@ -1,12 +1,17 @@
 {
   "name": "@prontosite/mcp",
-  "version": "0.10.0",
+  "version": "0.10.3",
   "description": "MCP-Server für ProntoSite – Website-Status, Export, Deploy und Veröffentlichen direkt aus einem KI-Assistenten (Claude Desktop). Dünner Client um die ProntoSite Deploy-API.",
   "type": "module",
   "bin": {
     "prontosite-mcp": "index.mjs"
   },
   "main": "index.mjs",
+  "scripts": {
+    "check": "node --check index.mjs && node --check api.mjs",
+    "smoke": "node smoke.mjs",
+    "prepublishOnly": "npm run check && npm run smoke"
+  },
   "files": [
     "index.mjs",
     "api.mjs",