菜单
審查本專案 (Manifest V3 Chrome extension) 的程式碼變更，含 Chrome API 用法、權限最小化、Puppeteer 測試 race condition、scripts/lint-check.sh 自動化檢查、references/{security-checklist,performance-patterns,style-guide}.md。當使用者提到「review PR、審查、code review、程式碼審查、檢查 PR、看一下這個 PR」時觸發。
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
建立本專案的 Pull Request：雙語描述 (zh-TW/en)、gh CLI、.agent/skills/pull-request/scripts/check-pr.sh 預驗證、references/pr_template.md 與 checklist.md。當使用者提到「開 PR、建 PR、create PR、pull request、提交變更、發 PR、新建 PR」時觸發。
本專案 (MV3 Chrome extension) 的除錯紀律與 playbook：先重現再修、證據優先於模式匹配、單變因修改、跨 context 時序問題、service worker 終止、storage 問題、flaky E2E 分流。當使用者提到「除錯、debug、壞掉、不會動、修不好、查原因、root cause、flaky、時好時壞」時觸發。
維護本專案 AI agent harness（AGENTS.md / CLAUDE.md / GEMINI.md / .agent/{rules,skills,workflows} / docs/wiki）的 SOP：單一事實來源對照表、drift 稽核步驟、新增/刪除 skill 的判準、wiki 同步機制。當使用者提到「更新 harness、同步 AGENTS、文件過時、drift、稽核設定檔、整理 skills、維護 wiki」時觸發。
交付前驗證紀律：依變更類型選擇最小充分驗證組合（unit / 目標 E2E / happy-path CI / make build）、Makefile 打包檢查、i18n 連動、GEMINI.md key_files 同步、誠實回報。當使用者提到「驗證、收尾、完成了嗎、可以交付嗎、送 PR 前、檢查一下」或任何實作即將收尾時觸發。
清理已合併進 main 的本地與遠端 git 分支。本專案採 squash merge，PR 合併後特性分支可直接刪除。當使用者提到「清理分支、cleanup branches、刪除舊分支、清掉已合併分支、整理 branch」時觸發。
本專案 (Manifest V3 Chrome extension) 的 commit 規範與範例：subject 使用英文 Conventional Commits、body 使用繁中說明背景與原因。當使用者提到「commit、提交訊息、commit message、寫 commit、commit 規範」時觸發。
| name | code-review |
| description | 審查本專案 (Manifest V3 Chrome extension) 的程式碼變更，含 Chrome API 用法、權限最小化、Puppeteer 測試 race condition、scripts/lint-check.sh 自動化檢查、references/{security-checklist,performance-patterns,style-guide}.md。當使用者提到「review PR、審查、code review、程式碼審查、檢查 PR、看一下這個 PR」時觸發。 |
當進行程式碼審查時，請遵循以下步驟：
由於這是一個 Chrome 擴充功能專案，額外檢查以下項目：
chrome.tabs、chrome.bookmarks 等 API審查 Puppeteer 測試時，特別注意潛在的 race condition：
| 反模式 ❌ | 正確模式 ✅ | 說明 |
|---|---|---|
await new Promise(r => setTimeout(r, 2000)) | await page.waitForFunction(...) | 固定延遲會在 CI 環境不穩定 |
setTimeout(r, 500) debounce | waitForSelector + visibility | 等待實際 DOM 狀態 |
| 等待 active tab URL 精確匹配 | 檢查 tab count 或使用 includes | Headless 環境 navigation 可能被阻擋 |
| 直接 click 後立即驗證 | waitForFunction 後再驗證 | 給予 Chrome API 回應時間 |
// ✅ 等待 tab 出現在 DOM
await page.waitForFunction(
(id) => document.querySelector(`.tab-item[data-tab-id="${id}"]`),
{ timeout: 10000 },
createdTabId
);
// ✅ 等待 modal 關閉
await page.waitForFunction(
() => !document.querySelector('.modal-content'),
{ timeout: 5000 }
);
// ✅ 等待 tab count 增加
await page.waitForFunction(
(expected) => {
return new Promise(resolve => {
chrome.tabs.query({}, tabs => resolve(tabs.length >= expected));
});
},
{ timeout: 10000 },
initialTabCount + 1
);
在開始人工審查之前，先執行 lint 檢查腳本以快速發現常見問題：
.agent/skills/code-review/scripts/lint-check.sh .
| 檢查項目 | 嚴重程度 | 說明 |
|---|---|---|
console.log | ⚠️ 警告 | 殘留的除錯語句 |
innerHTML | ⚠️ 警告 | 潛在 XSS 風險 |
eval() | ❌ 錯誤 | 嚴重安全風險 |
TODO/FIXME | 📝 資訊 | 未完成項目 |
| 未處理的 Promise | ⚠️ 警告 | 可能的錯誤遺漏 |
setTimeout 固定等待 | ⚠️ 警告 | 測試中使用可能造成 CI race condition |
詳細的審查標準請參考：