Skip to content

send full diff to PR title/body generator#1394

Merged
adboio merged 1 commit intomainfrom
03-31-send_full_diff_to_pr_title_body_generator
Apr 1, 2026
Merged

send full diff to PR title/body generator#1394
adboio merged 1 commit intomainfrom
03-31-send_full_diff_to_pr_title_body_generator

Conversation

@adboio
Copy link
Copy Markdown
Contributor

@adboio adboio commented Mar 31, 2026

Problem

pr title/body LLM call only sends a "summary" with commit messages and changed file paths, so it often hallucinates changes

the diff check is also not accurate to what will actually be in a PR against the default

Changes

updates diff checking, sends actual diff content to llm

How did you test this?

manually

Copy link
Copy Markdown
Contributor Author

adboio commented Mar 31, 2026

@adboio adboio force-pushed the 03-31-send_full_diff_to_pr_title_body_generator branch from a27b8da to 03f27e5 Compare March 31, 2026 18:16
@adboio adboio force-pushed the fix/dedupe-suggested-branch-names branch from 12fe24e to bca7194 Compare March 31, 2026 18:25
@adboio adboio force-pushed the 03-31-send_full_diff_to_pr_title_body_generator branch from 03f27e5 to c0aaf55 Compare March 31, 2026 18:25
@adboio adboio requested a review from a team March 31, 2026 18:25
@adboio adboio marked this pull request as ready for review March 31, 2026 18:26
@adboio adboio force-pushed the fix/dedupe-suggested-branch-names branch from bca7194 to 8afc4a6 Compare March 31, 2026 19:17
@adboio adboio force-pushed the 03-31-send_full_diff_to_pr_title_body_generator branch from c0aaf55 to 86ef9fd Compare March 31, 2026 19:17
joshsny
joshsny previously approved these changes Mar 31, 2026
Copy link
Copy Markdown
Contributor Author

adboio commented Mar 31, 2026

Merge activity

@adboio adboio changed the base branch from fix/dedupe-suggested-branch-names to graphite-base/1394 March 31, 2026 20:48
@adboio adboio force-pushed the graphite-base/1394 branch from 8afc4a6 to 4bf2c11 Compare March 31, 2026 21:33
@adboio adboio force-pushed the 03-31-send_full_diff_to_pr_title_body_generator branch from 86ef9fd to 479b56d Compare March 31, 2026 21:33
@adboio adboio mentioned this pull request Mar 31, 2026
@adboio adboio force-pushed the graphite-base/1394 branch from 4bf2c11 to 46131fa Compare March 31, 2026 22:15
@adboio adboio force-pushed the 03-31-send_full_diff_to_pr_title_body_generator branch from 479b56d to 2ae8173 Compare March 31, 2026 22:15
@graphite-app graphite-app bot changed the base branch from graphite-base/1394 to main March 31, 2026 22:15
@graphite-app graphite-app bot dismissed joshsny’s stale review March 31, 2026 22:15

The base branch was changed.

@adboio adboio merged commit 56aba1c into main Apr 1, 2026
16 checks passed
@adboio adboio deleted the 03-31-send_full_diff_to_pr_title_body_generator branch April 1, 2026 07:08
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.

2 participants