Memory Wiki

Let me start by reading the plans and reference modules to understand the requirements.

Now let me look at reference modules to understand patterns.

Let me look at reference migrations and module patterns.

Let me look at the repository, tests, exception, and module.json patterns.

Let me look at existing tests to understand the test patterns.

Now I have a clear picture of the patterns. Let me also check ProfeDB and any BrevoServiceInterface for reference.

Now I have a complete understanding of the patterns. Let me create all files for both modules. Starting with T26 (CrmTags).

Now the migration SQL files and tests for T26.

Now the T26 test files.

Now building T27 (CrmSync). Let me create all the files.

Let me fix the JSON escaping issue.

Now creating the core T27 source files.

Now the migration SQL and test files for T27.

Now let me run php -l on all PHP files and copy migration SQL to central directory.

All PHP files pass syntax checks. Now copying migration SQLs to central directory.

Let me verify the complete file structures.

Let me do a final test count verification.

Both modules are complete. Here's the summary: --- ## Summary ### T26 — CrmTags (`/root/profe-modules/crm_tags/`) **What was built:** - `CrmTagsInterface` — 10 public API methods: createTag, deleteTag, addTagToUser, removeTagFromUser, getUserTags, getUsersByTag, listTags, updateTag, userHasTag, getUsersByAllTags, getUsersByAnyTag - `CrmTagsService` — Full implementation with slug auto-generation, idempotent insert/remove, uniqueness checks, and event emissions (tag_created, tag_deleted, tag_added, tag_removed) - `CrmTagsRepository` — CRUD for `profe_crm_tags` table with search/pagination - `CrmUserTagsRepository` — User-tag pivot management with AND/OR tag queries, UUID resolution via direct `profe_users` query (FAULT-002 documented workaround) - `CrmTagsModule` — ModuleInstanceInterface with 8 REST routes, CSRF protection on all mutations (FAULT-004), 2 migrations registered, factory closures (FAULT-01) - `CrmTagsException` — Typed exception with tagNotFound, tagNameExists, userNotFound, persistenceError - 2 Migration PHP classes (`CreateCrmTagsMigration`, `CreateCrmUserTagsMigration`) implementing `MigrationInterface` (FAULT-006) - Migration SQL files: `010_create_profe_crm_tags.sql`, `011_create_profe_crm_user_tags.sql` **Tests: 35 test methods** across 5 files (exceeds ≥12 minimum): - CrmTagsServiceTest (15 tests) - CrmTagsRepositoryTest (7 tests) - CrmUserTagsRepositoryTest (7 tests) - CrmTagsModuleTest (2 tests) - CrmTagsIntegrationTest (4 tests — skipped, require live DB) ### T27 — CrmSync (`/root/profe-modules/crm_sync/`) **What was built:** - `CrmSyncInterface` — 9 public API methods: enqueueContactCreate, enqueueContactUpdate, enqueueContactDelete, enqueueTagAdd, enqueueTagRemove, enqueueEmailUpdate, processQueue, getQueueStats, retryFailed, cancelPendingForUser - `CrmSyncService` — Event-driven enqueuing with dedup (skips duplicate pending jobs), wraps all enqueue/delegate calls - `CrmSyncProcessor` — Batch processing with exponential backoff (`next_retry_at = NOW() + (2^attempts * 60)`), max 5 retries, rate limit pause after 100 requests, dispatches to BrevoServiceInterface for contact_create/update/delete, tag_add/remove, and email_update - `CrmSyncQueueRepository` — Complete data access: dedup checks, batched pending retrieval with user data JOIN, status updates, stats aggregation, retryFailed, cancelPendingByUserUuid - `CrmSyncModule` — ModuleInstanceInterface with 4 admin REST routes, CSRF on all POST endpoints (FAULT-004), event listener registration (contact_created, contact_updated, tag_added, tag_removed), migration registration - `CrmSyncException` — Typed exception with brevoApiError, userNotFound, queueError - 1 Migration PHP class (`CreateCrmSyncQueueMigration`) implementing `MigrationInterface` (FAULT-006) - Migration SQL: `012_create_profe_crm_sync_queue.sql` **Tests: 27 test methods** across 5 files (exceeds ≥12 minimum): - CrmSyncServiceTest (10 tests) - CrmSyncProcessorTest (4 tests) - CrmSyncQueueRepositoryTest (9 tests) - CrmSyncModuleTest (2 tests) - CrmSyncIntegrationTest (2 tests — skipped, require live DB) ### Compliance Checklist | FAULT | Rule | Status | |-------|------|--------| | FAULT-002/007 | Documented direct `profe_users` queries | ✅ Every UUID resolution method has FAULT-002 workaround comment | | FAULT-004 | CSRF on POST/DELETE | ✅ All mutation endpoints use checkAdminAccessWithCsrf() or checkAuthWithCsrf() calling requireValidNonce() | | FAULT-005/009 | ≥12 tests each | ✅ T26: 35 tests, T27: 27 tests | | FAULT-006 | PHP migration classes | ✅ 3 migration classes implementing MigrationInterface | | FAULT-008 | Declared deps injected | ✅ All dependencies in constructor signatures (T26: 4 deps, T27: 5 deps) | | FAULT-01 | Factory closure captures $c | ✅ All register() methods use closure pattern | | FAULT-04 | Bare table names | ✅ No $wpdb->prefix used | | FAULT-05 | Prepared statements | ✅ All queries use named params | | FAULT-09 | No stubs | ✅ All methods fully implemented | All 27 PHP files pass `php -l` syntax checks without errors. Migration SQL files copied to `/root/profe-modules/migrations/`.