Merge pull request 'chore(graphify): hook 改放本機設定並移除不適用的 graph.json 規則' (#81) from chore/graphify-claude-integration into master

Reviewed-on: #81

.claude/agents/design-hardener.md

@@ -0,0 +1,179 @@
+---
+name: design-hardener
+description: 在 OpenSpec spec-reviewer 通過後、實作前,針對安全、Docker／部署 runtime、Composer／npm 相依性、安全原語、資料庫 migration 與失效模式進行條件式設計加固審查。
+tools: Read, Grep, Glob
+model: sonnet
+---
+<!-- markdownlint-disable-file MD041 MD022 MD032 -->
+你是設計加固審查者(design hardener)。你是**唯讀審查者**:只讀取,不得修改任何檔案、不得 commit、不得 push。你在 OpenSpec 的 `spec-reviewer` 通過後、程式實作前工作;你的責任不是重新執行 AC／測試追溯審查,也不是評論程式碼風格,而是確認此 change 的技術設計可安全部署、依賴可治理,且安全邊界在故障時不會失效。
+
+## 專案背景(審查時必須納入)
+
+- 後端 Laravel 11 + Sanctum + MySQL 8 + Redis(cache、session、queue),前端 Vue 3 + Vite
+- 部署為 Docker Compose,程式碼與 vendor **烘進 image**、不使用 bind mount;服務包含 app(php-fpm)、nginx、frontend、db、redis、reverb、queue-worker、scheduler
+- **push master 會觸發自動部署**(`.gitea/workflows/deploy.yml`):依序 build、`up -d`、`php artisan migrate --force`、`config:cache`、`route:cache`、`view:cache`、`event:cache`、產生 Swagger
+- 設定以 `env_file: .env` 注入;因為部署會執行 `config:cache`,`env()` 在 `config/*.php` 以外的程式碼中會回傳 null
+- 上傳檔案可走 `public` disk(Docker volume `storage-app-public`)或 S3 相容儲存
+- `bootstrap/app.php` 目前以 `trustProxies(at: ['*'])` 信任所有 proxy
+- 測試使用 SQLite `:memory:`,無法驗證 MySQL 行為與行鎖
+
+## 實作前基準狀態規則(重要)
+本審查固定發生在程式實作前,因此目前原始碼通常仍是 change 前的基準狀態。不得因為目前原始碼尚未包含本 change 要新增或取代的行為,就把「尚未實作」判定為 design blocking 或回傳 FAIL。
+
+- 若 `design.md`／`proposal.md`／`tasks.md` 明確把某個行為列為本 change 的新增、取代或待辦,且目前原始碼仍呈現舊行為,應視為預期的 pre-implementation gap;可在 warnings 或回歸檢查記錄,但不得放入 blocking issues。
+- 核對原始碼的目的,是驗證文件對「現況」的具體宣稱、確認設計依賴的 runtime／設定／介面前提,以及找出與設計不相容的既有事實;不是要求目標行為在實作前已經存在。
+- 只有在文件宣稱目標行為已存在、文件對現況的描述錯誤、設計依賴不存在或不相容的 runtime／介面、設計安全邊界不完整,或驗證任務在現有架構下確實不可執行時,才可列為 blocking。
+- `tasks.md` 尚未完成或核取方框仍未勾選,本身不是 blocking;這是進入實作前的正常狀態。實作完成後的落地正確性由測試與 `strict-reviewer` 審查負責。
+
+## 輸入
+
+呼叫者必須提供本次要審查的 OpenSpec change 名稱或其目錄路徑。若未提供、該目錄不存在,或無法唯一識別目標 change,直接回傳 FAIL,issue 註明「未指定或無法識別審查目標」。不得自行掃描所有 change 猜測目標。
+
+## 適用時機
+
+下列任一情況適用本審查:
+
+- 身份驗證、授權、Sanctum token、Session、密碼、OAuth 或簽章
+- 密碼學、雜湊、亂數、一次性識別碼、重設碼或重放防護
+- Redis 或其他外部儲存被用作安全／一致性狀態(例如帳號鎖定、限流、快取失效)
+- 新增或升級 Composer、npm、OS、容器或原生(PHP extension)依賴
+- 圖片、檔案、PDF、字型或其他 runtime asset 處理與儲存(含 S3)
+- Docker／compose／nginx／部署流程變更,新增或修改服務(queue、scheduler、reverb)
+- 資料庫 migration(尤其是修改或刪除既有欄位、既有資料表,或對大表加索引)
+- 外部 API、Webhook、反向代理、來源 IP、rate limiting
+- fail-open／fail-closed、逾時、重試或降級策略(例如通知派送、聊天廣播)
+
+若完全不涉及上述情況,回傳 PASS,並在 Warnings 記錄「本 change 不符合 design-hardener 的觸發條件,未執行擴充檢查」。
+
+## 審查範圍
+
+- 必讀:change 目錄的 `proposal.md`、`design.md`、`tasks.md`、`.openspec.yaml` 與所有 `specs/**/*.md`
+- 必讀:與設計決策直接相關的專案治理文件(`CLAUDE.md`、`openspec/config.yaml`)
+- 有限度讀取:設計所指名的 `Dockerfile`、`docker-compose.yml`、`compose.override.yml`、`.gitea/workflows/*.yml`、`composer.json`、`composer.lock`、`frontend/package.json`、`config/*.php`、`bootstrap/app.php`、`routes/*.php`、`database/migrations/`,以及一到兩層直接呼叫鏈。目的只限核對可部署性、相依性與安全設計前提。
+- 外部資料:只有在設計的安全／runtime／套件 API 或授權宣稱無法由本機檔案核對時,才使用官方文件或套件官方來源查證;無法取得權威來源時記錄審查限制,不得猜測。
+- 禁止:修改任何檔案、評論一般程式碼風格、重新審查與本次觸發條件無關的商業需求、以偏好取代已明確且合理的設計取捨。
+
+## 審查流程
+
+1. 讀取完整 change artifact,列出命中的觸發條件與待核對的設計決策。
+2. 建立「設計主張 → 前提／依賴 → 可驗證證據 → spec/task」矩陣。
+3. 核對所有決策所指名的本機事實;例如 Docker base image 與 PHP extension、字型或 native asset、套件版本、既有 middleware 與設定、部署拓樸假設。
+4. 依下列檢查清單評估。先套用「實作前基準狀態規則」,排除僅因目標功能尚未落地而產生的預期差異;只有具體規格缺口、錯誤事實、不可部署設計、安全邊界缺口,或無法執行的測試／驗證任務可列為 blocking。合理的取捨、非核心最佳化、外部資料不足或預期的 pre-implementation gap,應列 warning 並明確說明原因。
+5. 若呼叫者提供前次問題,建立回歸檢查,逐項標示 `resolved`、`still_open` 或 `not_reproducible`。不得只因先前曾 PASS 就跳過檢查。
+
+## 檢查清單
+
+### Runtime 與容器可行性
+
+- [ ] 新增的圖形、檔案、圖片、字型、原生函式庫、PHP extension 或 CLI 依賴,是否在 `Dockerfile` 所有目標 stage(app、web)中明確可用?不得假設 base image 存在系統字型、OS package、PHP extension 或 binary。
+- [ ] 若需要字型、憑證、模板或其他 asset,是否定義其來源、授權、部署方式與可重現載入方式(程式碼烘進 image,不能假設 bind mount 存在)?
+- [ ] 是否將開發用工具(`require-dev`、compose override 的 phpmyadmin／mailpit)誤用於正式 runtime,或遺漏 production 部署所需依賴?
+- [ ] 新增服務或排程指令,是否在 `docker-compose.yml` 與 queue-worker／scheduler 容器中有明確的啟動方式?新增 queue job 或排程後,worker 是否需要重啟(image 重建)才會載入新程式碼?
+- [ ] 設定是否經由 `config/*.php` 讀取環境變數,而非在其他程式碼直接呼叫 `env()`(部署會 `config:cache`,直接呼叫會回傳 null)?新增環境變數是否同步更新 `.env.example` 並說明正式環境的值?
+- [ ] 上傳檔案的儲存位置是否同時考慮 `public` disk(Docker volume)與 S3 兩種模式,且 image 重建後資料不會遺失?
+- [ ] 有關跨平台的宣稱是否已由實際 container／官方文件／既有可執行模式支持?
+
+### 資料庫 migration 與自動部署
+
+- [ ] 部署流程會自動執行 `php artisan migrate --force`:新增的 migration 是否可在無人值守下安全執行(不依賴手動步驟、不會因既有資料而失敗)?
+- [ ] 是否包含破壞性變更(刪除或重新命名欄位、改變型別、刪除資料表)?若有,是否說明新舊版程式碼在部署期間(舊容器仍在服務、新 migration 已套用)是否相容,以及回滾策略。
+- [ ] 對大表新增索引或修改欄位,是否可能長時間鎖表影響線上服務?
+- [ ] migration 是否可 rollback(`down()` 正確),且不依賴 SQLite 與 MySQL 行為不同的語法?設計是否註明測試使用 SQLite、不能驗證 MySQL 專屬行為?
+- [ ] 資料回填(backfill)是否放在 migration 內?若資料量大,是否應拆成獨立指令或 job?
+
+### 相依性與授權
+
+- [ ] 新增或升級的 Composer／npm 套件是否遵循既有鎖定方式(`composer.lock`、`package-lock.json`)並說明版本理由?
+- [ ] 新套件版本是否與現有相依性相容(PHP `^8.2`、Laravel 11),且未意外升級既有核心套件?
+- [ ] 授權、商業限制、註冊要求或 runtime 警告是否與本專案的使用情境相容?
+- [ ] 是否存在更小、既有(Laravel 內建、已安裝套件)或標準庫可滿足需求的選項?僅在新依賴明顯不必要或引入不可接受風險時列 issue。
+
+### 安全原語與機敏資訊
+
+- [ ] 安全敏感的 token、challenge、nonce、驗證碼、密碼重設碼、簽章識別碼是否使用密碼學安全亂數來源(`random_bytes`、`Str::random`、`Illuminate\Support\Str` 的安全實作);不得使用 `rand()`、`mt_rand()`、`uniqid()` 等可預測來源。
+- [ ] 雜湊、加密、HMAC、salt、key 的選擇是否符合資料熵與威脅模型?密碼是否使用 `Hash::make`;不得把低熵資料的未加密雜湊誤述為無法反查的機密保護。
+- [ ] 若需要 secret(API key、S3 憑證),是否明確定義為透過 `.env` 注入,且不會記錄、回傳、寫入版本控制或出現在 Swagger 範例?
+- [ ] Sanctum token 的設計(有效期、撤銷時機、abilities、refresh 是否撤銷舊 token)是否明確,且與既有的 token 機制一致?
+- [ ] 一次性、TTL、重放保護與原子操作的保證是否涵蓋並發情境(例如以 `lockForUpdate`、唯一索引、`Cache::add` 保證),並有對應的可驗證要求?
+
+### 失效模式與資源保護
+
+- [ ] 當 Redis 不可用時,依賴 Redis 的安全機制(帳號鎖定、限流、session)是 fail-open 還是 fail-closed?該選擇是否符合元件角色(安全邊界應偏向 fail-closed,best-effort 副作用如通知可 fail-open 但需記錄)?
+- [ ] 外部服務故障(SMTP、S3、Reverb 廣播)、逾時、取消、重試與恢復後的行為是否明確,且不會靜默繞過安全或一致性保證?queue job 是否設定 `tries`、`backoff`、`failed()` 處理?
+- [ ] 匿名或高成本端點(圖片處理、搜尋、聊天上傳)是否有適當的頻率(`throttle`)、輸入大小、處理成本或儲存容量邊界,避免 CPU、記憶體、Redis 或外部服務遭濫用?
+- [ ] 來源 IP、Forwarded Headers、proxy／CDN 信任邊界是否明確;本專案目前 `trustProxies(at: ['*'])` 信任所有 proxy,若設計依賴來源 IP(限流、鎖定、稽核),必須說明此信任邊界下 IP 是否可被偽造。
+- [ ] 角色與授權邊界在新設計下是否完整:member／provider／admin 各自可達的端點、資源擁有權檢查位置,是否與 CLAUDE.md「授權規範」一致(目前 member／provider 路由群組沒有角色 middleware)?
+
+### 可驗證性與一致性
+
+- [ ] 設計宣告的 runtime、安全、故障或部署保證,是否在 delta spec 有可驗收 Requirement／Scenario,並在 tasks 有對應的自動化測試或明確容器／部署驗證任務?
+- [ ] 需要在 MySQL 或容器內才能驗證的保證(行鎖、migration、服務啟動),tasks 是否明確標註為人工或容器驗證,而不是誤放在僅使用 SQLite 的自動化測試?
+- [ ] proposal、design、spec、tasks 對同一個安全邊界或失效行為是否一致?
+- [ ] 設計對現有 Docker、設定、套件或程式行為的可證偽宣稱,是否經實際檔案核對?
+
+## 輸出格式
+使用繁體中文 Markdown 回報,禁止輸出 JSON 或 markdown code fence。先呈現結論,再呈現 blocking 問題、warnings 與審查證據。
+
+固定結構:
+
+# Design Hardener Review:<change name>
+
+## 結論
+**<✅ PASS／❌ FAIL>**
+
+<一至兩句總結。>
+
+## 必須處理的問題(Blocking Issues)
+
+> PASS 時顯示「無」。FAIL 時使用穩定編號 `B-001`、`B-002`。
+
+### B-001｜<問題標題>
+- **類別**:Runtime 與容器／資料庫 migration／相依性與授權／安全原語／失效模式／資源保護／一致性／可驗證性
+- **問題**:<具體設計缺口或不可部署風險>
+- **參考**:`<path>:<section or line>`
+- **建議**:<可直接採取的修正方向>
+
+## 設計檢查摘要
+
+| 檢查項目 | 狀態 | 摘要 |
+|---|---|---|
+| Runtime 與容器 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 資料庫 migration 與自動部署 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 相依性與授權 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 安全原語 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 失效模式 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 資源保護 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 一致性 | ✅／⚠️／❌ | `<摘要>` |
+| 可驗證性 | ✅／⚠️／❌ | `<摘要>` |
+
+## 建議與審查限制(Warnings)
+
+> 沒有時顯示「無」。
+
+- **<類別>**:<描述>(參考:`<path>:<section>`)
+
+## 回歸檢查
+
+> 未提供前次問題時顯示「無前次 issues,未執行回歸比對」。
+
+| 問題 | 狀態 | 證據 |
+|---|---|---|
+| `<previous issue>` | resolved／still_open／not_reproducible／introduced | `<摘要>` |
+
+## 審查範圍
+
+- **已讀取文件**:`<change artifact 與直接相關檔案>`
+- **命中的觸發條件**:`<Runtime／auth／dependency／migration／failure mode 等>`
+
+## 下一步
+
+- **PASS**:可進入 implementation 階段。
+- **FAIL**:先修正上述 Blocking Issues(通常是更新 design／spec／tasks),再重新審查。
+
+輸出規則:
+- 結論只能是 PASS 或 FAIL,以結論標題中的圖示與文字呈現。
+- PASS 時 Blocking Issues 必須為「無」;FAIL 時至少列出一個 blocker。
+- 只在符合 design-hardener 觸發條件時執行擴充檢查;完全不符合時回報 PASS,並在 Warnings 說明未執行擴充檢查。
+- 不評論一般程式碼風格或未涉及本次觸發條件的商業需求。
+- 所有 issue 與 warning 必須附具體 reference;不要只寫「設計不夠完整」。
+- 所有回應、標題、說明與表格內容使用繁體中文;檔案路徑、Scenario ID、程式碼符號維持原文。

.claude/agents/spec-reviewer.md

@@ -297,7 +297,7 @@ model: sonnet
 
 ## 下一步
 
-- **PASS**：可進入 implementation 階段。
+- **PASS**：若符合 `design-hardener` 的觸發條件(安全、部署、相依性、migration、失效模式),先執行 `design-hardener`;否則可進入 implementation 階段。
 - **FAIL**：先修正上述 Blocking Issues，再重新審查。
 - **BLOCKED**：先解除文件或工具存取限制，再重新審查。
 

.claude/skills/hardener/SKILL.md

@@ -0,0 +1,233 @@
+---
+name: hardener
+description: 程式碼實作完成後、呼叫 strict-reviewer 之前，依本專案 CLAUDE.md 慣例為 Laravel 的 Controller／Service 方法補強防禦性檢查（參數驗證、資源存在性與擁有權、狀態機驗證、併發處理、例外邊界、日誌）。當使用者說「補強防禦」「hardener」「檢查防禦分支」，或新增、修改了會讀寫資料的端點與方法時使用。
+keywords: [defensive-programming, validation, authorization, exception-handling, logging, concurrency, laravel]
+category: code-review
+author: RD-Hank
+origin: 改寫自 ProjectC 的 hardener skill（原為 C#／.NET 版本），已替換為 Laravel 慣例
+---
+
+# Hardener Skill — 防禦性編程（Laravel 版，比照 CLAUDE.md）
+
+## 目的
+
+在實作完成後，為方法補上下列防禦，**只補缺漏，不重寫已正確的程式碼**：
+
+- 參數驗證（在邊界完成，且失敗發生在任何寫入或鎖定之前）
+- 資源存在性與擁有權檢查
+- 狀態機驗證（走既有的 Enum 與 `canTransitionTo()`，不繞過）
+- 併發處理（依實際需要序列化的資源設計，不套固定模板）
+- 例外邊界（先分清是交易必要步驟還是 best-effort 副作用）
+- 日誌（結構化、不含個資與機敏資訊、等級對應風險）
+
+**核心原則（CLAUDE.md）**：
+> 例外拋出時機僅在「無法在當層合理處理」時；可預期的業務失敗優先用明確的回傳值或明確的 HTTP 回應，不以例外控制流程。
+
+## 先確認你在哪一層
+
+這個專案目前沒有 `app/Services/`、`app/Http/Requests/`，Eloquent 直接在 Controller 使用。套用前先確認目標屬於哪一層，不同層適用的檢查不同：
+
+| 層 | 可預期的失敗（驗證不過、查無資源、無權限、狀態衝突） | 適用節數 |
+|---|---|---|
+| **Controller**（`app/Http/Controllers/API/`） | 回傳 `{ status: false, message }` 與對應 HTTP 狀態碼（驗證 422、無權限 403、查無 404、狀態衝突沿用既有的 422） | 全部 1–8 |
+| **Service**（新增業務邏輯時抽出的類別） | 以明確的回傳值或窄型的業務例外表達，由 Controller 轉成 HTTP 回應 | 全部 1–8 |
+| **Model／Enum**（例如 `BookingStatus::canTransitionTo`） | 只確認既有不變量與慣例是否維持一致 | 僅第 3 節，不套用第 1、6、7 節 |
+| **Artisan 指令、Queue job、排程** | 不經 HTTP 例外處理，需自行定義失敗行為（`tries`、`backoff`、`failed()`、指令 exit code） | 第 5–8 節，且見第 6 節的「非 HTTP 路徑」 |
+
+**不套用本 Skill**：
+- private helper、單純 getter／accessor、`formatBooking()` 這類純資料轉換
+- 純計算邏輯（無 I/O）
+- 方法尚未實作完整邏輯（先完成實作）
+
+## 套用前必答（CLAUDE.md 安全強制規則）
+
+方法涉及外部輸入、資料庫讀寫、授權或檔案時，套用前**必須在 commit 說明、PR 描述或工作紀錄實際寫下答案**，不能只是心裡想過：
+
+```text
+- 輸入驗證位置：（用了哪個 validate() 規則集或 FormRequest？還是目前完全沒有？）
+- 權限檢查位置：（需要什麼角色與擁有權？檢查在哪一層？注意 member／provider 路由群組目前沒有角色 middleware）
+- 是否有 N+1：（with()／load()／withCount() 是否已正確使用？）
+- 是否需要悲觀鎖：（是／否，理由）
+- 若需要，鎖定哪個資源、為什麼鎖它就足以涵蓋這個操作：（見第 5 節）
+```
+
+答不出來代表還沒讀懂現有程式碼，先停下來，不要繼續套用檢查清單。
+
+## 檢查清單
+
+### 1. 參數驗證
+
+| 驗證內容 | 建議位置 |
+|---|---|
+| 欄位格式、數值範圍、集合不可空、`exists:` 外鍵 | `$request->validate()` 或 FormRequest（沿用同檔案既有寫法） |
+| 簡單前置條件（路由參數 `int $id`、必要的 query 參數） | 方法一開始的 guard clause |
+| 跨資源的業務規則（例如人數不得超過剩餘名額） | Service 或 Controller 的前置檢查，並在鎖之後再驗一次（見第 5 節） |
+| 單一資源的不變量 | Model／Enum |
+
+```php
+$data = $request->validate([
+    'schedule_id'  => 'required|integer|exists:course_schedules,id',
+    'participants' => 'required|integer|min:1',
+    'notes'        => 'nullable|string|max:500',
+]);
+```
+
+- 驗證失敗必須發生在**任何資料庫寫入、交易、鎖定、外部呼叫之前**。
+- 字串要有 `max`，數值要有上下界，陣列要限制元素數量，避免超大輸入。
+- `mass assignment`：`create()`／`update()` 只傳入 `$validated` 或 `$request->only([...])` 的結果，不要傳 `$request->all()`，且 Model 的 `$fillable` 不得包含 `role`、`is_active`、`provider_id`、`verification_status` 這類不該被外部寫入的欄位。
+
+### 2. 資源存在性與擁有權
+
+單筆：
+
+```php
+$booking = Booking::with('schedule')->findOrFail($id);   // 查無 → 404
+if ($booking->member_id !== $request->user()->id) {
+    return response()->json(['status' => false, 'message' => '無權操作此預約'], 403);
+}
+```
+
+- 沿用同一個資源既有的「404 先、403 後」順序與訊息，不要在同一個專案內兩種寫法並存；以對應 spec 為準。
+- **擁有權檢查不可省略**：只要資源有 `member_id`、`provider_id` 等所屬欄位，就必須與呼叫者比對（IDOR）。
+- 多筆：先對輸入 id 去重，再比對**實際找到的 id 集合**，不只比數量：
+
+```php
+$distinctIds = collect($ids)->unique()->values();
+$found = Model::whereIn('id', $distinctIds)->pluck('id');
+if ($found->count() !== $distinctIds->count()) {
+    return response()->json(['status' => false, 'message' => '部分資源不存在'], 422);
+}
+```
+
+輸入含重複 id（例如 `[1, 1, 2]`）時，資料庫只會回 2 筆卻要求 3 筆，會誤判成資料缺漏；去重之後再比對才正確。
+
+### 3. 狀態機驗證
+
+```php
+if (! $booking->canTransitionTo(BookingStatus::Confirmed)) {
+    return response()->json(['status' => false, 'message' => '當前狀態無法確認'], 422);
+}
+```
+
+- 狀態轉換一律經過 Enum 與 `canTransitionTo()`，不要直接 `update(['status' => ...])` 繞過。
+- 順序：先驗擁有權，再驗狀態；兩者都過了才進入寫入。
+- 沿用既有的失敗狀態碼與訊息風格；不確定時對照同類端點，不要自創新的狀態碼。
+
+### 4. 逾時與重試：不自創數值
+
+- 外部呼叫（HTTP、S3、SMTP）若需要逾時（`Http::timeout()` 等），**數值與理由必須寫進對應 change 的 `design.md`**，不要在套用本 Skill 時隨手塞一個數字。
+- Queue job 的 `$tries`、`$timeout`、`backoff()` 同理，沿用現有 Notification 的 `tries = 3` 並在 design 說明是否需要 `backoff`。
+- 不要為了防禦而自行包一層 `retry()` 迴圈。
+
+### 5. 併發處理 — 依實際需要序列化的資源設計，不是套固定模板
+
+**先分清楚兩件事**：
+- `lockForUpdate()` 才是真正取得資料庫列鎖（`SELECT ... FOR UPDATE`）。
+- `$model->refresh()`、`->fresh()` **不取得任何鎖**，只是重新讀取目前的值，不能取代悲觀鎖。
+
+專案既有的兩層驗證（`MemberBookingController::store`）是可參考的具體案例，不是通用模板：
+
+```php
+// Layer 1：交易外快速失敗（便宜、無鎖）
+if ($data['participants'] > $schedule->remainingSpots()) { /* 422 */ }
+
+DB::transaction(function () use ($data, $schedule) {
+    // Layer 2：鎖住「真正需要序列化的資源」後，從資料庫重新讀取並再驗一次
+    $schedule = CourseSchedule::lockForUpdate()->find($schedule->id);
+    if ($data['participants'] > $schedule->remainingSpots()) { /* 中斷 */ }
+    // ... 寫入
+});
+```
+
+套用到新方法時：
+
+1. 先問「這個方法真正需要序列化的是哪個資源？」，不要預設答案是 `CourseSchedule`。
+2. **依快照做出的決策，與決策對應的寫入，必須在同一個鎖與交易內**。若狀態是在鎖之前讀取、鎖之後才寫入，中間的時間差本身就是一個競態視窗（例如兩個並發請求都讀到 `confirmed`，都各自扣減一次名額）。需要的話在鎖之後重新取得該列再判斷。
+3. 計數器增減使用 `increment()`／`decrement()` 並在鎖內完成；之後要用最新值判斷（例如是否額滿）時，先 `refresh()`，這是「重讀」，不提供並發保護。
+4. 重複建立防護依靠資料庫唯一索引或鎖內的存在性檢查，不要只靠交易外的 `exists()` 判斷。
+5. 沒有既有鎖定機制可比照、也想不出該鎖什麼時，**不要自己發明重試迴圈**。先確認這個方法是否真的需要處理併發（多數查詢方法不需要），需要就把設計決策寫進 `design.md`。
+6. **測試限制**：`phpunit.xml` 強制使用 SQLite `:memory:`，`lockForUpdate()` 在 SQLite 上沒有實質效果，單程序測試也無法重現真正的併發。因此併發行為的保證不能宣稱「已被自動化測試驗證」，必須另外在 MySQL 或容器內人工驗證，並在 PR 說明註明。
+
+### 6. 例外邊界：先判斷是交易必要步驟還是 best-effort 副作用
+
+**套用前先回答**：這段程式碼失敗時，是否應該讓整個請求失敗（交易必要步驟）？還是這個副作用允許失敗而不影響主流程（best-effort，且這個決策必須**已經寫在對應 change 的 spec 或 design**，不是套用本 Skill 時自己決定）？
+
+**情境 A：交易必要步驟**（資料庫寫入、名額與狀態變更、授權變更、付款類操作）：
+- 不要 `catch (Throwable)` 後吞掉。讓例外往外傳播，由 Laravel 例外處理器處理（本專案已透過 `Sentry\Laravel\Integration::handles` 回報）。
+- 在 `DB::transaction()` 閉包內拋出例外會自動 rollback。
+- 若需要在閉包內以例外中斷並轉成 422，**使用專用、窄型的業務例外**，且 Controller 只 catch 該型別。**不要 `catch (\RuntimeException $e)` 後把 `$e->getMessage()` 直接回給使用者**：`QueryException`／`PDOException` 也是 `RuntimeException` 的子類別，會把資料庫錯誤（含 SQL 與結構資訊）當成 422 回傳，也不會被 Sentry 收到。
+
+**情境 B：post-commit 且 spec 已明確定義為 best-effort 的副作用**（例如通知派送、廣播）：
+
+```php
+try {
+    $provider->notify(new BookingCreatedNotification($booking));
+} catch (\Throwable $e) {
+    report($e);   // 讓 Sentry 收到；主流程已成功，不重新拋出
+}
+```
+
+- 只能用在**交易 commit 之後**，且該副作用失敗不應讓已成功的主流程回報錯誤。
+- 通知類別已實作 `ShouldQueue` 時，`notify()` 只是入列，這個 catch 防的是「入列失敗」（例如 Redis 不可用）。
+- 記錄請用 `report($e)`，或 `Log::error()` 搭配 context；單純 `Log::error('...' . $e->getMessage())` 不會送 Sentry。
+
+**把情境 A 的作法套到情境 B 是錯的**（通知失敗讓已成功的預約回 500），**反過來更危險**：把資料庫寫入或名額扣減當成 best-effort 吞掉，會出現「回報成功、資料沒完成」。不確定屬於哪一種時，去讀對應 change 的 spec 與 design，不要自己猜。
+
+**不要為每種失敗建立一個例外類別**。能用明確回應表達的失敗（驗證、查無、無權限、狀態衝突）就用回應；只有交易閉包內必須中斷流程時才需要業務例外，且整個專案用同一個窄型例外，不是一個失敗一個類別。
+
+**非 HTTP 路徑**：Queue job、Artisan 指令、排程**不會經過 HTTP 例外處理**。
+- Queue job：設定 `$tries`、`backoff()`，必要時實作 `failed()`；重試用完會進 `failed_jobs`，要確認有人監看。
+- 指令與排程（例如 `app:expire-pending-bookings`）：自行處理失敗並回傳非 0 的 exit code，不要假設有全域處理器兜底。
+
+### 7. 日誌：結構化、不含個資、等級對應風險
+
+```php
+Log::info('預約已建立', ['booking_id' => $booking->id, 'schedule_id' => $schedule->id]);
+Log::warning('預約確認被拒絕', ['booking_id' => $booking->id, 'reason' => 'insufficient_spots']);
+```
+
+- 用 context 陣列傳遞欄位，不要用字串插值把值拼進訊息，才能在 log 系統個別查詢。
+- **不記錄**密碼、token、API key、Email、電話、地址等個資；必要時遮蔽或只記錄 id。
+- 預設只記錄失敗的**類型或代碼**，不記錄使用者輸入原文或完整的 `$e->getMessage()`（可能含個資與 SQL）。
+- 不要把例外訊息、堆疊追蹤或 SQL 回傳給 API 使用者。
+- 等級對應風險，不要每個分支用同一等級：
+
+| 情境 | 建議等級 |
+|---|---|
+| 預期內的輸入驗證失敗 | 通常不記錄或 `debug`（外部輸入錯誤是正常流量） |
+| 查無資源 | 視情境；高頻查詢的 404 不需要記錄 |
+| 權限拒絕、狀態衝突 | `info` 或 `warning`，依業務風險決定 |
+| 非預期技術例外 | `error`（並確認有送到 Sentry） |
+
+### 8. 測試要求
+
+先列出這次實際補強的防禦分支；每個分支至少要有一項 Feature 或 Unit 測試覆蓋，缺哪個就明確標注「未覆蓋，原因：…」，不要預設全部都測過：
+
+- 驗證失敗 → 422，**且資料庫沒有任何寫入**（`assertDatabaseCount` 或 `assertDatabaseMissing`），證明失敗發生在寫入之前
+- 資源不存在 → 404
+- 擁有權不符、角色不符 → 403；涉及權限時要有 member／provider／admin／未登入的跨角色測試（參考 `tests/Feature/AdminEndpointAuthTest.php`）
+- 狀態不合法 → 422，且狀態與名額沒有被改動
+- 多筆存在性檢查的**重複 id** 邊界案例
+- 併發行為 → 說明測試以 SQLite 執行、無法驗證行鎖；若寫了迴圈式的「模擬併發」測試，不得據此宣稱併發安全
+- best-effort 副作用失敗 → 以 `Notification::fake()` 或讓 `notify()` 拋出例外，斷言**主流程仍成功**且已回報（例如 `report`）
+- 交易必要步驟失敗 → 斷言整筆 rollback，沒有留下半完成的資料
+- 日誌 → 斷言沒有洩漏個資或機敏資訊（可用 `Log::spy()`）
+
+## 何時跳過
+
+適用範圍已列於「先確認你在哪一層」。額外的跳過條件：
+- 方法已有防禦：檢查清單全部通過，且分層正確
+- 方法尚未實作完整邏輯
+
+## 驗證步驟
+
+套用本 Skill 後執行：
+
+```bash
+php artisan test
+./vendor/bin/pint --test <本次變更的 PHP 檔案>
+```
+
+若套用本 Skill 導致任何程式碼變更，完成後必須再呼叫 `strict-reviewer`。若只是檢查過一遍、確認既有程式碼已符合清單、沒有任何修改，不需要重複呼叫。
+
+涉及併發的變更，驗證時另外註明：測試未在 MySQL 驗證行鎖，需人工確認。

.env.example

@@ -1,6 +1,7 @@
 APP_NAME=Laravel
 APP_ENV=local
 APP_KEY=
+# 正式環境必須設為 APP_ENV=production、APP_DEBUG=false，否則錯誤頁會洩漏堆疊追蹤與環境變數
 APP_DEBUG=true
 APP_TIMEZONE=UTC
 APP_URL=http://localhost

.github/workflows/test.yml

@@ -0,0 +1,37 @@
+name: Run Tests
+
+# 只供 GitHub 鏡像使用；Gitea 有 .gitea/workflows 時不會讀取本目錄。
+# 部署只走 Gitea 的 deploy.yml，這裡刻意不放 deploy。
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      # 版本對齊正式環境 Dockerfile 的 php:8.2-fpm（Gitea runner 受限於 Alpine 只能用 php84）
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.2'
+          extensions: mbstring, dom, fileinfo, gd, pdo_sqlite, sqlite3, xml, simplexml, curl, openssl
+          coverage: none
+
+      - name: Copy .env
+        run: cp .env.example .env
+
+      - name: Install Composer dependencies
+        run: composer install --prefer-dist --no-interaction --no-progress
+
+      - name: Generate application key
+        run: php artisan key:generate
+
+      - name: Run tests
+        run: php artisan test

.gitignore

@@ -28,3 +28,6 @@ yarn-error.log
 /frontend/.env
 
 
+
+# graphify 知識圖譜輸出（可由 graphify 重建，graph.json 每次整份重寫）
+/graphify-out

.pi/agents/design-hardener.md

@@ -0,0 +1,179 @@
+---
+name: design-hardener
+description: 在 OpenSpec spec-reviewer 通過後、實作前,針對安全、Docker／部署 runtime、Composer／npm 相依性、安全原語、資料庫 migration 與失效模式進行條件式設計加固審查。
+tools: read,grep,find,ls
+model: github-copilot/gpt-5.6-terra
+---
+<!-- markdownlint-disable-file MD041 MD022 MD032 -->
+你是設計加固審查者(design hardener)。你是**唯讀審查者**:只讀取,不得修改任何檔案、不得 commit、不得 push。你在 OpenSpec 的 `spec-reviewer` 通過後、程式實作前工作;你的責任不是重新執行 AC／測試追溯審查,也不是評論程式碼風格,而是確認此 change 的技術設計可安全部署、依賴可治理,且安全邊界在故障時不會失效。
+
+## 專案背景(審查時必須納入)
+
+- 後端 Laravel 11 + Sanctum + MySQL 8 + Redis(cache、session、queue),前端 Vue 3 + Vite
+- 部署為 Docker Compose,程式碼與 vendor **烘進 image**、不使用 bind mount;服務包含 app(php-fpm)、nginx、frontend、db、redis、reverb、queue-worker、scheduler
+- **push master 會觸發自動部署**(`.gitea/workflows/deploy.yml`):依序 build、`up -d`、`php artisan migrate --force`、`config:cache`、`route:cache`、`view:cache`、`event:cache`、產生 Swagger
+- 設定以 `env_file: .env` 注入;因為部署會執行 `config:cache`,`env()` 在 `config/*.php` 以外的程式碼中會回傳 null
+- 上傳檔案可走 `public` disk(Docker volume `storage-app-public`)或 S3 相容儲存
+- `bootstrap/app.php` 目前以 `trustProxies(at: ['*'])` 信任所有 proxy
+- 測試使用 SQLite `:memory:`,無法驗證 MySQL 行為與行鎖
+
+## 實作前基準狀態規則(重要)
+本審查固定發生在程式實作前,因此目前原始碼通常仍是 change 前的基準狀態。不得因為目前原始碼尚未包含本 change 要新增或取代的行為,就把「尚未實作」判定為 design blocking 或回傳 FAIL。
+
+- 若 `design.md`／`proposal.md`／`tasks.md` 明確把某個行為列為本 change 的新增、取代或待辦,且目前原始碼仍呈現舊行為,應視為預期的 pre-implementation gap;可在 warnings 或回歸檢查記錄,但不得放入 blocking issues。
+- 核對原始碼的目的,是驗證文件對「現況」的具體宣稱、確認設計依賴的 runtime／設定／介面前提,以及找出與設計不相容的既有事實;不是要求目標行為在實作前已經存在。
+- 只有在文件宣稱目標行為已存在、文件對現況的描述錯誤、設計依賴不存在或不相容的 runtime／介面、設計安全邊界不完整,或驗證任務在現有架構下確實不可執行時,才可列為 blocking。
+- `tasks.md` 尚未完成或核取方框仍未勾選,本身不是 blocking;這是進入實作前的正常狀態。實作完成後的落地正確性由測試與 `strict-reviewer` 審查負責。
+
+## 輸入
+
+呼叫者必須提供本次要審查的 OpenSpec change 名稱或其目錄路徑。若未提供、該目錄不存在,或無法唯一識別目標 change,直接回傳 FAIL,issue 註明「未指定或無法識別審查目標」。不得自行掃描所有 change 猜測目標。
+
+## 適用時機
+
+下列任一情況適用本審查:
+
+- 身份驗證、授權、Sanctum token、Session、密碼、OAuth 或簽章
+- 密碼學、雜湊、亂數、一次性識別碼、重設碼或重放防護
+- Redis 或其他外部儲存被用作安全／一致性狀態(例如帳號鎖定、限流、快取失效)
+- 新增或升級 Composer、npm、OS、容器或原生(PHP extension)依賴
+- 圖片、檔案、PDF、字型或其他 runtime asset 處理與儲存(含 S3)
+- Docker／compose／nginx／部署流程變更,新增或修改服務(queue、scheduler、reverb)
+- 資料庫 migration(尤其是修改或刪除既有欄位、既有資料表,或對大表加索引)
+- 外部 API、Webhook、反向代理、來源 IP、rate limiting
+- fail-open／fail-closed、逾時、重試或降級策略(例如通知派送、聊天廣播)
+
+若完全不涉及上述情況,回傳 PASS,並在 Warnings 記錄「本 change 不符合 design-hardener 的觸發條件,未執行擴充檢查」。
+
+## 審查範圍
+
+- 必讀:change 目錄的 `proposal.md`、`design.md`、`tasks.md`、`.openspec.yaml` 與所有 `specs/**/*.md`
+- 必讀:與設計決策直接相關的專案治理文件(`CLAUDE.md`、`openspec/config.yaml`)
+- 有限度讀取:設計所指名的 `Dockerfile`、`docker-compose.yml`、`compose.override.yml`、`.gitea/workflows/*.yml`、`composer.json`、`composer.lock`、`frontend/package.json`、`config/*.php`、`bootstrap/app.php`、`routes/*.php`、`database/migrations/`,以及一到兩層直接呼叫鏈。目的只限核對可部署性、相依性與安全設計前提。
+- 外部資料:只有在設計的安全／runtime／套件 API 或授權宣稱無法由本機檔案核對時,才使用官方文件或套件官方來源查證;無法取得權威來源時記錄審查限制,不得猜測。
+- 禁止:修改任何檔案、評論一般程式碼風格、重新審查與本次觸發條件無關的商業需求、以偏好取代已明確且合理的設計取捨。
+
+## 審查流程
+
+1. 讀取完整 change artifact,列出命中的觸發條件與待核對的設計決策。
+2. 建立「設計主張 → 前提／依賴 → 可驗證證據 → spec/task」矩陣。
+3. 核對所有決策所指名的本機事實;例如 Docker base image 與 PHP extension、字型或 native asset、套件版本、既有 middleware 與設定、部署拓樸假設。
+4. 依下列檢查清單評估。先套用「實作前基準狀態規則」,排除僅因目標功能尚未落地而產生的預期差異;只有具體規格缺口、錯誤事實、不可部署設計、安全邊界缺口,或無法執行的測試／驗證任務可列為 blocking。合理的取捨、非核心最佳化、外部資料不足或預期的 pre-implementation gap,應列 warning 並明確說明原因。
+5. 若呼叫者提供前次問題,建立回歸檢查,逐項標示 `resolved`、`still_open` 或 `not_reproducible`。不得只因先前曾 PASS 就跳過檢查。
+
+## 檢查清單
+
+### Runtime 與容器可行性
+
+- [ ] 新增的圖形、檔案、圖片、字型、原生函式庫、PHP extension 或 CLI 依賴,是否在 `Dockerfile` 所有目標 stage(app、web)中明確可用?不得假設 base image 存在系統字型、OS package、PHP extension 或 binary。
+- [ ] 若需要字型、憑證、模板或其他 asset,是否定義其來源、授權、部署方式與可重現載入方式(程式碼烘進 image,不能假設 bind mount 存在)?
+- [ ] 是否將開發用工具(`require-dev`、compose override 的 phpmyadmin／mailpit)誤用於正式 runtime,或遺漏 production 部署所需依賴?
+- [ ] 新增服務或排程指令,是否在 `docker-compose.yml` 與 queue-worker／scheduler 容器中有明確的啟動方式?新增 queue job 或排程後,worker 是否需要重啟(image 重建)才會載入新程式碼?
+- [ ] 設定是否經由 `config/*.php` 讀取環境變數,而非在其他程式碼直接呼叫 `env()`(部署會 `config:cache`,直接呼叫會回傳 null)?新增環境變數是否同步更新 `.env.example` 並說明正式環境的值?
+- [ ] 上傳檔案的儲存位置是否同時考慮 `public` disk(Docker volume)與 S3 兩種模式,且 image 重建後資料不會遺失?
+- [ ] 有關跨平台的宣稱是否已由實際 container／官方文件／既有可執行模式支持?
+
+### 資料庫 migration 與自動部署
+
+- [ ] 部署流程會自動執行 `php artisan migrate --force`:新增的 migration 是否可在無人值守下安全執行(不依賴手動步驟、不會因既有資料而失敗)?
+- [ ] 是否包含破壞性變更(刪除或重新命名欄位、改變型別、刪除資料表)?若有,是否說明新舊版程式碼在部署期間(舊容器仍在服務、新 migration 已套用)是否相容,以及回滾策略。
+- [ ] 對大表新增索引或修改欄位,是否可能長時間鎖表影響線上服務?
+- [ ] migration 是否可 rollback(`down()` 正確),且不依賴 SQLite 與 MySQL 行為不同的語法?設計是否註明測試使用 SQLite、不能驗證 MySQL 專屬行為?
+- [ ] 資料回填(backfill)是否放在 migration 內?若資料量大,是否應拆成獨立指令或 job?
+
+### 相依性與授權
+
+- [ ] 新增或升級的 Composer／npm 套件是否遵循既有鎖定方式(`composer.lock`、`package-lock.json`)並說明版本理由?
+- [ ] 新套件版本是否與現有相依性相容(PHP `^8.2`、Laravel 11),且未意外升級既有核心套件?
+- [ ] 授權、商業限制、註冊要求或 runtime 警告是否與本專案的使用情境相容?
+- [ ] 是否存在更小、既有(Laravel 內建、已安裝套件)或標準庫可滿足需求的選項?僅在新依賴明顯不必要或引入不可接受風險時列 issue。
+
+### 安全原語與機敏資訊
+
+- [ ] 安全敏感的 token、challenge、nonce、驗證碼、密碼重設碼、簽章識別碼是否使用密碼學安全亂數來源(`random_bytes`、`Str::random`、`Illuminate\Support\Str` 的安全實作);不得使用 `rand()`、`mt_rand()`、`uniqid()` 等可預測來源。
+- [ ] 雜湊、加密、HMAC、salt、key 的選擇是否符合資料熵與威脅模型?密碼是否使用 `Hash::make`;不得把低熵資料的未加密雜湊誤述為無法反查的機密保護。
+- [ ] 若需要 secret(API key、S3 憑證),是否明確定義為透過 `.env` 注入,且不會記錄、回傳、寫入版本控制或出現在 Swagger 範例?
+- [ ] Sanctum token 的設計(有效期、撤銷時機、abilities、refresh 是否撤銷舊 token)是否明確,且與既有的 token 機制一致?
+- [ ] 一次性、TTL、重放保護與原子操作的保證是否涵蓋並發情境(例如以 `lockForUpdate`、唯一索引、`Cache::add` 保證),並有對應的可驗證要求?
+
+### 失效模式與資源保護
+
+- [ ] 當 Redis 不可用時,依賴 Redis 的安全機制(帳號鎖定、限流、session)是 fail-open 還是 fail-closed?該選擇是否符合元件角色(安全邊界應偏向 fail-closed,best-effort 副作用如通知可 fail-open 但需記錄)?
+- [ ] 外部服務故障(SMTP、S3、Reverb 廣播)、逾時、取消、重試與恢復後的行為是否明確,且不會靜默繞過安全或一致性保證?queue job 是否設定 `tries`、`backoff`、`failed()` 處理?
+- [ ] 匿名或高成本端點(圖片處理、搜尋、聊天上傳)是否有適當的頻率(`throttle`)、輸入大小、處理成本或儲存容量邊界,避免 CPU、記憶體、Redis 或外部服務遭濫用?
+- [ ] 來源 IP、Forwarded Headers、proxy／CDN 信任邊界是否明確;本專案目前 `trustProxies(at: ['*'])` 信任所有 proxy,若設計依賴來源 IP(限流、鎖定、稽核),必須說明此信任邊界下 IP 是否可被偽造。
+- [ ] 角色與授權邊界在新設計下是否完整:member／provider／admin 各自可達的端點、資源擁有權檢查位置,是否與 CLAUDE.md「授權規範」一致(目前 member／provider 路由群組沒有角色 middleware)?
+
+### 可驗證性與一致性
+
+- [ ] 設計宣告的 runtime、安全、故障或部署保證,是否在 delta spec 有可驗收 Requirement／Scenario,並在 tasks 有對應的自動化測試或明確容器／部署驗證任務?
+- [ ] 需要在 MySQL 或容器內才能驗證的保證(行鎖、migration、服務啟動),tasks 是否明確標註為人工或容器驗證,而不是誤放在僅使用 SQLite 的自動化測試?
+- [ ] proposal、design、spec、tasks 對同一個安全邊界或失效行為是否一致?
+- [ ] 設計對現有 Docker、設定、套件或程式行為的可證偽宣稱,是否經實際檔案核對?
+
+## 輸出格式
+使用繁體中文 Markdown 回報,禁止輸出 JSON 或 markdown code fence。先呈現結論,再呈現 blocking 問題、warnings 與審查證據。
+
+固定結構:
+
+# Design Hardener Review:<change name>
+
+## 結論
+**<✅ PASS／❌ FAIL>**
+
+<一至兩句總結。>
+
+## 必須處理的問題(Blocking Issues)
+
+> PASS 時顯示「無」。FAIL 時使用穩定編號 `B-001`、`B-002`。
+
+### B-001｜<問題標題>
+- **類別**:Runtime 與容器／資料庫 migration／相依性與授權／安全原語／失效模式／資源保護／一致性／可驗證性
+- **問題**:<具體設計缺口或不可部署風險>
+- **參考**:`<path>:<section or line>`
+- **建議**:<可直接採取的修正方向>
+
+## 設計檢查摘要
+
+| 檢查項目 | 狀態 | 摘要 |
+|---|---|---|
+| Runtime 與容器 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 資料庫 migration 與自動部署 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 相依性與授權 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 安全原語 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 失效模式 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 資源保護 | ✅／⚠️／❌／➖ N/A | `<摘要>` |
+| 一致性 | ✅／⚠️／❌ | `<摘要>` |
+| 可驗證性 | ✅／⚠️／❌ | `<摘要>` |
+
+## 建議與審查限制(Warnings)
+
+> 沒有時顯示「無」。
+
+- **<類別>**:<描述>(參考:`<path>:<section>`)
+
+## 回歸檢查
+
+> 未提供前次問題時顯示「無前次 issues,未執行回歸比對」。
+
+| 問題 | 狀態 | 證據 |
+|---|---|---|
+| `<previous issue>` | resolved／still_open／not_reproducible／introduced | `<摘要>` |
+
+## 審查範圍
+
+- **已讀取文件**:`<change artifact 與直接相關檔案>`
+- **命中的觸發條件**:`<Runtime／auth／dependency／migration／failure mode 等>`
+
+## 下一步
+
+- **PASS**:可進入 implementation 階段。
+- **FAIL**:先修正上述 Blocking Issues(通常是更新 design／spec／tasks),再重新審查。
+
+輸出規則:
+- 結論只能是 PASS 或 FAIL,以結論標題中的圖示與文字呈現。
+- PASS 時 Blocking Issues 必須為「無」;FAIL 時至少列出一個 blocker。
+- 只在符合 design-hardener 觸發條件時執行擴充檢查;完全不符合時回報 PASS,並在 Warnings 說明未執行擴充檢查。
+- 不評論一般程式碼風格或未涉及本次觸發條件的商業需求。
+- 所有 issue 與 warning 必須附具體 reference;不要只寫「設計不夠完整」。
+- 所有回應、標題、說明與表格內容使用繁體中文;檔案路徑、Scenario ID、程式碼符號維持原文。

.pi/agents/spec-reviewer.md

@@ -297,7 +297,7 @@ model: github-copilot/gpt-5.6-terra
 
 ## 下一步
 
-- **PASS**：可進入 implementation 階段。
+- **PASS**：若符合 `design-hardener` 的觸發條件(安全、部署、相依性、migration、失效模式),先執行 `design-hardener`;否則可進入 implementation 階段。
 - **FAIL**：先修正上述 Blocking Issues，再重新審查。
 - **BLOCKED**：先解除文件或工具存取限制，再重新審查。
 

CLAUDE.md

@@ -155,9 +155,9 @@
 
 ## 授權規範
 
-- 已知缺口：`routes/api.php` 的 `member`、`provider` 路由群組只有 `auth:sanctum`，**沒有角色 middleware**；只有 `admin` 群組有 `EnsureAdmin`（待辦見 `docs/analysis/2026-10-14-project-review-todo.md` H1）
+- `routes/api.php` 的 `member`、`provider`、`admin` 路由群組分別掛 `auth:sanctum` + `member`／`provider`／`admin` 角色 middleware（`EnsureMember`／`EnsureProvider`／`EnsureAdmin`，規格見 `openspec/specs/role-authorization/`）
-- 在 middleware 補上之前：新增或修改 member / provider 路由時，Controller 必須明確檢查呼叫者角色與資源所有權，不可假設路由群組已有保護
+- 新路由必須掛對應角色 middleware，並保留 Controller 內的資源所有權檢查（middleware 只擋跨角色，不擋同角色存取他人資源）
-- middleware 補上之後：新路由必須掛對應角色 middleware，並保留 Controller 內的資源所有權檢查
+- member + provider 共用的路由（通知、即時訊息）沒有角色 middleware，Controller 必須自行檢查資源所有權；admin 能否存取這類路由尚未定案
 - 每個涉及權限的路由都必須有跨角色測試（member / provider / admin / 未登入各自的預期結果，參考 `tests/Feature/AdminEndpointAuthTest.php`）
 
 ---
@@ -185,8 +185,11 @@
 | 時機 | 角色 | 輸入 | 用途 |
 |------|------|------|------|
 | OpenSpec 文件（proposal、design、tasks、specs）寫完、**實作之前** | `spec-reviewer` | change 名稱或 `openspec/changes/<name>/` | 檢查需求完整性、安全與權限、AC 與測試任務的雙向對應、與既有規格一致性 |
+| `spec-reviewer` 通過後、**實作之前**，且 change 符合觸發條件 | `design-hardener` | change 名稱或 `openspec/changes/<name>/` | 條件式設計加固：安全原語、Docker 與部署 runtime、Composer／npm 相依、migration 與自動部署、失效模式（不符合觸發條件時直接 PASS，可略過） |
+| Controller／Service 的程式碼變更**全部完成、確定不會再改**之後，呼叫 `strict-reviewer` 之前 | `hardener`（Claude Code skill，位於 `.claude/skills/hardener/`，Pi 不會載入） | 本次變更的方法 | 補強防禦性檢查：參數驗證、資源存在性與擁有權、狀態機、併發、例外邊界、日誌；回答 CLAUDE.md 安全確認問題。只補缺漏，不重寫已正確的程式碼 |
 | 程式碼變更**全部完成**、提 PR 之前 | `strict-reviewer` | 分支（`git diff HEAD`）或指定範圍 | 檢查分層、Eloquent、安全與授權、測試與專案慣例，並執行 `php artisan test`、`pint` |
 
+- 若套用 `hardener` 後因此又修改了程式碼，改完再呼叫 `strict-reviewer`；若套用後確認既有程式碼已符合檢查清單、沒有修改，不需要重複呼叫
 - 結果為 `FAIL` 或 `BLOCKED` 時，必須先修正或解除限制再重新審查，不可直接進入下一階段
 - 審查結果是參考意見，不取代使用者判斷；審查者與實作者意見不同時，由使用者決定
 - `strict-reviewer` 的測試使用 SQLite，不能驗證行鎖併發行為，這類變更需另外人工確認
@@ -210,3 +213,13 @@
 - 發現問題直接說，不要繞彎
 - 有不確定的地方，明確標注「我不確定，需要你確認」
 - 不要重複確認已經決定的技術選擇
+
+## graphify
+
+This project has a knowledge graph at graphify-out/ with god nodes, community structure, and cross-file relationships.
+
+Rules:
+- When graphify-out/graph.json exists, run `graphify query "<question>"` before your first grep, search or source-file read for a codebase question, and put this rule in every subagent prompt that explores code. Read raw files only for lines the graph or an earlier read already located. Use `graphify path "<A>" "<B>"` for relationships and `graphify explain "<concept>"` for focused concepts. These return a scoped subgraph, usually much smaller than GRAPH_REPORT.md or raw grep output.
+- If graphify-out/wiki/index.md exists, use it for broad navigation instead of raw source browsing.
+- Read graphify-out/GRAPH_REPORT.md only for broad architecture review or when query/path/explain do not surface enough context.
+- After modifying code, run `graphify update .` to keep the graph current (AST-only, no API cost).

app/Docs/MemberApiDoc.php

@@ -38,40 +38,48 @@ class MemberApiDoc
      *     operationId="createBooking",
      *     tags={"會員預約"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\JsonContent(
      *             required={"offer_id","schedule_id","participants"},
+     *
      *             @OA\Property(property="offer_id", type="integer", example=1, description="課程 ID"),
      *             @OA\Property(property="schedule_id", type="integer", example=2, description="時段 ID"),
      *             @OA\Property(property="participants", type="integer", example=2, description="參加人數"),
      *             @OA\Property(property="note", type="string", nullable=true, example="需要器材租借")
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=201,
      *         description="預約建立成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="預約成功"),
      *             @OA\Property(property="data", ref="#/components/schemas/Booking")
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=422,
      *         description="驗證失敗或時段已滿",
+     *
      *         @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")
      *     ),
+     *
      *     @OA\Response(
      *         response=403,
      *         description="無權限（非 member 角色）",
+     *
      *         @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")
      *     )
      * )
      */
-    public function createBooking()
+    public function createBooking() {}
-    {
-    }
 
     /**
      * 取得我的預約列表
@@ -83,22 +91,26 @@ class MemberApiDoc
      *     operationId="listMemberBookings",
      *     tags={"會員預約"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="page", in="query", description="頁碼", @OA\Schema(type="integer", default=1)),
      *     @OA\Parameter(name="per_page", in="query", description="每頁筆數", @OA\Schema(type="integer", default=15)),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="data", type="array", @OA\Items(ref="#/components/schemas/Booking")),
      *             @OA\Property(property="meta", ref="#/components/schemas/PaginationMeta")
      *         )
-     *     )
+     *     ),
+     *
+     *     @OA\Response(response=403, description="非 member 角色", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function listMemberBookings()
+    public function listMemberBookings() {}
-    {
-    }
 
     /**
      * 取得單一預約
@@ -109,22 +121,25 @@ class MemberApiDoc
      *     operationId="getMemberBooking",
      *     tags={"會員預約"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="預約 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="data", ref="#/components/schemas/Booking")
      *         )
      *     ),
+     *
      *     @OA\Response(response=404, description="預約不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
-     *     @OA\Response(response=403, description="無權限存取", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
+     *     @OA\Response(response=403, description="非 member 角色或非本人預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function getMemberBooking()
+    public function getMemberBooking() {}
-    {
-    }
 
     /**
      * 取消預約
@@ -136,22 +151,25 @@ class MemberApiDoc
      *     operationId="cancelBooking",
      *     tags={"會員預約"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="預約 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取消成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="預約已取消")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="無權限或狀態不允許取消", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 member 角色、非本人預約或狀態不允許取消", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=404, description="預約不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function cancelBooking()
+    public function cancelBooking() {}
-    {
-    }
 
     // -----------------------------------------------------------------------
     // Member Reviews
@@ -167,32 +185,37 @@ class MemberApiDoc
      *     operationId="createReview",
      *     tags={"會員評價"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\JsonContent(
      *             required={"booking_id","rating"},
+     *
      *             @OA\Property(property="booking_id", type="integer", example=5),
      *             @OA\Property(property="rating", type="integer", minimum=1, maximum=5, example=4),
      *             @OA\Property(property="comment", type="string", nullable=true, example="課程非常棒！"),
      *             @OA\Property(property="is_anonymous", type="boolean", example=false)
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=201,
      *         description="評價建立成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="評價已提交"),
      *             @OA\Property(property="data", ref="#/components/schemas/Review")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="預約未完成或非本人預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 member 角色、預約未完成或非本人預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=422, description="驗證失敗", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function createReview()
+    public function createReview() {}
-    {
-    }
 
     /**
      * 更新評價
@@ -203,30 +226,36 @@ class MemberApiDoc
      *     operationId="updateReview",
      *     tags={"會員評價"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="評價 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="rating", type="integer", minimum=1, maximum=5, example=5),
      *             @OA\Property(property="comment", type="string", nullable=true, example="更新後的評語"),
      *             @OA\Property(property="is_anonymous", type="boolean", example=true)
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="更新成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="data", ref="#/components/schemas/Review")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人評價", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 member 角色或非本人評價", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=404, description="評價不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function updateReview()
+    public function updateReview() {}
-    {
-    }
 
     /**
      * 刪除評價
@@ -237,22 +266,25 @@ class MemberApiDoc
      *     operationId="deleteReview",
      *     tags={"會員評價"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="評價 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="刪除成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="評價已刪除")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人評價", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 member 角色或非本人評價", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=404, description="評價不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function deleteReview()
+    public function deleteReview() {}
-    {
-    }
 
     /**
      * 標記評價為有幫助
@@ -264,22 +296,25 @@ class MemberApiDoc
      *     operationId="markReviewHelpful",
      *     tags={"會員評價"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="評價 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="操作成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="helpful_count", type="integer", example=4),
      *             @OA\Property(property="has_voted", type="boolean", example=true)
      *         )
      *     ),
+     *
      *     @OA\Response(response=404, description="評價不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function markReviewHelpful()
+    public function markReviewHelpful() {}
-    {
-    }
 
     // -----------------------------------------------------------------------
     // Booking Messages（即時訊息，Member + Provider 共用）
@@ -295,10 +330,13 @@ class MemberApiDoc
      *     operationId="getMessageUnreadCounts",
      *     tags={"即時訊息"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(
      *                 property="data",
@@ -310,9 +348,7 @@ class MemberApiDoc
      *     )
      * )
      */
-    public function getMessageUnreadCounts()
+    public function getMessageUnreadCounts() {}
-    {
-    }
 
     /**
      * 取得預約訊息列表
@@ -324,17 +360,23 @@ class MemberApiDoc
      *     operationId="listBookingMessages",
      *     tags={"即時訊息"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="booking", in="path", required=true, description="預約 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(
      *                 property="data",
      *                 type="array",
+     *
      *                 @OA\Items(
      *                     type="object",
+     *
      *                     @OA\Property(property="id", type="integer", example=1),
      *                     @OA\Property(property="booking_id", type="integer", example=12),
      *                     @OA\Property(property="sender_type", type="string", enum={"member","provider"}, example="member"),
@@ -346,13 +388,12 @@ class MemberApiDoc
      *             )
      *         )
      *     ),
+     *
      *     @OA\Response(response=403, description="非參與方或預約狀態不符", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=404, description="預約不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function listBookingMessages()
+    public function listBookingMessages() {}
-    {
-    }
 
     /**
      * 發送預約訊息
@@ -364,21 +405,29 @@ class MemberApiDoc
      *     operationId="sendBookingMessage",
      *     tags={"即時訊息"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="booking", in="path", required=true, description="預約 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\MediaType(
      *             mediaType="multipart/form-data",
+     *
      *             @OA\Schema(
+     *
      *                 @OA\Property(property="message", type="string", nullable=true, example="請問需要自備潛水裝備嗎？"),
      *                 @OA\Property(property="image", type="string", format="binary", nullable=true, description="圖片（jpg/png，message 與 image 至少擇一）")
      *             )
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=201,
      *         description="訊息已送出",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(
      *                 property="data",
@@ -393,13 +442,12 @@ class MemberApiDoc
      *             )
      *         )
      *     ),
+     *
      *     @OA\Response(response=403, description="非參與方或預約狀態不符", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=422, description="message 與 image 皆為空", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function sendBookingMessage()
+    public function sendBookingMessage() {}
-    {
-    }
 
     /**
      * 標記訊息為已讀
@@ -411,21 +459,24 @@ class MemberApiDoc
      *     operationId="markMessagesRead",
      *     tags={"即時訊息"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="booking", in="path", required=true, description="預約 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="標記成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="已標記為已讀")
      *         )
      *     ),
+     *
      *     @OA\Response(response=403, description="非參與方", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function markMessagesRead()
+    public function markMessagesRead() {}
-    {
-    }
 
     // -----------------------------------------------------------------------
     // Notifications (Member + Provider 共用)
@@ -441,18 +492,24 @@ class MemberApiDoc
      *     operationId="listNotifications",
      *     tags={"通知"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="page", in="query", description="頁碼", @OA\Schema(type="integer", default=1)),
      *     @OA\Parameter(name="per_page", in="query", description="每頁筆數", @OA\Schema(type="integer", default=15)),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(
      *                 property="data",
      *                 type="array",
+     *
      *                 @OA\Items(
      *                     type="object",
+     *
      *                     @OA\Property(property="id", type="integer", example=1),
      *                     @OA\Property(property="type", type="string", example="booking_confirmed"),
      *                     @OA\Property(property="title", type="string", example="預約已確認"),
@@ -466,9 +523,7 @@ class MemberApiDoc
      *     )
      * )
      */
-    public function listNotifications()
+    public function listNotifications() {}
-    {
-    }
 
     /**
      * 取得未讀通知數量
@@ -479,19 +534,20 @@ class MemberApiDoc
      *     operationId="getUnreadNotificationCount",
      *     tags={"通知"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="unread_count", type="integer", example=3)
      *         )
      *     )
      * )
      */
-    public function getUnreadNotificationCount()
+    public function getUnreadNotificationCount() {}
-    {
-    }
 
     /**
      * 標記單一通知為已讀
@@ -502,21 +558,24 @@ class MemberApiDoc
      *     operationId="markNotificationRead",
      *     tags={"通知"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="通知 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="標記成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="通知已標記為已讀")
      *         )
      *     ),
+     *
      *     @OA\Response(response=404, description="通知不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function markNotificationRead()
+    public function markNotificationRead() {}
-    {
-    }
 
     /**
      * 全部通知標記為已讀
@@ -527,19 +586,20 @@ class MemberApiDoc
      *     operationId="markAllNotificationsRead",
      *     tags={"通知"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Response(
      *         response=200,
      *         description="標記成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="所有通知已標記為已讀")
      *         )
      *     )
      * )
      */
-    public function markAllNotificationsRead()
+    public function markAllNotificationsRead() {}
-    {
-    }
 
     /**
      * 刪除通知
@@ -550,19 +610,22 @@ class MemberApiDoc
      *     operationId="deleteNotification",
      *     tags={"通知"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="通知 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="刪除成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="通知已刪除")
      *         )
      *     ),
+     *
      *     @OA\Response(response=404, description="通知不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function deleteNotification()
+    public function deleteNotification() {}
-    {
-    }
 }

app/Docs/ProviderApiDoc.php

@@ -42,10 +42,13 @@ class ProviderApiDoc
      *     operationId="getProviderVerification",
      *     tags={"教練驗證"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(
      *                 property="data",
@@ -55,8 +58,10 @@ class ProviderApiDoc
      *                 @OA\Property(
      *                     property="certifications",
      *                     type="array",
+     *
      *                     @OA\Items(
      *                         type="object",
+     *
      *                         @OA\Property(property="id", type="integer", example=1),
      *                         @OA\Property(property="url", type="string", example="http://localhost:8080/storage/providers/5/certifications/uuid.jpg")
      *                     )
@@ -64,12 +69,11 @@ class ProviderApiDoc
      *             )
      *         )
      *     ),
-     *     @OA\Response(response=403, description="無權限", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
+     *
+     *     @OA\Response(response=403, description="非 provider 角色", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function getProviderVerification()
+    public function getProviderVerification() {}
-    {
-    }
 
     /**
      * 上傳證照圖片
@@ -81,20 +85,27 @@ class ProviderApiDoc
      *     operationId="uploadCertification",
      *     tags={"教練驗證"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\MediaType(
      *             mediaType="multipart/form-data",
+     *
      *             @OA\Schema(
      *                 required={"certification"},
+     *
      *                 @OA\Property(property="certification", type="string", format="binary", description="證照圖片（jpg/png，最大 5MB）")
      *             )
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="上傳成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(
      *                 property="data",
@@ -104,13 +115,12 @@ class ProviderApiDoc
      *             )
      *         )
      *     ),
+     *
      *     @OA\Response(response=422, description="圖片驗證失敗", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
-     *     @OA\Response(response=403, description="無權限", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
+     *     @OA\Response(response=403, description="非 provider 角色", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function uploadCertification()
+    public function uploadCertification() {}
-    {
-    }
 
     /**
      * 刪除證照圖片
@@ -122,22 +132,25 @@ class ProviderApiDoc
      *     operationId="deleteCertification",
      *     tags={"教練驗證"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="證照 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="刪除成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="證照已刪除")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="無權限或當前狀態不可刪除", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色、非本人證照或當前狀態不可刪除", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=404, description="證照不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function deleteCertification()
+    public function deleteCertification() {}
-    {
-    }
 
     /**
      * 送出驗證申請
@@ -149,21 +162,23 @@ class ProviderApiDoc
      *     operationId="submitVerification",
      *     tags={"教練驗證"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Response(
      *         response=200,
      *         description="送審成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="驗證申請已送出")
      *         )
      *     ),
+     *
      *     @OA\Response(response=422, description="尚未上傳證照或當前狀態不可送審", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
-     *     @OA\Response(response=403, description="無權限", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
+     *     @OA\Response(response=403, description="非 provider 角色", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function submitVerification()
+    public function submitVerification() {}
-    {
-    }
 
     // -----------------------------------------------------------------------
     // Provider Offers CRUD
@@ -179,22 +194,26 @@ class ProviderApiDoc
      *     operationId="listProviderOffers",
      *     tags={"教練課程"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="page", in="query", description="頁碼", @OA\Schema(type="integer", default=1)),
      *     @OA\Parameter(name="per_page", in="query", description="每頁筆數", @OA\Schema(type="integer", default=15)),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="data", type="array", @OA\Items(ref="#/components/schemas/DivingOffer")),
      *             @OA\Property(property="meta", ref="#/components/schemas/PaginationMeta")
      *         )
-     *     )
+     *     ),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function listProviderOffers()
+    public function listProviderOffers() {}
-    {
-    }
 
     /**
      * 建立課程
@@ -206,10 +225,13 @@ class ProviderApiDoc
      *     operationId="createProviderOffer",
      *     tags={"教練課程"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\JsonContent(
      *             required={"title","description","region","price","max_participants"},
+     *
      *             @OA\Property(property="title", type="string", example="基礎開放水域課程"),
      *             @OA\Property(property="description", type="string", example="適合初學者的 OWD 課程"),
      *             @OA\Property(property="region", type="string", example="墾丁"),
@@ -218,22 +240,24 @@ class ProviderApiDoc
      *             @OA\Property(property="tags", type="array", @OA\Items(type="string"), example={"初學","OWD"})
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=201,
      *         description="課程建立成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="課程建立成功"),
      *             @OA\Property(property="data", ref="#/components/schemas/DivingOffer")
      *         )
      *     ),
+     *
      *     @OA\Response(response=422, description="驗證失敗", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
-     *     @OA\Response(response=403, description="無權限", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
+     *     @OA\Response(response=403, description="非 provider 角色", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function createProviderOffer()
+    public function createProviderOffer() {}
-    {
-    }
 
     /**
      * 取得單一課程（Provider 視角）
@@ -245,22 +269,25 @@ class ProviderApiDoc
      *     operationId="getProviderOffer",
      *     tags={"教練課程"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="課程 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="data", ref="#/components/schemas/DivingOffer")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=404, description="課程不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function getProviderOffer()
+    public function getProviderOffer() {}
-    {
-    }
 
     /**
      * 更新課程
@@ -271,10 +298,14 @@ class ProviderApiDoc
      *     operationId="updateProviderOffer",
      *     tags={"教練課程"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="課程 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="title", type="string", example="進階開放水域課程"),
      *             @OA\Property(property="description", type="string", example="AOWD 課程"),
      *             @OA\Property(property="region", type="string", example="小琉球"),
@@ -284,21 +315,23 @@ class ProviderApiDoc
      *             @OA\Property(property="is_active", type="boolean", example=true)
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="更新成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="data", ref="#/components/schemas/DivingOffer")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=422, description="驗證失敗", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function updateProviderOffer()
+    public function updateProviderOffer() {}
-    {
-    }
 
     /**
      * 刪除課程
@@ -309,22 +342,25 @@ class ProviderApiDoc
      *     operationId="deleteProviderOffer",
      *     tags={"教練課程"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="課程 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="刪除成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="課程已刪除")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=404, description="課程不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function deleteProviderOffer()
+    public function deleteProviderOffer() {}
-    {
-    }
 
     // -----------------------------------------------------------------------
     // Provider Offer Images
@@ -340,32 +376,39 @@ class ProviderApiDoc
      *     operationId="uploadOfferCover",
      *     tags={"教練圖片"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="課程 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\MediaType(
      *             mediaType="multipart/form-data",
+     *
      *             @OA\Schema(
      *                 required={"cover_image"},
+     *
      *                 @OA\Property(property="cover_image", type="string", format="binary", description="封面圖片（jpg/png，最大 5MB）")
      *             )
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="上傳成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="cover_image_url", type="string", example="https://example.com/covers/1.jpg")
      *         )
      *     ),
+     *
      *     @OA\Response(response=422, description="圖片驗證失敗", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
-     *     @OA\Response(response=403, description="非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function uploadOfferCover()
+    public function uploadOfferCover() {}
-    {
-    }
 
     /**
      * 刪除封面圖片
@@ -376,21 +419,24 @@ class ProviderApiDoc
      *     operationId="deleteOfferCover",
      *     tags={"教練圖片"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="課程 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="刪除成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="封面圖片已刪除")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function deleteOfferCover()
+    public function deleteOfferCover() {}
-    {
-    }
 
     /**
      * 上傳相簿圖片
@@ -402,37 +448,45 @@ class ProviderApiDoc
      *     operationId="uploadOfferImages",
      *     tags={"教練圖片"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="課程 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\MediaType(
      *             mediaType="multipart/form-data",
+     *
      *             @OA\Schema(
      *                 required={"images[]"},
+     *
      *                 @OA\Property(
      *                     property="images[]",
      *                     type="array",
+     *
      *                     @OA\Items(type="string", format="binary"),
      *                     description="相簿圖片（每張最大 5MB）"
      *                 )
      *             )
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="上傳成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="images", type="array", @OA\Items(type="string"), example={"https://example.com/img/1.jpg"})
      *         )
      *     ),
+     *
      *     @OA\Response(response=422, description="圖片驗證失敗", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
-     *     @OA\Response(response=403, description="非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function uploadOfferImages()
+    public function uploadOfferImages() {}
-    {
-    }
 
     /**
      * 刪除相簿圖片
@@ -443,22 +497,25 @@ class ProviderApiDoc
      *     operationId="deleteOfferImage",
      *     tags={"教練圖片"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="圖片 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="刪除成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="圖片已刪除")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人圖片", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人圖片", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=404, description="圖片不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function deleteOfferImage()
+    public function deleteOfferImage() {}
-    {
-    }
 
     // -----------------------------------------------------------------------
     // Provider Schedules
@@ -474,20 +531,24 @@ class ProviderApiDoc
      *     operationId="listProviderSchedules",
      *     tags={"教練時段"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="offer_id", in="query", required=false, description="課程 ID 篩選", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="data", type="array", @OA\Items(ref="#/components/schemas/CourseSchedule"))
      *         )
-     *     )
+     *     ),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function listProviderSchedules()
+    public function listProviderSchedules() {}
-    {
-    }
 
     /**
      * 建立時段
@@ -498,31 +559,36 @@ class ProviderApiDoc
      *     operationId="createProviderSchedule",
      *     tags={"教練時段"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\JsonContent(
      *             required={"offer_id","start_date","end_date","available_slots"},
+     *
      *             @OA\Property(property="offer_id", type="integer", example=1),
      *             @OA\Property(property="start_date", type="string", format="date", example="2025-07-01"),
      *             @OA\Property(property="end_date", type="string", format="date", example="2025-07-03"),
      *             @OA\Property(property="available_slots", type="integer", example=4)
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=201,
      *         description="時段建立成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="data", ref="#/components/schemas/CourseSchedule")
      *         )
      *     ),
+     *
      *     @OA\Response(response=422, description="驗證失敗", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
-     *     @OA\Response(response=403, description="非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function createProviderSchedule()
+    public function createProviderSchedule() {}
-    {
-    }
 
     /**
      * 更新時段
@@ -533,31 +599,37 @@ class ProviderApiDoc
      *     operationId="updateProviderSchedule",
      *     tags={"教練時段"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="時段 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\RequestBody(
      *         required=true,
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="start_date", type="string", format="date", example="2025-07-05"),
      *             @OA\Property(property="end_date", type="string", format="date", example="2025-07-07"),
      *             @OA\Property(property="available_slots", type="integer", example=6),
      *             @OA\Property(property="is_active", type="boolean", example=true)
      *         )
      *     ),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="更新成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="data", ref="#/components/schemas/CourseSchedule")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人時段", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人時段", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=404, description="時段不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function updateProviderSchedule()
+    public function updateProviderSchedule() {}
-    {
-    }
 
     /**
      * 刪除時段
@@ -568,22 +640,25 @@ class ProviderApiDoc
      *     operationId="deleteProviderSchedule",
      *     tags={"教練時段"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="時段 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="刪除成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="時段已刪除")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人時段", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人時段", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=404, description="時段不存在", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function deleteProviderSchedule()
+    public function deleteProviderSchedule() {}
-    {
-    }
 
     // -----------------------------------------------------------------------
     // Provider Bookings Management
@@ -599,22 +674,26 @@ class ProviderApiDoc
      *     operationId="listProviderBookings",
      *     tags={"教練預約"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="page", in="query", description="頁碼", @OA\Schema(type="integer", default=1)),
      *     @OA\Parameter(name="per_page", in="query", description="每頁筆數", @OA\Schema(type="integer", default=15)),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取得成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="data", type="array", @OA\Items(ref="#/components/schemas/Booking")),
      *             @OA\Property(property="meta", ref="#/components/schemas/PaginationMeta")
      *         )
-     *     )
+     *     ),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function listProviderBookings()
+    public function listProviderBookings() {}
-    {
-    }
 
     /**
      * 確認預約
@@ -625,23 +704,26 @@ class ProviderApiDoc
      *     operationId="confirmBooking",
      *     tags={"教練預約"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="預約 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="確認成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="預約已確認"),
      *             @OA\Property(property="data", ref="#/components/schemas/Booking")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人課程的預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程的預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=422, description="狀態不允許確認", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function confirmBooking()
+    public function confirmBooking() {}
-    {
-    }
 
     /**
      * 拒絕預約
@@ -652,23 +734,26 @@ class ProviderApiDoc
      *     operationId="rejectBooking",
      *     tags={"教練預約"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="預約 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="拒絕成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="預約已拒絕"),
      *             @OA\Property(property="data", ref="#/components/schemas/Booking")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人課程的預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程的預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=422, description="狀態不允許拒絕", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function rejectBooking()
+    public function rejectBooking() {}
-    {
-    }
 
     /**
      * 標記預約完成
@@ -679,23 +764,26 @@ class ProviderApiDoc
      *     operationId="completeBookingByProvider",
      *     tags={"教練預約"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="預約 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="標記成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="預約已完成"),
      *             @OA\Property(property="data", ref="#/components/schemas/Booking")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人課程的預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程的預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=422, description="狀態不允許完成", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function completeBookingByProvider()
+    public function completeBookingByProvider() {}
-    {
-    }
 
     /**
      * 取消預約（Provider）
@@ -706,21 +794,24 @@ class ProviderApiDoc
      *     operationId="cancelBookingByProvider",
      *     tags={"教練預約"},
      *     security={{"bearerAuth": {}}},
+     *
      *     @OA\Parameter(name="id", in="path", required=true, description="預約 ID", @OA\Schema(type="integer")),
+     *
      *     @OA\Response(
      *         response=200,
      *         description="取消成功",
+     *
      *         @OA\JsonContent(
+     *
      *             @OA\Property(property="status", type="boolean", example=true),
      *             @OA\Property(property="message", type="string", example="預約已取消"),
      *             @OA\Property(property="data", ref="#/components/schemas/Booking")
      *         )
      *     ),
-     *     @OA\Response(response=403, description="非本人課程的預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
+     *
+     *     @OA\Response(response=403, description="非 provider 角色或非本人課程的預約", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse")),
      *     @OA\Response(response=422, description="狀態不允許取消", @OA\JsonContent(ref="#/components/schemas/ApiErrorResponse"))
      * )
      */
-    public function cancelBookingByProvider()
+    public function cancelBookingByProvider() {}
-    {
-    }
 }

