McRitchie Studio
Say Hi 👋
Agents Builders
← All Tasks
Sizing

release_notes API: route DiscordClient delivery failures into ErrorLog

Archived
task-fbca827303be

Created

Jun 21, 14:10

Started

Jun 22, 04:47

Completed

Jun 22, 13:55

DevOps handoff

Type

Chore

Shape

backend

Worktree Slug

—

Repositories

mcritchie-studio

Release Train

—

Branch

feat/release-notes-errorlog

Pull Request

https://github.com/amcritchie/mcritchie-studio/pull/86

Local URL

—

QA URL

—

Production URL

—

observability

Acceptance Criteria

  • POST /api/v1/release_notes delivery failures (MissingWebhook/DeliveryError) are recorded to ErrorLog so they surface on the board, not just returned as an API error

Expected Test Plan

  • integration

Checks Run

  • [integration] rails test test/controllers/api/v1/release_notes_controller_test.rb — POST /api/v1/release_notes now records MissingWebhook + DeliveryError to ErrorLog; dry-run logs nothing; 7 runs green
  • [unit] full suite green: 916 runs / 0 failures / 0 errors / 4 skips (Ruby 3.3.11); rubocop clean

Stage Timeline

Who handled each stage, the time it took (measured), and the model / tokens / cost reported (best-effort) — plus who's on it right now. — means the agent didn't report that metric.

No stage changes recorded yet.

Conversation

QA review feedback, agent handoffs, and follow-up notes for this task.

Comment 5 days ago

Avi APPROVED (PR #86). Verified: create_error_log reachable from action, no double-log (typed rescues don't propagate to rescue_from; no @_error_logged interaction), no swallow (non-typed failures still fall through to handle_unexpected_error → logged). MissingWebhook+DeliveryError are the only typed DiscordClient failures; MISSING_TASKS/UNKNOWN_TASKS correctly NOT logged (user input). Ran tests himself 7/30 green. Added his happy-path no-difference nit (now 7/32). Agreed the kind:chore dor-check gap is worth a ticket (groomed).

Sealed-bid sizing

Edit →

Alex (PM)

—

Avi (PO)

—

Dev

—

Actual

—