Add category to moderator notification emails

[dthaler]

• If the "comments" field is populated in unreviewed detections, use it to get a category to use in the emails.
• Instead of just saying "Orca" for SRKW in one place, use "Southern Resident Killer Whale" for consistency.
• Instead of saying "PDT" when it is actually PST for half the year, just say "Pacific"
• Add unit tests for moderator email methods

[Copilot]

Pull request overview

This PR updates the NotificationSystem’s email generation to include a whale category (e.g., SRKW vs transient vs humpback) in both moderator and subscriber notifications, and expands unit test coverage for the email template helpers.

Changes:

• Extend EmailTemplate email body/subject APIs to accept a category, and add GetCategory(comments) + GetModeratorEmailSubject(...).
• Update SendModeratorEmail/SendSubscriberEmail to pass category into subjects/bodies.
• Add/extend unit tests covering moderator email bodies/subjects and category derivation.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File	Description
NotificationSystem/NotificationSystem/Template/EmailTemplate.cs	Adds category-aware email subject/body generation and GetCategory helper.
NotificationSystem/NotificationSystem/SendSubscriberEmail.cs	Passes a category into subscriber email subject/body generation.
NotificationSystem/NotificationSystem/SendModeratorEmail.cs	Derives category from document comments and uses it in moderator subject/body generation.
NotificationSystem/NotificationSystem.Tests.Unit/EmailTemplateTests.cs	Adds/updates unit tests for category-aware email methods and GetCategory.

Comments suppressed due to low confidence (1)

NotificationSystem/NotificationSystem/Template/EmailTemplate.cs:146

• Same issue as subscriber email: the moderator template hardcodes "PDT" in the copy, which will be incorrect during standard time for some timestamps. Use a neutral label ("PT") or compute PST/PDT dynamically to match the actual conversion performed.

                {category} Call Candidate
                </h1>
                <p>
                Dear moderator, a potential {category} call was detected on {timeString} PDT at {location} location. 
                </p>

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

[dbainj1]

Historically, the system was developed to detect southern residents, which is why defaults and hard coding apply to them. It's only in the last few months that we've started giving serious thought to having multiple categories. Get Outlook for Android<https://aka.ms/AAb9ysg>

________________________________ From: Tong W. ***@***.***> Sent: Monday, May 25, 2026 10:31:38 PM To: orcasound/orcahello ***@***.***> Cc: David Bain ***@***.***>; Review requested ***@***.***> Subject: Re: [orcasound/orcahello] Add category to moderator notification emails (PR #507) @tongwynn commented on this pull request.

________________________________ In NotificationSystem/NotificationSystem/SendSubscriberEmail.cs<#507 (comment)>:

@@ -64,8 +64,9 @@ public async Task Run(

foreach (var message in messages) { string location = EmailTemplate.GetLocation(message); - string emailSubject = EmailTemplate.GetSubscriberEmailSubject(location); - string body = CreateBody(message); + string category = "Southern Resident Killer Whale"; why is the category hard coded? — Reply to this email directly, view it on GitHub<#507?email_source=notifications&email_token=AWBBIJYLYN4OK342PUQNSGD44UT3VA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIMZWGA4DGMBVGMYKM4TFMFZW63VQOJSXM2LFO5PXEZLROVSXG5DFMSSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#pullrequestreview-4360830530>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AWBBIJ323HQ7AZZWY7HZ7LL44UT3VAVCNFSM6AAAAACZKPHBLKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DGNRQHAZTANJTGA>. Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because your review was requested.Message ID: ***@***.***>

[dthaler]

FYI, I would like to merge #508 (which adds more tests) before this PR, and rebase this on top of it before this is merged. So I need reviews of #508 too, thanks!