diff --git a/README.md b/README.md index 356eb5f..258cf0f 100644 --- a/README.md +++ b/README.md @@ -144,9 +144,9 @@ rm -rf ~/.config/opencode/skills/code-review | 助理 | headless 指令 | 執行 `code-review` skill | | --- | --- | --- | -| Claude Code | `claude -p ""` | `claude -p "/jsc:code-review main feature/login all"` | -| Codex | `codex exec ""` | `codex exec '$code-review main feature/login all'` | -| Antigravity | `agy -p ""` | `agy -p "/jsc:code-review main feature/login all"` | +| Claude Code | `claude -p ""` | `claude -p "/jsc:code-review main feature/login all --exclusions .codereview.md"` | +| Codex | `codex exec ""` | `codex exec '$code-review main feature/login all --exclusions .codereview.md'` | +| Antigravity | `agy -p ""` | `agy -p "/jsc:code-review main feature/login all --exclusions .codereview.md"` | | OpenCode | `opencode run ""` | `opencode run "用攻防角色 review main 與 feature/login 的差異"` | - Claude / Antigravity 支援 `/jsc:` 前綴，直接 `-p "/jsc:"` 即可。 @@ -167,11 +167,11 @@ rm -rf ~/.config/opencode/skills/code-review 以 RPG 攻防對決方式審查 `git diff` 的程式碼審查 skill。審查範圍是兩個分支的差異，**來源分支與目標分支缺一不可**（缺漏會反問補齊）。角色分**攻擊方**（吟遊詩人=風格 🎼／法師=邏輯 🔮／盜賊=效率 ⚡／刺客=安全性 🗡️）與**防守方**（聖騎士=裁決 🛡️），每個角色定義在 `skills/code-review/roles/.md`（含英文名稱／專案／個性／徽章／代表色）。攻擊方分析 diff 找出問題（問題／等級／描述／建議／檔案位置／所在行數）；防守方依專案根目錄排除事項設定檔與原始碼脈絡裁決每條問題（🚫 略過／❌ 誤判／✅ 成立）。使用者可選擇單一角色、整個攻擊方、整個防守方或全部；複選時以 sub agent 並行執行。 -參數格式：` [角色...]`（目標在前、來源在後；角色可省略，會詢問）。 +參數格式：` [角色...] [--exclusions <排除事項檔案路徑>]`（目標在前、來源在後；角色可省略，會詢問；`--exclusions` 指定排除事項檔案路徑，省略時若選到防守方會反問）。 -- **Claude Code / Antigravity**：`/jsc:code-review`（反問分支與角色），或帶參數 `/jsc:code-review main feature/login`、`/jsc:code-review main feature/login mage`、`/jsc:code-review main feature/login all` -- **Codex**：`$code-review main feature/login attack`（可接參數），或用 `/skills` 選單 -- **OpenCode**：描述需求（如「用攻防角色 review main 與 feature/login 的差異」）自動觸發 +- **Claude Code / Antigravity**：`/jsc:code-review`（反問分支與角色），或帶參數 `/jsc:code-review main feature/login`、`/jsc:code-review main feature/login mage`、`/jsc:code-review main feature/login all --exclusions .codereview.md` +- **Codex**：`$code-review main feature/login attack`，或指定排除事項 `$code-review main feature/login all --exclusions docs/review-rules.md`，或用 `/skills` 選單 +- **OpenCode**：描述需求（如「用攻防角色 review main 與 feature/login 的差異，排除事項看 docs/review-rules.md」）自動觸發 diff --git a/skills/code-review/SKILL.md b/skills/code-review/SKILL.md index c2e7386..1170ba4 100644 --- a/skills/code-review/SKILL.md +++ b/skills/code-review/SKILL.md @@ -37,7 +37,8 @@ description: 以 RPG 攻防對決方式審查 git diff 的程式碼審查 skill ### 2. 選擇角色 -slash 參數格式：`/jsc:code-review [角色...]`（角色接在兩個分支之後）。 +slash 參數格式：`/jsc:code-review [角色...] [--exclusions <排除事項檔案路徑>]` +（角色接在兩個分支之後；`--exclusions` 旗標可放在任意位置，指定排除事項檔案路徑）。 - **若已指定角色** → 直接採用。可接受： - 單一面向：`bard` / `mage` / `rogue` / `assassin` / `paladin` @@ -51,7 +52,10 @@ slash 參數格式：`/jsc:code-review [角色...]`（角色 排除事項設定檔位於**專案根目錄**（建議檔名 `.codereview.md`，列出已知技術債／團隊慣例／刻意取捨）。 -- **若不知道排除事項檔案的位置 → 必須詢問使用者**（提供建議預設 `.codereview.md`）。 +- **若 slash 參數帶了 `--exclusions <路徑>`** → 即為使用者明確指定，直接採用該路徑，**不需再問**。 +- **否則只要使用者沒有明確告知檔案路徑 → 一律先詢問**。預設檔名 `.codereview.md` 僅作為詢問時的**建議選項**， + **不可**在未取得使用者明確指定前自行假設或直接採用該預設路徑。 +- 使用者明確指定路徑（透過 `--exclusions`，或在詢問中確認用預設 / 確認無檔）後才繼續。 - **檔案允許不存在或為空** → 視為「無排除事項」，防守方直接進入原始碼判斷，**不**因缺檔而中斷。 ### 4. 攻擊方審查 @@ -104,10 +108,11 @@ slash 參數格式：`/jsc:code-review [角色...]`（角色 ## 呼叫方式 -格式：` [角色...]` — **來源/目標分支缺一不可**，缺漏會反問補齊；角色可省略（會詢問）。 +格式：` [角色...] [--exclusions <排除事項檔案路徑>]` — **來源/目標分支缺一不可**，缺漏會反問補齊； +角色可省略（會詢問）；`--exclusions` 可省略（省略時若選到防守方會反問檔案路徑）。 | 助理 | 呼叫 | | --- | --- | -| Claude Code / Antigravity | `/jsc:code-review`（反問分支與角色），或 `/jsc:code-review main feature/login`、`/jsc:code-review main feature/login mage`、`/jsc:code-review main feature/login all` | -| Codex | `$code-review main feature/login attack`（可接參數），或用 `/skills` 選單 | -| OpenCode | 描述需求（如「幫我用攻防角色 review main 與 feature/login 的差異」）自動觸發 | +| Claude Code / Antigravity | `/jsc:code-review`（反問分支與角色），或 `/jsc:code-review main feature/login`、`/jsc:code-review main feature/login mage`、`/jsc:code-review main feature/login all --exclusions .codereview.md` | +| Codex | `$code-review main feature/login attack`，或指定排除事項 `$code-review main feature/login all --exclusions docs/review-rules.md`，或用 `/skills` 選單 | +| OpenCode | 描述需求（如「用攻防角色 review main 與 feature/login 的差異，排除事項看 docs/review-rules.md」）自動觸發 | diff --git a/skills/code-review/roles/paladin.md b/skills/code-review/roles/paladin.md index 0ea8cb3..6694c74 100644 --- a/skills/code-review/roles/paladin.md +++ b/skills/code-review/roles/paladin.md @@ -22,9 +22,10 @@ personality: 沉穩公正、就事論事，不護短也不冤枉，只依排除 排除事項設定檔位於**專案根目錄**（建議檔名 `.codereview.md`，列出已知技術債／團隊慣例／刻意取捨）。 -1. **若不知道排除事項檔案的位置 → 必須先詢問使用者**（提供建議預設 `.codereview.md`）。 +1. **只要使用者沒有明確告知檔案路徑 → 一律先詢問**。預設檔名 `.codereview.md` 僅是詢問時的**建議選項**， + **不可**在未取得使用者明確指定前自行假設或直接採用該預設路徑。 2. **檔案允許不存在或為空** → 視為「無排除事項」，直接進入原始碼判斷，不因缺檔而中斷。 -3. 取得位置（或確認無檔）後，以該檔內容作為排除事項依據。 +3. 取得使用者明確指定的路徑（或在詢問中確認用預設 / 確認無檔）後，才以該檔內容作為排除事項依據。 ## 裁決準則（對攻擊方的每一條 finding 依序處理）