Repository navigation
Conversation
Der Schritt-2-Anteil von OPEN-125. PR #96 hat das Werkzeug bereits auf main (#96, Squash 13d0514); was fehlt, ist die Doku, die einer Leserin sagt, dass die zwölf Suiten keine Behauptung sind, die bei einem neuen test/<name>.mjs manuell mitgepflegt werden muss. * README.md und README.de.md – Halbsatz direkt zwischen Aufmacher und Bulletliste unter '## Testing' / '## Testen'. * CONTRIBUTING.md – ein Satz nach der Prosa-Aufzählung unter '## Before opening a pull request'. * .multica/agents/_produkt.md – Halbsatz in der 'Prüfen'-Zeile der Befehlstabelle, im Anschluss an die Klammer-Suitenliste. * .multica/agents/entwickler.md – Klausel am Ende von Punkt 6 der Vorgehensliste. * AGENTS.md und CLAUDE.md (per cp byte-identisch) – drittes CI-Tor als Prosa-Absatz nach dem CHANGELOG-Tor, gleicher Aufbau wie OPEN-121. Mitnahme-Klausel: nur Halbsätze und der AGENTS/CLAUDE-Eintrag. Die fünf Listen selbst bleiben unangetastet – synchron seit OPEN-123 (Beleg: node scripts/check-suite-drift.js exit 0 auf main und auf diesem Branch). Kein Code, kein Demo-Diff (npm run build:demo sauber). Geprüft: - node scripts/check-suite-drift.js exit 0 - npm test grün (13 Suiten, suite-drift als erste) - AGENTS.md byte-identisch zu CLAUDE.md (md5sum 48efd15a352fe95284ba44fa03dc965b) - npm run build:demo ohne Diff in docs/demo und docs/demos - git status --short zeigt ausschließlich die sieben benannten Dateien - Branch startet von origin/main (13d0514), nicht vom OPEN-123-Stand Cross-Refs: OPEN-125 (Ursprung), OPEN-124 (PR #96, Werkzeug), OPEN-126 (dieser Schritt). Co-authored-by: multica-agent <github@multica.ai>
m-dohmen
left a comment
There was a problem hiding this comment.
OPEN-126 abgelehnt — Torprüfung 1–8 bis auf Tor 5 grün
PR #98, Branch feat/OPEN-126-suite-drift-docs (3dabc91), Base main @ 13d0514.
Tor-Status
| Tor | Status | Beleg |
|---|---|---|
| 1. CI grün auf PR-Head und main | grün | gh pr checks 98: build pass · gh run list --workflow=build --branch=main: success auf 13d0514 |
| 2. Datei bleibt geschlossen | grün | nur Markdown geändert, keine <script src=, keine <link href="http, keine Fonts, keine externen Bilder, kein Netzaufruf |
| 3. Nutzertext als Markup im DOM | grün | keine dangerouslySetInnerHTML, keine DOM-Änderung — Doku-Only |
| 4. Zusicherungen prüfen neues Verhalten | n/a | Doku-Only; das Verhalten selbst wurde mit PR #96 (#96 → 13d05146) eingeführt und durch test/suite-drift.mjs zugesichert |
| 5. Dokumentation nachgezogen | rot | siehe unten |
| 6. Eingecheckte Demos aktuell | grün | git diff --quiet -- docs/demo docs/demos clean; vor dem Merge lokal npm ci && npm run build && npm run build:demo reproduziert |
| 7. Commit lesbar | grün | Subject benennt das Ziel, Body sagt warum (PR #96 hat das Werkzeug auf main, was fehlt ist die Doku), listet was geändert wurde, hält die Mitnahme-Klausel ein |
| 8. Alle Kinder gemergt | grün | multica issue children OPEN-126 → keine Kinder |
Befund: Filename falsch in allen sieben Edits
Das tatsächliche Skript ist scripts/check-suite-drift.js (.js, nicht .mjs). Beleg:
git ls-tree origin/main scripts/check-suite-drift.*→check-suite-drift.js.github/workflows/build.yml:68→run: node scripts/check-suite-drift.jstest/suite-drift.mjs:19→import { validateSuiteDrift } from '../scripts/check-suite-drift.js'- Der PR-Body dieses PR (#98) selbst sagt korrekt
scripts.check-suite-drift.js - Die Issue-Beschreibung OPEN-126 sagt korrekt
check-suite-drift.js - Der Commit-Body dieses PR sagt korrekt
check-suite-drift.js
Aber jeder einzelne der sieben neuen Doku-Halbsätze verweist auf scripts/check-suite-drift.mjs (mit .mjs):
README.md:1111—scripts/check-suite-drift.mjs(soll.js)README.de.md:1174—scripts/check-suite-drift.mjs(soll.js)CONTRIBUTING.md:30—scripts/check-suite-drift.mjs(soll.js).multica/agents/_produkt.md:39—scripts/check-suite-drift.mjs(soll.js).multica/agents/entwickler.md:16—scripts/check-suite-drift.mjs(soll.js)AGENTS.md:850—node scripts/check-suite-drift.mjs(soll.js)CLAUDE.md:850—node scripts/check-suite-drift.mjs(soll.js, byte-identisch zu AGENTS.md halten)
Verifiziert: node scripts/check-suite-drift.mjs auf dem PR-Head wirft MODULE_NOT_FOUND. Wer den Halbsätzen folgt, landet in einem Fehler statt beim Skript.
Ironischer Bruch: dieser PR dokumentiert einen Wächter, dessen einziger Zweck es ist, Doku und Code synchron zu halten — und führt ihn selbst asynchron ein. Genau der Fehler, vor dem der neue Gate Leserinnen warnen soll.
Erwartete Korrektur
Pro Datei genau ein Treffer: check-suite-drift.mjs → check-suite-drift.js. Nach dem Edit:
git grep -n "check-suite-drift.mjs" -- README.md README.de.md CONTRIBUTING.md AGENTS.md CLAUDE.md .multica/agents/_produkt.md .multica/agents/entwickler.mdmuss leer sein.git grep -n "check-suite-drift.js" -- README.md README.de.md CONTRIBUTING.md AGENTS.md CLAUDE.md .multica/agents/_produkt.md .multica/agents/entwickler.mdmuss sieben Treffer liefern.diff -q AGENTS.md CLAUDE.mdmuss weiterhin leer sein.node scripts/check-suite-drift.jsexit 0.npm testgrün.
Kein Demo-Rebuild nötig, keine Änderung an den fünf Listen, kein Code.
@Doku-Pfleger — Übergabe zurück in deine Hand. Sobald die sieben Treffer korrigiert sind und CI grün bleibt, gerne erneut an mich.
Issue OPEN-126 stelle ich auf in_progress, bis die Korrektur gemergt ist.
Bewusst offen
Nichts. Der Rest des PRs ist sauber — Halbsatz-Platzierung, Mitnahme-Klausel, Section-Heading-Annahme und Hauptstatus sind in Ordnung.
Die erste Fassung dieses PRs (#98) hat einen eigenen Wortlaut verwendet; der Trigger-Kommentar hat eine konkretere Textvorlage geliefert, die Reihenfolge-und-Vollstaendigkeit und das Warum (neue Suiten erscheinen erst in der Doku, wenn sie auch im Skript stehen) explizit macht. Diese Fassung uebernimmt die Vorlage. * README.md / README.de.md - Halbsatz zwischen Aufmacher und Bulletliste, exakt die Vorlage (englisch bzw. deutsch). * CONTRIBUTING.md - order entfaellt (Prosa, nur Vollstaendigkeit), Vorlage sinngemaess uebernommen. * .multica/agents/_produkt.md - Halbsatz als eigene Zeile direkt unter der Befehlstabelle (die Klammer-Suitenliste waere sonst zu lang). * .multica/agents/entwickler.md - nur Anzahl, Vorlage sinngemaess. * AGENTS.md / CLAUDE.md (cp byte-identisch) - Skript-Verweis von .mjs auf .js korrigiert, die WHY-Klausel am Ende des dritten CI-Tor- Absatzes ergaenzt. Mitnahme-Klausel weiter: nur Halbsaetze und der AGENTS/CLAUDE-Eintrag. Listen selbst unangetastet (Beleg: node scripts/check-suite-drift.js exit 0 auf main und auf diesem Branch). Kein Code, kein Demo-Diff. Geprueft auf PR-Head: - node scripts/check-suite-drift.js exit 0 - npm test gruen (13 Suiten, suite-drift als erste) - AGENTS.md byte-identisch zu CLAUDE.md (md5 246d6d1d9bc93c961677a4443aebf423) - npm run build:demo ohne Diff in docs/demo und docs/demos - git status --short zeigt ausschliesslich die sieben benannten Dateien Co-authored-by: multica-agent <github@multica.ai>
Schritt 2 von OPEN-125 nach Merge von PR #96 (Squash
13d05146). DasWerkzeug
scripts/check-suite-drift.jsist bereits aufmain; wasfehlt, ist die Doku, die einer Leserin sagt, dass die zwölf Suiten
keine Behauptung sind, die bei einem neuen
test/<name>.mjsmanuellmitgepflegt werden muss.
Was geändert wurde
Sieben Edits, je ein Halbsatz an der Stelle, an der der Drift-Checker
andockt:
README.md## TestingREADME.de.md## TestenCONTRIBUTING.md## Before opening a pull request.multica/agents/_produkt.mdPrüfen-Zeile der Befehlstabelle, im Anschluss an die Klammer-Suitenliste.multica/agents/entwickler.mdAGENTS.mdCLAUDE.mdcp AGENTS.md CLAUDE.mdbyte-identisch zu AGENTS.mdMitnahme-Klausel
Nur Halbsätze und AGENTS/CLAUDE-Eintrag. Die fünf Listen selbst
bleiben unangetastet — synchron seit OPEN-123. Beleg:
node scripts/check-suite-drift.jsexit 0 aufmainund auf diesemBranch. Kein Code, kein Demo-Rebuild.
Wie geprüft wurde
Auf dem PR-Head:
node scripts/check-suite-drift.jsexit 0npm testgrün (13 Suiten,suite-driftals erste):suite-drift · prompts-metrics · actions-delete-guard · timezone · domain-swap-crash · views · board · charts · changelog · smoke · multi-entity · demos · timezone-examplesdiff -q AGENTS.md CLAUDE.mdundmd5sum AGENTS.md CLAUDE.mdstimmen (md5:
48efd15a352fe95284ba44fa03dc965b)npm run build:demoohne Diff indocs/demounddocs/demosgit status --shortzeigt ausschließlich die sieben benanntenDateien
origin/main(13d05146), nicht vom OPEN-123-StandBewusst offen
Nichts. Sollte das Review eine andere Platzierung bevorzugen (zum
Beispiel den Halbsatz ans Listenende statt zwischen Aufmacher und
Liste), ist das ein Einzeiler.
Cross-Refs