Files
dogfather-universe/server/workspace-suche.js
T
DogFatherGit 389d8c1213 Workspace: neue Rolle Manager, feste Rollenreihenfolge, echte Rollenwahl
DREI TEILE.

1) ROLLE "MANAGER"

Ein Manager darf alles, was DogFather darf -- mit genau zwei
Vorbehalten: Er kann keine Leitung ANLEGEN und an keiner Leitung etwas
AENDERN. Sonst koennte er sich einen zweiten Vollzugang schaffen oder
DogFather aussperren. "Nur DogFather hat alle endgueltigen Rechte"
heisst genau das.

Umgesetzt ueber istLeitung() an EINER Stelle statt 44 einzelner
Vergleiche auf "admin" im Server und 26 im Browser.

DATENBANK-UMSTELLUNG: CREATE TABLE IF NOT EXISTS fasst eine vorhandene
Tabelle nicht an -- die CHECK-Regel stand also weiter auf den alten drei
Rollen, und ein Manager waere daran gescheitert, obwohl der Code stimmt.
SQLite kann eine CHECK-Regel nicht aendern, also: neue Tabelle, Daten
hinueber, alte weg, umbenennen. Davor schreibt der Server eine
vollstaendige Sicherung (VACUUM INTO, in sich konsistent). Ohne
Sicherung wird NICHT umgestellt.

Geprueft nach der Umstellung: alle 13 Tabellen mit gleicher Zeilenzahl,
PRAGMA integrity_check ok, keine verwaisten Verweise. Die einzige
Abweichung war eine Sitzung mehr -- die eigene Anmeldung, die die
Umstellung ausgeloest hat.

2) EIN SICHERHEITSLOCH, DAS DER TEST GEFUNDEN HAT

Der erste Entwurf sicherte "Person anlegen" und "Person sperren" ab --
und liess "neuer Zugangscode" offen. Ein Manager konnte DogFather einen
neuen Code ausstellen, bekam ihn angezeigt und haette ihn damit aus
seinem eigenen Konto ausgesperrt. Im Test aufgefallen, weil ich den
negativen Fall durchgespielt habe.

Behoben nicht durch eine dritte Einzelpruefung, sondern durch eine
Schranke an JEDEM Weg mit einer :id. Der naechste Weg, der dazukommt,
ist damit automatisch mitgeschuetzt.

Nachgeprueft: Manager bekommt 403 beim Code-Erneuern und Sperren von
DogFather UND von sich selbst, darf aber Creator und Scouts verwalten.

3) FOLGEFEHLER DER MASSENERSETZUNG

Die Regel "niemals den letzten aktiven DogFather sperren" hatte durch
die Umstellung auf istLeitung() ploetzlich auch Manager blockiert --
gezaehlt werden aber nur DogFather-Zugaenge. Jetzt istDogFather().
Geprueft: DogFather kann einen Manager sperren, sich selbst nicht.

4) REIHENFOLGE UND ROLLENWAHL

Ueberall DogFather, Manager, Scout, Creator. "ORDER BY rolle" waere
alphabetisch gewesen (admin, creator, manager, scout) -- also fast genau
falsch herum. Jetzt ein gemeinsamer Sortierausdruck aus workspace.js.

Das Auswahlmenue fuer die Rolle ist weg. Es kam als weisses
Windows-Menue mitten in einer dunklen Oberflaeche und schnitt "Creator"
zu "Crea" ab -- gestalten laesst sich ein aufgeklapptes Systemmenue
nicht. Ersetzt durch vier sichtbare Schalter mit Symbol, Farbe je Rolle
und einer Zeile, was die Rolle bedeutet. Bei "Manager" gegen
"DogFather" ist das der Unterschied zwischen Raten und Wissen.

DogFather und Manager stehen dort nur zur Wahl, wenn DogFather selbst
davorsitzt -- ein Knopf, der immer scheitert, gehoert nicht hin.

Nebenbei: Das Namensfeld war auf eine von zwoelf Spalten gequetscht, weil
seine Umgebung keine .feld-Klasse trug. Alle Formulare daraufhin
durchsucht, keine weiteren Faelle.
2026-08-31 11:32:22 +02:00

271 lines
11 KiB
JavaScript

