Repository navigation
ci: repair broken issue-summary workflow without Copilot dependency - #70
Conversation
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 21 |
| Duplication | 0 |
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
Pull Request Overview
✅ No blocking issues noticed
No blocking code, security, or quality findings were identified.
The required live opened-issue smoke check is not present in the supplied test evidence, leaving end-to-end workflow behavior unverified. The reported coverage information is also incomplete.
Test suggestions
- Generate a deterministic summary with bounded title and body content, neutralized mentions, and extracted P0-P3 priority.
- Ensure shell-looking issue text is preserved as data and is not executed.
- Handle missing, null, or invalid issue data with safe fallback values.
- Run the CLI against an event JSON file and write the generated summary to the requested output file.
- Verify the complete workflow performs a live opened-issue smoke check and posts exactly one comment.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify the complete workflow performs a live opened-issue smoke check and posts exactly one comment.
Low confidence findings
- Please add or document an automated live workflow smoke check that verifies an opened issue produces exactly one deterministic comment.
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
Why
The
Summarize new issuesworkflow fails on every new issue becauseactions/ai-inference@v3selects a Copilot CLI provider, but the runner has nocopilotbinary (and no configuredCOPILOT_GITHUB_TOKEN). We confirmed the failure in run #41366840183. This should not be an ongoing failed workflow or require spending on a model just to triage incoming issues.Changes
GITHUB_EVENT_PATHas data; do not interpolate untrusted issue body in a shell command.@mentions; extract stated P0-P3 priority without independently asserting severity.gh issue commentwith the existing standardGITHUB_TOKEN, no external AI or credentials.Tests
issues: openedsmoke check produces one non-AI, deterministic comment without failing.No generated models, network subscriptions or paid resources are required.