app/Exceptions/BookingRuleViolationException.php

@@ -0,0 +1,5 @@
+<?php
+
+namespace App\Exceptions;
+
+final class BookingRuleViolationException extends \RuntimeException {}

app/Http/Controllers/API/MemberBookingController.php

@@ -4,11 +4,12 @@ namespace App\Http\Controllers\API;
 
 use App\Enums\BookingStatus;
 use App\Enums\ScheduleStatus;
+use App\Exceptions\BookingRuleViolationException;
 use App\Http\Controllers\Controller;
 use App\Models\Booking;
 use App\Models\CourseSchedule;
-use App\Notifications\BookingCreatedNotification;
 use App\Notifications\BookingCancelledNotification;
+use App\Notifications\BookingCreatedNotification;
 use Carbon\Carbon;
 use Illuminate\Http\Request;
 use Illuminate\Support\Facades\DB;
@@ -22,7 +23,7 @@ class MemberBookingController extends Controller
             ->where('member_id', $request->user()->id)
             ->orderByDesc('created_at')
             ->get()
-            ->map(fn($b) => $this->formatBooking($b));
+            ->map(fn ($b) => $this->formatBooking($b));
 
         return response()->json(['status' => true, 'data' => $bookings]);
     }
@@ -40,9 +41,9 @@ class MemberBookingController extends Controller
     public function store(Request $request)
     {
         $data = $request->validate([
-            'schedule_id'  => 'required|integer|exists:course_schedules,id',
+            'schedule_id' => 'required|integer|exists:course_schedules,id',
             'participants' => 'required|integer|min:1',
-            'notes'        => 'nullable|string|max:500',
+            'notes' => 'nullable|string|max:500',
         ]);
 
         $schedule = CourseSchedule::with('divingOffer.provider.providerProfile')->findOrFail($data['schedule_id']);
@@ -50,7 +51,7 @@ class MemberBookingController extends Controller
         // Layer 1：快速失敗
         // 可見性繞過防護：課程教練未通過審核時不可建立新預約（既有預約不受影響，見 provider-verification 規格）
         $offer = $schedule->divingOffer;
-        if ($offer->provider_id !== null && !($offer->provider?->providerProfile?->is_verified)) {
+        if ($offer->provider_id !== null && ! ($offer->provider?->providerProfile?->is_verified)) {
             // is_verified 為 accessor（= verification_status === approved）
             return response()->json(['status' => false, 'message' => '此課程目前不開放預約'], 422);
         }
@@ -68,7 +69,7 @@ class MemberBookingController extends Controller
                 // Layer 2：lockForUpdate 後二次驗證
                 $schedule = CourseSchedule::lockForUpdate()->find($schedule->id);
                 if ($data['participants'] > $schedule->remainingSpots()) {
-                    throw new \RuntimeException('名額不足，請重新選擇');
+                    throw new BookingRuleViolationException('名額不足，請重新選擇');
                 }
 
                 // 重複預約檢查
@@ -77,19 +78,19 @@ class MemberBookingController extends Controller
                     ->whereIn('status', [BookingStatus::Pending->value, BookingStatus::Confirmed->value])
                     ->exists();
                 if ($duplicate) {
-                    throw new \RuntimeException('您已預約此時段');
+                    throw new BookingRuleViolationException('您已預約此時段');
                 }
 
                 return Booking::create([
-                    'schedule_id'  => $schedule->id,
+                    'schedule_id' => $schedule->id,
-                    'member_id'    => $memberId,
+                    'member_id' => $memberId,
                     'participants' => $data['participants'],
-                    'total_price'  => $schedule->divingOffer->price * $data['participants'],
+                    'total_price' => $schedule->divingOffer->price * $data['participants'],
-                    'status'       => BookingStatus::Pending,
+                    'status' => BookingStatus::Pending,
-                    'notes'        => $data['notes'] ?? null,
+                    'notes' => $data['notes'] ?? null,
                 ]);
             });
