From fc890ab2bdc4d059cedc458a99d47802e3f53e9e Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Mon, 2 Nov 2026 23:34:36 +0000
Subject: [PATCH 01/18] fix: read display-only page data and props through
 $derived

Plain `let x = data.x` copies only the initial value, so a reused page
component would keep showing stale data after a same-route navigation.
Clears 14 state_referenced_locally warnings.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 src/lib/components/InputFieldWrapper.svelte     | 2 +-
 src/routes/(admin)/tags/+page.svelte            | 2 +-
 src/routes/(admin)/tags/[tag_id]/+page.svelte   | 4 ++--
 src/routes/(admin)/tasks/[task_id]/+page.svelte | 4 ++--
 src/routes/problems/[slug]/+page.svelte         | 4 ++--
 src/routes/users/[username]/+page.svelte        | 8 ++++----
 src/routes/workbooks/[slug]/+page.svelte        | 4 ++--
 7 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/lib/components/InputFieldWrapper.svelte b/src/lib/components/InputFieldWrapper.svelte
index b45058253..9b5b7049d 100644
--- a/src/lib/components/InputFieldWrapper.svelte
+++ b/src/lib/components/InputFieldWrapper.svelte
@@ -26,7 +26,7 @@
     message = '',
   }: Props = $props();
 
-  let isReadOnly = !isEditable;
+  let isReadOnly = $derived(!isEditable);
 </script>
 
 <Label class="flex flex-col gap-2">
diff --git a/src/routes/(admin)/tags/+page.svelte b/src/routes/(admin)/tags/+page.svelte
index e22807c60..f99fcc7d3 100644
--- a/src/routes/(admin)/tags/+page.svelte
+++ b/src/routes/(admin)/tags/+page.svelte
@@ -3,7 +3,7 @@
   import TagListForEdit from '#lib/components/TagListForEdit.svelte';
 
   let { data } = $props();
-  let tags = data.tags;
+  let tags = $derived(data.tags);
 </script>
 
 <div class="container mx-auto w-5/6">
diff --git a/src/routes/(admin)/tags/[tag_id]/+page.svelte b/src/routes/(admin)/tags/[tag_id]/+page.svelte
index ddf77522c..baaf4a1a4 100644
--- a/src/routes/(admin)/tags/[tag_id]/+page.svelte
+++ b/src/routes/(admin)/tags/[tag_id]/+page.svelte
@@ -3,8 +3,8 @@
 
   let { data } = $props();
 
-  let tag = data.tag;
-  let tasks = data.tasks;
+  let tag = $derived(data.tag);
+  let tasks = $derived(data.tasks);
 </script>
 
 <div class="container mx-auto w-3/4">
diff --git a/src/routes/(admin)/tasks/[task_id]/+page.svelte b/src/routes/(admin)/tasks/[task_id]/+page.svelte
index fd269c19e..15bdb2c30 100644
--- a/src/routes/(admin)/tasks/[task_id]/+page.svelte
+++ b/src/routes/(admin)/tasks/[task_id]/+page.svelte
@@ -5,8 +5,8 @@
 
   let { data } = $props();
 
-  let task = data.task;
-  let tags = data.tags;
+  let task = $derived(data.task);
+  let tags = $derived(data.tags);
 </script>
 
 <div class="container mx-auto w-5/6 md:w-1/2">
diff --git a/src/routes/problems/[slug]/+page.svelte b/src/routes/problems/[slug]/+page.svelte
index 7e6cc0578..5fc16ed98 100644
--- a/src/routes/problems/[slug]/+page.svelte
+++ b/src/routes/problems/[slug]/+page.svelte
@@ -12,8 +12,8 @@
 
   let { data } = $props();
 
-  let taskResult = data.taskResult;
-  let buttons = data.buttons;
+  let taskResult = $derived(data.taskResult);
+  let buttons = $derived(data.buttons);
 </script>
 
 <!-- TODO: 回答状況は、一覧表のリストやテーブルから更新する -->
diff --git a/src/routes/users/[username]/+page.svelte b/src/routes/users/[username]/+page.svelte
index f6524b6ac..bfda211b7 100644
--- a/src/routes/users/[username]/+page.svelte
+++ b/src/routes/users/[username]/+page.svelte
@@ -8,10 +8,10 @@
 
   let { data } = $props();
 
-  let username = data.username;
-  let atcoder_username = data.atcoder_username;
-  let isLoggedIn = data.isLoggedIn;
-  let taskResults = data.taskResults;
+  let username = $derived(data.username);
+  let atcoder_username = $derived(data.atcoder_username);
+  let isLoggedIn = $derived(data.isLoggedIn);
+  let taskResults = $derived(data.taskResults);
 
   const problemsHref = resolve(PROBLEMS_PAGE);
 </script>
diff --git a/src/routes/workbooks/[slug]/+page.svelte b/src/routes/workbooks/[slug]/+page.svelte
index dd6b36fd4..213d06d86 100644
--- a/src/routes/workbooks/[slug]/+page.svelte
+++ b/src/routes/workbooks/[slug]/+page.svelte
@@ -28,12 +28,12 @@
 
   let { data } = $props();
 
-  let workBook = data.workBook;
+  let workBook = $derived(data.workBook);
   let workBookTasks: WorkBookTaskBase[] = $state([]);
   let taskResults: Map<string, TaskResult> = $derived(data.taskResults);
   let voteStatisticsMap = $derived(data.voteStatisticsMap);
 
-  let isLoggedIn = data.isLoggedIn;
+  let isLoggedIn = $derived(data.isLoggedIn);
   let isAtCoderVerified = $derived(data.isAtCoderVerified);
 
   // TODO: 関数をutilへ移動させる

From cdcd6f8f0c8b084ae48b1eb159bd6466d609b14c Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Mon, 2 Nov 2026 23:35:06 +0000
Subject: [PATCH 02/18] docs: add the plan for fixing build, check, and lint
 warnings

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 .../2026-11-01/fix-build-warnings/plan.md     | 80 +++++++++++++++++++
 1 file changed, 80 insertions(+)
 create mode 100644 docs/dev-notes/2026-11-01/fix-build-warnings/plan.md

