Ein Hinweis auf eine Seite, die es hier nicht gibt, ist kein Hinweis

Filipe, mit dem Bildschirmfoto auf crew.: "wieso steht es noch da?" --
"1 Punkt im Start-Check braucht Handlung", mit einem Pfeil auf eine
Seite, die seit einer Stunde dort gesperrt ist.

MEIN FEHLER VON HEUTE MITTAG. Ich hatte die Seiten gesperrt und
ausdruecklich dazugeschrieben "nur die Seiten, nicht die
Schnittstellen" -- und dabei uebersehen, dass es eine DRITTE Stelle
gibt. Die Hinweise auf der Startseite sind keine Seite und keine
Schnittstelle, sondern eine Liste von VERWEISEN. Der Satz im Kommentar
klang vollstaendig und war es nicht.

Nachgesehen, wer sonst noch Seitenziele ausliefert, statt zu raten:

  workspace-hinweise.js   alle 16 Hinweise, ueber dazu()   -> behoben
  workspace-suche.js      jeder Treffer traegt ein Ziel    -> behoben
  workspace-push.js       Benachrichtigungen tragen eines  -> OFFEN

Die ersten beiden bekommen dieselbe Regel wie die Seiten, nicht eine
zweite: gehoertAufDieseAdresse(). Was hier keine Seite hat, bekommt
hier auch keinen Hinweis und keinen Treffer. Bei der Suche fallen leere
Gruppen mit weg (eine Ueberschrift ohne Treffer sieht aus, als waere
etwas kaputt) und `gesamt` wird danach abgeleitet, sonst nennt die Zahl
Treffer, die gar nicht dastehen.

DIE ADRESSE IST IMMER DIE ECHTE: Beide Stellen benutzen `req.sicht ||
req.person`. Der Sicht-Umschalter aendert, WESSEN Zahlen dastehen --
nicht, auf welcher Wand man steht. Das Haus kommt deshalb aus
req.person.

PUSH BLEIBT OFFEN, und zwar bewusst: Eine Benachrichtigung entsteht,
wenn niemand auf einer Adresse steht. Sie geht an ein GERAET, und das
oeffnet die Adresse, als die es installiert wurde. Das ist eine andere
Frage als diese hier, und eine halbe Antwort waere schlechter als
keine. Gehoert eigens angesehen.

Was die Gegenprobe gefunden hat -- an mir selbst, zweimal:

  Der erste Entwurf der Pruefung legte die Tabelle `startcheck` mit
  erfundenen Spalten selbst an. Die echte war laengst da, das
  IF NOT EXISTS schwieg, der INSERT scheiterte. Die Pruefung haette
  gemeldet, der Hinweis stehe nirgends -- richtig und wertlos.

  Und "kein Suchtreffer zeigt dorthin" stand ueber NULL Treffern. Jetzt
  wird erst nachgewiesen, dass die Suche denselben Menschen auf der
  Agenturadresse wirklich findet (1 Treffer -> profil.html).

  server/pruef-haus-seiten.mjs   34 Pruefungen (vorher 24), 0 Fehler
  pruef-sicht, pruef-haus-trennung 66, pruef-betreuung,
  pruef-manager-sicht 43 -- alle 0 Fehler

