docs: Add lessons and fix path (#3169)

Author: KATO-Hiro

docs/dev-notes/2026-02-14/refactor-for-contest-table/plan.md

@@ -25,29 +25,6 @@
 
 ---
 
-## 検証
-
-1. `pnpm test:unit` でテストが全て通ること
-2. `pnpm check` で型エラーがないこと
-3. `pnpm build` でビルドが通ること
-4. `/problems` ページで TaskTable が正常に表示されること（開発サーバー確認）
-
----
-
-## 教訓
-
-1. **`git mv` でファイルを移動する**: `cp` + `rm` より git 履歴が追跡しやすい。`Write` ツールで新規作成しない。
-
-2. **`$features` エイリアスを先に設定してからインポートを書く**: エイリアス未設定のまま移動ファイルを参照すると型チェックが壊れる。設定ファイル変更 → ファイル移動 → import 修正の順が安全。
-
-3. **共有 test_cases は使用元を grep で確認してから移動判断する**: `contest_name_and_task_index.ts` 等は `contest.test.ts` からも参照されており、フィクスチャに移動すると `src/test/lib` 残存テストが壊れる。
-
-4. **同一ディレクトリ内の相対 import は `./` を使う**: `TaskTable.svelte` → `TaskTableBodyCell.svelte` のように同じディレクトリに移動したコンポーネントは `$features/...` より `./` の方が明確。
-
-5. **`pnpm check` の既存エラーは事前に把握しておく**: リファクタ起因か既存問題かを切り分けるために、作業前の状態を確認しておくとよい。
-
----
-
 # ファイル分割リファクタリング (第二段階)
 
 ## Context
@@ -301,3 +278,30 @@ import { classifyContest } from '$lib/utils/contest'; // no mock needed
 - `pnpm lint` — ESLint エラーなし
 - `pnpm build` — ビルド成功
 - ブラウザで `/problems` ページの表示確認
+
+## 実装から得た教訓
+
+1. **`git mv` でファイルを移動する**: `cp` + `rm` より git 履歴が追跡しやすい。`Write` ツールで新規作成しない。
+
+2. **`$features` エイリアスを先に設定してからインポートを書く**: エイリアス未設定のまま移動ファイルを参照すると型チェックが壊れる。設定ファイル変更 → ファイル移動 → import 修正の順が安全。
+
+3. **共有 test_cases は使用元を grep で確認してから移動判断する**: `contest_name_and_task_index.ts` 等は `contest.test.ts` からも参照されており、フィクスチャに移動すると `src/test/lib` 残存テストが壊れる。
+
+4. **同一ディレクトリ内の相対 import は `./` を使う**: `TaskTable.svelte` → `TaskTableBodyCell.svelte` のように同じディレクトリに移動したコンポーネントは `$features/...` より `./` の方が明確。
+
+5. **`pnpm check` の既存エラーは事前に把握しておく**: リファクタ起因か既存問題かを切り分けるために、作業前の状態を確認しておくとよい。
+
+6. vi.mock 廃止で発見されたバグ
+
+旧テストでは `classifyContest` や `getContestNameLabel` を mock していたため、実装との乖離が隠れていた。mock を外して直接 import に切り替えたことで以下が発覚：
+
+- **Tessoku sub-provider のメタデータ不一致**: テストでは `"力試し問題"` 等を期待していたが、実装は `"C. 力試し問題"` のように接頭辞付き。`abbreviationName` も `tessoku-book-examples` ではなく `tessoku-book-for-examples`（`for-` 接頭辞）が正しかった。
+- **JOI の無効 ID ラベル**: mock では小文字パススルーを期待、実装は `toUpperCase()` で `'INVALID-ID'` を返す。部分一致するパターン（`joi2024yo1d`）では想定外の `'2024d'` が返る。
+
+→ **mock は実装との一致を保証しない。純粋関数は直接テストすべき。**
+
+### 分割の実用的な効果
+
+- 1ファイル 100〜300 行に抑えられ、変更時の影響範囲が明確になった
+- テスト実行時間が短縮（vitest の並列実行が効く）
+- 新規 provider 追加時に既存ファイルを触る必要がほぼなくなった

docs/guides/how-to-add-contest-table-provider.md

@@ -71,11 +71,11 @@
 2. **テスト実装フェーズ**
    - Vitest でテストコードを記述
    - Provider クラスをスケルトン実装（下記参照）
-   - `src/lib/utils/contest_table_provider.ts` の `prepareContestProviderPresets()` と `contestTableProviderGroups` に登録
+   - `src/features/tasks/utils/contest-table/contest_table_provider_groups.ts` の `prepareContestProviderPresets()` と `contestTableProviderGroups` に登録
 
 3. **Provider 実装フェーズ**
    - テストが RED → GREEN になるまで段階的に実装
-   - 実行: `pnpm test:unit src/test/lib/utils/contest_table_provider.test.ts`
+   - 実行: `pnpm test:unit src/features/tasks/utils/contest-table/`
 
 ### スケルトン Provider の最小例
 
@@ -105,7 +105,7 @@ export class MyNewProvider extends ContestTableProviderBase {
 
 ### Provider 登録（テスト実行環境構築）
 
-**ファイル**: `src/lib/utils/contest_table_provider.ts`
+**ファイル**: `src/features/tasks/utils/contest-table/contest_table_provider_groups.ts`
 
 ```typescript
 function prepareContestProviderPresets() {
@@ -124,7 +124,7 @@ export const contestTableProviderGroups: Record<ProviderKey, ContestTableProvide
 
 **重要**: これらを登録しないとテストが実行できません。
 
-詳細な実装例は [Provider実装ファイル](../src/lib/utils/contest_table_provider.ts) を参照してください。
+詳細な実装例は [Provider実装ディレクトリ](../src/features/tasks/utils/contest-table/) を参照してください。
 
 ---
 
@@ -152,7 +152,7 @@ protected setFilterCondition(): (taskResult: TaskResult) => boolean {
 }
 ```
 
-その他の実装（`getMetadata()`、`getDisplayConfig()` など）は [Provider実装ファイル](../src/lib/utils/contest_table_provider.ts) を参照してください。
+その他の実装（`getMetadata()`、`getDisplayConfig()` など）は [Provider実装ディレクトリ](../src/features/tasks/utils/contest-table/) を参照してください。
 
 **注意**: ARC、AGC も同じパターン。範囲フィルタ型を参照してください。
 
@@ -180,7 +180,7 @@ protected setFilterCondition(): (taskResult: TaskResult) => boolean {
 }
 ```
 
-その他の実装は [Provider実装ファイル](../src/lib/utils/contest_table_provider.ts) を参照してください。
+その他の実装は [Provider実装ディレクトリ](../src/features/tasks/utils/contest-table/) を参照してください。
 
 **注意**: TDPC、FPS_24 も同じパターン。`contest_id` とメタデータだけ異なります。
 
@@ -232,7 +232,7 @@ class TessokuBookSectionProvider extends TessokuBookProvider {
 }
 ```
 
-その他の実装は [Provider実装ファイル](../src/lib/utils/contest_table_provider.ts) を参照してください。
+その他の実装は [Provider実装ディレクトリ](../src/features/tasks/utils/contest-table/) を参照してください。
 
 ---
 
@@ -373,13 +373,13 @@ Provider クラスのテスト完了後、`prepareContestProviderPresets()` か
 
 **セクション識別子の指定**:
 
-複数プロバイダーを含むグループでは、`getProvider()` の第2引数にセクション識別子を渡します。セクション定数は `src/lib/types/contest_table_provider.ts` で定義されています：
+複数プロバイダーを含むグループでは、`getProvider()` の第2引数にセクション識別子を渡します。セクション定数は `src/features/tasks/types/contest-table/contest_table_provider.ts` で定義されています：
 
 - `TESSOKU_SECTIONS`: `examples`、`practicals`、`challenges`
 - `JOI_SECOND_QUAL_ROUND_SECTIONS`: `'2020Onwards'`、`from2006To2019`
 - `JOI_FINAL_ROUND_SECTIONS`: `semiFinal`
 
-詳細な実装例は [単体テストファイル](../src/test/lib/utils/contest_table_provider.test.ts) を参照してください。
+詳細な実装例は [単体テストディレクトリ](../src/features/tasks/utils/contest-table/) の各 `*.test.ts` ファイルを参照してください。
 
 ---
 
@@ -534,9 +534,9 @@ describe('CustomProvider with unique config', () => {
 
 ### 実装ファイル
 
-- [Provider 実装](../src/lib/utils/contest_table_provider.ts)
-- [単体テスト](../src/test/lib/utils/contest_table_provider.test.ts)
-- [モックデータ](../src/test/lib/utils/test_cases/contest_table_provider.ts)
+- [Provider 実装ディレクトリ](../src/features/tasks/utils/contest-table/)
+- [単体テスト](../src/features/tasks/utils/contest-table/) (各 `*.test.ts` ファイル)
+- [テストフィクスチャ](../src/features/tasks/fixtures/contest-table/contest_table_provider.ts)
 
 ---