diff --git a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
new file mode 100644
index 000000000..6bb39fe93
--- /dev/null
+++ b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
@@ -0,0 +1,80 @@
+# build・check・lint の警告の解消
+
+## 概要
+
+`pnpm build`・`pnpm check`・`pnpm lint` の警告31件を0件にし、CI で再発を止める（2026-11-01、staging `3eb7eadd`）。
+
+| 警告                                    | 出どころ                        | 件数 |
+| --------------------------------------- | ------------------------------- | ---- |
+| `state_referenced_locally`              | Svelte（build と `pnpm check`） | 29   |
+| `a11y_no_noninteractive_tabindex`       | 同上                            | 1    |
+| `unicorn/no-useless-fallback-in-spread` | oxlint（`pnpm lint`）           | 1    |
+
+- build は SSR と client で2回 compile するので Svelte の警告は60個表示されるが、ユニークなのは30件で `pnpm check` と一致する。`pnpm format` の警告は0件。
+- 警告を抑える設定は無い。svelte-check と oxlint は既定では警告があっても exit 0 を返すので、CI が通っている。
+- error にする予定は見当たらない。`state_referenced_locally` は Svelte 5.45.3 から props にも出るようになり、メンテナーは「props の初期値を使うのはほとんどの場合バグ」として戻さない（[sveltejs/svelte#17289](https://github.com/sveltejs/svelte/issues/17289)）。実害は、同じ route の中で移動すると SvelteKit がコンポーネントを再利用し、`let x = data.x` が古い値のまま残ること（[SvelteKit docs](https://svelte.dev/docs/kit/state-management#Component-and-page-state-is-preserved)）。今はそうした移動をするリンクが無く、潜在的なバグにとどまる。
+- svelte.dev などに devcontainer から接続できなかったので、docs は GitHub 上の source で、Superforms は issue で確かめた。
+- 範囲外: chunk size の警告（末尾）、`(admin)/account_transfer` の `svelte/legacy` の `run`。`pnpm dev`・`test:unit`・`test:e2e` の出力は見ていない。
+
+## 方針
+
+- 値の変化に追従してよい所は `$derived` に、初期値だけを使う意図の所は `untrack(() => ...)` にする。迷う所は、今の挙動を保つ `untrack` にする。前者は SvelteKit の docs と `svelte-runes.md` に、後者は Superforms の docs（[superforms#667](https://github.com/ciscoheat/sveltekit-superforms/issues/667)、`superForm(untrack(() => data.form))`）と `AtCoderVerificationForm.svelte:23-24` に揃える。
+- 挙動の分岐を足さないので test-first は省く。各 phase で `pnpm check` の残りの件数と既存の E2E を確かめてから commit する。手で変えるのは18ファイルで、クロスレビューの条件には当たらない。
+
+## フェーズ
+
+### Phase 1: oxlint（1件）
+
+- [x] `src/test/lib/utils/auth_forms.test.ts:222-224` の stub を `vi.stubGlobal('crypto', { randomUUID: undefined })` にし、`prevCrypto` を消して、コメントを実態に合わせる。`?? {}` は、object の spread が null・undefined を無視するので要らない。`prevCrypto` は `beforeEach` の stub（`randomUUID` だけを持つ object）なので、元のコードも `{ randomUUID: undefined }` を作っていた。コメントの「preserve other properties」は実態と合っていなかった。
+- [x] 確認: `pnpm exec oxlint .` が0件になり、`auth_forms.test.ts` が通る。
+
+### Phase 2: 表示にだけ使う値を `$derived` にする（14件、残り16件）
+
+- [ ] `InputFieldWrapper.svelte:29`・`(admin)/tags/+page.svelte:6`・`(admin)/tags/[tag_id]/+page.svelte:6-7`・`(admin)/tasks/[task_id]/+page.svelte:8-9`・`problems/[slug]/+page.svelte:15-16`・`users/[username]/+page.svelte:11-14`・`workbooks/[slug]/+page.svelte:31,36` を、`let tags = $derived(data.tags);` の形にする。
+- [ ] `workbooks/[slug]` の `$effect` は、今も `taskResults` を通じて `data` が変わるたびに再実行されている。`workBook` を `$derived` にしても、表示は変わらない見込み。
+- [ ] 確認: E2E の `submission_status`・`workbooks_list`・`redirect_after_login` が通り、上の6画面を開いて表示が変わっていない。
+
+### Phase 3: コンポーネントの props（5件、残り11件）
+
+- [ ] `KanbanColumn.svelte:21,23`: `type`・`accept` も `id` と同じく getter にする。`@dnd-kit/svelte` の `createDroppable` は、getter の値を `$effect` の中で読み直す。
+- [ ] `VotableGrade.svelte:63`: `componentId` を `$derived` にする。
+- [ ] `VotableGrade.svelte:59`: 投票後の表示に使う state の初期値なので、`untrack` で包む（同じファイルの78行目と同じ形）。
+- [ ] 確認: E2E の `workbook_order`・`votes`・`workbooks_list` が通る。
+
+### Phase 4: a11y（1件、残り10件）
+
+- [ ] `votes/+page.svelte:69` の `<span role="img" tabindex="0">` を `<button type="button" aria-label="暫定グレード">` にする。`id`・`class` はそのまま残す。flowbite-svelte の `Tooltip` はトリガーの `focusin` で開くので、この `tabindex` はキーボードで Tooltip を開くためのもの。`<button>` にしても、フォーカスで開く挙動は変わらない。
+- [ ] 確認: Tab でアイコンにフォーカスすると Tooltip が開き、見た目（cursor・色・位置）が変わらない。E2E の `votes` が通る。
+
+### Phase 5: Superforms の初期値（10件、残り0件）
+
+- [ ] `untrack` で包む: `(admin)/account_transfer/+page.svelte:31`、`AuthForm.svelte:43`・`WorkBookForm.svelte:41`（受け取った superForm の object を分割代入する所）、`workbooks/create/+page.svelte:14,22,23`、`workbooks/edit/[slug]/+page.svelte:21`。
+- [ ] `$derived` にする: `workbooks/create/+page.svelte:26`、`workbooks/edit/[slug]/+page.svelte:16,29`。
+- [ ] 確認: E2E の `account_transfer`・`signin`・`redirect_after_login`・`workbook_edit`・`error_pages` が通る。問題集を作ると `authorId`・`isOfficial`・`workBookType` が今と同じ値で保存され、編集画面に今の値が入っている。
+
+### Phase 6: CI で警告を止める
+
+- [ ] `package.json` の `check` に `--fail-on-warnings` を、`lint` の oxlint に `--deny-warnings` を付ける。`check:watch` は変えない。`lefthook.yml` の oxlint の job にも `--deny-warnings` を付ける。
+- [ ] 確認: どちらも exit 0 で終わる。試しに `let x = data.x` を1行足すと `pnpm check` が exit 1 になる（`rtk bypass` で確かめ、元に戻す）。
+- [ ] これ以後、`no-explicit-any`（`warn`）も disable コメントが無ければ CI で失敗する。今ある `any` には、すべて disable コメントが付いている。
+
+## 採らなかった案
+
+- **`warningFilter`・`svelte-ignore` で消す:** 本物のバグも隠れてしまう。意図的な所も、`untrack` の方が意図をコードで表せる。
+- **Superforms に渡す値を `$derived` にする:** `superForm` は初期化した後、自分で state を持つ。`AuthForm` は呼び出し側が template の式で `superForm(data?.form)` を渡すので、子で読み直すと初期化の外で `superForm()` が呼ばれうる。
+- **`VotableGrade` の `displayGrade` を上書きできる `$derived` にする:** 親が新しい `taskResult` を渡すと、投票した直後の表示が元に戻る。挙動の変更を chore に混ぜない。
+- **`{#key}` でコンポーネントを作り直す:** `$derived` で足りる。
+- **a11y の警告を `tabindex` を消して直す:** キーボードで Tooltip を開けなくなる。
+- **ESLint にも `--max-warnings 0` を付ける:** 今の警告は0件で、今回の警告の出どころでもない。
+
+## 範囲外: chunk size の警告
+
+- devcontainer でだけ出る。`compose.yaml` の `NODE_ENV=development` のため Vite が dev モードで build し、Svelte が dev モードでだけ出す `Component[FILENAME] = ...` を Rolldown が副作用として扱う。その結果、flowbite-svelte の使っていないコンポーネント（135個、約 158 kB）が残る。本番モードでは最大の chunk は 301 kB で、警告は出ない。
+- flowbite-svelte を Rolldown の `treeshake.moduleSideEffects` で副作用なしと宣言しても、本番では 75 kB（約9%）しか減らない。upstream が止まっているので、この設定はずっと残り、flowbite-svelte を上げるたびに確認が要る。
+- deep import（`flowbite-svelte/Button.svelte`）にしても、`Button.svelte` 自体がルートの barrel を読み込むので縮まない。`chunkSizeWarningLimit` を上げるのは、本番では出ない警告のために本番の設定を変えることになる。
+
+## 残る作業
+
+- devcontainer の `pnpm build` と E2E（Playwright の `webServer`）は dev モードのまま。本番と揃えるなら、`compose.yaml` の `NODE_ENV=development` を消す。`NODE_ENV` を読むコードは無いが、devcontainer の作り直しが要る。
+- `.coderabbit.yaml` の `src/**/*utils/**` の指示（「Must be pure functions with no side effects」）はテストファイルにも当たるので、`auth_forms.test.ts` の `vi.stubGlobal` を副作用として指摘された（Phase 1 の review）。直すなら、指示の文面に「本番コード向けで、テストでは `afterEach` で戻すなら `vi.stubGlobal` を使ってよい」と1行足す。この glob に当たるテストは63ファイルあるが、global を stub しているのはこのファイルだけ。`path_instructions` の glob が `!` による除外に対応しているかは確かめていない。
+- UI の3層構造（Tailwind CSS → Flowbite → flowbite-svelte）の見直しは、別の task・PR にする。flowbite-svelte は2026-04-07から release も merge も無く、66ファイルが46コンポーネントを import している。flowbite は3.1.2のまま（最新は v4.0.2）。

From 52c1cf004087c73e7688ba07e039ec3e8d22f21e Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Mon, 2 Nov 2026 23:45:13 +0000
Subject: [PATCH 03/18] fix: keep dnd-kit options and component IDs in sync
 with props

Pass type/accept to createDroppable as getters like id, so its internal
effect re-reads them. Derive componentId, and mark the displayGrade seed
as an intentional initial value with untrack.
Clears 5 state_referenced_locally warnings.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 src/features/votes/components/VotableGrade.svelte         | 7 ++++---
 .../workbooks/order/_components/KanbanColumn.svelte       | 8 ++++++--
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/src/features/votes/components/VotableGrade.svelte b/src/features/votes/components/VotableGrade.svelte
index d721ae804..ed08a00bf 100644
--- a/src/features/votes/components/VotableGrade.svelte
+++ b/src/features/votes/components/VotableGrade.svelte
@@ -56,11 +56,12 @@
   // 表示用のグレード（投票後に画面リロードなしで差し替えるためのローカル状態）
   // PENDING かつ estimatedGrade（集計済み中央値）があればそれを優先表示。
   // DBグレード付与済みの場合はそちらを優先。
-  const initialGrade = resolveDisplayGrade(taskResult.grade, estimatedGrade);
-  let displayGrade = $state<TaskGrade | string>(initialGrade);
+  let displayGrade = $state<TaskGrade | string>(
+    untrack(() => resolveDisplayGrade(taskResult.grade, estimatedGrade)),
+  );
 
   // Use task_id as a deterministic component ID to avoid SSR/hydration mismatches.
-  const componentId = taskResult.task_id;
+  const componentId = $derived(taskResult.task_id);
 
   const editProfileHref = `${resolve(EDIT_PROFILE_PAGE)}?tab=atcoder`;
 
diff --git a/src/routes/(admin)/workbooks/order/_components/KanbanColumn.svelte b/src/routes/(admin)/workbooks/order/_components/KanbanColumn.svelte
index 4d642cf55..f9ff73ddb 100644
--- a/src/routes/(admin)/workbooks/order/_components/KanbanColumn.svelte
+++ b/src/routes/(admin)/workbooks/order/_components/KanbanColumn.svelte
@@ -18,9 +18,13 @@
     get id() {
       return columnId;
     },
-    type: group,
+    get type() {
+      return group;
+    },
     collisionPriority: 1, // CollisionPriority.Low
-    accept: group,
+    get accept() {
+      return group;
+    },
   });
 
   function attachDroppable(node: HTMLElement): { destroy: () => void } {

From a30a21c1247f5e4f2192ec0aceba37de39a7eaf8 Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Mon, 2 Nov 2026 23:45:43 +0000
Subject: [PATCH 04/18] docs: record the results of phases 2 and 3 for fixing
 build warnings

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 .../2026-11-01/fix-build-warnings/plan.md          | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
index 6bb39fe93..37502cafb 100644
--- a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
+++ b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
@@ -30,16 +30,16 @@
 
 ### Phase 2: 表示にだけ使う値を `$derived` にする（14件、残り16件）
 
-- [ ] `InputFieldWrapper.svelte:29`・`(admin)/tags/+page.svelte:6`・`(admin)/tags/[tag_id]/+page.svelte:6-7`・`(admin)/tasks/[task_id]/+page.svelte:8-9`・`problems/[slug]/+page.svelte:15-16`・`users/[username]/+page.svelte:11-14`・`workbooks/[slug]/+page.svelte:31,36` を、`let tags = $derived(data.tags);` の形にする。
-- [ ] `workbooks/[slug]` の `$effect` は、今も `taskResults` を通じて `data` が変わるたびに再実行されている。`workBook` を `$derived` にしても、表示は変わらない見込み。
-- [ ] 確認: E2E の `submission_status`・`workbooks_list`・`redirect_after_login` が通り、上の6画面を開いて表示が変わっていない。
+- [x] `InputFieldWrapper.svelte:29`・`(admin)/tags/+page.svelte:6`・`(admin)/tags/[tag_id]/+page.svelte:6-7`・`(admin)/tasks/[task_id]/+page.svelte:8-9`・`problems/[slug]/+page.svelte:15-16`・`users/[username]/+page.svelte:11-14`・`workbooks/[slug]/+page.svelte:31,36` を、`let tags = $derived(data.tags);` の形にする。
+- [x] `workbooks/[slug]` の `$effect` は、今も `taskResults` を通じて `data` が変わるたびに再実行されている。`workBook` を `$derived` にしても、表示は変わらない見込み。
+- [x] 確認: E2E の `submission_status`・`workbooks_list`・`redirect_after_login` が通り、上の6画面を開いて表示が変わっていない。結果: `pnpm check` は30件から16件（Phase 2 の7ファイルは0件）。E2E は50件 pass・1件 skip（データが無いときの条件付き skip）。admin で6画面と `/workbooks/create` の本文と input（`readonly`・value）を変更の前後で書き出し、差分は無かった。
 
 ### Phase 3: コンポーネントの props（5件、残り11件）
 
-- [ ] `KanbanColumn.svelte:21,23`: `type`・`accept` も `id` と同じく getter にする。`@dnd-kit/svelte` の `createDroppable` は、getter の値を `$effect` の中で読み直す。
-- [ ] `VotableGrade.svelte:63`: `componentId` を `$derived` にする。
-- [ ] `VotableGrade.svelte:59`: 投票後の表示に使う state の初期値なので、`untrack` で包む（同じファイルの78行目と同じ形）。
-- [ ] 確認: E2E の `workbook_order`・`votes`・`workbooks_list` が通る。
+- [x] `KanbanColumn.svelte:21,23`: `type`・`accept` も `id` と同じく getter にする。`@dnd-kit/svelte` の `createDroppable` は、getter の値を `$effect` の中で読み直す。
+- [x] `VotableGrade.svelte:63`: `componentId` を `$derived` にする。
+- [x] `VotableGrade.svelte:59`: 投票後の表示に使う state の初期値なので、`untrack` で包む（同じファイルの78行目と同じ形）。
+- [x] 確認: E2E の `workbook_order`・`votes`・`workbooks_list` が通る。結果: `pnpm check` は16件から11件。E2E は54件のうち53件 pass・1件 skip（Phase 2 と同じ条件付き skip）。`VotableGrade:59` は、78行目に揃えて `initialGrade` を消し、`$state(untrack(() => ...))` にまとめた。
 
 ### Phase 4: a11y（1件、残り10件）
 

From 982cf47b178233868d6f948b28b9a0951fc14fba Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Mon, 2 Nov 2026 23:51:56 +0000
Subject: [PATCH 05/18] fix: drop the redundant tabindex on the
 provisional-grade icon

flowbite-svelte's Tooltip already sets tabIndex=0 on its trigger at
runtime (Popper.svelte), so keyboard users can still focus the icon
and open the tooltip. Clears the a11y_no_noninteractive_tabindex warning.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 src/routes/votes/+page.svelte | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/routes/votes/+page.svelte b/src/routes/votes/+page.svelte
index a78acea3f..be97efcbc 100644
--- a/src/routes/votes/+page.svelte
+++ b/src/routes/votes/+page.svelte
@@ -69,7 +69,6 @@
                     <span
                       id="flask-{task.task_id}"
                       class="cursor-help text-gray-500 dark:text-gray-400"
-                      tabindex="0"
                       role="img"
                       aria-label="暫定グレード"
                     >

From dfeee10bc13e883d2b8620a3528a540ce52f64cd Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Mon, 2 Nov 2026 23:51:59 +0000
Subject: [PATCH 06/18] docs: revise the a11y fix in the plan and record the
 phase 4 result

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 docs/dev-notes/2026-11-01/fix-build-warnings/plan.md | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
index 37502cafb..09a944c86 100644
--- a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
+++ b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
@@ -43,8 +43,9 @@
 
 ### Phase 4: a11y（1件、残り10件）
 
-- [ ] `votes/+page.svelte:69` の `<span role="img" tabindex="0">` を `<button type="button" aria-label="暫定グレード">` にする。`id`・`class` はそのまま残す。flowbite-svelte の `Tooltip` はトリガーの `focusin` で開くので、この `tabindex` はキーボードで Tooltip を開くためのもの。`<button>` にしても、フォーカスで開く挙動は変わらない。
-- [ ] 確認: Tab でアイコンにフォーカスすると Tooltip が開き、見た目（cursor・色・位置）が変わらない。E2E の `votes` が通る。
+- [x] `votes/+page.svelte:69` の `<span role="img">` から `tabindex="0"` を消す。flowbite-svelte 1.33.1 の `Popper.svelte:201` が、`Tooltip` のトリガーに実行時に `tabIndex = 0` を付けるので、markup の `tabindex` は重複している。`VotableGrade.svelte` の同じフラスコのアイコンも、`tabindex` を書かずにトリガーにしている。
+- [x] 実行時の DOM は今と同じく「フォーカスできる `role="img"`」のままで、a11y の状態は良くも悪くもならない。キーボードで Tooltip の説明を読むには、フォーカスできる必要がある。
+- [x] 確認: Tab でアイコンにフォーカスすると Tooltip が開き、見た目（cursor・色・位置）が変わらない。E2E の `votes` が通る。結果: `pnpm check` は11件から10件。E2E の `votes` は15件 pass。dev の DB に暫定グレードの問題が無いので、Playwright で `/votes/__data.json` を書き換えて PENDING の問題1件に `estimatedGrade` を足し、アイコンを出して確かめた。dev server のコンパイル結果に `tabindex` は無いが、実行時の要素には `tabindex="0"` が付き、Tab でフォーカスすると Tooltip が開いて、フォーカスが離れると閉じた。`cursor: help` と色も変わらない。E2E にはこのアイコンのテストが無い。
 
 ### Phase 5: Superforms の初期値（10件、残り0件）
 
@@ -64,7 +65,7 @@
 - **Superforms に渡す値を `$derived` にする:** `superForm` は初期化した後、自分で state を持つ。`AuthForm` は呼び出し側が template の式で `superForm(data?.form)` を渡すので、子で読み直すと初期化の外で `superForm()` が呼ばれうる。
 - **`VotableGrade` の `displayGrade` を上書きできる `$derived` にする:** 親が新しい `taskResult` を渡すと、投票した直後の表示が元に戻る。挙動の変更を chore に混ぜない。
 - **`{#key}` でコンポーネントを作り直す:** `$derived` で足りる。
-- **a11y の警告を `tabindex` を消して直す:** キーボードで Tooltip を開けなくなる。
+- **a11y の警告を、アイコンを `<button>` にして直す:** 押しても何も起きないのに、スクリーンリーダーに「ボタン」と読まれる。flowbite-svelte がトリガーにフォーカスを付けるので、`<button>` にする必要も無い（当初はこの案だったが、`Popper.svelte` を読んで変えた）。
 - **ESLint にも `--max-warnings 0` を付ける:** 今の警告は0件で、今回の警告の出どころでもない。
 
 ## 範囲外: chunk size の警告

From 7e3769e594ad7b51f21a87ca7cb76f3afd23b8fd Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:00:49 +0000
Subject: [PATCH 07/18] fix: seed Superforms from load data once with untrack

superForm owns the form state after init, so its initial data and the
destructured superForm objects are read once on purpose; untrack makes
that explicit. Lookup-only data (workBook, tasksMapByIds) is derived.
Clears the last 10 state_referenced_locally warnings.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 .../workbooks/components/shared/WorkBookForm.svelte  |  4 +++-
 src/lib/components/AuthForm.svelte                   |  4 ++--
 src/routes/(admin)/account_transfer/+page.svelte     |  4 ++--
 src/routes/workbooks/create/+page.svelte             | 12 +++++++-----
 src/routes/workbooks/edit/[slug]/+page.svelte        | 10 ++++++----
 5 files changed, 20 insertions(+), 14 deletions(-)

diff --git a/src/features/workbooks/components/shared/WorkBookForm.svelte b/src/features/workbooks/components/shared/WorkBookForm.svelte
index e0639a2d8..1553d14a0 100644
--- a/src/features/workbooks/components/shared/WorkBookForm.svelte
+++ b/src/features/workbooks/components/shared/WorkBookForm.svelte
@@ -1,4 +1,6 @@
 <script lang="ts">
+  import { untrack } from 'svelte';
+
   import { Breadcrumb, BreadcrumbItem } from 'flowbite-svelte';
 
   import HeadingOne from '#lib/components/HeadingOne.svelte';
@@ -38,7 +40,7 @@
     submitButtonLabel = '',
   }: Props = $props();
 
