fix: Consolidate resolve() arguments to satisfy eslint-plugin-svelte …

[KATO-Hiro]

…3.16.0+

• KanbanBoard, workbooks: pass pathname+search+hash as single string inside resolve() (concatenating resolve(path) + search is now rejected by no-navigation-without-resolve)
• users/edit: comment out status state and assignments until AtCoderUserValidationForm is restored
• sveltekit.md: update resolve() rule with correct pattern for query string and hash

close #2345

src/routes/users/edit/+page.svelte の TODO: コメント は 別の PR で近日中に修正するものなので意図です。
レビューの対象外として。

Summary by CodeRabbit

リリースノート

• Documentation

  • ナビゲーション設定ルールを更新しました。

• Refactor

  • URLパス解決のメカニズムを改善しました。
  • 不要なステータス管理機能を削除しました。

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 8319de80-99ed-4a2f-ac33-f783fea4db74

📥 Commits

Reviewing files that changed from the base of the PR and between 78585d3 and d2f94d2.

📒 Files selected for processing (4)

• .claude/rules/sveltekit.md
• src/routes/(admin)/workbooks/order/_components/KanbanBoard.svelte
• src/routes/users/edit/+page.svelte
• src/routes/workbooks/+page.svelte

---

📝 Walkthrough

Walkthrough

eslint-plugin-svelte 3.16.0+ のアップデートに対応し、resolve() の第1引数に path/search/hash を連結した単一文字列を渡す形式に統一しました。ナビゲーション規則ドキュメントと複数の SvelteKit ページコンポーネントの URL 構築方法を更新しています。

Changes

Cohort / File(s)	Summary
SvelteKit ナビゲーション規則の更新 .claude/rules/sveltekit.md	resolve(path) + search パターンを廃止し、resolve(path + search + hash) への統一を指示。クエリ文字列・ハッシュの取り扱いに関する具体例を追加。
URL 構築形式の統一 src/routes/(admin)/workbooks/order/_components/KanbanBoard.svelte, src/routes/workbooks/+page.svelte	連結した pathname/search/hash を resolve() の単一引数として渡すように修正。
status プロップの削除 src/routes/users/edit/+page.svelte	ページコンポーネントから Props の status フィールドと関連する state 初期化・条件分岐を削除。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

📦 Lint の声に耳傾けて
resolve() の形も整いて
eslint-plugin-svelte と手を取り
パスと検索を一つにまとめ
スッキリと統一、エラーなし 🎉

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PRのタイトルは、eslint-plugin-svelteの更新に対応してresolve()の引数を統合する主要な変更を正確に要約しており、変更内容と一致している。
Linked Issues check	✅ Passed	すべてのコード変更が、issue #2345で指定されたeslint-plugin-svelteの``no-navigation-without-resolveルール対応という要件を満たしている。
Out of Scope Changes check	✅ Passed	変更はすべてissue #2345のeslint-plugin-svelte対応に関連しており、スコープ外の変更は見当たらない。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

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

---

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.}

[KATO-Hiro]

LGTM