creating-pull-requestslisted
Install: claude install-skill dork-labs/dorkos
# Creating Pull Requests
How DorkOS PRs are opened and how the automated Claude review behaves on them.
This repo is routinely multi-agent, so the mechanics below keep PRs clean and the
review loop cheap.
## When to use
- You have finished a branch and are deciding when to open the PR.
- You are about to open a PR (from an agent or by hand).
- A PR already has review feedback and you want another pass once it is addressed.
- You want to dial a PR's review up, down, or off.
## The order: review the branch, then open the PR
The independent adversarial review runs **against a pushed branch, before a PR
exists**. Opening the PR is the last step, not the first:
1. Build in an isolated worktree and run the local gates.
2. Push the branch. **Open nothing.**
3. A reviewer fetches and checks out that branch. It does not need a PR.
4. Findings, fixes, convergence.
5. Squash to one clean commit carrying one changelog fragment.
6. **Then** open the PR, already reviewed, so the repo's automated review spends
its single pass on final content.
Every reason below is a cost measured on 2026-07-27/28, not a preference:
- **Merge churn.** A PR held open across review rounds watches `main` move under
it. One night of that took roughly a dozen `gh pr update-branch` calls and still
left a `DIRTY` PR needing a semantic conflict resolved by hand.
- **The changelog gate.** A fragment claims the commits that existed when it was
written. Review-fix commits arrive afterwards uncovered,