-  const { form, message, errors, enhance } = superFormObject;
+  const { form, message, errors, enhance } = untrack(() => superFormObject);
 
   let workBookTasksForTable: WorkBookTasksCreate | WorkBookTasksEdit = $derived(
     $form.workBookTasks
diff --git a/src/lib/components/AuthForm.svelte b/src/lib/components/AuthForm.svelte
index 977f5c363..8dcd69014 100644
--- a/src/lib/components/AuthForm.svelte
+++ b/src/lib/components/AuthForm.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-  import { tick } from 'svelte';
+  import { tick, untrack } from 'svelte';
   import { goto } from '$app/navigation';
   import { resolve } from '$app/paths';
 
@@ -40,7 +40,7 @@
     alternativePageLink,
   }: Props = $props();
 
-  const { form, message, errors, submitting, enhance } = formProperties;
+  const { form, message, errors, submitting, enhance } = untrack(() => formProperties);
 
   const homeHref = resolve(HOME_PAGE);
   const forgotPasswordHref = resolve(FORGOT_PASSWORD_PAGE);
diff --git a/src/routes/(admin)/account_transfer/+page.svelte b/src/routes/(admin)/account_transfer/+page.svelte
index ff52a06ca..fb5b9f66a 100644
--- a/src/routes/(admin)/account_transfer/+page.svelte
+++ b/src/routes/(admin)/account_transfer/+page.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-  import { onMount, onDestroy } from 'svelte';
+  import { onMount, onDestroy, untrack } from 'svelte';
   import { run } from 'svelte/legacy';
 
   import { superForm } from 'sveltekit-superforms';
@@ -28,7 +28,7 @@
 
   let { data }: Props = $props();
 
-  const { form, errors, message, submitting, enhance } = superForm(data.form);
+  const { form, errors, message, submitting, enhance } = superForm(untrack(() => data.form));
 
   let accountTransferMessages: FloatingMessages = $state([]);
 
diff --git a/src/routes/workbooks/create/+page.svelte b/src/routes/workbooks/create/+page.svelte
index 78aaa6e2e..ce473ac1a 100644
--- a/src/routes/workbooks/create/+page.svelte
+++ b/src/routes/workbooks/create/+page.svelte
@@ -1,4 +1,5 @@
 <script lang="ts">
+  import { untrack } from 'svelte';
   import { superForm } from 'sveltekit-superforms/client';
 
   import type { Task } from '#lib/types/task.js';
@@ -8,22 +9,23 @@
 
   let { data } = $props();
 
+  // superForm owns the form state after init, so seed it once from the load data.
   // See:
   // https://superforms.rocks/concepts/nested-data
-  const initialData = {
+  const initialData = untrack(() => ({
     ...data.form,
     workBookTasks: [] as WorkBookTasksBase,
-  };
+  }));
   const superFormObject = superForm(initialData, {
     dataType: 'json',
   });
   const { form } = superFormObject;
 
-  $form.authorId = data.author.id;
-  $form.isOfficial = data.isAdmin;
+  $form.authorId = untrack(() => data.author.id);
+  $form.isOfficial = untrack(() => data.isAdmin);
   $form.workBookType = $form.isOfficial ? WorkBookType.CURRICULUM : WorkBookType.CREATED_BY_USER;
 
-  const tasksMapByIds: Map<string, Task> = data.tasksMapByIds;
+  const tasksMapByIds: Map<string, Task> = $derived(data.tasksMapByIds);
 </script>
 
 <WorkBookForm
diff --git a/src/routes/workbooks/edit/[slug]/+page.svelte b/src/routes/workbooks/edit/[slug]/+page.svelte
index 1d9891191..097e26c6f 100644
--- a/src/routes/workbooks/edit/[slug]/+page.svelte
+++ b/src/routes/workbooks/edit/[slug]/+page.svelte
@@ -1,4 +1,5 @@
 <script lang="ts">
+  import { untrack } from 'svelte';
   import { superForm } from 'sveltekit-superforms/client';
 
   import ErrorMessageAndReturnButton from '#lib/components/Messages/ErrorMessageAndReturnButton.svelte';
@@ -13,20 +14,21 @@
   let { data } = $props();
   let canView = $derived(data.status === FORBIDDEN ? false : true);
 
-  let workBook = data.workBook;
+  let workBook = $derived(data.workBook);
 
+  // superForm owns the form state after init, so seed it once from the load data.
   // See:
   // https://superforms.rocks/concepts/nested-data
-  const initialData = {
+  const initialData = untrack(() => ({
     ...data.form,
     workBookTasks: workBook.workBookTasks as WorkBookTasksBase,
-  };
+  }));
   const superFormObject = superForm(initialData, {
     dataType: 'json',
   });
 
   // Create data of workbook for edit based on database.