/* =====================================================================
workspace-suche.js — Suche über alle Bereiche (Konzept, Phase 4).
Zwei Dinge machen den Unterschied zwischen einer Suche und einer
brauchbaren Suche:
1. SIE DARF NICHTS FINDEN, WAS DIE SEITE VERBERGEN WUERDE. Eine Suche
ist die verlockendste Stelle fuer ein Datenleck: Man tippt einen
Namen und bekommt Treffer aus Bereichen, die man nie oeffnen
duerfte. Deshalb wird jede Quelle mit der Sichtbarkeitsregel ihres
eigenen Moduls abgefragt -- importiert, nicht abgeschrieben. Und
die management-internen Profilfelder (admin_notiz, plan_start,
naechster_review) werden gar nicht erst durchsucht, fuer niemanden
ausser dem Management.
2. SIE MUSS SAGEN, WO ETWAS STEHT. Ein Treffer ohne Textstelle und
ohne Weg dorthin zwingt zum Weitersuchen. Jeder Treffer traegt
deshalb einen Ausschnitt rund um die Fundstelle und ein Ziel.
Bewusst kein Volltextindex (FTS5). Die Datenmengen hier sind klein --
ein paar tausend Zeilen -- und LIKE braucht keinen zweiten Datenstand,
der irgendwann auseinanderlaeuft.
===================================================================== */
import express from "express";
import {
db, sitzungLesen, betreuteIds, istLeitung,
} from "./workspace.js";
import { sichtbar as sichtbarAufgaben } from "./workspace-aufgaben.js";
import { sichtbar as sichtbarTermine } from "./workspace-kalender.js";
import { sichtbar as sichtbarDateien } from "./workspace-dateien.js";
import { sichtbar as sichtbarEintraege } from "./workspace-bereiche.js";
export const sucheRouter = express.Router();
const MIN = 2;
const JE_BEREICH = 6;
const AUSSCHNITT = 110;
const BEREICHSNAME = {
live: "LIVE-Analyse", content: "Content", technik: "Technik",
community: "Community", schutz: "Schutz",
};
function angemeldet(req, res, next) {
const person = sitzungLesen(req);
if (!person) return res.status(401).json({ fehler: "nicht_angemeldet" });
req.person = person;
next();
}
sucheRouter.use("/workspace/api/suche", angemeldet);
/* LIKE-Sonderzeichen unschaedlich machen. Ohne das waere die Suche nach
"100%" eine Suche nach "100 gefolgt von irgendwas" und "a_b" faende
auch "axb" -- beides falsch und beides faellt erst spaet auf. */
function muster(text) {
return "%" + text.replace(/[\\%_]/g, (z) => "\\" + z) + "%";
}
/* Ausschnitt rund um die Fundstelle, damit man sieht, WARUM etwas
gefunden wurde -- nicht nur die ersten Zeichen des Feldes. */
function ausschnitt(text, suche) {
if (!text) return null;
const flach = String(text).replace(/\s+/g, " ").trim();
const wo = flach.toLowerCase().indexOf(suche.toLowerCase());
if (wo < 0) return flach.length > AUSSCHNITT ? flach.slice(0, AUSSCHNITT) + " …" : flach;
const von = Math.max(0, wo - 35);
const bis = Math.min(flach.length, wo + suche.length + 70);
return (von > 0 ? "… " : "") + flach.slice(von, bis) + (bis < flach.length ? " …" : "");
}
sucheRouter.get("/workspace/api/suche", (req, res) => {
try {
const person = req.person;
const q = String(req.query.q ?? "").trim().slice(0, 100);
if (q.length < MIN) {
return res.json({ frage: q, zu_kurz: true, gruppen: [], gesamt: 0 });
}
const m = muster(q);
const gruppen = [];
/* Eine Quelle abfragen. Faellt eine aus, bleibt die Suche im Rest
benutzbar -- eine leere Trefferliste waere schlimmer als eine
unvollstaendige. */
const quelle = (titel, sql, werte, bau) => {
try {
const zeilen = db().prepare(sql).all(...werte);
if (!zeilen.length) return;
gruppen.push({ titel, treffer: zeilen.map(bau) });
} catch (fehler) {
console.error(`[workspace] Suche (${titel}):`, fehler?.message);
}
};
/* ---------- Aufgaben ---------- */
const a = sichtbarAufgaben(person);
if (a) {
quelle("Aufgaben", `
SELECT a.id, a.titel, a.beschreibung, a.status, a.frist, p.name AS creator_name
FROM aufgaben a LEFT JOIN personen p ON p.id = a.creator_id
WHERE ${a.wo} AND (a.titel LIKE ? ESCAPE '\\' OR a.beschreibung LIKE ? ESCAPE '\\')
ORDER BY a.status = 'erledigt', a.id DESC LIMIT ${JE_BEREICH}`,
[...a.werte, m, m],
(z) => ({
titel: z.titel,
text: ausschnitt(z.beschreibung, q),
zusatz: [z.creator_name, z.status].filter(Boolean).join(" · "),
ziel: "aufgaben.html",
}));
}
/* ---------- Termine, Calls und Protokolle ---------- */
const t = sichtbarTermine(person);
if (t) {
quelle("Termine & Calls", `
SELECT t.id, t.titel, t.beschreibung, t.art, t.beginn, t.ort, p.name AS creator_name
FROM termine t LEFT JOIN personen p ON p.id = t.creator_id
WHERE ${t.wo} AND (t.titel LIKE ? ESCAPE '\\' OR t.beschreibung LIKE ? ESCAPE '\\'
OR t.ort LIKE ? ESCAPE '\\')
ORDER BY t.beginn DESC LIMIT ${JE_BEREICH}`,
[...t.werte, m, m, m],
(z) => ({
titel: z.titel,
text: ausschnitt(z.beschreibung, q),
zusatz: [z.beginn?.replace("T", " · "), z.creator_name].filter(Boolean).join(" · "),
ziel: z.art === "termin" ? "kalender.html" : "calls.html",
}));
/* Protokolle haengen an Terminen und erben deren Sichtbarkeit.
Hier steckt oft das, wonach man wirklich sucht: was besprochen
und was entschieden wurde. */
quelle("Gesprächsprotokolle", `
SELECT pr.id, pr.punkte, pr.entscheidungen, t.titel, t.beginn
FROM protokolle pr JOIN termine t ON t.id = pr.termin_id
WHERE ${t.wo} AND (pr.punkte LIKE ? ESCAPE '\\' OR pr.entscheidungen LIKE ? ESCAPE '\\')
ORDER BY t.beginn DESC LIMIT ${JE_BEREICH}`,
[...t.werte, m, m],
(z) => {
const treffer = (z.entscheidungen || "").toLowerCase().includes(q.toLowerCase())
? { feld: "Entscheidung", wert: z.entscheidungen }
: { feld: "Besprochen", wert: z.punkte };
return {
titel: z.titel,
text: ausschnitt(treffer.wert, q),
zusatz: `${treffer.feld} · ${(z.beginn || "").slice(0, 10).split("-").reverse().join(".")}`,
ziel: "calls.html",
};
});
}
/* ---------- Dateien ---------- */
const d = sichtbarDateien(person);
if (d) {
quelle("Dateien", `
SELECT d.id, d.name_original, d.notiz, d.status, p.name AS creator_name
FROM dateien d LEFT JOIN personen p ON p.id = d.creator_id
WHERE ${d.wo} AND (d.name_original LIKE ? ESCAPE '\\' OR d.notiz LIKE ? ESCAPE '\\')
ORDER BY d.id DESC LIMIT ${JE_BEREICH}`,
[...d.werte, m, m],
(z) => ({
titel: z.name_original,
text: ausschnitt(z.notiz, q),
zusatz: [z.creator_name, z.status].filter(Boolean).join(" · "),
ziel: "dateien.html",
}));
}
/* ---------- Betreuungsbereiche ---------- */
const e = sichtbarEintraege(person);
if (e) {
quelle("Betreuungsbereiche", `
SELECT e.id, e.bereich, e.titel, e.text, e.datum, e.status, p.name AS creator_name
FROM eintraege e LEFT JOIN personen p ON p.id = e.creator_id
WHERE ${e.wo} AND (e.titel LIKE ? ESCAPE '\\' OR e.text LIKE ? ESCAPE '\\')
ORDER BY e.datum DESC LIMIT ${JE_BEREICH}`,
[...e.werte, m, m],
(z) => ({
titel: z.titel,
text: ausschnitt(z.text, q),
zusatz: [BEREICHSNAME[z.bereich] || z.bereich, z.creator_name].filter(Boolean).join(" · "),
ziel: `bereich.html?b=${z.bereich}`,
}));
}
/* ---------- Scout-Pipeline ---------- */
if (istLeitung(person) || person.rolle === "scout") {
const nur = istLeitung(person) ? "1=1" : "l.scout_id = ?";
const werte = istLeitung(person) ? [] : [person.id];
quelle("Scout-Pipeline", `
SELECT l.id, l.name, l.plattform, l.handle, l.status, l.notizen, l.potenzial, l.aktivitaet
FROM leads l
WHERE ${nur} AND (l.name LIKE ? ESCAPE '\\' OR l.handle LIKE ? ESCAPE '\\'
OR l.notizen LIKE ? ESCAPE '\\' OR l.potenzial LIKE ? ESCAPE '\\'
OR l.aktivitaet LIKE ? ESCAPE '\\')
ORDER BY l.id DESC LIMIT ${JE_BEREICH}`,
[...werte, m, m, m, m, m],
(z) => ({
titel: z.name,
text: ausschnitt(z.notizen || z.potenzial || z.aktivitaet, q),
zusatz: [z.plattform, z.handle, z.status].filter(Boolean).join(" · "),
ziel: "scouting.html",
}));
}
/* ---------- Creator-Profile ----------
Nur die offenen Felder. admin_notiz, plan_start und
naechster_review werden NICHT durchsucht -- auch nicht fuer das
Management, denn ein Treffer daraus taucht sonst spaeter in einer
Ansicht auf, die diese Felder gar nicht zeigen darf. Wer die Notiz
lesen will, oeffnet das Profil. */
if (istLeitung(person) || person.rolle === "scout") {
let nur = "";
let werte = [];
if (person.rolle === "scout") {
const ids = betreuteIds(person);
if (!ids.length) nur = null;
else {
nur = ` AND p.id IN (${ids.map(() => "?").join(",")})`;
werte = ids;
}
}
if (nur !== null) {
quelle("Creator-Profile", `
SELECT p.id, p.name, f.nische, f.handles, f.live_zeiten, f.technik,
f.ziel_live, f.ziel_content, f.ziel_community, f.ziel_technik,
f.plan_prio1, f.plan_prio2, f.plan_prio3
FROM personen p LEFT JOIN profile f ON f.person_id = p.id
WHERE p.rolle = 'creator' AND p.aktiv = 1${nur}
AND (p.name LIKE ? ESCAPE '\\' OR f.nische LIKE ? ESCAPE '\\'
OR f.handles LIKE ? ESCAPE '\\' OR f.live_zeiten LIKE ? ESCAPE '\\'
OR f.technik LIKE ? ESCAPE '\\'
OR f.ziel_live LIKE ? ESCAPE '\\' OR f.ziel_content LIKE ? ESCAPE '\\'
OR f.ziel_community LIKE ? ESCAPE '\\' OR f.ziel_technik LIKE ? ESCAPE '\\'
OR f.plan_prio1 LIKE ? ESCAPE '\\' OR f.plan_prio2 LIKE ? ESCAPE '\\'
OR f.plan_prio3 LIKE ? ESCAPE '\\')
ORDER BY p.name LIMIT ${JE_BEREICH}`,
[...werte, ...Array(12).fill(m)],
(z) => {
/* Zeigen, WELCHES Feld getroffen hat -- sonst raetselt man,
warum ein Profil auftaucht. */
const felder = [
["Nische", z.nische], ["Handles", z.handles], ["LIVE-Zeiten", z.live_zeiten],
["Technik", z.technik], ["Ziel LIVE", z.ziel_live], ["Ziel Content", z.ziel_content],
["Ziel Community", z.ziel_community], ["Ziel Technik", z.ziel_technik],
["Priorität 1", z.plan_prio1], ["Priorität 2", z.plan_prio2],
["Priorität 3", z.plan_prio3],
];
const t2 = felder.find(([, w]) => w && w.toLowerCase().includes(q.toLowerCase()));
return {
titel: z.name,
text: t2 ? ausschnitt(t2[1], q) : null,
zusatz: t2 ? t2[0] : "Name",
ziel: "profil.html",
};
});
}
}
res.json({
frage: q,
zu_kurz: false,
gruppen,
gesamt: gruppen.reduce((s, g) => s + g.treffer.length, 0),
});
} catch (fehler) {
console.error("[workspace] Suche:", fehler?.message);
res.status(503).json({ fehler: "nicht_verfuegbar" });
}
});