Add working agreement: return a ready PR to draft for rework #32

Merged
geusebio merged 1 commit from docs/redraft-pr-on-rework into main 2026-08-22 16:40:50 +00:00
Owner

Marking a PR ready is currently one-way in this document: there's a rule for
opening as draft and for stripping WIP: once all-green and mergeable, but
nothing for the reverse — a PR that gets sent back for rework after review
(or after a second look surfaces something bigger than a quick fix) just
sits "ready" while it's actually not.

Adds a new subsection distinguishing rework (design change, a bug needing
more than a patch, a coverage gap needing new scaffolding) from a quick fix
to a red check, documents re-adding the WIP: prefix the same way it's
added the first time, and wires it into the working-agreement-mode table
and the quick reference.

Docs-only diff — opened ready, not draft, per this file's own convention.

Marking a PR ready is currently one-way in this document: there's a rule for opening as draft and for stripping `WIP:` once all-green and mergeable, but nothing for the reverse — a PR that gets sent back for rework after review (or after a second look surfaces something bigger than a quick fix) just sits "ready" while it's actually not. Adds a new subsection distinguishing rework (design change, a bug needing more than a patch, a coverage gap needing new scaffolding) from a quick fix to a red check, documents re-adding the `WIP:` prefix the same way it's added the first time, and wires it into the working-agreement-mode table and the quick reference. Docs-only diff — opened ready, not draft, per this file's own convention.
A PR marked ready is one-way today, so a PR that gets reworked after review
(a design change, a bug needing more than a patch, a coverage gap) keeps
sitting "ready" while it's actually not — inviting re-review of unfinished
work. Document re-adding the WIP: prefix for genuine rework, distinct from a
quick fix to a red check, and add it to the working-agreement-mode table and
quick reference.
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Grey.ooo/agent-standards!32
No description provided.