Skip to content

docs: document the one-PR-per-change archive convention - #22

Merged
turnipdabeets merged 1 commit into
mainfrom
anna/document-one-pr-per-change
Jul 31, 2026
Merged

docs: document the one-PR-per-change archive convention#22
turnipdabeets merged 1 commit into
mainfrom
anna/document-one-pr-per-change

Conversation

@turnipdabeets

Copy link
Copy Markdown
Collaborator

Writes down the workflow rule that a change should land as one PR: once review approves the proposal, run apply + archive on the same branch, so the PR merges already archived — canonical spec synced, capability lists updated, archive-time tasks checked.

Until now the docs left this ambiguous and both patterns exist in history: #12 merged already archived, while #14#15 and #13#21 split propose and archive into two PRs. The split turned out to be a mistake — it leaves a merged proposal in a half-done state, invites review questions about unchecked archive-time task boxes (as happened on #13), and doubles the PR overhead for no benefit.

The rule goes in two places on purpose:

  • openspec/project.md Conventions — this file is injected into every AI-agent proposal/apply/archive run, so agents working in this repo will follow the one-PR flow automatically
  • README.md workflow section — for human contributors

🤖 Generated with Claude Code

https://claude.ai/code/session_01JJ7w12bnM5JqcDD7pEEqhQ


Generated by Claude Code

@turnipdabeets
turnipdabeets merged commit 50a99d8 into main Jul 31, 2026
10 checks passed
@turnipdabeets
turnipdabeets deleted the anna/document-one-pr-per-change branch July 31, 2026 12:06
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.

3 participants