fix: PR-Chat bei Follow-ups + WIP: während der Arbeit wieder setzen #62

Closed
opened 2026-07-13 10:29:01 +02:00 by frank · 1 comment
Owner

Problem

Bei PR-Mention-Follow-ups (RunMentionReply) antwortet der Agent nicht mehr im PR-Chat (keine 💭 Agent progress-Kommentare). Zusätzlich bleibt der PR nach dem ersten Finalize ohne WIP:-Prefix, obwohl wieder aktiv gearbeitet wird.

Erwartetes Verhalten

  1. PR-Chat auch bei Follow-ups: Während Mention-/CI-/Conflict-Follow-ups sollen Agent-Antworten wie beim Initial-Run wieder als PR-Kommentare erscheinen (PRChatCommenter / ACP-Events).
  2. WIP während der Arbeit: Sobald Follow-up-Arbeit startet, soll der PR-Titel wieder den Prefix WIP: bekommen (analog WIPPRTitle). Nach erfolgreichem Abschluss / Finalize wird WIP: wie bisher entfernt.

Vermutete Ursache

  • In RunIssue wird p.prChat = NewPRChatCommenter(...) gesetzt; in RunMentionReply (und vermutlich RunCIFix / RunConflictFix) fehlt das.
  • Nach Finalize wird StripWIPPrefix angewendet; beim erneuten Start von Follow-up-Arbeit wird der Prefix nicht wiederhergestellt.

Akzeptanzkriterien

  • Mention-Follow-up postet wieder Zwischenantworten als 💭 **Agent progress** auf dem PR
  • Beim Start von Follow-up-Arbeit (mindestens Mention; idealerweise auch CI-/Conflict-Repair) erhält der offene PR wieder WIP: im Titel
  • Nach erfolgreichem Abschluss wird WIP: wieder entfernt / finalisiert
  • Tests für Re-WIP und PR-Chat-Anbindung im Follow-up-Pfad

Hinweise

Relevante Stellen: internal/agent/pipeline.go (RunIssue, RunMentionReply, Finalize), internal/agent/pr_chat.go, internal/agent/pr_wip.go.

## Problem Bei PR-Mention-Follow-ups (`RunMentionReply`) antwortet der Agent nicht mehr im PR-Chat (keine `💭 Agent progress`-Kommentare). Zusätzlich bleibt der PR nach dem ersten Finalize ohne `WIP:`-Prefix, obwohl wieder aktiv gearbeitet wird. ## Erwartetes Verhalten 1. **PR-Chat auch bei Follow-ups:** Während Mention-/CI-/Conflict-Follow-ups sollen Agent-Antworten wie beim Initial-Run wieder als PR-Kommentare erscheinen (`PRChatCommenter` / ACP-Events). 2. **WIP während der Arbeit:** Sobald Follow-up-Arbeit startet, soll der PR-Titel wieder den Prefix `WIP:` bekommen (analog `WIPPRTitle`). Nach erfolgreichem Abschluss / Finalize wird `WIP:` wie bisher entfernt. ## Vermutete Ursache - In `RunIssue` wird `p.prChat = NewPRChatCommenter(...)` gesetzt; in `RunMentionReply` (und vermutlich `RunCIFix` / `RunConflictFix`) fehlt das. - Nach Finalize wird `StripWIPPrefix` angewendet; beim erneuten Start von Follow-up-Arbeit wird der Prefix nicht wiederhergestellt. ## Akzeptanzkriterien - [x] Mention-Follow-up postet wieder Zwischenantworten als `💭 **Agent progress**` auf dem PR - [x] Beim Start von Follow-up-Arbeit (mindestens Mention; idealerweise auch CI-/Conflict-Repair) erhält der offene PR wieder `WIP:` im Titel - [x] Nach erfolgreichem Abschluss wird `WIP:` wieder entfernt / finalisiert - [x] Tests für Re-WIP und PR-Chat-Anbindung im Follow-up-Pfad ## Hinweise Relevante Stellen: `internal/agent/pipeline.go` (`RunIssue`, `RunMentionReply`, Finalize), `internal/agent/pr_chat.go`, `internal/agent/pr_wip.go`.
Collaborator

🤖 forge agent started

  • Agent: cursor-agent
  • Model: (default)
  • Mode: pr
  • Trigger: assignee=agent|+4 mapped
  • Open ToDos: 4

fix: PR-Chat bei Follow-ups + WIP: während der Arbeit wieder setzen — Bei PR-Mention-Follow-ups (RunMentionReply) antwortet der Agent nicht mehr im PR-Chat (keine 💭 Agent progress-Kommentare). Zusätzlich bleibt der PR nach dem ersten Finalize ohne WIP:-Prefix,…

🤖 **forge agent started** - Agent: `cursor-agent` - Model: `(default)` - Mode: `pr` - Trigger: `assignee=agent|+4 mapped` - Open ToDos: 4 fix: PR-Chat bei Follow-ups + WIP: während der Arbeit wieder setzen — Bei PR-Mention-Follow-ups (`RunMentionReply`) antwortet der Agent nicht mehr im PR-Chat (keine `💭 Agent progress`-Kommentare). Zusätzlich bleibt der PR nach dem ersten Finalize ohne `WIP:`-Prefix,…
frank closed this issue 2026-07-13 14:20:46 +02:00
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
frank/forgecli#62
No description provided.