Skip to content

PDO移行ExecPlanの明確化:インストーラーのAJAX処理、リポジトリ検索コマンド、期待結果 - #469

Open
yama wants to merge 2 commits into
feature/pdo-migration-execplanfrom
codex/2026-09-22-github-mention-docsplan-pdodbapiexecplan
Open

yama wants to merge 2 commits into
feature/pdo-migration-execplanfrom
codex/2026-09-22-github-mention-docsplan-pdodbapiexecplan

Conversation

@yama

@yama yama commented Sep 22, 2026 •

Copy link
Copy Markdown
Member

動機

  • インストーラーの接続テスト用エンドポイントと $conn への直接アクセスについて、レビューで判明した挙動を明確化・文書化し、PDO移行作業で public な $conn プロパティを安全に削除できるようにする。
  • リポジトリ横断の検索・検証手順について、grep ではなく rg(ripgrep)を使うよう更新し、期待される検索結果もより具体的に記載する。
  • 互換性チェック、PHP拡張チェック、ドキュメント上の前提を新しいPDOベースの方針に合わせる。これまで mysqli を参照していた箇所では pdo_mysql を使用する。

変更内容

  • ExecPlan の Markdown ファイル 2026-07-08-pdo-migration-dbapi-refactor.md を更新し、インストーラー関連で判明した内容を追記した。特に、接続テスト用エンドポイントが install/tpl/connection.tpl から呼び出される AJAX の送信先であることを明記した。

  • 計画内の grep 使用例を rg(ripgrep)に置き換え、それに合わせて期待される検索結果も調整した。

  • Steps および Validation セクションに、rg 検索、php -l による構文チェック、docker compose のヘルスチェック/CLIチェックについて、期待される結果を明示的に追加した。

  • いくつかの説明文も更新した。

    • ->conn の検索範囲を rg を使ったリポジトリ全体の検索に変更。
    • 引き続き対象外とするファイルとして、非推奨の DocumentParser::dbConnect() を明確化。
    • 一部ファイルおよび health:check では、mysqli の文字列チェックを pdo_mysql の拡張チェックへ置き換えることを期待事項として追加。

テスト

  • 今回はドキュメントのみの変更であるため、自動テストは実行していない。
  • このPRは、今後実施するコード変更および検証作業の手順と期待結果を更新するもの。

Codex Task:
https://chatgpt.com/codex/cloud/tasks/task_e_6a4da963479c832dbeb70ac6e1c27b3b

@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: modxcms-jp/evolution-jp/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: d3c49e41-d6f3-425b-9ea0-fe0418c6d8a2

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a12f2be423

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


これらは実処理に影響しないため、Step 1〜5完了後に更新する。

期待される観測結果: `rg -n "extension_loaded\\('mysqli'\\)" manager/includes/cli install manager/includes/default.config.php` が0件になり、`docker compose exec <app-service> php evo health:check` が `pdo_mysql` 拡張チェックを含めて成功する。

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Step 6 の全変更対象を検証する

Step 6 で CLI の2ファイルだけを更新し、install/instprocessor.php の database_type = 'mysqli' と manager/includes/default.config.php の mysqli_set_charset 判定を残した場合でも、この rg は extension_loaded('mysqli') しか検索しないため0件となり、health:check も成功します。その結果、必須として列挙した残り2変更が未実施でも検証を通過するので、database_type と charset 関数の旧表記、または各ファイルの期待する pdo_mysql 表記も個別に検査してください。

AGENTS.md reference: AGENTS.md:L3-L5

Useful? React with 👍 / 👎.

@yama yama changed the title Clarify PDO migration ExecPlan: installer AJAX handling, repo search commands, and expectations PDO移行ExecPlanの明確化:インストーラーのAJAX処理、リポジトリ検索コマンド、期待結果 Sep 22, 2026
…ithub-mention-docsplan-pdodbapiexecplan

PDO移行ExecPlanのコンフリクトを解消:
- $conn直接アクセス: PHP全体へのrg確認と「計3箇所」を併記
- Step 3: PHP全体の再検索を含むPR側の観測結果を採用
- Step 6: ベース側の検索パターンにdefault.config.phpを追加し、
  database_typeの照合を `'database_type'] =` に合う形へ修正
- Step 7: フルパス照合のPR側を採用

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AZ1r9wx8cJk5E211ZW9Nxb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant