fix(pruefungen): acknowledge the reworded wiki finding, record what the run taught
The first CI run after the acknowledgement list went in was still red, so the "all three green" line in #0104 was premature and is corrected rather than left standing. Two things it showed. Disabling the wiki mirror changed the finding's wording from "Mirror wiki: counterpart unreadable" to "wiki: no active push mirror", and since acknowledgements match on substrings, the entry no longer applied. Added. That is the cost of a list readable without special knowledge: fix or move a cause and the acknowledgement has to follow, otherwise it goes red - correctly. The second finding, management mirror divergence, is deliberately NOT acknowledged. It appears on any run starting minutes after a push because the mirror lags, and acknowledging it is tempting - but that same message is the only signal if a mirror ever really stops (#0028). It stays sharp; a run right after a push is briefly red. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
795639a1d8
commit
84e87238f6
@@ -90,3 +90,65 @@ hier festgehalten, weil sie dieselbe Klasse betreffen:
|
||||
Löschen des Zweigs auf **beiden** Remotes meldete der Job ihn weiter. Löschen
|
||||
wäre also gar kein Ausweg aus dem Dauer-Rot gewesen — live beobachtet an
|
||||
Pipeline 492.
|
||||
|
||||
## Umsetzung 2026-08-18 — Quittierungsliste gebaut (Weg 3)
|
||||
|
||||
Entscheidung sorb: Weg 3. Umgesetzt als `scripts/quittungen.py` plus
|
||||
`scripts/befund_quittungen.tsv`, eingehängt in **beide** geplanten Prüfungen.
|
||||
|
||||
**Wirkung, gemessen:** gruppenpruefung von 20 offenen Befunden auf **0** (20
|
||||
quittiert), stillstandspruefung von 5 auf **0**. `canonize_rotation` war bereits
|
||||
grün.
|
||||
|
||||
**Die drei Regeln gegen die eigene Blindstelle** — der naheliegende Einwand gegen
|
||||
Quittierungslisten überhaupt:
|
||||
|
||||
- Jede Zeile braucht eine **Frist**. Läuft sie ab, quittiert sie nicht mehr und
|
||||
meldet sich; der Befund zählt wieder.
|
||||
- **„Dauerhaft" ist nur als ADR-Verweis formulierbar.** Eine dauerhafte Ausnahme
|
||||
ohne Entscheidungs-Record ist laut AGENTS.md ohnehin ein Fehler — hier lässt sie
|
||||
sich nicht einmal hinschreiben.
|
||||
- Eine Zeile, auf die **kein Befund mehr passt**, meldet sich selbst zum Entfernen.
|
||||
|
||||
**Die #0053-Befunde sind einzeln nach SHA quittiert**, nicht per Sammelmuster. Ein
|
||||
Muster wie `: Echtzeit-Stempel` wäre eine Zeile statt siebzehn gewesen und hätte
|
||||
jeden künftigen Verstoß mitverschluckt — genau die Blindstelle, gegen die das hier
|
||||
gebaut ist. Nebeneffekt: Nach dem Historien-Rewrite passen sie auf nichts mehr und
|
||||
melden sich.
|
||||
|
||||
**Ein Konstruktionsfehler unterwegs**, hier notiert weil er sich wiederholen kann:
|
||||
Die erste Fassung hatte eine gemeinsame Liste ohne Geltungsbereich — jede Prüfung
|
||||
meldete daraufhin die Einträge der anderen als wirkungslos. Behoben durch die
|
||||
Spalte `pruefung`.
|
||||
|
||||
### Acceptance-Stand
|
||||
|
||||
| Kriterium | Stand |
|
||||
|---|---|
|
||||
| Alle drei Prüfungen laufen grün durch | ⏳ canonize ✅; die beiden management-Prüfungen erst nach dem Nachzug unten |
|
||||
| Ein *neu* eingeführter Befund färbt nachweislich rot | ✅ am Beispiel gezeigt, nicht abgeleitet |
|
||||
| Quittiertes bleibt sichtbar, mit Grund und Datum | ✅ |
|
||||
| Bedingung in AGENTS.md ergänzen | ⏳ **offen — braucht sorbs Zustimmung** |
|
||||
|
||||
Der letzte Punkt bleibt bewusst liegen: Änderungen an AGENTS.md werden nur mit
|
||||
sorb abgestimmt. Vorschlag für die Ergänzung neben „die rote Pipeline ist der
|
||||
Alarm": *„Das gilt nur, solange Grün der Normalzustand ist. Bekanntes wird
|
||||
quittiert (`scripts/befund_quittungen.tsv`), nicht toleriert — eine Prüfung, die
|
||||
dauerhaft rot steht, meldet nichts mehr."*
|
||||
|
||||
### Nachzug am selben Tag — zwei Befunde, die der erste Lauf zeigte
|
||||
|
||||
**1. Ein Befundtext ändert sich, wenn man die Ursache anfasst.** Das Abschalten des
|
||||
`wiki`-Mirrors (#0105) verwandelte `Mirror wiki: Gegenseite nicht lesbar` in
|
||||
`wiki: kein aktiver Push-Mirror` — die Quittung greift über Teilzeichenketten und
|
||||
passte nicht mehr. Nachgetragen. Das ist kein Fehler des Verfahrens, sondern seine
|
||||
Kehrseite: Textbindung ist der Preis für eine Liste, die man ohne Spezialwissen
|
||||
lesen kann. Wer eine Ursache behebt oder verschiebt, muss die Quittung nachziehen —
|
||||
sonst färbt es zu Recht rot.
|
||||
|
||||
**2. Ein flüchtiger Befund, der bewusst NICHT quittiert wird.**
|
||||
`Mirror management auseinander` erscheint bei jedem Lauf, der kurz nach einem Push
|
||||
startet, weil der Spiegel ein paar Minuten nachhinkt. Verlockend zu quittieren —
|
||||
und genau falsch: Dieselbe Meldung ist der einzige Hinweis, wenn ein Spiegel
|
||||
*wirklich* stehenbleibt (MIRROR-01, #0028). Sie bleibt scharf; ein Lauf unmittelbar
|
||||
nach einem Push ist eben kurz rot.
|
||||
|
||||
Reference in New Issue
Block a user