Skip to content

Add WordPress operation provider factory - #45

Draft
alexstandiford wants to merge 1 commit into
codex/wordpress-coordinationfrom
codex/wordpress-operation-provider
Draft

alexstandiford wants to merge 1 commit into
codex/wordpress-coordinationfrom
codex/wordpress-operation-provider

Conversation

@alexstandiford

@alexstandiford alexstandiford commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Stacked on PR44 (codex/wordpress-coordination, d502c26). Adds the WordPress adapter implementation of DB PR42's OperationDatabaseProviderFactory (4b0b770) for official core wpdb only.

  • Creates fresh operation-local WordPress QueryBuilder and ClauseBuilder instances from the exact active coordinator/pinned wpdb resource before handler clause preparation.
  • Refuses factory use outside the matching active coordinated callback; preserves the operation query strategy, cache/event wrappers, source logger, and independent builders.
  • Binds the factory through WordPressInitializer and verifies the production container shares one ordinary/coordinated strategy instance; covers the cache TTL/database provider alias seam.
  • Adds real official WordPress 6.8.3 + MySQL 8 handler-bridge coverage for rollback suppression, post-commit invalidation/events, fresh reads, callback/resource identity, and builder affinity.

The hosted MySQL 8 + official WordPress 6.8.3 integration job with --fail-on-skipped is included in the stacked PR44 coordinator layer.

Verification

  • vendor/bin/phpunit --colors=never — 78 tests, 139 assertions.
  • Official WP 6.8.3 + MySQL 8: vendor/bin/phpunit -c phpunit-integration.xml --colors=never --fail-on-skipped — 76 tests, 161 assertions.

@alexstandiford
alexstandiford force-pushed the codex/wordpress-coordination branch from b56d5cf to d502c26 Compare September 20, 2026 23:57
@alexstandiford
alexstandiford force-pushed the codex/wordpress-operation-provider branch from 2e0284e to 75e3331 Compare September 20, 2026 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant