Skip to content

Add Email Routing rule surface#16

Merged
rogu3bear merged 1 commit into
mainfrom
feature/email-routing-rule-surface
May 20, 2026
Merged

Add Email Routing rule surface#16
rogu3bear merged 1 commit into
mainfrom
feature/email-routing-rule-surface

Conversation

@rogu3bear

@rogu3bear rogu3bear commented May 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a first-class email.routing_rule cfctl surface for zone Email Routing rules
  • support list/get/verify and plan/ack-safe upsert from literal recipient to Email Worker
  • update generated capabilities docs

Verification

  • jq empty catalog/surfaces.json
  • bash -n scripts/cf_inventory_email_routing_rules.sh scripts/cf_mutate_email_routing_rule.sh scripts/lib/cfctl.sh commands/cfctl.sh
  • ./scripts/verify_static_contract.sh -> static-contract verification passed
  • CF_TOKEN_LANE=global ./cfctl list email.routing_rule --zone adapteros.com --name founders@adapteros.com -> existing rule read successfully
  • CF_TOKEN_LANE=global ./cfctl apply email.routing_rule upsert --zone adapteros.com --name founders@adapteros.com --service star-maildesk-cf-router --plan -> preview produced a PUT plan, no live mutation performed

Notes

This intentionally stops at a previewed Email Routing rule mutation. Do not ack live maildesk routing until the maildesk router forwards/notifies operators or the desk UI is ready to be the primary mailbox surface.

@kilo-code-bot

kilo-code-bot Bot commented May 20, 2026

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (7 files)
  • catalog/surfaces.json
  • commands/cfctl.sh
  • docs/capabilities.md
  • scripts/cf_inventory_email_routing_rules.sh
  • scripts/cf_mutate_email_routing_rule.sh
  • scripts/lib/cfctl.sh

Reviewed by grok-code-fast-1:optimized:free · 97,776 tokens

@rogu3bear rogu3bear merged commit 8e9d07b into main May 20, 2026
3 checks passed
@rogu3bear rogu3bear deleted the feature/email-routing-rule-surface branch May 20, 2026 00:36
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