Skip to content

Bootstrap shared production bindings and refine PlanetScale scrape detection#210

Merged
Makisuo merged 3 commits into
mainfrom
codex/review-alchemy-v2-bindings
Jul 15, 2026
Merged

Bootstrap shared production bindings and refine PlanetScale scrape detection#210
Makisuo merged 3 commits into
mainfrom
codex/review-alchemy-v2-bindings

Conversation

@Makisuo

@Makisuo Makisuo commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Bootstrap retained production-only Cloudflare resources for email sending and Workers observability, then reuse them across API, alerting, and chat-flue workers
  • Tighten PlanetScale permission detection by probing an actual metrics data-plane endpoint instead of relying on service discovery alone
  • Improve PlanetScale discovery keying and logging so fallback endpoints stay distinct and label drift is easier to diagnose

Testing

  • Not run (not requested)

Open in Devin Review

@pullfrog

pullfrog Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Your Pullfrog Router balance is exhausted.

You have a card on file but auto-reload is disabled, so runs paused once your balance went past the overdraft buffer.

Top up balance → · Enable auto-reload →

Pullfrog  | ⚠️ this action is pinned to a commit SHA, which freezes the cleanup step — switch to @v0 or keep the SHA fresh with Dependabot | Rerun failed job ➔View workflow run | via Pullfrog | Using Claude Opus𝕏

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@Makisuo Makisuo merged commit d1a1212 into main Jul 15, 2026
7 of 8 checks passed
@Makisuo Makisuo deleted the codex/review-alchemy-v2-bindings branch July 15, 2026 12:47
Makisuo added a commit that referenced this pull request Jul 15, 2026
The shared-bindings bootstrap in #210 added Email.Routing + SendingSubdomain
(zone-level enable + DKIM/SPF DNS writes) that the prod CLOUDFLARE_API_TOKEN
lacks scopes for, so every push-to-main prd deploy 403'd in the plan phase
(POST /zones/{id}/email/routing/enable → Forbidden).

Revert the email plumbing to the pre-#210 wiring: both api and alerting keep
their plain Cloudflare.Email.SendEmail("email", ...) binding (unchanged from
before, needs no extra token scope). The Workers ObservabilityDestination
bootstrap stays — that permission is granted and its plan call now succeeds.

Removes the emailBinding param from createMapleApi/createAlertingWorker and the
now-unused Input/Output imports.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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