fix(daemon): deduplicate Dolt alerts and use nudge instead of mail#3474
Open
outdoorsea wants to merge 1 commit intogastownhall:mainfrom
Open
fix(daemon): deduplicate Dolt alerts and use nudge instead of mail#3474outdoorsea wants to merge 1 commit intogastownhall:mainfrom
outdoorsea wants to merge 1 commit intogastownhall:mainfrom
Conversation
github-actions
bot
added
the
status/needs-triage
outdoorsea
force-pushed
the
pr/daemon-dedup-alerts
branch
from
b8b3578 to
c3d5238
Compare
jw409
added a commit
to jw409/gastown-upstream
that referenced
this pull request
Apr 2, 2026
- Add doltAlertTracker with fingerprint-based cooldowns (5min) to prevent wisp storms during Dolt crash loops - Switch sendCrashAlert and sendUnhealthyAlert from gt mail send to gt nudge (ephemeral, zero-cost — no Dolt commit per alert) - Keep escalation on gt mail send (must survive session death) with --wisp-type escalation tag - Remove witness mail fan-out; witnesses detect degraded state via DOLT_UNHEALTHY signal file instead - Reset alert tracker when backoff resets after healthy period - Fix wispTypeToCategory test to match current 2-arg signature - Add nolint:misspell annotations for bead ID references (gt-ect) - Fix serialised → serialized typo in convoy_manager Fixes gastownhall#3474
4 tasks
…q-fo5) - Add doltAlertTracker with 5-minute cooldown to suppress alert storms - Switch sendCrashAlert and sendUnhealthyAlert from gt mail to gt nudge - Remove witness fan-out from crash/unhealthy/escalation alerts (witnesses detect degraded state via DOLT_UNHEALTHY signal file) - Add --wisp-type escalation to sendEscalationMail - Reset alert tracker on recovery in maybeResetBackoff
outdoorsea
force-pushed
the
pr/daemon-dedup-alerts
branch
from
c3d5238 to
c26024e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
gt mail send(permanent Dolt commit) togt nudge(zero cost)Test plan
🤖 Generated with Claude Code