1# hooks/pre_commit.py
 2def run_git_command(cmd):
 3    result = subprocess.run(cmd, capture_output=True, text=True)
 4    return result.stdout.strip()
 5
 6# hooks/post_merge.py -- 完全相同的程式碼
 7def run_git_command(cmd):
 8    result = subprocess.run(cmd, capture_output=True, text=True)
 9    return result.stdout.strip()
10
11# hooks/branch_check.py -- 又是一模一樣
12# hooks/worktree_guardian.py -- 第四份...

1def parse_worktree_line(line):
2    if line.startswith("worktree "):
3        return line[9:]  # 三個月後，你還記得 9 是什麼嗎？

 1# 一個 800+ 行的 Hook 檔案
 2PROTECTED_BRANCHES = ["main", "master", "develop"]
 3ALLOWED_PATTERNS = ["feat/*", "fix/*", "chore/*"]
 4ERROR_MESSAGES = {
 5    "branch_not_allowed": "分支名稱不符合規範",
 6    "missing_ticket": "缺少 Ticket 引用",
 7    # ... 數十行配置繼續
 8}
 9
10def check_branch():
11    # 真正的邏輯只有幾十行，卻埋在幾百行配置之下
12    pass

 1# 修正前：logger 是全域變數，所有函式都看得到
 2logger = setup_hook_logging("hook-name")
 3
 4def helper_function():
 5    logger.info("doing something")  # OK，全域可見
 6
 7def main():
 8    result = helper_function()
 9
10# 修正後：logger 搬進 main()，但 helper 沒收到通知
11def helper_function():
12    logger.info("doing something")  # NameError! logger 不見了
13
14def main():
15    logger = setup_hook_logging("hook-name")  # 現在是區域變數
16    result = helper_function()  # helper 找不到 logger

 1# 偵測 IMP-001：找出重複的函式定義
 2grep -rh "^def " hooks/*.py | sort | uniq -c | sort -rn | head -10
 3#  4 def run_git_command(cmd):    <-- 出現 4 次，高度疑似重複
 4#  2 def parse_output(line):      <-- 出現 2 次，需要確認
 5
 6# 偵測 IMP-002：找出魔法數字
 7grep -rn -E "\[[0-9]+:\]" hooks/*.py      # 數字切片 [9:]
 8grep -rn "sleep([0-9]" hooks/*.py        # 硬編碼的等待時間
 9grep -rn "range([0-9]" hooks/*.py        # 硬編碼的迴圈次數
10
11# 偵測 ARCH-001：找出超長檔案
12find hooks/ -name "*.py" -exec wc -l {} \; | awk '$1 > 500'
13# 847 hooks/user_prompt_submit.py    <-- 紅色警報
14
15# 偵測 IMP-005：模組遷移後殘留的舊 Import
16grep -rn "from common_functions import" hooks/*.py
17# 如果 common_functions.py 已經搬到 lib/，這些都是未更新的引用

 1import ast
 2import sys
 3
 4def find_scope_references(filename, variable_name):
 5    """找出所有在非 main 函式中引用特定變數的位置。
 6    限制：此函式只做名稱比對，無法追蹤賦值或閉包捕獲。"""
 7    with open(filename) as f:
 8        tree = ast.parse(f.read())
 9
10    issues = []
11    for node in ast.walk(tree):
12        if isinstance(node, ast.FunctionDef) and node.name != "main":
13            param_names = {arg.arg for arg in node.args.args}
14            if variable_name in param_names:
15                continue  # 函式已接收此變數為參數，非問題
16            for child in ast.walk(node):
17                if isinstance(child, ast.Name) and child.id == variable_name:
18                    issues.append(f"  {node.name}() 在第 {child.lineno} 行引用 {variable_name}")
19                    break
20    return issues
21
22# 使用方式
23issues = find_scope_references("hooks/pre_commit.py", "logger")
24for issue in issues:
25    print(issue)

 1# py_compile：語法 OK 不代表能跑
 2python3 -m py_compile hooks/pre_commit.py  # 通過！但 logger 根本找不到
 3
 4# pylint：能抓到更多問題
 5pylint hooks/pre_commit.py
 6# E0602: Undefined variable 'logger' (undefined-variable)
 7
 8# 實際執行：最可靠的驗證
 9python3 hooks/pre_commit.py < /dev/null
10# NameError: name 'logger' is not defined

 1問題：單一 Hook 檔案超過 800 行，其中約一半是硬編碼的配置資料
 2
 3Why 1: 為什麼檔案會有 800 行？
 4--> 因為配置資料（分支規則、錯誤訊息、檔案模式）和程式邏輯
 5    全部寫在同一個檔案中
 6
 7Why 2: 為什麼配置和邏輯放在一起？
 8--> 因為開發時為求快速，直接在程式碼中定義配置常數
 9
10Why 3: 為什麼選擇快速做法而非分離？
11--> 因為缺乏配置管理策略，沒有標準化的做法可以遵循
12
13Why 4: 為什麼沒有配置管理策略？
14--> 因為 Hook 系統初期設計時，只考慮了功能實現，
15    沒有考慮到配置會不斷增長
16
17Why 5: 為什麼初期設計沒考慮配置增長？
18--> 【根本原因】缺乏明確的架構原則指導配置與程式碼分離

1.claude/error-patterns/
2├── README.md              # 系統說明與索引
3├── test/                  # 測試相關：TEST-001, TEST-002, ...
4├── documentation/         # 文件相關：DOC-001, DOC-002, ...
5├── architecture/          # 架構相關：ARCH-001, ARCH-002, ...
6└── implementation/        # 實作相關：IMP-001, IMP-002, ...

 1# [Pattern ID]: [簡短標題]
 2
 3## 基本資訊
 4
 5- **Pattern ID**: {CATEGORY}-{NNN}
 6- **風險等級**: 高/中/低
 7- **發現日期**: YYYY-MM-DD
 8
 9## 問題描述
10
11### 症狀
12
13[用程式碼範例展示問題的外在表現]
14
15### 根本原因 (5 Why 分析)
16
171. Why 1: ...
182. Why 2: ...
193. Why 3: ...
204. Why 4: ...
215. Why 5: (根本原因)
22
23## 解決方案
24
25### 正確做法
26
27[程式碼範例]
28
29### 錯誤做法 (避免)
30
31[程式碼範例]
32
33## 檢測方法
34
35[grep 指令、AST 腳本或工具配置]

 1發現壞味道
 2    |
 3    v
 4影響正確性嗎？（會導致 Bug）
 5    |
 6    +-- 是 --> 立即修復，建立 Ticket
 7    |
 8    +-- 否 --> 影響多個檔案嗎？
 9                |
10                +-- 是 --> 記錄 Error Pattern + 建立 Ticket
11                |
12                +-- 否 --> 認知負擔高嗎？（函式超長、巢狀太深）
13                            |
14                            +-- 是 --> 排入下次重構
15                            |
16                            +-- 否 --> 記錄，暫不處理

 1BRANCH_RULES = {
 2    "protected": ["main", "master"],
 3    "max_length": 50,
 4    "patterns": ["feat/*", "fix/*", "chore/*"],
 5}
 6
 7def check(data):
 8    res = []
 9    for i in range(len(data)):
10        if data[i]["type"] == "A":
11            if data[i]["status"] == 1:
12                if data[i]["value"] > 100:
13                    res.append(data[i]["name"][5:])
14    return res