fix: 編集で任意項目を空にできるようにする

[hduehgw0]

関連 issue

resolve #43

やったこと

概要

編集で任意項目を空にしても既存値を消せなかった問題を、「空欄を明示 null として end-to-end で運ぶ」ことで修正した。原因は「空欄→undefined→JSON.stringify でキー欠落→prisma.update は欠けたキーを更新しない」。

変更点

• スキーマ（src/lib/schemas/bottle.ts）：任意項目（region・subRegion・age・caskType・note）を .optional() → .nullish()（null を受理）。作成の「空→undefined→省略」挙動は不変（null を追加で許すだけ）
• 編集フォーム（edit-bottle-form.tsx）：
  • onSubmit で空欄の任意項目を明示 null に正規化して送信（undefined だと JSON から落ちて消せないため）
  • region に「未選択」項目を追加してクリア可能に。Radix は空文字 SelectItem を禁止するため、センチネル値（"__none__"）→ onValueChange で null に正規化（センチネルは送信データに出さない）。null のときは value ?? NONE で「未選択」項目を選択状態にする（レビュー指摘の UX 整合）
• PATCH ルート：変更なし（bottleSchema が null を受理し parsed.data を updateMany に渡すため、既存コードで null が保存される）
• 作成フォーム（bottle-form.tsx）：スキーマ nullish 化に伴う subRegion の value 型安全化（value={field.value ?? ""}）。1 行・挙動不変

受け入れ条件（セルフチェック）

• 全任意項目（age・region・subRegion・caskType・note）を空にして保存すると、その値が削除される（DB が null になる）
• region を「未選択」に戻せる UI がある
• PATCH が null 入りボディを受理し（400 にしない）、null を保存する
• 回帰テスト：null を送ると updateMany の data に null が渡り、対象が消える（複数の任意項目を null にした代表 1 本）

動作確認

• pnpm lint が通る
• pnpm test が通る（PATCH の null クリア回帰テストを追加。23 本）
• pnpm build が通る
• モバイル幅で表示が崩れない（要ローカル確認）
• 手動：編集で年数・国・地域・樽・メモを空／未選択にして保存 → 消えることを確認（フォーム UI は E2E 領域のためユニット未実施）

Issue 要件外の対応（あれば）

• 作成フォーム bottle-form.tsx の subRegion value を型安全化（1 行）。共有スキーマを nullish にした型的波及への追随で、挙動は変えていない。

備考

• 由来：feat: ボトルを編集する（US-5） #42 のレビュー指摘（discussion_r3631569760）。方針は fix: 編集で任意項目を空にできるようにする #43 起票時に CodeRabbit 承諾済み
• 作成側の region クリア affordance は本 PR では扱わない（refactor: 登録・編集フォームを共通化する（重複解消） #41 のフォーム共通化時に編集側のクリア UI が共有され自然に反映）

Summary by CodeRabbit

• New Features

  • Added support for clearing optional bottle details by submitting empty fields.
  • Updated bottle forms to handle unselected regions and empty sub-regions consistently.

• Bug Fixes

  • Fixed optional values such as age, region, cask type, and notes not being cleared when set to empty.
  • Improved validation for fields explicitly cleared with null.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
mycellar	Ready	Preview, Comment	Jul 23, 2026 8:43am

[coderabbitai]

📝 Walkthrough

Walkthrough

Optional bottle fields now support clearing through explicit null values. The edit form maps empty fields and an unselected region to null, shared validation accepts those values, and API tests verify they reach the Prisma update payload.

Changes

Bottle field clearing

Layer / File(s)	Summary
Nullable bottle input contract src/lib/schemas/bottle.ts	Optional text, region, and age fields accept null or undefined, while empty text continues to normalize to undefined.
Edit-form null clearing flow src/app/bottles/[id]/edit/edit-bottle-form.tsx, src/app/bottles/new/bottle-form.tsx, src/app/api/bottles/[id]/route.test.ts	The edit form sends empty optional fields as null, supports resetting region through a sentinel option, controls subRegion inputs, and verifies null values reach updateMany.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  actor Editor
  participant EditBottleForm
  participant BottleSchema
  participant Prisma
  Editor->>EditBottleForm: Clear optional fields and submit
  EditBottleForm->>EditBottleForm: Convert empty values to null
  EditBottleForm->>BottleSchema: Send PATCH payload
  BottleSchema-->>EditBottleForm: Accept nullish optional fields
  EditBottleForm->>Prisma: Update bottle fields with null

Loading

Possibly related PRs

• hduehgw0/mycellar#42: Introduced the related bottle editing PATCH flow and its route coverage.
• hduehgw0/mycellar#33: Introduced the bottle schema extended by this nullable-field validation change.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The changes satisfy #43 by sending null for optional edit fields, allowing region to be cleared, accepting null in PATCH, and adding a regression test.
Out of Scope Changes check	✅ Passed	No clearly unrelated code changes stand out; the BottleForm type-safety tweak is a small supporting change tied to the schema update.
Title check	✅ Passed	編集フォームで任意項目を空にして保存できる修正を端的に表しており、変更内容と一致しています。
Description check	✅ Passed	関連issue、概要、変更点、セルフチェック、動作確認、備考までテンプレートの必須項目が概ね揃っています。

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/#43/edit-clear-optional

---

_{Comment @coderabbitai help to get the list of available commands.}

[hduehgw0]

Out of Scope Changes check: create-form subRegion value change

ご指摘ありがとうございます。解決策 (b)（スコープに属する理由の記載）で対応します。

bottle-form.tsx の subRegion の value={field.value ?? ""} 追加は、共有スキーマ bottleSchema を nullish() にした（本 #43 の層1）ことの必須の型的帰結です。subRegion の型が string | null | undefined に広がり、React の <Input value> が null を受け付けないため、この 1 行が無いと tsc / build が失敗します。

そのため解決策 (a)（別 PR へ分離）は取れません。型エラーはその原因（スキーマ変更）と不可分で、分離すると本ブランチ単体がコンパイル不能になります。挙動は変えておらず（作成は従来どおり空→undefined→省略）、純粋な型安全化です。

この理由は PR 本文の「Issue 要件外の対応」にも記載済みです。以上より、本 PR のスコープ内の変更として維持します。