Co-Authored-By: Claude Opus 5 <[email protected]>
This commit is contained in:
2026-09-15 14:30:31 +02:00
co-authored by Claude Opus 5
parent 44a6b7cb83
commit 418b6211e2
33 changed files with 495 additions and 362 deletions
+83
View File
@@ -81,6 +81,22 @@ const jetzt = new Date().toISOString();
anlegen("Rieke", "hand", "CODE-HAND-0001");
anlegen("Frida", "modi", "CODE-MODI-0001");
anlegen("Mara", "manager", "CODE-MANA-0001");
anlegen("Sam", "creator", "CODE-CREA-0001");
const idSam = d.prepare("SELECT id FROM personen WHERE name = 'Sam'").get().id;
/* EIN ECHTER ANLASS FÜR DEN HINWEIS. Ohne ihn gäbe es ihn nirgends,
und „er steht nicht auf crew.“ wäre ein grüner Haken über der Null.
DIE TABELLE NICHT SELBST ANLEGEN: Der erste Entwurf schrieb hier
ein CREATE TABLE IF NOT EXISTS mit erfundenen Spalten hin. Die
echte Tabelle war längst da, das IF NOT EXISTS schwieg, und der
INSERT scheiterte an „no column named schluessel“ -- die Prüfung
hätte gemeldet, der Hinweis stehe nirgends, und das wäre richtig
und wertlos gewesen. Gefunden hat es die Gegenprobe. */
const spalten = d.prepare("PRAGMA table_info(startcheck)").all().map((x) => x.name);
if (!spalten.includes("bewertung")) throw new Error("startcheck sieht anders aus als erwartet");
d.prepare("INSERT INTO startcheck (creator_id, feld, punkt, bewertung, geaendert)"
+ " VALUES (?,?,?,?,?)")
.run(idSam, "technik", "licht", "handlung", jetzt);
d.close();
}
@@ -262,6 +278,73 @@ melde("=== 5. Der Durchgang über alle Seiten ===");
`für einen Modi mehr als für DogFather (${zu.modi.length})`);
}
/* =======================================================================
6. EIN HINWEIS AUF EINE SEITE, DIE ES HIER NICHT GIBT
Filipe, nach dem Ausliefern: „wieso steht es noch da?" -- „1 Punkt im
Start-Check braucht Handlung", auf crew., mit einem Pfeil auf eine
Seite, die dort gesperrt ist.
Das war die dritte Stelle: nicht Seite, nicht Schnittstelle, sondern
eine Liste von VERWEISEN. Sie bekommt jetzt dieselbe Regel.
======================================================================= */
melde("");
melde("=== 6. Hinweise und Suche ===");
{
const hinweise = async (host, keks) => {
const a = await ruf("/workspace/api/hinweise", host, "GET", { cookie: keks });
try { return JSON.parse(a.text).hinweise || []; } catch { return []; }
};
/* ZUERST DIE GEGENPROBE: Auf der Agenturadresse MUSS er dastehen.
Ohne sie beweist „er ist auf crew. weg" nur, dass es ihn nirgends
gibt. */
const drueben = await hinweise(HAUS, kDogi);
const dort = drueben.find((h) => h.art === "startcheck_handlung");
ok(!!dort, `auf der Agenturadresse steht der Hinweis (${dort?.text || "—"})`);
ok(dort?.ziel === "startcheck.html", `und zeigt auf den Start-Check (${dort?.ziel})`);
/* UND HIER IST DER GEMELDETE FEHLER. */
const hier = await hinweise(CREW, kDogiCrew);
ok(!hier.some((h) => h.art === "startcheck_handlung"),
"auf der Team-Adresse steht er nicht mehr");
ok(!hier.some((h) => /startcheck|scouting|content|profil|calls/.test(String(h.ziel || ""))),
`und kein einziger Hinweis zeigt dorthin (${hier.length} Hinweise insgesamt)`);
/* ABER DIE HINWEISE DES TEAMS BLEIBEN. Eine Regel, die alle
wegnimmt, wäre keine Lösung, sondern eine leere Startseite. */
ok(hier.every((h) => !h.ziel
|| ws.gehoertAufDieseAdresse({ rolle: "admin", haus: "crew" },
"/workspace/" + String(h.ziel).split("?")[0])),
"jeder verbliebene Hinweis führt auf eine Seite, die es hier gibt");
/* DIESELBE SORTE VERWEIS: die Suche. */
const suche = async (host, keks, q) => {
const a = await ruf(`/workspace/api/suche?q=${encodeURIComponent(q)}`, host, "GET",
{ cookie: keks });
try { return JSON.parse(a.text); } catch { return { gruppen: [] }; }
};
/* ZUERST WIEDER DIE GEGENPROBE. „Kein Treffer zeigt dorthin“ ist
wertlos, solange es gar keinen Treffer gibt -- genau der grüne
Haken über der Null, vor dem die Hausregel warnt. Also erst
nachweisen, dass die Suche diesen Menschen auf der Agenturadresse
WIRKLICH findet. */
const s0 = await suche(HAUS, kDogi, "Sam");
const zieleH = (s0.gruppen || []).flatMap((g) => g.treffer || []).map((t) => t.ziel || "");
ok(zieleH.length > 0, `auf der Agenturadresse findet die Suche Sam (${zieleH.length} Treffer)`);
ok(zieleH.some((z) => /profil.html/.test(z)),
`und ein Treffer führt auf sein Creator-Profil (${zieleH.join(", ")})`);
const s1 = await suche(CREW, kDogiCrew, "Sam");
const ziele = (s1.gruppen || []).flatMap((g) => g.treffer || []).map((t) => t.ziel || "");
ok(!ziele.some((z) => /profil|scouting|calls|content/.test(z)),
`auf der Team-Adresse verweist keiner dorthin (${ziele.length} Treffer)`);
ok(s1.gesamt === ziele.length,
`und die Zahl nennt nur, was wirklich dasteht (${s1.gesamt})`);
ok((s1.gruppen || []).every((g) => (g.treffer || []).length > 0),
"keine Überschrift ohne Treffer darunter");
}
/* ------------------------------------------------------------------- */
melde("");
melde(`${geprueft} Pruefungen, ${fehler} Fehler`);