[Klaud Cold] AGENTS: note sweep doesn't trigger while a PR has merge conflicts#1495
Conversation
A stale claude/* or update-* branch with a perf-changelog.yaml conflict (most common case) sits in NO_SWEEP / NO_SUCCESS even when sweep-enabled or full-sweep-enabled is applied — the run-sweep.yml workflow needs the PR to cleanly merge into main first. Adds a paragraph under "Pull Request Sweep Labels" pointing at the existing canonical resolution recipe in KLAUD_DEBUG.md §1.1. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
LGTM — single-paragraph doc note in AGENTS.md clarifying that the sweep won't run on PRs with merge conflicts.
Extended reasoning...
Overview
The PR adds a single paragraph to AGENTS.md under the "Pull Request Sweep Labels" section, explaining that run-sweep.yml won't trigger while a PR has merge conflicts even with sweep labels applied, and points to the canonical resolution recipe already documented in KLAUD_DEBUG.md §1.1.
Security risks
None. This is a documentation-only change with no executable code, no configuration, and no impact on auth/permissions/data handling.
Level of scrutiny
Minimal scrutiny is appropriate. This is a pure prose addition to an agent-facing markdown file. There is no code path, workflow, or schema affected. The content is consistent with the existing perf-changelog.yaml whitespace warnings already present in the same file.
Other factors
The note accurately reflects standard GitHub Actions behavior (PR-triggered workflows are gated on a clean mergeable state) and references an existing canonical recipe rather than duplicating it. No outstanding reviewer comments, no bugs flagged by the bug-hunting system.
1 participant
Summary
Single-paragraph doc edit in
AGENTS.mdunder "Pull Request Sweep Labels": calls out that a PR sitting on a stale branch with aperf-changelog.yamlconflict will land in NO_SWEEP / NO_SUCCESS even whensweep-enabled/full-sweep-enabledis applied — therun-sweep.ymlworkflow needs the PR to be cleanly mergeable into main first.Surfaced this repeatedly today (#1422, #1451) — the dashboard would show
NO_SWEEPand it wasn't obvious that "no sweep" meant "rebase your branch" rather than "the label dropped" or "something is broken upstream". Points readers at the existing canonical resolution recipe inKLAUD_DEBUG.md §1.1.Test plan
🤖 Generated with Claude Code