Skip to content

Revert Daily yamllint Fixer to Claude to avoid Copilot 429 incomplete runs#43950

Merged
pelikhan merged 2 commits into
mainfrom
copilot/aw-fix-daily-yamllint-fixer
Jul 7, 2026
Merged

Revert Daily yamllint Fixer to Claude to avoid Copilot 429 incomplete runs#43950
pelikhan merged 2 commits into
mainfrom
copilot/aw-fix-daily-yamllint-fixer

Conversation

Copilot AI commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

The Daily yamllint Fixer started failing after its engine change to Copilot, producing report_incomplete outcomes caused by provider 429s rather than an actual yamllint remediation result. This change restores the workflow to the last known-good execution path.

  • Restore stable runtime

    • switch daily-yamllint-fixer from copilot back to claude
    • remove the now-unneeded copilot-requests: write permission
    • keep the workflow behavior and prompt intact; only the execution engine changes
  • Regenerate compiled workflow

    • recompile daily-yamllint-fixer.lock.yml so the emitted runtime metadata, engine wiring, and allowed domains match the restored Claude configuration
  • Net effect

    • scheduled runs should stop surfacing infrastructure-only report_incomplete results caused by Copilot rate limiting
    • the workflow continues to do the same yamllint analysis and PR generation work, but on the engine used by the last successful baseline run
engine:
  id: claude

permissions:
  contents: read
  pull-requests: read
  issues: read

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix workflow failure in Daily yamllint Fixer Revert Daily yamllint Fixer to Claude to avoid Copilot 429 incomplete runs Jul 7, 2026
Copilot AI requested a review from pelikhan July 7, 2026 07:02
@pelikhan pelikhan marked this pull request as ready for review July 7, 2026 07:22
Copilot AI review requested due to automatic review settings July 7, 2026 07:22
@pelikhan pelikhan merged commit 3a511d4 into main Jul 7, 2026
1 check passed
@pelikhan pelikhan deleted the copilot/aw-fix-daily-yamllint-fixer branch July 7, 2026 07:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Reverts the daily-yamllint-fixer workflow runtime engine from Copilot back to Claude to avoid provider 429-induced report_incomplete runs, while keeping the workflow’s behavior/prompt intact and regenerating the compiled workflow to match the restored engine wiring.

Changes:

  • Switch workflow engine from copilot to claude and remove copilot-requests: write permission.
  • Regenerate daily-yamllint-fixer.lock.yml to reflect Claude runtime metadata (engine id/version, secret validation, domains allowlist, log parsing, concurrency keying).
Show a summary per file
File Description
.github/workflows/daily-yamllint-fixer.md Updates frontmatter to use the Claude engine and removes the Copilot-specific permission.
.github/workflows/daily-yamllint-fixer.lock.yml Recompiled workflow reflecting Claude engine wiring (Anthropic secret validation, engine/env updates, concurrency rename, log parsing updates).

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Low

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.82.3

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[aw] Daily yamllint Fixer reported incomplete result

3 participants