Skip to content

Modern Restructuring#1643

Merged
jakejackson1 merged 5 commits intohot-patchfrom
modern-restructuring
Apr 21, 2026
Merged

Modern Restructuring#1643
jakejackson1 merged 5 commits intohot-patchfrom
modern-restructuring

Conversation

@jakejackson1
Copy link
Copy Markdown
Member

@jakejackson1 jakejackson1 commented Apr 16, 2026

Modern Restructuring

This branch modernises the development toolchain, replaces the legacy TestCafe E2E suite with Playwright, and brings the codebase up to date with WordPress 7.0 and PHP 8.5.

Changes

Add Playwright E2E tests

Replaces the legacy TestCafe-based tests/e2e/ suite with a new Playwright test suite under tests/playwright/. Tests are split into two projects — core/ (standard permalinks, port 8702) and permalinks/ (pretty permalinks, port 8703) — and cover PDF shortcode rendering, entry list/detail views, PDF settings (general, appearance, advanced, notifications, templates), font manager, template manager, and PDF access controls. A dedicated GitHub Actions workflow (.github/workflows/playwright-e2e.yml) runs the suite on CI.

Modernize toolchain and update dependencies

  • Consolidates dev tooling under a tools/ directory: PHP-Scoper config moves from .php-scoper/ to tools/php-scoper/, PHPUnit config and fixtures move from tests/phpunit/ to tools/phpunit/, PHPCS config moves to tools/phpcs/, and release scripts move to tools/release/
  • Replaces the single .wp-env.json with four dedicated environment configs (tools/wp-env/development.json, integration.json, e2e.json, e2e-permalinks.json) each on their own port (8700–8703)
  • Updates Composer and npm dependencies; removes babel.config.js and phpunit.xml.dist in favour of config files under tools/
  • Adds tools/potomatic/ scripts for automated translation generation
  • Updates CLAUDE.md to document all new commands and environment setup

Update for WordPress 7.0 and PHP 8.5 compatibility

Updates PHPUnit tests and admin styles for compatibility with WordPress 7.0 and PHP 8.5.

Fix various bugs

  • Fixes a fatal error when an add-on is improperly registered
  • Fixes a syncing issue with the font selector and an issue reinitialising merge tags
  • Fixes the colour picker
  • Fixes Jest errors introduced by the font selector changes
  • Removes the non-canonical URL notice
  • Corrects @since tags carried over from hot-patch code

Add translator comments and update translations

Adds /* translators: */ comments to all translatable strings that reference placeholders (%s, %1$s, etc.), resolving all warnings produced by wp i18n make-pot. Updates .po/.mo files and adds .l10n.php files for all bundled locales; moves language files from src/assets/languages/ to languages/ at the plugin root.

@jakejackson1 jakejackson1 mentioned this pull request Apr 16, 2026
4 tasks
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.85%. Comparing base (4bc6ea4) to head (3bfbdf7).
⚠️ Report is 99 commits behind head on hot-patch.

Additional details and impacted files
@@              Coverage Diff              @@
##           hot-patch    #1643      +/-   ##
=============================================
+ Coverage      92.72%   92.85%   +0.12%     
=============================================
  Files             58       58              
  Lines           1526     1511      -15     
  Branches         420      419       -1     
=============================================
- Hits            1415     1403      -12     
  Misses           103      103              
+ Partials           8        5       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jakejackson1 jakejackson1 force-pushed the modern-restructuring branch from 7cf4436 to 5e833f6 Compare April 16, 2026 05:01
@jakejackson1 jakejackson1 force-pushed the modern-restructuring branch 3 times, most recently from a2ec4ff to 40c557b Compare April 20, 2026 10:37
@jakejackson1 jakejackson1 force-pushed the modern-restructuring branch 2 times, most recently from a98cc72 to 2d3565a Compare April 21, 2026 00:49
@jakejackson1 jakejackson1 force-pushed the modern-restructuring branch from 5bf4b02 to 3bfbdf7 Compare April 21, 2026 03:16
@jakejackson1 jakejackson1 merged commit 0568faa into hot-patch Apr 21, 2026
20 of 21 checks passed
@jakejackson1 jakejackson1 deleted the modern-restructuring branch April 21, 2026 03:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant