Skip to content

Docs/incident response playbook - #638

Merged
thlpkee20-wq merged 8 commits into
RevoraOrg:masterfrom
heymide:docs/incident-response-playbook
Jul 29, 2026
Merged

Docs/incident response playbook#638
thlpkee20-wq merged 8 commits into
RevoraOrg:masterfrom
heymide:docs/incident-response-playbook

Conversation

@heymide

@heymide heymide commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Close: #562

All 6 fixes applied and verified:

  1. parseMappingTable — simplified to match any line starting with | ` (no more separator detection needed). Handles both |---|---| and | --- | --- | formats.
  2. Duplicate oidc_discovery_changed — removed the second entry at line 35 of the script.
  3. Missing email_alarm_* entries — added email_alarm_alignment_failure and email_alarm_high_bounce_ratio rows to the README mapping table.
  4. fail() → throw new Error() — Jest 30 removed the global fail() function.
  5. KNOWN_ALERTS mutation leak — added finally block to restore the array after the mutation test.
  6. Verification — npm run validate:alert-mappings exits 0, all 8 Jest tests pass.

@drips-wave

drips-wave Bot commented Jul 28, 2026

Copy link
Copy Markdown

@heymide Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@heymide

heymide commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

Done, Close: #638

@thlpkee20-wq
thlpkee20-wq merged commit d6fcc20 into RevoraOrg:master Jul 29, 2026
1 check failed
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.

Documentation: incident-response playbook mapping alerts to runbooks

2 participants