Skip to content

feat(mailgun): add US/EU region selection to the Mailgun block and tools - #8423

Merged
waleedlatif1 merged 1 commit into
stagingfrom
feat/mailgun-region
Sep 29, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
feat/mailgun-region

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Added a Region dropdown (US / EU, default US) to the Mailgun block, shown for every operation
  • Added an optional region param (user-only) to all 8 Mailgun tools; each request.url now resolves its host through one shared getMailgunApiBaseUrl helper in tools/mailgun/utils.ts
  • EU maps to https://api.eu.mailgun.net/v3; anything else (including no value) stays on https://api.mailgun.net/v3, so existing workflows keep calling the US host unchanged
  • Regenerated tool metadata and the Mailgun integration docs

Closes #8420

Type of Change

  • New feature

Testing

  • Routed every operation (all 8) through the real block tools.config.tool/params → tool request.url for three states: legacy block with no region value, us, and eu. All 24 resolved to the expected host, including add_list_member (the one operation that remaps params)
  • Hit both live hosts with an invalid key: api.mailgun.net and api.eu.mailgun.net both answer 401, so both are real endpoints
  • Deploy change detection resolves a field added after deploy against the current definition's default on both sides, so already-deployed Mailgun workflows do not flip to "needs redeploy"
  • bun run type-check (apps/sim), bun run lint, block-registry check, bun run check:audits (52/52), docs-manifest:check
  • vitest run blocks/ tools/index.test.ts tools/utils.test.ts tools/params.test.ts lib/workflows/comparison: 1017 passed

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 29, 2026 7:26pm UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 13 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium risk] Adds region selection to Mailgun integration tools.

The PR appears safe to merge; no new actionable issue was identified.

Summary

The PR adds a US/EU region selector to the Mailgun block and routes all eight Mailgun tools through a shared host-selection helper.

  • Omitted regions continue to use the US endpoint.
  • Tool metadata and integration documentation include the new parameter.
Diagram
%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Mailgun block region or user-provided tool parameter] --> B[Mailgun tool request URL]
  B --> C{Region is eu?}
  C -- Yes --> D[EU API host]
  C -- No or omitted --> E[US API host]
Loading

Reviews (2) · Last reviewed commit: "feat(mailgun): add US/EU region selectio..."

Comment thread apps/sim/tools/mailgun/utils.ts
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1
waleedlatif1 merged commit bf66b16 into staging Sep 29, 2026
33 checks passed
@waleedlatif1
waleedlatif1 deleted the feat/mailgun-region branch September 29, 2026 19:39

This branch was successfully deployed

1 active deployment
Preview — 6d327506 Deployed Sep 29, 2026 by vercel[bot]
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