Die rechte Hand steht ueberall an zweiter Stelle - und im Kalender ueberhaupt

Filipe, mit Bildschirmfoto der Liste "Wen gehst du durch?" (Diene, Dogi, VanVan - alphabetisch, die rechte Hand ganz rechts): "rechte hand soll immer als erstes sein. ueberall. wie in dem sinne soll sie ganz links sein. ausser bei personen und zugaenge soll sie ueber jedem aber unter dogfather sein. ich will dass auch ueberall immer nach der hirarchie gearbeitet wird." EINE REIHENFOLGE, NICHT ZWEI REGELN ROLLEN_REIHE in workspace.js ist jetzt: Spicy Media > DogFather > Rechte Hand > Manager > Scout > Creator > Modi Er hat zwei Faelle beschrieben, aber es braucht nur eine Reihenfolge: In den Team-Listen kommt DogFather gar nicht vor (er beurteilt, er wird nicht beurteilt) - dort steht sie dadurch automatisch ganz vorn. In "Personen & Zugaenge" steht er drin, also steht sie dort hinter ihm. Zwei Sonderfaelle waeren zwei Stellen, an denen es auseinanderlaeuft. Dass Spicy Media davor bleibt, ist seine ausdrueckliche Entscheidung auf Nachfrage. 'gast' steht bewusst nicht in der Liste und faellt ans Ende. ROLLEN_SORTIERUNG (der SQL-Ausdruck) wird jetzt aus ROLLEN_REIHE ABGELEITET statt danebengeschrieben. Bis heute stand die Reihenfolge zweimal da; beim Hochziehen der rechten Hand haetten beide geaendert werden muessen. Eine Liste, die niemand pflegt, kann nicht veralten - derselbe Grundsatz wie beim Spaltenverlust vom 06.09. WAS DABEI AUFFIEL, OHNE DASS JEMAND DANACH GESUCHT HAT Der Server sortierte laengst richtig. DREI Auswahllisten im Browser haben seine Reihenfolge wieder verworfen und nach einer eigenen Liste mit fuenf Agentur-Rollen neu gezeichnet - Team Dogi kommt darin nicht vor und DARF es nicht (bereiche.js laedt jeder herunter). - Chat-Auswahl und Sicht-Umschalter: Team Dogi landete in einem Nachzuegler-Block ganz unten, hinter jedem Creator. - Teilnehmerwahl im Kalender: dort gab es nicht einmal einen Nachzuegler-Block. Die rechte Hand und die Modis standen GAR NICHT zur Auswahl. DogFather konnte sein eigenes Team zu keinem Termin einladen, und auf dem Bildschirm sah das vollkommen normal aus. Das ist derselbe Fehler zum vierten Mal (Chat 10.09., Personenliste 10.09., Sicht-Umschalter 11.09., Kalender 11.09.). Deshalb keine vierte Einzelreparatur, sondern eine Stelle: window.Bereiche.gruppieren() gruppiert in genau der Reihenfolge, in der der Server die Menschen schickt - ohne einen einzigen Rang zu kennen. Fehlt der Helfer, wird eine Gruppe mit allen gezeichnet: nicht schoen, aber sichtbar, und niemand verschwindet. Der Kalender-Weg schickt die Ueberschrift jetzt mit, wie der Chat es laengst tut. PRUEFUNGEN pruef-nachwuchs 109 -> 123 (Abschnitt 12: die Reihenfolge, mit der Gegenprobe, dass sie NICHT alphabetisch ist -- Rieke steht alphabetisch hinten, mit "Anna" waere jede Zeile gruen ohne etwas zu messen) pruef-dabei-optik misst die Wahl jetzt im Browser: ist Team Dogi ueberhaupt da, und steht es vorn pruef-rollen 315 (vorher 312), 423 s gemessen. Die Notbremse lag bei 480 s und hat angeschlagen - kein Haenger, sondern zu wenig Luft, seit die rechte Hand vier Kacheln mehr hat. Jetzt 900 s, mit der Messung daneben und dem Hinweis, beim naechsten Mal nicht die Zahl zu erhoehen, sondern nachzusehen. Co-Authored-By: Claude Opus 5 <[email protected]>
2026-09-11 23:58:50 +02:00
parent bba3642664
commit 1ab2729e47
39 changed files with 638 additions and 454 deletions
@@ -97,6 +97,21 @@ const idDogi = anlegen("Filipe Fernandes Ribeiro", "admin", "CODE-DOGI-0001");
 const idCig = anlegen("Cigdem", "manager", "CODE-MANA-0001");
 const idPat = anlegen("Patrick von Hohenstein", "scout", "CODE-SCOU-0001");
 const idLuna = anlegen("Luna", "creator", "CODE-CREA-0001");
