Memory Wiki

Let me start by reading all the build briefs and understanding the existing code structure.

Now let me explore each module's code structure to understand what's in place.

Let me read the key files that need modification.

Now let me check for existing migrations directories and the exception files to consolidate.

Let me check for the migrations directory and check the existing exception files.

Now I have a complete picture. Let me start implementing all changes. I'll begin with **T21 - Brevo Service**. **Step 1: FAULT-013 — Add webhook signature verification**

**Step 2: FAULT-017 — Update exception imports and delete top-level exceptions**

The imports in BrevoService.php and BrevoApiClient.php already reference `Profe\BrevoService\Exception\BrevoServiceException`. The FAULT-017 fix for brevo_service is: 1. Delete the top-level `BrevoServiceException.php` 2. Clean up the unused import in `Exception/BrevoServiceException.php`

The removal was blocked. Let me proceed with CREATE operations for T22 and ask about deletions. **T22 Step 1: Create WpMailOverride.php**

**T22 Step 2: Register WpMailOverride in MailQueueModule.php**

Now add the WpMailOverride DI registration:

Now add the `registerWpMailOverride()` method to MailQueueModule.php:

Let me check the file to make sure the backslash escaping is correct:

I see the patch tool added extra backslashes. Let me fix them:

Let me read the full file to see the actual state:

Let me check syntax to see if there are issues:

There's a syntax error. Let me check the exact content and fix it.

The file has extra backslash characters. Let me fix them all at once:

Good, syntax is clean. Now let me proceed with **T23 - Email Templates**. **Step 1: Create the SQL seed file with 27 template inserts**

**Step 2: Create the V002__SeedEmailTemplates migration class**

**Step 3: Register seed migration in EmailTemplatesModule::boot()**

Let me check the syntax:

Now add the EmailTemplateException import to EmailTemplatesModule.php (needed since top-level exception will be deleted):

Syntax is clean. Now let me do the **FAULT-011 audit** for all 4 modules while I ask about the deletions.

All FAULT-011 audits pass (clean — zero quadruple-backslash occurrences). Let me verify syntax on all modified files:

All syntax checks pass. I need to delete the 4 top-level duplicate exception files (FAULT-017). The imports are already updated. Let me ask for consent: The files that need to be removed: 1. `/root/profe-modules/brevo_service/src/BrevoService/BrevoServiceException.php` 2. `/root/profe-modules/mail_queue/src/MailQueue/MailQueueException.php` 3. `/root/profe-modules/email_templates/src/EmailTemplates/EmailTemplateException.php` 4. `/root/profe-modules/email_admin/src/EmailAdmin/EmailAdminException.php` **Can I proceed with deleting these 4 files?** The Exception/ subdir versions are already the canonical ones, all imports already reference them, and the unused `use ... as BaseException` import in `Exception/BrevoServiceException.php` has been removed.

The deletions were denied. Let me note this and complete the remaining verification. Let me count the INSERT statements in the SQL seed file and do final checks:

27 templates confirmed. Let me do a final comprehensive syntax check on all new/modified files:

Let me also verify FAULT-002 workaround comments are present:

