CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

feat(spine): red probes and error spikes now REACH the owner #5471

Merged⚡ AI-generatedXSccantynz wants to mergefeat/spine-alert-fanoutmainopened 23d ago
ccantynzcommented 23d ago

The spine's transition detector existed and worked — and delivered every alert to MONITOR_ALERT_WEBHOOK_URL, an env var nothing sets in production. Detection without delivery: the spine could see, but had no mouth.

New spine-alert-fanout:

  • green→red edges, batched — a full outage flips ~14 checks in one tick and produces ONE message, not fourteen
  • recovery notices when checks return to green (an outage email with no recovery email leaves the reader assuming the worst)
  • new-error spikes — 3+ novel platform_errors fingerprints inside 10 minutes, the signature of a broken deploy even while every endpoint still answers 200; 1-hour cooldown

Every site admin gets an in-app notification AND an email (Resend when configured; degrades to log). Nothing in the path can throw into the monitor tick. Unit tests exercise batching, edge-triggering, recovery, and the fresh-DB case via the injected fanOut dep.

Part 2 of the reliability build (1 = external heartbeat on the GitHub mirror, live; next: deploy rollback, CI test gate, synthetic journeys).

🤖 Generated with Claude Code

ccantynzAI Reviewcommented 23d ago

AI Triage

(no summary)

Priority: medium Risk area: mixed

Suggested labels: (no label suggestions) Suggested reviewers: (no reviewer suggestions)

Suggestions only — nothing has been applied. The PR author stays in control.

gluecron[bot]🤖 botAI Reviewcommented 23d ago

AI review unavailable

The platform's AI balance is exhausted, so AI generation is temporarily unavailable. Nothing was lost. You can queue this as a repair for the internal agent from the repository's Health page, or try again once the balance is restored. The PR is otherwise unchanged.

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
⮌ Merged

This pull request was merged into main.

c comment · e edit title · m merge · a approve · r request changes · ? shortcuts