fix: Livewire Pagesコンポーネントのページ状態管理を一本化

[128na]

Summary

• WithPaginationトレイトの内部状態($paginators)と、SearchActionに渡す$pageプロパティが二重管理になっており、render()冒頭のresetPage()は$paginatorsしか操作しないため実質何もしていなかった。結果、検索条件（キーワード/pak/サイト）を変更しても表示ページ番号がリセットされず、フィルタ後の件数によっては空ページが表示されうるバグがあった。
• WithPaginationを削除し、#[Url] public int $pageのみをページ状態の唯一の情報源にすることでこのバグを修正。
• onConditionUpdate()がrender()を直接呼んでViewを返す設計をやめ、状態変更（ページを1に戻す）に専念させ、再描画はLivewireのライフサイクルに委譲。
• 検索結果を#[Computed] pages()に切り出し、render()を単純化。
• $keywordに#[Validate('string|max:191')]を追加し、Blade側にエラー表示を追加。
• Pagesコンポーネントのテストが存在しなかったため新規に6件追加（ページリセットのバグ修正を検証するテストを含む）。

E:\chore\best_practice\livewire.md のアンチパターンチェックリスト(1,2,4,5,6)に基づく対応。

Test plan

• ./vendor/bin/pint --test
• ./vendor/bin/phpstan analyse（level 9, 全体, エラーなし）
• ./vendor/bin/rector process --dry-run（提案を適用済み）
• php artisan test（63件全て通過、新規6件含む）

🤖 Generated with Claude Code

[128na]

セルフレビュー結果（8観点・並列調査＋検証）

自動セルフレビューで以下を検出し、対応済み。

重大: WithPagination除去でページネーションリンクが405になる回帰

use WithPagination; を外した際、その内部で行われていた Paginator::currentPathResolver の差し替え（Livewire::originalPath()）まで一緒に失っていた。これにより検索・フィルタ操作を1回でも行った後は、ページネーションリンクの遷移先がLivewireの内部updateエンドポイント（POST専用）になり、通常のリンククリック（GET）が405になっていた。boot() を無効化した状態と有効化した状態を実際にテストで比較して再現・修正を確認済み（href="http://localhost/livewire-xxx/update?page=2" → 修正後は正しいページURLに戻る）。回帰防止テストを追加。

中: 負のページ番号が未クランプ

?page=-1 のような値がそのまま SearchAction/paginate() に渡っていた。max(1, $this->page) でクランプ。

中: keywordバリデーションの最大文字数が既存API側と不一致

新規追加した #[Validate('string|max:191')] が、同じ検索機能の既存API側バリデーション（PageSearchRequest::rules() の keyword => 'present|max:20'）と食い違っていた。max:20 に統一。

軽微: 新規テストの指摘2件

不要なアクション呼び出しの削除、テスト名が謳う sites リセットの検証漏れを解消。

対応見送り（既存の制限、スコープ外として明記）

検索条件（keyword/paks/sites）自体が #[Url] 化されておらず、ページネーションのプレーンリンク遷移時にURLへ乗らず消える既存の制限を発見。本PRのスコープ（ページ番号の二重管理解消）外のため、コード上にコメントで明記するに留めた。別issue化を検討。

🤖 Generated with Claude Code