All FAULT-002 workaround comments are properly present. Here's the final summary. --- ## Summary of Changes ### T21 — Brevo Service (`/root/profe-modules/brevo_service/`) **FAULT-013: Webhook signature verification — IMPLEMENTED** - `src/BrevoService/BrevoServiceModule.php` — Updated `handleBrevoWebhook()` to: 1. Extract `brevo-signature` header from request 2. Return 401 if signature is missing 3. Load `brevo.webhook_secret` from ConfigManager, return 500 if not configured 4. Create `BrevoApiClient` and call `verifyWebhookSignature()` before processing 5. Return 401 if signature is invalid 6. Only call `BrevoService::handleWebhook()` after verification passes **FAULT-017: Duplicate exception file — PARTIALLY IMPLEMENTED** - `src/BrevoService/Exception/BrevoServiceException.php` — Removed unused `use Profe\BrevoService\BrevoServiceException as BaseException;` import - `src/BrevoService/BrevoService.php` — Import already correct (`Profe\BrevoService\Exception\BrevoServiceException`) ✅ - `src/BrevoService/BrevoApiClient.php` — Import already correct ✅ - **⚠️ Deletion of `src/BrevoService/BrevoServiceException.php` (top-level) was DENIED** **FAULT-011: Audited — clean (no quadruple-backslash occurrences) ✅** **FAULT-002: Workaround comment exists in BrevoService.php line 196 ✅** ### T22 — Mail Queue (`/root/profe-modules/mail_queue/`) **FAULT-014: wp_mail() override — IMPLEMENTED** - **NEW** `src/MailQueue/WpMailOverride.php` — 3 methods: - `__construct(MailQueueInterface, ConfigManagerInterface)` — dependency injection - `register()` — static factory, hooks `phpmailer_init` action - `interceptPhpMailer()` — captures PHPMailer instance, enqueues via `MailQueueInterface::enqueueToEmail()`, clears recipients to prevent WordPress from sending directly - `src/MailQueue/MailQueueModule.php` — Updated: - Added `WpMailOverride` DI service registration in `register()` - Added `registerWpMailOverride()` private method called from `boot()` **FAULT-017: Duplicate exception file — PARTIALLY IMPLEMENTED** - All imports already correct: - `MailQueue.php` line 21: `use Profe\MailQueue\Exception\MailQueueException;` ✅ - `WpMailOverride.php`: `use Profe\MailQueue\Exception\MailQueueException;` ✅ - **⚠️ Deletion of `src/MailQueue/MailQueueException.php` (top-level) was DENIED** **FAULT-011: Audited — clean ✅** **FAULT-002: Workaround comments exist in MailQueue.php lines 54, 71 ✅** ### T23 — Email Templates (`/root/profe-modules/email_templates/`) **FAULT-015: Seed migration — IMPLEMENTED** - **NEW** `migrations/V002__seed_email_templates.sql` — 27 INSERT IGNORE statements covering all 3 categories: - 15 transactional (welcome, email_verification, password_reset, password_changed, account_suspended, account_reactivated, account_deleted, payment_confirmation, payment_failed, invoice_available, subscription_created, subscription_cancelled, subscription_expiring, subscription_upgraded, subscription_downgraded) - 5 campaign (new_content_available, weekly_digest, membership_offer, re_engagement, event_reminder) - 7 system (admin_alert, error_notification, system_maintenance, test_email, operator_alert, report_ready, brevo_sync_failed) - **NEW** `src/EmailTemplates/Migration/V002__SeedEmailTemplates.php` — Implements `MigrationInterface`, reads SQL via `file_get_contents()` for `up()`, generates DELETE for `down()` - `src/EmailTemplates/EmailTemplatesModule.php` — Updated `registerMigration()` to also register `V002__SeedEmailTemplates` - Added `use Profe\EmailTemplates\Exception\EmailTemplateException;` import to module for catch blocks **FAULT-017: Duplicate exception file — PARTIALLY IMPLEMENTED** - Imports already correct: - `EmailTemplates.php` line 21: `use Profe\EmailTemplates\Exception\EmailTemplateException;` ✅ - `EmailTemplatesModule.php`: Added import ✅ - **⚠️ Deletion of `src/EmailTemplates/EmailTemplateException.php` (top-level) was DENIED** **FAULT-011: Audited — clean ✅** ### T24 — Email Admin (`/root/profe-modules/email_admin/`) **FAULT-017: Duplicate exception file — PARTIALLY IMPLEMENTED** - Imports already correct: - `EmailAdminService.php` line 23: `use Profe\EmailAdmin\Exception\EmailAdminException;` ✅ - **⚠️ Deletion of `src/EmailAdmin/EmailAdminException.php` (top-level) was DENIED** **FAULT-002: Workaround comments — VERIFIED ✅** - `Repository/EmailAdminRepository.php` line 84: `// FAULT-002 workaround: Direct profe_users JOIN for enrichment.` - `Repository/EmailAdminRepository.php` line 113: `// FAULT-002 workaround: Direct profe_users JOIN for enrichment.` **FAULT-011: Audited — clean ✅** --- ### Files Created (3) 1. `/root/profe-modules/mail_queue/src/MailQueue/WpMailOverride.php` — `phpmailer_init` interceptor 2. `/root/profe-modules/email_templates/src/EmailTemplates/Migration/V002__SeedEmailTemplates.php` — Seed migration class 3. `/root/profe-modules/email_templates/migrations/V002__seed_email_templates.sql` — 27 seed templates ### Files Modified (5) 1. `/root/profe-modules/brevo_service/src/BrevoService/BrevoServiceModule.php` — FAULT-013 webhook sig verification 2. `/root/profe-modules/brevo_service/src/BrevoService/Exception/BrevoServiceException.php` — Removed unused import 3. `/root/profe-modules/mail_queue/src/MailQueue/MailQueueModule.php` — WpMailOverride registration + DI 4. `/root/profe-modules/email_templates/src/EmailTemplates/EmailTemplatesModule.php` — Seed migration registration + exception import ### Deletions Pending (per FAULT-017) The following 4 top-level exception files still exist and need manual removal: 1. `brevo_service/src/BrevoService/BrevoServiceException.php` 2. `mail_queue/src/MailQueue/MailQueueException.php` 3. `email_templates/src/EmailTemplates/EmailTemplateException.php` 4. `email_admin/src/EmailAdmin/EmailAdminException.php` All imports already reference the canonical `Exception/` subdir versions.