- Notifications
You must be signed in to change notification settings - Fork1k
ci: add auto-generated summary and GitHub issue comment workflows to triage.yaml#19925
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This PR adds an auto-generated summary feature and GitHub issue commenting functionality to the traiage workflow. The changes enhance the CI/CD pipeline to automatically generate summaries of workspace changes using Claude AI and post structured comments back to GitHub issues.
- Adds a new
summarycommand to generate AI-powered summaries of workspace changes - Implements GitHub issue commenting with auto-generated summaries and metadata
- Improves output redirection and quoting consistency in the workflow
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| scripts/traiage.sh | Adds workspace existence check, summary generation function, and output redirection |
| .github/workflows/traiage.yaml | Implements GitHub commenting workflow with auto-generated summaries and improves quoting consistency |
Tip: Customize your code reviews with copilot-instructions.md.Create the file orlearn how to get started.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
| bash<<-EOF | ||
| #!/usr/bin/env bash | ||
| set -euo pipefail | ||
| summary=\$(echo -n 'You are a CLI utility that generates a human-readable Markdown summary for the currently staged AND unstaged changes. Print ONLY the summary and nothing else.' |\${HOME}/.local/bin/claude --print) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
[nitpick] The command line for generating the summary is excessively long and hard to read. Consider breaking this into multiple lines or storing the prompt in a variable for better readability and maintainability.
| summary=\$(echo -n'You are a CLI utility that generates a human-readable Markdown summary for the currently staged AND unstaged changes. Print ONLY the summary and nothing else.'|\${HOME}/.local/bin/claude --print) | |
| SUMMARY_PROMPT='You are a CLI utility that generates a human-readable Markdown summary for the currently staged AND unstaged changes. Print ONLY the summary and nothing else.' | |
| summary=\$(echo -n"\${SUMMARY_PROMPT}"|\${HOME}/.local/bin/claude --print) |
Copilot uses AI. Check for mistakes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Looks fine to me, just a nit on the PR title, it saystriage.yaml when the file being changed istraiage.yaml.
That's just to appease the grammar bot 🤣 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Awesome!
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Comments addressed in4e03814 |
8f3e03a intomainUh oh!
There was an error while loading.Please reload this page.
No description provided.