docs(agent): warn on missing workflow labels - #2815
Conversation
5cb5409 to
bea6264
Compare
elezar
left a comment
There was a problem hiding this comment.
Changes requested
P1 — Include write-level maintainers: The repository’s issue-triage workflow treats write alongside maintain and admin as maintainer permission. Excluding it here means issues authored by write-level maintainers remain blocked despite this PR’s goal; use the same permission set or document why it differs.
While working on some triage tooling, I noted some inconsistency in the triage skill with respect to the new state labels and opened #2809 to address that. This PR and #2809 overlap in .agents/skills/triage-issue/SKILL.md, so they will need to be reconciled when this change is updated.
bea6264 to
90d218d
Compare
Signed-off-by: Kris Hicks <khicks@nvidia.com>
90d218d to
9665b42
Compare
|
I thought about this more and loosened this further so that the agent just does what a human wants, warning when the labels on the issue don't align with the request, because we don't want a robot to refuse a human's request to work on something, no matter if they're a maintainer or not. |
Yes, having to toggle labels (or convince it that it's "just a POC") when starting on something was quite frustrating. Thanks for looking into this. |
Summary
Makes it so the agent does what I say when I want it to work on an issue that doesn't have the
state:acceptedlabel.Related Issue
Changes
Testing
mise run pre-commitpassesChecklist