Skip to content

fix: preserve PR title across reviewer iterations#90

Merged
mdelapenya merged 1 commit intomainfrom
ralph/issue-89
Mar 23, 2026
Merged

fix: preserve PR title across reviewer iterations#90
mdelapenya merged 1 commit intomainfrom
ralph/issue-89

Conversation

@mdelapenya
Copy link
Copy Markdown
Owner

Closes #89

Status

SHIP — Iteration 1

Summary

  • Updated reviewer system prompt to instruct the reviewer to evaluate the ENTIRE branch diff when generating PR titles, not just the latest iteration's work
  • Added instruction to preserve existing PR titles unless the scope of changes has fundamentally changed (e.g., a feat became a refactor)
  • Updated PR body generation to build a comprehensive summary of ALL changes on the branch across all iterations

Changes

  • prompts/reviewer-system.md: Added CRITICAL instruction for full-branch-aware PR title generation, existing title preservation logic, and comprehensive work summary building

Generated by Claude Ralph GitHub Action

The reviewer was generating PR titles based only on the latest iteration's
work, causing misleading titles like 'chore: use subtests' when the main
change was 'feat: new feature'. Now the reviewer is instructed to review
the full branch diff and preserve existing PR titles unless the scope
fundamentally changes.

Ralph-Run-Id: 23434416100
Ralph-Run-Url: https://github.com/mdelapenya/claude-ralph-github-action/actions/runs/23434416100
Ralph-Worker-Model: opus
Copilot AI review requested due to automatic review settings March 23, 2026 11:13
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the reviewer agent system prompt to keep PR metadata (title/body) aligned with the full branch scope across iterative Ralph cycles, addressing Issue #89 where titles drift to reflect only the latest iteration.

Changes:

  • Instructs the reviewer to generate PR titles based on the entire branch diff/commit history, not just the latest iteration.
  • Adds guidance to preserve an existing PR title unless the overall scope/type has fundamentally changed.
  • Updates PR body work-summary guidance to be comprehensive across all branch changes, using commit history as a source of truth.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mdelapenya mdelapenya merged commit f8a8ef2 into main Mar 23, 2026
6 checks passed
@mdelapenya mdelapenya deleted the ralph/issue-89 branch March 23, 2026 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Do not change PR title when reworking on a branch

2 participants