Memory Wiki

I'll start by reading all the build briefs, the SA plan, the past faults log, and the existing code files that need modification.

Now let me read the existing code files that need to be modified.

Let me also check the composer.json files and README patterns:

Now I have full context. Let me start building all three tasks. First, T52 — the new google_ads_conversion module.

Now creating all T52 module files:

Now creating the test files for T52:

T52 module files created. Let me verify syntax and then proceed to T17 (WebhookHandler modifications).

T52 is clean. Now implementing T17 — WebhookHandler modifications:

Now replace the handleCheckoutSessionCompleted method and add conversion upload + retry methods:

Now add the new conversion upload methods at the end of the WebhookHandler class:

Now update the WebhookHandlerModule:

Now update the WebhookEventRepository with conversion retry methods:

Now update the webhook_handler composer.json and README:

Let me fix the README table properly:

Now update the README with conversion upload documentation:

Verify the modified webhook_handler syntax:

Now let me create/update the WebhookHandler tests. Let me check for existing test files:

Now I need to update the WebhookHandlerTest and WebhookHandlerModuleTest to cover the new conversion upload functionality:

Now update the test fixtures to add the GCLID-enabled event:

Now update the WebhookHandlerModuleTest:

Now implementing T15 — StripeService and CheckoutService modifications:

Now update the StripeService implementation:

Now update the StripeServiceModule's handleCreateCheckoutSession to accept ui_mode:

Now update CheckoutServiceInterface and CheckoutService:

Now update CheckoutServiceModule:

Verify syntax on all T15 files:

Now create/update tests for T15 modules. Let me check existing test files:

Now let me add the embedded mode tests to StripeServiceTest:

Now let me add the embedded mode result fixture:

Now add the new embedded mode tests to CheckoutServiceTest:

Now add tests to StripeServiceTest for the new embedded mode + metadata features:

Now let me run syntax checks on all modified files across all tasks:

All syntax checks clean, zero quadruple-backslash. Let me do a final comprehensive test count verification:

All test counts exceed the minimum of 12 per module. Let me provide the final summary. ## Summary — All 3 Build Briefs Implemented ### T52 — New `google_ads_conversion` Module **Files created** (under `/root/profe-modules/google_ads_conversion/`): - `src/GoogleAdsConversion/GoogleAdsConversionInterface.php` — Interface contract (3 methods: `uploadConversion()`, `isReady()`, `hashEmailForEnhancedConversions()`) - `src/GoogleAdsConversion/GoogleAdsConversionService.php` — Implementation with credential validation, click conversion building, email hashing - `src/GoogleAdsConversion/GoogleAdsConversionModule.php` — Module registration with DI wiring - `src/GoogleAdsConversion/Exception/GoogleAdsConversionException.php` — Typed exception with `fromApiError()`, `credentialsNotConfigured()`, `tokenRefreshFailed()` factory methods - `composer.json` — PSR-4 autoloading, profe module config - `README.md` — Config keys, usage, exceptions, fault compliance - `tests/Unit/GoogleAdsConversionServiceTest.php` — 17 tests (isReady, hashEmail, uploadConversion) - `tests/Unit/GoogleAdsConversionModuleTest.php` — 11 tests - `tests/Unit/GoogleAdsConversionExceptionTest.php` — 9 tests **Total: 37 tests** ✅ (12+) ### T17 — WebhookHandler Revision **Files modified** (under `/root/profe-modules/webhook_handler/`): - `src/WebhookHandler/WebhookHandler.php` — Added `GoogleAdsConversionInterface` injection, `uploadConversionToGoogleAds()`, `hashEmailForConversion()`, `enqueueConversionRetry()`, `processConversionRetries()`, `calculateNextRetry()` methods. `handleCheckoutSessionCompleted()` now calls `uploadConversionToGoogleAds()` after emitting `payment_succeeded`. - `src/WebhookHandler/WebhookHandlerModule.php` — Added `GoogleAdsConversionInterface` to `getDependencies()` and `register()` factory - `src/WebhookHandler/Repository/WebhookEventRepository.php` — Added `storeConversionRetry()`, `findPendingConversionRetries()`, `updateConversionRetry()`, `clearConversionRetry()` methods with prepared statements - `composer.json` — Added dependency on `Profe\GoogleAdsConversion\GoogleAdsConversionInterface` - `README.md` — Documented conversion upload behavior, retry schedule, new columns - `tests/Unit/WebhookHandlerTest.php` — 16 tests (added conversion upload, retry, GCLID, failure tests) - `tests/Unit/WebhookHandlerModuleTest.php` — 5 tests (updated for new dependency) - `tests/Fixture/WebhookHandlerFixtures.php` — Added `getCheckoutSessionCompletedEventWithGclid()` **Total: 21 tests** ✅ (12+) ### T15 — Embedded Checkout + GCLID Capture **Files modified** (stripe_service): - `src/StripeService/StripeServiceInterface.php` — Added `$uiMode` (default `'hosted'`) and `$metadata` params to `createCheckoutSession()` - `src/StripeService/StripeService.php` — Implemented embedded mode: adds `ui_mode='embedded'` + `return_url` to Stripe API params, returns `client_secret` in result array - `src/StripeService/StripeServiceModule.php` — Accepts `ui_mode` and `gclid` from request params, validates ui_mode, passes metadata - `tests/Unit/StripeServiceTest.php` — 20 tests (added 5 for embedded mode, metadata, interface signature) **Files modified** (checkout_service): - `src/CheckoutService/CheckoutServiceInterface.php` — Added `$uiMode` param (default `'hosted'`) - `src/CheckoutService/CheckoutService.php` — Added `getGclidFromCookie()` (reads `_gcl_aw`/`_gcl_dc` cookies + `$_GET['gclid']` fallback), passes metadata with gclid to StripeService, returns `client_secret` in embedded mode, emits `uiMode` and `gclid` in event - `src/CheckoutService/CheckoutServiceModule.php` — Accepts `ui_mode` from request, validates it, passes to service - `tests/Unit/CheckoutServiceTest.php` — 16 tests (added 6 for embedded mode, GCLID, metadata passing) - `tests/Fixture/CheckoutServiceFixtures.php` — Added `getEmbeddedCheckoutSessionResult()` **StripeService: 20 tests** ✅ (12+) **CheckoutService: 16 tests** ✅ (12+) ### Verification Results - **PHP Syntax**: All files pass `php -l` - **Quadruple-backslash**: 0 occurrences across all modules (FAULT-011) - **Prepared statements**: All DB queries use `:param` placeholders (FAULT-005) - **Backwards compatibility**: Both `$uiMode` and `$metadata` parameters default to `'hosted'` and `[]` respectively — no existing callers break - **Non-blocking conversion upload**: `uploadConversionToGoogleAds()` catches all exceptions internally, never propagates to webhook handler's main try/catch