refactor: 設定系ストアを汎用ファクトリへ集約し class を廃止

[ansanloms]

概要

設定系データの永続化ストア (server/store/) を、class ベースからクロージャベースの汎用ファクトリへ集約するリファクタ。挙動は変更しない。

動機

KeywordRuleStore と NotificationSettingsStore は「KV 配線 + CRUD」の骨格が重複しており、各 store が #kv の保持と注入を個別に書いていた。骨格を汎用化し、各ドメインは「キー設計・型ガード・正規化」を渡すだけの薄い構成にする。あわせて class を廃止し、依存はクロージャに束ねる。

変更点

• store/kv.ts
  • Kv を class → createKv() クロージャへ。lazy-open 状態 (旧 #kv private) はクロージャ変数に閉じる。
  • 汎用 collectionStore（prefix 配下に複数レコード、各 id + createdAt 持ち）と singletonStore（単一キー）を追加。型 CollectionStore<T, Input> / SingletonStore<T> も export。
  • collection の既定ソートは createdAt 降順 → id（登録の新しい順）。
• store/keyword-rules.ts / store/notification-settings.ts
  • class を廃止し、createKeywordRuleStore(kv) / createNotificationSettingsStore(kv) に。prefix・型ガード（isKeywordRule）・正規化（normalizeNotificationSettings）を渡すだけの薄い factory に縮小。
• main.ts: new X() → createX() の置換のみ。routes はストアのサブセット型（*Like）に構造的に依存するため無変更。
• store/*.test.ts: new X(kv) → createX(kv) に追従。挙動契約は不変。

設計判断

• kv.ts 一本化はしない: collection 型（複数レコード + id）と singleton 型（単一値）は形が異なり、単一 CRUD interface に押し込めると singleton 側に無意味な list/add/remove(id) が生える。2 種のファクトリへ分けた。
• 読み戻し検証は据え置き: keyword-rules はスキーマ駆動の isKeywordRule（= matchesStoredSchema）、notification-settings は旧形状補完を含む normalizeNotificationSettings。汎用 list<T>() の無検証キャストでは KV に残る旧形状・壊れた値を drop できないため、検証はドメイン側に残す。
• id は現状維持: Deno KV はレコードに id を自動採番しない。versionstamp は更新のたびに変わり安定 id にならないため、crypto.randomUUID() による採番をそのまま使う。

確認

• deno task test:server → 92 passed / 0 failed（store/keyword-rules.ts・store/notification-settings.ts は行・分岐 100%）。
• deno check server/main.ts（ファクトリ返り値 → routes の *Like 構造適合）通過。
• deno lint / deno fmt --check（変更ファイル）通過。

🤖 Generated with Claude Code