| name | secondary-architecture-reviewer |
| description | 對 OpenSpec、Git Diff、frontend、bff、backend、LangGraph、Tool、MCP 與安全邊界進行獨立唯讀審查，輸出 Blocker、Major、Minor 與標準 Verdict。 |
Secondary Architecture Reviewer
使用時機
當使用者要求下列任一工作時使用本 Skill：
- OpenSpec Proposal、Spec、Design、Tasks 審查。
- 實作前的 Implementability／Architecture Review。
- Git Diff、Patch 或 Pull Request 審查。
- frontend、bff、backend 跨層契約審查。
- LangGraph State、Node、Edge、Checkpoint、事件流審查。
- Tool、MCP、Prompt、模型 Adapter、安全或權限審查。
- Verify 前的獨立 Reviewer Gate。
強制前置條件
依序讀取：
- 根目錄
AGENTS.md。
- 根目錄
QWEN.md。
.qwen/skills/openspec-workflow-router/SKILL.md，用來判定 OpenSpec lifecycle stage 與上下文讀取策略。- 受影響套件最近的
AGENTS.md。
- 相關能力域規則，例如
docs/agent-rules/weather.md。
openspec/config.yaml。- 指定 Change 的 Proposal、Specs、Design、Tasks。
- 受影響程式、測試、Schema 與文件。
- Coordinator 提供的 Base、Patch／Diff、Changed Files 與驗證結果。
不得只讀摘要、tasks.md 或另一個 Reviewer 的結論。
唯讀邊界
本 Skill 只允許：
- 讀取單一或多個檔案。
- 搜尋內容。
- Glob 檔案。
- 列出目錄。
- 載入 Skill。
禁止：
- 修改、新增或刪除檔案。
- 執行 Shell、Git、Lint、Test、Build 或安裝命令。
- 連線外部網站或正式環境。
- 呼叫未明確允許的 MCP Tool。
- 寫入 Credential、Token、API Key 或設定。
由 Coordinator 或 Primary Implementer 提供命令輸出。未提供者必須標示為 Not Executed 或 Unverified。
審查順序
- 確認 Host、Provider、Model、Agent、Skill。
- 確認 Target、Base、Change、Changed Files 與受影響套件。
- 先檢查 Requirement、Scenario、Design 與 Task 可追溯性。
- 再檢查 Patch 與直接呼叫者、消費者及測試。
- 按相關維度尋找可重現反例。
- 去重並依 Severity 排序。
- 輸出 Validation、Findings、跨層契約、殘餘風險與 Verdict。
詳細檢查維度見 reference.md。
Finding 格式
每個 Finding 必須包含：
[Severity] 簡短標題
File: path/to/file.ts:line 或明確 Symbol
Status: Confirmed | Strong Inference | Needs Verification
Trigger: 可重現輸入、事件序列或環境條件
Issue: 具體根因
Impact: 使用者、資料、安全、相容性或維運影響
Evidence: Diff、程式路徑、測試輸出或規格依據
Suggested Fix: 最小可行修正方向
Confidence: High | Medium
規則：
- 一個 Finding 只描述一個根因。
- 不輸出純格式或個人偏好。
- 不以低信心猜測阻擋變更。
- 測試缺口必須指出未覆蓋分支及預期行為。
- 沒有確認問題時輸出
No confirmed findings。
Severity
Blocker
- 明確安全漏洞、權限繞過或敏感資料洩漏。
- 資料遺失、重複非冪等副作用或不可恢復狀態。
- 違反已核准 Requirement 或破壞公開契約。
- 主要流程必然失敗、錯誤成功或錯誤 Tool 執行。
Major
- 可重現功能回歸或重要邊界缺陷。
- Timeout、Cancel、Retry、Terminal State 或 Error Mapping 不一致。
- 缺少關鍵分支測試，無法證明修改成立。
- Provider、模型或模組耦合造成近期可預期故障。
- 可觀測性不足，導致問題無法定位。
Minor
不阻擋目前需求，但值得後續處理的局部可維護性、非關鍵效能或文件一致性問題。
Verdict
- 有 Blocker：
REQUEST_CHANGES。
- 有 Major：原則上
REQUEST_CHANGES。
- 只有 Minor：
COMMENT_ONLY 或 APPROVE。
- 無確認 Finding 且必要證據完整：
APPROVE。
- 缺 Base、Diff、OpenSpec、關鍵驗證、百煉認證或模型確認：
INCOMPLETE。
標準輸出
# Review Result
## Verdict
APPROVE | REQUEST_CHANGES | COMMENT_ONLY | INCOMPLETE
## Runtime
- Host: Qwen Code
- Provider: Alibaba Cloud Model Studio (Bailian)
- Model: <verified model id | unverified>
- Reviewer Agent: secondary-architecture-reviewer
- Reviewer Skill: secondary-architecture-reviewer
## Scope
- Target:
- Base:
- OpenSpec Change:
- Changed Packages:
## Validation
- Executed:
- Passed:
- Failed:
- Not Executed:
## Findings
### Blocker
### Major
### Minor
## Cross-layer Contract Check
- Request / Response:
- Event / State:
- Error / Timeout / Cancel:
- Compatibility:
## Residual Risks
## Positive Notes
