Skip to content

Fix personal WP Health Check recovery boot#3637

Open
akirk wants to merge 1 commit into
WordPress:trunkfrom
akirk:fix/personal-wp-health-check-recovery
Open

Fix personal WP Health Check recovery boot#3637
akirk wants to merge 1 commit into
WordPress:trunkfrom
akirk:fix/personal-wp-health-check-recovery

Conversation

@akirk
Copy link
Copy Markdown
Member

@akirk akirk commented May 14, 2026

What changed

  • Mark Health Check recovery URLs with an explicit ?playground-recovery-mode=health-check query param. → This also will allow us in support cases to just send the URL https://my.wordpress.net/?playground-recovery-mode=health-check to a user
  • Detect that marker before boot probes WordPress, so recovery mode skips the pre-blueprint wp-load.php path that can crash on broken active plugins.
  • Remove the recovery marker from the URL after the blueprint is consumed.
  • Add coverage for recovery URL generation and router cleanup.

Why

The previous recovery flow inferred recovery mode from the decoded blueprint landing page. Personal WP could still run the WordPress install check before recovery setup, and that check loads wp-load.php. If an active plugin fatals there, Health Check never gets installed.

Testing

  • NX_ISOLATE_PLUGINS=false NX_DAEMON=false npm_config_cache=/tmp/npm-cache npm_config_update_notifier=false npm exec --offline -- nx test playground-personal-wp
  • NX_ISOLATE_PLUGINS=false NX_DAEMON=false npm_config_cache=/tmp/npm-cache npm_config_update_notifier=false npm exec --offline -- nx run playground-personal-wp:typecheck
  • NX_ISOLATE_PLUGINS=false NX_DAEMON=false npm_config_cache=/tmp/npm-cache npm_config_update_notifier=false npm exec --offline -- nx run playground-personal-wp:lint

@akirk akirk requested review from a team, brandonpayton and Copilot May 14, 2026 14:14
@akirk akirk force-pushed the fix/personal-wp-health-check-recovery branch from 972d143 to 71bccc9 Compare May 14, 2026 14:23
@akirk akirk force-pushed the fix/personal-wp-health-check-recovery branch from 71bccc9 to c7cf897 Compare May 14, 2026 14:42
@akirk akirk requested review from Copilot and removed request for Copilot May 15, 2026 07:25
@akirk akirk self-assigned this May 15, 2026
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