 Leo
|
67dae3b814
|
推送閘改成判目標，不判整條指令裡有沒有那個字
一個晚上誤攔六次，全都不是在推預設分支：
① checkout -b 建新分支時把預設分支寫在後面，再推那條新分支
② gh pr create 指定 base——根本不是 git push
③ 推 tag（refs/tags/…）
④ 推 feature 分支（帶 -u）
⑤ 它擋住了我用來**測試它自己**的那條指令
⑥ 它擋住了這一筆的 commit——因為 message 裡引用了那幾個字
leo 2026-08-17 早就講過這個形狀：文字層封路必敗，
「紅線寫得越細，命中關鍵字的機率越高 ⇒ 那些閘在懲罰謹慎」。
舊版掃整條指令字串，正是文字層。
改成解析 push 的目標 refspec：
旗標跳過／第一個非旗標＝remote／a:b 取 b／refs/tags/* 不算分支
一個 refspec 都沒給，才退回看當前分支
八向實測（hooks/tests/main-and-prod-push-guard.test.sh，8/8）：
五種該放行的（今晚誤攔的原形狀，含分支名帶 domain 那種）全過
三種該擋的全擋
中途自己抓到一個 bug：tag 被跳過後目標清單變空 → 退回猜當前分支
⇒ 當前分支剛好叫預設名時誤擋。改成看到 refspec 就不退回猜測。
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
2026-08-21 01:29:55 +08:00 |
|
|
Leo
|
87186585d6
|
fix(hooks): sdd-guard.sh 修「解析失敗仍照擋、且訊息洩漏 /nonexistent」（InkStoneCo#22）
症狀（總管 2026-08-12 實撞）：寫暫存腳本進 scratchpad
（/private/tmp/.../scratchpad/foo.py）被 sdd-guard.sh 攔下，訊息印出字面的
「/nonexistent/3-specs/ 下找不到任何 SDD」。
兩個洞：
- 洞 A：scratchpad 不在任何 git repo 裡，卻被當成「repo 裡的 code 變動」誤判
需要 SDD。改成先問 path_in_git_worktree()（見 hooks/lib/path-resolve.sh）：
不在任何 git repo 裡 → SDD 天生管不到，直接放行，不必先猜專案根。
這個檢查放在 $_root 的 case 分岔之前、對兩邊都適用——第一版只放進「專案外」
分支，被本次新增的 hooks/tests/sdd-guard.test.sh 抓到一個不對稱漏洞（cwd 剛好
等於 scratchpad 祖先目錄時會漏判），改成統一檢查後修掉。
- 洞 B：舊版用內部 sentinel `/nonexistent/3-specs` 重用既有的擋下路徑，但這個
假路徑被直接印進使用者看到的訊息。改用 RESOLVED 旗標記解析成不成功，訊息
改用人話描述原因，不洩漏假路徑。
fail-closed / fail-open 的判準（票上明確要求回答，不能各憑運氣）：
真的落在某個 git repo 裡、但那個 repo 沒有 3-specs（或沒有 active SDD）→
仍然 fail-closed（擋）。理由：這道閘存在的目的就是防止「沒有 SDD 卻能動
code」，把「判斷不出來」直接放行，等於把環境跑歪（cwd 被切走、
$CLAUDE_PROJECT_DIR 沒設）悄悄變成「這道閘關掉了、且沒人知道」——silent
bypass 的代價遠高於多打一次確認。#22 紅線亦明寫「不要把閘改成解析失敗就
放行」。
順手修的殘留 cwd 依賴：SPECS_DIR 的預設值原本是相對路徑
「system-dev/docs/3-specs」，專案內迴圈找不到時會被拿去跟 hook 執行當下的
cwd 兜；改成絕對路徑 $_root/system-dev/docs/3-specs。
同時修 ADR-0001（ISEP 自建 wiki）：標題與內文原本會讓人誤解成「ISEP plugin
裝到哪個 repo，就會在那裡自建一份 wiki」，但實際查證（marketplace.json 只宣告
hooks/commands/skills、README 明文排除 wiki/docs、hooks 一律用
${CLAUDE_PLUGIN_ROOT} 讀自己不是寫別處）並非如此——那份 wiki 只是 ISEP 這個
repo自己的開發歷史，跟裝 plugin 無關。唯一真的會在某 repo 建 wiki 的
scripts/install.sh 是 system-dev-template 的獨立安裝器殘留，要手動執行，
作用對象是 cwd 不是「plugin 裝到的地方」——這多半是誤解的真正來源，已在
ADR 的「常見誤解」段說明。
驗證：
- 造出 08-12 原始事故情境（cwd=InkStoneCo、CLAUDE_PROJECT_DIR 未設、寫
scratchpad），修前擋（印 /nonexistent）、修後放行——實測輸出見票留言。
- 造出「真的在 git repo 裡但沒有 3-specs」情境，修後仍擋、訊息不含
/nonexistent。
- 新增 hooks/tests/sdd-guard.test.sh：8 案例全過（洞 A／洞 B／fail-open
陷阱／單一活性違反／恰好一份 active／改文件放行）。
- 既有六套 scripts/test-*.sh 全過，無退步。
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
2026-08-20 21:03:14 +08:00 |
|
|
Leo
|
c1d80756d7
|
審核補件：第二個空殼閘（irreversible_dispatch_check.py）＋既有測試套
總管複驗 PR 時發現 subagent 只補了一個空殼，還有第二個同款的：
arcrun-intent-guard.sh → exec 不存在的 .py ⇒ 擋掉全部（吵，PR 已修）
irreversible-dispatch-guard.sh → 同款，但寫法是
python3 <不存在> 2>/dev/null || echo '{"verdict":"OK"}'
⇒ **靜默放行全部**（危險，本 commit 補）
實測同一份派工單「驗過了就把舊分支刪掉」：
InkStoneCo 版（有 .py）exit=2 擋 ／ ISEP 版（缺 .py）exit=0 放行
補完後：不可逆派工 exit=2、正常派工 exit=0、合規工作流 exit=0。
系統性掃描 ISEP 全部 hooks 引用的同目錄檔案：補完後 0 個缺檔（InkStoneCo 本來就是 0）。
順帶把 InkStoneCo 的 hooks/tests/ 四支既有測試一併帶過來。
🔴 這件事很重要：若先併 InkStoneCo#64（刪掉 .claude/hooks/），
這台機器會失去那支唯一還能用的副本——變成真的沒有那道閘。
|
2026-08-20 20:06:23 +08:00 |
|