feat(pr-review): add summary-prompt input for context-aware review summaries#119
feat(pr-review): add summary-prompt input for context-aware review summaries#119daniel-richter wants to merge 4 commits into
Conversation
|
This update adds a new Walkthrough
Model: o4-mini-2025-04-16 | Prompt Tokens: 1707 | Completion Tokens: 829 |
There was a problem hiding this comment.
Thank you for sharing your code. I’ve offered ideas supported by AI to improve readability, control flow and maintainability while respecting your judgment. Feel free to apply what fits your style as you guide the decisions. The feedback covers extracting summary generation into its own helper, adding a guard to skip unnecessary API calls, improving display text assembly through a utility function, updating action.yml inputs with consistent structure and examples, and clarifying summary-prompt defaults and formatting in the README.
Model: o4-mini-2025-04-16 | Prompt Tokens: 3468 | Completion Tokens: 4933
|
This update aligns CI workflows with the latest Node setup action, brings core libraries and tooling dependencies up to date in both the PR review and summary packages, and streamlines configuration parsing by removing unnecessary type assertions. Walkthrough
Model: o4-mini-2025-04-16 | Prompt Tokens: 1893 | Completion Tokens: 487 | Diff Range: 699875e...68c8f91 |
|
Summary prompt is now optional: if left unset, only the disclaimer is sent and review findings are excluded. Documentation has been updated to clarify this behavior. The system’s chat payload now uses additional spacing for readability, and the review comments step is conditionally omitted based on the summary prompt. These adjustments empower users to control whether detailed review insights are shared with the LLM. Walkthrough
Model: o4-mini-2025-04-16 | Prompt Tokens: 697 | Completion Tokens: 1364 | Diff Range: 68c8f91...9fe97d3 |
There was a problem hiding this comment.
This collaborative code review is powered by AI. These ideas come from predictive analysis and may not always hit the mark. Use what aligns with your workflow and expertise. You remain in control of every choice—AI is here solely to support your decisions.
Model: o4-mini-2025-04-16 | Prompt Tokens: 1289 | Completion Tokens: 2451 | Diff Range: 699875e...9fe97d3
|
This change simplifies the code formatting by collapsing a multi-line conditional spread into a single-line expression without altering any logic or user-facing behavior. Walkthrough
Model: o4-mini-2025-04-16 | Prompt Tokens: 356 | Completion Tokens: 457 | Diff Range: 9fe97d3...e854784 |
There was a problem hiding this comment.
I’m pleased to provide a supportive code review enhanced by AI insights. While some suggestions are informed predictions, trust your judgment to decide what fits best. You guide the process and AI is here to support you.
Model: o4-mini-2025-04-16 | Prompt Tokens: 916 | Completion Tokens: 1881 | Diff Range: 9fe97d3...e854784
Context: The action previously generated the review body text using
disclaimer-promptin isolation, with no access to the actual review findings. This meant the summary could not include context-aware content such as finding counts or severity breakdowns, since the findings only existed after the review was generated.Changes:
summary-promptinput parameter (default:"Append a brief summary of the key review findings. No lists, no headings.") to allow customizing the summary textdisclaimer-promptAI call with a combined system prompt that mergesdisclaimer-promptandsummary-prompt, passing the actual review findings as user content so the model can produce a context-aware summarycomments,review, andsummaryoutput parameters toaction.ymland updated their descriptionssummary-promptparameter, the new outputs, and added the new "Customize the review summary" section