docs/style: Update Prisma and Svelte component rules, fix TaskList

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Author: KATO-Hiro

.claude/rules/prisma-db.md

@@ -141,6 +141,12 @@ When a migration leaves `finished_at = NULL` in `_prisma_migrations`:
 
 A `--rolled-back` migration is permanently skipped by `migrate deploy`; fixing the original file has no effect.
 
+## Merged Data: Document Uniqueness Loss
+
+Service functions that merge records from multiple sources (e.g., `getMergedTasksMap()` combining base tasks + `ContestTaskPair` entries) lose individual field uniqueness. Document that results contain duplicates in "unique" DB fields.
+
+Example: `getMergedTasksMap()` returns multiple entries with same `task_id` but different `contest_id` — callers must not assume `task_id` is unique. UI layer must use composite keys (see svelte-components.md `{#each}` patterns).
+
 ## Validate Constraints
 
 Prisma does not support `@@check`. To add one:

.claude/rules/svelte-components.md

@@ -37,6 +37,7 @@ Use `$props()`, `$state()`, `$derived()`, `$effect()` in all components:
 ## `{#each}` Patterns
 
 - Always key: `(item.id)` or `(i)`
+- **Key MUST be unique per iteration** — if domain allows duplicates, use composite key (e.g. `contest_id + task_id`)
 - Filter **before**, not inside with `{#if}`
 - Use `{:else}` for empty lists
 
@@ -48,6 +49,8 @@ Use `$props()`, `$state()`, `$derived()`, `$effect()` in all components:
 {/each}
 ```
 
+**Common Trap:** `task_id` alone is NOT unique when same task appears in multiple contests. Use `contest_id + '-' + task_id` as composite key (see issue #3460 & PR #3442).
+
 ## Snippets vs Components
 
 - **Snippet**: needs `$state` access, pure display, same-file; params require explicit type `{#snippet label(item: Item)}`

docs/dev-notes/2026-04-26/fix-grade-accordion-each-key/plan.md

@@ -0,0 +1,69 @@
+# グレード別一覧で展開されないグレードの修正
+
+## 概要
+
+[issue #3460](https://github.com/AtCoder-NoviSteps/AtCoderNoviSteps/issues/3460): 問題一覧のグレード別タブで、一部グレードのアコーディオンが展開できない不具合を修正する。
+
+## 原因調査
+
+### 根本原因
+
+`TaskList.svelte` の `{#each}` キーに `task_id` 単独を使用しているが、`task_id` はユニークではない場合がある。
+
+`getMergedTasksMap()`（`src/lib/services/tasks.ts`）は同一の `task_id` を持ちながら `contest_id` が異なるタスク（同じ問題が複数コンテストに登場するケース）を生成する。`ContestTaskPair` モデルがこのケースを管理している。
+
+```
+task_id: "typical90_s"  contest_id: "typical90"     ← baseTaskMap
+task_id: "typical90_s"  contest_id: "tessoku-book"  ← additionalTaskMap（ContestTaskPairで追加）
+```
+
+この重複した `task_id` を持つ `TaskResult` が同一グレードに含まれると、`{#each}` のキーが重複する。Svelte の keyed `{#each}` は重複キーを正しく reconcile できず、`AccordionItem` の状態が不正になって展開できないグレードが発生する。
+
+### PR #3442 との対比
+
+PR #3442（[#3442](https://github.com/AtCoder-NoviSteps/AtCoderNoviSteps/pull/3442)）は同一構造のバグを修正：
+
+- `TaskListForEdit.svelte` で `importTask.task_id` → `importTask.id`（真のユニークキーへ変更）
+
+今回も同様に、真のユニーク識別子である `contest_id + task_id` の複合キーを使用する必要がある。
+
+### 証拠
+
+- `getMergedTasksMap` のコメント：「Handling cases where the same problem is used in different contests」
+- `TableBodyRow` の `id={taskResult.contest_id + '-' + taskResult.task_id}`：すでに複合キーで DOM id の一意性を確保済み
+
+## 全 `{#each}` キーの調査結果
+
+コードベース全体で `{#each ... (key)}` パターンを持つ箇所を調査した結果：
+
+| ファイル                                             | キー                  | 判定                          |
+| ---------------------------------------------------- | --------------------- | ----------------------------- |
+| `src/lib/components/TaskList.svelte:100`             | `taskResult.task_id`  | **要修正**                    |
+| `src/lib/components/TaskListSorted.svelte:37`        | `taskResult.task_id`  | 安全（Mapで重複排除済み）     |
+| `src/routes/votes/+page.svelte:60`                   | `task.task_id`        | 安全（DB `@unique` 制約あり） |
+| `src/routes/(admin)/vote_management/+page.svelte:39` | `stat.taskId`         | 安全（集計テーブルのPK）      |
+| その他                                               | `id` / enum値 / index | 安全                          |
+
+**`TaskListSorted.svelte`** は `getTaskResultsOnlyResultExists()` が Map（`task_id` キー）で重複排除するため現時点では安全。ただし `getMergedTasksMap` を使うデータソースに変わった場合は壊れる可能性あり。
+
+## 設計の判断
+
+修正対象は `TaskList.svelte` のみ。`TableBodyRow` の `id` 属性が既に `contest_id + '-' + task_id` を使用しているため、同じセパレーターで統一する。
+
+## 修正内容
+
+**対象ファイル:** `src/lib/components/TaskList.svelte` (line 100)
+
+```svelte
+<!-- Before -->
+{#each taskResults as taskResult (taskResult.task_id)}
+
+<!-- After -->
+{#each taskResults as taskResult (taskResult.contest_id + '-' + taskResult.task_id)}
+```
+
+## 検証方法
+
+1. `pnpm test:unit` — ユニットテストの確認
+2. `pnpm dev` でローカルサーバーを起動し `/problems` → グレード別タブで全グレードが展開できることを確認
+3. 複数コンテストに登場する問題を含むグレード（例：典型90問、鉄則本）で特に確認

src/lib/components/TaskList.svelte

@@ -97,7 +97,7 @@
         </TableHead>
 
         <TableBody class="divide-y divide-gray-200 dark:divide-gray-700">
-          {#each taskResults as taskResult (taskResult.task_id)}
+          {#each taskResults as taskResult (taskResult.contest_id + '-' + taskResult.task_id)}
             <TableBodyRow
               id={taskResult.contest_id + '-' + taskResult.task_id}
               class={getBackgroundColorFrom(taskResult.status_name)}