+/* DIE RECHTE HAND (11.09.2026). Sie stand in dieser Auswahl bis heute
+   GAR NICHT -- kalender.js zeichnete nach einer Liste aus fuenf
+   Agentur-Rollen, und wer nicht darin vorkam, fiel lautlos heraus.
+   DogFather konnte sein eigenes Team zu keinem Termin einladen, und
+   der Bildschirm sah dabei vollkommen normal aus.
+
+   Sie heisst hier "VanVan" und damit alphabetisch HINTER allen anderen
+   ausser Patrick -- mit einem Namen wie "Anna" waere die Zeile
+   "sie steht vorn" gruen, ohne etwas zu messen. */
+anlegen("VanVan", "hand", "CODE-HAND-0001");
+
+/* Die Zahl steht hier und nicht in den Pruefzeilen: Wer unten einen
+   Menschen dazusetzt, muss sonst drei Zahlen nachziehen -- und die
+   vergessene ist die, die dann still falsch ist. */
+const ZUR_WAHL = 4;   // alle aktiven ausser DogFather selbst
 d.prepare("INSERT INTO betreuung (creator_id, betreuer_id, seit) VALUES (?,?,?)").run(idLuna, idPat, jetzt);
 d.prepare("INSERT INTO scout_zuteilung (scout_id, manager_id, seit) VALUES (?,?,?)").run(idPat, idCig, jetzt);
 d.close();
@@ -139,13 +154,33 @@ console.log("\n=== Die Wahl aus Sicht von DogFather ===");
  const knoepfe = await seite.$$("#dabei-wahl .k-dabei__knopf");
  /* DIE ANZAHL IN DIE BEDINGUNG: Wären es null, würde jede folgende
     Messung über eine leere Menge laufen und grün melden. */
-  ok(knoepfe.length === 3,
-    `${knoepfe.length} Personen zur Auswahl (erwartet 3 -- alle außer ihm selbst)`);
+  ok(knoepfe.length === ZUR_WAHL,
+    `${knoepfe.length} Personen zur Auswahl (erwartet ${ZUR_WAHL} -- alle außer ihm selbst)`);

  const gruppen = await seite.$$eval(".k-dabei__gruppe", (g) => g.map((x) => x.dataset.rolle));
