Skip to content

Comments

bonk: improve PR review steering#6150

Open
elithrar wants to merge 6 commits intomainfrom
elithrar-patch-1
Open

bonk: improve PR review steering#6150
elithrar wants to merge 6 commits intomainfrom
elithrar-patch-1

Conversation

@elithrar
Copy link
Contributor

@elithrar elithrar commented Feb 24, 2026

Improve prompt to steer PR review comments.

  • Force the $PR_NUMBER into the prompt
  • Improved fetch-depth for diffs (balance of larger checkouts / time)
  • Stronger prompt guidance

These are also now exposed in Bonk's GitHub Action (which feeds into OpenCode) as of ask-bonk/ask-bonk#117

Improve prompt to steer PR review comments.
@elithrar elithrar requested review from a team as code owners February 24, 2026 11:39
@ask-bonk

This comment was marked as outdated.

@dom96
Copy link
Contributor

dom96 commented Feb 24, 2026

Hmm, I guess the fact that it did it again in this PR suggests that it's a bug in Bonk rather than a prompt issue.

@elithrar elithrar requested a review from a team as a code owner February 24, 2026 12:07
@elithrar elithrar enabled auto-merge February 24, 2026 12:09
@elithrar elithrar self-assigned this Feb 24, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 24, 2026

Merging this PR will not alter performance

✅ 70 untouched benchmarks
⏩ 129 skipped benchmarks1


Comparing elithrar-patch-1 (b25819b) with main (1a5413f)

Open in CodSpeed

Footnotes

  1. 129 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

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.

5 participants