Skip to content

fix(ci-cleaner): add mandatory exit protocol to always produce safe outputs#24182

Merged
pelikhan merged 2 commits intomainfrom
copilot/fix-ci-cleaner-failure
Apr 2, 2026
Merged

fix(ci-cleaner): add mandatory exit protocol to always produce safe outputs#24182
pelikhan merged 2 commits intomainfrom
copilot/fix-ci-cleaner-failure

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 2, 2026

CI Cleaner agent ran for ~41 minutes working on CI fixes, exhausted its conversation turns, and exited without calling any safe-output tool — producing the "no safe outputs generated" failure.

Changes

  • .github/workflows/hourly-ci-cleaner.md — Added ## Mandatory Exit Protocol section between "Token Budget Awareness" and "Pull Request Guidelines"
  • .github/agents/ci-cleaner.agent.md — Added ## Mandatory Exit Protocol section before "Workflow & Best Practices"

Both sections enforce:

  • Call create_pull_request if any changes were made (even partial), after git add -A && git commit
  • Call noop if no changes were made (CI already passing, can't reproduce, or too complex)
  • If running low on turns: commit current work immediately and call create_pull_request rather than continuing to attempt more fixes — never exit without a safe output

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/graphql
    • Triggering command: /usr/bin/gh /usr/bin/gh api graphql -f query=query($owner: String!, $name: String!) { repository(owner: $owner, name: $name) { hasDiscussionsEnabled } } -f owner=github -f name=gh-aw (http block)
    • Triggering command: /usr/bin/gh /usr/bin/gh api graphql -f query=query($owner: String!, $name: String!) { repository(owner: $owner, name: $name) { hasDiscussionsEnabled } } -f owner=github -f name=gh-aw -pack /home/REDACTED/work/gh-aw/gh-aw/cmd/gh-aw/main.go (http block)
  • https://api.github.com/repos/astral-sh/setup-uv/git/ref/tags/eac588ad8def6316056a12d4907a9d4d84ff7a3b
    • Triggering command: /usr/bin/gh gh api /repos/astral-sh/setup-uv/git/ref/tags/eac588ad8def6316056a12d4907a9d4d84ff7a3b --jq .object.sha (http block)
  • https://api.github.com/repos/github/gh-aw
    • Triggering command: /usr/bin/gh gh api /repos/github/gh-aw --jq .visibility (http block)
  • https://api.github.com/repos/github/gh-aw-actions/git/ref/tags/v0
    • Triggering command: /usr/bin/gh gh api /repos/github/gh-aw-actions/git/ref/tags/v0 --jq .object.sha (http block)
  • https://api.github.com/repos/githubnext/agentics/git/ref/tags/
    • Triggering command: /usr/bin/gh gh api /repos/githubnext/agentics/git/ref/tags/# --jq .object.sha (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI linked an issue Apr 2, 2026 that may be closed by this pull request
…re always produced

Agent-Logs-Url: https://github.com/github/gh-aw/sessions/e198ac68-27c1-421b-af25-e35d1913ca99

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI Cleaner workflow failure fix(ci-cleaner): add mandatory exit protocol to always produce safe outputs Apr 2, 2026
Copilot AI requested a review from pelikhan April 2, 2026 22:26
@pelikhan pelikhan marked this pull request as ready for review April 2, 2026 22:33
Copilot AI review requested due to automatic review settings April 2, 2026 22:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a “Mandatory Exit Protocol” to the CI cleaner workflow/agent docs to ensure every run ends by producing a safe output (create_pull_request or noop), even when time/turns are running out.

Changes:

  • Add a Mandatory Exit Protocol section to the CI cleaner workflow instructions.
  • Add a Mandatory Exit Protocol section to the ci-cleaner agent guidance.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/hourly-ci-cleaner.md Documents a strict requirement to call create_pull_request/noop before session end.
.github/agents/ci-cleaner.agent.md Aligns the CI cleaner agent instructions with the same mandatory safe-output exit protocol.
Comments suppressed due to low confidence (1)

.github/workflows/hourly-ci-cleaner.md:181

  • Same issue here: git add -A && git commit may open an editor and hang. Recommend using a non-interactive commit command (e.g., always include -m) to ensure the exit protocol reliably produces a safe output.
- Stage and commit whatever changes you have made so far (`git add -A && git commit`)

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

@pelikhan pelikhan merged commit 3298808 into main Apr 2, 2026
67 checks passed
@pelikhan pelikhan deleted the copilot/fix-ci-cleaner-failure branch April 2, 2026 23:19
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] CI Cleaner failed

3 participants