-  ok(gruppen.length === 3 && !gruppen.includes("admin"),
+  ok(gruppen.length === ZUR_WAHL && !gruppen.includes("admin"),
    `Gruppen: ${gruppen.join(", ") || "keine"} — er selbst ist die einzige admin-Person und steht nicht drin`);

+  /* ---- DIE REIHENFOLGE (Filipe, 11.09.2026) --------------------------
+
+     "rechte hand soll immer als erstes sein. ueberall."
+
+     Zwei getrennte Fragen, und beide muessen gestellt werden: Ist sie
+     ueberhaupt da? Und steht sie vorn? Die erste allein war jahrelang
+     mit Nein zu beantworten, ohne dass es jemand sah. */
+  ok(gruppen.includes("hand"),
+    `die rechte Hand steht ueberhaupt zur Wahl (${gruppen.join(", ")})`);
+  ok(gruppen[0] === "hand",
+    `und sie steht ganz vorn (erste Gruppe: ${gruppen[0]})`);
+
+  /* GEGENPROBE: Alphabetisch stuende sie NICHT vorn. Ohne diese Zeile
+     waere "sie steht vorn" auch dann gruen, wenn gar nicht sortiert
+     wird -- die Namen kaemen zufaellig in der richtigen Folge. */
+  const namen = await seite.$$eval("#dabei-wahl .k-dabei__name", (n) => n.map((x) => x.textContent.trim()));
+  const alphabetisch = [...namen].sort((a, b) => a.localeCompare(b, "de"));
+  ok(namen[0] === "VanVan" && alphabetisch[0] !== "VanVan",
+    `gezeichnet: ${namen.join(", ")} — alphabetisch waere ${alphabetisch.join(", ")}`);
+
  /* DER KERN: Kleben Name und Zeichen? Gemessen wird der echte
     Abstand zwischen den zwei Kästchen, nicht der CSS-Wert. */
  const abstand = await seite.evaluate(() => {
@@ -168,8 +203,13 @@ console.log("\n=== Die Wahl aus Sicht von DogFather ===");

  /* Die Rolle steht stattdessen über der Gruppe. */
  const titel = await seite.$$eval(".k-dabei__titel", (t) => t.map((x) => x.textContent.trim()));
-  ok(titel.length === 3 && titel.some((t) => /Manager/.test(t)),
+  ok(titel.length === ZUR_WAHL && titel.some((t) => /Manager/.test(t)),
    `Gruppenüberschriften: ${titel.join(" · ")}`);
+  /* Die Ueberschrift der Team-Gruppe kommt vom SERVER -- diese Datei im
+     Browser darf den Rollennamen gar nicht kennen. Steht sie da, hat
+     der Weg funktioniert. */
+  ok(/Team Dogi/.test(titel[0] || ""),
+    `und die erste heißt „${titel[0]}" — sie kommt vom Server, nicht aus dem Browser`);

  /* Daumengröße. 40 px ist die Grenze, die dieses Haus überall nutzt. */
  const hoehe = await seite.$eval("#dabei-wahl .k-dabei__knopf",
@@ -647,6 +647,78 @@ melde("=== 11. Probe → Zugang ===");
  ok(z?.rolle === "modi" && z?.aktiv === 1, "und er steht als aktiver Modi in der Datenbank");
 }

+/* =======================================================================
+   12. DIE REIHENFOLGE — NACH RANG, NICHT NACH NAMEN
+
+   Filipe, 11.09.2026 mit Bildschirmfoto der Liste "Wen gehst du
+   durch?" (Diene, Dogi, VanVan — alphabetisch, die rechte Hand
+   ganz rechts): "rechte hand soll immer als erstes sein. ueberall.
+   ... ausser bei personen und zugaengen soll sie ueber jedem aber
+   unter dogfather sein."
+
+   WARUM DIE NAMEN IM PRUEFSTAND SO GEWAEHLT SIND: Rieke (rechte Hand)
+   steht alphabetisch HINTER Frida, Mara und Nina. Waere die Sortierung
+   auch nur an einer Stelle wieder die alte, stuende sie dort hinten —
+   und die Pruefung wuerde es sehen. Mit einem Namen wie "Anna" waere
+   jede Zeile hier gruen, ohne irgendetwas zu messen.
+   ======================================================================= */
+melde("");
+melde("=== 12. Die Reihenfolge ===");
+{
+  /* Die Quelle selbst: EINE Liste, und der SQL-Ausdruck daraus
+     abgeleitet. Zwei gepflegte Listen waeren die Stelle, an der es
+     beim naechsten Mal auseinanderlaeuft. */
+  const w = await import("./workspace.js");
+  const reihe = w.ROLLEN_REIHE;
+  ok(reihe.indexOf("hand") === reihe.indexOf("admin") + 1,
+    `die rechte Hand steht direkt hinter DogFather (${reihe.join(" > ")})`);
+  ok(reihe.every((r, i) => w.ROLLEN_SORTIERUNG.includes(`WHEN '${r}' THEN ${i}`)),
+    "der SQL-Ausdruck kennt jede Rolle mit genau ihrer Nummer");
+  ok(w.ROLLEN_SORTIERUNG.split("WHEN").length - 1 === reihe.length,
+    `und keine darueber hinaus (${w.ROLLEN_SORTIERUNG.split("WHEN").length - 1} von ${reihe.length})`);
+  /* Die Aufrufer stellen "rolle" per .replace() auf "p.rolle" um --
+     das trifft nur die erste Fundstelle. Kaeme das Wort zweimal vor,
+     waere die zweite Haelfte des Ausdrucks still falsch. */
+  ok(w.ROLLEN_SORTIERUNG.split("rolle").length - 1 === 1,
+    "das Wort „rolle“ kommt darin genau einmal vor");
+  ok(!w.ROLLEN_REIHE.includes("gast"),
+    "die Community steht nicht in der Rangfolge — sie faellt ans Ende");
+
+  /* ---- Die Liste aus dem Bildschirmfoto ---- */
+  const lage = json(await roh("/workspace/api/entwicklung/lage", HAUS, { cookie: kDogi }));
+  const namen = (lage.personen || []).map((p) => p.name);
+  ok(namen.length >= 3, `${namen.length} Menschen in „Wen gehst du durch?“ (${namen.join(", ")})`);
+  ok(namen[0] === "Rieke", `die rechte Hand steht ganz links (${namen[0]})`);
+  /* GEGENPROBE, und ohne sie beweist die Zeile darueber nichts:
+     Alphabetisch stuende sie HINTEN. Dass beides verschieden ist, ist
+     der eigentliche Beweis, dass hier nach Rang sortiert wird. */
+  const alphabetisch = [...namen].sort((a, b) => a.localeCompare(b, "de"));
+  ok(alphabetisch[0] !== "Rieke",
+    `alphabetisch stuende dort ${alphabetisch[0]} — die Reihenfolge ist also keine Namensfolge`);
+
+  /* ---- Personen & Zugaenge: unter DogFather, ueber allen anderen ---- */
+  const liste = json(await roh("/workspace/api/verwaltung/personen", HAUS, { cookie: kDogi }));
+  const alle = (liste.personen || []).map((p) => ({ name: p.name, rolle: p.rolle }));
+  ok(alle.length >= 4, `${alle.length} Menschen in „Personen & Zugaenge“`);
+  const platz = (rolle) => alle.findIndex((p) => p.rolle === rolle);
+  const vonHand = platz("hand");
+  ok(vonHand >= 0, "die rechte Hand steht in der Liste");
+  ok(platz("admin") >= 0 && platz("admin") < vonHand,
+    `DogFather steht ueber ihr (Platz ${platz("admin") + 1} vor ${vonHand + 1})`);
+  const darunter = alle.filter((p, i) => i > vonHand).map((p) => p.rolle);
+  const darueber = alle.filter((p, i) => i < vonHand).map((p) => p.rolle);
+  ok(darueber.every((r) => r === "admin" || r === "spicy"),
+    `ueber ihr steht nur die Leitung (${darueber.join(", ") || "niemand"})`);
+  ok(darunter.includes("manager") && darunter.includes("modi"),
+    `unter ihr stehen Manager und Modis (${[...new Set(darunter)].join(", ")})`);
+
+  /* ---- Und dieselbe Regel im Haus nebenan ---- */
+  const crew = json(await roh("/workspace/api/entwicklung/lage", CREW, { cookie: kHand }));
+  const crewNamen = (crew.personen || []).map((p) => p.name);
+  ok(crewNamen[0] === "Rieke",
+    `auch auf der Team-Adresse steht sie vorn (${crewNamen.join(", ")})`);
+}
+
 /* ------------------------------------------------------------------- */
 melde("");
 melde(`${geprueft} Pruefungen, ${fehler} Fehler`);
@@ -49,7 +49,25 @@ await import("./index.js");
   richtig fuer den Betrieb, fatal fuer eine Pruefung. Ohne diese
   Zeile bleibt der Prozess nach einem Fehler ewig stehen, weil der
   Server ihn am Leben haelt (siehe helfer-notbremse.mjs). */
-notbremse(480_000, "pruef-rollen");
+/* DIE FRIST MUSS ZUR ARBEIT PASSEN -- 900 s seit dem 11.09.2026.
+
+   Diese Pruefung besucht JEDE Kachel JEDER Rolle einzeln und wartet
+   dabei auf networkidle. Ihre Dauer haengt also an der Zahl der
+   Kacheln, und die ist heute gewachsen: Die rechte Hand hat vier
+   dazubekommen (Entwicklung, Talente, Wer sieht was, Wie geht es dir).
+
+   Bei 480 s ist der Lauf danach in die Notbremse gelaufen -- mitten im
+   Zaehlen, nichts haing. Gemessen am 11.09.2026 auf diesem Rechner:
+   423 s fuer 315 Pruefungen, also 12 % Luft. Das ist zu knapp; eine
+   Notbremse, die bei normaler Arbeit ausloest, ist keine Notbremse,
+   sondern ein Fehlalarm -- und ein Fehlalarm, der regelmaessig kommt,
+   wird ueberlesen (siehe die Projektnotiz zur Loeschwarnung).
+
+   900 s sind gut das Doppelte der Messung. Wer hier wieder anstoesst,
+   soll NICHT die Zahl erhoehen, sondern zuerst nachsehen, ob der Lauf
+   noch vorankommt: eine Frist, die man dreimal hochsetzt, misst
+   irgendwann gar nichts mehr. */
+notbremse(900_000, "pruef-rollen");
 await new Promise((r) => setTimeout(r, 700));
 const BASIS = "http://127.0.0.1:4193";
 setTimeout(() => { console.log("ABBRUCH"); process.exit(1); }, 540_000).unref?.();
@@ -61,6 +61,7 @@
 import express from "express";
 import {
  db, sitzungLesen, protokolliere, echteIp, fuehrtTeamDogi, ROLLEN_NAME,
+  ROLLEN_SORTIERUNG,
 } from "./workspace.js";
 import {
  ENTWICKLUNG_BLOECKE, ENTWICKLUNG_PUNKTE, ENTWICKLUNG_STUFEN,
@@ -211,7 +212,8 @@ entwicklungRouter.get("/workspace/api/entwicklung/lage", (req, res) => {
    const platz = KARTEN_ROLLEN.map(() => "?").join(", ");
    const personen = leitung
      ? db().prepare(`SELECT id, name, rolle, erstellt FROM personen
-           WHERE rolle IN (${platz}) AND aktiv = 1 ORDER BY name COLLATE NOCASE`)
+           WHERE rolle IN (${platz}) AND aktiv = 1
+           ORDER BY ${ROLLEN_SORTIERUNG}, name COLLATE NOCASE`)
        .all(...KARTEN_ROLLEN)
      : [];

@@ -15,7 +15,7 @@ import express from "express";
 import {
  db, protokolliere, echteIp, sitzungLesen, istLeitung, istDogFather, siehtAlles, betreutWo, termineSichtbar, heuteLokal,
  kategorienFuer,
-  externPruefen, externSql, betreuteIds, einladbareIds, verborgeneIds, ROLLEN_SORTIERUNG,
+  externPruefen, externSql, betreuteIds, einladbareIds, verborgeneIds, ROLLEN_SORTIERUNG, ROLLEN_GRUPPE,
 } from "./workspace.js";
 import {
  nachfuellenAlle, serienPruefen, serieAnlegen, zuordnungErzwingen,
@@ -265,10 +265,15 @@ kalenderRouter.get("/workspace/api/einladbar", (req, res) => {
       tritt ein, sobald jemand niemandem zugeordnet ist. */
    if (ids !== null && !ids.length) return res.json({ personen: [] });

+    /* DIE UEBERSCHRIFT KOMMT MIT (11.09.2026) -- wie beim Chat. Ohne
+       sie kann der Browser eine Gruppe nicht beschriften, deren
+       Rollennamen er gar nicht kennen darf, und liess sie deshalb
+       einfach weg. Team Dogi stand in dieser Auswahl nie. */
    res.json({
      personen: db().prepare(
        `SELECT id, name, rolle FROM personen WHERE aktiv = 1${wo}
-         ORDER BY ` + ROLLEN_SORTIERUNG + ", name").all(...werte),
+         ORDER BY ` + ROLLEN_SORTIERUNG + ", name").all(...werte)
+        .map((p) => ({ ...p, gruppe: ROLLEN_GRUPPE[p.rolle] || "Weitere" })),
    });
  } catch (fehler) {
    console.error("[workspace] Einladbar:", fehler?.message);
@@ -69,10 +69,32 @@ const VERSUCHE_MAX = 8;             // pro IP
 const VERSUCHE_FENSTER_MIN = 10;
 const ROLLEN = new Set(["spicy", "admin", "manager", "scout", "creator", "hand", "modi", "gast"]);

-/* Die Reihenfolge, in der Rollen ueberall erscheinen: Spicy Media
-   zuerst, dann DogFather, Manager, Scout, Creator. Steht hier einmal,
-   damit keine Liste eine eigene Reihenfolge erfindet. */
-export const ROLLEN_REIHE = ["spicy", "admin", "manager", "scout", "creator", "hand", "modi"];
+/* DIE EINE REIHENFOLGE, IN DER ROLLEN UEBERALL ERSCHEINEN.
+
+   Sie steht hier einmal, damit keine Liste eine eigene erfindet.
+
+   DIE RECHTE HAND STEHT AN ZWEITER STELLE (Filipe, 11.09.2026:
+   "rechte hand soll immer als erstes sein. ueberall. ... ausser bei
+   personen und zugaengen soll sie ueber jedem aber unter dogfather
+   sein. ich will dass auch ueberall immer nach der hirarchie
+   gearbeitet wird.").
+
+   EINE ZAHL STATT ZWEIER REGELN: Er hat zwei Saetze gesagt, aber es
+   braucht nur eine Reihenfolge. In den Team-Listen kommt DogFather
+   gar nicht vor (er beurteilt, er wird nicht beurteilt) -- dort steht
+   sie damit automatisch ganz vorn. In "Personen & Zugaenge" steht er
+   drin, also steht sie dort hinter ihm. Zwei Sonderfaelle waeren zwei
+   Stellen, an denen es auseinanderlaufen kann.
+
+   SPICY MEDIA BLEIBT DAVOR -- seine ausdrueckliche Entscheidung auf
+   Nachfrage am 11.09.2026. Die Agentur ist keine Stufe in seinem Team,
+   sondern steht daneben; an Rechten aendert die Reihenfolge ohnehin
+   nichts, sie sortiert nur Listen.
+
+   'gast' (Community) steht mit Absicht NICHT in der Liste: Sie faellt
+   dadurch ans Ende (siehe ELSE unten), und das ist richtig -- die
+   Community ist keine Stufe im Team. */
+export const ROLLEN_REIHE = ["spicy", "admin", "hand", "manager", "scout", "creator", "modi"];

 /* DIE ROLLEN MIT EINER KACHEL AUF DER ANMELDESEITE.

@@ -132,10 +154,22 @@ const CREW_KACHEL = new Set(["admin", "hand", "modi", "gast"]);

 /* Als SQL-Ausdruck fuer ORDER BY. "ORDER BY rolle" waere alphabetisch
   (admin, creator, manager, scout, spicy) -- also fast genau falsch
-   herum. */
-export const ROLLEN_SORTIERUNG =
-  "CASE rolle WHEN 'spicy' THEN 0 WHEN 'admin' THEN 1 WHEN 'manager' THEN 2"
-  + " WHEN 'scout' THEN 3 WHEN 'creator' THEN 4 WHEN 'hand' THEN 5 ELSE 6 END";
+   herum.
+
+   ABGELEITET UND NICHT ABGESCHRIEBEN (11.09.2026). Bis heute stand die
+   Reihenfolge zweimal da: einmal als Liste, einmal als CASE mit
+   Zahlen. Beim Hochziehen der rechten Hand haetten beide geaendert
+   werden muessen -- und eine gepflegte Liste, die mit einer zweiten
+   uebereinstimmen muss, ist genau die Bauart, an der im Projekt schon
+   dreimal etwas verlorengegangen ist (zuletzt drei Spalten beim
+   Tabellenumbau). Eine Liste, die niemand pflegt, kann nicht veralten.
+
+   Das Wort "rolle" kommt hier genau EINMAL vor -- in "CASE rolle".
+   Darauf verlassen sich die Aufrufer, die es per .replace() auf
+   "p.rolle" umstellen; kein Rollenname enthaelt die Zeichenfolge. */
+export const ROLLEN_SORTIERUNG = "CASE rolle "
+  + ROLLEN_REIHE.map((r, i) => `WHEN '${r}' THEN ${i}`).join(" ")
+  + ` ELSE ${ROLLEN_REIHE.length} END`;

 /* LEITUNG = DogFather und Manager. Ein Manager darf alles, was
   DogFather darf -- mit genau zwei Ausnahmen, die in