Skip to content

Add task-preflight skill to stop zero-diff retry PRs on container-vuln and refactor tasks - #53674

Closed
pelikhan with Copilot wants to merge 2 commits into
mainfrom
copilot/copilot-opt-reduce-retry-waste
Closed

Add task-preflight skill to stop zero-diff retry PRs on container-vuln and refactor tasks#53674
pelikhan with Copilot wants to merge 2 commits into
mainfrom
copilot/copilot-opt-reduce-retry-waste

Conversation

CopilotAI commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Container-vulnerability-fix and large-refactor tasks repeatedly burned full agent sessions: 5 of 6 closed PRs across three topics had 0 additions/0 deletions and no comments, so each retry restarted from zero with no record of the blocker.

Changes

  • .github/skills/task-preflight/SKILL.md (new) — feasibility check that must pass before any code is pushed, covering the two topics that produce zero-diff PRs:

    • Container vulnerability tasks: reproduce with gh aw compile --force-refresh-container-pins --syft --grype --grant; unreachable image ⇒ blocker report. Otherwise route to one of the two real remediations — refresh the pin (commit changed .lock.yml) or add a scoped .grype.yaml ignore entry with a reason. Stale finding ⇒ report with scan output, no PR.
    • Large-refactor tasks: derive the offender list from make golint-custom (largefunc, 60-line default) instead of guessing, scope to a small named subset with a written extraction plan, keep it behaviour-preserving. Repo-wide rewrites like the 6096/6203-line attempt in #49825 are explicitly out.
    • Hard rules: never finish at 0/0 diff without explanation; never leave an empty placeholder PR; when blocked, emit a structured report (attempted / failure output / root cause / what unblocks it).
  • AGENTS.md — one routing line in "Use these skills by intent", keeping the ambient file short per the repo's lazy-loading policy.

Blocker report shape the skill prescribes:

## Blocked: <tasktitle>**Attempted**: <commandsrun>
**Failure**: <exacterroroutput>
**Why it cannot proceed**: <rootcause>
**Unblocked by**: <access, upstream fix, scope decision needed from a maintainer>

Notes

The issue asks for a plan comment on the issue before opening a PR. No issue-comment tool was available in this environment, so the plan and links to the six prior closed PRs were published in the PR description before any content changes were pushed.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
CopilotAI changed the title [WIP] Fix zero-diff retries on container vulnerability and refactor tasksAdd task-preflight skill to stop zero-diff retry PRs on container-vuln and refactor tasksAug 18, 2026
CopilotAI requested a review from pelikhanAugust 18, 2026 12:25
@github-actions

Copy link
Copy Markdown
Contributor

PR Triage

Category: chore (WIP) | Risk: low | Priority: low (score 20)

  • Draft, opened minutes ago with 0 diff — agent still generating changes (Fix zero-diff retries on container vulnerability and refactor tasks)
  • Batch: pr-batch:wip-drafts (6 similar just-opened WIP PRs)

Recommended action: defer — recheck once agent finishes and diff is populated

Generated by 🔧 PR Triage Agent · auto · 53.1 AIC · ⌖ 2.52 AIC · ⊞ 8.3K ·

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[copilot-opt] Reduce zero-diff retry waste on container vulnerability & refactor tasks

2 participants

@pelikhan