ci(agent-commerce): 結合 E2E の依存プラグイン参照を EC-CUBE org の 4.4 ブランチへ切り替える - #7056
ci(agent-commerce): 結合 E2E の依存プラグイン参照を EC-CUBE org の 4.4 ブランチへ切り替える#7056nanasess wants to merge 1 commit into
Conversation
eccube-api4#191 と sample-payment-plugin#54 が両リポジトリの 4.4 ブランチへ マージされたため、開発期間中に参照していた fork を org 参照に差し替える。 - API44_VCS / SAMPLEPAYMENT44_VCS を EC-CUBE org の URL に変更 - 参照ブランチの既定値を feature/agentic-commerce から 4.4 に変更 (workflow_dispatch / workflow_call の inputs、Job A/B のフォールバック) path リポジトリ方式とダミー extra.id の注入は org 移行後も引き続き必要なため維持した。 両リポジトリのデフォルトブランチは 4.2 系で composer 名も旧名 (ec-cube/api42 / ec-cube/samplepayment42) のため、VCS ドライバではデフォルトブランチ由来の名前で repo がキーされ、新名の 4.4 ブランチが名前不一致でスキップされる。また api44 は 4.4 ブランチでも composer.json に extra.id を持たない。コメントの理由書きのみ 実態に合わせて更新している。 あわせて e2e/agent のドキュメントが実装と食い違っていた箇所を修正する。 - landing 待ちを前提にした記述を、landing 済みかつ skip ゲートはローカル実行用と いう位置づけに更新 (README / acp-checkout.php の docblock・skip メッセージ・TODO) - ローカル実行手順の router を codeception/router.php から index.php に修正。 router.php は GET に対し false を返し、静的フォールバックの挙動が PHP バージョン 依存で 404 になり得るため、CI と同じく front controller を直接 router にする - AGENT_E2E_ITEM_ID の既定値の記述を修正。README は ACP=1 としていたが実装は ACP/UCP ともに 2 で、1 は visible=0 のダミー規格のため purchase flow が明細を除去する - 統合ブランチ運用を説明していた注意書きを、コア実装のマージ経緯の記載に差し替え Co-Authored-By: Claude <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughAgentic Commerce E2E の参照先を EC-CUBE の ChangesAgentic Commerce 4.4 移行
Estimated code review effort: 2 (Simple) | ~10 minutes Mergeability Score: ⚪ Minimal · up to This PR updates E2E dependency references and aligns workflow defaults and documentation with the 4.4 branch; no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 4.4 #7056 +/- ##
==========================================
+ Coverage 77.67% 77.75% +0.07%
==========================================
Files 597 597
Lines 29333 29333
==========================================
+ Hits 22785 22808 +23
+ Misses 6548 6525 -23
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
1 participant
概要(Overview・Refs Issue)
エージェントコマース結合 E2E (
agentic-commerce-e2e.yml) が参照している依存プラグインを、開発期間中の fork から EC-CUBE org の4.4ブランチへ切り替えます。以下 2 件がマージされ、fork を参照する必要がなくなったためです。
4.4(merge commitbca124e6)4.4(merge commitd5ab9b33)あわせて、
e2e/agent/のドキュメントが実装と食い違っていた箇所を修正します（#6872 で本体にマージされた後、記述が実態と乖離していました）。方針(Policy)
path リポジトリ方式とダミー
extra.idの注入は org 移行後も必要なため維持しました。 「org に移ったので通常の VCS 参照に戻せる」とは判断できず、実際に確認した結果は次のとおりです。4.4の composer 名4.4のextra.id4.2ec-cube/api42ec-cube/api44(4.4.0)4.2ec-cube/samplepayment42ec-cube/samplepayment44(3.0.0)9999994.4ブランチが名前不一致でスキップされ "not found" になります。この制約は fork 固有ではなく org でも同じでした。api44は4.4ブランチでもcomposer.jsonにextra.idを持たないため、ec-cube/plugin-installer用のダミー id 注入も引き続き必要です。そのため機構は変えず、コメントの理由書きのみ実態（デフォルトブランチが 4.2 系）に合わせて更新しています。
実装に関する補足(Appendix)
CI 設定
API44_VCS/SAMPLEPAYMENT44_VCSを EC-CUBE org の URL に変更feature/agentic-commerce→4.4に変更（workflow_dispatch/workflow_callの inputs 各 2 箇所、Job A/B のフォールバック 4 箇所）main.ymlの caller 側コメントを更新（「org 移行後に別 PR で差し替え」の TODO を解消）ドキュメントの実装との齟齬修正
landing 待ち表記の更新に加え、突き合わせで見つかった以下 3 件を修正しました。
php -S 127.0.0.1:8000 codeception/router.phpを指示していましたが、codeception/router.phpは GET に対しreturn falseを返し、PHP built-in server の静的フォールバック挙動が PHP バージョン依存で app ルートを 404 にし得ます。workflow 自身はこれを理由にindex.phpを直接 router にしているため、README も CI と同じ方式に統一しました。AGENT_E2E_ITEM_IDの既定値が誤り — README は「ACP=1/ UCP=2」としていましたが、実装はacp-checkout.php/ucp-checkout.phpともに2です。1は fixtures のvisible=0なダミー規格で purchase flow が明細を除去するため、ACP で1を既定とする記述は成立しません。feature/agentic-commerce-e2e) はテスト専用の統合ブランチ」という注記が、feat(agent-commerce): 結合 E2E を通常 CI に統合 + UCP discovery payment_handlers 自動広告 #6872 で本体 4.4 にマージされた後も残っており参照先が存在しない状態でした。コア実装のマージ経緯（feat(agent-commerce): エージェントコマース共通基盤 Phase 1a (#6777) #6802/feat(agent-commerce): Product Feed/Catalog/Discovery を実装 (#6794 トラックA) #6815/feat(agent-commerce): CheckoutSession 中核を実装 (#6777 トラックB前提・Phase 1b) #6825/feat(agent-commerce): UCP checkout (#6574) のコア実装 #6837/feat(agent-commerce): ACP checkout (#6776) のコア実装 #6843/feat(agent-commerce): 結合 E2E を通常 CI に統合 + UCP discovery payment_handlers 自動広告 #6872）の記載に差し替えました。acp-checkout.phpの TODO は、ブロッカー（トークン発行の landing）は解消済みですが作業本体（payload をAcpCheckoutSessionMapperの契約に実データで突き合わせる）は未実施のため、項目自体は残し前提条件の記述のみ更新しています。テスト（Test)
ローカルで実施済み
4.4ブランチのgit clone --depth 1 --branch 4.4（両リポジトリ・匿名 HTTPS）が成功し、HEAD が上記 merge commit そのものであることを確認composer.jsonの名前・version が workflow の require（ec-cube/api44:*/ec-cube/samplepayment44:*）と、plugin code が--code=Api44/--code=SamplePayment44と一致することを確認dev.ucp.payment.cardがUcpSampleCardHandler::HANDLER_IDに存在することを確認client_credentials/acp:checkoutucp:checkoutscope /ECCUBE_OAUTH2_ENCRYPTION_KEY）が api44 の4.4に存在することを確認4.4になることを確認php -l e2e/agent/*.php/ Rector / PHPStan level 6 いずれも green（pre-push hook で実行）CI で確認したいこと
Agentic Commerce E2Eの Job A (integration smoke, PHP 8.2-8.5 × pgsql/mysql) が green相談（Discussion）
Packagist 公開後は path リポジトリ方式ごと通常の
composer requireに単純化できます。本 PR では機構を変えず参照先の切り替えに留めました。マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目
🤖 Generated with Claude Code
Summary by CodeRabbit