-  const tasksMapByIds: Map<string, Task> = data.tasksMapByIds;
+  const tasksMapByIds: Map<string, Task> = $derived(data.tasksMapByIds);
 </script>
 
 {#if canView}

From d10d82c74cb570ebf5ad43549e721e56a17f6f19 Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:00:52 +0000
Subject: [PATCH 08/18] docs: record the phase 5 result in the build warnings
 plan

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 docs/dev-notes/2026-11-01/fix-build-warnings/plan.md | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
index 09a944c86..b6da53546 100644
--- a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
+++ b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
@@ -49,9 +49,9 @@
 
 ### Phase 5: Superforms の初期値（10件、残り0件）
 
-- [ ] `untrack` で包む: `(admin)/account_transfer/+page.svelte:31`、`AuthForm.svelte:43`・`WorkBookForm.svelte:41`（受け取った superForm の object を分割代入する所）、`workbooks/create/+page.svelte:14,22,23`、`workbooks/edit/[slug]/+page.svelte:21`。
-- [ ] `$derived` にする: `workbooks/create/+page.svelte:26`、`workbooks/edit/[slug]/+page.svelte:16,29`。
-- [ ] 確認: E2E の `account_transfer`・`signin`・`redirect_after_login`・`workbook_edit`・`error_pages` が通る。問題集を作ると `authorId`・`isOfficial`・`workBookType` が今と同じ値で保存され、編集画面に今の値が入っている。
+- [x] `untrack` で包む: `(admin)/account_transfer/+page.svelte:31`、`AuthForm.svelte:43`・`WorkBookForm.svelte:41`（受け取った superForm の object を分割代入する所）、`workbooks/create/+page.svelte:14,22,23`、`workbooks/edit/[slug]/+page.svelte:21`。
+- [x] `$derived` にする: `workbooks/create/+page.svelte:26`、`workbooks/edit/[slug]/+page.svelte:16,29`。
+- [x] 確認: E2E の `account_transfer`・`signin`・`redirect_after_login`・`workbook_edit`・`error_pages` が通る。問題集を作ると `authorId`・`isOfficial`・`workBookType` が今と同じ値で保存され、編集画面に今の値が入っている。結果: `pnpm check` は10件から0件。E2E は38件 pass。`workbooks/create`・`edit` の `initialData` は、object 全体を `untrack` で包んだ（`edit` は `$derived` にした `workBook` も初期化で読むため）。admin で作成・編集の画面を開き、input の値と送信内容（`__superform_json`）を変更の前後で書き出して比べ、差分は無かった（毎回変わる `__superform_id` は除いた）。送信は Playwright で abort したので、DB には保存していない。`workbooks/create` は admin 専用（`+page.server.ts` が admin 以外に 403 を返す）なので、`isOfficial` は実際には常に `true` で、`CREATED_BY_USER` の分岐はこの画面からは通らない。
 
 ### Phase 6: CI で警告を止める
 

From 575516d68fba3bac3af9c169a023d124403d4026 Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:04:47 +0000
Subject: [PATCH 09/18] chore: fail check and lint on warnings

svelte-check and oxlint exit 0 on warnings by default, which let the
30 warnings fixed in this branch pile up unnoticed. Add
--fail-on-warnings to check and --deny-warnings to oxlint in lint and
the pre-commit hook.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 lefthook.yml | 2 +-
 package.json | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/lefthook.yml b/lefthook.yml
index a1a61ee44..349903494 100644
--- a/lefthook.yml
+++ b/lefthook.yml
@@ -9,7 +9,7 @@ pre-commit:
       glob: '**/*.{js,jsx,ts,tsx,md,svelte}'
 
     - name: oxlint
-      run: pnpm exec oxlint {staged_files}
+      run: pnpm exec oxlint --deny-warnings {staged_files}
       glob: '**/*.{js,jsx,ts,tsx}'
 
     - name: eslint
diff --git a/package.json b/package.json
index 31d6a5e75..0aedfb681 100644
--- a/package.json
+++ b/package.json
@@ -11,9 +11,9 @@
     "build": "prisma generate && vite build",
     "preview": "vite preview",
     "test": "npm run test:e2e && npm run test:unit",
-    "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json",
+    "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --fail-on-warnings",
     "check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch",
-    "lint": "oxfmt --check && oxlint . && eslint .",
+    "lint": "oxfmt --check && oxlint --deny-warnings . && eslint .",
     "format": "oxfmt",
     "test:e2e": "playwright test",
     "test:unit": "vitest",

From 0d1cee00374532e5ea49f25774125d210e8a9cd6 Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:04:47 +0000
Subject: [PATCH 10/18] docs: record the phase 6 result in the build warnings
 plan

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 docs/dev-notes/2026-11-01/fix-build-warnings/plan.md | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
index b6da53546..2344d5294 100644
--- a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
+++ b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
@@ -55,9 +55,9 @@
 
 ### Phase 6: CI で警告を止める
 
-- [ ] `package.json` の `check` に `--fail-on-warnings` を、`lint` の oxlint に `--deny-warnings` を付ける。`check:watch` は変えない。`lefthook.yml` の oxlint の job にも `--deny-warnings` を付ける。
-- [ ] 確認: どちらも exit 0 で終わる。試しに `let x = data.x` を1行足すと `pnpm check` が exit 1 になる（`rtk bypass` で確かめ、元に戻す）。
-- [ ] これ以後、`no-explicit-any`（`warn`）も disable コメントが無ければ CI で失敗する。今ある `any` には、すべて disable コメントが付いている。
+- [x] `package.json` の `check` に `--fail-on-warnings` を、`lint` の oxlint に `--deny-warnings` を付ける。`check:watch` は変えない。`lefthook.yml` の oxlint の job にも `--deny-warnings` を付ける。CI（`ci.yml`）は `pnpm lint`・`pnpm check` を呼ぶので、workflow は変えなくてよい。
+- [x] 確認: どちらも exit 0 で終わる。試しに `let x = data.x` を1行足すと `pnpm check` が exit 1 になる（`rtk bypass` で確かめ、元に戻す）。結果: 警告が無いと `pnpm check`・`pnpm lint` はどちらも exit 0。`(admin)/tags/+page.svelte` に `let tagsSnapshot = data.tags;` を足すと、`pnpm check` は exit 1。`any` を書いた一時的な `.ts` では、`pnpm lint` も lefthook と同じ形の `oxlint --deny-warnings <file>` も exit 1。足した行と一時ファイルは消した。
+- [x] これ以後、`no-explicit-any`（`warn`）も、JS/TS のファイルでは disable コメントが無ければ CI で失敗する。`.svelte` は oxlint が見ず、ESLint の警告だけになるので、`any` を書いた一時的な `.svelte` でも `pnpm lint` は exit 0 のまま（下の「ESLint にも `--max-warnings 0` を付ける」）。今ある `any` には、すべて disable コメントが付いている。
 
 ## 採らなかった案
 
@@ -66,7 +66,7 @@
 - **`VotableGrade` の `displayGrade` を上書きできる `$derived` にする:** 親が新しい `taskResult` を渡すと、投票した直後の表示が元に戻る。挙動の変更を chore に混ぜない。
 - **`{#key}` でコンポーネントを作り直す:** `$derived` で足りる。
 - **a11y の警告を、アイコンを `<button>` にして直す:** 押しても何も起きないのに、スクリーンリーダーに「ボタン」と読まれる。flowbite-svelte がトリガーにフォーカスを付けるので、`<button>` にする必要も無い（当初はこの案だったが、`Popper.svelte` を読んで変えた）。
-- **ESLint にも `--max-warnings 0` を付ける:** 今の警告は0件で、今回の警告の出どころでもない。
+- **ESLint にも `--max-warnings 0` を付ける:** 今の警告は0件で、今回の警告の出どころでもない。ただし付けないので、`.svelte` の `no-explicit-any` は CI で止まらず、`.ts` と扱いが揃わない。
 
 ## 範囲外: chunk size の警告
 

From c5d9386786a346d1d9d2eb42b89c3253ca55e2fc Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:06:27 +0000
Subject: [PATCH 11/18] chore: fail lint on ESLint warnings too

oxlint does not lint .svelte files, so warnings such as no-explicit-any
in Svelte components passed CI while the same rule failed in .ts files.
Add --max-warnings 0 to eslint in lint and the pre-commit hook.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 lefthook.yml | 2 +-
 package.json | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lefthook.yml b/lefthook.yml
index 349903494..09458e37b 100644
--- a/lefthook.yml
+++ b/lefthook.yml
@@ -13,7 +13,7 @@ pre-commit:
       glob: '**/*.{js,jsx,ts,tsx}'
 
     - name: eslint
-      run: pnpm exec eslint {staged_files}
+      run: pnpm exec eslint --max-warnings 0 {staged_files}
       glob: '**/*.svelte'
 
     - name: plan-lambda-check
diff --git a/package.json b/package.json
index 0aedfb681..840c3071f 100644
--- a/package.json
+++ b/package.json
@@ -13,7 +13,7 @@
     "test": "npm run test:e2e && npm run test:unit",
     "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --fail-on-warnings",
     "check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch",
-    "lint": "oxfmt --check && oxlint --deny-warnings . && eslint .",
+    "lint": "oxfmt --check && oxlint --deny-warnings . && eslint --max-warnings 0 .",
     "format": "oxfmt",
     "test:e2e": "playwright test",
     "test:unit": "vitest",

From 6f694d1ea3ddd1bc555766c46440a5d1a3aaf622 Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:10:19 +0000
Subject: [PATCH 12/18] docs: rewrite the build warnings plan into decisions
 and findings

Fold the phase checklists into decisions, rejected alternatives,
verification results, unchecked items, and the Codex cross-review.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 .../2026-11-01/fix-build-warnings/plan.md     | 112 ++++++++++--------
 1 file changed, 63 insertions(+), 49 deletions(-)

diff --git a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
index 2344d5294..fdd994919 100644
--- a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
+++ b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
@@ -2,7 +2,7 @@
 
 ## 概要
 
-`pnpm build`・`pnpm check`・`pnpm lint` の警告31件を0件にし、CI で再発を止める（2026-11-01、staging `3eb7eadd`）。
+`pnpm build`・`pnpm check`・`pnpm lint` の警告31件を0件にし、警告が出たら CI（`pnpm check`・`pnpm lint`）で失敗するようにした（2026-11-01 着手、staging `3eb7eadd`）。pre-commit では oxlint と ESLint の警告だけを止め、svelte-check（`state_referenced_locally` など）は走らせていない。oxlint の1件は #4277 で、残りはブランチ `#4285` で直した。
 
 | 警告                                    | 出どころ                        | 件数 |
 | --------------------------------------- | ------------------------------- | ---- |
@@ -10,54 +10,22 @@
 | `a11y_no_noninteractive_tabindex`       | 同上                            | 1    |
 | `unicorn/no-useless-fallback-in-spread` | oxlint（`pnpm lint`）           | 1    |
 
-- build は SSR と client で2回 compile するので Svelte の警告は60個表示されるが、ユニークなのは30件で `pnpm check` と一致する。`pnpm format` の警告は0件。
-- 警告を抑える設定は無い。svelte-check と oxlint は既定では警告があっても exit 0 を返すので、CI が通っている。
-- error にする予定は見当たらない。`state_referenced_locally` は Svelte 5.45.3 から props にも出るようになり、メンテナーは「props の初期値を使うのはほとんどの場合バグ」として戻さない（[sveltejs/svelte#17289](https://github.com/sveltejs/svelte/issues/17289)）。実害は、同じ route の中で移動すると SvelteKit がコンポーネントを再利用し、`let x = data.x` が古い値のまま残ること（[SvelteKit docs](https://svelte.dev/docs/kit/state-management#Component-and-page-state-is-preserved)）。今はそうした移動をするリンクが無く、潜在的なバグにとどまる。
-- svelte.dev などに devcontainer から接続できなかったので、docs は GitHub 上の source で、Superforms は issue で確かめた。
-- 範囲外: chunk size の警告（末尾）、`(admin)/account_transfer` の `svelte/legacy` の `run`。`pnpm dev`・`test:unit`・`test:e2e` の出力は見ていない。
+- build は SSR と client で2回 compile するので Svelte の警告は60個表示されるが、ユニークなのは30件で `pnpm check` と一致する。
+- svelte-check・oxlint・ESLint は、既定では警告があっても exit 0 を返す。そのため、警告が溜まっていても CI は通っていた。
+- `state_referenced_locally` は、Svelte 5.45.3 から props にも出るようになった。メンテナーは「props の初期値を使うのはほとんどの場合バグ」として、この変更を戻さない（[sveltejs/svelte#17289](https://github.com/sveltejs/svelte/issues/17289)）。実害は、同じ route の中で移動すると SvelteKit がコンポーネントを再利用し、`let x = data.x` が古い値のまま残ること（[SvelteKit docs](https://svelte.dev/docs/kit/state-management#Component-and-page-state-is-preserved)）。今はそうした移動をするリンクが無く、潜在的なバグにとどまる。
 
-## 方針
+## 決めたこと
 
-- 値の変化に追従してよい所は `$derived` に、初期値だけを使う意図の所は `untrack(() => ...)` にする。迷う所は、今の挙動を保つ `untrack` にする。前者は SvelteKit の docs と `svelte-runes.md` に、後者は Superforms の docs（[superforms#667](https://github.com/ciscoheat/sveltekit-superforms/issues/667)、`superForm(untrack(() => data.form))`）と `AtCoderVerificationForm.svelte:23-24` に揃える。
-- 挙動の分岐を足さないので test-first は省く。各 phase で `pnpm check` の残りの件数と既存の E2E を確かめてから commit する。手で変えるのは18ファイルで、クロスレビューの条件には当たらない。
-
-## フェーズ
-
-### Phase 1: oxlint（1件）
-
-- [x] `src/test/lib/utils/auth_forms.test.ts:222-224` の stub を `vi.stubGlobal('crypto', { randomUUID: undefined })` にし、`prevCrypto` を消して、コメントを実態に合わせる。`?? {}` は、object の spread が null・undefined を無視するので要らない。`prevCrypto` は `beforeEach` の stub（`randomUUID` だけを持つ object）なので、元のコードも `{ randomUUID: undefined }` を作っていた。コメントの「preserve other properties」は実態と合っていなかった。
-- [x] 確認: `pnpm exec oxlint .` が0件になり、`auth_forms.test.ts` が通る。
-
-### Phase 2: 表示にだけ使う値を `$derived` にする（14件、残り16件）
-
-- [x] `InputFieldWrapper.svelte:29`・`(admin)/tags/+page.svelte:6`・`(admin)/tags/[tag_id]/+page.svelte:6-7`・`(admin)/tasks/[task_id]/+page.svelte:8-9`・`problems/[slug]/+page.svelte:15-16`・`users/[username]/+page.svelte:11-14`・`workbooks/[slug]/+page.svelte:31,36` を、`let tags = $derived(data.tags);` の形にする。
-- [x] `workbooks/[slug]` の `$effect` は、今も `taskResults` を通じて `data` が変わるたびに再実行されている。`workBook` を `$derived` にしても、表示は変わらない見込み。
-- [x] 確認: E2E の `submission_status`・`workbooks_list`・`redirect_after_login` が通り、上の6画面を開いて表示が変わっていない。結果: `pnpm check` は30件から16件（Phase 2 の7ファイルは0件）。E2E は50件 pass・1件 skip（データが無いときの条件付き skip）。admin で6画面と `/workbooks/create` の本文と input（`readonly`・value）を変更の前後で書き出し、差分は無かった。
-
-### Phase 3: コンポーネントの props（5件、残り11件）
-
-- [x] `KanbanColumn.svelte:21,23`: `type`・`accept` も `id` と同じく getter にする。`@dnd-kit/svelte` の `createDroppable` は、getter の値を `$effect` の中で読み直す。
-- [x] `VotableGrade.svelte:63`: `componentId` を `$derived` にする。
-- [x] `VotableGrade.svelte:59`: 投票後の表示に使う state の初期値なので、`untrack` で包む（同じファイルの78行目と同じ形）。
-- [x] 確認: E2E の `workbook_order`・`votes`・`workbooks_list` が通る。結果: `pnpm check` は16件から11件。E2E は54件のうち53件 pass・1件 skip（Phase 2 と同じ条件付き skip）。`VotableGrade:59` は、78行目に揃えて `initialGrade` を消し、`$state(untrack(() => ...))` にまとめた。
-
-### Phase 4: a11y（1件、残り10件）
-
-- [x] `votes/+page.svelte:69` の `<span role="img">` から `tabindex="0"` を消す。flowbite-svelte 1.33.1 の `Popper.svelte:201` が、`Tooltip` のトリガーに実行時に `tabIndex = 0` を付けるので、markup の `tabindex` は重複している。`VotableGrade.svelte` の同じフラスコのアイコンも、`tabindex` を書かずにトリガーにしている。
-- [x] 実行時の DOM は今と同じく「フォーカスできる `role="img"`」のままで、a11y の状態は良くも悪くもならない。キーボードで Tooltip の説明を読むには、フォーカスできる必要がある。
-- [x] 確認: Tab でアイコンにフォーカスすると Tooltip が開き、見た目（cursor・色・位置）が変わらない。E2E の `votes` が通る。結果: `pnpm check` は11件から10件。E2E の `votes` は15件 pass。dev の DB に暫定グレードの問題が無いので、Playwright で `/votes/__data.json` を書き換えて PENDING の問題1件に `estimatedGrade` を足し、アイコンを出して確かめた。dev server のコンパイル結果に `tabindex` は無いが、実行時の要素には `tabindex="0"` が付き、Tab でフォーカスすると Tooltip が開いて、フォーカスが離れると閉じた。`cursor: help` と色も変わらない。E2E にはこのアイコンのテストが無い。
-
-### Phase 5: Superforms の初期値（10件、残り0件）
-
-- [x] `untrack` で包む: `(admin)/account_transfer/+page.svelte:31`、`AuthForm.svelte:43`・`WorkBookForm.svelte:41`（受け取った superForm の object を分割代入する所）、`workbooks/create/+page.svelte:14,22,23`、`workbooks/edit/[slug]/+page.svelte:21`。
-- [x] `$derived` にする: `workbooks/create/+page.svelte:26`、`workbooks/edit/[slug]/+page.svelte:16,29`。
-- [x] 確認: E2E の `account_transfer`・`signin`・`redirect_after_login`・`workbook_edit`・`error_pages` が通る。問題集を作ると `authorId`・`isOfficial`・`workBookType` が今と同じ値で保存され、編集画面に今の値が入っている。結果: `pnpm check` は10件から0件。E2E は38件 pass。`workbooks/create`・`edit` の `initialData` は、object 全体を `untrack` で包んだ（`edit` は `$derived` にした `workBook` も初期化で読むため）。admin で作成・編集の画面を開き、input の値と送信内容（`__superform_json`）を変更の前後で書き出して比べ、差分は無かった（毎回変わる `__superform_id` は除いた）。送信は Playwright で abort したので、DB には保存していない。`workbooks/create` は admin 専用（`+page.server.ts` が admin 以外に 403 を返す）なので、`isOfficial` は実際には常に `true` で、`CREATED_BY_USER` の分岐はこの画面からは通らない。
-
-### Phase 6: CI で警告を止める
-
-- [x] `package.json` の `check` に `--fail-on-warnings` を、`lint` の oxlint に `--deny-warnings` を付ける。`check:watch` は変えない。`lefthook.yml` の oxlint の job にも `--deny-warnings` を付ける。CI（`ci.yml`）は `pnpm lint`・`pnpm check` を呼ぶので、workflow は変えなくてよい。
-- [x] 確認: どちらも exit 0 で終わる。試しに `let x = data.x` を1行足すと `pnpm check` が exit 1 になる（`rtk bypass` で確かめ、元に戻す）。結果: 警告が無いと `pnpm check`・`pnpm lint` はどちらも exit 0。`(admin)/tags/+page.svelte` に `let tagsSnapshot = data.tags;` を足すと、`pnpm check` は exit 1。`any` を書いた一時的な `.ts` では、`pnpm lint` も lefthook と同じ形の `oxlint --deny-warnings <file>` も exit 1。足した行と一時ファイルは消した。
-- [x] これ以後、`no-explicit-any`（`warn`）も、JS/TS のファイルでは disable コメントが無ければ CI で失敗する。`.svelte` は oxlint が見ず、ESLint の警告だけになるので、`any` を書いた一時的な `.svelte` でも `pnpm lint` は exit 0 のまま（下の「ESLint にも `--max-warnings 0` を付ける」）。今ある `any` には、すべて disable コメントが付いている。
+- **値の使い方で書き方を分ける:**
+  - 表示や検索に使うだけで、今の値に追従すべきもの（`data.*`・props）は `$derived` にする。
+  - 受け取った値を初期値にして、その後は自分で書き換える state は `untrack(() => ...)` で包む。Superforms の初期値（`superForm(untrack(() => data.form))`、[superforms#667](https://github.com/ciscoheat/sveltekit-superforms/issues/667)）と、`VotableGrade` の `displayGrade` がこれに当たる。実行時の挙動は `untrack` が無い時と同じで、「初期値だけを使うのは意図したことだ」とコードに残すために使う。
+  - ライブラリに渡す option のうち、ライブラリが自分の `$effect` の中で読み直すもの（`createDroppable` の `id`・`type`・`accept`）は getter で渡す。
+  - 迷う所は、今の挙動を保つ `untrack` にする。
+- **Superforms の初期値:** `workbooks/create`・`edit` の `initialData` は、object 全体を `untrack` で包んだ。`edit` では、`$derived` にした `workBook` も初期化のときに読むため。`AuthForm`・`WorkBookForm` で受け取った superForm の object を分割代入する所も、`untrack` で包んだ。
+- **a11y の警告:** `votes/+page.svelte` のフラスコのアイコンから、`tabindex="0"` を消すだけにした。flowbite-svelte 1.33.1 の `Popper.svelte:201` が、`Tooltip` のトリガーに実行時に `tabIndex = 0` を付けるので、markup の `tabindex` は重複していた。実行時の DOM は今と同じく「フォーカスできる `role="img"`」のままで、a11y の状態は良くも悪くもならない。`VotableGrade.svelte` の同じアイコンも、`tabindex` を書かずにトリガーにしている。
+- **警告で失敗させる:** `check` に `--fail-on-warnings` を付けた。`lint` では、oxlint に `--deny-warnings` を、ESLint に `--max-warnings 0` を付けた。`lefthook.yml` の oxlint と eslint の job にも同じフラグを付けた。CI（`ci.yml`）は `pnpm lint`・`pnpm check` を呼ぶので、workflow は変えていない。`check:watch` も変えていない。
+- **ESLint の扱い:** ESLint に `--max-warnings 0` を付けるのは、当初は不要とした。しかし oxlint は `.svelte` を見ないので、付けないと `.svelte` の `no-explicit-any` だけが CI で止まらず、`.ts` と扱いが揃わない。それがわかったので付けた。今ある `any` には、すべて disable コメントが付いている。
+- **テスト:** 挙動の分岐を足さないので test-first は省いた。phase ごとに `pnpm check` の残りの件数・既存の E2E・変更前後の画面の比較で確かめてから commit した。
 
 ## 採らなかった案
 
@@ -65,8 +33,53 @@
 - **Superforms に渡す値を `$derived` にする:** `superForm` は初期化した後、自分で state を持つ。`AuthForm` は呼び出し側が template の式で `superForm(data?.form)` を渡すので、子で読み直すと初期化の外で `superForm()` が呼ばれうる。
 - **`VotableGrade` の `displayGrade` を上書きできる `$derived` にする:** 親が新しい `taskResult` を渡すと、投票した直後の表示が元に戻る。挙動の変更を chore に混ぜない。
 - **`{#key}` でコンポーネントを作り直す:** `$derived` で足りる。
-- **a11y の警告を、アイコンを `<button>` にして直す:** 押しても何も起きないのに、スクリーンリーダーに「ボタン」と読まれる。flowbite-svelte がトリガーにフォーカスを付けるので、`<button>` にする必要も無い（当初はこの案だったが、`Popper.svelte` を読んで変えた）。
-- **ESLint にも `--max-warnings 0` を付ける:** 今の警告は0件で、今回の警告の出どころでもない。ただし付けないので、`.svelte` の `no-explicit-any` は CI で止まらず、`.ts` と扱いが揃わない。
+- **a11y の警告を、アイコンを `<button>` にして直す:** 当初の plan はこの案だった。押しても何も起きないのに、スクリーンリーダーに「ボタン」と読まれる。flowbite-svelte がトリガーにフォーカスを付けるので、`<button>` にする必要も無い。
+- **a11y の警告を、`tabindex` を消すとキーボードで開けなくなるとして残す:** 当初の plan の前提だったが誤り。上の `Popper.svelte:201` のとおり、消しても実行時に付く。
+
+## わかったこと（コードや git に残らない事実）
+
+- `let x = data.x` は、Svelte 4 の時も初期値を写すだけで、追従していなかった（追従させるには `$: x = data.x` が要った）。`users/[username]`・`problems/[slug]` は作った時から `export let data; let x = data.x;` で、runes に移した `01fffb99`（2025-02-22）は `export let data` を `$props()` に置き換えただけ。そのため、v5 で壊れたのではなく、v4 の頃からある潜在的なバグに警告が出るようになった、というのが実態。
+- `@dnd-kit/svelte` 0.5.1 の `createDroppable` は、input の `id`・`accept`・`type` を内部の `$effect` で読み直す。dnd-kit の docs には、「reactive な値と同期させたい option は getter で定義する」とある。一方、quickstart では `createDroppable({id})` と値をそのまま渡している。`KanbanBoard.svelte` は `group` に固定の文字列を渡すので、getter にしても今の動きは変わらない。
+- `workbooks/create` は admin 専用で、`+page.server.ts` が admin 以外に 403 を返す。そのため `isOfficial` は実際には常に `true` で、`CREATED_BY_USER` の分岐はこの画面からは通らない。
+- oxlint は `.svelte` を lint しない。`.svelte` の lint は ESLint だけが担っている。
+- **調べた資料:** svelte.dev・dndkit.com には devcontainer から接続できなかった。代わりに、手元の Svelte 5.57.3 の型定義（`untrack` の JSDoc）と、GitHub 上の source（Svelte の `compile-warnings.md`・`$effect.md`、dnd-kit の `create-droppable.mdx`・`quickstart.mdx`）を見た。Superforms は issue で確かめた。
+
+## 確かめたこと
+
+- **警告の件数:** `pnpm check` は30件 → 16件（`$derived`）→ 11件（props）→ 10件（a11y）→ 0件（Superforms）になった。
+- **E2E:** 各 phase で関係する spec を流した。次以外はすべて pass した。`workbooks_list` の1件は、データが無いときに飛ばす条件付きの skip。
+  - Phase 2: `submission_status`・`workbooks_list`・`redirect_after_login`
+  - Phase 3: `workbook_order`・`votes`・`workbooks_list`
+  - Phase 4: `votes`
+  - Phase 5: `account_transfer`・`signin`・`redirect_after_login`・`workbook_edit`・`error_pages`
+- **unit テスト:** 各 phase で `pnpm test:unit` の2994件が pass した。
+- **Phase 2 の画面:** admin で6画面と `/workbooks/create` を開き、本文と input（`readonly`・value）を Playwright で書き出した。変更前・変更後・2回目の変更後で比べ、差分は無かった。
+- **Phase 4 のアイコン:** dev の DB に暫定グレードの問題が無いので、Playwright で `/votes/__data.json` を書き換え、PENDING の問題1件に `estimatedGrade` を足してアイコンを出した。dev server のコンパイル結果に `tabindex` は無かった。それでも実行時の要素には `tabindex="0"` が付き、Tab でフォーカスすると Tooltip が開いて、フォーカスが離れると閉じた。`cursor: help` と色も変わらない。
+- **Phase 5 の form:** admin で作成・編集の画面を開き、input の値と送信内容（`__superform_json`）を変更の前後で書き出して比べた。差分は無かった（毎回変わる `__superform_id` は除いた）。
+- **警告で失敗するか:**
+  - 警告が無い今の状態では、`pnpm check`・`pnpm lint` はどちらも exit 0。
+  - `let tagsSnapshot = data.tags;` を1行足すと、`pnpm check` は exit 1。
+  - `any` を書いた一時的な `.ts` では、`pnpm lint` も lefthook と同じ形の `oxlint --deny-warnings <file>` も exit 1。
+  - `any` を書いた一時的な `.svelte` では、ESLint にフラグを付ける前は `pnpm lint` が exit 0 だった。付けた後は、`pnpm lint` も lefthook と同じ形の `eslint --max-warnings 0 <file>` も exit 1。
+  - 足した行と一時ファイルは消した。
+
+## 確かめていないこと
+
+- 問題集を実際に保存した時の値。Phase 5 では送信を Playwright で abort したので、DB には保存していない。
+- 同じ route の中での移動（`/users/a` → `/users/b` など）で、`$derived` にした値が新しくなること。今はそうしたリンクが無い。
+- フラスコのアイコンの E2E。今は E2E にこのアイコンのテストが無く、Phase 4 の確認は手で動かした1回だけ。
+- JS が読み込まれる前の SSR の HTML では、アイコンにフォーカスできない。これは実際に困ることがまず無いとして、見ていない。
+- `pnpm dev` と E2E の警告の出力。
+
+## レビュー
+
+- ユーザーの依頼で、Codex（`codex exec -s read-only`、codex-cli 0.166.0）にクロスレビューを頼んだ（2026-11-03）。Codex の sandbox は devcontainer で起動しない（`bwrap: No permissions to create a new namespace`）。そのため、plan・差分・変更したファイルの全文・関係するライブラリの source・ルールを prompt に埋め込んで渡した。
+- Critical・Severe の指摘は無かった。dnd-kit の getter、`untrack` の追加、`tabindex` の削除、`package.json` のフラグと CI の呼び出し経路は、問題無しとされた。
+- 指摘は次の3件で、どれを直すかはユーザーの選択待ち。
+  - **Moderate: pre-commit で svelte-check が走らない（`lefthook.yml`）。** 指摘どおりで、`.svelte` に `let x = data.x` を足しても commit できる。止まるのは CI の `pnpm check`。`pnpm check` は全体を見るので staged のファイルだけには絞れず、devcontainer で11秒かかった。
+  - **Minor: ESLint が除外するディレクトリの `.svelte` を、lefthook が明示的に渡すと失敗する（`lefthook.yml`）。** 再現した。`package/` に一時的な `.svelte` を置いて `eslint --max-warnings 0 <file>` を呼ぶと、「File ignored because of a matching ignore pattern」の警告で exit 1 になり、通常のファイルと混ぜても exit 1 だった。`--no-warn-ignored` を付けると exit 0。ただし、ESLint の `ignores`（`build`・`coverage`・`.svelte-kit`・`package` など）の下にある git 管理の `.svelte` は0件で、今は起きない。
+  - **未検証（Codex 自身が断定していない）: 編集画面 A から B へ、コンポーネントを使い回したまま移ると、見出しと `tasksMapByIds` は B に追従するが、form は A の値のまま残りうる（`workbooks/edit/[slug]/+page.svelte`）。** 初期値を固定する構造は変更前からある。編集画面へのリンクは一覧（`WorkbookAuthorActionsCell`）にしか無く、編集画面の中には `/workbooks` へのパンくずリストしか無い。更新に成功すると `/workbooks` に redirect する。そのため、今はこの移動をする経路が無い。Superforms がページデータの変化で form を同期するかは確かめていない。
+- Codex は、plan の「ESLint にも付ける案を採らなかった」が最終のコードと矛盾するとも指摘した。これは Codex に渡した後に plan を書き直した時に解消した。
 
 ## 範囲外: chunk size の警告
 
@@ -79,3 +92,4 @@
 - devcontainer の `pnpm build` と E2E（Playwright の `webServer`）は dev モードのまま。本番と揃えるなら、`compose.yaml` の `NODE_ENV=development` を消す。`NODE_ENV` を読むコードは無いが、devcontainer の作り直しが要る。
 - `.coderabbit.yaml` の `src/**/*utils/**` の指示（「Must be pure functions with no side effects」）はテストファイルにも当たるので、`auth_forms.test.ts` の `vi.stubGlobal` を副作用として指摘された（Phase 1 の review）。直すなら、指示の文面に「本番コード向けで、テストでは `afterEach` で戻すなら `vi.stubGlobal` を使ってよい」と1行足す。この glob に当たるテストは63ファイルあるが、global を stub しているのはこのファイルだけ。`path_instructions` の glob が `!` による除外に対応しているかは確かめていない。
 - UI の3層構造（Tailwind CSS → Flowbite → flowbite-svelte）の見直しは、別の task・PR にする。flowbite-svelte は2026-04-07から release も merge も無く、66ファイルが46コンポーネントを import している。flowbite は3.1.2のまま（最新は v4.0.2）。
+- `(admin)/account_transfer` に残っている `svelte/legacy` の `run` は、今回の範囲外として残した。

From 1f661ce2d4e29a22b80acd4782eac9eff34e5a2d Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:12:22 +0000
Subject: [PATCH 13/18] docs: condense the build warnings plan

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 .../2026-11-01/fix-build-warnings/plan.md     | 129 +++++++-----------
 1 file changed, 50 insertions(+), 79 deletions(-)

diff --git a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
index fdd994919..9c61f218a 100644
--- a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
+++ b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
@@ -2,94 +2,65 @@
 
 ## 概要
 
-`pnpm build`・`pnpm check`・`pnpm lint` の警告31件を0件にし、警告が出たら CI（`pnpm check`・`pnpm lint`）で失敗するようにした（2026-11-01 着手、staging `3eb7eadd`）。pre-commit では oxlint と ESLint の警告だけを止め、svelte-check（`state_referenced_locally` など）は走らせていない。oxlint の1件は #4277 で、残りはブランチ `#4285` で直した。
-
-| 警告                                    | 出どころ                        | 件数 |
-| --------------------------------------- | ------------------------------- | ---- |
-| `state_referenced_locally`              | Svelte（build と `pnpm check`） | 29   |
-| `a11y_no_noninteractive_tabindex`       | 同上                            | 1    |
-| `unicorn/no-useless-fallback-in-spread` | oxlint（`pnpm lint`）           | 1    |
-
-- build は SSR と client で2回 compile するので Svelte の警告は60個表示されるが、ユニークなのは30件で `pnpm check` と一致する。
-- svelte-check・oxlint・ESLint は、既定では警告があっても exit 0 を返す。そのため、警告が溜まっていても CI は通っていた。
-- `state_referenced_locally` は、Svelte 5.45.3 から props にも出るようになった。メンテナーは「props の初期値を使うのはほとんどの場合バグ」として、この変更を戻さない（[sveltejs/svelte#17289](https://github.com/sveltejs/svelte/issues/17289)）。実害は、同じ route の中で移動すると SvelteKit がコンポーネントを再利用し、`let x = data.x` が古い値のまま残ること（[SvelteKit docs](https://svelte.dev/docs/kit/state-management#Component-and-page-state-is-preserved)）。今はそうした移動をするリンクが無く、潜在的なバグにとどまる。
+- `pnpm check`・`pnpm lint` の警告31件（`state_referenced_locally` 29、`a11y_no_noninteractive_tabindex` 1、oxlint 1）を0件にし、警告が出たら CI で失敗するようにした（2026-11-01 着手、staging `3eb7eadd`）。oxlint の1件は #4277、残りはブランチ `#4285`。
+- svelte-check・oxlint・ESLint は、既定では警告があっても exit 0 を返す。そのため、警告が溜まっても CI は通っていた。
+- `state_referenced_locally` は、Svelte 5.45.3 から props にも出る（[sveltejs/svelte#17289](https://github.com/sveltejs/svelte/issues/17289)）。実害は、同じ route の中で移動するとコンポーネントが再利用され、`let x = data.x` が古い値のまま残ること。今はそうしたリンクが無く、潜在的なバグにとどまる。
 
 ## 決めたこと
 
 - **値の使い方で書き方を分ける:**
-  - 表示や検索に使うだけで、今の値に追従すべきもの（`data.*`・props）は `$derived` にする。
-  - 受け取った値を初期値にして、その後は自分で書き換える state は `untrack(() => ...)` で包む。Superforms の初期値（`superForm(untrack(() => data.form))`、[superforms#667](https://github.com/ciscoheat/sveltekit-superforms/issues/667)）と、`VotableGrade` の `displayGrade` がこれに当たる。実行時の挙動は `untrack` が無い時と同じで、「初期値だけを使うのは意図したことだ」とコードに残すために使う。
-  - ライブラリに渡す option のうち、ライブラリが自分の `$effect` の中で読み直すもの（`createDroppable` の `id`・`type`・`accept`）は getter で渡す。
+  - 表示や検索に使うだけの `data.*`・props は `$derived` にする。
+  - 初期値にして、その後は自分で持つ state は `untrack(() => ...)` で包む（Superforms の初期値、`VotableGrade` の `displayGrade`）。実行時の挙動は変わらず、「初期値だけを使うのは意図したことだ」とコードに残す。
+  - ライブラリが自分の `$effect` で読み直す option（`createDroppable` の `type`・`accept`）は getter で渡す。
   - 迷う所は、今の挙動を保つ `untrack` にする。
-- **Superforms の初期値:** `workbooks/create`・`edit` の `initialData` は、object 全体を `untrack` で包んだ。`edit` では、`$derived` にした `workBook` も初期化のときに読むため。`AuthForm`・`WorkBookForm` で受け取った superForm の object を分割代入する所も、`untrack` で包んだ。
-- **a11y の警告:** `votes/+page.svelte` のフラスコのアイコンから、`tabindex="0"` を消すだけにした。flowbite-svelte 1.33.1 の `Popper.svelte:201` が、`Tooltip` のトリガーに実行時に `tabIndex = 0` を付けるので、markup の `tabindex` は重複していた。実行時の DOM は今と同じく「フォーカスできる `role="img"`」のままで、a11y の状態は良くも悪くもならない。`VotableGrade.svelte` の同じアイコンも、`tabindex` を書かずにトリガーにしている。
-- **警告で失敗させる:** `check` に `--fail-on-warnings` を付けた。`lint` では、oxlint に `--deny-warnings` を、ESLint に `--max-warnings 0` を付けた。`lefthook.yml` の oxlint と eslint の job にも同じフラグを付けた。CI（`ci.yml`）は `pnpm lint`・`pnpm check` を呼ぶので、workflow は変えていない。`check:watch` も変えていない。
-- **ESLint の扱い:** ESLint に `--max-warnings 0` を付けるのは、当初は不要とした。しかし oxlint は `.svelte` を見ないので、付けないと `.svelte` の `no-explicit-any` だけが CI で止まらず、`.ts` と扱いが揃わない。それがわかったので付けた。今ある `any` には、すべて disable コメントが付いている。
-- **テスト:** 挙動の分岐を足さないので test-first は省いた。phase ごとに `pnpm check` の残りの件数・既存の E2E・変更前後の画面の比較で確かめてから commit した。
-
-## 採らなかった案
-
-- **`warningFilter`・`svelte-ignore` で消す:** 本物のバグも隠れてしまう。意図的な所も、`untrack` の方が意図をコードで表せる。
-- **Superforms に渡す値を `$derived` にする:** `superForm` は初期化した後、自分で state を持つ。`AuthForm` は呼び出し側が template の式で `superForm(data?.form)` を渡すので、子で読み直すと初期化の外で `superForm()` が呼ばれうる。
-- **`VotableGrade` の `displayGrade` を上書きできる `$derived` にする:** 親が新しい `taskResult` を渡すと、投票した直後の表示が元に戻る。挙動の変更を chore に混ぜない。
-- **`{#key}` でコンポーネントを作り直す:** `$derived` で足りる。
-- **a11y の警告を、アイコンを `<button>` にして直す:** 当初の plan はこの案だった。押しても何も起きないのに、スクリーンリーダーに「ボタン」と読まれる。flowbite-svelte がトリガーにフォーカスを付けるので、`<button>` にする必要も無い。
-- **a11y の警告を、`tabindex` を消すとキーボードで開けなくなるとして残す:** 当初の plan の前提だったが誤り。上の `Popper.svelte:201` のとおり、消しても実行時に付く。
-
-## わかったこと（コードや git に残らない事実）
-
-- `let x = data.x` は、Svelte 4 の時も初期値を写すだけで、追従していなかった（追従させるには `$: x = data.x` が要った）。`users/[username]`・`problems/[slug]` は作った時から `export let data; let x = data.x;` で、runes に移した `01fffb99`（2025-02-22）は `export let data` を `$props()` に置き換えただけ。そのため、v5 で壊れたのではなく、v4 の頃からある潜在的なバグに警告が出るようになった、というのが実態。
-- `@dnd-kit/svelte` 0.5.1 の `createDroppable` は、input の `id`・`accept`・`type` を内部の `$effect` で読み直す。dnd-kit の docs には、「reactive な値と同期させたい option は getter で定義する」とある。一方、quickstart では `createDroppable({id})` と値をそのまま渡している。`KanbanBoard.svelte` は `group` に固定の文字列を渡すので、getter にしても今の動きは変わらない。
-- `workbooks/create` は admin 専用で、`+page.server.ts` が admin 以外に 403 を返す。そのため `isOfficial` は実際には常に `true` で、`CREATED_BY_USER` の分岐はこの画面からは通らない。
-- oxlint は `.svelte` を lint しない。`.svelte` の lint は ESLint だけが担っている。
-- **調べた資料:** svelte.dev・dndkit.com には devcontainer から接続できなかった。代わりに、手元の Svelte 5.57.3 の型定義（`untrack` の JSDoc）と、GitHub 上の source（Svelte の `compile-warnings.md`・`$effect.md`、dnd-kit の `create-droppable.mdx`・`quickstart.mdx`）を見た。Superforms は issue で確かめた。
-
-## 確かめたこと
-
-- **警告の件数:** `pnpm check` は30件 → 16件（`$derived`）→ 11件（props）→ 10件（a11y）→ 0件（Superforms）になった。
-- **E2E:** 各 phase で関係する spec を流した。次以外はすべて pass した。`workbooks_list` の1件は、データが無いときに飛ばす条件付きの skip。
-  - Phase 2: `submission_status`・`workbooks_list`・`redirect_after_login`
-  - Phase 3: `workbook_order`・`votes`・`workbooks_list`
-  - Phase 4: `votes`
-  - Phase 5: `account_transfer`・`signin`・`redirect_after_login`・`workbook_edit`・`error_pages`
-- **unit テスト:** 各 phase で `pnpm test:unit` の2994件が pass した。
-- **Phase 2 の画面:** admin で6画面と `/workbooks/create` を開き、本文と input（`readonly`・value）を Playwright で書き出した。変更前・変更後・2回目の変更後で比べ、差分は無かった。
-- **Phase 4 のアイコン:** dev の DB に暫定グレードの問題が無いので、Playwright で `/votes/__data.json` を書き換え、PENDING の問題1件に `estimatedGrade` を足してアイコンを出した。dev server のコンパイル結果に `tabindex` は無かった。それでも実行時の要素には `tabindex="0"` が付き、Tab でフォーカスすると Tooltip が開いて、フォーカスが離れると閉じた。`cursor: help` と色も変わらない。
-- **Phase 5 の form:** admin で作成・編集の画面を開き、input の値と送信内容（`__superform_json`）を変更の前後で書き出して比べた。差分は無かった（毎回変わる `__superform_id` は除いた）。
-- **警告で失敗するか:**
-  - 警告が無い今の状態では、`pnpm check`・`pnpm lint` はどちらも exit 0。
-  - `let tagsSnapshot = data.tags;` を1行足すと、`pnpm check` は exit 1。
-  - `any` を書いた一時的な `.ts` では、`pnpm lint` も lefthook と同じ形の `oxlint --deny-warnings <file>` も exit 1。
-  - `any` を書いた一時的な `.svelte` では、ESLint にフラグを付ける前は `pnpm lint` が exit 0 だった。付けた後は、`pnpm lint` も lefthook と同じ形の `eslint --max-warnings 0 <file>` も exit 1。
-  - 足した行と一時ファイルは消した。
-
-## 確かめていないこと
-
-- 問題集を実際に保存した時の値。Phase 5 では送信を Playwright で abort したので、DB には保存していない。
-- 同じ route の中での移動（`/users/a` → `/users/b` など）で、`$derived` にした値が新しくなること。今はそうしたリンクが無い。
-- フラスコのアイコンの E2E。今は E2E にこのアイコンのテストが無く、Phase 4 の確認は手で動かした1回だけ。
-- JS が読み込まれる前の SSR の HTML では、アイコンにフォーカスできない。これは実際に困ることがまず無いとして、見ていない。
-- `pnpm dev` と E2E の警告の出力。
+  - 採らなかった案:
+    - `warningFilter`・`svelte-ignore`: 本物のバグも隠れてしまう。
+    - `{#key}`: `$derived` で足りる。
+    - Superforms の値を `$derived` にする: `superForm` は初期化した後、自分で state を持つ。
+    - `displayGrade` を `$derived` にする: 投票した直後の表示が、親の値で元に戻る。
+- **a11y:** `tabindex="0"` を消すだけにした。flowbite-svelte 1.33.1 の `Popper.svelte:201` が、Tooltip のトリガーに実行時に `tabIndex = 0` を付けるので、キーボードでの操作は変わらない。
+  - 採らなかった案: `<button>` にする（当初の plan）。押しても何も起きないのに、スクリーンリーダーに「ボタン」と読まれる。
+- **警告で失敗させる:**
+  - `check` に `--fail-on-warnings` を付けた。
+  - `lint` の oxlint に `--deny-warnings` を、ESLint に `--max-warnings 0` を付け、lefthook の同じ job にも付けた。
+  - svelte-check は CI でだけ走り、pre-commit では走らない。
+  - ESLint のフラグは当初は不要としたが、付けた。oxlint は `.svelte` を lint しないので、付けないと `.svelte` の `no-explicit-any` だけが止まらないため。
+- **テスト:** 挙動の分岐を足さないので test-first は省いた。各 phase で、`pnpm check` の件数・既存の E2E・変更前後の比較で確かめた。
+
+## 事実（コードや git に残らない）
+
+- `let x = data.x` は、Svelte 4 の時も追従していなかった（追従させるには `$:` が要った）。runes に移した `01fffb99` は、`export let data` を `$props()` に置き換えただけ。つまり v5 で壊れたのではなく、v4 の頃からある潜在的なバグに警告が出るようになった。
+- `@dnd-kit/svelte` 0.5.1 の `createDroppable` は、`id`・`accept`・`type` を内部の `$effect` で読み直す。`KanbanBoard` は `group` に固定の文字列を渡すので、getter にしても今の動きは変わらない。
+- `workbooks/create` は admin 専用（admin 以外には 403）。そのため、`isOfficial` は常に `true` になる。
+- svelte.dev・dndkit.com には devcontainer から接続できなかった。代わりに、手元の Svelte 5.57.3 の型定義と、GitHub 上の docs の source を見た。Superforms は issue（[#667](https://github.com/ciscoheat/sveltekit-superforms/issues/667)）で確かめた。
+
+## 検証
+
+- **警告:** `pnpm check` は 30 → 16 → 11 → 10 → 0 件になった。
+- **テスト:** 各 phase で、関係する E2E と unit テスト（2994件）がすべて pass した。例外は `workbooks_list` の1件で、データが無いときに飛ばす条件付きの skip。
+- **変更前後の比較（Playwright）:**
+  - 表示: 6画面と `/workbooks/create` の本文と input に、差分は無かった。
+  - 問題集の作成・編集: input の値と送信内容（`__superform_json`）に、差分は無かった。送信は abort した。
+- **フラスコのアイコン:** DB に暫定グレードの問題が無いので、`/votes/__data.json` を書き換えてアイコンを出した。実行時に `tabindex="0"` が付き、Tab でフォーカスすると Tooltip が開いて、フォーカスが離れると閉じた。
+- **警告で失敗するか:** 警告が無ければ `check`・`lint` は exit 0 だった。`let x = data.x`、`any` を書いた `.ts`・`.svelte` を一時的に足すと exit 1 になった。lefthook と同じ形の呼び出しでも、同じ結果だった。
+- **確かめていないこと:**
+  - DB に実際に保存したときの値
+  - 同じ route の中での移動
+  - SSR の HTML で、JS が読み込まれる前のフォーカス
+  - `pnpm dev` の出力
+  - フラスコのアイコンの E2E（今はテストが無く、手での確認1回のみ）
 
 ## レビュー
 
-- ユーザーの依頼で、Codex（`codex exec -s read-only`、codex-cli 0.166.0）にクロスレビューを頼んだ（2026-11-03）。Codex の sandbox は devcontainer で起動しない（`bwrap: No permissions to create a new namespace`）。そのため、plan・差分・変更したファイルの全文・関係するライブラリの source・ルールを prompt に埋め込んで渡した。
-- Critical・Severe の指摘は無かった。dnd-kit の getter、`untrack` の追加、`tabindex` の削除、`package.json` のフラグと CI の呼び出し経路は、問題無しとされた。
-- 指摘は次の3件で、どれを直すかはユーザーの選択待ち。
-  - **Moderate: pre-commit で svelte-check が走らない（`lefthook.yml`）。** 指摘どおりで、`.svelte` に `let x = data.x` を足しても commit できる。止まるのは CI の `pnpm check`。`pnpm check` は全体を見るので staged のファイルだけには絞れず、devcontainer で11秒かかった。
-  - **Minor: ESLint が除外するディレクトリの `.svelte` を、lefthook が明示的に渡すと失敗する（`lefthook.yml`）。** 再現した。`package/` に一時的な `.svelte` を置いて `eslint --max-warnings 0 <file>` を呼ぶと、「File ignored because of a matching ignore pattern」の警告で exit 1 になり、通常のファイルと混ぜても exit 1 だった。`--no-warn-ignored` を付けると exit 0。ただし、ESLint の `ignores`（`build`・`coverage`・`.svelte-kit`・`package` など）の下にある git 管理の `.svelte` は0件で、今は起きない。
-  - **未検証（Codex 自身が断定していない）: 編集画面 A から B へ、コンポーネントを使い回したまま移ると、見出しと `tasksMapByIds` は B に追従するが、form は A の値のまま残りうる（`workbooks/edit/[slug]/+page.svelte`）。** 初期値を固定する構造は変更前からある。編集画面へのリンクは一覧（`WorkbookAuthorActionsCell`）にしか無く、編集画面の中には `/workbooks` へのパンくずリストしか無い。更新に成功すると `/workbooks` に redirect する。そのため、今はこの移動をする経路が無い。Superforms がページデータの変化で form を同期するかは確かめていない。
-- Codex は、plan の「ESLint にも付ける案を採らなかった」が最終のコードと矛盾するとも指摘した。これは Codex に渡した後に plan を書き直した時に解消した。
-
-## 範囲外: chunk size の警告
+Codex（`codex exec -s read-only`、0.166.0）に頼んだ。sandbox が起動しないので、必要なファイルは prompt に埋め込んだ。Critical・Severe の指摘は無かった。次の3件は、ユーザーの選択待ち。
 
-- devcontainer でだけ出る。`compose.yaml` の `NODE_ENV=development` のため Vite が dev モードで build し、Svelte が dev モードでだけ出す `Component[FILENAME] = ...` を Rolldown が副作用として扱う。その結果、flowbite-svelte の使っていないコンポーネント（135個、約 158 kB）が残る。本番モードでは最大の chunk は 301 kB で、警告は出ない。
-- flowbite-svelte を Rolldown の `treeshake.moduleSideEffects` で副作用なしと宣言しても、本番では 75 kB（約9%）しか減らない。upstream が止まっているので、この設定はずっと残り、flowbite-svelte を上げるたびに確認が要る。
-- deep import（`flowbite-svelte/Button.svelte`）にしても、`Button.svelte` 自体がルートの barrel を読み込むので縮まない。`chunkSizeWarningLimit` を上げるのは、本番では出ない警告のために本番の設定を変えることになる。
+- **Moderate:** pre-commit で svelte-check が走らない。指摘どおり。`pnpm check` は staged のファイルだけに絞れず、11秒かかる。
+- **Minor:** ESLint が除外するディレクトリの `.svelte` を lefthook が渡すと、「ignored」の警告で失敗する。再現し、`--no-warn-ignored` で避けられる。ただし、該当する git 管理の `.svelte` は0件。
+- **未検証:** 編集画面 A から B へ移ると、見出しは B に追従するが、form は A のまま残りうる。初期値を固定する構造は変更前からあり、今はこの移動をする経路が無い。
 
 ## 残る作業
 
-- devcontainer の `pnpm build` と E2E（Playwright の `webServer`）は dev モードのまま。本番と揃えるなら、`compose.yaml` の `NODE_ENV=development` を消す。`NODE_ENV` を読むコードは無いが、devcontainer の作り直しが要る。
-- `.coderabbit.yaml` の `src/**/*utils/**` の指示（「Must be pure functions with no side effects」）はテストファイルにも当たるので、`auth_forms.test.ts` の `vi.stubGlobal` を副作用として指摘された（Phase 1 の review）。直すなら、指示の文面に「本番コード向けで、テストでは `afterEach` で戻すなら `vi.stubGlobal` を使ってよい」と1行足す。この glob に当たるテストは63ファイルあるが、global を stub しているのはこのファイルだけ。`path_instructions` の glob が `!` による除外に対応しているかは確かめていない。
-- UI の3層構造（Tailwind CSS → Flowbite → flowbite-svelte）の見直しは、別の task・PR にする。flowbite-svelte は2026-04-07から release も merge も無く、66ファイルが46コンポーネントを import している。flowbite は3.1.2のまま（最新は v4.0.2）。
-- `(admin)/account_transfer` に残っている `svelte/legacy` の `run` は、今回の範囲外として残した。
+- **chunk size の警告（範囲外）:** devcontainer でだけ出る。原因は `compose.yaml` の `NODE_ENV=development` で、Vite が dev モードで build し、flowbite-svelte の使っていないコンポーネント（約 158 kB）が残る。本番では出ない。直すなら `NODE_ENV` を消す（devcontainer の作り直しが要る）。`treeshake.moduleSideEffects`・deep import・`chunkSizeWarningLimit` は、効果が小さいか、本番の設定を変えることになる。
+- **`.coderabbit.yaml`:** `src/**/*utils/**` の「pure functions」の指示がテストにも当たり、`vi.stubGlobal` が指摘された。直すなら、テストは除くと1行足す。
+- **UI の3層構造:** Tailwind → Flowbite → flowbite-svelte の見直しは、別の PR にする。flowbite-svelte は2026-04-07から更新が止まっている。
+- **`account_transfer`:** `svelte/legacy` の `run` が残っている。

From e42566ece9cce39f2ab40d5d6d42c742acea4494 Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:19:31 +0000
Subject: [PATCH 14/18] chore: ignore ESLint-ignored files passed by the
 pre-commit hook

With --max-warnings 0, ESLint's "File ignored because of a matching
ignore pattern" warning for an explicitly passed file would block the
commit. Add --no-warn-ignored (found in the Codex cross-review).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 lefthook.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lefthook.yml b/lefthook.yml
index 09458e37b..038c0beb6 100644
--- a/lefthook.yml
+++ b/lefthook.yml
@@ -13,7 +13,7 @@ pre-commit:
       glob: '**/*.{js,jsx,ts,tsx}'
 
     - name: eslint
-      run: pnpm exec eslint --max-warnings 0 {staged_files}
+      run: pnpm exec eslint --max-warnings 0 --no-warn-ignored {staged_files}
       glob: '**/*.svelte'
 
     - name: plan-lambda-check

From 4d317999c545a24862b5b2c0dface5e20980238f Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:21:12 +0000
Subject: [PATCH 15/18] docs: add lessons from fixing build warnings to the
 agent rules

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 AGENTS.md                                | 2 ++
 docs/guides/agent-rules/accessibility.md | 2 ++
 docs/guides/agent-rules/coding-style.md  | 1 +
 docs/guides/agent-rules/svelte-runes.md  | 2 ++
 4 files changed, 7 insertions(+)

diff --git a/AGENTS.md b/AGENTS.md
index 16f2d7ae0..04649a06d 100644
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -82,6 +82,8 @@ Run project tools with `pnpm exec`, never `pnpm dlx`: `dlx` downloads the latest
 
 Lefthook runs oxfmt, oxlint for JS/TS, and ESLint for Svelte before commit.
 
+`check` and `lint` fail on warnings: fix them or disable with a reason. oxlint skips `.svelte`, so set lint rules in ESLint too.
+
 Formatting belongs to oxfmt alone. Do not add ESLint or oxlint rules whose `meta.type` is `layout` (e.g. `svelte/indent`), because they would conflict with the formatter.
 
 ## Verification and Cross-review Before a PR
diff --git a/docs/guides/agent-rules/accessibility.md b/docs/guides/agent-rules/accessibility.md
index bf39e8ea0..2a949ad45 100644
--- a/docs/guides/agent-rules/accessibility.md
+++ b/docs/guides/agent-rules/accessibility.md
@@ -33,6 +33,8 @@ When `{#if}` reveals a modal, dialog, or focus-trap region, move focus programma
 
 Icon-only `<Button>` variants require an explicit `aria-label`; Flowbite Svelte does not generate one automatically. Omitting `color` only applies the `'primary'` default style — it has no effect on `aria-label`.
 
+`Tooltip`/`Popover` already make their trigger focusable; no `tabindex` or `<button>` needed.
+
 ## Form Labels
 
 Every `<input>`, `<select>`, and `<textarea>` must have an associated `<label>` (via `for`/`id`) or `aria-label`. Placeholder text does not substitute for a label.
diff --git a/docs/guides/agent-rules/coding-style.md b/docs/guides/agent-rules/coding-style.md
index d86a76a14..b869ec781 100644
--- a/docs/guides/agent-rules/coding-style.md
+++ b/docs/guides/agent-rules/coding-style.md
@@ -70,6 +70,7 @@ Before removing an import, grep the entire file for all usages — removing one
 When removing a dependency or renaming a symbol, sweep the **whole repo**, not just `src`:
 
 - `grep --include='*.{ts,svelte,md}'` does NOT brace-expand — it silently matches nothing. Use `rg -ni 'name' -g '*.ts' -g '*.svelte' -g '*.md'` instead.
+- `rg` alternation is `a|b`, not `a\|b`.
 - Include root files (CONTRIBUTING.md, README), `e2e/`, and config (`vite.config.ts`) — not only `src prisma .claude`.
 
 ## Documentation
diff --git a/docs/guides/agent-rules/svelte-runes.md b/docs/guides/agent-rules/svelte-runes.md
index bfea707a0..3dac3f3d2 100644
--- a/docs/guides/agent-rules/svelte-runes.md
+++ b/docs/guides/agent-rules/svelte-runes.md
@@ -12,6 +12,8 @@ paths:
 
 - `let` captures initial value only; use `$derived` for props/server data
 - Referencing `$props()` in `$state()` initializer? Wrap with `untrack` if intentional (seed-only)
+- Superforms: `superForm(untrack(() => data.form))`, never `$derived`
+- Pass reactive library options as getters: `get id() { return id; }`
 - `pnpm check` warns: "This reference only captures the initial value"
 
 ## `SvelteMap<K, V>` — Always Type

From ca6628e7a84055dc9545dd5dda422b1db27c938e Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:28:13 +0000
Subject: [PATCH 16/18] test: cover the provisional-grade tooltip and workbook
 create defaults

Turn two manual checks from the warning fixes into E2E tests:
- the provisional-grade icon on /votes opens its tooltip on focus,
  which relies on flowbite-svelte making the trigger focusable;
- the admin workbook create form pre-fills authorId, isOfficial, and
  the curriculum type.
Seed a median (D3, from production) for the PENDING task zone2021_f so
the icon is shown.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 e2e/votes.spec.ts           | 19 +++++++++++++++++++
 e2e/workbook_create.spec.ts | 21 +++++++++++++++++++++
 prisma/seed.ts              |  6 ++++--
 3 files changed, 44 insertions(+), 2 deletions(-)
 create mode 100644 e2e/workbook_create.spec.ts

diff --git a/e2e/votes.spec.ts b/e2e/votes.spec.ts
index f416b2b6b..85972880d 100644
--- a/e2e/votes.spec.ts
+++ b/e2e/votes.spec.ts
@@ -7,6 +7,7 @@ const VOTES_LIST_URL = '/votes';
 const TASKS_GRADE_URL = '/tasks/grade';
 const KNOWN_TASK_ID = 'abc422_a'; // From prisma/tasks.ts seed data
 const KNOWN_VOTE_DETAIL_URL = '/votes/abc422_a'; // From prisma/tasks.ts seed data
+const PROVISIONAL_TASK_ID = 'zone2021_f'; // PENDING task with a median in prisma/seed.ts
 
 // ---------------------------------------------------------------------------
 // Votes list page (/votes)
@@ -51,6 +52,24 @@ test.describe('votes list page (/votes)', () => {
       timeout: TIMEOUT,
     });
   });
+
+  test('provisional grade icon opens its tooltip on keyboard focus', async ({ page }) => {
+    await page.goto(VOTES_LIST_URL);
+    const searchInput = page.getByPlaceholder('問題名・問題ID・出典で検索');
+    await expect(searchInput).toBeVisible({ timeout: TIMEOUT });
+    await searchInput.fill(PROVISIONAL_TASK_ID);
+
+    const icon = page.getByRole('img', { name: '暫定グレード' });
+    const tooltip = page.getByRole('tooltip');
+    await expect(icon).toBeVisible({ timeout: TIMEOUT });
+    await expect(tooltip).toBeHidden();
+
+    // The markup has no tabindex; flowbite-svelte's Tooltip adds it to the trigger at runtime.
+    await expect(icon).toHaveAttribute('tabindex', '0');
+    await icon.focus();
+    await expect(icon).toBeFocused();
+    await expect(tooltip).toContainText('暫定グレード');
+  });
 });
 
 // ---------------------------------------------------------------------------
diff --git a/e2e/workbook_create.spec.ts b/e2e/workbook_create.spec.ts
new file mode 100644
index 000000000..9de0fb79c
--- /dev/null
+++ b/e2e/workbook_create.spec.ts
@@ -0,0 +1,21 @@
+import { test, expect } from '@playwright/test';
+
+import { loginAsAdmin } from './helpers/auth';
+
+const TIMEOUT = 60 * 1000;
+const WORKBOOK_CREATE_URL = '/workbooks/create';
+const ADMIN_USER_ID = '1'; // From prisma/users.ts seed data
+
+test.describe('workbook create form (admin)', () => {
+  test('pre-fills the admin as author of an official curriculum workbook', async ({ page }) => {
+    await loginAsAdmin(page);
+    await page.goto(WORKBOOK_CREATE_URL);
+
+    // Hidden inputs have no label, so select them by name.
+    await expect(page.locator('input[name="userId"]')).toHaveValue(ADMIN_USER_ID, {
+      timeout: TIMEOUT,
+    });
+    await expect(page.locator('input[name="isOfficial"]')).toHaveValue('true');
+    await expect(page.getByLabel('問題集の種類')).toHaveValue('CURRICULUM');
+  });
+});
diff --git a/prisma/seed.ts b/prisma/seed.ts
index 6e45ebea5..db50bfa1d 100755
--- a/prisma/seed.ts
+++ b/prisma/seed.ts
@@ -680,10 +680,12 @@ async function addAnswer(
   });
 }
 
-// Demo data: tasks with VotedGradeStatistics so the ±0 badge is visible in the UI.
-// grade matches the task's official grade → diff=0 → ±0 badge displayed.
+// Demo data: tasks with VotedGradeStatistics so vote-based UI is visible.
+// APG4bPython_co: grade matches the official grade → diff=0 → ±0 badge displayed.
+// zone2021_f: PENDING task with a median → provisional grade icon displayed (used by E2E).
 const VOTE_STATS_DEMO_DATA = [
   { id: 'demo-vote-stat-apg4b-co', taskId: 'APG4bPython_co', grade: 'Q8' },
+  { id: 'demo-vote-stat-zone2021-f', taskId: 'zone2021_f', grade: 'D3' },
 ] as const;
 
 async function addVoteStatisticsDemoData() {

From 126431f343611dce0fd7b1d50117198aedbcff75 Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:28:14 +0000
Subject: [PATCH 17/18] docs: record the added E2E tests and review outcome in
 the plan

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 .../2026-11-01/fix-build-warnings/plan.md      | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
index 9c61f218a..6fb8dfa5a 100644
--- a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
+++ b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
@@ -26,6 +26,11 @@
   - svelte-check は CI でだけ走り、pre-commit では走らない。
   - ESLint のフラグは当初は不要としたが、付けた。oxlint は `.svelte` を lint しないので、付けないと `.svelte` の `no-explicit-any` だけが止まらないため。
 - **テスト:** 挙動の分岐を足さないので test-first は省いた。各 phase で、`pnpm check` の件数・既存の E2E・変更前後の比較で確かめた。
+- **手での確認を E2E にする基準:** テストが無く、ライブラリの更新で何度も流す価値があり、安定して書けるものだけにした。当てはまったのは2件。
+  - `votes.spec.ts`: フラスコのアイコンに focus すると Tooltip が開く。そのために、seed の `VOTE_STATS_DEMO_DATA` に、PENDING の `zone2021_f` を足した（中央値は本番のデータの D3）。
+  - `workbook_create.spec.ts`（新規）: admin の作成画面で、`userId`・`isOfficial`・「問題集の種類」の初期値を見る。送信しないので DB は汚れない。
+  - 画面の文字の比較はデータで変わり、同じ route の中での移動は経路が無いので、テストにしなかった。
+- **ルールへの追記:** 今回の教訓を英語で1行ずつ足した。警告でも CI が落ちることと ESLint 側のルール（AGENTS.md）、Superforms と getter（`svelte-runes.md`）、Tooltip のフォーカス（`accessibility.md`）、`rg` の `|`（`coding-style.md`）。
 
 ## 事実（コードや git に残らない）
 
@@ -41,22 +46,23 @@
 - **変更前後の比較（Playwright）:**
   - 表示: 6画面と `/workbooks/create` の本文と input に、差分は無かった。
   - 問題集の作成・編集: input の値と送信内容（`__superform_json`）に、差分は無かった。送信は abort した。
-- **フラスコのアイコン:** DB に暫定グレードの問題が無いので、`/votes/__data.json` を書き換えてアイコンを出した。実行時に `tabindex="0"` が付き、Tab でフォーカスすると Tooltip が開いて、フォーカスが離れると閉じた。
+- **フラスコのアイコン:** 最初は、`/votes/__data.json` を書き換えてアイコンを出し、手で確かめた。実行時に `tabindex="0"` が付き、Tab でフォーカスすると Tooltip が開いて、フォーカスが離れると閉じた。その後、E2E にした。
+- **足した E2E:** 2件とも通り、E2E 全体も121件 pass した（1件は条件付きの skip）。production のコードを壊すと、2件とも失敗した。Tooltip を消すと `tabindex` が付かずに失敗し、`$form.isOfficial` の代入を消すと `isOfficial` が `false` になって失敗した。
 - **警告で失敗するか:** 警告が無ければ `check`・`lint` は exit 0 だった。`let x = data.x`、`any` を書いた `.ts`・`.svelte` を一時的に足すと exit 1 になった。lefthook と同じ形の呼び出しでも、同じ結果だった。
 - **確かめていないこと:**
   - DB に実際に保存したときの値
   - 同じ route の中での移動
   - SSR の HTML で、JS が読み込まれる前のフォーカス
   - `pnpm dev` の出力
-  - フラスコのアイコンの E2E（今はテストが無く、手での確認1回のみ）
+  - E2E は CI でコメントアウトされているので、足した E2E はローカルでしか流れない
 
 ## レビュー
 
-Codex（`codex exec -s read-only`、0.166.0）に頼んだ。sandbox が起動しないので、必要なファイルは prompt に埋め込んだ。Critical・Severe の指摘は無かった。次の3件は、ユーザーの選択待ち。
+Codex（`codex exec -s read-only`、0.166.0）に頼んだ。sandbox が起動しないので、必要なファイルは prompt に埋め込んだ。Critical・Severe の指摘は無かった。ユーザーが直すのに選んだのは、Minor の1件だけ。
 
-- **Moderate:** pre-commit で svelte-check が走らない。指摘どおり。`pnpm check` は staged のファイルだけに絞れず、11秒かかる。
-- **Minor:** ESLint が除外するディレクトリの `.svelte` を lefthook が渡すと、「ignored」の警告で失敗する。再現し、`--no-warn-ignored` で避けられる。ただし、該当する git 管理の `.svelte` は0件。
-- **未検証:** 編集画面 A から B へ移ると、見出しは B に追従するが、form は A のまま残りうる。初期値を固定する構造は変更前からあり、今はこの移動をする経路が無い。
+- **Minor（直した）:** ESLint が除外するディレクトリの `.svelte` を lefthook が渡すと、「ignored」の警告で失敗する。再現したので、lefthook の eslint に `--no-warn-ignored` を足した。該当する git 管理の `.svelte` は0件。
+- **Moderate（直していない）:** pre-commit で svelte-check が走らない。指摘どおりで、svelte-check の警告は CI でだけ止まる。`pnpm check` は staged のファイルだけに絞れず、11秒かかる。
+- **未検証（直していない）:** 編集画面 A から B へ移ると、見出しは B に追従するが、form は A のまま残りうる。初期値を固定する構造は変更前からあり、今はこの移動をする経路が無い。
 
 ## 残る作業
 

From 1c457c3e5cc35129971148b023320e8908bf4a75 Mon Sep 17 00:00:00 2001
From: "k.hiro1818" <k.hiro1818@gmail.com>
Date: Tue, 3 Nov 2026 00:29:57 +0000
Subject: [PATCH 18/18] docs: remove the build warnings plan

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 .../2026-11-01/fix-build-warnings/plan.md     | 72 -------------------
 1 file changed, 72 deletions(-)
 delete mode 100644 docs/dev-notes/2026-11-01/fix-build-warnings/plan.md

diff --git a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md b/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
deleted file mode 100644
index 6fb8dfa5a..000000000
--- a/docs/dev-notes/2026-11-01/fix-build-warnings/plan.md
+++ /dev/null
@@ -1,72 +0,0 @@
-# build・check・lint の警告の解消
-
-## 概要
-
-- `pnpm check`・`pnpm lint` の警告31件（`state_referenced_locally` 29、`a11y_no_noninteractive_tabindex` 1、oxlint 1）を0件にし、警告が出たら CI で失敗するようにした（2026-11-01 着手、staging `3eb7eadd`）。oxlint の1件は #4277、残りはブランチ `#4285`。
-- svelte-check・oxlint・ESLint は、既定では警告があっても exit 0 を返す。そのため、警告が溜まっても CI は通っていた。
-- `state_referenced_locally` は、Svelte 5.45.3 から props にも出る（[sveltejs/svelte#17289](https://github.com/sveltejs/svelte/issues/17289)）。実害は、同じ route の中で移動するとコンポーネントが再利用され、`let x = data.x` が古い値のまま残ること。今はそうしたリンクが無く、潜在的なバグにとどまる。
-
-## 決めたこと
-
-- **値の使い方で書き方を分ける:**
-  - 表示や検索に使うだけの `data.*`・props は `$derived` にする。
-  - 初期値にして、その後は自分で持つ state は `untrack(() => ...)` で包む（Superforms の初期値、`VotableGrade` の `displayGrade`）。実行時の挙動は変わらず、「初期値だけを使うのは意図したことだ」とコードに残す。
-  - ライブラリが自分の `$effect` で読み直す option（`createDroppable` の `type`・`accept`）は getter で渡す。
-  - 迷う所は、今の挙動を保つ `untrack` にする。
-  - 採らなかった案:
-    - `warningFilter`・`svelte-ignore`: 本物のバグも隠れてしまう。
-    - `{#key}`: `$derived` で足りる。
-    - Superforms の値を `$derived` にする: `superForm` は初期化した後、自分で state を持つ。
-    - `displayGrade` を `$derived` にする: 投票した直後の表示が、親の値で元に戻る。
-- **a11y:** `tabindex="0"` を消すだけにした。flowbite-svelte 1.33.1 の `Popper.svelte:201` が、Tooltip のトリガーに実行時に `tabIndex = 0` を付けるので、キーボードでの操作は変わらない。
-  - 採らなかった案: `<button>` にする（当初の plan）。押しても何も起きないのに、スクリーンリーダーに「ボタン」と読まれる。
-- **警告で失敗させる:**
-  - `check` に `--fail-on-warnings` を付けた。
-  - `lint` の oxlint に `--deny-warnings` を、ESLint に `--max-warnings 0` を付け、lefthook の同じ job にも付けた。
-  - svelte-check は CI でだけ走り、pre-commit では走らない。
-  - ESLint のフラグは当初は不要としたが、付けた。oxlint は `.svelte` を lint しないので、付けないと `.svelte` の `no-explicit-any` だけが止まらないため。
-- **テスト:** 挙動の分岐を足さないので test-first は省いた。各 phase で、`pnpm check` の件数・既存の E2E・変更前後の比較で確かめた。
-- **手での確認を E2E にする基準:** テストが無く、ライブラリの更新で何度も流す価値があり、安定して書けるものだけにした。当てはまったのは2件。
-  - `votes.spec.ts`: フラスコのアイコンに focus すると Tooltip が開く。そのために、seed の `VOTE_STATS_DEMO_DATA` に、PENDING の `zone2021_f` を足した（中央値は本番のデータの D3）。
-  - `workbook_create.spec.ts`（新規）: admin の作成画面で、`userId`・`isOfficial`・「問題集の種類」の初期値を見る。送信しないので DB は汚れない。
-  - 画面の文字の比較はデータで変わり、同じ route の中での移動は経路が無いので、テストにしなかった。
-- **ルールへの追記:** 今回の教訓を英語で1行ずつ足した。警告でも CI が落ちることと ESLint 側のルール（AGENTS.md）、Superforms と getter（`svelte-runes.md`）、Tooltip のフォーカス（`accessibility.md`）、`rg` の `|`（`coding-style.md`）。
-
-## 事実（コードや git に残らない）
-
-- `let x = data.x` は、Svelte 4 の時も追従していなかった（追従させるには `$:` が要った）。runes に移した `01fffb99` は、`export let data` を `$props()` に置き換えただけ。つまり v5 で壊れたのではなく、v4 の頃からある潜在的なバグに警告が出るようになった。
-- `@dnd-kit/svelte` 0.5.1 の `createDroppable` は、`id`・`accept`・`type` を内部の `$effect` で読み直す。`KanbanBoard` は `group` に固定の文字列を渡すので、getter にしても今の動きは変わらない。
-- `workbooks/create` は admin 専用（admin 以外には 403）。そのため、`isOfficial` は常に `true` になる。
-- svelte.dev・dndkit.com には devcontainer から接続できなかった。代わりに、手元の Svelte 5.57.3 の型定義と、GitHub 上の docs の source を見た。Superforms は issue（[#667](https://github.com/ciscoheat/sveltekit-superforms/issues/667)）で確かめた。
-
-## 検証
-
-- **警告:** `pnpm check` は 30 → 16 → 11 → 10 → 0 件になった。
-- **テスト:** 各 phase で、関係する E2E と unit テスト（2994件）がすべて pass した。例外は `workbooks_list` の1件で、データが無いときに飛ばす条件付きの skip。
-- **変更前後の比較（Playwright）:**
-  - 表示: 6画面と `/workbooks/create` の本文と input に、差分は無かった。
-  - 問題集の作成・編集: input の値と送信内容（`__superform_json`）に、差分は無かった。送信は abort した。
-- **フラスコのアイコン:** 最初は、`/votes/__data.json` を書き換えてアイコンを出し、手で確かめた。実行時に `tabindex="0"` が付き、Tab でフォーカスすると Tooltip が開いて、フォーカスが離れると閉じた。その後、E2E にした。
-- **足した E2E:** 2件とも通り、E2E 全体も121件 pass した（1件は条件付きの skip）。production のコードを壊すと、2件とも失敗した。Tooltip を消すと `tabindex` が付かずに失敗し、`$form.isOfficial` の代入を消すと `isOfficial` が `false` になって失敗した。
-- **警告で失敗するか:** 警告が無ければ `check`・`lint` は exit 0 だった。`let x = data.x`、`any` を書いた `.ts`・`.svelte` を一時的に足すと exit 1 になった。lefthook と同じ形の呼び出しでも、同じ結果だった。
-- **確かめていないこと:**
-  - DB に実際に保存したときの値
-  - 同じ route の中での移動
-  - SSR の HTML で、JS が読み込まれる前のフォーカス
-  - `pnpm dev` の出力
-  - E2E は CI でコメントアウトされているので、足した E2E はローカルでしか流れない
-
-## レビュー
-
-Codex（`codex exec -s read-only`、0.166.0）に頼んだ。sandbox が起動しないので、必要なファイルは prompt に埋め込んだ。Critical・Severe の指摘は無かった。ユーザーが直すのに選んだのは、Minor の1件だけ。
-
-- **Minor（直した）:** ESLint が除外するディレクトリの `.svelte` を lefthook が渡すと、「ignored」の警告で失敗する。再現したので、lefthook の eslint に `--no-warn-ignored` を足した。該当する git 管理の `.svelte` は0件。
-- **Moderate（直していない）:** pre-commit で svelte-check が走らない。指摘どおりで、svelte-check の警告は CI でだけ止まる。`pnpm check` は staged のファイルだけに絞れず、11秒かかる。
-- **未検証（直していない）:** 編集画面 A から B へ移ると、見出しは B に追従するが、form は A のまま残りうる。初期値を固定する構造は変更前からあり、今はこの移動をする経路が無い。
-
-## 残る作業
-
-- **chunk size の警告（範囲外）:** devcontainer でだけ出る。原因は `compose.yaml` の `NODE_ENV=development` で、Vite が dev モードで build し、flowbite-svelte の使っていないコンポーネント（約 158 kB）が残る。本番では出ない。直すなら `NODE_ENV` を消す（devcontainer の作り直しが要る）。`treeshake.moduleSideEffects`・deep import・`chunkSizeWarningLimit` は、効果が小さいか、本番の設定を変えることになる。
-- **`.coderabbit.yaml`:** `src/**/*utils/**` の「pure functions」の指示がテストにも当たり、`vi.stubGlobal` が指摘された。直すなら、テストは除くと1行足す。
-- **UI の3層構造:** Tailwind → Flowbite → flowbite-svelte の見直しは、別の PR にする。flowbite-svelte は2026-04-07から更新が止まっている。
-- **`account_transfer`:** `svelte/legacy` の `run` が残っている。