Claude/overmanager alert control sync dsu4ko#30
Merged
proofofprints merged 6 commits intoJun 19, 2026
Conversation
…obile acknowledge_alert previously only flipped the local flag — nothing reached the cloud, so a read marked on the desktop never showed up on the web portal or mobile app. It now POSTs the alert's (ruleName, minerIp, timestamp) tuple to the new /ingest/alert-read endpoint after marking it locally, falling back to the offline queue (drained by the sync loop) when offline or on failure. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01YYr4XG3SBHoSpP33d84Jqa
… queue acknowledge_alert always returned Ok(()) regardless of whether the cloud push succeeded, and the success path had no logging at all — making it impossible to tell from the console whether an ack ever reached the cloud. Also the offline queue's CHECK constraint never included 'alert-read' (or 'alerts-read-all'), so a queued ack while offline would silently fail to insert.
…ert-control-sync-dsu4ko # Conflicts: # src-tauri/src/commands/alerts.rs
A 5-minute suppression window after launch made the alerting system look broken to users who'd just opened the app — no alerts fire, no ack to test. 30s is enough for a poll cycle or two to establish a baseline for stateful rules (NoShares, MinerOffline) without the long dead zone.
Elevate the cloud->desktop "no local match" branch from debug to warn (was indistinguishable from "message never arrived" at default log level) and log receipt of every alert-read WS message. Also log the two no-op paths in acknowledge_alert (already-acked, unknown id) that previously failed silently with zero console output.
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
2 participants
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
Changes
Checklist
npx tsc --noEmitpasses with no errorscd src-tauri && cargo checkpasses with no errors#[serde(default)]lib.rsinvoke_handler!Testing
Related issues