Workflow-Probleme aus End-to-End Test (area-basic)

[MarkusNeusinger]

Gefundene Probleme beim Test von Issue #99 (area-basic)

Bei der End-to-End-Testung des Plot-Generation Workflows wurden folgende Probleme identifiziert:

1. Spec-Datei Konflikte (Kritisch)

Problem: Alle 8 Library-Branches versuchen specs/{spec-id}.md zu erstellen/ändern. Wenn PRs sequenziell mergen, entstehen Konflikte für alle verbleibenden Branches.

Symptom: Nach jedem PR-Merge müssen alle anderen Branches rebased werden.

Lösung:

• Option A: Orchestrator (gen-new-plot.yml) erstellt Spec-Datei ZUERST auf main, bevor Library-Branches gestartet werden
• Option B: Library-Branches fügen nur Implementation hinzu, überspringen Spec-Änderungen wenn Datei existiert

---

2. Auto-Merge nicht getriggert durch Workflow-Labels (Kritisch)

Problem: Labels die mit GITHUB_TOKEN innerhalb eines Workflows hinzugefügt werden, triggern KEINE anderen Workflows (GitHub Actions Limitation).

Symptom: bot-ai-review.yml fügt ai-approved Label hinzu, aber bot-auto-merge.yml wird nicht getriggert.

Lösung:

• Implementiert: workflow_dispatch Trigger in bot-auto-merge.yml
• Implementiert: bot-ai-review.yml triggert auto-merge via gh workflow run
• Status: Fix vorbereitet (lokal), noch nicht gepusht

---

3. Code-Formatierung fehlt in Generation (Mittel)

Problem: Claude generiert Code der nicht mit ruff format formatiert ist.

Symptom: Linting-Check schlägt fehl mit "Would reformat: plots/..."

Lösung: In gen-library-impl.yml nach Code-Generierung uv run ruff format ausführen vor dem Commit.

---

4. Altair Retry-Loop nicht aktiviert (Mittel)

Problem: Altair wurde nach erstem AI-Review als ai-rejected markiert, aber kein Retry-Versuch gestartet.

Beobachtung:

• PR feat(altair): implement area-basic #108 hat nur ai-rejected Label, keine ai-attempt-* Labels
• Sub-Issue [area-basic] altair implementation #104 hat ai-rejected aber keine Attempt-Kommentare

Zu untersuchen:

• Wird gen-update-plot.yml bei ai-rejected getriggert?
• Fehlen die Attempt-Labels für das Counting?

---

5. Post-Merge Summary Job läuft nicht (Niedrig)

Problem: Der post-merge-summary Job in bot-auto-merge.yml läuft nicht nach dem Merge.

Ursache: Der Job prüft auf action == "closed", aber bei --auto Merge kommt kein neues Event.

Symptom: Sub-Issues werden nicht automatisch geschlossen, Main-Issue nicht als completed markiert.

---

6. Highcharts noch offen (Info)

Status: PR #115 ist noch offen, noch im Test/Review-Prozess.

---

Aktueller Status (area-basic #99)

Library	PR	Status
matplotlib	#109	✅ Merged
seaborn	#110	✅ Merged
plotly	#111	✅ Merged
bokeh	#113	✅ Merged
plotnine	#114	✅ Merged
pygal	#112	✅ Merged
altair	#108	❌ ai-rejected (kein Retry)
highcharts	#115	⏳ Offen

---

Nächste Schritte

1. Fix für Spec-Datei Konflikte implementieren
2. Auto-Merge workflow_dispatch Fix pushen und testen
3. Code-Formatierung in gen-library-impl.yml hinzufügen
4. Retry-Loop für ai-rejected PRs untersuchen und fixen
5. Post-Merge Summary Logik überarbeiten

---

Erstellt während End-to-End Test Session

[MarkusNeusinger]

Zusätzliche Erkenntnisse

7. Repair-Loop wird nicht getriggert (Kritisch)

Problem: gen-update-plot.yml triggert auf pull_request: types: [labeled] wenn ai-rejected hinzugefügt wird. Aber das ai-rejected Label wird von bot-ai-review.yml (via Claude) mit GITHUB_TOKEN hinzugefügt.

Gleiche Ursache wie Problem #2: Labels die mit GITHUB_TOKEN hinzugefügt werden, triggern keine anderen Workflows.

Lösung: In bot-ai-review.yml nach dem Hinzufügen von ai-rejected auch gen-update-plot.yml via workflow_dispatch triggern.

---

8. Sub-Issue #100 (matplotlib) nicht geschlossen

Problem: PR #109 (matplotlib) wurde gemerged, aber Sub-Issue #100 ist noch OPEN.

Status der Sub-Issues:

Issue	Library	Status	Labels
#100	matplotlib	OPEN ❌	ai-approved
#101	seaborn	CLOSED ✅	merged
#102	plotly	CLOSED ✅	merged
#103	bokeh	CLOSED ✅	merged
#104	altair	OPEN	ai-rejected
#105	plotnine	CLOSED ✅	merged
#106	pygal	CLOSED ✅	merged
#107	highcharts	OPEN	testing

Beobachtung: Einige Sub-Issues wurden korrekt geschlossen (post-merge-summary lief), aber #100 nicht. Möglicherweise ein Timing-Problem beim ersten PR.

---

Zusammenfassung der Label-Trigger-Probleme

Alle diese Workflows haben das gleiche Problem - sie reagieren auf Labels, die von anderen Workflows hinzugefügt werden:

1. bot-auto-merge.yml - reagiert auf ai-approved
2. gen-update-plot.yml - reagiert auf ai-rejected

Universelle Lösung: Statt auf Label-Events zu warten, sollte der auslösende Workflow (bot-ai-review.yml) direkt die Folge-Workflows via workflow_dispatch triggern.