Memory Wiki

Let me check existing modules for reference patterns before building.

Let me examine reference modules for patterns.

Let me check a couple more reference files for test and service patterns.

Let me check a couple more reference files for the composer.json and README patterns.

Now I have a thorough understanding of the patterns. Let me start creating all files. I'll batch the writes for efficiency. **Phase 1: Create all T25 (CrmContacts) files.**

Now let me create the CrmContactsModule.php, README.md, and all test files for T25.

Now creating the T25 test files:

Let me check PHP syntax on T25 and start creating T28 files simultaneously.

All T25 files pass PHP syntax. Now creating T28 (CrmImportExport) files.

Now creating T28 service, module, and test files:

Now creating T28 test files and README:

Checking T28 syntax and starting T29 files:

All T28 passes syntax. Now creating T29 (CrmAdmin):

Now creating T29 test files and README:

Now verifying all PHP syntax across all 3 modules:

All syntax checks pass. Let me do a final file count and test count verification:

Let me get accurate per-module test counts:

Here is the completed build summary: --- ## Build Complete: T25, T28, T29 ### What was built **T25 - CrmContacts** (14 files, 18 tests) - `/root/profe-modules/crm_contacts/` — Full module with interface, service, repository, exception, module, and migration - Companion table `profe_crm_contacts` with FK to `profe_users.uuid` - `CreateCrmContactsMigration.php` implements `MigrationInterface` (FAULT-006) - Migration SQL: `/root/profe-modules/migrations/009_create_profe_crm_contacts.sql` - 6 methods on interface: `getContact` (lazy init), `updateContact`, `searchContacts`, `addNote`, `getContactsByOptInStatus`, `getExportBatch` - 5 REST routes: GET/PUT contacts, POST notes, GET export, GET search - All mutation endpoints CSRF-protected (FAULT-004) **T28 - CrmImportExport** (15 files, 17 tests) - `/root/profe-modules/crm_import_export/` — CSV import/export with parser, writer, service, module - `CsvParser` handles BOM, delimiter config, max file size, validation - `CsvWriter` generates CSV with BOM for Excel with proper escaping - 4 methods on interface: `exportContacts`, `getExportColumns`, `importContacts`, `validateCsv`, `getDefaultMapping` - 4 REST routes: GET columns + POST export/import/validate (all POST CSRF-protected) - Per-row transaction import with configurable skip_errors/create_tags - FAULT-002 documented: direct profe_users queries for batch import efficiency **T29 - CrmAdmin** (13 files, 18 tests) - `/root/profe-modules/crm_admin/` — Admin panel backend orchestrating CRM data - `CrmAdminRepository` with complex JOIN queries across 5 tables, sync_status filter - `CrmAdminService` delegates to CoreUsersInterface, CrmContactsInterface, CrmTagsInterface, CrmSyncInterface - 8 methods on interface: `listContacts`, `getContactDetail`, `updateContact`, `deleteContact`, `getDashboardSummary`, `listTags`, `getSyncQueue`, `retrySyncJob` - 10 REST routes: GET/PUT/DELETE contacts, GET dashboard, GET/POST/DELETE tags, GET sync, POST retry - Soft-delete only (status=cancelled, no physical delete) - FAULT-002 documented: cross-module direct queries for complex admin views ### FAULT Compliance | FAULT | Status | |-------|--------| | FAULT-004 (CSRF) | ✅ All PUT/POST/DELETE endpoints call `requireValidNonce()` via `checkAdminAccessWithCsrf()` | | FAULT-005/009 (≥12 tests) | ✅ T25=18, T28=17, T29=18 tests each | | FAULT-006 (Migration class) | ✅ `CreateCrmContactsMigration` implements `MigrationInterface` with up/down SQL | | FAULT-008 (Injected deps) | ✅ All declared deps injected in constructors | | FAULT-002/007 (Direct queries) | ✅ Every direct `profe_users` query has `// FAULT-002 workaround:` comment | | FAULT-04 (Bare tables) | ✅ No `$wpdb->prefix` in any query | | FAULT-05 (Prepared statements) | ✅ All queries use named parameters with `$wpdb->prepare()` | | FAULT-01 (Factory closures) | ✅ `register()` closures capture `$c`, never store container | ### Issues encountered None. All 33 PHP files pass `php -l` syntax check. Integration tests are marked as skipped with explanatory comments since they require a live database connection.