Auto-update PR description after each fix iteration

[chanakyav]

The fix prompt tells the agent to update PR title/description via gh pr edit when the approach changes significantly, but the agent almost never does it. The result is stale PR descriptions that don't reflect the work done across multiple fix iterations.

Proposed Approach

Move PR description updates from agent discretion to orchestrator-controlled — the orchestrator already has the fix summary data and knows exactly what changed.

PR Body: Update after each iteration

In _do_resolve_comments, after resolving all comments, append a changelog section to the PR body:

---

### Iteration 2 — autopilot-loop

**Comments addressed:**
- Comment 42 (billing.rb:15): FIXED — Extracted billing concern into separate module
- Comment 43 (user.rb:30): SKIPPED — Current pattern is idiomatic Ruby

**Commits:** abc1234, def5678

This uses gh pr edit --body to append (not replace) the iteration summary. Each iteration adds a new section, building a history of what was done.

PR Title: Update once at completion

In _do_parse_review when 0 unresolved comments are found (clean review → COMPLETE), optionally update the title:

gh pr edit --title "..."

The final title could be generated by asking the agent in the last fix pass, or by using the original task prompt as the base.

Alternatively, skip auto-title and just do the body updates — that's the higher-value change.

Implementation

1. In _do_resolve_comments (orchestrator.py): after resolving comments, call gh pr edit to append iteration summary to PR body
2. New function update_pr_body_with_summary(pr_number, iteration, summaries, commit_sha) in github_api.py
3. Remove step 8 from fix_prompt (agent no longer responsible for this)
4. Optional: update PR title at COMPLETE time

Open Questions

• Should we append to the existing body or maintain a separate "autopilot-loop activity" section?
• Should the iteration summary include the full comment text or just a brief note?
• Should we get the PR's current body via gh pr view and append, or use a separate comment?
• Alternative: post iteration summaries as PR comments instead of editing the body (less disruptive, more visible in the timeline)

[chanakyav]

Simplified scope: update PR body and title together once at the end when no feedback is left (PARSE_REVIEW finds 0 unresolved comments).

No per-iteration body updates. No PR comments for now — can add later if needed.

Implementation: in _do_parse_review, when transitioning to COMPLETE (0 comments), call gh pr edit to update both title and body based on the cumulative work done across all iterations.

[chanakyav]

Correction: the orchestrator should NOT call gh pr edit directly — that bypasses PR templates.

Instead, at COMPLETE time, run a lightweight agent invocation with a prompt like:

Update the PR title and body to reflect all the work done across iterations. Use the repo's PR template (.github/PULL_REQUEST_TEMPLATE.md) to structure the body. Review the git log and diff to write an accurate description.

This lets the agent:

• Read the PR template and honor its structure
• Review the actual diff and commit history
• Write a proper title and description

The orchestrator just triggers the agent call — it doesn't compose the text itself.

[gaurav-310]

I looked through the current flow, and the cleanest implementation seems to be:

• keep the latest clarified scope (final update only, no per-iteration PR edits)
• trigger a lightweight final agent run from _do_parse_review() when it finds 0 unresolved comments, just before returning COMPLETE
• add a dedicated prompt in prompts.py for refreshing PR title/body from the repo PR template, git diff, and commit history
• remove the current PR-update step from fix_prompt() so PR metadata refresh is orchestrator-controlled rather than discretionary in each fix pass

I think this avoids introducing a new state unless we specifically want resumable/retriable PR-metadata updates. My default would be to treat the final PR refresh as best-effort: log a warning on failure but still allow the task to complete once review is clean.

[chanakyav]

Closed — fully addressed by #70 (merged).

All items from this issue are implemented:

1. Orchestrator-controlled PR description — UPDATE_DESCRIPTION state runs after RESOLVE_COMMENTS every iteration
2. Removed step 8 from fix_prompt — agent no longer has discretionary PR edit
3. Dedicated agent invocation — update_description_prompt() via _run_agent_with_retry()
4. Preserves PR template — prompt checks .github/PULL_REQUEST_TEMPLATE.md
5. Non-fatal — failure logs a warning, doesn't block the loop

---

🤖 autopilot-loop