Skip to content

fix: resolve attached recurrences again - #82

Merged
ernescz merged 1 commit into
mainfrom
fix/issue-81-resolved-recurrence
Sep 4, 2026
Merged

fix: resolve attached recurrences again#82
ernescz merged 1 commit into
mainfrom
fix/issue-81-resolved-recurrence

Conversation

@ernescz

@ernescz ernescz commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • atomically reopen a resolved incident when a genuine recurrence attaches, allowing the existing one-shot resolution transition to notify again
  • keep duplicate resolved deliveries silent and make recurrence re-judgment share the same resolution ownership rule
  • serialize per-incident Slack root-card writes so delayed recurrence edits cannot overwrite the resolved card
  • add the first Unreleased fix candidate toward application v0.13.8; chart metadata remains unchanged until the coordinated release step

Closes #81

Verification

  • GOCACHE=/tmp/alertint-issue81-go-cache go test -race -count=1 ./...
  • go build ./...
  • go vet ./...
  • golangci-lint run ./... (0 issues)
  • gosec -quiet ./...
  • independent code review: no remaining critical or important findings

Checklist

  • Tests were added or updated where behavior changed.
  • Documentation was updated where user-facing behavior changed.
  • CHANGELOG.md was updated for a notable user-facing change.
  • Every commit includes a DCO Signed-off-by trailer.

Signed-off-by: ernescz <ernescz@gmail.com>
@ernescz
ernescz merged commit 0cd9011 into main Sep 4, 2026
13 checks passed
@ernescz
ernescz deleted the fix/issue-81-resolved-recurrence branch September 4, 2026 17:39
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.

Reattached occurrences on an already-resolved incident never get re-resolved (or re-notified) — Slack card sticks on "recurred ×N"

1 participant