Bug: kontext.md wird bei Pipeline-Failure nicht aufgeräumt #39

Closed
opened 2026-03-30 19:54:26 +00:00 by David · 1 comment
Collaborator

Beschreibung

Wenn die Pipeline fehlschlägt (z.B. während Review oder Claude Code Run), bleibt die kontext.md im Repository liegen. Nachfolgende Runs finden stale Kontext-Dateien.

Hintergrund

pipeline.py:363 räumt kontext.md nur bei Erfolg auf (_cleanup_kontext). Bei Failures wird die Cleanup-Funktion nicht aufgerufen. Wenn das Ticket erneut gestartet wird, existiert die alte kontext.md noch im Repo.

Akzeptanzkriterien

  • kontext.md wird auch bei Pipeline-Failure aufgeräumt (try/finally Block)
  • Cleanup läuft in allen Failure-Pfaden: Review-Failure, Claude-Code-Timeout, Git-Push-Failure
  • Beim Start eines neuen Runs: Prüfung ob alte kontext.md existiert und Warnung/Cleanup
  • Test für Cleanup-Verhalten bei verschiedenen Failure-Szenarien

Technische Hinweise

  • Fix in: backend/services/pipeline.py_cleanup_kontext() in finally-Block verschieben
  • Zusätzlich: Check zu Beginn von _run_preparation_and_execution() ob kontext.md bereits existiert
  • Migration nötig: nein

Aufwand: S

## Beschreibung Wenn die Pipeline fehlschlägt (z.B. während Review oder Claude Code Run), bleibt die kontext.md im Repository liegen. Nachfolgende Runs finden stale Kontext-Dateien. ## Hintergrund `pipeline.py:363` räumt kontext.md nur bei Erfolg auf (`_cleanup_kontext`). Bei Failures wird die Cleanup-Funktion nicht aufgerufen. Wenn das Ticket erneut gestartet wird, existiert die alte kontext.md noch im Repo. ## Akzeptanzkriterien - [ ] kontext.md wird auch bei Pipeline-Failure aufgeräumt (try/finally Block) - [ ] Cleanup läuft in allen Failure-Pfaden: Review-Failure, Claude-Code-Timeout, Git-Push-Failure - [ ] Beim Start eines neuen Runs: Prüfung ob alte kontext.md existiert und Warnung/Cleanup - [ ] Test für Cleanup-Verhalten bei verschiedenen Failure-Szenarien ## Technische Hinweise - Fix in: `backend/services/pipeline.py` — `_cleanup_kontext()` in `finally`-Block verschieben - Zusätzlich: Check zu Beginn von `_run_preparation_and_execution()` ob kontext.md bereits existiert - Migration nötig: nein ## Aufwand: S
Author
Collaborator

Superseded by #78 (Error Recovery & Cleanup). kontext.md Cleanup ist dort als Teil der Aufräumlogik abgedeckt (Finding #5: kontext.md-Write nicht atomar, Finding #8: Kein Cleanup nach Partial Failure).

Superseded by #78 (Error Recovery & Cleanup). kontext.md Cleanup ist dort als Teil der Aufräumlogik abgedeckt (Finding #5: kontext.md-Write nicht atomar, Finding #8: Kein Cleanup nach Partial Failure).
David closed this issue 2026-03-30 20:39:21 +00:00
Sign in to join this conversation.
No description provided.