Files
DogFatherGitandClaude Opus 5 372a0eb95a Ampel und Rueckmeldungen - bewerten darf nur die Betreuung
Der Wunsch: "der Creator sieht, ob es gut ist oder schlecht. Und wenn
nicht gut, dann muss es verbessert werden -- und nur der Ansprechpartner
kann das aendern. Die Creator koennen nur Nachrichten hinterlassen zum
Kommunizieren."

ZWEI GETRENNTE DINGE, und sie getrennt zu halten ist der ganze Punkt:

  DIE AMPEL ist eine Beurteilung. Sie gehoert dem Betreuer, und nur er
  setzt sie. Koennte ein Creator sich selbst auf gruen stellen, waere
  sie wertlos -- dann stuende ueberall gruen. Der Server lehnt es mit
  403 ab und sagt dabei, wer es kann.

  DIE NACHRICHT ist ein Gespraech. Sie gehoert beiden. Ein Creator, der
  auf eine Bewertung nicht antworten kann, bekommt ein Urteil statt
  einer Betreuung.

DREI STUFEN, NICHT FUENF. Eine Zahl von 1 bis 5 klingt genauer und ist
es nicht: Niemand kann den Unterschied zwischen 3 und 4 erklaeren, und
am Ende steht ueberall die 3. Die Frage lautet "reicht das schon?", und
darauf gibt es drei ehrliche Antworten -- passt so, noch verbessern,
noch nicht angesehen.

"NOCH VERBESSERN" VERLANGT EINE BEGRUENDUNG. Eine Bewertung, mit der der
Creator nichts anfangen kann, ist nicht streng, sondern nur
entmutigend. Der Server lehnt sie ohne Grund ab; die Oberflaeche fragt
deshalb gleich danach, statt hinterher eine Fehlermeldung zu zeigen.
"Passt so" braucht keinen -- da gibt es nichts zu erklaeren.

Die Stufe steht IMMER an der Karte, auch fuer den Creator, auch wenn sie
"noch nicht angesehen" lautet. Er soll sehen, wo er steht, ohne fragen
zu muessen. Der Grund steht daneben in voller Breite, nicht in einer
Ecke.

FREMDE NACHRICHTEN BLEIBEN STEHEN -- auch fuer DogFather. Ein Gespraech
nachtraeglich umzuschreiben waere schlimmer, als eine unbedachte
Aeusserung stehen zu lassen. Wer etwas richtigstellen will, schreibt
eine neue.

VORLAGEN AUCH FUER COMMUNITY UND TECHNIK (Screens 11 und 12): 14
Moderations-, Aktions- und Konfliktpunkte, 12 Technikpunkte. Ton steht
vorn, weil schlechter Ton der Grund Nummer eins ist, warum Leute einen
Stream verlassen. Die drei Bereiche laufen jetzt ueber EINE Zuordnung
statt drei fast gleicher Bloecke -- sonst weicht der dritte irgendwann
ab. Die beiden alten Zweige wurden entfernt: Toter Code, den man stehen
laesst, wird beim naechsten Mal fuer lebenden gehalten.

Die Ampeln einer ganzen Liste kommen in EINER Abfrage. Zwanzig
Eintraege einzeln zu fragen waeren zwanzig Anfragen, und die Seite
ruckelte sichtbar beim Aufbau. Die Sichtbarkeitspruefung laeuft dabei
je Eintrag, nicht einmal pauschal -- ein fremder Eintrag taucht auch in
der Sammelabfrage nicht auf.

Co-Authored-By: Claude Opus 5 <[email protected]>
2026-09-01 16:49:56 +02:00

277 lines
12 KiB
JavaScript
Raw Permalink Blame History

