Skip to content

[miniflare] Add logging of emails sent by workers to .wrangler/tmp.#14627

Open
tpmmorris wants to merge 20 commits into
cloudflare:mainfrom
tpmmorris:tmorris/EmailSupport
Open

[miniflare] Add logging of emails sent by workers to .wrangler/tmp.#14627
tpmmorris wants to merge 20 commits into
cloudflare:mainfrom
tpmmorris:tmorris/EmailSupport

Conversation

@tpmmorris

@tpmmorris tpmmorris commented Jul 9, 2026

Copy link
Copy Markdown

Add storing of emails sent by workers to temporary directory within project.

Previously, email send logs were only saved to a the system's temporary directory. This new change keeps this location to prevent breaking, but adds an additional location (for wrangler sending a simple text email: .wrangler/tmp/email/<session-id>/email-text/<message-uuid>). This will improve user's ability to test their workers' email capability. Implementation of feature request #13809.


image
Open in Devin Review

@changeset-bot

changeset-bot Bot commented Jul 9, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 5035f85

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages
Name Type
miniflare Patch
wrangler Patch
@cloudflare/deploy-helpers Patch
@cloudflare/pages-shared Patch
@cloudflare/runtime-types Patch
@cloudflare/vite-plugin Patch
@cloudflare/vitest-pool-workers Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@workers-devprod workers-devprod requested review from a team and ascorbic and removed request for a team July 9, 2026 14:25
@workers-devprod

workers-devprod commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • @cloudflare/wrangler
Show detailed file reviewers
  • .changeset/clever-months-speak.md: [@cloudflare/wrangler]
  • packages/miniflare/src/index.ts: [@cloudflare/wrangler]
  • packages/miniflare/src/plugins/core/index.ts: [@cloudflare/wrangler]
  • packages/miniflare/src/plugins/email/index.ts: [@cloudflare/wrangler]
  • packages/miniflare/src/plugins/shared/index.ts: [@cloudflare/wrangler]
  • packages/miniflare/src/workers/email/send_email.worker.ts: [@cloudflare/wrangler]
  • packages/miniflare/test/plugins/email/index.spec.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/api/integrations/platform/index.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/api/startDevWorker/LocalRuntimeController.ts: [@cloudflare/wrangler]
  • packages/wrangler/src/dev/miniflare/index.ts: [@cloudflare/wrangler]

devin-ai-integration[bot]

This comment was marked as resolved.

Comment thread .changeset/clever-months-speak.md Outdated
Comment thread packages/miniflare/src/plugins/email/index.ts
Comment thread .changeset/clever-months-speak.md Outdated
Comment thread packages/miniflare/src/plugins/email/index.ts Outdated
Comment thread packages/miniflare/src/plugins/email/index.ts Outdated
Comment thread packages/miniflare/src/plugins/email/index.ts Outdated
Comment thread packages/miniflare/src/plugins/email/index.ts Outdated
Comment thread packages/miniflare/src/workers/email/send_email.worker.ts
Comment thread packages/miniflare/src/index.ts Outdated
@pkg-pr-new

pkg-pr-new Bot commented Jul 9, 2026

Copy link
Copy Markdown
@cloudflare/autoconfig

npm i https://pkg.pr.new/@cloudflare/autoconfig@14627

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@14627

@cloudflare/deploy-helpers

npm i https://pkg.pr.new/@cloudflare/deploy-helpers@14627

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@14627

miniflare

npm i https://pkg.pr.new/miniflare@14627

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@14627

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@14627

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@14627

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@14627

@cloudflare/workers-auth

npm i https://pkg.pr.new/@cloudflare/workers-auth@14627

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@14627

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@14627

wrangler

npm i https://pkg.pr.new/wrangler@14627

commit: bd8a6ed

@emily-shen emily-shen 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.

not sure if you've done this already, but it can be really useful to use our fixtures to play around and test things manually - have a look at fixtures/email-worker

devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

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

Labels

None yet

Projects

Status: Untriaged

Development

Successfully merging this pull request may close these issues.

3 participants