DogFatherGit/dogfather-universe
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Talente: der Uebergang, an dem die meisten verlorengehen

Browse Source 
Filipe: "ich will dass du die 2 kategorien perfektionnierst. ich will
die noch viel besser, perfektionniert, viel geiler und krasser. es
soll so einfach wie moeglich sein fuer jeden."

Die AUSWAHL war der gut gebaute Teil dieser Seite: 21 Merkmale, sechs
Warnzeichen, ein Trichter mit Standzeiten. Die Stelle, an der Teams
tatsaechlich Leute verlieren, liegt dahinter -- zwischen dem "ja" und
der ersten echten Schicht. Wer zusagt und danach zwei Wochen nichts
hoert, hat innerlich schon abgesagt, und auf der Karte steht weiter
"angesprochen".

In der Beschreibung der Stufe "Probe" stand seit dem ersten Tag
"fester Buddy". Ein Feld dafuer gab es nicht. Ein Versprechen im Text
ist keine Eigenschaft des Systems -- erst eine Regel, die NEIN sagen
kann, ist eine.

Ab jetzt:

  - Auf die Probe kommt niemand ohne einen Namen und ein Datum. Die
    Absage sagt, welches von beiden fehlt, und die Karte bleibt dabei
    stehen, wo sie war.
  - Die Karte zeigt beides: wer einarbeitet, wann die erste Schicht
    ist -- in einem Satz, den man laut vorlesen kann ("Erste Schicht
    ist morgen."). Ein Termin weiter als 14 Tage ist erlaubt, wird
    aber benannt.
  - Drei Dinge fuer den ersten Tag stehen auf der Probekarte: was
    gilt, was du darfst, wen du fragst. Keine Haken -- der Buddy soll
    sie lesen, nicht abarbeiten.
  - Buddy und Termin lassen sich spaeter aendern, OHNE die Standzeit
    zurueckzusetzen: Sie ist die Auskunft ueber uns, nicht ueber den
    Kandidaten.
  - Wer geht, ist kein Buddy mehr -- stillgelegt wie geloescht. Die
    Karte sagt dann "bitte neu bestimmen" und ist am Rand markiert.

Recherche (Quellen im Kopf von workspace-talent-punkte.js): Discord
nennt Buddy und Mentoring als die beiden Trainingswege, die
funktionieren; aus der Freiwilligenarbeit kommt dieselbe Aussage mit
Zahlen. Und: Am ersten Tag braucht jemand drei Dinge, nicht dreissig.

Was die Pruefung gefunden hat: buddy_weg fragte "buddy_id gesetzt UND
Person weg" -- damit war der zweite Weg blind. Wird ein Mensch
geloescht, setzt ON DELETE SET NULL die Spalte auf NULL, die Karte sah
unauffaellig aus, und eine laufende Probe stand ohne jeden Buddy da.

  server/pruef-uebergang.mjs   61 Pruefungen, 0 Fehler (Port 4420)
  pruef-nachwuchs 123, pruef-entwicklung 43, pruef-css-klassen -- 0 Fehler

Co-Authored-By: Claude Opus 5 <[email protected]>
This commit is contained in:
DogFatherGit
2026-09-15 13:55:26 +02:00
co-authored by Claude Opus 5
parent d4730deb30
commit b85740dc20
35 changed files with 1355 additions and 367 deletions
Download Patch File Download Diff File Expand all files Collapse all files
server/workspace-entwicklung.js
+199 -5
View File
@@ -70,6 +70,7 @@ import {
} from "./workspace-entwicklungs-punkte.js";
import {
TALENT_STUFEN, TALENT_GRUPPEN, TALENT_MERKMALE, TALENT_WARNZEICHEN,
SCHICHT_WEIT_TAGE, SCHICHT_MAX_TAGE, ERSTER_TAG,
TALENT_NACH_SCHLUESSEL, talentStufe, talentNaechste, ansprechtext,
} from "./workspace-talent-punkte.js";
@@ -92,6 +93,15 @@ export const VERBLASST_NACH_TAGEN = 90;
automatisch eine Personalakte bekommen. */
const KARTEN_ROLLEN = ["hand", "modi"];
/** Wer einen Neuen einarbeiten kann.
*
* DIESELBEN WIE BEI DER KARTE, PLUS DOGFATHER. Er steht nicht ueber
* dem Team, er gehoert dazu -- und wer eine Probe begleitet, macht
* dieselbe Arbeit wie alle anderen. Ein Manager oder Scout dagegen
* arbeitet im anderen Haus und kann niemanden durch eine Schicht
* begleiten, die er selbst nicht macht. */
const BUDDY_ROLLEN = ["admin", "hand", "modi"];
/* =====================================================================
DIE TABELLEN
===================================================================== */
@@ -155,6 +165,30 @@ export function tabellen() {
notiz TEXT
);
`);
/* DIE BEIDEN SPALTEN DES ÜBERGANGS (15.09.2026).
Sie stehen hier und nicht in der Umstellungsliste von workspace.js:
Die ueberspringt eine Tabelle stillschweigend, wenn es sie noch
nicht gibt -- und talent_stufe entsteht erst beim ersten Aufruf
dieser Seite, nicht beim Start. Hier, zwei Zeilen nach dem CREATE,
ist sie sicher da.
ON DELETE SET NULL beim Buddy: Wer geht, nimmt die Probe eines
anderen nicht mit. Die Karte verliert den Namen und faellt damit
genau dort auf, wo jemand einen neuen Buddy bestimmen muss. */
for (const [spalte, typ] of [
["buddy_id", "INTEGER REFERENCES personen(id) ON DELETE SET NULL"],
["erste_schicht", "TEXT"],
]) {
try {
const da = db().prepare("PRAGMA table_info(talent_stufe)").all().map((s) => s.name);
if (!da.includes(spalte)) {
db().exec(`ALTER TABLE talent_stufe ADD COLUMN ${spalte} ${typ}`);
console.log(`[talente] Spalte '${spalte}' ergaenzt.`);
}
} catch (f) { console.error(`[talente] Spalte '${spalte}':`, f?.message); }
}
bereit = true;
}
@@ -618,9 +652,72 @@ entwicklungRouter.get("/workspace/api/entwicklung/ampel", nurLeitung, (req, res)
TALENTE
===================================================================== */
/** Ein Tagesdatum in Ortszeit, YYYY-MM-DD.
*
* NICHT toISOString().slice(0,10) -- das rechnet in UTC, und zwischen
* Mitternacht und zwei Uhr morgens liefert es in Deutschland den
* Vortag. Genau diese Zeile steht als offener Befund in pruef-struktur. */
function tagIn(d = new Date()) {
return new Intl.DateTimeFormat("sv-SE", { timeZone: "Europe/Berlin" }).format(d);
}
/** Wie viele Tage liegen zwischen heute und einem Tagesdatum?
* Negativ heisst: vorbei. */
function tageBis(tag) {
if (!tag) return null;
const heute = Date.parse(tagIn() + "T12:00:00Z");
const ziel = Date.parse(tag + "T12:00:00Z");
if (!Number.isFinite(ziel)) return null;
return Math.round((ziel - heute) / 86400000);
}
/** Der Übergang: ein Buddy und ein Datum — mehr braucht die Probe nicht.
*
* ER WIRD ABGELEITET, NICHT GESPEICHERT. Ein Satz wie „die erste
* Schicht war vor drei Tagen" in der Datenbank waere ab morgen falsch.
* Aus zwei Feldern wird hier jedes Mal neu gerechnet, was gerade gilt. */
function uebergangVon(stand) {
if (!stand?.erste_schicht && !stand?.buddy_id) return null;
const buddy = stand.buddy_id
? db().prepare("SELECT name, rolle, aktiv FROM personen WHERE id = ?").get(stand.buddy_id)
: null;
const bis = tageBis(stand.erste_schicht);
return {
buddy_id: stand.buddy_id || null,
buddy_name: buddy?.name || null,
/* WER GEGANGEN IST, IST KEIN BUDDY MEHR -- und zwar auf beiden
Wegen, auf denen jemand gehen kann.
Der erste Entwurf fragte "`buddy_id` gesetzt UND die Person weg".
Damit war der zweite Weg blind: Wird ein Mensch GELOESCHT, setzt
ON DELETE SET NULL die Spalte auf NULL -- die Bedingung war
falsch, die Karte sah unauffaellig aus, und eine laufende Probe
stand ohne jeden Buddy da. Gefunden hat das die Pruefung, nicht
das Lesen; sie loescht die Person wirklich.
Richtig ist die Frage, auf die es ankommt: Gibt es jemanden, der
gerade einarbeitet? */
buddy_weg: !buddy?.aktiv,
erste_schicht: stand.erste_schicht || null,
bis_schicht: bis,
/* Drei Lagen, drei Saetze -- derselbe Grund wie bei der Ampel:
ein Satz fuer alle ergibt bei der Haelfte falsches Deutsch. */
satz: !stand.erste_schicht ? null
: bis > SCHICHT_WEIT_TAGE
? `Erste Schicht in ${bis} Tagen. Das ist weit weg – bis dahin hört man nichts `
+ "voneinander, und genau da springen die meisten ab."
: bis > 1 ? `Erste Schicht in ${bis} Tagen.`
: bis === 1 ? "Erste Schicht ist morgen."
: bis === 0 ? "Erste Schicht ist heute."
: bis === -1 ? "Erste Schicht war gestern."
: `Erste Schicht war vor ${-bis} Tagen.`,
};
}
/** Die Karte eines Kandidaten -- Stufe, Standzeit, was zu tun ist. */
function talentKarte(e, merkmale) {
const stand = db().prepare("SELECT stufe, seit, notiz FROM talent_stufe WHERE eintrag_id = ?")
const stand = db().prepare(
"SELECT stufe, seit, notiz, buddy_id, erste_schicht FROM talent_stufe WHERE eintrag_id = ?")
.get(e.id);
const stufe = talentStufe(stand?.stufe) || TALENT_STUFEN[0];
const seit = stand?.seit || e.erstellt;
@@ -640,6 +737,10 @@ function talentKarte(e, merkmale) {
merkmale: gesetzt.filter((s) => !TALENT_NACH_SCHLUESSEL.get(s)?.warnung),
warnzeichen: warn,
notiz: stand?.notiz || null,
/* DER ÜBERGANG. Er steht an der Karte und nicht in einem eigenen
Abruf: Er aendert sich mit jedem Tag, und was man extra holen
muss, ist beim Holen schon veraltet. */
uebergang: uebergangVon(stand),
/* DER VORLAGENTEXT -- gebaut aus den Merkmalen, die wirklich
angeklickt wurden. Er kommt MIT der Karte und nicht ueber einen
eigenen Weg: Er aendert sich mit jedem Klick, und ein Text, den
@@ -665,6 +766,15 @@ entwicklungRouter.get("/workspace/api/talente/lage", (req, res) => {
stufen: TALENT_STUFEN,
gruppen: TALENT_GRUPPEN.map((g) => ({ ...g, merkmale: TALENT_MERKMALE[g.schluessel] || [] })),
warnzeichen: TALENT_WARNZEICHEN,
/* WER BUDDY SEIN KANN -- in der Reihenfolge des Hauses, nicht in
der der Datenbank. Sie kommt mit der Lage, damit die Auswahl
keine zweite Liste braucht, die man pflegen muesste. */
buddys: db().prepare(
`SELECT id, name, rolle FROM personen
WHERE aktiv = 1 AND rolle IN (${BUDDY_ROLLEN.map(() => "?").join(", ")})
ORDER BY ${ROLLEN_SORTIERUNG}, name COLLATE NOCASE`).all(...BUDDY_ROLLEN),
erster_tag: ERSTER_TAG,
schicht_weit_tage: SCHICHT_WEIT_TAGE,
kandidaten: karten,
/* Wie viele auf jeder Stufe liegen -- der Trichter als Zahlen.
Abgeleitet, damit er nicht danebenliegen kann. */
@@ -713,6 +823,67 @@ entwicklungRouter.put("/workspace/api/talente/:id/merkmal/:schluessel", gleicheH
}
});
/** Buddy und Datum prüfen. Gibt `{ fehlt: [...] }` oder `{ buddyId, schicht }`.
*
* EINE STELLE, ZWEI AUFRUFER: der Stufenwechsel auf „Probe" und das
* spaetere Aendern. Zwei Abschriften derselben Regeln waeren zwei
* Regeln, und die zweite ist immer die, die etwas durchlaesst. */
function uebergangPruefen(koerper, { pflicht }) {
const fehlt = [];
const buddyId = nummer(koerper?.buddy_id);
const b = buddyId
? db().prepare("SELECT id, aktiv, rolle FROM personen WHERE id = ?").get(buddyId) : null;
if (buddyId && (!b || !b.aktiv || !BUDDY_ROLLEN.includes(b.rolle))) fehlt.push("buddy_unbekannt");
else if (!buddyId && pflicht) fehlt.push("buddy");
const roh = String(koerper?.erste_schicht || "").trim();
if (roh) {
if (!/^\d{4}-\d{2}-\d{2}$/.test(roh)) fehlt.push("datum_unlesbar");
else {
const bis = tageBis(roh);
/* VERGANGENHEIT IST KEIN TERMIN. Wer eine Schicht nachtraegt, die
schon war, meint etwas anderes -- naemlich, dass die Probe
laeuft. Dann gehoert das Datum von damals hinein, nicht heute:
deshalb sind bis zu sieben Tage rueckwaerts erlaubt. */
if (bis === null) fehlt.push("datum_unlesbar");
else if (bis < -7) fehlt.push("datum_zu_alt");
else if (bis > SCHICHT_MAX_TAGE) fehlt.push("datum_zu_weit");
}
} else if (pflicht) fehlt.push("erste_schicht");
return fehlt.length ? { fehlt } : { buddyId: buddyId || null, schicht: roh || null };
}
/** Buddy oder Datum ändern, ohne die Stufe anzufassen.
*
* Ein eigener Weg, weil es ein eigener Vorgang ist: Ein Buddy hoert
* auf, ein Termin verschiebt sich. Wer dafuer die Stufe neu setzen
* muesste, setzt dabei `seit` zurueck -- und die Standzeit, die genau
* diese Verschleppung sichtbar macht, faengt wieder bei null an. */
entwicklungRouter.put("/workspace/api/talente/:id/uebergang", gleicheHerkunft, (req, res) => {
try {
const id = nummer(req.params.id);
const e = id && db().prepare("SELECT id, bereich, titel FROM eintraege WHERE id = ?").get(id);
if (!e || e.bereich !== "talente") return res.status(404).json({ fehler: "nicht_gefunden" });
const stand = db().prepare("SELECT stufe FROM talent_stufe WHERE eintrag_id = ?").get(id);
if (!stand) return res.status(404).json({ fehler: "nicht_gefunden" });
const p = uebergangPruefen(req.body, { pflicht: stand.stufe === "probe" });
if (p.fehlt) return res.status(400).json({ fehler: "uebergang_offen", fehlt: p.fehlt });
db().prepare("UPDATE talent_stufe SET buddy_id = ?, erste_schicht = ? WHERE eintrag_id = ?")
.run(p.buddyId, p.schicht, id);
protokolliere("talent_uebergang", {
personId: req.person.id, ip: echteIp(req),
detail: `${e.titel}: Buddy/erste Schicht geaendert`.slice(0, 120),
});
res.json({ ok: true });
} catch (f) {
console.error("[talente] Uebergang:", f?.message);
res.status(503).json({ fehler: "nicht_verfuegbar" });
}
});
entwicklungRouter.put("/workspace/api/talente/:id/stufe", gleicheHerkunft, (req, res) => {
try {
const id = nummer(req.params.id);
@@ -722,12 +893,35 @@ entwicklungRouter.put("/workspace/api/talente/:id/stufe", gleicheHerkunft, (req,
if (!e || e.bereich !== "talente") return res.status(404).json({ fehler: "nicht_gefunden" });
if (!talentStufe(stufe)) return res.status(400).json({ fehler: "unbekannte_stufe" });
db().prepare(`INSERT INTO talent_stufe (eintrag_id, stufe, seit, von_id, notiz)
VALUES (?,?,?,?,?)
/* AUF DIE PROBE KOMMT NIEMAND OHNE BUDDY UND DATUM.
In der Stufenbeschreibung stand „fester Buddy" seit dem ersten
Tag -- ein Feld dafuer gab es nicht. Ein Versprechen ohne Feld
ist eines, an das sich nach zwei Wochen niemand erinnert, und
genau an dieser Stelle verlieren Teams die Leute, die sie gerade
gewonnen haben.
EINE ABSAGE MIT GRUND, nicht ein „ging nicht": Die Oberflaeche
fragt beides vorher ab, aber die Regel gehoert hierher. Was nur
im Browser steht, gilt nur fuer den, der den Browser benutzt. */
const p = stufe === "probe"
? uebergangPruefen(req.body, { pflicht: true })
: uebergangPruefen(req.body, { pflicht: false });
if (p.fehlt) return res.status(400).json({ fehler: "uebergang_offen", fehlt: p.fehlt });
/* COALESCE, NICHT ÜBERSCHREIBEN: Wer aus der Probe ins Team geht,
schickt Buddy und Datum nicht noch einmal mit -- und die beiden
sollen bleiben, wo sie sind. Sie sind die Geschichte dieser
Probe, nicht ein Formularfeld. */
db().prepare(`INSERT INTO talent_stufe
(eintrag_id, stufe, seit, von_id, notiz, buddy_id, erste_schicht)
VALUES (?,?,?,?,?,?,?)
ON CONFLICT(eintrag_id) DO UPDATE SET
stufe = excluded.stufe, seit = excluded.seit,
von_id = excluded.von_id, notiz = excluded.notiz`)
.run(id, stufe, jetzt(), req.person.id, notiz || null);
von_id = excluded.von_id, notiz = excluded.notiz,
buddy_id = COALESCE(excluded.buddy_id, talent_stufe.buddy_id),
erste_schicht = COALESCE(excluded.erste_schicht, talent_stufe.erste_schicht)`)
.run(id, stufe, jetzt(), req.person.id, notiz || null, p.buddyId, p.schicht);
protokolliere("talent_stufe", {
personId: req.person.id, ip: echteIp(req),