This file contains ambiguous Unicode characters
This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.
/* =====================================================================
workspace-ampel.js — Bewertung und Rückmeldungen an einem Eintrag.
Der Wunsch vom 01.09.2026 zur LIVE-Analyse: "der Creator sieht, ob es
gut ist oder schlecht. Und wenn nicht gut, dann muss es verbessert
werden – und nur der Ansprechpartner kann das ändern. Die Creator
können nur Nachrichten hinterlassen zum Kommunizieren."
Das sind zwei getrennte Dinge, und sie getrennt zu halten ist der
ganze Punkt:
DIE AMPEL ist eine Beurteilung. Sie gehört dem Betreuer, und nur
er setzt sie. Könnte der Creator sie selbst auf grün
stellen, wäre sie wertlos.
DIE NACHRICHT ist ein Gespräch. Sie gehört beiden. Ein Creator, der
auf eine Bewertung nicht antworten kann, bekommt ein
Urteil statt einer Betreuung.
Drei Stufen, nicht fünf. Eine Zahl von 1 bis 5 klingt genauer und ist
es nicht: Niemand kann den Unterschied zwischen 3 und 4 erklären, und
am Ende steht überall 3. Die Frage lautet "reicht das schon?", und
darauf gibt es drei ehrliche Antworten:
gut so lassen
verbessern noch nicht gut genug -- mit dem Satz, was fehlt
offen noch nicht angesehen
"verbessern" verlangt eine Begründung. Eine Bewertung ohne Grund ist
für den Creator nicht umsetzbar, sondern nur entmutigend -- und der
Server lehnt sie deshalb ab.
===================================================================== */
import express from "express";
import {
db, sitzungLesen, istLeitung, darfCreator, protokolliere, echteIp,
} from "./workspace.js";
export const ampelRouter = express.Router();
const jetzt = () => new Date().toISOString();
const STUFEN = new Set(["gut", "verbessern", "offen"]);
const NACHRICHT_MAX = 2000;
let bereit = false;
function tabellen() {
if (bereit) return;
db().exec(`
/* Die Ampel haengt am Eintrag, steht aber in einer eigenen Tabelle:
Sie hat einen eigenen Verfasser und einen eigenen Zeitpunkt, und
beides gehoert nicht in die Zeile des Eintrags. */
CREATE TABLE IF NOT EXISTS eintrag_ampel (
eintrag_id INTEGER PRIMARY KEY REFERENCES eintraege(id) ON DELETE CASCADE,
stufe TEXT NOT NULL CHECK (stufe IN ('gut','verbessern','offen')),
begruendung TEXT,
von INTEGER REFERENCES personen(id) ON DELETE SET NULL,
am TEXT NOT NULL
);
/* Rueckmeldungen. Jeder Beteiligte darf schreiben -- auch der
Creator, fuer den das oft der einzige Weg ist, etwas zu sagen. */
CREATE TABLE IF NOT EXISTS eintrag_nachricht (
id INTEGER PRIMARY KEY AUTOINCREMENT,
eintrag_id INTEGER NOT NULL REFERENCES eintraege(id) ON DELETE CASCADE,
text TEXT NOT NULL,
von INTEGER REFERENCES personen(id) ON DELETE SET NULL,
am TEXT NOT NULL
);
CREATE INDEX IF NOT EXISTS idx_nachricht_eintrag
ON eintrag_nachricht (eintrag_id);
`);
bereit = true;
}
function angemeldet(req, res, next) {
const person = sitzungLesen(req);
if (!person) return res.status(401).json({ fehler: "nicht_angemeldet" });
req.person = person;
try { tabellen(); } catch (f) {
console.error("[workspace] Ampeltabellen:", f?.message);
return res.status(503).json({ fehler: "nicht_verfuegbar" });
}
next();
}
ampelRouter.use("/workspace/api/eintrag", angemeldet);
function gleicheHerkunft(req, res, next) {
const herkunft = req.get("origin");
if (!herkunft) return next();
let erlaubt;
try { erlaubt = new URL(herkunft).host === req.get("host"); } catch { erlaubt = false; }
if (!erlaubt) return res.status(403).json({ fehler: "fremde_herkunft" });
next();
}
/** Darf die Person diesen Eintrag ueberhaupt sehen? Dieselbe Regel wie
* im Bereichsmodul -- sie hier zu wiederholen waere der sichere Weg,
* dass die beiden irgendwann auseinanderlaufen, deshalb wird sie aus
* denselben Bausteinen gebildet. */
function sichtbarerEintrag(person, id) {
const e = db().prepare("SELECT id, creator_id, bereich FROM eintraege WHERE id = ?").get(id);
if (!e) return null;
if (istLeitung(person)) return e;
if (person.rolle === "creator") return e.creator_id === person.id ? e : null;
return e.creator_id && darfCreator(person, e.creator_id) ? e : null;
}
/* ---------- Lesen ---------------------------------------------------------- */
ampelRouter.get("/workspace/api/eintrag/:id(\\d+)/rueckmeldung", (req, res) => {
try {
const e = sichtbarerEintrag(req.person, Number(req.params.id));
if (!e) return res.status(404).json({ fehler: "nicht_gefunden" });
const ampel = db().prepare(`
SELECT a.*, p.name AS von_name
FROM eintrag_ampel a LEFT JOIN personen p ON p.id = a.von
WHERE a.eintrag_id = ?`).get(e.id) || null;
const nachrichten = db().prepare(`
SELECT n.*, p.name AS von_name, p.rolle AS von_rolle
FROM eintrag_nachricht n LEFT JOIN personen p ON p.id = n.von
WHERE n.eintrag_id = ? ORDER BY n.id`).all(e.id);
res.json({
ampel, nachrichten,
/* Die Oberflaeche soll nicht selbst raten muessen, wer was darf --
sonst zeigt sie einem Creator einen Knopf, der beim Klicken 403
sagt. Das ist schlechter als kein Knopf. */
darf_bewerten: req.person.rolle !== "creator",
ich: req.person.id,
});
} catch (fehler) {
console.error("[workspace] Rückmeldung lesen:", fehler?.message);
res.status(503).json({ fehler: "nicht_verfuegbar" });
}
});
/* ---------- Bewerten (NUR die Betreuung) ---------------------------------- */
ampelRouter.put("/workspace/api/eintrag/:id(\\d+)/ampel", gleicheHerkunft,
express.json({ limit: "8kb" }), (req, res) => {
try {
/* Ein Creator bewertet sich nicht selbst. Waere das moeglich,
waere die Ampel wertlos -- jeder stuende auf gruen. */
if (req.person.rolle === "creator") {
return res.status(403).json({
fehler: "Die Bewertung setzt deine Betreuung. Du kannst hier antworten.",
});
}
const e = sichtbarerEintrag(req.person, Number(req.params.id));
if (!e) return res.status(404).json({ fehler: "nicht_gefunden" });
const stufe = String(req.body.stufe || "");
if (!STUFEN.has(stufe)) return res.status(400).json({ fehler: "Unbekannte Stufe." });
const begruendung = String(req.body.begruendung ?? "").trim().slice(0, NACHRICHT_MAX);
/* "verbessern" ohne Grund ist fuer den Creator nicht umsetzbar --
nur entmutigend. Deshalb Pflicht. */
if (stufe === "verbessern" && begruendung.length < 5) {
return res.status(400).json({
fehler: "Bitte kurz schreiben, was verbessert werden soll – sonst kann er nichts damit anfangen.",
});
}
db().prepare(`INSERT INTO eintrag_ampel (eintrag_id, stufe, begruendung, von, am)
VALUES (?,?,?,?,?)
ON CONFLICT(eintrag_id) DO UPDATE SET
stufe = excluded.stufe, begruendung = excluded.begruendung,
von = excluded.von, am = excluded.am`)
.run(e.id, stufe, begruendung || null, req.person.id, jetzt());
protokolliere("ampel_gesetzt", req.person, echteIp(req), `Eintrag ${e.id}: ${stufe}`);
const neu = db().prepare(`
SELECT a.*, p.name AS von_name FROM eintrag_ampel a
LEFT JOIN personen p ON p.id = a.von WHERE a.eintrag_id = ?`).get(e.id);
res.json({ ampel: neu });
} catch (fehler) {
console.error("[workspace] Ampel setzen:", fehler?.message);
res.status(503).json({ fehler: "nicht_verfuegbar" });
}
});
/* ---------- Nachricht schreiben (JEDER Beteiligte) ------------------------ */
ampelRouter.post("/workspace/api/eintrag/:id(\\d+)/nachricht", gleicheHerkunft,
express.json({ limit: "8kb" }), (req, res) => {
try {
const e = sichtbarerEintrag(req.person, Number(req.params.id));
if (!e) return res.status(404).json({ fehler: "nicht_gefunden" });
const text = String(req.body.text ?? "").trim();
if (text.length < 2) return res.status(400).json({ fehler: "Die Nachricht ist leer." });
if (text.length > NACHRICHT_MAX) {
return res.status(400).json({ fehler: "Die Nachricht ist zu lang." });
}
db().prepare("INSERT INTO eintrag_nachricht (eintrag_id, text, von, am) VALUES (?,?,?,?)")
.run(e.id, text, req.person.id, jetzt());
protokolliere("nachricht", req.person, echteIp(req), `Eintrag ${e.id}`);
const alle = db().prepare(`
SELECT n.*, p.name AS von_name, p.rolle AS von_rolle
FROM eintrag_nachricht n LEFT JOIN personen p ON p.id = n.von
WHERE n.eintrag_id = ? ORDER BY n.id`).all(e.id);
res.json({ nachrichten: alle });
} catch (fehler) {
console.error("[workspace] Nachricht:", fehler?.message);
res.status(503).json({ fehler: "nicht_verfuegbar" });
}
});
/* ---------- Eigene Nachricht zurücknehmen --------------------------------
Nur die eigene, und nur die eigene. Fremde Nachrichten zu löschen
hiesse, ein Gespräch nachträglich umzuschreiben -- selbst für die
Leitung nicht. Wer etwas richtigstellen will, schreibt eine neue. */
ampelRouter.delete("/workspace/api/eintrag/:id(\\d+)/nachricht/:nid(\\d+)", gleicheHerkunft,
(req, res) => {
try {
const e = sichtbarerEintrag(req.person, Number(req.params.id));
if (!e) return res.status(404).json({ fehler: "nicht_gefunden" });
const n = db().prepare("SELECT * FROM eintrag_nachricht WHERE id = ? AND eintrag_id = ?")
.get(Number(req.params.nid), e.id);
if (!n) return res.status(404).json({ fehler: "nicht_gefunden" });
if (n.von !== req.person.id) {
return res.status(403).json({
fehler: "Fremde Nachrichten bleiben stehen. Schreib eine neue, wenn etwas richtigzustellen ist.",
});
}
db().prepare("DELETE FROM eintrag_nachricht WHERE id = ?").run(n.id);
res.json({ ok: true });
} catch (fehler) {
console.error("[workspace] Nachricht löschen:", fehler?.message);
res.status(503).json({ fehler: "nicht_verfuegbar" });
}
});
/* ---------- Sammelabfrage für eine Liste ---------------------------------
Damit die Seite nicht für zwanzig Einträge zwanzig Abfragen schickt.
Geliefert wird nur, was die Person ohnehin sehen darf -- die Prüfung
läuft je Eintrag, nicht einmal pauschal. */
ampelRouter.get("/workspace/api/eintrag/ampeln", (req, res) => {
try {
const ids = String(req.query.ids || "").split(",")
.map((x) => Number(x)).filter((x) => Number.isInteger(x) && x > 0).slice(0, 200);
if (!ids.length) return res.json({ ampeln: {}, anzahlNachrichten: {} });
const erlaubt = ids.filter((id) => sichtbarerEintrag(req.person, id));
if (!erlaubt.length) return res.json({ ampeln: {}, anzahlNachrichten: {} });
const platz = erlaubt.map(() => "?").join(",");
const ampeln = {};
for (const a of db().prepare(`
SELECT a.*, p.name AS von_name FROM eintrag_ampel a
LEFT JOIN personen p ON p.id = a.von
WHERE a.eintrag_id IN (${platz})`).all(...erlaubt)) {
ampeln[a.eintrag_id] = a;
}
const anzahlNachrichten = {};
for (const z of db().prepare(`
SELECT eintrag_id, COUNT(*) AS n FROM eintrag_nachricht
WHERE eintrag_id IN (${platz}) GROUP BY eintrag_id`).all(...erlaubt)) {
anzahlNachrichten[z.eintrag_id] = z.n;
}
res.json({ ampeln, anzahlNachrichten, darf_bewerten: req.person.rolle !== "creator" });
} catch (fehler) {
console.error("[workspace] Ampeln lesen:", fehler?.message);
res.status(503).json({ fehler: "nicht_verfuegbar" });
}
});