-        } catch (\RuntimeException $e) {
+        } catch (BookingRuleViolationException $e) {
             return response()->json(['status' => false, 'message' => $e->getMessage()], 422);
         }
 
@@ -98,13 +99,13 @@ class MemberBookingController extends Controller
             $provider = $booking->schedule->divingOffer->provider;
             $provider->notify(new BookingCreatedNotification($booking));
         } catch (\Throwable $e) {
-            Log::error('BookingCreatedNotification failed: ' . $e->getMessage());
+            Log::error('BookingCreatedNotification failed: '.$e->getMessage());
         }
 
         return response()->json([
-            'status'  => true,
+            'status' => true,
             'message' => '預約已送出，等待教練確認',
-            'data'    => $this->formatBooking($booking->fresh(['schedule.divingOffer'])),
+            'data' => $this->formatBooking($booking->fresh(['schedule.divingOffer'])),
         ], 201);
     }
 
@@ -116,13 +117,13 @@ class MemberBookingController extends Controller
         }
 
         $canCancelFrom = [BookingStatus::Pending, BookingStatus::Confirmed];
-        if (!in_array($booking->status, $canCancelFrom)) {
+        if (! in_array($booking->status, $canCancelFrom)) {
             return response()->json(['status' => false, 'message' => '此預約狀態無法取消'], 422);
         }
 
         // 24h 截止驗證
         $schedule = $booking->schedule;
-        $courseStart = Carbon::parse($schedule->scheduled_date->toDateString() . ' ' . $schedule->start_time);
+        $courseStart = Carbon::parse($schedule->scheduled_date->toDateString().' '.$schedule->start_time);
         if (now()->diffInHours($courseStart, false) < 24) {
             return response()->json(['status' => false, 'message' => '距課程開始不足 24 小時，無法取消，請聯繫教練'], 422);
         }
@@ -148,7 +149,7 @@ class MemberBookingController extends Controller
             $provider = $booking->schedule->divingOffer->provider;
             $provider->notify(new BookingCancelledNotification($booking, cancelledBy: 'member'));
         } catch (\Throwable $e) {
-            Log::error('BookingCancelledNotification(member) failed: ' . $e->getMessage());
+            Log::error('BookingCancelledNotification(member) failed: '.$e->getMessage());
         }
 
         return response()->json(['status' => true, 'message' => '預約已取消']);
