swe-knowledge

# swe-knowledge — 軟體工程的 Definition of Done 這裡放的是**所有增量共用**的完成條件。跟每張單獨有的驗收條件（acceptance criteria）分得很 開：AC 寫在那張單凍結的 assertion 裡，DoD 寫在這裡。**一張單可以通過全部 AC 卻沒有 done**—— assertion 全綠但改動還躺在預設分支上、沒有任何人看得到它，就是那個情況。 所以這幾條不要抄進任何一張單的凍結區。抄進去等於每張單都重簽一次同樣幾行不承載新資訊的 東西，而漏抄的那一張就靜默地少了一條完成條件。 ## 五條 1. **改動住在一條 branch 上，不躺在預設分支。** 開的時機是「判定要立案之後、動手之前」—— 一個還沒開工的成功定義直接躺在預設分支上，等於它已經是既成事實。 **那條 branch 是自己建的。** 改動只推到自己建的 branch。別人建的、或有別人的 PR 在用的 branch，任何情況都不推——推不推得動、用不用 force 都一樣。要在別人還沒併的 branch 之上 做事，從它切一條自己的，PR 的 base 指向它。 **所以別人的 branch 推不動的時候，那不是要解決的問題。** 它從一開始就不是推的對象， 沒有「換個方法推上去」這一題。2026-09-10 的實例：一顆 PR 的 base 是同事還沒併的 branch，遠端規則讓兩條都推不動，討論裡出現了「那條也一樣推不動」——那條本來就不是我們 會去推的東西。 2. **有一個 PR。** **PR 開出來就是實作完成**：它是那份改動變成可被別人看見、可被判定的 東西的那一刻。沒有 PR 的改動不管本機多綠都還沒 done。 3. **判定過才進預設分支。** 進去的路徑是那個 PR，不是直接推。 4. **push 之前，本機跑完跑得動的驗證。** type check、lint、單元測試、受影響路徑的冒煙。 把 CI 當第一道防線等於把 reviewer 當驗證工具。 5. **reviewer 提的每一條，處置回到那條意見上。** 不是「有沒有處理」——是**提出者拿不拿 得到那個處置**。他看的是他留言的地方，回在別處他收不到。 **處置有四種：照做、回覆說明為什麼不照做、拒絕、另外開單。照做不是預設。**「處置」這個 字是中性的，而它常被讀成「照做」。一條意見可以是對的、而仍然不該在這一輪做；也可以是 這個 repo 根本不做那件事。讀成「照做」的話，這一條會把每一則意見都變成一次施工。 **每一串都算，nit 也算。** 「妥善處理」的量法是：每一串的最後一則是作者，而且那一則說出了 四種處置裡的哪一種。標成 nit 的意見只是提出者在說「這條不擋你」，不是在說「這條不用回」。 第 2 條有一個這個 repo 自己的教訓：2026-08-03，「開 PR」這個能力在腳本歸位時被刪掉，因為 它在三站裡沒有主人；43 分鐘後有人寫了一句跟腳本矛盾的散文把洞蓋住，於是「PR 算不算完成」 有兩個互相矛盾的答案在流通。現在它有主人了，就是這一條。 第 5 條是同一個形狀又發生了一次。2026-08-11：一個 PR 收到三位 reviewer 的意見，全部查證、 修掉、assertion 重跑全綠，也回了另一個系統的訊息串——**但那個 PR 上一則回覆都沒有**，是人自己 發現的（「我好像...