Skip to content

auto-recover: orphaned worker branch for #1446#1448

Merged
superdav42 merged 1 commit into
mainfrom
feature/auto-20260616-232816-gh1446
Jun 17, 2026
Merged

auto-recover: orphaned worker branch for #1446#1448
superdav42 merged 1 commit into
mainfrom
feature/auto-20260616-232816-gh1446

Conversation

@superdav42

@superdav42 superdav42 commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Orphan recovery PR — worker pushed this branch but exited before opening a PR.

Branch feature/auto-20260616-232816-gh1446 was pushed by headless worker session issue-1446 which released as worker_branch_orphan. This PR was auto-created by the orphan-recovery path (GH#20819) so the change can land via the normal review and merge pipeline.

Resolves #1446

Summary by CodeRabbit

  • Improvements
    • Enhanced email domain verification workflow to publish DNS records at the appropriate verification initiation stage across all supported provider integrations (Amazon SES, Cloudflare, and OCI Email), improving synchronization timing.

@superdav42 superdav42 added status:in-review PR open, awaiting review/merge origin:worker-takeover Worker took over from interactive session labels Jun 17, 2026
@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e120641c-4949-43d8-af1c-b4ea2d8c1e81

📥 Commits

Reviewing files that changed from the base of the PR and between ba9e5e0 and 3b1b450.

📒 Files selected for processing (3)
  • inc/integrations/providers/amazon-ses/class-amazon-ses-transactional-email.php
  • inc/integrations/providers/cloudflare/class-cloudflare-domain-mapping.php
  • inc/integrations/providers/oci-email/class-oci-email-transactional-email.php

📝 Walkthrough

Walkthrough

Three provider integration files are updated to rename the wu_domain_verified WordPress action hook to wu_domain_verification_initiated. Amazon SES and OCI Email now emit the new hook (with domain, site ID, and DNS records) and re-emit the old hook name via do_action_deprecated (deprecated since 2.5.1). Cloudflare's hook subscription is updated to listen to the new hook name.

Changes

Hook rename: wu_domain_verified → wu_domain_verification_initiated

Layer / File(s) Summary
Hook emission and deprecation in SES and OCI Email providers
inc/integrations/providers/amazon-ses/class-amazon-ses-transactional-email.php, inc/integrations/providers/oci-email/class-oci-email-transactional-email.php
Both providers' on_domain_added() now fire wu_domain_verification_initiated with $domain, $site_id, and $dns_records, and replace the direct wu_domain_verified emission with do_action_deprecated() pointing to the new hook at version 2.5.1. Docblocks updated accordingly.
Cloudflare hook subscription update
inc/integrations/providers/cloudflare/class-cloudflare-domain-mapping.php
register_hooks() subscribes publish_transactional_email_dns_records to wu_domain_verification_initiated instead of wu_domain_verified, accepting 3 arguments at the same priority.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐇 Hop, hop — the hook's been renamed today,
wu_domain_verified has had its day.
"Initiated" is the truer name,
for DNS records are not yet the same.
Deprecated with a trail to follow,
the rabbit leaves no path hollow! 🌿

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title references an orphaned worker branch recovery for issue #1446, accurately describing the PR's nature as an automated recovery mechanism for abandoned work.
Linked Issues check ✅ Passed The PR successfully implements the required fix from issue #1446: renaming the misleading wu_domain_verified action hook to wu_domain_verification_initiated across OCI Email and Amazon SES integrations, and updating Cloudflare to subscribe to the new hook.
Out of Scope Changes check ✅ Passed All changes are directly scoped to resolving issue #1446: renaming the action hook, deprecating the old one, and updating hook subscriptions in related provider modules.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/auto-20260616-232816-gh1446

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown

🔨 Build Complete - Ready for Testing!

📦 Download Build Artifact (Recommended)

Download the zip build, upload to WordPress and test:

🌐 Test in WordPress Playground (Very Experimental)

Click the link below to instantly test this PR in your browser - no installation needed!
Playground support for multisite is very limitied, hopefully it will get better in the future.

🚀 Launch in Playground

Login credentials: admin / password

@superdav42 superdav42 merged commit 89d010a into main Jun 17, 2026
11 checks passed
@superdav42

Copy link
Copy Markdown
Collaborator Author

Orphan recovery PR — worker pushed this branch but exited before opening a PR.
Branch feature/auto-20260616-232816-gh1446 was pushed by headless worker session issue-1446 which released as worker_branch_orphan. This PR was auto-created by the orphan-recovery path (GH#20819) so the change can land via the normal review and merge pipeline.


Merged via PR #1448 to main.
Merged by deterministic merge pass (pulse-wrapper.sh).


aidevops.sh v3.20.86 spent 20m on this as a headless bash routine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

origin:worker-takeover Worker took over from interactive session review-feedback-scanned Merged PR already scanned for quality feedback status:in-review PR open, awaiting review/merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review followup: PR #1440 — feat: add OCI Email Delivery integration

1 participant