Skip to content

fix: Track and report partial send failures in sendDigest (apn-obg9x)#295

Merged
paulvanbrenk merged 1 commit intomainfrom
merge/apn-al1amv
Feb 16, 2026
Merged

fix: Track and report partial send failures in sendDigest (apn-obg9x)#295
paulvanbrenk merged 1 commit intomainfrom
merge/apn-al1amv

Conversation

@paulvanbrenk
Copy link
Collaborator

Summary

  • Adds failure tracking to sendDigest: collects per-recipient send results with sent/failed/skipped counts
  • Logs a summary line after processing all users
  • Throws an error with failed recipient details if any sends fail, replacing the previous silent-success behavior

Test plan

  • All 392 .NET tests pass (dotnet test PatchNotes.slnx)
  • Includes 189-line test file for sendDigest failure scenarios
  • Clean rebase onto current main

MR: apn-al1amv | Bead: apn-obg9x | Worker: rictus

Collect send results during the digest loop and throw an error if any
sends fail, so the function invocation is properly marked as failed
instead of silently returning success. Adds vitest and regression tests.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@paulvanbrenk paulvanbrenk merged commit 0d2042d into main Feb 16, 2026
4 checks passed
@paulvanbrenk paulvanbrenk deleted the merge/apn-al1amv branch February 16, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant