feat: add URL param persistence, admin visibility, and filter fixes for workbooks list

- Restore filter state via nav link click instead of direct URL navigation in E2E test
- Pass includeUnpublished flag to getAvailableSolutionCategories for admin users
- Fix empty state check in CreatedByUserTable to use visible (readable) count
- Add keyed #each blocks in CurriculumWorkBookList and SolutionWorkBookList
- Move import to top of kanban.ts to fix declaration order
- Update coding-style rule with CodeRabbit Findings exception
- Fix code block formatting in svelte-components rule
- Add E2E note about avoiding src/ imports in testing rule

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: KATO-Hiro

.claude/rules/coding-style.md

@@ -75,6 +75,8 @@ Shared helper functions (used by two or more exports) should be grouped at the e
 
 Write all project documentation (plans, dev-notes, guides, refactor notes) in Japanese. Write all source code comments, TSDoc, commit messages, and test titles in English. This keeps documentation readable for the team while keeping code comments universally accessible and searchable.
 
+**Exception**: The `## CodeRabbit Findings` section in `refactor.md` must quote findings verbatim in their original language (English). Do not translate CodeRabbit output.
+
 ### TSDoc
 
 Add TSDoc comments to every exported function, type, and class. The minimum required fields are `@param` (for non-obvious parameters) and `@returns` (when the return value is not evident from the type). One-liner `/** ... */` is sufficient for simple cases; use multi-line only when behavior needs explanation.

.claude/rules/svelte-components.md

@@ -41,10 +41,12 @@ When copying button styles from a reference component, always check all three ax
 
 Plain `let` or `const` in Svelte 5 component `<script>` executes once at component creation. Values derived from props or server data must use `$derived()`:
 
-```svelte
-// Bad: captures only the initial value — won't update when data reloads let user =
-data.loggedInUser; const categories = availableCategories.filter(...); // Good let user =
-$derived(data.loggedInUser); let categories = $derived(availableCategories.filter(...));
+```md
+// Bad: captures only the initial value — won't update when data reloads
+let user = data.loggedInUser; const categories = availableCategories.filter(...);
+
+// Good
+let user = $derived(data.loggedInUser); let categories = $derived(availableCategories.filter(...));
 ```
 
 `pnpm check` warns: "This reference only captures the initial value."

.claude/rules/testing.md

@@ -153,7 +153,10 @@ test.describe('logged-in user', () => {
 Playwright has no native `test.each`. Use `for...of` loops — the official recommended pattern:
 
 ```typescript
-for (const grade of [TaskGrade.Q10, TaskGrade.Q9, TaskGrade.Q8]) {
+// Mirrors TaskGrade from $lib/types/task — do not import from src/ in E2E files
+const GRADES = ['Q10', 'Q9', 'Q8'] as const;
+
+for (const grade of GRADES) {
   await gradeButton(grade).click();
   await expect(page).toHaveURL(`?grades=${grade}`);
 }

e2e/workbooks_list.spec.ts

@@ -163,8 +163,8 @@ test.describe('logged-in user (general)', () => {
       await page.goto('/');
       await expect(page).toHaveURL('/', { timeout: TIMEOUT });
 
-      // Return to /workbooks via nav link (no params)
-      await page.goto(WORKBOOK_LIST_URL);
+      // Return to /workbooks by clicking the nav link (no params)
+      await page.getByRole('link', { name: '問題集', exact: true }).click();
 
       // URL should be restored to the saved filter state
       await expect(page).toHaveURL(new RegExp(`tab=${TAB_SOLUTION}`), { timeout: TIMEOUT });

src/features/workbooks/components/list/CreatedByUserTable.svelte

@@ -21,9 +21,13 @@
   import { getTaskResult } from '$features/workbooks/utils/workbooks';
 
   let { workbooks, userId, role, taskResults }: SolutionTableProps = $props();
+
+  let visibleCount = $derived(
+    workbooks.filter((workbook) => canRead(workbook.isPublished, userId, workbook.authorId)).length,
+  );
 </script>
 
-{#if workbooks.length === 0}
+{#if visibleCount === 0}
   <EmptyWorkbookList />
 {:else}
   <div class="overflow-auto rounded-md border border-gray-200 dark:border-gray-100">

src/features/workbooks/components/list/CurriculumWorkBookList.svelte

@@ -72,7 +72,7 @@
 <div class="mb-6">
   <div class="flex items-center space-x-4">
     <div class="flex flex-wrap gap-1">
-      {#each AVAILABLE_GRADES as grade}
+      {#each AVAILABLE_GRADES as grade (grade)}
         <Button
           onclick={() => filterByGradeMode(grade)}
           color="alternative"

src/features/workbooks/components/list/SolutionWorkBookList.svelte

@@ -47,7 +47,7 @@
 </script>
 
 <div class="mb-6 flex flex-wrap gap-1">
-  {#each AVAILABLE_CATEGORIES as category}
+  {#each AVAILABLE_CATEGORIES as category (category)}
     <Button
       onclick={() => onCategoryChange(category)}
       color="alternative"

src/features/workbooks/services/workbooks.test.ts

@@ -267,6 +267,37 @@ describe('getAvailableSolutionCategories', () => {
 
     expect(result).toEqual([SolutionCategory.GRAPH]);
   });
+
+  test('passes isPublished: true filter by default', async () => {
+    vi.mocked(prisma.workBookPlacement.findMany).mockResolvedValue([]);
+
+    await getAvailableSolutionCategories();
+
+    expect(prisma.workBookPlacement.findMany).toHaveBeenCalledWith(
+      expect.objectContaining({
+        where: expect.objectContaining({
+          workBook: expect.objectContaining({ isPublished: true }),
+        }),
+      }),
+    );
+  });
+
+  test('omits isPublished filter when includeUnpublished is true', async () => {
+    vi.mocked(prisma.workBookPlacement.findMany).mockResolvedValue([
+      { solutionCategory: SolutionCategory.GRAPH },
+    ] as unknown as Awaited<ReturnType<typeof prisma.workBookPlacement.findMany>>);
+
+    const result = await getAvailableSolutionCategories(true);
+
+    expect(result).toEqual([SolutionCategory.GRAPH]);
+    expect(prisma.workBookPlacement.findMany).toHaveBeenCalledWith(
+      expect.objectContaining({
+        where: expect.objectContaining({
+          workBook: expect.not.objectContaining({ isPublished: expect.anything() }),
+        }),
+      }),
+    );
+  });
 });
 
 describe('getWorkBook', () => {

src/features/workbooks/services/workbooks.ts

@@ -120,13 +120,20 @@ export async function getWorkBooksCreatedByUsers(): Promise<WorkbooksWithAuthors
 }
 
 /**
- * Returns the list of SolutionCategory values that have at least one published
- * SOLUTION workbook with a placement record.
+ * Returns the list of SolutionCategory values that have at least one SOLUTION
+ * workbook with a placement record.
+ *
+ * @param includeUnpublished - When true, includes categories from unpublished workbooks. Defaults to false.
  */
-export async function getAvailableSolutionCategories(): Promise<SolutionCategories> {
+export async function getAvailableSolutionCategories(
+  includeUnpublished = false,
+): Promise<SolutionCategories> {
   const placements = await db.workBookPlacement.findMany({
     where: {
-      workBook: { isPublished: true, workBookType: WorkBookTypeConst.SOLUTION },
+      workBook: {
+        ...(includeUnpublished ? {} : { isPublished: true }),
+        workBookType: WorkBookTypeConst.SOLUTION,
+      },
       solutionCategory: { not: null },
     },
     select: { solutionCategory: true },

src/routes/(admin)/workbooks/order/_types/kanban.ts

@@ -1,5 +1,6 @@
 import type { DragDropManager, Draggable, Droppable } from '@dnd-kit/dom';
 import type { DragDropEvents } from '@dnd-kit/abstract';
+import type { WorkBookTab } from '$features/workbooks/types/workbook';
 
 // DnD event types derived from dnd-kit abstractions
 type DndEvents = DragDropEvents<Draggable, Droppable, DragDropManager>;
@@ -9,8 +10,6 @@ export type DragEndEventArg = Parameters<DndEvents['dragend']>[0];
 
 export type ColumnKey = 'solutionCategory' | 'taskGrade';
 
-import type { WorkBookTab } from '$features/workbooks/types/workbook';
-
 /** Tabs available on the admin order page — excludes CREATED_BY_USER which has no placement config. */
 export type ActiveTab = Exclude<WorkBookTab, 'created_by_user'>;