Unified · Split
| @@ -114,7 +114,6 @@ An empty database is a bad test. Seed your worktree's `.t3` with a copy of real · 113 unmodified lines | |||
| 114 | - Never make a PR unless the developer explicitly asks you to do so. | 114 | - Never make a PR unless the developer explicitly asks you to do so. |
| 115 | - Conventional commit titles, plain language: `fix(web): new threads no longer spike CPU`. | 115 | - Conventional commit titles, plain language: `fix(web): new threads no longer spike CPU`. |
| 116 | - Body: the problem in a sentence or two, then how you fixed it. End with the model and harness that did the work. | 116 | - Body: the problem in a sentence or two, then how you fixed it. End with the model and harness that did the work. |
| 117 | - **Rebase onto latest main before opening.** Stale branches conflict and burn a review round. | ||
| 118 | - UI changes need before/after images. Motion or timing needs a short video. | 117 | - UI changes need before/after images. Motion or timing needs a short video. |
| 119 | - One concern per PR. If the description says "also", split it. | 118 | - One concern per PR. If the description says "also", split it. |
| 120 | - When babysitting: poll checks and comments newer than the last push, verify each bot finding against the source, fix real ones, dismiss false positives with a written reason. Stay quiet when nothing is new. Stop when the bots are green on the latest commit. | 119 | - When babysitting: poll checks and comments newer than the last push, verify each bot finding against the source, fix real ones, dismiss false positives with a written reason. Stay quiet when nothing is new. Stop when the bots are green on the latest commit. |