@@ -15,10 +15,12 @@ Return exactly five PR URLs, each with:
15151616- bug summary
1717- why the fix is low-risk
18-- proof: local/CI/Testbox/live commands or run IDs
18+- proof: rebased-head local/Testbox/live commands or run IDs
19+- CI green on the exact pushed PR head
1920- issue/duplicate cleanup done or still pending
20212122The five URLs may be existing PRs that were reviewed/fixed, or new PRs created from issues/clusters.
23+Do not present a PR as one of the five until it has been refreshed on current `main`, left-tested, pushed, and verified green in live GitHub CI.
22242325## Companion Skills
2426@@ -40,6 +42,7 @@ Accept only when all are true:
4042- no plugin SDK, public plugin API, or `src/plugin-sdk/**` boundary change
4143- no broad refactor smell
4244- focused proof is feasible
45+- branch can be rebased/refreshed and pushed, or a replacement PR can be created
43464447Good examples:
4548@@ -54,6 +57,8 @@ Reject:
5457- plugin SDK/API boundary changes, including compatibility shims, new SDK methods, SDK exports, or plugin-facing channel/provider seams
5558- auth/security boundary changes unless explicitly assigned
5659- bugs needing live credentials that are unavailable
60+- PRs with red CI unless you fix, rebase, push, and recheck them green
61+- PRs you only reviewed locally but did not refresh/push/check live
5762- fixes whose clean shape is a larger architecture move
5863- speculative reports without reproducible/provable cause
5964- UI/UX changes requiring product judgment
@@ -82,14 +87,16 @@ Reject:
8287- if no PR exists, create one
8388- add regression test when it fits
8489- changelog for user-facing fixes; thank credited human reporter/contributor
85-6. Review and publish:
86-- run focused proof
90+6. Review, refresh, and publish:
91+- rebase or otherwise refresh the PR branch on current `origin/main`
92+- resolve drift, including newly exposed CI failures, rather than counting the PR as ready
93+- left-test the rebased head with the smallest meaningful local/Testbox/live command that proves the bug
8794- run `$autoreview` until no accepted/actionable findings remain
8895- create/update PR with real body and proof fields
89-- push branch
90-- get CI green or document exact external blocker; do not count blocked PRs in the five
96+- push the exact reviewed head
97+- verify live GitHub CI is green for that pushed head; do not count pending, red, dirty, conflicting, or externally blocked PRs in the five
91987. Hygiene:
92-- close duplicates and fixed-on-main issues with proof
99+- close duplicates and fixed-on-main issues/PRs with proof as soon as you notice them during the sweep
93100- never mutate more than five associated items in one cluster without explicit confirmation
94101- comments must be kind, concrete, and include proof/PR/commit links
951028. Repeat until five landable PR URLs are ready.
@@ -110,10 +117,11 @@ What was not tested:
110117## Existing PR Rules
111118112119- Review code path beyond the diff before trusting it.
113-- If PR is good: fix small issues, run proof, push if writable, get CI green.
120+- If PR is good: rebase/refresh on current `main`, fix small issues, left-test, autoreview, push, and get CI green before counting it.
114121- If PR is not good but has a useful idea: recreate locally, co-author when warranted, close original with thanks and explanation.
115122- If PR is duplicate or fixed on `main`: comment proof, close.
116123- If maintainer cannot push to contributor branch: create own branch/PR, preserve useful commits or credit.
124+- If CI turns red after local proof, treat that as normal work: inspect the failing job, fix or reject, rerun, and only count the PR once green.
117125118126## Output Ledger
119127@@ -127,7 +135,9 @@ accepted:
127135 root cause:
128136 fix:
129137 risk:
130- proof:
138+ rebase/head:
139+ left-test:
140+ autoreview:
131141 CI:
132142 credit/thanks:
133143 cleanup:
此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。 原文来自 — 版权归原作者所有。