@prontosite/mcp 0.14.7 → 0.16.0

package/CHANGELOG.md

@@ -3,6 +3,24 @@
 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.16.0 – 2026-11-03
+
+- Formular-Anfragen per MCP (Scope `anfragen` bzw. `anfragen:schreiben`, beim Verbinden ein eigener, nie vorausgewählter Punkt;
+  API-Tokens von Kunden und Betreuern haben das Recht): `site_inbox_list` (Filter Status/Formular/Zeitraum/Volltext, max. 50 je Seite),
+  `site_inbox_get` (alle Felder, Anhänge mit 15-Minuten-Download-Link, Verlauf), `site_inbox_set_status` (neu/gelesen/erledigt,
+  spam ↔ kein-spam, mehrere IDs; „kein-spam" sendet die Benachrichtigung nach), `site_inbox_delete` (Papierkorb, 30 Tage),
+  `site_inbox_restore`, `site_inbox_stats`. Keine Antwort-Mails über MCP.
+- `site_status`: `canInbox`/`canInboxWrite` und – mit Recht – `anfragen` (Anzahl je Status). Instructions 5f (Besucherdaten nie ungefragt
+  zusammenfassen oder weitergeben, Löschen nur nach Bestätigung).
+
+## 0.15.0 – 2026-11-03
+
+- JS-Dateien bearbeitbar ohne die JS-Freigabe auszuhebeln: `site_edit_text` mit `scope:"all"` ersetzt auch in Skriptdateien (nur in
+  Texten/Zahlen/Kommentaren, Fundstellen in `skripte`); `site_file_put` darf JS für Kunden ändern. Der Server klassifiziert gegen den
+  zuletzt freigegebenen Stand: nur Zahlen/Texte/Kommentare → sofort wirksam, sonst `jsReview.status: "pending"` (live bleibt der freigegebene Stand).
+- Neues Tool `site_js_review` (nur Betreuer-Token): ausstehende Prüfungen mit Diff ansehen, freigeben oder ablehnen.
+- `site_status.jsReview`; Instructions 2e (Preise in Skripten, Hinweis „wartet auf Freigabe" an den Nutzer).
+
 ## 0.14.7 – 2026-10-29
 
 - `site_image_set`: neue Option `zoom` (true/false) – Bild per Klick vergrößern (Lightbox der ProntoSite-Runtime, Bildunterschrift

package/api.mjs

@@ -599,3 +599,33 @@ export async function apiLock(cfg, { site, action = null, note = null } = {}, fe
   if (!action) return asJson(await fetchImpl(url, { headers: authHeaders(token, cfg) }));
   return asJsonOr409(await fetchImpl(url, { method: "POST", headers: { ...authHeaders(token, cfg), "content-type": "application/json" }, body: JSON.stringify({ action, note }) }));
 }
+
+/** GET/POST /api/js-review/<site> – JS-Prüfungen (nur Betreuer-Token): ohne action Liste + Diff, sonst approve|reject. */
+export async function apiJsReview(cfg, { site, path: rel = null, action = null, note = null } = {}, fetchImpl = fetch) {
+  const { token, slug } = resolveTarget(cfg, site);
+  const url = `${baseUrl(cfg)}/js-review/${encodeURIComponent(slug)}`;
+  if (!action) return asJson(await fetchImpl(url, { headers: authHeaders(token, cfg) }));
+  return asJson(await fetchImpl(url, { method: "POST", headers: { ...authHeaders(token, cfg), "content-type": "application/json" }, body: JSON.stringify({ path: rel, action, note }) }));
+}
+
+/** GET /api/inbox/<site> – Formular-Anfragen: Liste (Filter), eine Anfrage (id) oder Statistik (stats). */
+export async function apiInboxGet(cfg, { site, id = null, stats = false, status = null, form = null, seit = null, bis = null, q = null, limit = null, offset = null } = {}, fetchImpl = fetch) {
+  const { token, slug } = resolveTarget(cfg, site);
+  const p = new URLSearchParams();
+  if (id) p.set("id", String(id));
+  else if (stats) p.set("stats", "1");
+  else {
+    for (const s of [].concat(status ?? [])) if (s) p.append("status", String(s));
+    for (const [k, v] of Object.entries({ form, seit, bis, q, limit, offset })) if (v != null && v !== "") p.set(k, String(v));
+  }
+  const qs = p.toString();
+  return asJson(await fetchImpl(`${baseUrl(cfg)}/inbox/${encodeURIComponent(slug)}${qs ? `?${qs}` : ""}`, { headers: authHeaders(token, cfg) }));
+}
+/** POST /api/inbox/<site> – Status setzen (status), in den Papierkorb (delete) oder zurückholen (restore); mehrere IDs. */
+export async function apiInboxPost(cfg, { site, action, ids, status = null } = {}, fetchImpl = fetch) {
+  const { token, slug } = resolveTarget(cfg, site);
+  return asJson(await fetchImpl(`${baseUrl(cfg)}/inbox/${encodeURIComponent(slug)}`, {
+    method: "POST", headers: { ...authHeaders(token, cfg), "content-type": "application/json" },
+    body: JSON.stringify({ action, ids: [].concat(ids ?? []), ...(status ? { status } : {}) }) }));
+}
+

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prontosite/mcp",
-  "version": "0.14.7",
+  "version": "0.16.0",
   "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": {

package/tools.mjs

@@ -9,7 +9,7 @@
  *                site_domain_connect nur mit Betreuer-Token (Entscheidung 9a).
  * `z` wird vom Aufrufer übergeben (dieselbe zod-Instanz wie sein MCP-SDK).
  */
-import { apiEditText, apiEditHtml, apiPageTexts, apiPageTextsSet, apiPageCopy, apiMediaUploadBegin, apiMediaUploadFile, apiMediaCommit, apiMediaList, apiImageSet, apiUndo, apiMediaDelete, apiMediaRestore, apiFormsGet, apiFormsSet, apiMediaSearch, apiMediaImport, apiPreviewScreenshot, apiStatus, apiExport, apiExportFetch, apiDeploy, apiPublish, sitesList, apiAccountCreate, apiCompanyGet, apiCompanySet, apiLegalOptions, apiDomainStatus, apiDomainConnect, apiEmbeds, apiEmbedConsent, apiWidgets, apiWidgetSet, apiFiles, apiFileGet, apiFilePut, apiFileDelete, apiFileRename, apiLock } from "./api.mjs";
+import { apiEditText, apiEditHtml, apiPageTexts, apiPageTextsSet, apiPageCopy, apiMediaUploadBegin, apiMediaUploadFile, apiMediaCommit, apiMediaList, apiImageSet, apiUndo, apiMediaDelete, apiMediaRestore, apiFormsGet, apiFormsSet, apiMediaSearch, apiMediaImport, apiPreviewScreenshot, apiStatus, apiExport, apiExportFetch, apiDeploy, apiPublish, sitesList, apiAccountCreate, apiCompanyGet, apiCompanySet, apiLegalOptions, apiDomainStatus, apiDomainConnect, apiEmbeds, apiEmbedConsent, apiWidgets, apiWidgetSet, apiFiles, apiFileGet, apiFilePut, apiFileDelete, apiFileRename, apiLock, apiJsReview, apiInboxGet, apiInboxPost } from "./api.mjs";
 
 /**
  * Arbeitsweise (MCP-`instructions` beim Initialize – lokal UND /mcp): Weg B ist der Standard (Nick 19.10.).
@@ -22,13 +22,18 @@ const instructionLines = mode => [
     + "Danach dem Nutzer sagen, was geändert wurde, und den Vorschau-Link (previewUrl aus site_status) nennen.",
   "1a. Website wählen: Wenn der Nutzer eine Website beim Firmennamen nennt, erst sites_list, dann den passenden Slug verwenden; bei "
     + "Unklarheit nachfragen. (Die Tools lösen Firmenname/Domain/Titel auch selbst auf; mehrdeutig → Kandidatenliste, nichts geändert.)",
-  "1b. Rechte aus site_status.can* lesen (canRead, canEditDraft, canPublish, canDeploy, Gründe in canReasons); nicht aus anderen Feldern "
+  "1b. Rechte aus site_status.can* lesen (canRead, canEditDraft, canPublish, canDeploy, canInbox, canInboxWrite, Gründe in canReasons); nicht aus anderen Feldern "
     + "schließen – restApiEnabled betrifft nur die REST-API, nicht diese Tools.",
   "2. AUSNAHME: site_export + Deploy (site_deploy bzw. deploy_upload_begin/deploy_commit) NUR, wenn der Nutzer ausdrücklich lokal arbeiten "
     + "möchte oder die Website komplett umgebaut wird.",
   "2a. Für Text-, Link- und Kontaktdaten-Änderungen (Telefonnummer, E-Mail, Adresse, Öffnungszeiten) IMMER site_edit_text mit "
     + "scope:\"all\" (bei Bedarf region:\"footer\"/\"header\") – EIN Aufruf für alle Seiten, inkl. tel:/mailto:-Links. NIE Seite für Seite per "
     + "site_file_put. Meldet die Antwort company, zusätzlich site_company_set für die Firmendaten (Impressum/Datenschutz/Kontakt).",
+  "2e. Preise, Texte und Zahlen in Skriptdateien (.js, z. B. Preisrechner) ändert site_edit_text mit scope:\"all\" mit – ersetzt wird dort "
+    + "nur in Texten/Zahlen/Kommentaren, nie im Code. Nur-Zahlen/Texte-Änderungen wirken sofort; alles andere prüft der Betreuer. Meldet eine "
+    + "Antwort jsReview.status \"pending\" (bzw. site_status jsReview.pending), dem Nutzer sagen: Änderung gespeichert, sie wartet auf Freigabe "
+    + "durch den Betreuer, der Betreuer ist benachrichtigt – bis dahin läuft auf der Website der bisherige Stand. Nicht erneut versuchen und "
+    + "nicht umgehen (z. B. nicht ins HTML verlagern).",
   "2c. Für Markup über alle Seiten (z. B. Footer-HTML) site_edit_html mit scope:\"all\" (ggf. region:\"footer\"), nie Seite für Seite per site_file_put. "
     + "Entwurfs-Limit je Website: 300 Änderungen je Stunde, davon 30 direkt hintereinander (Kunden und Betreuer gleich) – dieselbe Änderung auf vielen Seiten ist EIN Aufruf (site_edit_text scope:\"all\" bzw. site_edit_html). Bei 429 (rateLimited) NICHT aufgeben: retryAfter Sekunden warten, dann weitermachen; bisherige Änderungen bleiben gespeichert.",
   "2d. Badge „Made with ProntoSite\" NUR als dieses Snippet einbauen (Footer, letzte Zeile, bündig mit dem Copyright), nie manuell stylen: "
@@ -63,6 +68,11 @@ const instructionLines = mode => [
   "5c. Bei „rückgängig“ site_undo (bzw. „doch wieder“ site_redo), nie Dateien manuell zurückbauen. Medien löschen nur mit site_media_delete "
     + "(Papierkorb, 30 Tage; zurückholen mit site_media_restore).",
   "5d. Formular-Empfänger, Spam-Stufe und Danke-Seite über site_forms_set, nie per HTML (site_forms_get zeigt den Stand).",
+  "5f. Formular-Anfragen (Recht canInbox in site_status): site_inbox_list (Kurzform) → site_inbox_get (vollständig) → site_inbox_set_status "
+    + "(neu/gelesen/erledigt, spam ↔ kein-spam). Anfragen enthalten Daten der Absender: NIE ungefragt zusammenfassen, auswerten oder an Dritte "
+    + "weitergeben – nur, was der Nutzer ausdrücklich verlangt. Löschen (site_inbox_delete = Papierkorb, 30 Tage, site_inbox_restore holt zurück) "
+    + "NUR nach Bestätigung des Nutzers. Antworten verschickt ProntoSite nicht: Absenderadresse nennen, der Nutzer antwortet im eigenen Mailprogramm. "
+    + "Fehlt canInbox, sagen: im Dashboard (Konto → KI verbinden) neu verbinden und „Formular-Anfragen“ erlauben.",
   "5. Jede Änderung lässt sich rückgängig machen; Rechtstexte (Impressum/Datenschutz) nicht umformulieren – sie kommen aus den Firmendaten.",
 ];
 /** Server-Instructions je Modus ("local" = stdio-Paket mit Dateizugriff, "remote" = /mcp). */
@@ -73,8 +83,8 @@ export const SERVER_INSTRUCTIONS = instructionsFor("remote");
 // destructive = live/kundenwirksam, ersetzt Bestand oder nimmt zurück (Bestätigung sinnvoll). Entwurfs-Änderungen sind
 // rückgängig machbar und daher nicht destruktiv. openWorld = wirkt nach außen (öffentliche Website/DNS).
 const READ_ONLY = new Set(["sites_list", "site_status", "site_forms_get", "media_search", "site_page_texts", "site_media_list", "site_preview_screenshot", "site_company_get",
-  "site_domain_status", "site_embeds", "site_widgets", "site_files", "site_file_get"]);
-const DESTRUCTIVE = new Set(["site_publish", "site_deploy", "deploy_commit", "site_file_delete", "site_media_delete", "site_undo", "site_redo", "site_domain_connect"]);
+  "site_domain_status", "site_embeds", "site_widgets", "site_files", "site_file_get", "site_inbox_list", "site_inbox_get", "site_inbox_stats"]);
+const DESTRUCTIVE = new Set(["site_publish", "site_deploy", "deploy_commit", "site_file_delete", "site_media_delete", "site_inbox_delete", "site_undo", "site_redo", "site_domain_connect"]);
 const OPEN_WORLD = new Set(["site_publish", "site_deploy", "deploy_commit", "site_domain_connect", "media_search", "media_import"]);
 export function toolAnnotations(name, mode = "local") {
   // site_export: remote nur ein Download-Link (lesend), lokal schreibt es eine ZIP auf die Platte.
@@ -192,6 +202,8 @@ export function registerTools(server0, { z, cfg, mode = "local", remote = null }
       + "meta content) ALLER Seiten und ersetzt durch `replace` – EIN Aufruf, EIN Rückgängig-Schritt, Formatierung bleibt. "
       + "scope:\"all\" = alle Fundstellen auf allen Seiten sofort ersetzen (für Kontaktdaten im Kopf-/Fußbereich: plus region:\"footer\" bzw. "
       + "\"header\"); ohne scope ändert der Server bei mehr als einer Fundstelle NICHTS und liefert nur die Liste (oder pages:[…]). "
+      + "Mit scope:\"all\" auch in Skriptdateien (.js, z. B. Preisrechner) – dort nur in Texten/Zahlen/Kommentaren (skripte = Fundstellen); "
+      + "jsReview.status \"pending\" = wartet auf Freigabe durch den Betreuer (dem Nutzer sagen). "
       + "Telefonnummern werden unabhängig von Schreibweise gefunden (Leerzeichen, „/“, „-“, Klammern, geschütztes Leerzeichen); ersetzt "
       + "wird wörtlich durch `replace`. Antwort: treffer = getrennt nach text/linktext/linkziel/attribut; linktextNichtGefunden = tel:-Ziel "
       + "geändert, sichtbarer Linktext aber nicht (dem Nutzer sagen). pages = je Seite gefunden/geändert. Rechtstexte und geschützte Bereiche bleiben unberührt (skipped); steht der Wert in "
@@ -363,6 +375,58 @@ export function registerTools(server0, { z, cfg, mode = "local", remote = null }
       captcha: z.boolean().optional().describe("Sicherheitscode (Bild-Captcha) für dieses Formular an/aus.") },
     async (a) => { try { return asText(await apiFormsSet(cfg, a)); } catch (e) { return asError(e); } }
   );
+  // ---- Formular-Anfragen (Scope anfragen / anfragen:schreiben; Besucherdaten) ----
+  const INBOX_IDS = z.array(z.string()).min(1).max(100).describe("Anfrage-IDs aus site_inbox_list (max. 100).");
+  server.tool(
+    "site_inbox_list",
+    "LESEND: Formular-Anfragen der Website in Kurzform (Datum, Formular, Absender-Name/-Mail, Betreff bzw. erste 120 Zeichen, Anzahl "
+      + "Anhänge, Spam-Grund), neueste zuerst. Ohne status: neu, gelesen und erledigt (ohne Spam und Papierkorb). Enthält Besucherdaten – "
+      + "nur zeigen, was der Nutzer wissen will; nichts ungefragt zusammenfassen oder weitergeben.",
+    { site: z.string().optional().describe(SITE_DESC),
+      status: z.array(z.enum(["neu", "gelesen", "erledigt", "spam", "papierkorb"])).optional().describe("Filter, z. B. ['neu'] oder ['spam']."),
+      form: z.string().optional().describe("Formular-ID oder -Name (site_forms_get)."),
+      seit: z.string().optional().describe("Ab Datum (JJJJ-MM-TT)."), bis: z.string().optional().describe("Bis Datum einschließlich (JJJJ-MM-TT)."),
+      q: z.string().optional().describe("Volltextsuche (Name, E-Mail, Betreff, Nachricht, Felder)."),
+      limit: z.number().int().min(1).max(50).default(20).describe("Seitengröße (max. 50)."),
+      offset: z.number().int().min(0).optional().describe("Weiterblättern: nextOffset der vorigen Antwort.") },
+    async (a) => { try { return asText(await apiInboxGet(cfg, a)); } catch (e) { return asError(e); } }
+  );
+  server.tool(
+    "site_inbox_get",
+    "LESEND: eine Formular-Anfrage vollständig – alle Felder, Absender (Name, E-Mail, Telefon), Anhänge als Liste mit Download-Links "
+      + "(15 Minuten gültig) und Verlauf (Status-Änderungen, von wem). Keine Antwort-Mail über ProntoSite: die Absenderadresse nennen, der "
+      + "Nutzer antwortet im eigenen Mailprogramm. Markiert die Anfrage NICHT automatisch als gelesen (dafür site_inbox_set_status).",
+    { site: z.string().optional().describe(SITE_DESC), id: z.string().describe("Anfrage-ID aus site_inbox_list.") },
+    async ({ site, id }) => { try { return asText(await apiInboxGet(cfg, { site, id })); } catch (e) { return asError(e); } }
+  );
+  server.tool(
+    "site_inbox_set_status",
+    "SCHREIBEND: Status einer oder mehrerer Anfragen setzen: neu | gelesen | erledigt | spam | kein-spam. kein-spam holt eine Anfrage aus "
+      + "dem Spam in den Posteingang (Status neu) und sendet die Empfänger-Benachrichtigung nach, falls sie wegen des Spam-Treffers "
+      + "unterblieben war. Der Spam-Filter lernt aus Korrekturen nicht automatisch. Anfragen im Papierkorb erst mit site_inbox_restore zurückholen.",
+    { site: z.string().optional().describe(SITE_DESC), ids: INBOX_IDS,
+      status: z.enum(["neu", "gelesen", "erledigt", "spam", "kein-spam"]) },
+    async ({ site, ids, status }) => { try { return asText(await apiInboxPost(cfg, { site, action: "status", ids, status })); } catch (e) { return asError(e); } }
+  );
+  server.tool(
+    "site_inbox_delete",
+    "SCHREIBEND: verschiebt Anfragen in den PAPIERKORB – 30 Tage wiederherstellbar (site_inbox_restore), danach endgültig gelöscht. "
+      + "Endgültig löschen geht nur im Dashboard. NUR nach ausdrücklicher Bestätigung des Nutzers (welche Anfragen, wie viele).",
+    { site: z.string().optional().describe(SITE_DESC), ids: INBOX_IDS },
+    async ({ site, ids }) => { try { return asText(await apiInboxPost(cfg, { site, action: "delete", ids })); } catch (e) { return asError(e); } }
+  );
+  server.tool(
+    "site_inbox_restore",
+    "SCHREIBEND: holt Anfragen aus dem Papierkorb zurück (vorheriger Status bzw. Spam bleibt erhalten). IDs aus site_inbox_list mit status ['papierkorb'].",
+    { site: z.string().optional().describe(SITE_DESC), ids: INBOX_IDS },
+    async ({ site, ids }) => { try { return asText(await apiInboxPost(cfg, { site, action: "restore", ids })); } catch (e) { return asError(e); } }
+  );
+  server.tool(
+    "site_inbox_stats",
+    "LESEND: Statistik der Formular-Anfragen – Anzahl je Status und je Formular, Eingänge der letzten 30 und 90 Tage, Spam-Quote. Ohne Inhalte.",
+    { site: z.string().optional().describe(SITE_DESC) },
+    async ({ site }) => { try { return asText(await apiInboxGet(cfg, { site, stats: true })); } catch (e) { return asError(e); } }
+  );
   server.tool(
     "site_undo",
     "SCHREIBEND (Entwurf): macht die letzte(n) Änderung(en) rückgängig – derselbe Verlauf wie der Rückgängig-Knopf im Dashboard (auch "
@@ -564,6 +628,21 @@ export function registerTools(server0, { z, cfg, mode = "local", remote = null }
       }
     );
   }
+  // JS-Prüfungen entscheiden (nur Betreuer-Token; der Server prüft die Rolle zusätzlich).
+  if (mode === "local" || remote?.isManager) {
+    server.tool(
+      "site_js_review",
+      "NUR BETREUER-TOKEN: ausstehende Prüfungen von Skript-Änderungen (JS) einer Website. Ohne action: Liste mit Gründen und Zeilen-Diff "
+        + "(freigegebener Stand → Entwurf; lesend). action='approve' gibt die Änderung frei (danach normaler Publish bzw. mit der nächsten "
+        + "Veröffentlichung), action='reject' setzt die Datei auf den freigegebenen Stand zurück (note = Grund, sieht der Kunde). Nur freigeben, "
+        + "was im Diff verstanden ist; im Zweifel ablehnen oder den Nutzer fragen.",
+      { site: z.string().optional().describe(SITE_DESC),
+        path: z.string().optional().describe("Datei (z. B. 'js/rechner.js') – für approve/reject Pflicht."),
+        action: z.enum(["approve", "reject"]).optional().describe("Leer = nur ansehen."),
+        note: z.string().optional().describe("Notiz/Grund (bei reject sinnvoll).") },
+      async ({ site, path, action, note }) => { try { return asText(await apiJsReview(cfg, { site, path, action, note })); } catch (e) { return asError(e); } }
+    );
+  }
   server.tool(
     "site_embeds",
     "LESEND: erkannte eingebettete Inhalte Dritter (iframes) einer Website – je Embed Host, Seite(n) und "
@@ -622,7 +701,9 @@ export function registerTools(server0, { z, cfg, mode = "local", remote = null }
   // ---- Datei-API: kleine Änderungen am Entwurf direkt auf dem Server (Konzept §7) ----
   const FILE_SMALL = "STANDARDWEG: Änderungen direkt am Server-Entwurf (für reine Textänderungen am einfachsten site_edit_text); "
     + "Export + Deploy nur als Ausnahme für größere Umbauten oder auf ausdrücklichen Wunsch. "
-    + "Skriptdateien (JavaScript) ändern nur Betreuer und nur bei JavaScript-Freigabe der Website – Kunden bekommen dort 403. ";
+    + "Skriptdateien (JavaScript) nur bei JavaScript-Freigabe der Website; Kunden-Änderungen daran prüft der Server: nur Zahlen/Texte/"
+    + "Kommentare → wirkt sofort, sonst gespeichert, aber erst nach Freigabe durch den Betreuer live (Antwort jsReview.status \"pending\" – "
+    + "dem Nutzer so sagen). Preise/Texte in Skripten besser mit site_edit_text scope:\"all\". ";
   server.tool(
     "site_files",
     "LESEND: Listet die Dateien des Entwurfs einer Website (Pfad, Größe, Typ, ob bearbeitbar) und die aktuelle draftRev. "