【P2】請求書番号の採番競合（レースコンディション）対策

[Yuto299]

概要

createInvoice は「番号を生成 → 請求書を作成」という 2 ステップで、その間がアトミックではありません。

const invoiceNumber =
  validatedData.invoiceNumber || (await generateInvoiceNumber(userId));
// ...
const invoice = await invoiceRepository.create({ ... invoiceNumber ... });

Invoice.invoiceNumber は @unique のため、同時作成でユニーク制約違反となり、ユーザーには原因不明のエラーが表示されます。

補足

現状の @unique はグローバルユニークです。将来的にマルチユーザー化（#51）すると、別ユーザーが同じ番号を使えなくなる設計上の問題もあります。@@unique([userId, invoiceNumber]) が適切かを合わせて検討します。

実装内容

• 採番と作成をトランザクション内で行う（【P0】請求書作成・入金登録のトランザクション化（$transaction 未使用） #68 と同時対応）
• ユニーク制約違反時のリトライ、または採番用シーケンステーブルの導入
• @unique → @@unique([userId, invoiceNumber]) への変更を検討（マイグレーション必要）
• ユーザー向けエラーメッセージの改善

対象ファイル

• src/lib/invoice/invoiceNumberGenerator.ts
• src/application/usecases/invoices/createInvoice.ts
• prisma/schema.prisma

受け入れ条件

• 同時に請求書を作成しても番号が重複しない
• 採番の一意性スコープの方針が docs/design/database.md に記載されている

関連

• 請求書番号の意味 #12（請求書番号の意味、クローズ済み）/ 【P0】請求書作成・入金登録のトランザクション化（$transaction 未使用） #68 / 【P4】複数ユーザー・権限管理（チーム機能） #51