Fixed race condition when scheduling email analytics jobs

[EvanHahn]

closes https://linear.app/ghost/issue/NY-1449

If scheduleRecurringJobs was called twice in rapid succession, we might schedule jobs twice. This fixes that.

This is a long-standing issue that could rear its head when we call this function more often, which we plan to do soon. However, I think this is a useful bugfix on its own.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 622f09a2-284a-4ef6-b6d9-d70956a459ae

📥 Commits

Reviewing files that changed from the base of the PR and between 9c3e04b and cc09906.

📒 Files selected for processing (2)

• ghost/core/core/server/services/email-analytics/jobs/email-analytics-job-scheduler.ts
• ghost/core/test/unit/server/services/email-analytics/email-analytics-job-scheduler.test.ts

---

Walkthrough

Updated email analytics recurring-job scheduling to check internal newsletter and automation scheduling state before registering jobs. Added concurrency-focused tests using deferred query resolution to verify that concurrent scheduling calls register only one newsletter job and one automation job, while preserving the expected query invocation behavior.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: fixing a race condition in email analytics job scheduling.
Description check	✅ Passed	The description matches the changeset and explains the rapid-succession scheduling bug it fixes.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch evanhahn-ny-1449-fix-race-condition-when-scheduling-recurring-jobs

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[nx-cloud]

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci

---

View your CI Pipeline Execution ↗ for commit cc09906

Command	Status	Duration	Result
nx run ghost:test:ci:integration	✅ Succeeded	2m 48s	View ↗
nx run ghost:test:integration	✅ Succeeded	3m 4s	View ↗
nx run ghost:test:legacy	✅ Succeeded	3m 5s	View ↗
nx run ghost:test:e2e	✅ Succeeded	2m 33s	View ↗
nx run-many --target=build --projects=tag:publi...	✅ Succeeded	1s	View ↗
nx run-many -t test:unit -p ghost	✅ Succeeded	29s	View ↗
nx run ghost-monorepo:lint:boundaries	✅ Succeeded	7s	View ↗
nx run @tryghost/admin:build	✅ Succeeded	7s	View ↗
Additional runs (3)	✅ Succeeded	...	View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗

---

☁️ Nx Cloud last updated this comment at 2026-07-13 17:45:58 UTC