feat: deliver review fixes inline (suggestion + collapsible AI prompt) - #180
Merged
Merged
Conversation
When the github-pr-labeled-review agent leaves a comment requesting a change, the fix now travels with the comment instead of in a separate ticket: - Each change-requesting comment is an inline review comment anchored to the exact file:line. - When the fix is a concrete line edit, the comment carries a GitHub ```suggestion block at the top level so the author gets a one-click "Apply suggestion" button. - Every such comment also offers an optional, copy-pasteable AI-agent fix prompt, hidden in a collapsible <details> block so it stays out of the way until wanted. - No follow-up issue/ticket is opened; a clean approval adds neither. Reworks only the route's promptTemplate plus the integrations-gateway docs -- no new config knob. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Doi7cuiWVA94FrUbBKRT1n
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
Replaces the previous "open a follow-up ticket" approach (closed PR #178) per feedback. When the PR review agent (
github-pr-labeled-review, triggered when a human applies theai-reviewlabel) leaves a comment requesting a change, the fix now travels with the comment — nothing to open, nothing to re-transcribe.Each change-requesting comment is an inline review comment anchored to the exact
file:line, and carries:suggestionblock at the comment's top level whenever the fix is a concrete, localized line edit — so GitHub renders a one-click Apply suggestion button the author can commit directly. The block holds the complete replacement for the anchored line range; the agent widens the range if the fix spills onto neighboring lines.<details>block (🤖 Prompt to fix this with an AI agent) so it stays out of the way until the author wants it. Written as a self-contained instruction (goal,file:line, acceptance criteria) another agent can act on without re-reading the diff.Fixes that aren't a mechanical line edit (architectural / multi-file) skip the suggestion and rely on the copyable prompt. A clean approval adds neither, and no issue or ticket is opened either way.
Changes
charts/community-components/templates/integrationroute-github-pr-labeled-review.yaml— reworked the route'spromptTemplateto instruct inline suggestion + collapsible prompt delivery; removed the ticket-opening instructions.docs/integrations-gateway.md— documents the inline-fix behavior under thepull_request.labeledsection.No new config knob is introduced (the earlier
fixLabelNamevalue is not needed here) — this is purely prompt behavior. All in-prompt placeholders keep the{{`{{owner}}`}}Helm escaping; brace/delimiter balance verified.Notes / caveats
helm templaterender.🤖 Generated with Claude Code
https://claude.ai/code/session_01Doi7cuiWVA94FrUbBKRT1n