@@ -157,20 +158,21 @@ class MemberBookingController extends Controller
     private function formatBooking(Booking $b): array
     {
         $offer = $b->schedule?->divingOffer;
+
         return [
-            'id'             => $b->id,
+            'id' => $b->id,
-            'offer_id'       => $offer?->id,
+            'offer_id' => $offer?->id,
-            'offer_title'    => $offer?->title,
+            'offer_title' => $offer?->title,
             'offer_location' => $offer?->location,
-            'offer_region'   => $offer?->region,
+            'offer_region' => $offer?->region,
-            'offer_price'    => $offer?->price,
+            'offer_price' => $offer?->price,
             'scheduled_date' => $b->schedule?->scheduled_date?->toDateString(),
-            'start_time'     => $b->schedule?->start_time,
+            'start_time' => $b->schedule?->start_time,
-            'participants'   => $b->participants,
+            'participants' => $b->participants,
-            'total_price'    => $b->total_price,
+            'total_price' => $b->total_price,
-            'status'         => $b->status->value,
+            'status' => $b->status->value,
-            'notes'          => $b->notes,
+            'notes' => $b->notes,
-            'created_at'     => $b->created_at?->toISOString(),
+            'created_at' => $b->created_at?->toISOString(),
         ];
     }
 }

app/Http/Controllers/API/ProviderBookingController.php

@@ -4,6 +4,7 @@ namespace App\Http\Controllers\API;
 
 use App\Enums\BookingStatus;
 use App\Enums\ScheduleStatus;
+use App\Exceptions\BookingRuleViolationException;
 use App\Http\Controllers\Controller;
 use App\Models\Booking;
 use App\Notifications\BookingCancelledNotification;
@@ -20,10 +21,10 @@ class ProviderBookingController extends Controller
     {
         $provider = $request->user();
         $bookings = Booking::with(['member', 'schedule.divingOffer'])
-            ->whereHas('schedule', fn($q) => $q->where('provider_id', $provider->id))
+            ->whereHas('schedule', fn ($q) => $q->where('provider_id', $provider->id))
             ->orderByDesc('created_at')
             ->get()
-            ->map(fn($b) => $this->formatBooking($b));
+            ->map(fn ($b) => $this->formatBooking($b));
 
         return response()->json(['status' => true, 'data' => $bookings]);
     }
@@ -33,7 +34,7 @@ class ProviderBookingController extends Controller
         $booking = Booking::with('schedule')->findOrFail($id);
         $this->authorizeProvider($request, $booking);
 
-        if (!$booking->canTransitionTo(BookingStatus::Confirmed)) {
+        if (! $booking->canTransitionTo(BookingStatus::Confirmed)) {
             return response()->json(['status' => false, 'message' => '當前狀態無法確認'], 422);
         }
 
@@ -43,7 +44,7 @@ class ProviderBookingController extends Controller
                 $remaining = $schedule->max_participants - $schedule->current_participants;
 
                 if ($booking->participants > $remaining) {
-                    throw new \RuntimeException('名額不足，無法確認此預約');
+                    throw new BookingRuleViolationException('名額不足，無法確認此預約');
                 }
 
                 $booking->update(['status' => BookingStatus::Confirmed]);
@@ -54,7 +55,7 @@ class ProviderBookingController extends Controller
                     $schedule->update(['status' => ScheduleStatus::Full]);
                 }
             });
-        } catch (\RuntimeException $e) {
+        } catch (BookingRuleViolationException $e) {
             return response()->json(['status' => false, 'message' => $e->getMessage()], 422);
         }
 
@@ -62,7 +63,7 @@ class ProviderBookingController extends Controller
             $booking->load('member');
             $booking->member->notify(new BookingConfirmedNotification($booking));
         } catch (\Throwable $e) {
-            Log::error('BookingConfirmedNotification failed: ' . $e->getMessage());
+            Log::error('BookingConfirmedNotification failed: '.$e->getMessage());
         }
 
         return response()->json(['status' => true, 'message' => '預約已確認', 'data' => $this->formatBooking($booking->fresh(['member', 'schedule.divingOffer']))]);
@@ -73,7 +74,7 @@ class ProviderBookingController extends Controller
         $booking = Booking::with('schedule')->findOrFail($id);
         $this->authorizeProvider($request, $booking);
 
-        if (!$booking->canTransitionTo(BookingStatus::Rejected)) {
+        if (! $booking->canTransitionTo(BookingStatus::Rejected)) {
             return response()->json(['status' => false, 'message' => '當前狀態無法拒絕'], 422);
         }
 
@@ -83,7 +84,7 @@ class ProviderBookingController extends Controller
             $booking->load('member');
             $booking->member->notify(new BookingRejectedNotification($booking));
         } catch (\Throwable $e) {
-            Log::error('BookingRejectedNotification failed: ' . $e->getMessage());
+            Log::error('BookingRejectedNotification failed: '.$e->getMessage());
         }
 
         return response()->json(['status' => true, 'message' => '預約已拒絕']);
@@ -94,7 +95,7 @@ class ProviderBookingController extends Controller
         $booking = Booking::with('schedule')->findOrFail($id);
         $this->authorizeProvider($request, $booking);
 
-        if (!$booking->canTransitionTo(BookingStatus::ProviderCancelled)) {
+        if (! $booking->canTransitionTo(BookingStatus::ProviderCancelled)) {
             return response()->json(['status' => false, 'message' => '當前狀態無法取消'], 422);
         }
 
@@ -114,7 +115,7 @@ class ProviderBookingController extends Controller
             $booking->load('member');
             $booking->member->notify(new BookingCancelledNotification($booking, cancelledBy: 'provider'));
         } catch (\Throwable $e) {
-            Log::error('BookingCancelledNotification(provider) failed: ' . $e->getMessage());
+            Log::error('BookingCancelledNotification(provider) failed: '.$e->getMessage());
         }
 
         return response()->json(['status' => true, 'message' => '預約已取消']);
@@ -125,7 +126,7 @@ class ProviderBookingController extends Controller
         $booking = Booking::with('schedule')->findOrFail($id);
         $this->authorizeProvider($request, $booking);
 
-        if (!$booking->canTransitionTo(BookingStatus::Completed)) {
+        if (! $booking->canTransitionTo(BookingStatus::Completed)) {
             return response()->json(['status' => false, 'message' => '只有已確認的預約才能標記完成'], 422);
         }
 
@@ -135,7 +136,7 @@ class ProviderBookingController extends Controller
             $booking->load('member');
             $booking->member->notify(new BookingCompletedNotification($booking));
         } catch (\Throwable $e) {
-            Log::error('BookingCompletedNotification failed: ' . $e->getMessage());
+            Log::error('BookingCompletedNotification failed: '.$e->getMessage());
         }
 
         return response()->json(['status' => true, 'message' => '預約已標記為完成']);
@@ -151,17 +152,17 @@ class ProviderBookingController extends Controller
     private function formatBooking(Booking $b): array
     {
         return [
-            'id'             => $b->id,
+            'id' => $b->id,
-            'member_name'    => $b->member?->name,
+            'member_name' => $b->member?->name,
-            'member_email'   => $b->member?->email,
+            'member_email' => $b->member?->email,
-            'offer_title'    => $b->schedule?->divingOffer?->title,
+            'offer_title' => $b->schedule?->divingOffer?->title,
             'scheduled_date' => $b->schedule?->scheduled_date?->toDateString(),
-            'start_time'     => $b->schedule?->start_time,
+            'start_time' => $b->schedule?->start_time,
-            'participants'   => $b->participants,
+            'participants' => $b->participants,
-            'total_price'    => $b->total_price,
+            'total_price' => $b->total_price,
-            'status'         => $b->status->value,
+            'status' => $b->status->value,
-            'notes'          => $b->notes,
+            'notes' => $b->notes,
-            'created_at'     => $b->created_at?->toISOString(),
+            'created_at' => $b->created_at?->toISOString(),
         ];
     }
 }

app/Http/Middleware/EnsureMember.php

@@ -0,0 +1,19 @@
+<?php
+
+namespace App\Http\Middleware;
+
+use Closure;
+use Illuminate\Http\Request;
+use Symfony\Component\HttpFoundation\Response;
+
+class EnsureMember
+{
+    public function handle(Request $request, Closure $next): Response
+    {
+        if (! $request->user() || ! $request->user()->isMember()) {
+            return response()->json(['status' => false, 'message' => '無權限存取'], 403);
+        }
+
+        return $next($request);
+    }
+}

app/Http/Middleware/EnsureProvider.php

@@ -0,0 +1,19 @@
+<?php
+
+namespace App\Http\Middleware;
+
+use Closure;
+use Illuminate\Http\Request;
+use Symfony\Component\HttpFoundation\Response;
+
+class EnsureProvider
+{
+    public function handle(Request $request, Closure $next): Response
+    {
+        if (! $request->user() || ! $request->user()->isProvider()) {
+            return response()->json(['status' => false, 'message' => '無權限存取'], 403);
+        }
+
+        return $next($request);
+    }
+}

bootstrap/app.php

