Skip to content

refactor(email-service): Separate email api and pubsub workers into separate services#594

Merged
evanhutnik merged 21 commits intomainfrom
eavn/comms-46-bins
Jan 6, 2026
Merged

refactor(email-service): Separate email api and pubsub workers into separate services#594
evanhutnik merged 21 commits intomainfrom
eavn/comms-46-bins

Conversation

@evanhutnik
Copy link
Copy Markdown
Contributor

@evanhutnik evanhutnik commented Dec 12, 2025

🚨 🚨 🚨 🚨 🚨 Do Not Merge! Until Evan is back 🚨 🚨 🚨 🚨 🚨

Separate the email api and the email pubsub workers into separate bins/services.

The code still all lives in the email-service crate, but the pubsub workers now have their own bin and fargate service.

This required some refactoring of the pulumi code, namely moving the instantiation of a few things (IAM role, policies, s3 bucket) from service.ts into index.ts.

Both things still use the same Config struct (and env vars) for now.

I had to move some code into modules under lib.rs so it could be accessed by the pubsub_workers bin as well.

Tested locally and in dev.

Screenshots, GIFs, and Videos

@linear
Copy link
Copy Markdown

linear bot commented Dec 12, 2025

Copy link
Copy Markdown
Member

@whutchinson98 whutchinson98 left a comment

Choose a reason for hiding this comment

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

seems good to me

@evanhutnik evanhutnik merged commit ad7f12f into main Jan 6, 2026
38 checks passed
@evanhutnik evanhutnik deleted the eavn/comms-46-bins branch January 6, 2026 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants