fix: Name <email> format for Postmark sender in health workflows

[beveradb]

Summary

The Postmark switch in #9 also silently broke the email-on-failure path. Spotted while investigating run #26064397324 where Postmark replied 501 5.1.7 Bad sender address syntax and the alert was dropped.

dawidd6/action-send-mail@v3 builds the From header like this:

function getFrom(from, username) {
    if (from.match(/.+ <.+@.+>/)) return from
    return `"${from}" <${username}>`
}

Our from: admin@nomadkaraoke.com (no Name <email> form) hit the fallback branch, so the action constructed:

"admin@nomadkaraoke.com" <POSTMARK_SERVER_TOKEN_UUID>

That's not a real mailbox, hence the 501 from Postmark. SendGrid happened to accept the same shape because apikey was the username — Postmark uses the actual token, which makes the bug visible.

Fix: wrap each workflow's from: in "Name <admin@nomadkaraoke.com>" so the regex matches and the value is used verbatim. Also makes alerts distinguishable in the inbox.

Files changed

• .github/workflows/decide-health.yml — Karaoke Decide Health <admin@nomadkaraoke.com>
• .github/workflows/gen-health.yml — Karaoke Gen Health <admin@nomadkaraoke.com>
• .github/workflows/flacfetch-health.yml — Flacfetch Health <admin@nomadkaraoke.com>

Test plan

• YAML parses (python3 -c "import yaml; yaml.safe_load(open(f))" for each).
• Verification of actual email delivery has to wait until the next real health-check failure — there's no harmless way to trigger the notification step without faking a failure.

@coderabbitai ignore

🤖 Generated with Claude Code