Add code-review and needs-info-sweeper workflows#7614
Open
kotlarmilos wants to merge 5 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds GitHub Agentic Workflows (gh-aw) automation to improve PR review signal, recurring CI failure triage, Dependabot auto-merge throughput, and “need info” issue hygiene in the dotnet/machinelearning repo.
Changes:
- Introduces 3 new gh-aw agent workflows (
code-review,ci-scan,needs-info-sweeper) with compiled lock workflows. - Adds a Dependabot reviewer agent plus a scheduled finalize workflow to auto-merge eligible Dependabot PRs.
- Updates
.github/aw/actions-lock.jsonto include an additional pinned action entry.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/needs-info-sweeper.agent.md | Defines the “need info” issue sweep policy and fixed comment wording/caps. |
| .github/workflows/needs-info-sweeper.agent.lock.yml | Compiled gh-aw workflow implementing the sweeper with Safe Outputs enforcement. |
| .github/workflows/dependabot-auto-merge.agent.md | Reviewer agent that labels safe Dependabot PRs as auto-merge candidates. |
| .github/workflows/dependabot-auto-merge-finalize.yml | Scheduled job that merges labeled Dependabot PRs after checks/age gates. |
| .github/workflows/code-review.agent.md | Read-only PR review agent prompt/config for high-signal correctness feedback. |
| .github/workflows/code-review.agent.lock.yml | Compiled gh-aw workflow for the code review agent. |
| .github/workflows/ci-scan.agent.md | Agent prompt/config for detecting recurring CI failures and filing issues. |
| .github/workflows/ci-scan.agent.lock.yml | Compiled gh-aw workflow for the CI failure scanner. |
| .github/aw/actions-lock.json | Updates pinned action references used by gh-aw compilation/execution. |
Copilot's findings
- Files reviewed: 10/10 changed files
- Comments generated: 7
Adds three gh-aw workflows. ci-scan runs every 6 hours, walks AzDO definition 167 (MachineLearning-CI) on main, classifies failures (build break, test failure, Helix-routed), dedups against open Known Build Error issues, applies the follow-up-build presence gate to filter transient regressions, and files structured KBEs with literal substrings that Build Analysis can match against PR CI. code-review runs on every non-draft PR with idempotent sha-keyed markers; scope is IDataView contract, trainer numerical correctness, tokenizer round-trips, ONNX opset compatibility, AutoML correctness, and public API baseline. needs-info-sweeper sweeps need-info-labeled issues every 2 days: removes the label on reply, nudges at 14 days, closes as not planned at 30 days, capped at 30 nudges and 15 closes per run with fixed wording and idempotent markers. All agent runs read-only; writes go through capped, allowlisted safe-outputs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
kotlarmilos
force-pushed
the
ai/agentic-workflows
branch
from
744650f to
5299107
Compare
kotlarmilos
changed the title
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
kotlarmilos
changed the title
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Align description with every-2-day cadence. - Tighten reply-check rule with explicit non-bot, non-labeler logic. - Update nudge wording so the 14-day close window is consistent with the 30-day age gate. - Note that close_issue safe-output sets state=closed (reason 'completed'); 'not planned' is not exposed by the schema. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
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.
Adds two gh-aw workflows.
code-reviewruns on each non-draft PR from the same repo. Posts one read-only review comment scoped to IDataView contracts, trainer numerical correctness, Tokenizer round-trips, ONNX, AutoML, and breaking public-API changes. Skips style and formatting. One comment per head sha.needs-info-sweeperruns every 2 days. For issues labeledneed info: removes the label if a non-labeler replied; posts a fixed nudge after 14 days of silence; closes asnot plannedafter 30 days, but only if the issue was previously nudged by this workflow. Caps at 30 nudges and 15 closes per run.