fix(openspec): 新增 fix-booking-exception-handling change 規劃 #74

Merged
a620906209 merged 1 commits from fix/booking-exception-handling into master 2026-10-17 17:12:33 +00:00
Owner

摘要

M7(docs/analysis/2026-10-14-project-review-todo.md)的 change 規劃,只含 OpenSpec 文件,不含程式碼。

  • 問題:MemberBookingController::store、ProviderBookingController::confirm 的 catch (\RuntimeException) 會接住 QueryException,把 SQL 以 422 回給使用者,且不送 Sentry
  • 規劃:新增 BookingRuleViolationException,Controller 只 catch 業務例外(422 與訊息不變);其他例外交給 Laravel 例外處理器(500 {"message":"Server Error"} + Sentry)
  • 規格:booking-lifecycle 新增「預約交易錯誤分類」requirement,5 個 Scenario 皆對應測試任務 2.1–2.5

審查

  • spec-reviewer:PASS(rollback 斷言、500 body、Exceptions::fake()、變異驗證歸類、Risks 等 warnings 已修正)
  • design-hardener:PASS(2.2 測試細節已補進 tasks)
  • openspec validate fix-booking-exception-handling --strict 通過

🤖 Generated with Claude Code

## 摘要 M7(`docs/analysis/2026-10-14-project-review-todo.md`)的 change 規劃,只含 OpenSpec 文件,不含程式碼。 - 問題:`MemberBookingController::store`、`ProviderBookingController::confirm` 的 `catch (\RuntimeException)` 會接住 `QueryException`,把 SQL 以 422 回給使用者,且不送 Sentry - 規劃:新增 `BookingRuleViolationException`,Controller 只 catch 業務例外(422 與訊息不變);其他例外交給 Laravel 例外處理器(500 `{"message":"Server Error"}` + Sentry) - 規格:`booking-lifecycle` 新增「預約交易錯誤分類」requirement,5 個 Scenario 皆對應測試任務 2.1–2.5 ## 審查 - spec-reviewer:PASS(rollback 斷言、500 body、`Exceptions::fake()`、變異驗證歸類、Risks 等 warnings 已修正) - design-hardener:PASS(2.2 測試細節已補進 tasks) - `openspec validate fix-booking-exception-handling --strict` 通過 🤖 Generated with [Claude Code](https://claude.com/claude-code)
a620906209 added 1 commit 2026-10-17 16:59:42 +00:00
fix(openspec): 新增 fix-booking-exception-handling change 規劃
Run Tests / test (pull_request) Successful in 39s
dcbf3e4a9a
- M7:預約交易的 catch (\RuntimeException) 會吞掉 QueryException,把 SQL 以 422 回給使用者且不送 Sentry
- 規劃新增 BookingRuleViolationException,Controller 只 catch 業務例外,其他例外交給 Laravel 處理器(500 + 回報)
- booking-lifecycle 新增「預約交易錯誤分類」requirement(5 個 Scenario 皆對應測試任務)
- spec-reviewer、design-hardener 皆 PASS,warnings 已併入文件

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
a620906209 merged commit 0ee6ce9966 into master 2026-10-17 17:12:33 +00:00
a620906209 deleted branch fix/booking-exception-handling 2026-10-17 17:12:33 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: a620906209/CFDivePlatform#74