Handle empty email recipient configuration

[KristopherKubicki]

Summary

• normalize the email recipient configuration by trimming entries and skipping empty strings
• log and exit early when no recipients remain to avoid attempting delivery
• add unit coverage asserting that blank recipients do not trigger SMTP usage

Testing

• flake8
• pytest

---

Codex Task

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch codex/normalize-email_recipients-handling

---

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 and usage tips.}

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[Copilot]

Pull Request Overview

This PR adds robust handling for empty or whitespace-only email recipient configurations to prevent unnecessary SMTP operations.

• Normalizes email recipients by trimming whitespace and filtering out empty strings
• Implements early exit with informative logging when no valid recipients remain after normalization
• Adds comprehensive unit test coverage for the empty recipients scenario

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
app/utils/email_alerts.py	Refactors recipient parsing to filter out empty/whitespace entries and adds validation to skip SMTP operations when no valid recipients exist
tests/test_email_alerts.py	Adds test case verifying that SMTP is not invoked and appropriate logging occurs when recipients are empty or whitespace-only

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Stale pull request message