Feature: Add Tests for Search and SyncNotion Actions

[128na]

SearchActionおよびSyncNotion Actionに対するテストを追加しました。また、SearchActionにおける除外検索の論理バグを修正しました。

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[gemini-code-assist]

Code Review

This pull request groups search query conditions within closures in SearchAction to ensure correct logical operator precedence, and introduces comprehensive feature tests for both SearchAction and SyncAction. Feedback suggests skipping empty strings during keyword parsing to prevent redundant LIKE '%%' queries, and avoiding the use of reflection on external library classes in tests to reduce fragility.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

キーワードに複数の連続したスペースが含まれている場合、explode(' ', $keyword) によって空文字列 "" が生成されます。このとき、str_starts_with($word, '-') は false となり、else ブロックが実行されて LIKE '%%' という不要なクエリが追加されてしまいます。

これを防ぐために、ループの先頭で空文字列をスキップすることをお勧めします。

また、日本語環境では全角スペース（　）が区切り文字として使われることが多いため、将来的に全角スペースへの対応（preg_split への変更など）も検討するとより良くなります。

                if ($word === '') {
                    continue;
                }
                if (str_starts_with($word, '-')) {
                    $word = trim(substr($word, 1));
                    if ($word !== '' && $word !== '0') {

[gemini-code-assist]

テスト内でリフレクション（ReflectionClass や ReflectionProperty）を使用して、外部ライブラリ（Notion SDK）の内部プロパティや読み取り専用プロパティを強制的に設定しています。

これはライブラリの内部実装に強く依存するため、ライブラリのアップデートやPHPのバージョンアップ（特に readonly プロパティの挙動変更など）によってテストが突然破損するリスクがあります。

可能であれば、PropertyCollection も Mockery でモックするか、ライブラリが提供する公式のファクトリメソッドやビルダーを使用することを検討してください。

[128na]

AIレビューの指摘内容が妥当であったため、以下の通り修正を行いました。

1. \SearchAction.php: キーワード検索時に空文字が渡された場合、スキップ処理（\continue;\）を追加し、不要な \LIKE '%%'\ クエリが発生しないよう改善しました。
2. \SyncActionTest.php: 外部ライブラリ（Notion SDK）の private プロパティ等を Reflection を使って強引にモックしていた箇所を廃止し、SDK に用意されている \Database::fromArray()\ などの公式なファクトリメソッドを活用したモックに置き換えることで、テストの堅牢性を高めました。

[128na]

AIレビューの指摘内容が妥当であったため、以下の通り修正を行いました。

1. \SearchAction.php: キーワード検索時に空文字が渡された場合、スキップ処理（\continue;\）を追加し、不要な \LIKE '%%'\ クエリが発生しないよう改善しました。
2. \SyncActionTest.php: 外部ライブラリ（Notion SDK）の private プロパティ等を Reflection を使って強引にモックしていた箇所を廃止し、SDK に用意されている \Database::fromArray()\ などの公式なファクトリメソッドを活用したモックに置き換えることで、テストの堅牢性を高めました。