fix: add ClickHouse-specific UPDATE syntax for alert status updates

[devin-ai-integration]

Summary

Fixes ClickHouse syntax error (Code 62) when running edr monitor. ClickHouse does not support standard SQL UPDATE statements - it requires ALTER TABLE ... UPDATE syntax for mutations.

This PR adds adapter dispatch for the update_skipped_alerts and update_sent_alerts macros to use the correct ClickHouse syntax when running on ClickHouse adapters. The default implementation preserves existing behavior for all other adapters.

Error being fixed:

Code: 62. DB::Exception: Syntax error: failed at position 1008 (FORMAT)...

Root cause: The macros were generating UPDATE table SET ... which ClickHouse doesn't support. ClickHouse requires ALTER TABLE table UPDATE ....

Review & Testing Checklist for Human

• Verify ClickHouse syntax is correct - The ALTER TABLE ... UPDATE syntax should match ClickHouse documentation. I have not tested this on an actual ClickHouse instance.
• Test on ClickHouse environment - Run edr monitor on a ClickHouse setup where a model previously failed to confirm the fix works end-to-end
• Verify adapter dispatch namespace - Confirm "elementary_cli" is the correct namespace for the dispatch pattern in this context

Recommended Test Plan

1. Set up a ClickHouse environment with Elementary
2. Create a scenario where a model fails (to trigger alert creation)
3. Run edr monitor and verify it completes without the syntax error
4. Verify alerts are properly updated to 'sent' or 'skipped' status in the alerts_v2 table

Notes

• Pattern follows existing ClickHouse-specific implementations in dbt-data-reliability/macros/utils/table_operations/delete_and_insert.sql
• Link to Devin run: https://app.devin.ai/sessions/e91ca640d865476d89fd10dfac12fe35
• Requested by: @arbiv

Summary by CodeRabbit

• Refactor
  • Enhanced alert management system with improved database engine support, including ClickHouse compatibility.
  • Optimized sent and skipped alert update operations for better performance and flexibility across different database platforms.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment and CI monitoring

[github-actions]

👋 @devin-ai-integration[bot]
Thank you for raising your pull request.
Please make sure to add tests and document all user-facing changes.
You can do this by editing the docs files in this pull request.

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR refactors two alert management macros to use adapter dispatch pattern. It replaces inline SQL query construction with delegated calls to dispatcher macros that dynamically select engine-specific implementations for updating sent and skipped alerts.

Changes

Cohort / File(s)	Summary
Alert Management Refactoring elementary/monitor/dbt_project/macros/alerts/manage/update_sent_alerts.sql	Introduced adapter dispatch pattern: added get_update_sent_alerts_query(alert_ids, sent_at) dispatcher macro and engine-specific variants (default__get_update_sent_alerts_query, clickhouse__get_update_sent_alerts_query) to centralize query selection. Main update_sent_alerts macro now delegates query construction instead of inlining SQL.
Alert Management Refactoring elementary/monitor/dbt_project/macros/alerts/manage/update_skipped_alerts.sql	Applied same adapter dispatch refactoring: added get_update_skipped_alerts_query(alert_ids) dispatcher and engine-specific implementations (default__get_update_skipped_alerts_query, clickhouse__get_update_skipped_alerts_query). Updated update_skipped_alerts to use delegated query execution via dispatcher.

Estimated Code Review Effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 Hopping through the alert dispatch,
Where queries find their database match,
ClickHouse and default, side by side,
Adapter patterns, our whisker-guide,
Refactored paths make our dbt hop with pride!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: adding ClickHouse-specific UPDATE syntax for alert status updates, which is the core purpose of refactoring update_sent_alerts and update_skipped_alerts macros.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}