fix: implement CodeRabbit findings (#3364)

[KATO-Hiro]

close #3364

Summary by CodeRabbit

リリースノート

• 新機能

  • 投票送信時のキャンセル機能とリアルタイムデータ更新を追加
  • フォーム送信中の状態表示を改善
  • 検証コードのクリップボードコピーをUI改善

• バグ修正

  • タスク更新時の エラーハンドリングを強化（見つからないタスクへの対応）
  • 投票送信レスポンスを明示的に返すよう修正
  • 投票詳細ページへのナビゲーション安定性を向上

• データベース

  • 投票データの整合性を確保するCHECK制約を追加

[coderabbitai]

📝 Walkthrough

概要

投票機能の内部クライアントヘルパー化、フォーム送信状態の統一的な管理、データベース制約の追加、関連サービスのエラーハンドリング強化、および複数のコーディング・テストガイドラインの整理を行うPRです。

変更内容

コホート / ファイル(s)	概要
投票機能：クライアントヘルパー src/features/votes/internal_clients/vote_grade.ts, src/features/votes/internal_clients/vote_grade.test.ts	新規モジュール。fetchMyVote, fetchMedianVote, submitVote の3つの非同期ヘルパーを導出し、投票データ取得・送信をカプセル化。AbortSignal 対応。
投票機能：コンポーネント & サービス更新 src/features/votes/components/VotableGrade.svelte, src/features/votes/services/vote_grade.ts, src/features/votes/actions/vote_actions.ts	VotableGrade で直接 fetch を廃止し、新ヘルパーと AbortController を使用。サービスの戻り値を Promise<{ success: true }> に更新。投票後に中央値を再取得する制御フロー。
投票定数の一元化 src/features/votes/constants/statistics.ts, src/features/votes/utils/median.ts, src/routes/votes/[slug]/+page.svelte	MIN_VOTES_FOR_STATISTICS 定数を新規定義し、ハードコーディング値 3 を統一管理。
タスク更新サービス：エラーハンドリング src/lib/services/tasks.ts, src/test/lib/services/tasks.test.ts	Prisma P2025 エラー（レコード未検出）を null に変換。呼び出し側で 404 相応の処理が可能に。テスト追加。
フォーム送信状態管理 src/lib/components/FormWrapper.svelte, src/lib/components/SubmissionButton.svelte	FormWrapper が Svelte context 経由で isSubmitting を提供。SubmissionButton が context 読取 + ローカル loading prop とマージ。
データベース制約 prisma/schema.prisma, prisma/ERD.md, prisma/migrations/20260409000000_fix_vote_grade_check_constraints/migration.sql	投票テーブルへの CHECK 制約を追加：grade != 'PENDING'、VotedGradeCounter.count >= 0。スキーマコメントで制約を説明。
型・定数の整理 src/lib/types/flowbite-svelte-wrapper.ts, src/lib/utils/task_grade.ts, src/lib/utils/task_grade.test.ts	ButtonColor を Flowbite Svelte の型から派生。isValidTaskGrade 型ガード関数を新規実装。テスト追加。
ルートハンドラの条件分岐 src/routes/(admin)/tasks/+page.server.ts, src/routes/(admin)/vote_management/+page.server.ts	updateTask 戻り値を検査し、null の場合は success: false を返す制御追加。
クリップボード UI 更新 src/features/account/components/settings/AtCoderVerificationForm.svelte	Flowbite Svelte Clipboard コンポーネントで置換。カスタム copy ロジックを廃止。
テスト拡充 src/features/account/services/atcoder_verification.test.ts, src/test/lib/services/users.test.ts	テスト構造を整理。vi.stubEnv で環境変数管理。Prisma mock ヘルパー拡張。
ドキュメント & ガイドライン整理 .claude/rules/coding-style.md, .claude/rules/prisma-db.md, .claude/rules/svelte-components.md, .claude/rules/svelte-runes.md, .claude/rules/sveltekit.md, .claude/rules/testing-e2e.md, .claude/rules/testing.md	複数のコーディングガイドを再構成：死活コード削除ルール、FK 関係明示化、テストパターン標準化、SSR 安全性、AbortController 活用等を詳述。
その他 & CI/CD .github/workflows/resolve-migration.yml, docs/dev-notes/2026-04-09/fix-vote-grade-check-constraints/plan.md, e2e/votes.spec.ts, vite.config.ts	Migration 失敗時の一時的な手動解決ワークフローを追加。E2E テストの ID 定数化。Vitest include パターンに src/lib を追加。

推定レビュー労力

🎯 3 (中程度) | ⏱️ ~25 分

理由： 複数の小～中規模な機能変更（投票ヘルパー化、フォーム状態管理、エラーハンドリング）と新規テストが含まれ、各領域で独立した判断が必要。一方、大部分がドキュメント更新と境界が明確な機能追加であり、制御フロー自体の複雑性は中程度。

詩

🎯 投票のフロー、堅牢に
AbortController で優雅にキャンセル
制約はDB で把守し
型ガード、null チェック、力を合わせ
スキーマ、テスト、ドキュメント
すべてが一つの方向へ — 信頼へ

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

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

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR タイトルは「fix: implement CodeRabbit findings (#3364)」で、コードレビューツールの指摘を実装する変更内容を明確に表現している。変更セット全体（ドキュメント更新、テスト改善、マイグレーション修正、ユーティリティの追加）の主な目的を適切に要約している。
Linked Issues check	✅ Passed	Issue #3364 は「レビューツールからの指摘事項のうち妥当だと判断したものを反映させましょう」で、PR ではドキュメント改善（Prisma 制約ガイダンス、テスト ドキュメント修正）、テスト強化（新パターン追加、テストファイル再配置、vite 設定更新）、コード修正（MIN_VOTES_FOR_STATISTICS 利用、globalThis リーク防止、getBaseUrl 簡略化）、マイグレーション修正を実装しており、全て要件を満たしている。
Out of Scope Changes check	✅ Passed	GitHub Actions ワークフロー（resolve-migration.yml）追加とマイグレーションドキュメント（plan.md）は、staging DB の P3009 スタック状態の一時的な解決と密接に関連しており、vote grade CHECK 制約実装を支援するための必要な変更。その他の変更も全て指摘事項への対応範囲内であり、スコープ外の変更は認識されない。

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch #3364

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 7

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.claude/rules/prisma-db.md:
- Around line 99-102: The policy text conflicts about where to record DB-level
CHECK constraints; pick and state one canonical source (either inline
schema.prisma comments or ERD.md) and update the "DB-Level Value Constraints"
section to reflect that single choice: if choosing schema.prisma, remove the
claim that "ERD.md is the only visualization target", instruct authors to add
manual migration SQL plus inline comments like "/// CHECK: count >= 0" and note
that prisma-erd-generator may overwrite ERD.md; if choosing ERD.md, remove the
instruction to record constraints in schema.prisma, require adding constraints
to manual migration SQL and documenting them in ERD.md, and add a short NOTE
about prisma-erd-generator behavior. Reference the section headers "DB-Level
Value Constraints", the file/comment pattern "schema.prisma" and the tool name
"prisma-erd-generator" when making the change.

In @.claude/rules/testing-e2e.md:
- Line 37: Update the heading text "Parametrized Tests" to correct the spelling:
change "Parametrized" to "Parameterized" so the heading reads "Parameterized
Tests"; locate the heading string in the file where the current heading symbol
"Parametrized Tests" appears and replace it accordingly.

In @.claude/rules/testing.md:
- Line 78: ヘッダー文字列 "Parametrized Tests" の綴りが誤っているため、該当ヘッダーを "Parameterized
Tests" に修正してください（該当テキストはファイル内の見出し "Parametrized Tests" を探して置換する形で対応してください）。

In `@src/features/votes/constants/statistics.ts`:
- Around line 1-5: The default hardcoded minVotes in median.ts should be
replaced with the shared constant to centralize the threshold: import and use
MIN_VOTES_FOR_STATISTICS from src/features/votes/constants/statistics in the
computeMedianGrade function (median.ts) instead of a literal number, update the
function signature/default parameter and any internal references to use
MIN_VOTES_FOR_STATISTICS so the same threshold is used across vote_grade.ts and
median.ts.

In `@src/features/votes/internal_clients/vote_grade.test.ts`:
- Around line 13-24: The tests overwrite globalThis.location in beforeEach; save
the original value (e.g., const originalLocation = globalThis.location) before
assigning { origin: BASE_URL } and restore it in afterEach (globalThis.location
= originalLocation) so tests do not leak state; keep the existing
nock.cleanAll() calls and ensure the restore handles undefined originals safely.

In `@src/features/votes/internal_clients/vote_grade.ts`:
- Around line 86-101: The getBaseUrl() fallback is overly complex and includes
an effectively unreachable 'http://localhost' value; simplify by returning the
browser origin when available via globalThis.location?.origin, and otherwise
returning an empty string for SSR/unknown contexts. Update getBaseUrl to check
globalThis.location?.origin and return that if present, otherwise return ''
(remove the 'http://localhost' fallback), and preserve the existing console
warning when browser is false so callers clearly get an empty string for
server-side usage; reference getBaseUrl, globalThis.location, and browser to
locate the change.

In `@src/lib/utils/task_grade.ts`:
- Around line 8-10: Move the unit test for isValidTaskGrade out of the central
test folder and create an adjacent test file next to the implementation (e.g.,
task_grade.test.ts) so the function has an adjacent unit test; ensure the test
imports isValidTaskGrade and taskGradeValues and verifies valid/invalid inputs
and purity (no side effects) to satisfy the coding guideline requiring pure
utils with adjacent tests.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9924d1b0-3287-4019-ba9c-11fbb76bae4f

📥 Commits

Reviewing files that changed from the base of the PR and between 794f5d3 and 4f16134.

📒 Files selected for processing (30)

• .claude/rules/coding-style.md
• .claude/rules/prisma-db.md
• .claude/rules/svelte-components.md
• .claude/rules/svelte-runes.md
• .claude/rules/sveltekit.md
• .claude/rules/testing-e2e.md
• .claude/rules/testing.md
• e2e/votes.spec.ts
• prisma/ERD.md
• prisma/migrations/20260406112057_add_vote_grade_check_constraints/migration.sql
• prisma/schema.prisma
• src/features/account/components/settings/AtCoderVerificationForm.svelte
• src/features/account/services/atcoder_verification.test.ts
• src/features/votes/actions/vote_actions.ts
• src/features/votes/components/VotableGrade.svelte
• src/features/votes/constants/statistics.ts
• src/features/votes/internal_clients/vote_grade.test.ts
• src/features/votes/internal_clients/vote_grade.ts
• src/features/votes/services/vote_grade.ts
• src/lib/components/FormWrapper.svelte
• src/lib/components/SubmissionButton.svelte
• src/lib/services/tasks.ts
• src/lib/types/flowbite-svelte-wrapper.ts
• src/lib/utils/task_grade.ts
• src/routes/(admin)/tasks/+page.server.ts
• src/routes/(admin)/vote_management/+page.server.ts
• src/routes/votes/[slug]/+page.svelte
• src/test/lib/services/tasks.test.ts
• src/test/lib/services/users.test.ts
• src/test/utils/task_grade.test.ts

[KATO-Hiro]

一旦、merge しないと CI の マイグレーションエラーが解消できないので、マージします

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/lib/utils/task_grade.test.ts`:
- Around line 14-22: The test for isValidTaskGrade currently passes implicitly
when the guard returns false; update the test that uses isValidTaskGrade(value)
so it explicitly fails when the guard is false by asserting the guard result
before using the narrowed value (e.g.,
expect(isValidTaskGrade(value)).toBe(true) or call a failure helper), then
continue with the narrowed-variable checks (referencing isValidTaskGrade and
TaskGrade in the same test) to ensure the test actually fails if the type guard
returns false.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 944d5e27-092b-4092-ad8f-be0cb42a49a9

📥 Commits

Reviewing files that changed from the base of the PR and between 4f16134 and bae4e80.

📒 Files selected for processing (11)

• .claude/rules/prisma-db.md
• .claude/rules/testing-e2e.md
• .claude/rules/testing.md
• .github/workflows/resolve-migration.yml
• docs/dev-notes/2026-04-09/fix-vote-grade-check-constraints/plan.md
• prisma/migrations/20260409000000_fix_vote_grade_check_constraints/migration.sql
• src/features/votes/internal_clients/vote_grade.test.ts
• src/features/votes/internal_clients/vote_grade.ts
• src/features/votes/utils/median.ts
• src/lib/utils/task_grade.test.ts
• vite.config.ts

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

型ガードテストで false 時の失敗ハンドリングがない。

isValidTaskGrade(value) が false を返した場合、テストは暗黙的にパスしてしまう。

🔧 修正案

     test('is a type guard that narrows type to TaskGrade', () => {
       const value: unknown = 'Q1';

       if (isValidTaskGrade(value)) {
         // value is narrowed to TaskGrade at this point
         const grade: typeof value = value;
         expect(grade).toBe(TaskGrade.Q1);
+      } else {
+        expect.fail('Expected isValidTaskGrade to return true for Q1');
       }
     });

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/lib/utils/task_grade.test.ts` around lines 14 - 22, The test for
isValidTaskGrade currently passes implicitly when the guard returns false;
update the test that uses isValidTaskGrade(value) so it explicitly fails when
the guard is false by asserting the guard result before using the narrowed value
(e.g., expect(isValidTaskGrade(value)).toBe(true) or call a failure helper),
then continue with the narrowed-variable checks (referencing isValidTaskGrade
and TaskGrade in the same test) to ensure the test actually fails if the type
guard returns false.