80 lines
3.4 KiB
Markdown
80 lines
3.4 KiB
Markdown
---
|
||
name: verify-and-report
|
||
description: 完成一個 task 或整個 change 的實作後，執行測試與格式檢查，並依 CLAUDE.md 要求輸出固定格式的完工報告（spec 同步狀態、偏差點、AC 測試涵蓋、重複邏輯、命名一致性）。當使用者說「驗證」「檢查一下」「幫我出完工報告」，或 openspec-apply-change 全部 task 完成時使用。
|
||
argument-hint: [change 名稱，可省略]
|
||
---
|
||
|
||
## Instructions
|
||
|
||
1. **確認範圍**
|
||
```
|
||
git branch --show-current
|
||
git diff master...HEAD --stat
|
||
git status --short
|
||
```
|
||
- 若在 `master` 上，停止並提醒使用者應該在功能分支上
|
||
- 若有 change 名稱（或可從分支與 `openspec list` 推斷），讀取 `openspec/changes/<name>/` 的 proposal、specs、design、tasks
|
||
|
||
2. **執行驗證（失敗就如實回報，不要隱藏或重跑到過為止）**
|
||
- `php artisan test`
|
||
- 對本次變更的 PHP 檔執行 `./vendor/bin/pint --test <檔案>`（只檢查變更的檔案，不要全專案格式化）
|
||
- `openspec validate --specs`
|
||
- 若有前端變更：在 `frontend/` 執行 `npm run build` 確認可建置
|
||
- 注意：`phpunit.xml` 強制使用 SQLite `:memory:`，`lockForUpdate` 等行鎖行為無法在此驗證，報告中不可宣稱「已驗證併發安全」
|
||
|
||
3. **AC 與測試涵蓋**
|
||
- 逐條列出 spec 的 Scenario，指出對應的測試檔與方法（用 `rg` 找實際存在的測試，不要憑印象）
|
||
- 找不到對應測試者標示「未涵蓋」並說明原因
|
||
|
||
4. **Spec 同步檢查**
|
||
- 本次改動是否改變了既有 spec 描述的行為？列出需要更新的 `openspec/specs/<capability>/spec.md`
|
||
- 實作與 spec 是否有偏差？列出偏差點，詢問是否更新 spec，不要自行改 spec
|
||
- 若為新增或修改 API 端點，Swagger（`app/Docs/`）是否同步更新
|
||
|
||
5. **安全確認（僅在觸及觸發條件時）**
|
||
若變更涉及外部輸入、資料庫讀寫、身份驗證/授權、檔案上傳、外部 API，依 CLAUDE.md「安全確認問題」逐項回答：驗證在哪一層、是否使用 Eloquent/Query Builder、權限檢查在哪一層、是否有 N+1、是否有 XSS 風險。
|
||
新增的 member / provider 路由要特別確認角色檢查（CLAUDE.md「授權規範」）。
|
||
|
||
6. **維護性檢查**
|
||
- 重複邏輯：本次新增的邏輯是否已存在於 codebase？用 `rg` 搜尋相似方法再判斷
|
||
- 命名一致性：變數、函式、API endpoint 是否與現有命名一致
|
||
|
||
7. **輸出報告**，使用下列格式，全程使用繁體中文。
|
||
|
||
## 報告格式
|
||
|
||
```
|
||
## 完工報告：<change 或 task 名稱>
|
||
|
||
### 驗證結果
|
||
| 項目 | 結果 | 備註 |
|
||
|------|------|------|
|
||
| php artisan test | ✅ / ❌ N 個失敗 | |
|
||
| pint | ✅ / ❌ | |
|
||
| openspec validate | ✅ / ❌ | |
|
||
|
||
### AC 測試涵蓋
|
||
| Scenario | 對應測試 | 狀態 |
|
||
|----------|----------|------|
|
||
|
||
### Spec 同步狀態
|
||
- 與 spec 一致 / 偏差點：...
|
||
- 需更新的 spec 文件：...
|
||
|
||
### 安全確認（若適用）
|
||
...
|
||
|
||
### 維護性
|
||
- 重複邏輯：...
|
||
- 命名一致性：...
|
||
|
||
### 未解決事項 / 我不確定的地方
|
||
...
|
||
```
|
||
|
||
## Notes
|
||
|
||
- 不要為了讓測試通過而修改測試的預期結果；測試失敗要回報原因。
|
||
- 本 skill 只負責檢查與回報，不修改程式、不 commit、不 push。要提交請接著使用 `finish-and-pr`。
|
||
- 不要在輸出中顯示 remote URL（可能含有憑證）。
|