metrics.py diff als CI-Schritt #125

New issue

Open

opened 2026-10-20 15:32:01 +00:00 by claude · 0 comments

claude commented

2026-10-20 15:32:01 +00:00

Collaborator

Anlass

Seit #120 (Teil 2, 01d0fad) gehört python3 .claude/skills/project-review/metrics.py diff zur technischen Abnahme vor dem Merge (CLAUDE.md, „Gemergt wird nach der technischen Abnahme“). Der Vergleich misst den Branch und seinen Abzweigpunkt von main mit derselben Fassung des Skripts, über alle Kriterien des Reviews. Er meldet, was neu ist und was sich verändert hat, und endet dann mit Exit-Code 1.

Heute ist das ein Handgriff der bauenden Session. Die CI prüft es nicht.

Vorschlag

Ein CI-Schritt in build.yml, der metrics.py diff --base origin/main fährt. Er braucht nur python3 und die Git-Historie bis zum Abzweigpunkt (fetch-depth: 0 oder genug Tiefe). Einen Build braucht er nicht, Laufzeit rund 6 s.

Warum nicht gleich

Bewusst verschoben (Olli, 20.10.2026): Zuerst soll sich bei zwei oder drei Merges zeigen, wie viele falsche Treffer der Vergleich liefert. Vor allem VERÄNDERT kann auch eine Verbesserung sein, zum Beispiel weniger doppelte Zeilen, und meldet trotzdem Exit-Code 1. Ein CI-Schritt, der dann rot wird, müsste das unterscheiden können.

Offene Fragen

Soll VERÄNDERT in der CI rot werden oder nur als Hinweis erscheinen?
Auf main selbst: gegen welchen Stand vergleichen (der Vorgänger-Commit)?

## Anlass Seit #120 (Teil 2, `01d0fad`) gehört `python3 .claude/skills/project-review/metrics.py diff` zur technischen Abnahme vor dem Merge (CLAUDE.md, „Gemergt wird nach der technischen Abnahme“). Der Vergleich misst den Branch und seinen Abzweigpunkt von `main` mit derselben Fassung des Skripts, über alle Kriterien des Reviews. Er meldet, was neu ist und was sich verändert hat, und endet dann mit Exit-Code 1. Heute ist das ein Handgriff der bauenden Session. Die CI prüft es nicht. ## Vorschlag Ein CI-Schritt in `build.yml`, der `metrics.py diff --base origin/main` fährt. Er braucht nur `python3` und die Git-Historie bis zum Abzweigpunkt (`fetch-depth: 0` oder genug Tiefe). Einen Build braucht er nicht, Laufzeit rund 6 s. ## Warum nicht gleich Bewusst verschoben (Olli, 20.10.2026): Zuerst soll sich bei zwei oder drei Merges zeigen, wie viele falsche Treffer der Vergleich liefert. Vor allem VERÄNDERT kann auch eine Verbesserung sein, zum Beispiel weniger doppelte Zeilen, und meldet trotzdem Exit-Code 1. Ein CI-Schritt, der dann rot wird, müsste das unterscheiden können. ## Offene Fragen - Soll VERÄNDERT in der CI rot werden oder nur als Hinweis erscheinen? - Auf `main` selbst: gegen welchen Stand vergleichen (der Vorgänger-Commit)?