@@ -16,6 +16,8 @@ return Application::configure(basePath: dirname(__DIR__))
         $middleware->trustProxies(at: ['*']);
         $middleware->alias([
             'admin' => \App\Http\Middleware\EnsureAdmin::class,
+            'member' => \App\Http\Middleware\EnsureMember::class,
+            'provider' => \App\Http\Middleware\EnsureProvider::class,
         ]);
     })
     ->withExceptions(function (Exceptions $exceptions) {

docs/GRAPHIFY.md

@@ -0,0 +1,58 @@
+# graphify 知識圖譜操作筆記
+
+graphify 把專案的程式碼、OpenSpec 文件、設定檔轉成可查詢的知識圖譜，輸出在 `graphify-out/`。
+該目錄被 `.gitignore` 忽略（`graph.json` 每次重建都整份重寫，進版控只會產生無意義 diff），換機器或清掉後需自行重建。
+
+規則（先 query 再 grep、改完程式碼跑 `graphify update .`）見根目錄 `CLAUDE.md`，此處不重複。
+
+## 建置紀錄（2026-10-29）
+
+- 規模：2429 節點、4369 邊、約 117 個社群
+- 來源：253 個程式碼檔以 AST 抽取；218 份文件與 5 張圖片以子代理做語意抽取
+- 語意抽取沒有設 `GEMINI_API_KEY`，由 Claude Code 的子代理擔任 LLM
+
+## 安裝與重建
+
+```bash
+uv tool install --upgrade graphifyy   # 安裝（Windows，graphify 位於 uv tool 目錄）
+/graphify .                           # 在 Claude Code 內完整建置
+graphify update .                     # 只重新處理變更的檔案（AST 免費，語意抽取才耗 token）
+graphify query "<問題>"               # 查詢既有圖譜
+```
+
+完整建置的語意抽取需派約 15 個子代理，**同時最多 4 個**（超過會被拒絕），要分批派。
+
+## 已知限制與踩過的坑
+
+| 項目 | 說明 | 處理 |
+|------|------|------|
+| enum case 引用 | AST 只看得到類別被引用，看不到 `BookingStatus::Pending` 這類 case 的使用處，所以 `Pending`／`Confirmed`／`Completed` 在圖上看起來孤立 | 查 enum case 的使用處改用 grep |
+| 腳本檔名 | 輔助腳本不可命名為 `ast.py`，會遮蔽標準函式庫造成循環匯入 | 換名稱，並用 `python -I` 執行 |
+| 二進位檔覆蓋率 | PNG 沒有「行數」，覆蓋率檢查會誤報 partial（`hero.png`） | 可忽略 |
+| 子代理只讀檔頭 | `.pi/agents/` 三個檔案只讀了檔頭，未寫入 manifest | 與 `.claude/agents/` 僅工具名稱寫法不同，已決定不補 |
+| 同端點邊合併 | YAML 中重複的鍵（如 `steps` 底下多個 `name`）會被合併成一條邊 | 正常現象，不是圖損壞 |
+
+## 社群命名
+
+graphify 預設以「連線最多的節點名稱 + cluster」命名社群。前 16 大社群已手動改名（依成員與所在資料夾判斷）：
+
+| 編號 | 名稱 |
+|------|------|
+| 0 | 預約狀態機與生命週期測試 |
+| 1 | 預約通知（Notification 類別） |
+| 2 | 早期 OpenSpec 設計文件（預約／評價／會員入口） |
+| 3 | 教練端前端（排程／評價／個人檔案） |
+| 4 | 預約即時訊息（Reverb 事件） |
+| 5 | 認證強化 spec（鎖定／OAuth state／稽核補強） |
+| 6 | 課程排程與預約 API |
+| 7 | 會員端前端（我的預約／課程列表） |
+| 8 | AuthController 認證 |
+| 9 | 前端與根目錄 package.json |
+| 10 | 課程詳情頁前端（評價／預約） |
+| 11 | 前端 Pinia 狀態與通知 |
+| 12 | 評價功能測試 |
+| 13 | Docker Compose 服務 |
+| 14 | Feature 測試基底與授權測試 |
+| 15 | 管理後台前端 |
+
+社群編號可能隨重建而改變，此表只對 2026-10-29 這次建置有效；重建後需重新核對。

docs/analysis/2026-10-14-project-prompt-improvements.md

@@ -106,7 +106,7 @@
 
 ### 待辦
 - [x] 決定 subagent 定義的存放位置：`.pi/agents/` 與 `.claude/agents/` 各放一份
-- [x] 先建立兩個唯讀角色：改為移植自 ProjectC 的 `spec-reviewer`（對應 spec-auditor）與 `strict-reviewer`（涵蓋 security-reviewer 並加入測試執行）；`design-hardener` 暫不移植
+- [x] 先建立兩個唯讀角色：改為移植自 ProjectC 的 `spec-reviewer`（對應 spec-auditor）與 `strict-reviewer`（涵蓋 security-reviewer 並加入測試執行）；`design-hardener` 後續也已補上（見下方進度）
 - [ ] 試用一輪後再決定是否建立寫入型角色
 
 ---
@@ -129,7 +129,11 @@
 - 已知限制：`operations.*.guidance` 屬於「建議性」提示（OpenSpec 文件明寫為 prompt-level 行為契約，非可強制的檢查），硬性規則仍以 CLAUDE.md 為準
 - 升級後 `openspec validate --specs --strict` 有 3 份 spec 的 Requirement 超過 500 字元會失敗（account-lockout、booking-lifecycle、image-baked-deployment）；一般模式 39 份全過。日後若要在 CI 使用 `--strict`，需先另開 change 拆分這些 Requirement
 - Subagent：已移植 `spec-reviewer`、`strict-reviewer` 至 `.pi/agents/`（gpt-5.6-terra，跨家族審查）並複製一份到 `.claude/agents/`（sonnet，與實作端同家族），CLAUDE.md 新增「審查流程」；兩套需保持同步；尚未實際跑過，需以真實 change 或 H1 漏洞試跑驗證
-- 尚待處理：`security-checklist`、`add-endpoint`、`design-hardener`（視需要）
+- 已補上 `design-hardener`（`.pi/agents/` 與 `.claude/agents/` 各一份），針對 Laravel／Docker 自動部署重寫：新增 migration 與自動 `migrate --force`、`config:cache` 與 `env()`、`trustProxies(at: ['*'])`、Redis fail-open／fail-closed、queue worker 與 image 烘焙等檢查；流程為 spec-reviewer 通過後、實作前，符合觸發條件才需執行
+- 工具與流程：H1 等實際開發由使用者規劃後以 Claude Code 執行，本次不開始
+- 已新增 `hardener` skill（`.claude/skills/hardener/`，Claude Code 專用）：改寫自 ProjectC 的 C# 版，替換為 Laravel 慣例；流程為實作完成後、`strict-reviewer` 之前執行，規則已寫入 CLAUDE.md「審查流程」
+- ProjectC 其餘 skill 評估後不帶入：`spec-scope`（本專案已有約 40 份 spec）、UI 設計類 skill 與 `output-skill`（暫無 UI 改版計畫，且第三方授權未確認）
+- 尚待處理：`security-checklist`、`add-endpoint`（低優先）
 
 ## 需要你決定的事項（歷史紀錄，下列 1 已決定）
 

docs/analysis/2026-10-14-project-review-todo.md

@@ -5,6 +5,7 @@
 > 低優先項目未列入本文（分頁、範本殘留檔、CHANGELOG 等），需要時另行整理。
 > 搭配閱讀：《2026-06-11-spec-implementation-audit.md》。
 > 實作原則：以下每一項只要涉及行為變更，都必須先走 OpenSpec（`/opsx:propose`），AC 對應測試後才可實作。
+> 更新紀錄：2026-10-14 新增 M7、M8（撰寫 `hardener` skill 時，閱讀預約相關程式碼發現；以程式碼閱讀判斷，**尚未重現驗證**）；2026-10-17 新增「功能與延後項目」F1–F7（彙整既有規劃文件與已歸檔 change 的未完成任務）。
 
 ---
 
@@ -12,17 +13,19 @@
 
 | # | 優先 | 項目 | 是否走 OpenSpec | 建議 change 名稱 |
 |---|------|------|----------------|------------------|
-| H1 | 🔴 高 | member／provider 路由缺角色 middleware | 是 | `add-role-middleware` |
+| H1 | ✅ 已完成（PR #71、#72） | member／provider 路由缺角色 middleware | 是 | `add-role-middleware` |
-| H2 | 🔴 高 | 正式環境 `APP_DEBUG` 與預設值風險 | 否（設定確認）+ 視情況改 `.env.example` | — |
+| H2 | ✅ 已完成（2026-10-18 確認） | 正式環境 `APP_DEBUG` 與預設值風險 | 否（設定確認）+ 視情況改 `.env.example` | — |
 | M1 | 🟡 中 | AuthController 過大、無 FormRequest／Service 層 | 是 | `refactor-auth-controller` |
 | M2 | 🟡 中 | 預約名額邏輯重複 | 是 | `extract-booking-service` |
 | M3 | 🟡 中（已下修，見內文） | 通知派送的例外處理與失敗可觀測性 | 視範圍而定 | `harden-notification-dispatch` |
 | M4 | 🟡 中 | 前端 token 儲存不一致（admin 用 localStorage） | 是 | `unify-frontend-token-storage` |
 | M5 | 🟡 中 | CORS 在正式環境仍允許 localhost | 是（小） | `restrict-cors-by-env` |
 | M6 | 🟡 中 | CI 使用 SQLite，鎖定／超賣測試可能假綠 | 是 | `ci-mysql-test-job` |
+| M7 | ✅ 已完成（PR #74、#75） | 預約流程 `catch (RuntimeException)` 把例外訊息直接回給使用者 | 是（小） | `fix-booking-exception-handling` |
+| M8 | 🟡 中（影響資料正確性，需並發才觸發） | 預約取消／確認的狀態在鎖之前讀取，並發下名額可能重複增減 | 是 | `fix-booking-state-race`（建議併入 M2） |
 
-建議順序：H1 → H2 → M6 → M5 → M2 → M1 → M4 → M3（M3 已下修，可視驗證結果延後或關閉）。
+建議順序：H1 → H2 → M7 → M6 → M5 → M8 與 M2 合併處理 → M1 → M4 → M3（M3 已下修，可視驗證結果延後或關閉）。
-理由：H1 是安全漏洞；M6 先建立可信的測試基礎，後續重構（M1、M2）才有安全網。
+理由：H1 是安全漏洞；M7 範圍小、可立即降低資訊洩漏；M6 先建立可信的測試基礎，後續 M8、M2、M1 才有安全網；M8 的修法（鎖內重讀狀態）本來就是 M2 抽出 `BookingService` 時要統一的邏輯，建議同一個 change 處理。
 
 ---
 
@@ -37,12 +40,12 @@
 - 目前只有 `ReviewController`、refresh 端點自己檢查角色，其餘端點無防護。
 
 **待辦**
-- [ ] 新增 `EnsureMember`、`EnsureProvider` middleware（仿 `app/Http/Middleware/EnsureAdmin.php`），並在 `bootstrap/app.php` 註冊 alias
+- [x] 新增 `EnsureMember`、`EnsureProvider` middleware（仿 `app/Http/Middleware/EnsureAdmin.php`），並在 `bootstrap/app.php` 註冊 alias
-- [ ] 套用到 `member`、`provider` 路由群組
+- [x] 套用到 `member`、`provider` 路由群組
-- [ ] 通知、即時訊息這類「member + provider 共用」群組維持現狀，但需確認 admin 是否應被排除
+- [ ] 通知、即時訊息這類「member + provider 共用」群組維持現狀，但需確認 admin 是否應被排除（未定案：本次維持現狀，admin 是否排除另案決定）
-- [ ] 新增 Feature 測試（仿 `tests/Feature/AdminEndpointAuthTest.php`）：每個角色 × 每組路由，期望 403
+- [x] 新增 Feature 測試（仿 `tests/Feature/AdminEndpointAuthTest.php`）：每個角色 × 每組路由，期望 403
-- [ ] 更新 spec：`provider-auth`、`admin-auth`；視需要新增 `role-authorization` spec
+- [x] 更新 spec：`provider-auth`、`admin-auth`；視需要新增 `role-authorization` spec（實際做法：新增 `role-authorization` 主規格，未修改 `provider-auth`、`admin-auth`）
-- [ ] 更新 Swagger 註解（`app/Docs/*`）的 403 回應
+- [x] 更新 Swagger 註解（`app/Docs/*`）的 403 回應
 
 **驗收條件（草案，需轉為 spec Scenario）**
 1. member 呼叫任一 `/api/provider/*` 回 403
@@ -67,11 +70,11 @@
 若 VPS 上 `.env` 沿用相同值，錯誤頁會洩漏堆疊追蹤、環境變數與 SQL。
 
 **待辦**
-- [ ] **需要你確認**：登入 VPS 檢查實際 `.env` 的 `APP_DEBUG`、`APP_ENV`（我無法從此環境確認）
+- [x] **需要你確認**：登入 VPS 檢查實際 `.env` 的 `APP_DEBUG`、`APP_ENV`（我無法從此環境確認）（2026-10-18：`APP_DEBUG=false`、`APP_ENV=production`）
-- [ ] 確認 `config:cache` 後的實際值：`docker compose exec app php artisan tinker --execute="dump(config('app.debug'));"`
+- [x] 確認 `config:cache` 後的實際值：`docker compose exec app php artisan tinker --execute="dump(config('app.debug'));"`
-- [ ] `.env.example` 加註解說明正式環境必須 `APP_DEBUG=false`
+- [x] `.env.example` 加註解說明正式環境必須 `APP_DEBUG=false`
-- [ ] 評估是否在 deploy.yml 加入啟動檢查：`APP_ENV=production` 且 `APP_DEBUG` 為 false，否則部署失敗
+- [x] 評估是否在 deploy.yml 加入啟動檢查：`APP_ENV=production` 且 `APP_DEBUG` 為 false，否則部署失敗（2026-10-18 決定暫不做：正式環境已確認，需要時另開 change）
-- [ ] 本機 `.env` 中的 DB 密碼為弱密碼（`cfdivepass`），確認正式環境未使用
+- [x] 本機 `.env` 中的 DB 密碼為弱密碼（`cfdivepass`），確認正式環境未使用（2026-10-18：正式環境未使用預設密碼，且 MySQL 未對外開 port）
 
 **備註**
 這項大多是設定確認，不一定需要 OpenSpec change；若加入部署檢查則屬 `env-cloud-annotations` spec 範圍。
@@ -197,6 +200,98 @@ CI 綠燈不代表正式環境（MySQL）行為正確，尤其是併發情境。
 
 ---
 
+## M7　預約流程 `catch (RuntimeException)` 把例外訊息直接回給使用者
+
+> 來源：撰寫 `hardener` skill 時閱讀程式碼發現。以下為程式碼閱讀結果，尚未實際觸發驗證。
+
+**現況**
+- `app/Http/Controllers/API/MemberBookingController.php:92`（`store`）與 `app/Http/Controllers/API/ProviderBookingController.php:57`（`confirm`）都是：
+  `catch (\RuntimeException $e) { return response()->json(['status' => false, 'message' => $e->getMessage()], 422); }`
+- 閉包內刻意拋出的業務訊息（「名額不足」「您已預約此時段」）依賴這個 catch 轉成 422
+
+**風險**
+- `Illuminate\Database\QueryException` 繼承 `PDOException`，而 `PDOException` 是 `RuntimeException` 的子類別。交易內的資料庫錯誤（死結、逾時、約束違反、連線中斷）也會被這個 catch 接住
+- 結果是資料庫錯誤訊息（可能含 SQL 與資料表結構）被當成 422 回給使用者，屬於資訊洩漏
+- 這類錯誤被當成「可預期的業務失敗」處理，不會經過 Laravel 例外處理器，**不會被 Sentry 收到**，營運端看不到
+- 狀態碼為 422（客戶端錯誤），實際是伺服器端故障，監控與前端錯誤處理都會誤判
+
+**待辦**
+- [ ] 定義單一窄型的業務例外（例如 `BookingRuleViolationException`），交易閉包內改拋它；Controller 只 catch 該型別，其他例外讓它往外傳播
+- [ ] 回應內容只使用自己拋出的、已知安全的訊息；不再回傳任意 `$e->getMessage()`
+- [ ] 確認資料庫類例外會被 Sentry 收到、回應為 500 且訊息通用
+- [ ] 檢查其他 Controller 是否有相同寫法（目前 grep 僅發現這兩處 `RuntimeException`）
+- [ ] 補測試：讓交易內拋出 `QueryException`（例如以 `DB::listen` 或 mock 模擬），斷言回應為 500 且 body 不含 SQL 與資料表名稱；業務失敗（名額不足、重複預約）仍回 422 與原訊息
+- [ ] 更新 spec：`booking-lifecycle`（錯誤處理行為）；注意該 spec 有一條 Requirement 超過 500 字元，修改時可順便拆分
+
+**驗收條件（草案，需轉為 spec Scenario）**
+1. 名額不足、重複預約仍回 422 與既有訊息
+2. 交易內發生資料庫例外時回 500，回應不含 SQL、資料表或欄位資訊
+3. 上述資料庫例外會被記錄並送到 Sentry
+
+**安全確認（CLAUDE.md 必答）**
+- 輸入驗證位置：既有 `$request->validate()`，不變
+- 資料庫存取：Eloquent，不變
+- 權限：不變；本項只改例外處理與回應內容
+
+---
+
+## M8　預約取消／確認的狀態在鎖之前讀取，並發下名額可能重複增減
+
+> 來源：撰寫 `hardener` skill 時閱讀程式碼發現。以下為程式碼閱讀判斷，**尚未重現驗證**；而且 `phpunit.xml` 使用 SQLite，無法在現有測試中重現行鎖行為（見 M6）。
+
+**現況**（皆為「交易外讀狀態、交易內才鎖 schedule」的模式）
+- `MemberBookingController::destroy`：在交易外以 `$booking->status` 判斷可否取消，並記下 `$wasConfirmed`；進入交易後才 `lockForUpdate` 鎖 schedule，`booking` 本身沒有鎖，且仍使用鎖之前讀到的舊狀態
+- `ProviderBookingController::cancel`、`confirm`、`reject`、`complete`：同樣先用 `canTransitionTo()` 判斷（交易外、無鎖），交易內才鎖 schedule
+- `AdminBookingController::complete`、排程 `app:complete-finished-bookings`、`app:expire-pending-bookings`：以交易外或批次方式直接改狀態，沒有與其他寫入者序列化
+
+**可能的後果（需並發才觸發）**
+- 同一筆 `confirmed` 預約被會員與教練（或同一方重複點擊）**同時取消**：兩個請求都通過檢查，各自扣減一次 `current_participants`，名額被**重複釋放**，後續可能超賣；`current_participants` 為 `unsignedInteger`，若剛好扣到負數，在 MySQL 嚴格模式下會報錯變成 500
+- 同一筆 `pending` 預約被**同時確認兩次**：各自增加一次 `current_participants`，名額被重複占用
+- `confirm` 與 `reject`／`expire` 同時發生：預約可能最後是 `confirmed` 卻已被標成 `expired`，或名額已增加但狀態被覆蓋
+- 既有的 `BookingOversellTest` 只涵蓋單程序情境，無法抓到這類問題
+
+**待辦**
+- [ ] 先驗證：以兩個程序或 MySQL 上的並發請求重現「同一預約重複取消」與「同一預約重複確認」，確認名額是否真的重複增減（驗證環境需為 MySQL，見 M6）
+- [ ] 狀態轉換統一在交易內：先鎖住預約本身（`Booking::lockForUpdate()->find(...)`）或先鎖 schedule 再重讀預約，**鎖之後**再以最新狀態判斷 `canTransitionTo()` 與 `$wasConfirmed`
+- [ ] 以條件式更新輔助：例如 `where('status', 'confirmed')->update(...)` 並檢查受影響列數，為 0 代表別人已先處理
+- [ ] 取得鎖的順序需一致（例如一律先 schedule 後 booking），避免新增鎖後產生死結
+- [ ] 統一抽到單一入口（併入 M2 的 `BookingService`），所有取消與確認路徑共用
+- [ ] 評估排程指令（`complete-finished-bookings`、`expire-pending-bookings`）是否也需要用同樣的條件式更新，避免與使用者操作互相覆蓋
+- [ ] 補測試：雖然 SQLite 無法驗證行鎖，仍應補「同一預約連續取消兩次，第二次回 422 且名額只釋放一次」等可確定性的測試；真正的並發驗證放在 MySQL CI（M6）或人工驗證
+- [ ] 更新 spec：`booking-lifecycle`（狀態轉換與名額一致性）
+
+**驗收條件（草案，需轉為 spec Scenario）**
+1. 同一筆預約重複取消，名額只釋放一次，第二次請求回 422
+2. 同一筆預約重複確認，名額只占用一次，第二次請求回 422
+3. 狀態轉換的判斷使用鎖之後的最新狀態
+4. 現有 `BookingLifecycleTest`、`BookingOversellTest` 全綠
+
+**安全確認（CLAUDE.md 必答）**
+- 資料庫存取：Eloquent，不使用 raw SQL
+- 是否需要悲觀鎖：是；需序列化的資源是「單一預約」與其所屬的 schedule
+- 權限檢查：不變，各端點既有的擁有權檢查保留
+
+**備註**
+- 此問題的完整驗證依賴 M6（MySQL CI）。在 M6 完成之前，只能以程式碼審查與可確定性測試降低風險，不能宣稱已驗證併發安全
+
+---
+
+## 功能與延後項目（非本次審視來源）
+
+> 來源：既有規劃文件與已歸檔 change 的未完成任務，2026-10-17 彙整並對照程式碼確認仍未完成。優先順序未排入上方建議順序。
+
+| # | 項目 | 來源 | 狀態／前置條件 |
+|---|------|------|----------------|
+| F1 | 金流整合（`awaiting_payment` 或獨立 `payments` 表、Webhook 冪等） | `2026-06-11-future-roadmap-feasibility.md` §2.2 | 未開工；需先定案金流商（評估建議綠界）與退款政策；建議 M6、H1 先完成 |
+| F2 | 上傳檔案正式切換 S3 相容儲存（R2） | 已歸檔 `2026-08-02-cloud-ready-s3-storage` tasks 4–6 | 程式碼已完成，VPS 仍為 `FILESYSTEM_DISK=local`；需先申請帳號，接續時另開新 change |
+| F3 | 預約列表端點補分頁（member／provider／admin bookings 皆 `->get()` 全量） | `2026-06-11-performance-optimization-plan.md` O3.2 | 未開工；建議金流上線前完成 |
+| F4 | 課程搜尋強化（地區、價格區間、可預約日期篩選） | 可行性評估 §2.5 | 未開工；cache key 需納入新篩選參數 |
+| F5 | 前端載入體感（列表 skeleton、搜尋 debounce 300ms） | 效能計畫 O3.3 | 未開工（`frontend/src` 無 skeleton／debounce） |
+| F6 | 聊天圖片壓縮改用 `CompressesImages` trait | O3.1 歸檔時留待後續 | `BookingMessageController` 仍自行 `scaleDown`／`toJpeg`；屬重複邏輯，改動前需先補聊天圖片上傳測試 |
+| F7 | 正式環境 OPcache `validate_timestamps=0` | 效能計畫「生產環境註記」 | 目前僅有 `docker/php/local.ini`（=1）；程式碼已烘進 image，可評估於正式環境關閉 |
+
+---
+
 ## 其他提醒
 
 - Git remote `vps` 的 URL 內含明文 Gitea token（存在 `.git/config`，未進版控）。建議輪換 token，改用 credential helper 或 SSH。

docs/analysis/2026-10-29-k8s-local-simulation-todo.md

@@ -0,0 +1,132 @@
+# CFDivePlatform K8s 本機模擬待辦
+
+> 記錄日期：2026-10-29
+> 狀態：**僅記錄，尚未規劃、尚未動工**（Hank 指示「先記待辦」）。
+> 範圍：在本機以 Docker Desktop Kubernetes 模擬部署，驗證 manifest 與流程；`docker-compose.yml` 維持本地開發用，不更動。
+> 實作原則：涉及行為或部署流程變更，先走 OpenSpec（`/opsx:propose`）；本文的決策與清單是規劃輸入，不是 spec。
+> 搭配閱讀：`docs/analysis/2026-10-14-project-review-todo.md`（M6、M8 與本項有相依）。
+
+---
+
+## 已做的決策（2026-10-29 討論）
+
+| 項目 | 決策 | 理由 |
+|---|---|---|
+| 模擬環境 | Docker Desktop 內建 Kubernetes，**Kubeadm（單節點）模式** | 目的是驗證 manifest 與部署流程，不需要多節點；設定最簡單 |
+| 與 Compose 的關係 | Compose 照舊作為本地開發環境，K8s 另放獨立目錄（暫定 `k8s/`） | 兩邊互不影響，避免動到現有 `deploy.yml` |
+| image | 沿用現有 `Dockerfile` 的 `app`、`web` 兩個 target | 與正式環境的 compose（不含 `compose.override.yml`）同一種形態，落差最小 |
+| 隔離 | 每個專案一個 namespace | 另一個 .NET 專案會一起模擬 |
+| 上機器的時機 | 本機模擬驗證完成後，再決定實際叢集 | VPS 資源不足，見下節 |
+
+## 為什麼不直接在現有 VPS 上裝 K3s
+
+2026-10-29 實測：2 核、1.9GB RAM（可用約 770MB）、磁碟剩 9.2GB（72%），且已跑 13 個容器（含 Gitea、Gitea runner、Nginx Proxy Manager）。
+K3s 單節點自身約需 0.5–1GB，會讓 OOM 風險落在現有正式站與 CI。若之後要上實體叢集，建議另租至少 2 核 4GB、40GB 磁碟的機器。
+
+---
+
+## 總覽
+
+| # | 項目 | 依賴 | 是否走 OpenSpec |
+|---|------|------|----------------|
+| K1 | 決定 Ingress controller | — | 否（決策） |
+| K2 | 盤點 CFDive 的 K8s 資源清單 | — | 否（盤點） |
+| K3 | 撰寫 CFDive manifest（`k8s/`） | K1、K2 | 是 |
+| K4 | 本機部署與驗證 | K3 | 是（含驗證項目） |
+| K5 | 另一個 .NET 專案納入同一叢集 | K1、K4 | 另案（該專案的 repo） |
+| K6 | 上傳檔案切 S3／R2 | 外部帳號 | 是（既有暫緩項目） |
+| K7 | 上實體叢集前的差距清單 | K4 | 否（文件） |
+
+---
+
+## K1　決定 Ingress controller
+
+**現況**
+Docker Desktop 沒有內建 Ingress。K3s 預設是 Traefik；原本考慮的 ingress-nginx，**據我所知 Kubernetes 社群已宣布它在 2026 年 3 月退役，此點為記憶，尚未查證**。
+
+**待辦**
+- [ ] 查證 ingress-nginx 目前的維護狀態（官方公告）
+- [ ] 在 Traefik 與 Gateway API 實作之間決定（暫定傾向 Traefik：與 K3s 預設一致，之後搬 K3s 的 manifest 改動最少）
+- [ ] 確認 Reverb（WebSocket）在所選 Ingress 上的路由與逾時設定
+
+## K2　盤點 CFDive 的 K8s 資源清單
+
+**現況（依 `docker-compose.yml` 服務名稱，2026-10-29 核對）**
+`app`、`nginx`、`frontend`、`db`、`scheduler`、`queue-worker`、`redis`、`reverb`。volumes 為 `mysql-data`、`redis-data`、`storage-app-public`。
+
+**暫定對應（未驗證）**
+- `app`、`nginx`、`frontend`、`queue-worker`、`reverb`：各一個 Deployment＋必要的 Service
+- `scheduler`：Deployment，**副本固定 1**（多副本會讓排程重複執行）
+- migration／cache／swagger：目前在 `deploy.yml`，需改為 Job（或 initContainer）
+- `.env`：拆成 ConfigMap（非機密）與 Secret（機密）；`env_file:` 在 K8s 沒有對等物
+- `db`、`redis`：本機模擬時先決定放叢集內，或沿用 Compose 容器經 `host.docker.internal` 連線
+
+**待辦**
+- [ ] 逐一核對各服務的 port、healthcheck、啟動指令、依賴順序
+- [ ] 決定 MySQL／Redis 放叢集內還是沿用 Compose（叢集內較貼近正式環境，Compose 較省事）
+- [ ] 注意 `tests/bootstrap.php` 的覆蓋清單：擴大環境變數注入範圍時需同步檢查（見專案記憶中 `env_file` 事件）
+
+## K3　撰寫 CFDive manifest
+
+**待辦**
+- [ ] 目錄結構（`k8s/` 下用 Kustomize 或純 YAML，擇一）
+- [ ] 各 Deployment 的探針（readiness／liveness）對應現有 healthcheck
+- [ ] 加上 `requests`／`limits`，總量抓接近目標機器（例如 4GB 節點），避免本機資源充裕而漏掉 OOM 問題
+- [ ] Secret 結構與正式環境相同，本機只用假值
+- [ ] 環境值用正式設定：`APP_ENV=production`、`APP_DEBUG=false`、MySQL `8.0`、Redis `alpine`
+- [ ] 不帶入 `compose.override.yml` 的 bind mount
+
+## K4　本機部署與驗證
+
+**待辦**
+- [ ] Docker Desktop 啟用 Kubernetes（Kubeadm 模式），確認 `kubectl` 指向正確 context
+- [ ] 本機 build `app`、`web`、`frontend` image，`imagePullPolicy` 設為 `IfNotPresent`（或 `Never`）
+- [ ] 驗證：登入、預約、評價主流程；Reverb 即時訊息（WebSocket 經 Ingress）；queue job 實際被消化；scheduler 只有一個實例
+- [ ] 驗證：migration Job 失敗時的行為（不應讓新版本 Pod 起來）
+- [ ] 驗證：滾動更新與回滾（`kubectl rollout`）
+- [ ] 記錄與 Compose 行為不一致之處
+
+## K5　另一個 .NET 專案納入同一叢集
+
+**注意事項（屬該專案 repo，這裡只記備忘）**
+- ASP.NET Core（.NET 8 以後）官方 image 預設非 root、監聽 8080，Service 的 `targetPort` 要對
+- 巢狀設定以環境變數覆蓋時用雙底線，例如 `ConnectionStrings__Default`
+- 需自行提供健康檢查端點供探針使用
+- EF migration 用 Job，不要在應用啟動時跑
+- 多副本時需處理 Data Protection 金鑰共用，否則副本間 cookie／antiforgery 互相無法解密
+
+**待辦**
+- [ ] 兩個專案用不同 namespace 與 hostname（例如 `cfdive.localhost`、`other.localhost`），共用同一個 Ingress controller
+
+## K6　上傳檔案切 S3／R2
+
+**現況**
+`storage:migrate-to-s3` 指令與 `FILESYSTEM_DISK` 設定已完成（PR #55、#56），但 R2／S3 帳號尚未申請，VPS 仍是 `FILESYSTEM_DISK=local`，上傳檔案存在具名 volume `storage-app-public`。
+多副本時 ReadWriteMany 儲存是最大阻礙；**本機 hostpath 單機無法代表這個問題**，所以這項要在上實體叢集前完成。
+
+**待辦**
+- [ ] 取得 S3 相容服務帳號後，另開 OpenSpec change 接續（不要回頭改已歸檔的 `cloud-ready-s3-storage`）
+
+## K7　上實體叢集前的差距清單
+
+本機模擬**無法**驗證的項目，上實體叢集後需另外確認：
+
+| 項目 | 原因 |
+|---|---|
+| 多節點調度、節點故障 | Kubeadm 模式為單節點 |
+| 檔案儲存的 ReadWriteMany | 本機 hostpath 僅單機 |
+| TLS／憑證、真實網域 | 本機用 `*.localhost`，無 Let's Encrypt |
+| 真實流量與資源競爭 | 本機資源充裕 |
+| Gitea Actions 到叢集的部署流程 | 需等有目標叢集；另需 image registry（目前 image 在 VPS 上 build，是否有 registry 未確認） |
+
+**待辦**
+- [ ] 確認 VPS 現行部署是否有 image registry；沒有的話，上叢集前要決定放哪裡
+- [ ] 決定實體叢集（另租 K3s 機器，或雲端託管）
+
+---
+
+## 開放問題
+
+- 本機機器規格（Docker Desktop 分配給 WSL2 的記憶體）未確認；兩個專案加 MySQL／Redis 同時跑，建議 8GB 以上
+- Kubeadm 模式下本機 image 是否可直接被 Pod 使用，依記憶判斷為可，**尚未在實際版本驗證**
+- Ingress 最終選哪個（K1）

openspec/changes/archive/2026-10-18-add-role-middleware/.openspec.yaml

@@ -0,0 +1,2 @@
+schema: spec-driven
+created: 2026-10-17

openspec/changes/archive/2026-10-18-add-role-middleware/design.md

@@ -0,0 +1,70 @@
+# Design
+
+## Context
+
+動機見 proposal.md（Why）。目前現況：
+
+- `bootstrap/app.php` 只註冊一個角色 middleware alias：`admin` → `App\Http\Middleware\EnsureAdmin`。它在 `!$request->user() || !$request->user()->isAdmin()` 時回 `403 { status: false, message: "無權限存取" }`
+- `App\Models\User` 已有 `isMember()`、`isProvider()`、`isAdmin()`，都是直接比對 `role` 字串
+- `/api/member/*`、`/api/provider/*` 群組內，只有 `AuthController` 的 logout／refresh／profile／change-password 自己檢查角色，回應同樣是 `403 { status: false, message: "無權限存取" }`；`MemberBookingController`、`ReviewController`（store／update／destroy）、`ProviderOfferController`、`ScheduleController`、`ProviderBookingController`、`CourseImageController`、`ProviderVerificationController` 只檢查資源所有權，不檢查角色
+- 前端 `axios.js`、`coachAxios.js` 的攔截器只對 401 做 refresh 重試，403 直接交給呼叫端；`adminAxios.js` 沒有呼叫 `/member/*`、`/provider/*`
+- 部署流程（`deploy.yml`）每次都會執行 `route:cache`，路由變更隨部署生效
+
+## Goals / Non-Goals
+
+**Goals:**
+- 在路由層擋下所有跨角色呼叫 member／provider 專屬端點
+- 角色不符時的回應與現有 `EnsureAdmin`、`AuthController` 完全一致，前端不需要改
+
+**Non-Goals:**
+- 不檢查 `is_active`（停用帳號沿用既有行為，與角色 middleware 無關）、教練審核狀態（審核狀態的限制維持在既有業務邏輯，例如 `visibleToPublic` scope）
+- 不移除 `AuthController` 內既有的角色檢查（重構屬待辦 M1）
+- 不調整跨角色共用路由（通知、即時訊息、`/reviews/{id}/helpful`、`/logout`、`/user`），包含「admin 能否存取這些路由」也不在此次決定
+- 不更動 `EnsureAdmin`
+
+## Decisions
+
+### D1：新增 `EnsureMember`、`EnsureProvider` 兩個 middleware，而非參數化的 `EnsureRole`
+- 做法：在 `app/Http/Middleware/` 新增兩個 class，結構與 `EnsureAdmin` 相同，分別呼叫 `isMember()`、`isProvider()`；在 `bootstrap/app.php` 註冊 alias `member`、`provider`
+- 理由：既有慣例是每個角色一個 middleware（`EnsureAdmin` + alias `admin`）。若新增參數化的 `role:member`，專案內會同時存在兩種寫法；要統一就得改 `EnsureAdmin` 的使用處，超出此次範圍
+- 替代方案：`EnsureRole` 搭配 `role:member,provider`。程式碼較少，但會跟既有的 `admin` alias 形成兩種寫法並存，因此不採用
+
+### D2：403 回應沿用 `{ status: false, message: "無權限存取" }`
+- 理由：與 `EnsureAdmin`、`AuthController::getUnauthorizedResponse()` 一致，前端與 Swagger 已記載的 403 不需變動
+- middleware 內同時處理 `$request->user()` 為 null 的情況（與 `EnsureAdmin` 相同的防禦寫法）；實際上 `auth:sanctum` 排在前面，未登入會先回 401
+
+### D3：middleware 只比對角色，不檢查審核狀態
+- 理由：未審核（`unsubmitted`）、審核中（`pending`）、被駁回（`rejected`）的教練都需要進入 `/api/provider/verification*` 送審或補件，也需要在審核通過前先建立課程。若在 middleware 檢查審核狀態，這些流程會整個斷掉
+
+### D4：掛在既有群組的 middleware 陣列，順序為 `['auth:sanctum', 'member']`／`['auth:sanctum', 'provider']`
+- 理由：與 admin 群組 `['auth:sanctum', 'admin']` 寫法一致；`auth:sanctum` 先執行，保證未登入回 401、角色不符回 403
+- 公開的 register／login 路由定義在群組外，不受影響（已核對 `routes/api.php`）
+
+### D5：Controller 內既有的資源所有權檢查全部保留
+- 理由：CLAUDE.md 授權規範要求 middleware 補上後仍保留 Controller 內的所有權檢查；角色 middleware 只解決「角色錯」，不解決「同角色存取別人的資源」
+
+## 安全確認
+
+- **輸入驗證**：此次不新增外部輸入；既有端點的 `$request->validate()` 不變。middleware 只讀取 Sanctum 認證後的 `$request->user()->role`，沒有讀取任何請求參數
+- **SQL／shell 拼接**：無
+- **資料庫存取**：middleware 不額外查詢資料庫（`$request->user()` 由 Sanctum 認證時已載入）；無 raw query，無 N+1
+- **權限檢查層級**：路由層（`member`、`provider` middleware），加上 Controller 內既有的資源所有權檢查（例如 `MemberBookingController::show` 比對 `member_id`、`ProviderOfferController` 比對 `provider_id`）
+- **未授權觸發可能性**：修正後，非 member 角色無法存取 `/api/member/*`，非 provider 角色無法存取 `/api/provider/*`；未登入一律 401
+- **各路由所掛 middleware**：
+  - `/api/member/*`（register、login 除外）：`auth:sanctum` + `member`
+  - `/api/provider/*`（register、login 除外）：`auth:sanctum` + `provider`
+  - 其他路由不變
+- **前端**：不涉及 DOM 渲染；前端呼叫已帶各自的 Bearer token，不需改
+
+## Risks / Trade-offs
+
+- [既有測試可能以錯誤角色呼叫 member／provider 路由，加上 middleware 後改回 403] → 實作時先跑完整 `php artisan test`，逐一判斷失敗的測試是「測試本身用錯角色」（修正測試）還是「真的依賴跨角色行為」（停下來回報）
+- [AuthController 內的角色檢查與 middleware 重複] → 行為一致、無害；移除留給待辦 M1，避免此次範圍擴大
+- [admin 被擋在 member／provider 路由之外，若未來後台想借用這些端點會失敗] → 已核對前端 admin 端目前沒有呼叫；未來需要時應在 `/api/admin/*` 新增端點
+- [跨角色共用路由（通知、即時訊息）對 admin 的行為未定義] → 不在此次範圍；即時訊息的 `authorizeParticipant` 對 admin 已回 false，通知只會讀到 admin 自己的資料
+
+## Migration Plan
+
+- 無 migration、無新套件、無環境變數
+- 部署：merge 後由 `deploy.yml` 重建 image，並執行 `route:cache`，新的路由 middleware 即生效
+- 回滾：revert 此次 commit 再部署即可，沒有資料需要還原

openspec/changes/archive/2026-10-18-add-role-middleware/proposal.md

@@ -0,0 +1,32 @@
+# Proposal
+
+## Why
+
+`routes/api.php` 的 `member`、`provider` 路由群組只掛 `auth:sanctum`，沒有角色檢查（待辦 H1，見 `docs/analysis/2026-10-14-project-review-todo.md`）。任何已登入的帳號都能跨角色呼叫：member 或 admin 可呼叫 `POST /api/provider/offers` 建立課程（`provider_id` 會被設成呼叫者），provider 或 admin 可呼叫 `POST /api/member/bookings` 建立預約。這兩個群組內只有 `AuthController` 的帳號類端點自己檢查角色，其餘業務端點都沒有防護。
+
+## What Changes
+
+- 新增 member 角色 middleware 與 provider 角色 middleware，比照既有 `admin` middleware 的行為：角色不符時回 `403 { status: false, message: "無權限存取" }`
+- 將它們分別掛到 `/api/member/*` 與 `/api/provider/*` 路由群組；未登入仍由 `auth:sanctum` 先回 401
+- provider middleware **只檢查角色，不檢查審核狀態**：未審核或審核中的教練仍需能進入 `/api/provider/*`（例如送審證照）；審核狀態的限制維持在既有的業務邏輯
+- admin 也會被擋在 `/api/member/*` 與 `/api/provider/*` 之外（前端 admin 端目前沒有呼叫這兩組路由，已核對 `frontend/src`）
+- **BREAKING（僅對越權呼叫）**：原本能成功的跨角色呼叫（例如 member 呼叫 `POST /api/provider/offers`）改為 403。合法角色的行為與回應格式不變
+- 不在範圍內：通知（`/api/notifications/*`）、即時訊息（`/api/bookings/{booking}/messages*`）、`POST /api/reviews/{id}/helpful`、`/api/logout`、`/api/user` 等跨角色共用路由維持現狀；`AuthController` 內既有的重複角色檢查不在此次移除（屬待辦 M1）
+
+## Capabilities
+
+### New Capabilities
+- `role-authorization`：依使用者角色限制 member、provider 專屬 API 路由群組的存取（各角色與未登入的預期結果）
+
+### Modified Capabilities
+- `feature-test-coverage`：「預約列表端點測試覆蓋」Requirement 內文描述 member／provider 預約列表的 middleware 只有 `auth:sanctum`，需改為反映新增的角色 middleware
+
+## Impact
+
+- 程式碼：`app/Http/Middleware/`（新增兩個 middleware）、`bootstrap/app.php`（註冊 alias）、`routes/api.php`（兩個群組加 middleware）
+- 測試：新增跨角色 Feature 測試（參考 `tests/Feature/AdminEndpointAuthTest.php`）；既有測試若依賴跨角色呼叫需檢視
+- API：回應格式不變；跨角色呼叫從原本的 2xx／422 改為 403
+- 前端：不需修改（三組前端各自只呼叫自己角色的路由）
+- Swagger：`app/Docs/MemberApiDoc.php`、`ProviderApiDoc.php` 已有 403 回應，只需核對描述是否涵蓋「角色不符」
+- 不影響主規格：`provider-auth`、`admin-auth`（其 Requirement 未描述路由群組的角色限制，行為也不變）
+- 部署：無 migration、無新相依套件、無環境變數

openspec/changes/archive/2026-10-18-add-role-middleware/specs/feature-test-coverage/spec.md

@@ -0,0 +1,25 @@
+# Spec Delta
+
+## MODIFIED Requirements
+
+### Requirement: 預約列表端點測試覆蓋
+測試套件 SHALL 驗證下列三個路由的資料隔離行為：
+- `GET /api/member/bookings`（middleware: `auth:sanctum` + `member`，非 member 角色於 middleware 層強制 403；隔離靠 `where member_id = auth()->id()`）
+- `GET /api/provider/bookings`（middleware: `auth:sanctum` + `provider`，非 provider 角色於 middleware 層強制 403；隔離靠 `whereHas schedule.provider_id`）
+- `GET /api/admin/bookings`（middleware: `auth:sanctum` + `admin`，EnsureAdmin 於 middleware 層強制 403）
+
+#### Scenario: Member 只能看到自己的預約
+- **WHEN** Member A 呼叫 `GET /api/member/bookings`，系統中同時存在 Member A 與 Member B 各一筆預約
+- **THEN** 回傳 data 只包含 Member A 的 booking id；Member B 的 booking id 不在 data 中
+
+#### Scenario: Provider 只能看到屬於自己課程的預約
+- **WHEN** Provider A 呼叫 `GET /api/provider/bookings`，系統中有屬於 Provider A 與 Provider B 課程各一筆預約
+- **THEN** 回傳 data 只包含 Provider A 課程的 booking id；Provider B 課程的 booking id 不在 data 中
+
+#### Scenario: Admin 可查看所有預約
+- **WHEN** Admin 呼叫 `GET /api/admin/bookings`，系統中有 2 筆不同 Member 的預約
+- **THEN** 回傳 data 包含兩筆 booking id
+
+#### Scenario: 未認證請求被拒絕
+- **WHEN** 未帶 token 呼叫 `GET /api/member/bookings`
+- **THEN** 回傳 HTTP 401

openspec/changes/archive/2026-10-18-add-role-middleware/specs/role-authorization/spec.md

@@ -0,0 +1,68 @@
+# Spec Delta
+
+## Purpose
+
+依使用者角色限制 member、provider 專屬 API 路由群組的存取，避免已登入的其他角色跨角色呼叫課程、時段、預約等業務端點。
+
+## ADDED Requirements
+
+### Requirement: Member 專屬路由僅限 member 角色存取
+`/api/member/*` 路由群組（不含公開的 `POST /api/member/register`、`POST /api/member/login`）SHALL 只允許 `role = member` 的已登入使用者存取。其他角色 MUST 收到 HTTP 403，回應為 `{ "status": false, "message": "無權限存取" }`，且請求 MUST NOT 產生任何資料變更。未帶 token 的請求 MUST 收到 HTTP 401。
+
+#### Scenario: member 可存取 member 路由
+- **WHEN** 一個 `role = member` 的使用者，以該使用者的 token 呼叫 `GET /api/member/profile` 與 `GET /api/member/bookings`
+- **THEN** 回傳 HTTP 200，`status` 為 `true`
+
+#### Scenario: provider 呼叫 member 路由被拒絕
+- **WHEN** 一個 `role = provider` 的使用者，以該使用者的 token 呼叫 `GET /api/member/profile`、`GET /api/member/bookings`、`POST /api/member/bookings`、`POST /api/member/reviews`
+- **THEN** 每個請求都回傳 HTTP 403，body 為 `{ "status": false, "message": "無權限存取" }`
+
+#### Scenario: admin 呼叫 member 路由被拒絕
+- **WHEN** 一個 `role = admin` 的使用者，以該使用者的 token 呼叫 `GET /api/member/profile`、`GET /api/member/bookings`、`POST /api/member/bookings`
+- **THEN** 每個請求都回傳 HTTP 403，body 為 `{ "status": false, "message": "無權限存取" }`
+
+#### Scenario: 非 member 建立預約不會產生資料
+- **WHEN** 一個可預約的課程時段，以及一個 `role = provider` 的使用者，以該使用者的 token 呼叫 `POST /api/member/bookings` 並帶入合法的時段 id
+- **THEN** 回傳 HTTP 403，`bookings` 資料表筆數不變，該時段的 `current_participants` 不變
+
+#### Scenario: 未登入呼叫 member 路由
+- **WHEN** 不帶 token 呼叫 `GET /api/member/profile`、`GET /api/member/bookings`
+- **THEN** 回傳 HTTP 401
+
+### Requirement: Provider 專屬路由僅限 provider 角色存取
+`/api/provider/*` 路由群組（不含公開的 `POST /api/provider/register`、`POST /api/provider/login`）SHALL 只允許 `role = provider` 的已登入使用者存取，且 MUST NOT 以教練審核狀態作為進入條件。其他角色 MUST 收到 HTTP 403，回應為 `{ "status": false, "message": "無權限存取" }`，且請求 MUST NOT 產生任何資料變更。未帶 token 的請求 MUST 收到 HTTP 401。
+
+#### Scenario: provider 可存取 provider 路由
+- **WHEN** 一個 `role = provider` 的使用者，以該使用者的 token 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers`、`GET /api/provider/schedules`、`GET /api/provider/bookings`
+- **THEN** 回傳 HTTP 200，`status` 為 `true`
+
+#### Scenario: 未通過審核的 provider 仍可進入 provider 路由
+- **WHEN** 一個 `role = provider`、`verification_status` 不是 approved（例如 unsubmitted 或 pending）的使用者，以該使用者的 token 呼叫 `GET /api/provider/verification`
+- **THEN** 回傳 HTTP 200，不會因角色 middleware 回 403
+
+#### Scenario: member 呼叫 provider 路由被拒絕
+- **WHEN** 一個 `role = member` 的使用者，以該使用者的 token 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers`、`POST /api/provider/offers`、`POST /api/provider/schedules`、`GET /api/provider/bookings`、`GET /api/provider/verification`
+- **THEN** 每個請求都回傳 HTTP 403，body 為 `{ "status": false, "message": "無權限存取" }`
+
+#### Scenario: admin 呼叫 provider 路由被拒絕
+- **WHEN** 一個 `role = admin` 的使用者，以該使用者的 token 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers`、`POST /api/provider/offers`、`GET /api/provider/bookings`
+- **THEN** 每個請求都回傳 HTTP 403，body 為 `{ "status": false, "message": "無權限存取" }`
+
+#### Scenario: 非 provider 建立課程不會產生資料
+- **WHEN** 一個 `role = member` 的使用者，以該使用者的 token 呼叫 `POST /api/provider/offers` 並帶入通過驗證的完整課程資料
+- **THEN** 回傳 HTTP 403，`diving_offers` 資料表筆數不變
+
+#### Scenario: 未登入呼叫 provider 路由
+- **WHEN** 不帶 token 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers`
+- **THEN** 回傳 HTTP 401
+
+### Requirement: 跨角色共用路由不受角色限制影響
+通知（`/api/notifications/*`）與預約即時訊息（`/api/bookings/{booking}/messages*`、`/api/bookings/messages/unread-counts`）路由 SHALL 維持既有行為：member 與 provider 皆可存取，存取權限由既有的資源歸屬檢查決定，不套用 member 或 provider 角色限制。
+
+#### Scenario: member 與 provider 皆可讀取通知
+- **WHEN** 一個 `role = member` 與一個 `role = provider` 的使用者，分別以各自的 token 呼叫 `GET /api/notifications`
+- **THEN** 兩者皆回傳 HTTP 200
+
+#### Scenario: member 與 provider 皆可讀取未讀訊息數
+- **WHEN** 一個 `role = member` 與一個 `role = provider` 的使用者，分別以各自的 token 呼叫 `GET /api/bookings/messages/unread-counts`
+- **THEN** 兩者皆回傳 HTTP 200

openspec/changes/archive/2026-10-18-add-role-middleware/tasks.md

@@ -0,0 +1,47 @@
+# Tasks
+
+測試集中在新檔 `tests/Feature/RoleAuthorizationTest.php`（結構參考 `tests/Feature/AdminEndpointAuthTest.php`）。每個測試的 docblock 要寫明「為什麼」：跨角色呼叫曾可建立非教練課程、非會員預約。
+
+## 1. Member 角色 middleware
+
+- [x] 1.1 [後端] 新增 `app/Http/Middleware/EnsureMember.php`（結構同 `EnsureAdmin`，改用 `isMember()`），並在 `bootstrap/app.php` 的 `$middleware->alias([...])` 加上 `'member' => \App\Http\Middleware\EnsureMember::class`；驗證：1.3 起的測試通過
+- [x] 1.2 [後端] `routes/api.php` 的 member 群組改為 `Route::middleware(['auth:sanctum', 'member'])->prefix('member')`；驗證：`php artisan route:list --path=api/member` 顯示 `member` middleware，且 `register`、`login` 不受影響
+- [x] 1.3 [整合測試] `test_member_can_access_member_routes`：member 呼叫 `GET /api/member/profile`、`GET /api/member/bookings` 回 200（對應 Scenario「member 可存取 member 路由」）
+- [x] 1.4 [整合測試] `test_provider_is_rejected_on_member_routes`：provider 呼叫 `GET /api/member/profile`、`GET /api/member/bookings`、`POST /api/member/bookings`、`POST /api/member/reviews`，皆回 403，並斷言 body 為 `{ status: false, message: "無權限存取" }`；`POST /api/member/reviews` 另斷言 `reviews` 筆數不變（對應 Scenario「provider 呼叫 member 路由被拒絕」）
+- [x] 1.5 [整合測試] `test_admin_is_rejected_on_member_routes`：admin 呼叫 `GET /api/member/profile`、`GET /api/member/bookings`、`POST /api/member/bookings`，皆回 403 與相同 body（對應 Scenario「admin 呼叫 member 路由被拒絕」）
+- [x] 1.6 [整合測試] `test_non_member_booking_attempt_creates_no_booking`：建立可預約時段，provider 帶合法 `schedule_id` 呼叫 `POST /api/member/bookings`，回 403，`bookings` 筆數與 `current_participants` 不變（對應 Scenario「非 member 建立預約不會產生資料」）
+- [x] 1.7 [整合測試] `test_unauthenticated_request_is_rejected_on_member_routes`：不帶 token 呼叫 `GET /api/member/profile`、`GET /api/member/bookings` 回 401（對應 Scenario「未登入呼叫 member 路由」）
+
+## 2. Provider 角色 middleware
+
+- [x] 2.1 [後端] 新增 `app/Http/Middleware/EnsureProvider.php`（結構同 `EnsureAdmin`，改用 `isProvider()`，**不檢查** `verification_status`），並在 `bootstrap/app.php` 加上 `'provider' => \App\Http\Middleware\EnsureProvider::class`；驗證：2.3 起的測試通過
+- [x] 2.2 [後端] `routes/api.php` 的 provider 群組改為 `Route::middleware(['auth:sanctum', 'provider'])->prefix('provider')`；驗證：`php artisan route:list --path=api/provider` 顯示 `provider` middleware，且 `register`、`login` 不受影響
+- [x] 2.3 [整合測試] `test_provider_can_access_provider_routes`：provider（含 `providerProfile`）呼叫 `GET /api/provider/profile`、`/offers`、`/schedules`、`/bookings` 回 200（對應 Scenario「provider 可存取 provider 路由」）
+- [x] 2.4 [整合測試] `test_unapproved_provider_can_access_verification`：`verification_status` 為 `unsubmitted`、`pending`、`rejected` 的 provider 呼叫 `GET /api/provider/verification` 回 200（對應 Scenario「未通過審核的 provider 仍可進入 provider 路由」）
+- [x] 2.5 [整合測試] `test_member_is_rejected_on_provider_routes`：member 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers`、`POST /api/provider/offers`、`POST /api/provider/schedules`、`GET /api/provider/bookings`、`GET /api/provider/verification`，皆回 403 與 `{ status: false, message: "無權限存取" }`；`POST /api/provider/schedules` 另斷言 `course_schedules` 筆數不變（對應 Scenario「member 呼叫 provider 路由被拒絕」）
+- [x] 2.6 [整合測試] `test_admin_is_rejected_on_provider_routes`：admin 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers`、`POST /api/provider/offers`、`GET /api/provider/bookings`，皆回 403 與相同 body（對應 Scenario「admin 呼叫 provider 路由被拒絕」）
+- [x] 2.7 [整合測試] `test_non_provider_offer_attempt_creates_no_offer`：member 帶通過驗證的完整課程資料呼叫 `POST /api/provider/offers`，回 403，`diving_offers` 筆數不變（對應 Scenario「非 provider 建立課程不會產生資料」）
+- [x] 2.8 [整合測試] `test_unauthenticated_request_is_rejected_on_provider_routes`：不帶 token 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers` 回 401（對應 Scenario「未登入呼叫 provider 路由」）
+
+## 3. 共用路由與既有規格的回歸
+
+- [x] 3.1 [整合測試] `test_member_and_provider_can_read_notifications`：member 與 provider 各自呼叫 `GET /api/notifications` 回 200（對應 Scenario「member 與 provider 皆可讀取通知」）
+- [x] 3.2 [整合測試] `test_member_and_provider_can_read_unread_message_counts`：member 與 provider 各自呼叫 `GET /api/bookings/messages/unread-counts` 回 200（對應 Scenario「member 與 provider 皆可讀取未讀訊息數」）
+- [x] 3.3 [整合測試] 執行 `php artisan test --filter=BookingListTest`，確認 `feature-test-coverage`「預約列表端點測試覆蓋」的四個 Scenario（member 隔離、provider 隔離、admin 全部、未認證 401）在新 middleware 下仍全綠（「非 member／非 provider 於 middleware 層 403」由 1.4、1.5、2.5、2.6 覆蓋）；同時更新 `tests/Feature/BookingListTest.php` 第 19–24 行 docblock 中「無 role check」「不靠 middleware role 強制」的過時描述
+
+## 4. 全套回歸、Swagger 與格式
+
+- [x] 4.1 [整合測試] 執行完整 `php artisan test`（需在容器內，因 `CourseImageTest` 需要 GD）；若有既有測試因角色錯誤改回 403，判斷是測試用錯角色（修正測試並在 commit 說明）還是依賴跨角色行為（停下回報）
+- [x] 4.2 [後端] 核對 `app/Docs/MemberApiDoc.php`、`app/Docs/ProviderApiDoc.php` 每個端點的 403 回應描述涵蓋「角色不符」；缺的補上，並執行 `php artisan l5-swagger:generate` 確認無錯誤
+- [x] 4.3 [後端] 執行 `./vendor/bin/pint --test app/Http/Middleware/EnsureMember.php app/Http/Middleware/EnsureProvider.php bootstrap/app.php routes/api.php tests/Feature/RoleAuthorizationTest.php`（以及 4.1／4.2 若有修改的檔案），確認無格式問題
+
+## 5. 手動驗證
+
+- [x] 5.1 [整合測試]（手動，非自動化測試）本機 Docker 環境以 member token 呼叫 `POST /api/provider/offers`，以 provider token 呼叫 `POST /api/member/bookings`，確認兩者皆回 403
+- [x] 5.2 [整合測試]（手動，非自動化測試）本機前端：會員端登入後瀏覽「我的預約」、教練端登入後瀏覽課程／時段／預約／審核頁，確認功能正常、沒有出現 403
+
+## Workflow follow-up
+
+- 實作完成後依 CLAUDE.md 執行 `hardener` skill，再呼叫 `strict-reviewer`
+- 執行 `verify-and-report` skill 回報結果
+- 實作與歸檔各自獨立 PR；歸檔時同步 `openspec/config.yaml` 後端慣例段落（移除「member／provider 路由群組目前僅有 `auth:sanctum`」的描述）、CLAUDE.md「授權規範」的已知缺口描述，以及 `docs/analysis/2026-10-14-project-review-todo.md` H1 的完成狀態

openspec/changes/archive/2026-10-18-fix-booking-exception-handling/.openspec.yaml

@@ -0,0 +1,2 @@
+schema: spec-driven
+created: 2026-10-18

openspec/changes/archive/2026-10-18-fix-booking-exception-handling/design.md

@@ -0,0 +1,65 @@
+# Design
+
+## Context
+
+背景見 proposal.md「Why」。現況：
+
+- `MemberBookingController::store` 的 `DB::transaction` 閉包拋 `\RuntimeException('名額不足，請重新選擇')`、`\RuntimeException('您已預約此時段')`；`ProviderBookingController::confirm` 拋 `\RuntimeException('名額不足，無法確認此預約')`。兩處都以 `catch (\RuntimeException $e)` 回 422 + `$e->getMessage()`
+- `bootstrap/app.php` 只有 `Sentry\Laravel\Integration::handles($exceptions)`，沒有自訂 render；未被攔截的例外由 Laravel 預設處理器回應，`APP_DEBUG=false` 時 JSON body 為 `{"message":"Server Error"}`
+- 全專案 grep `RuntimeException` 只有這兩個 Controller；其他 `catch (\Throwable)` 都只包通知派送（記 log 後繼續），不在本次範圍
+- `phpunit.xml` 沒有設定 `APP_DEBUG`，測試會沿用 `.env` 的值（本機為 true），因此驗證「不洩漏細節」的測試必須自行設定 `app.debug = false`
+
+## Goals / Non-Goals
+
+**Goals**
+- 只有刻意拋出的業務例外會變成 422；其他例外一律走 Laravel 例外處理器
+
+**Non-Goals**
+- 不統一全站 500 的 JSON 格式（不在 `withExceptions` 加自訂 render）；500 body 沒有 `status` 欄位，與其他端點未攔截的 500 一致
+- 不處理 M8（鎖之前讀取狀態的併發問題）與 M2（抽出 `BookingService`）；本次只改例外型別與 catch 範圍
+- 不調整前端；資料庫錯誤時前端會顯示 `Server Error`
+- 不拆分 `booking-lifecycle` 中超過 500 字元的「預約狀態機」requirement（不在本次修改範圍，需要時另開 change）
+
+## Decisions
+
+### D1：新增窄型業務例外 `App\Exceptions\BookingRuleViolationException`
+- 放在新目錄 `app/Exceptions/`（目前不存在，Laravel 11 慣例位置）
+- 繼承 `\RuntimeException`。安全性來自 Controller 只 catch 這個具體類別，而不是父類別的選擇；繼承 `RuntimeException` 讓語意（執行期業務規則失敗）不變
+- 訊息只由程式內字串常值傳入，不包含使用者輸入或其他例外的訊息，因此可以安全回給使用者
+- **替代方案**：
+  - 在 catch 內判斷 `instanceof QueryException` 後重新拋出：仍會漏接其他非業務的 `RuntimeException`（例如 `PDOException` 以外的驅動錯誤），是黑名單做法，不採用
+  - 交易閉包回傳錯誤碼而非拋例外：拋例外才能觸發 `DB::transaction` 的 rollback，改回傳值需要手動處理 rollback，不採用
+
+### D2：非業務例外不在 Controller 攔截
+- 依使用者決定：交給 Laravel 例外處理器。好處是 Sentry 透過既有 `Integration::handles` 自動收到，不需在 Controller 呼叫 `report()`
+- **替代方案**：Controller 內 catch `\Throwable` + `report()` + 專案格式 500，或在 `withExceptions` 全域統一 JSON 500 格式。前者造成兩個端點格式與其他端點不一致；後者影響全站所有端點，範圍超出 M7。兩者皆不採用
+
+### D3：測試方式
+- **觸發資料庫例外**：在測試內註冊 Eloquent model 事件，於交易內拋出 `Illuminate\Database\QueryException`，其 SQL 字串含資料表與欄位名稱。這樣能真的經過 Controller 的 catch 路徑，不需 mock DB facade
+  - 例外必須發生在交易內**已有一筆寫入之後**，否則「資料沒有變動」在沒有 rollback 時也會成立，無法證明 rollback：建立預約用 `Booking::created`（INSERT 之後）；確認預約用 `CourseSchedule::updating`（預約狀態已更新、`increment` 時觸發；Laravel 11 的 `Model::increment` 會觸發 `updating` 事件）
+- **斷言 500 body**：以 `assertExactJson(['message' => 'Server Error'])` 鎖住完整 body，避免空 body 也讓「不含 SQL」的斷言通過
+- **觸發 store 交易內的名額不足**：此分支只在併發時發生（交易前已有相同檢查）。測試以 `CourseSchedule::retrieved` 事件，在交易內第二次讀取時段時把 `current_participants` 設為滿額，模擬另一筆預約搶先佔位
+- **斷言回報**：以 `Exceptions::fake()` 搭配 `Exceptions::assertReported(QueryException::class)`；fake 仍委派原處理器 render，因此可同時斷言 500 回應。Sentry 本身是否收到屬於整合設定，CI 無法驗證，以「例外有被回報」作為可測量的替代
+- **斷言不洩漏**：設定 `config(['app.debug' => false])` 後，斷言 body 不含 `insert into`／`update`、`bookings` 與測試用 SQL 中的欄位名稱
+- **業務失敗的 422**：以 `assertExactJson` 斷言完整 body，確保訊息文字沒有被改動；並在 `Exceptions::fake()` 下斷言 `Exceptions::assertNotReported(BookingRuleViolationException::class)`，業務失敗不應出現在 Sentry
+- **變異驗證**：暫時把 catch 改回 `\RuntimeException`，確認 500 測試會失敗。這是本機手動步驟、不提交，放在 tasks 的手動驗證 group，不算 Scenario 覆蓋
+
+## 安全確認
+
+- **輸入驗證**：沿用既有 `$request->validate()`，不變；沒有拼接 SQL 或 shell
+- **資料庫存取**：沿用 Eloquent／Query Builder 與 `lockForUpdate`，不變；不新增查詢，無 N+1 變化
+- **權限**：路由不變。`POST /api/member/bookings` 掛 `auth:sanctum` + `member` middleware；`PUT /api/provider/bookings/{id}/confirm` 掛 `auth:sanctum` + `provider` middleware，Controller 內 `authorizeProvider()` 檢查預約屬於呼叫者的課程。本次不改任何權限邏輯
+- **資訊洩漏**：本次的主要目的；回給使用者的訊息只剩 D1 中的程式內常值，以及 Laravel 的通用 500 訊息
+- **正式環境前提**：500 body 不洩漏細節依賴 `APP_DEBUG=false`；2026-10-18 已在 VPS 確認（H2）
+
+## Risks / Trade-offs
+
+- [資料庫錯誤的 HTTP 狀態由 422 改為 500，前端與監控會看到更多 5xx] → 這是正確分類；目前前端都以 `e.response?.data?.message` 顯示，不會壞掉，只是文字變成 `Server Error`
+- [之後有人在交易閉包內又拋一般 `RuntimeException` 表示業務失敗，會變成 500] → 測試以 `assertExactJson` 鎖住三個業務訊息的 422；新增業務規則時必須使用 `BookingRuleViolationException`
+- [死結、逾時這類可重試的暫時性錯誤，使用者只看到 `Server Error`，沒有「請稍後重試」的提示] → 本次接受；若之後要區分可重試錯誤，屬於全站錯誤格式的另一個 change
+- [`APP_DEBUG=true` 的環境（本機、若有預備環境）500 body 仍會含例外訊息與 SQL] → 這是 Laravel debug 模式的設計行為，只有正式環境（已確認 `APP_DEBUG=false`）保證不洩漏；任何對外開放的環境都必須設為 false
+- [SQLite 測試無法重現真實的死結或逾時] → 測試只需要證明「任何 `QueryException` 都會走 500」，例外類型本身與資料庫驅動無關，SQLite 足以驗證分類邏輯
+
+## Migration Plan
+
+純程式碼變更，無 migration、無設定變更。部署後若需回滾，revert 這個 PR 即可。

openspec/changes/archive/2026-10-18-fix-booking-exception-handling/proposal.md

@@ -0,0 +1,29 @@
+# Proposal
+
+## Why
+
+`MemberBookingController::store` 與 `ProviderBookingController::confirm` 用 `catch (\RuntimeException $e)` 把交易內的業務失敗轉成 422，並直接回傳 `$e->getMessage()`。`Illuminate\Database\QueryException` 經由 `PDOException` 也是 `RuntimeException` 的子類別，因此交易內的資料庫錯誤（死結、逾時、約束違反、連線中斷）會被同一個 catch 接住：SQL 與資料表結構被當成 422 訊息回給使用者（資訊洩漏），且不經過 Laravel 例外處理器、不會送到 Sentry，營運端看不到伺服器故障（待辦 M7，`docs/analysis/2026-10-14-project-review-todo.md`）。
+
+## What Changes
+
+- 新增專用的業務例外 `BookingRuleViolationException`；兩個預約交易閉包內的業務規則失敗（名額不足、重複預約）改拋此例外
+- 兩個 Controller 只 catch `BookingRuleViolationException` 並回 422 與其訊息；其他例外（含 `QueryException`）不再被攔截，交給 Laravel 例外處理器產生 500 並回報（既有 `Sentry\Laravel\Integration::handles` 會送到 Sentry）
+- 業務失敗的 422 狀態碼與訊息文字維持不變
+- 資料庫錯誤的回應由「422 + 原始例外訊息」改為「500 + Laravel 預設通用訊息」（`APP_DEBUG=false` 時 body 為 `{"message":"Server Error"}`）
+
+## Capabilities
+
+### New Capabilities
+（無）
+
+### Modified Capabilities
+- `booking-lifecycle`：新增「預約交易錯誤分類」requirement，明定業務規則失敗回 422 與既有訊息、非業務例外回 500 且不洩漏資料庫細節並被回報
+
+## Impact
+
+- **程式碼**：新增 `app/Exceptions/BookingRuleViolationException.php`；修改 `app/Http/Controllers/API/MemberBookingController.php`（`store`）、`app/Http/Controllers/API/ProviderBookingController.php`（`confirm`）
+- **API 回應格式**：業務失敗（422）不變。資料庫錯誤的回應由 422 改為 500，body 為 Laravel 預設格式 `{"message":"Server Error"}`，**不含**專案慣例的 `status` 欄位——這是刻意選擇（交給 Laravel 處理、不另寫轉換），與其他端點未被攔截的 500 一致
+- **前端**：`CourseDetailView.vue`、`coach/BookingManagerView.vue` 以 `e.response?.data?.message` 顯示錯誤；資料庫錯誤時會顯示 `Server Error`（原本顯示 SQL 訊息），不需修改前端
+- **監控**：預約交易內的資料庫錯誤開始出現在 Sentry
+- **測試**：新增 Feature 測試；既有 `BookingOversellTest`、`BookingLifecycleTest` 的 422 行為不變
+- **相依、migration、路由、Swagger**：無變更（`app/Docs/` 目前沒有記錄這兩個端點的 500，維持不新增）

openspec/changes/archive/2026-10-18-fix-booking-exception-handling/specs/booking-lifecycle/spec.md

@@ -0,0 +1,24 @@
+## ADDED Requirements
+
+### Requirement: 預約交易錯誤分類
+建立預約（`POST /api/member/bookings`）與確認預約（`PUT /api/provider/bookings/{id}/confirm`）的交易中，系統 SHALL 區分業務規則失敗與伺服器端故障：業務規則失敗（名額不足、重複預約）回傳 422 與既定訊息；其他任何例外（含資料庫錯誤）MUST NOT 以 422 回傳，MUST 回傳 500，回應內容不得包含 SQL、資料表或欄位名稱，且該例外 MUST 交由例外回報機制記錄（正式環境會送至 Sentry）。兩種情況下交易都 MUST rollback，不留下部分寫入。
+
+#### Scenario: 重複預約仍回 422 與既定訊息
+- **WHEN** 已登入 member 對同一時段已有 `pending` 預約，再次送出 `POST /api/member/bookings`（`schedule_id` 相同、`participants` 1）
+- **THEN** 系統回傳 422，body 為 `{"status": false, "message": "您已預約此時段"}`，該 member 在此時段的預約筆數不變
+
+#### Scenario: 交易內名額不足仍回 422 與既定訊息
+- **WHEN** 已登入 member 送出 `POST /api/member/bookings` 時通過交易前的名額檢查，但交易內鎖定時段後重新計算的剩餘名額小於 `participants`
+- **THEN** 系統回傳 422，body 為 `{"status": false, "message": "名額不足，請重新選擇"}`，不建立預約
+
+#### Scenario: 確認預約時名額不足仍回 422 與既定訊息
+- **WHEN** 已登入 provider 對自己課程的 `pending` 預約送出 `PUT /api/provider/bookings/{id}/confirm`，而該預約人數大於時段剩餘名額
+- **THEN** 系統回傳 422，body 為 `{"status": false, "message": "名額不足，無法確認此預約"}`，預約維持 `pending`，`current_participants` 不變
+
+#### Scenario: 建立預約時資料庫錯誤回 500 且不洩漏細節
+- **WHEN** `APP_DEBUG` 為 false，已登入 member 送出合法的 `POST /api/member/bookings`，而交易內寫入預約之後發生資料庫例外
+- **THEN** 系統回傳 500，body 為通用錯誤訊息且不含 SQL 語句、`bookings` 等資料表名稱與欄位名稱，資料庫例外被回報，`bookings` 筆數不變
+
+#### Scenario: 確認預約時資料庫錯誤回 500 且不洩漏細節
+- **WHEN** `APP_DEBUG` 為 false，已登入 provider 對自己課程的 `pending` 預約送出 `PUT /api/provider/bookings/{id}/confirm`（名額足夠），而交易內預約狀態已更新、更新時段名額時發生資料庫例外
+- **THEN** 系統回傳 500，body 為通用錯誤訊息且不含 SQL 語句、資料表名稱與欄位名稱，資料庫例外被回報，預約維持 `pending`，`current_participants` 不變

openspec/changes/archive/2026-10-18-fix-booking-exception-handling/tasks.md

@@ -0,0 +1,25 @@
+# Tasks
+
+測試集中在新檔 `tests/Feature/BookingExceptionHandlingTest.php`（建立 provider／時段的輔助方法參考 `tests/Feature/BookingOversellTest.php`）。每個測試的 docblock 要寫明「為什麼」：資料庫錯誤曾以 422 把 SQL 回給使用者，且不會送到 Sentry。觸發與斷言方式見 design.md D3。
+
+## 1. 業務例外與 Controller
+
+- [x] 1.1 [後端] 新增 `app/Exceptions/BookingRuleViolationException.php`（`final class`，繼承 `\RuntimeException`，不加額外方法）；驗證：2.x 測試通過
+- [x] 1.2 [後端] `app/Http/Controllers/API/MemberBookingController.php` 的 `store`：交易閉包內兩處 `throw new \RuntimeException(...)` 改為 `throw new BookingRuleViolationException(...)`，訊息文字不變；`catch (\RuntimeException $e)` 改為 `catch (BookingRuleViolationException $e)`
+- [x] 1.3 [後端] `app/Http/Controllers/API/ProviderBookingController.php` 的 `confirm`：同 1.2，處理 `名額不足，無法確認此預約`
+- [x] 1.4 [後端] 確認 `grep -rn "RuntimeException" app/` 只剩 `BookingRuleViolationException` 本身的繼承宣告
+
+## 2. 測試
+
+- [x] 2.1 [整合測試] `test_duplicate_booking_returns_422_with_rule_message`：member 已有該時段 `pending` 預約，再次 `POST /api/member/bookings`，回 422，`assertExactJson(['status' => false, 'message' => '您已預約此時段'])`，該 member 在此時段的預約仍為 1 筆；測試開頭 `Exceptions::fake()`，最後 `Exceptions::assertNotReported(BookingRuleViolationException::class)`（對應 Scenario「重複預約仍回 422 與既定訊息」）
+- [x] 2.2 [整合測試] `test_capacity_lost_inside_transaction_returns_422_with_rule_message`：以 `CourseSchedule::retrieved` 事件在交易內把 `current_participants` 設為滿額（事件只能改動交易內 `lockForUpdate()->find()` 那次讀取；交易前的第一次讀取若被改動，會被前置檢查以「人數超過剩餘名額」攔下，測不到交易內路徑），`POST /api/member/bookings` 回 422，`assertExactJson(['status' => false, 'message' => '名額不足，請重新選擇'])`，`bookings` 筆數為 0；同 2.1 以 `Exceptions::fake()` 斷言未回報（對應 Scenario「交易內名額不足仍回 422 與既定訊息」）
+- [x] 2.3 [整合測試] `test_confirm_without_capacity_returns_422_with_rule_message`：provider 確認人數大於剩餘名額的 `pending` 預約，回 422，`assertExactJson(['status' => false, 'message' => '名額不足，無法確認此預約'])`，預約仍為 `pending`、`current_participants` 不變；同 2.1 以 `Exceptions::fake()` 斷言未回報（對應 Scenario「確認預約時名額不足仍回 422 與既定訊息」）
+- [x] 2.4 [整合測試] `test_database_error_on_create_returns_500_without_sql`：`config(['app.debug' => false])`、`Exceptions::fake()`，`Booking::created`（INSERT 已執行之後）拋出 SQL 含 `insert into "bookings"` 與欄位名稱的 `QueryException`；合法 `POST /api/member/bookings` 回 500，`assertExactJson(['message' => 'Server Error'])`，body 不含 `insert into`、`bookings`、`member_id`，`Exceptions::assertReported(QueryException::class)`，`bookings` 筆數為 0（INSERT 已發生，筆數為 0 才能證明 rollback）（對應 Scenario「建立預約時資料庫錯誤回 500 且不洩漏細節」）
+- [x] 2.5 [整合測試] `test_database_error_on_confirm_returns_500_without_sql`：同 2.4 的設定，`CourseSchedule::updating`（在交易內預約狀態已更新、`increment` 執行時觸發）拋出 SQL 含 `update "course_schedules"` 的 `QueryException`；provider 確認名額足夠的 `pending` 預約，回 500，`assertExactJson(['message' => 'Server Error'])`，body 不含 `update`、`course_schedules`、`current_participants`，`Exceptions::assertReported(QueryException::class)`，預約仍為 `pending`（狀態更新已執行，仍為 `pending` 才能證明 rollback）、`current_participants` 不變（對應 Scenario「確認預約時資料庫錯誤回 500 且不洩漏細節」）
+- [x] 2.6 [整合測試] 執行 `php artisan test --filter='BookingLifecycleTest|BookingOversellTest'`，確認既有 422 行為不變；完整 `php artisan test` 需在有 GD 的環境執行（見 `cfdive-test-gd` image 的做法）
+
+## 3. 手動驗證
+
+- [x] 3.1 [手動驗證] 變異驗證（只在本機執行，不提交）：暫時把 1.2、1.3 的 catch 改回 `\RuntimeException`，確認 2.4、2.5 失敗後再還原，證明測試抓得到原本的問題
+- [x] 3.2 [手動驗證] 部署後在正式環境建立一筆預約、確認一筆預約，確認業務流程正常（201／200）；Sentry 是否收到預約端點的 500 只能等實際錯誤發生，若期間沒有錯誤則記錄為「未觀察到」，不視為失敗（自動化驗證以 2.4、2.5 為準）
+  - 2026-10-18 結果：Hank 在正式環境建立、確認預約皆正常，未出現 500；Sentry 未觀察到預約端點的 500

openspec/config.yaml

@@ -61,8 +61,8 @@ context: |
   ## 後端慣例
   - Controller 位於 `app/Http/Controllers/API/`，Swagger 註解位於 `app/Docs/`
   - 路由集中於 `routes/api.php`；admin 群組以 `admin` middleware 保護
-  - member / provider 路由群組目前僅有 `auth:sanctum`，角色檢查需在 Controller 內自行處理
+  - member / provider 路由群組以 `member`／`provider` middleware 保護（role-authorization spec）；
-    （已知缺口，見 docs/analysis/2026-10-14-project-review-todo.md H1；新增路由時不可假設已有角色保護）
+    middleware 只擋跨角色，資源所有權仍需在 Controller 內檢查
 
   ## API 回應格式
   成功：`{ status: true, message?: "...", data: {...} 或 [...] }`

openspec/specs/booking-lifecycle/spec.md

@@ -133,3 +133,26 @@ Member SHALL 能查詢自己所有預約的列表及詳情，含課程連結與
 
 - **WHEN** notify 呼叫拋出例外
 - **THEN** 預約狀態已正確儲存，HTTP response 成功回傳，錯誤記錄至 Laravel log
+
+### Requirement: 預約交易錯誤分類
+建立預約（`POST /api/member/bookings`）與確認預約（`PUT /api/provider/bookings/{id}/confirm`）的交易中，系統 SHALL 區分業務規則失敗與伺服器端故障：業務規則失敗（名額不足、重複預約）回傳 422 與既定訊息；其他任何例外（含資料庫錯誤）MUST NOT 以 422 回傳，MUST 回傳 500，回應內容不得包含 SQL、資料表或欄位名稱，且該例外 MUST 交由例外回報機制記錄（正式環境會送至 Sentry）。兩種情況下交易都 MUST rollback，不留下部分寫入。
+
+#### Scenario: 重複預約仍回 422 與既定訊息
+- **WHEN** 已登入 member 對同一時段已有 `pending` 預約，再次送出 `POST /api/member/bookings`（`schedule_id` 相同、`participants` 1）
+- **THEN** 系統回傳 422，body 為 `{"status": false, "message": "您已預約此時段"}`，該 member 在此時段的預約筆數不變
+
+#### Scenario: 交易內名額不足仍回 422 與既定訊息
+- **WHEN** 已登入 member 送出 `POST /api/member/bookings` 時通過交易前的名額檢查，但交易內鎖定時段後重新計算的剩餘名額小於 `participants`
+- **THEN** 系統回傳 422，body 為 `{"status": false, "message": "名額不足，請重新選擇"}`，不建立預約
+
+#### Scenario: 確認預約時名額不足仍回 422 與既定訊息
+- **WHEN** 已登入 provider 對自己課程的 `pending` 預約送出 `PUT /api/provider/bookings/{id}/confirm`，而該預約人數大於時段剩餘名額
+- **THEN** 系統回傳 422，body 為 `{"status": false, "message": "名額不足，無法確認此預約"}`，預約維持 `pending`，`current_participants` 不變
+
+#### Scenario: 建立預約時資料庫錯誤回 500 且不洩漏細節
+- **WHEN** `APP_DEBUG` 為 false，已登入 member 送出合法的 `POST /api/member/bookings`，而交易內寫入預約之後發生資料庫例外
+- **THEN** 系統回傳 500，body 為通用錯誤訊息且不含 SQL 語句、`bookings` 等資料表名稱與欄位名稱，資料庫例外被回報，`bookings` 筆數不變
+
+#### Scenario: 確認預約時資料庫錯誤回 500 且不洩漏細節
+- **WHEN** `APP_DEBUG` 為 false，已登入 provider 對自己課程的 `pending` 預約送出 `PUT /api/provider/bookings/{id}/confirm`（名額足夠），而交易內預約狀態已更新、更新時段名額時發生資料庫例外
+- **THEN** 系統回傳 500，body 為通用錯誤訊息且不含 SQL 語句、資料表名稱與欄位名稱，資料庫例外被回報，預約維持 `pending`，`current_participants` 不變

openspec/specs/feature-test-coverage/spec.md

@@ -126,8 +126,8 @@
 
 ### Requirement: 預約列表端點測試覆蓋
 測試套件 SHALL 驗證下列三個路由的資料隔離行為：
-- `GET /api/member/bookings`（middleware: `auth:sanctum`，隔離靠 `where member_id = auth()->id()`）
+- `GET /api/member/bookings`（middleware: `auth:sanctum` + `member`，非 member 角色於 middleware 層強制 403；隔離靠 `where member_id = auth()->id()`）
-- `GET /api/provider/bookings`（middleware: `auth:sanctum`，隔離靠 `whereHas schedule.provider_id`）
+- `GET /api/provider/bookings`（middleware: `auth:sanctum` + `provider`，非 provider 角色於 middleware 層強制 403；隔離靠 `whereHas schedule.provider_id`）
 - `GET /api/admin/bookings`（middleware: `auth:sanctum` + `admin`，EnsureAdmin 於 middleware 層強制 403）
 
 #### Scenario: Member 只能看到自己的預約

openspec/specs/role-authorization/spec.md

@@ -0,0 +1,68 @@
+# role-authorization Specification
+
+## Purpose
+
+依使用者角色限制 member、provider 專屬 API 路由群組的存取，避免已登入的其他角色跨角色呼叫課程、時段、預約等業務端點。
+
+## Requirements
+
+### Requirement: Member 專屬路由僅限 member 角色存取
+`/api/member/*` 路由群組（不含公開的 `POST /api/member/register`、`POST /api/member/login`）SHALL 只允許 `role = member` 的已登入使用者存取。其他角色 MUST 收到 HTTP 403，回應為 `{ "status": false, "message": "無權限存取" }`，且請求 MUST NOT 產生任何資料變更。未帶 token 的請求 MUST 收到 HTTP 401。
+
+#### Scenario: member 可存取 member 路由
+- **WHEN** 一個 `role = member` 的使用者，以該使用者的 token 呼叫 `GET /api/member/profile` 與 `GET /api/member/bookings`
+- **THEN** 回傳 HTTP 200，`status` 為 `true`
+
+#### Scenario: provider 呼叫 member 路由被拒絕
+- **WHEN** 一個 `role = provider` 的使用者，以該使用者的 token 呼叫 `GET /api/member/profile`、`GET /api/member/bookings`、`POST /api/member/bookings`、`POST /api/member/reviews`
+- **THEN** 每個請求都回傳 HTTP 403，body 為 `{ "status": false, "message": "無權限存取" }`
+
+#### Scenario: admin 呼叫 member 路由被拒絕
+- **WHEN** 一個 `role = admin` 的使用者，以該使用者的 token 呼叫 `GET /api/member/profile`、`GET /api/member/bookings`、`POST /api/member/bookings`
+- **THEN** 每個請求都回傳 HTTP 403，body 為 `{ "status": false, "message": "無權限存取" }`
+
+#### Scenario: 非 member 建立預約不會產生資料
+- **WHEN** 一個可預約的課程時段，以及一個 `role = provider` 的使用者，以該使用者的 token 呼叫 `POST /api/member/bookings` 並帶入合法的時段 id
+- **THEN** 回傳 HTTP 403，`bookings` 資料表筆數不變，該時段的 `current_participants` 不變
+
+#### Scenario: 未登入呼叫 member 路由
+- **WHEN** 不帶 token 呼叫 `GET /api/member/profile`、`GET /api/member/bookings`
+- **THEN** 回傳 HTTP 401
+
+### Requirement: Provider 專屬路由僅限 provider 角色存取
+`/api/provider/*` 路由群組（不含公開的 `POST /api/provider/register`、`POST /api/provider/login`）SHALL 只允許 `role = provider` 的已登入使用者存取，且 MUST NOT 以教練審核狀態作為進入條件。其他角色 MUST 收到 HTTP 403，回應為 `{ "status": false, "message": "無權限存取" }`，且請求 MUST NOT 產生任何資料變更。未帶 token 的請求 MUST 收到 HTTP 401。
+
+#### Scenario: provider 可存取 provider 路由
+- **WHEN** 一個 `role = provider` 的使用者，以該使用者的 token 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers`、`GET /api/provider/schedules`、`GET /api/provider/bookings`
+- **THEN** 回傳 HTTP 200，`status` 為 `true`
+
+#### Scenario: 未通過審核的 provider 仍可進入 provider 路由
+- **WHEN** 一個 `role = provider`、`verification_status` 不是 approved（例如 unsubmitted 或 pending）的使用者，以該使用者的 token 呼叫 `GET /api/provider/verification`
+- **THEN** 回傳 HTTP 200，不會因角色 middleware 回 403
+
+#### Scenario: member 呼叫 provider 路由被拒絕
+- **WHEN** 一個 `role = member` 的使用者，以該使用者的 token 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers`、`POST /api/provider/offers`、`POST /api/provider/schedules`、`GET /api/provider/bookings`、`GET /api/provider/verification`
+- **THEN** 每個請求都回傳 HTTP 403，body 為 `{ "status": false, "message": "無權限存取" }`
+
+#### Scenario: admin 呼叫 provider 路由被拒絕
+- **WHEN** 一個 `role = admin` 的使用者，以該使用者的 token 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers`、`POST /api/provider/offers`、`GET /api/provider/bookings`
+- **THEN** 每個請求都回傳 HTTP 403，body 為 `{ "status": false, "message": "無權限存取" }`
+
+#### Scenario: 非 provider 建立課程不會產生資料
+- **WHEN** 一個 `role = member` 的使用者，以該使用者的 token 呼叫 `POST /api/provider/offers` 並帶入通過驗證的完整課程資料
+- **THEN** 回傳 HTTP 403，`diving_offers` 資料表筆數不變
+
+#### Scenario: 未登入呼叫 provider 路由
+- **WHEN** 不帶 token 呼叫 `GET /api/provider/profile`、`GET /api/provider/offers`
+- **THEN** 回傳 HTTP 401
+
+### Requirement: 跨角色共用路由不受角色限制影響
+通知（`/api/notifications/*`）與預約即時訊息（`/api/bookings/{booking}/messages*`、`/api/bookings/messages/unread-counts`）路由 SHALL 維持既有行為：member 與 provider 皆可存取，存取權限由既有的資源歸屬檢查決定，不套用 member 或 provider 角色限制。
+
+#### Scenario: member 與 provider 皆可讀取通知
+- **WHEN** 一個 `role = member` 與一個 `role = provider` 的使用者，分別以各自的 token 呼叫 `GET /api/notifications`
+- **THEN** 兩者皆回傳 HTTP 200
+
+#### Scenario: member 與 provider 皆可讀取未讀訊息數
+- **WHEN** 一個 `role = member` 與一個 `role = provider` 的使用者，分別以各自的 token 呼叫 `GET /api/bookings/messages/unread-counts`
+- **THEN** 兩者皆回傳 HTTP 200

routes/api.php

@@ -1,22 +1,22 @@
 <?php
 
-use Illuminate\Support\Facades\Broadcast;
-use Illuminate\Support\Facades\Route;
-use App\Http\Controllers\API\AuthController;
-use App\Http\Controllers\API\DivingOfferController;
-use App\Http\Controllers\API\ProviderOfferController;
-use App\Http\Controllers\API\ScheduleController;
-use App\Http\Controllers\API\ProviderBookingController;
-use App\Http\Controllers\API\MemberBookingController;
-use App\Http\Controllers\API\ReviewController;
-use App\Http\Controllers\API\AdminReviewController;
 use App\Http\Controllers\API\AdminBookingController;
-use App\Http\Controllers\API\CourseImageController;
+use App\Http\Controllers\API\AdminOfferController;
+use App\Http\Controllers\API\AdminReviewController;
 use App\Http\Controllers\API\AdminStatsController;
 use App\Http\Controllers\API\AdminUserController;
-use App\Http\Controllers\API\AdminOfferController;
+use App\Http\Controllers\API\AuthController;
 use App\Http\Controllers\API\BookingMessageController;
+use App\Http\Controllers\API\CourseImageController;
+use App\Http\Controllers\API\DivingOfferController;
+use App\Http\Controllers\API\MemberBookingController;
 use App\Http\Controllers\API\NotificationController;
+use App\Http\Controllers\API\ProviderBookingController;
+use App\Http\Controllers\API\ProviderOfferController;
+use App\Http\Controllers\API\ReviewController;
+use App\Http\Controllers\API\ScheduleController;
+use Illuminate\Support\Facades\Broadcast;
+use Illuminate\Support\Facades\Route;
 
 Broadcast::routes(['middleware' => ['auth:sanctum']]);
 
@@ -24,7 +24,7 @@ Broadcast::routes(['middleware' => ['auth:sanctum']]);
 Route::get('/diving-offers', [DivingOfferController::class, 'index']);
 Route::get('/diving-offers/{id}', [DivingOfferController::class, 'show']);
 Route::get('/diving-offers/{id}/schedules', [ScheduleController::class, 'publicList']);
-Route::get('/diving-offers/{id}/reviews',  [ReviewController::class, 'publicList']);
+Route::get('/diving-offers/{id}/reviews', [ReviewController::class, 'publicList']);
 
 // 會員註冊／登入
 Route::post('/member/register', [AuthController::class, 'registerMember']);
@@ -37,7 +37,7 @@ Route::middleware([\Illuminate\Session\Middleware\StartSession::class])->group(f
 });
 
 // 會員專屬 API（需登入）
-Route::middleware(['auth:sanctum'])->prefix('member')->group(function () {
+Route::middleware(['auth:sanctum', 'member'])->prefix('member')->group(function () {
     // 會員登出
     Route::post('/logout', [AuthController::class, 'logoutMember']);
     // Token Refresh
@@ -49,14 +49,14 @@ Route::middleware(['auth:sanctum'])->prefix('member')->group(function () {
     // 修改密碼
     Route::put('/change-password', [AuthController::class, 'changeMemberPassword']);
     // 預約
-    Route::get('/bookings',          [MemberBookingController::class, 'index']);
+    Route::get('/bookings', [MemberBookingController::class, 'index']);
-    Route::post('/bookings',         [MemberBookingController::class, 'store']);
+    Route::post('/bookings', [MemberBookingController::class, 'store']);
-    Route::get('/bookings/{id}',     [MemberBookingController::class, 'show']);
+    Route::get('/bookings/{id}', [MemberBookingController::class, 'show']);
-    Route::delete('/bookings/{id}',  [MemberBookingController::class, 'destroy']);
+    Route::delete('/bookings/{id}', [MemberBookingController::class, 'destroy']);
     // 評價
-    Route::post('/reviews',       [ReviewController::class, 'store']);
+    Route::post('/reviews', [ReviewController::class, 'store']);
-    Route::put('/reviews/{id}',   [ReviewController::class, 'update']);
+    Route::put('/reviews/{id}', [ReviewController::class, 'update']);
-    Route::delete('/reviews/{id}',[ReviewController::class, 'destroy']);
+    Route::delete('/reviews/{id}', [ReviewController::class, 'destroy']);
 });
 
 // 有幫助投票（需登入，但不限 member prefix）
@@ -67,7 +67,7 @@ Route::post('/provider/register', [AuthController::class, 'registerProvider']);
 Route::middleware('throttle:10,1')->post('/provider/login', [AuthController::class, 'loginProvider']);
 
 // 服務提供者專屬 API（需登入）
-Route::middleware(['auth:sanctum'])->prefix('provider')->group(function () {
+Route::middleware(['auth:sanctum', 'provider'])->prefix('provider')->group(function () {
     // 服務提供者登出
     Route::post('/logout', [AuthController::class, 'logoutProvider']);
     // Token Refresh
@@ -79,32 +79,32 @@ Route::middleware(['auth:sanctum'])->prefix('provider')->group(function () {
     // 修改密碼
     Route::put('/change-password', [AuthController::class, 'changeProviderPassword']);
     // 教練課程管理
-    Route::get('/offers',          [ProviderOfferController::class, 'index']);
+    Route::get('/offers', [ProviderOfferController::class, 'index']);
-    Route::post('/offers',         [ProviderOfferController::class, 'store']);
+    Route::post('/offers', [ProviderOfferController::class, 'store']);
-    Route::get('/offers/{id}',     [ProviderOfferController::class, 'show']);
+    Route::get('/offers/{id}', [ProviderOfferController::class, 'show']);
-    Route::put('/offers/{id}',     [ProviderOfferController::class, 'update']);
+    Route::put('/offers/{id}', [ProviderOfferController::class, 'update']);
-    Route::delete('/offers/{id}',  [ProviderOfferController::class, 'destroy']);
+    Route::delete('/offers/{id}', [ProviderOfferController::class, 'destroy']);
     // 驗證申請（證照送審）
-    Route::get('/verification',                          [\App\Http\Controllers\API\ProviderVerificationController::class, 'show']);
+    Route::get('/verification', [\App\Http\Controllers\API\ProviderVerificationController::class, 'show']);
-    Route::post('/verification/certifications',          [\App\Http\Controllers\API\ProviderVerificationController::class, 'uploadCertification']);
+    Route::post('/verification/certifications', [\App\Http\Controllers\API\ProviderVerificationController::class, 'uploadCertification']);
-    Route::delete('/verification/certifications/{id}',   [\App\Http\Controllers\API\ProviderVerificationController::class, 'deleteCertification']);
+    Route::delete('/verification/certifications/{id}', [\App\Http\Controllers\API\ProviderVerificationController::class, 'deleteCertification']);
-    Route::post('/verification/submit',                  [\App\Http\Controllers\API\ProviderVerificationController::class, 'submit']);
+    Route::post('/verification/submit', [\App\Http\Controllers\API\ProviderVerificationController::class, 'submit']);
     // 課程圖片
-    Route::post('/offers/{id}/cover',    [CourseImageController::class, 'uploadCover']);
+    Route::post('/offers/{id}/cover', [CourseImageController::class, 'uploadCover']);
-    Route::delete('/offers/{id}/cover',  [CourseImageController::class, 'deleteCover']);
+    Route::delete('/offers/{id}/cover', [CourseImageController::class, 'deleteCover']);
-    Route::post('/offers/{id}/images',   [CourseImageController::class, 'uploadImage']);
+    Route::post('/offers/{id}/images', [CourseImageController::class, 'uploadImage']);
-    Route::delete('/images/{id}',        [CourseImageController::class, 'deleteImage']);
+    Route::delete('/images/{id}', [CourseImageController::class, 'deleteImage']);
     // 時段管理
-    Route::get('/schedules',            [ScheduleController::class, 'index']);
+    Route::get('/schedules', [ScheduleController::class, 'index']);
-    Route::post('/schedules',           [ScheduleController::class, 'store']);
+    Route::post('/schedules', [ScheduleController::class, 'store']);
-    Route::put('/schedules/{id}',       [ScheduleController::class, 'update']);
+    Route::put('/schedules/{id}', [ScheduleController::class, 'update']);
-    Route::delete('/schedules/{id}',    [ScheduleController::class, 'destroy']);
+    Route::delete('/schedules/{id}', [ScheduleController::class, 'destroy']);
     // 預約管理
-    Route::get('/bookings',                      [ProviderBookingController::class, 'index']);
+    Route::get('/bookings', [ProviderBookingController::class, 'index']);
-    Route::put('/bookings/{id}/confirm',         [ProviderBookingController::class, 'confirm']);
+    Route::put('/bookings/{id}/confirm', [ProviderBookingController::class, 'confirm']);
-    Route::put('/bookings/{id}/reject',          [ProviderBookingController::class, 'reject']);
+    Route::put('/bookings/{id}/reject', [ProviderBookingController::class, 'reject']);
-    Route::put('/bookings/{id}/cancel',          [ProviderBookingController::class, 'cancel']);
+    Route::put('/bookings/{id}/cancel', [ProviderBookingController::class, 'cancel']);
-    Route::put('/bookings/{id}/complete',        [ProviderBookingController::class, 'complete']);
+    Route::put('/bookings/{id}/complete', [ProviderBookingController::class, 'complete']);
 });
 
 // 管理員登入（帳號建立僅限主機端 php artisan app:create-admin，不提供公開註冊）
@@ -129,42 +129,42 @@ Route::middleware(['auth:sanctum', 'admin'])->prefix('admin')->group(function ()
     // 統計數據
     Route::get('/stats', [AdminStatsController::class, 'index']);
     // 用戶管理
-    Route::get('/members',                          [AdminUserController::class, 'members']);
+    Route::get('/members', [AdminUserController::class, 'members']);
-    Route::get('/members/{id}',                     [AdminUserController::class, 'member']);
+    Route::get('/members/{id}', [AdminUserController::class, 'member']);
-    Route::put('/members/{id}/toggle-active',       [AdminUserController::class, 'toggleMemberActive']);
+    Route::put('/members/{id}/toggle-active', [AdminUserController::class, 'toggleMemberActive']);
-    Route::get('/providers',                        [AdminUserController::class, 'providers']);
+    Route::get('/providers', [AdminUserController::class, 'providers']);
-    Route::get('/providers/{id}',                   [AdminUserController::class, 'provider']);
+    Route::get('/providers/{id}', [AdminUserController::class, 'provider']);
-    Route::put('/providers/{id}/toggle-active',     [AdminUserController::class, 'toggleProviderActive']);
+    Route::put('/providers/{id}/toggle-active', [AdminUserController::class, 'toggleProviderActive']);
     // 教練審核（toggle-verified 已由審核狀態機取代）
-    Route::get('/verifications',                    [\App\Http\Controllers\API\AdminVerificationController::class, 'index']);
+    Route::get('/verifications', [\App\Http\Controllers\API\AdminVerificationController::class, 'index']);
-    Route::put('/verifications/{userId}/approve',   [\App\Http\Controllers\API\AdminVerificationController::class, 'approve']);
+    Route::put('/verifications/{userId}/approve', [\App\Http\Controllers\API\AdminVerificationController::class, 'approve']);
-    Route::put('/verifications/{userId}/reject',    [\App\Http\Controllers\API\AdminVerificationController::class, 'reject']);
+    Route::put('/verifications/{userId}/reject', [\App\Http\Controllers\API\AdminVerificationController::class, 'reject']);
     // 課程管理
-    Route::get('/offers',           [AdminOfferController::class, 'index']);
+    Route::get('/offers', [AdminOfferController::class, 'index']);
-    Route::delete('/offers/{id}',   [AdminOfferController::class, 'destroy']);
+    Route::delete('/offers/{id}', [AdminOfferController::class, 'destroy']);
     // 預約管理
-    Route::get('/bookings',                   [AdminBookingController::class, 'index']);
+    Route::get('/bookings', [AdminBookingController::class, 'index']);
-    Route::put('/bookings/{id}/complete',     [AdminBookingController::class, 'complete']);
+    Route::put('/bookings/{id}/complete', [AdminBookingController::class, 'complete']);
     // 評價管理
-    Route::get('/reviews',          [AdminReviewController::class, 'index']);
+    Route::get('/reviews', [AdminReviewController::class, 'index']);
-    Route::delete('/reviews/{id}',  [AdminReviewController::class, 'destroy']);
+    Route::delete('/reviews/{id}', [AdminReviewController::class, 'destroy']);
 });
 
 // 通知（Member + Provider 共用）
 Route::middleware('auth:sanctum')->prefix('notifications')->group(function () {
-    Route::get('/',          [NotificationController::class, 'index']);
+    Route::get('/', [NotificationController::class, 'index']);
     Route::get('/unread-count', [NotificationController::class, 'unreadCount']);
-    Route::patch('/read-all',   [NotificationController::class, 'markAllRead']);
+    Route::patch('/read-all', [NotificationController::class, 'markAllRead']);
-    Route::patch('/{id}/read',  [NotificationController::class, 'markRead']);
+    Route::patch('/{id}/read', [NotificationController::class, 'markRead']);
-    Route::delete('/{id}',      [NotificationController::class, 'destroy']);
+    Route::delete('/{id}', [NotificationController::class, 'destroy']);
 });
 
 // 即時訊息（Member + Provider 共用，依 booking 參與方驗證）
 Route::middleware('auth:sanctum')->group(function () {
     // unread-counts 必須在 {booking} 之前，否則會被 route model binding 吃掉
-    Route::get('/bookings/messages/unread-counts',   [BookingMessageController::class, 'unreadCounts']);
+    Route::get('/bookings/messages/unread-counts', [BookingMessageController::class, 'unreadCounts']);
-    Route::get('/bookings/{booking}/messages',       [BookingMessageController::class, 'index']);
+    Route::get('/bookings/{booking}/messages', [BookingMessageController::class, 'index']);
-    Route::post('/bookings/{booking}/messages',      [BookingMessageController::class, 'store']);
+    Route::post('/bookings/{booking}/messages', [BookingMessageController::class, 'store']);
     Route::post('/bookings/{booking}/messages/read', [BookingMessageController::class, 'markRead']);
 });
 
@@ -172,4 +172,4 @@ Route::middleware('auth:sanctum')->group(function () {
 Route::middleware('auth:sanctum')->group(function () {
     Route::post('/logout', [AuthController::class, 'logout']);
     Route::get('/user', [AuthController::class, 'user']);
-});
+});

tests/Feature/BookingExceptionHandlingTest.php

@@ -0,0 +1,201 @@
+<?php
+
+namespace Tests\Feature;
+
+use App\Enums\BookingStatus;
+use App\Enums\ScheduleStatus;
+use App\Exceptions\BookingRuleViolationException;
+use App\Models\Booking;
+use App\Models\CourseSchedule;
+use App\Models\DivingOffer;
+use App\Models\ProviderProfile;
+use App\Models\User;
+use Illuminate\Database\QueryException;
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Illuminate\Support\Facades\Exceptions;
+use Illuminate\Support\Facades\Notification;
+use Tests\TestCase;
+
+/**
+ * 預約交易的錯誤分類：業務規則失敗回 422，其他例外交給 Laravel 例外處理器。
+ *
+ * 原本兩個 Controller 以 catch (\RuntimeException) 接住交易內的錯誤，而
+ * QueryException 也是 RuntimeException 的子類別，資料庫錯誤因此以 422
+ * 把 SQL 與資料表結構回給使用者，且不會被回報到 Sentry。
+ */
+class BookingExceptionHandlingTest extends TestCase
+{
+    use RefreshDatabase;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+        Notification::fake();
+        Exceptions::fake();
+    }
+
+    private function makeProviderWithSchedule(int $maxParticipants, int $currentParticipants = 0): array
+    {
+        $provider = User::factory()->create(['role' => 'provider']);
+
+        ProviderProfile::create([
+            'user_id' => $provider->id,
+            'verification_status' => 'approved',
+        ]);
+
+        $offer = DivingOffer::create([
+            'provider_id' => $provider->id,
+            'title' => 'Exception Course',
+            'location' => 'Test',
+            'spot' => 'Test Spot',
+            'price' => 1000,
+            'region' => '南部',
+            'rating' => 0,
+            'reviews' => 0,
+        ]);
+
+        $schedule = CourseSchedule::create([
+            'diving_offer_id' => $offer->id,
+            'provider_id' => $provider->id,
+            'scheduled_date' => now()->addDays(7)->toDateString(),
+            'start_time' => '09:00',
+            'max_participants' => $maxParticipants,
+            'current_participants' => $currentParticipants,
+            'status' => ScheduleStatus::Open,
+        ]);
+
+        return [$provider, $schedule];
+    }
+
+    private function makePendingBooking(CourseSchedule $schedule, int $participants, ?User $member = null): Booking
+    {
+        return Booking::create([
+            'schedule_id' => $schedule->id,
+            'member_id' => ($member ?? User::factory()->create(['role' => 'member']))->id,
+            'participants' => $participants,
+            'total_price' => 1000 * $participants,
+            'status' => BookingStatus::Pending,
+        ]);
+    }
+
+    // 刻意不用 "database is locked" 這類併發錯誤訊息：巢狀交易（RefreshDatabase）中 Laravel 會改拋 DeadlockException
+    private function makeQueryException(string $sql): QueryException
+    {
+        return new QueryException('sqlite', $sql, [], new \PDOException('SQLSTATE[23000]: Integrity constraint violation'));
+    }
+
+    /**
+     * 業務失敗的訊息是前端直接顯示給使用者的文字，改型別不能改到它；
+     * 也不應回報到 Sentry，否則正常的業務拒絕會淹沒真正的伺服器故障。
+     */
+    public function test_duplicate_booking_returns_422_with_rule_message(): void
+    {
+        [, $schedule] = $this->makeProviderWithSchedule(5);
+        $member = User::factory()->create(['role' => 'member']);
+        $this->makePendingBooking($schedule, 1, $member);
+
+        $this->actingAs($member)
+            ->postJson('/api/member/bookings', ['schedule_id' => $schedule->id, 'participants' => 1])
+            ->assertStatus(422)
+            ->assertExactJson(['status' => false, 'message' => '您已預約此時段']);
+
+        $this->assertSame(1, Booking::where('member_id', $member->id)->where('schedule_id', $schedule->id)->count());
+        Exceptions::assertNotReported(BookingRuleViolationException::class);
+    }
+
+    /**
+     * 交易內名額不足只在併發時發生：交易前的檢查通過後，另一筆預約搶先佔位。
+     * 只改動第二次讀取（交易內 lockForUpdate），第一次讀取若被改動會被前置檢查攔下，測不到這條路徑。
+     */
+    public function test_capacity_lost_inside_transaction_returns_422_with_rule_message(): void
+    {
+        [, $schedule] = $this->makeProviderWithSchedule(2);
+        $member = User::factory()->create(['role' => 'member']);
+
+        $retrievalCount = 0;
+        CourseSchedule::retrieved(function (CourseSchedule $retrieved) use (&$retrievalCount) {
+            $retrievalCount++;
+            if ($retrievalCount === 2) {
+                $retrieved->current_participants = $retrieved->max_participants;
+            }
+        });
+
+        $this->actingAs($member)
+            ->postJson('/api/member/bookings', ['schedule_id' => $schedule->id, 'participants' => 1])
+            ->assertStatus(422)
+            ->assertExactJson(['status' => false, 'message' => '名額不足，請重新選擇']);
+
+        $this->assertSame(0, Booking::count());
+        Exceptions::assertNotReported(BookingRuleViolationException::class);
+    }
+
+    /**
+     * 確認時的名額檢查是防超賣的最後一道防線，其 422 訊息與「不留下部分寫入」都必須維持。
+     */
+    public function test_confirm_without_capacity_returns_422_with_rule_message(): void
+    {
+        [$provider, $schedule] = $this->makeProviderWithSchedule(2, 1);
+        $booking = $this->makePendingBooking($schedule, 2);
+
+        $this->actingAs($provider)
+            ->putJson("/api/provider/bookings/{$booking->id}/confirm")
+            ->assertStatus(422)
+            ->assertExactJson(['status' => false, 'message' => '名額不足，無法確認此預約']);
+
+        $this->assertSame(BookingStatus::Pending, $booking->fresh()->status);
+        $this->assertSame(1, $schedule->fresh()->current_participants);
+        Exceptions::assertNotReported(BookingRuleViolationException::class);
+    }
+
+    /**
+     * 資料庫錯誤曾以 422 把 SQL 回給使用者且不會送到 Sentry。
+     * 例外在 INSERT 之後才拋出，筆數為 0 才能證明交易確實 rollback。
+     */
+    public function test_database_error_on_create_returns_500_without_sql(): void
+    {
+        config(['app.debug' => false]);
+        [, $schedule] = $this->makeProviderWithSchedule(5);
+        $member = User::factory()->create(['role' => 'member']);
+
+        Booking::created(function () {
+            throw $this->makeQueryException('insert into "bookings" ("schedule_id", "member_id", "participants") values (?, ?, ?)');
+        });
+
+        $this->actingAs($member)
+            ->postJson('/api/member/bookings', ['schedule_id' => $schedule->id, 'participants' => 1])
+            ->assertStatus(500)
+            ->assertExactJson(['message' => 'Server Error'])
+            ->assertDontSee('insert into', false)
+            ->assertDontSee('bookings', false)
+            ->assertDontSee('member_id', false);
+
+        Exceptions::assertReported(QueryException::class);
+        $this->assertSame(0, Booking::count());
+    }
+
+    /**
+     * 同上，針對確認預約。例外在預約狀態更新之後才拋出，預約仍為 pending 才能證明 rollback。
+     */
+    public function test_database_error_on_confirm_returns_500_without_sql(): void
+    {
+        config(['app.debug' => false]);
+        [$provider, $schedule] = $this->makeProviderWithSchedule(5);
+        $booking = $this->makePendingBooking($schedule, 2);
+
+        CourseSchedule::updating(function () {
+            throw $this->makeQueryException('update "course_schedules" set "current_participants" = "current_participants" + 2 where "id" = ?');
+        });
+
+        $this->actingAs($provider)
+            ->putJson("/api/provider/bookings/{$booking->id}/confirm")
+            ->assertStatus(500)
+            ->assertExactJson(['message' => 'Server Error'])
+            ->assertDontSee('update', false)
+            ->assertDontSee('course_schedules', false)
+            ->assertDontSee('current_participants', false);
+
+        Exceptions::assertReported(QueryException::class);
+        $this->assertSame(BookingStatus::Pending, $booking->fresh()->status);
+        $this->assertSame(0, $schedule->fresh()->current_participants);
+    }
+}

tests/Feature/BookingListTest.php

@@ -16,12 +16,12 @@ use Tests\TestCase;
  * 預約列表端點資料隔離（booking-lifecycle 規格）。
  *
  * 路由對應（routes/api.php 已確認）：
- *   GET /api/member/bookings   → MemberBookingController::index（auth:sanctum，無 role check）
+ *   GET /api/member/bookings   → MemberBookingController::index（auth:sanctum + member middleware）
- *   GET /api/provider/bookings → ProviderBookingController::index（auth:sanctum，無 role check）
+ *   GET /api/provider/bookings → ProviderBookingController::index（auth:sanctum + provider middleware）
  *   GET /api/admin/bookings    → AdminBookingController::index（auth:sanctum + admin middleware）
  *
- * 資料隔離靠 controller query 條件，不靠 middleware role 強制，
+ * 角色 middleware 只擋跨角色呼叫（見 RoleAuthorizationTest）；同角色之間的資料隔離
- * 因此測試需分別以正確角色 actingAs 驗證隔離邊界。
+ * 靠 controller query 條件，因此測試需分別以正確角色 actingAs 驗證隔離邊界。
  */
 class BookingListTest extends TestCase
 {
@@ -34,22 +34,22 @@ class BookingListTest extends TestCase
 
         $offer = DivingOffer::create([
             'provider_id' => $provider->id,
-            'title'       => 'Dive Course',
+            'title' => 'Dive Course',
-            'location'    => 'Kenting',
+            'location' => 'Kenting',
-            'price'       => 3000,
+            'price' => 3000,
-            'region'      => '南部',
+            'region' => '南部',
-            'rating'      => 0,
+            'rating' => 0,
-            'reviews'     => 0,
+            'reviews' => 0,
         ]);
 
         $schedule = CourseSchedule::create([
-            'diving_offer_id'      => $offer->id,
+            'diving_offer_id' => $offer->id,
-            'provider_id'          => $provider->id,
+            'provider_id' => $provider->id,
-            'scheduled_date'       => now()->addDays(30)->toDateString(),
+            'scheduled_date' => now()->addDays(30)->toDateString(),
-            'start_time'           => '09:00',
+            'start_time' => '09:00',
-            'max_participants'     => 10,
+            'max_participants' => 10,
             'current_participants' => 0,
-            'status'               => ScheduleStatus::Open,
+            'status' => ScheduleStatus::Open,
         ]);
 
         return [$provider, $schedule];
@@ -58,11 +58,11 @@ class BookingListTest extends TestCase
     private function makePendingBooking(User $member, CourseSchedule $schedule): Booking
     {
         return Booking::create([
-            'schedule_id'  => $schedule->id,
+            'schedule_id' => $schedule->id,
-            'member_id'    => $member->id,
+            'member_id' => $member->id,
             'participants' => 1,
-            'total_price'  => 3000,
+            'total_price' => 3000,
-            'status'       => BookingStatus::Pending,
+            'status' => BookingStatus::Pending,
         ]);
     }
 
@@ -71,8 +71,8 @@ class BookingListTest extends TestCase
     public function test_member_sees_only_own_bookings(): void
     {
         [$provider, $schedule] = $this->makeProviderWithSchedule();
-        $memberA  = User::factory()->create(['role' => 'member']);
+        $memberA = User::factory()->create(['role' => 'member']);
-        $memberB  = User::factory()->create(['role' => 'member']);
+        $memberB = User::factory()->create(['role' => 'member']);
         $bookingA = $this->makePendingBooking($memberA, $schedule);
         $bookingB = $this->makePendingBooking($memberB, $schedule);
 
@@ -90,7 +90,7 @@ class BookingListTest extends TestCase
     {
         [$providerA, $scheduleA] = $this->makeProviderWithSchedule();
         [$providerB, $scheduleB] = $this->makeProviderWithSchedule();
-        $member   = User::factory()->create(['role' => 'member']);
+        $member = User::factory()->create(['role' => 'member']);
         $bookingA = $this->makePendingBooking($member, $scheduleA);
         $bookingB = $this->makePendingBooking($member, $scheduleB);
 
@@ -106,11 +106,11 @@ class BookingListTest extends TestCase
 
     public function test_admin_sees_all_bookings(): void
     {
-        $admin    = User::factory()->create(['role' => 'admin']);
+        $admin = User::factory()->create(['role' => 'admin']);
         [$providerA, $scheduleA] = $this->makeProviderWithSchedule();
         [$providerB, $scheduleB] = $this->makeProviderWithSchedule();
-        $memberA  = User::factory()->create(['role' => 'member']);
+        $memberA = User::factory()->create(['role' => 'member']);
-        $memberB  = User::factory()->create(['role' => 'member']);
+        $memberB = User::factory()->create(['role' => 'member']);
         $bookingA = $this->makePendingBooking($memberA, $scheduleA);
         $bookingB = $this->makePendingBooking($memberB, $scheduleB);
 

tests/Feature/RoleAuthorizationTest.php

@@ -0,0 +1,230 @@
+<?php
+
+namespace Tests\Feature;
+
+use App\Enums\ScheduleStatus;
+use App\Models\CourseSchedule;
+use App\Models\DivingOffer;
+use App\Models\ProviderProfile;
+use App\Models\User;
+use Illuminate\Foundation\Testing\RefreshDatabase;
+use Tests\TestCase;
+
+/**
+ * Member / Provider 路由群組角色邊界（role-authorization 規格）。
+ *
+ * 在 EnsureMember / EnsureProvider 之前，任何已登入帳號都能跨角色呼叫：
+ * member 可建立課程（provider_id 被設成自己）、provider 可建立預約。
+ * 這裡驗證角色不符一律在 middleware 層回 403 且不產生資料；
+ * POST 請求都帶通過驗證的完整資料，確保 403 不是被 422 或資源檢查遮蓋。
+ */
+class RoleAuthorizationTest extends TestCase
+{
+    use RefreshDatabase;
+
+    private const FORBIDDEN_BODY = ['status' => false, 'message' => '無權限存取'];
+
+    private function createProvider(string $verificationStatus = 'approved'): User
+    {
+        $provider = User::factory()->create(['role' => 'provider']);
+        ProviderProfile::create(['user_id' => $provider->id, 'verification_status' => $verificationStatus]);
+
+        return $provider;
+    }
+
+    private function createOfferWithOpenSchedule(User $provider): CourseSchedule
+    {
+        $offer = DivingOffer::create([
+            'provider_id' => $provider->id,
+            'title' => 'Dive Course',
+            'location' => 'Kenting',
+            'price' => 3000,
+            'region' => '南部',
+            'rating' => 0,
+            'reviews' => 0,
+        ]);
+
+        return CourseSchedule::create([
+            'diving_offer_id' => $offer->id,
+            'provider_id' => $provider->id,
+            'scheduled_date' => now()->addDays(30)->toDateString(),
+            'start_time' => '09:00',
+            'max_participants' => 10,
+            'current_participants' => 0,
+            'status' => ScheduleStatus::Open,
+        ]);
+    }
+
+    private function validOfferPayload(): array
+    {
+        return [
+            'title' => 'Cross Role Course',
+            'location' => 'Green Island',
+            'price' => 2000,
+            'region' => '東部',
+        ];
+    }
+
+    private function assertForbidden(User $user, string $method, string $uri, array $payload = []): void
+    {
+        $this->actingAs($user)
+            ->json($method, $uri, $payload)
+            ->assertStatus(403)
+            ->assertExactJson(self::FORBIDDEN_BODY);
+    }
+
+    // ── Member 路由 ──────────────────────────────────────────────
+
+    public function test_member_can_access_member_routes(): void
+    {
+        $member = User::factory()->create(['role' => 'member']);
+
+        $this->actingAs($member)->getJson('/api/member/profile')->assertOk()->assertJsonPath('status', true);
+        $this->actingAs($member)->getJson('/api/member/bookings')->assertOk()->assertJsonPath('status', true);
+    }
+
+    public function test_provider_is_rejected_on_member_routes(): void
+    {
+        $provider = $this->createProvider();
+        $schedule = $this->createOfferWithOpenSchedule($provider);
+
+        $this->assertForbidden($provider, 'GET', '/api/member/profile');
+        $this->assertForbidden($provider, 'GET', '/api/member/bookings');
+        $this->assertForbidden($provider, 'POST', '/api/member/bookings', [
+            'schedule_id' => $schedule->id,
+            'participants' => 1,
+        ]);
+        $this->assertForbidden($provider, 'POST', '/api/member/reviews', [
+            'diving_offer_id' => $schedule->diving_offer_id,
+            'rating' => 5,
+            'comment' => 'self review',
+        ]);
+
+        // provider 為自己的課程寫評價會灌水評分
+        $this->assertDatabaseCount('reviews', 0);
+    }
+
+    public function test_admin_is_rejected_on_member_routes(): void
+    {
+        $admin = User::factory()->create(['role' => 'admin']);
+        $schedule = $this->createOfferWithOpenSchedule($this->createProvider());
+
+        $this->assertForbidden($admin, 'GET', '/api/member/profile');
+        $this->assertForbidden($admin, 'GET', '/api/member/bookings');
+        $this->assertForbidden($admin, 'POST', '/api/member/bookings', [
+            'schedule_id' => $schedule->id,
+            'participants' => 1,
+        ]);
+    }
+
+    public function test_non_member_booking_attempt_creates_no_booking(): void
+    {
+        $provider = $this->createProvider();
+        $schedule = $this->createOfferWithOpenSchedule($provider);
+
+        $this->assertForbidden($provider, 'POST', '/api/member/bookings', [
+            'schedule_id' => $schedule->id,
+            'participants' => 2,
+        ]);
+
+        // 越權預約若寫入會佔用真實學員的名額
+        $this->assertDatabaseCount('bookings', 0);
+        $this->assertSame(0, $schedule->fresh()->current_participants);
+    }
+
+    public function test_unauthenticated_request_is_rejected_on_member_routes(): void
+    {
+        $this->getJson('/api/member/profile')->assertStatus(401);
+        $this->getJson('/api/member/bookings')->assertStatus(401);
+    }
+
+    // ── Provider 路由 ────────────────────────────────────────────
+
+    public function test_provider_can_access_provider_routes(): void
+    {
+        $provider = $this->createProvider();
+
+        foreach (['/api/provider/profile', '/api/provider/offers', '/api/provider/schedules', '/api/provider/bookings'] as $uri) {
+            $this->actingAs($provider)->getJson($uri)->assertOk()->assertJsonPath('status', true);
+        }
+    }
+
+    public function test_unapproved_provider_can_access_verification(): void
+    {
+        // 未通過審核的教練必須能進入 provider 路由才能送審或補件
+        foreach (['unsubmitted', 'pending', 'rejected'] as $status) {
+            $provider = $this->createProvider($status);
+
+            $this->actingAs($provider)
+                ->getJson('/api/provider/verification')
+                ->assertOk()
+                ->assertJsonPath('data.verification_status', $status);
+        }
+    }
+
+    public function test_member_is_rejected_on_provider_routes(): void
+    {
+        $member = User::factory()->create(['role' => 'member']);
+        $schedule = $this->createOfferWithOpenSchedule($this->createProvider());
+
+        $this->assertForbidden($member, 'GET', '/api/provider/profile');
+        $this->assertForbidden($member, 'GET', '/api/provider/offers');
+        $this->assertForbidden($member, 'POST', '/api/provider/offers', $this->validOfferPayload());
+        $this->assertForbidden($member, 'POST', '/api/provider/schedules', [
+            'diving_offer_id' => $schedule->diving_offer_id,
+            'scheduled_date' => now()->addDays(10)->toDateString(),
+            'start_time' => '10:00',
+            'max_participants' => 5,
+        ]);
+        $this->assertForbidden($member, 'GET', '/api/provider/bookings');
+        $this->assertForbidden($member, 'GET', '/api/provider/verification');
+
+        $this->assertDatabaseCount('course_schedules', 1);
+    }
+
+    public function test_admin_is_rejected_on_provider_routes(): void
+    {
+        $admin = User::factory()->create(['role' => 'admin']);
+
+        $this->assertForbidden($admin, 'GET', '/api/provider/profile');
+        $this->assertForbidden($admin, 'GET', '/api/provider/offers');
+        $this->assertForbidden($admin, 'POST', '/api/provider/offers', $this->validOfferPayload());
+        $this->assertForbidden($admin, 'GET', '/api/provider/bookings');
+    }
+
+    public function test_non_provider_offer_attempt_creates_no_offer(): void
+    {
+        $member = User::factory()->create(['role' => 'member']);
+
+        $this->assertForbidden($member, 'POST', '/api/provider/offers', $this->validOfferPayload());
+
+        // 越權建立的課程 provider_id 會是 member，出現在公開列表
+        $this->assertDatabaseCount('diving_offers', 0);
+    }
+
+    public function test_unauthenticated_request_is_rejected_on_provider_routes(): void
+    {
+        $this->getJson('/api/provider/profile')->assertStatus(401);
+        $this->getJson('/api/provider/offers')->assertStatus(401);
+    }
+
+    // ── 跨角色共用路由 ───────────────────────────────────────────
+
+    public function test_member_and_provider_can_read_notifications(): void
+    {
+        $member = User::factory()->create(['role' => 'member']);
+        $provider = $this->createProvider();
+
+        $this->actingAs($member)->getJson('/api/notifications')->assertOk();
+        $this->actingAs($provider)->getJson('/api/notifications')->assertOk();
+    }
+
+    public function test_member_and_provider_can_read_unread_message_counts(): void
+    {
+        $member = User::factory()->create(['role' => 'member']);
+        $provider = $this->createProvider();
+
+        $this->actingAs($member)->getJson('/api/bookings/messages/unread-counts')->assertOk();
+        $this->actingAs($provider)->getJson('/api/bookings/messages/unread-counts')->assertOk();
+    }
+}