feat(enterprise): add alert management commands (#300) (#307)

- List and filter alerts by type and severity
- Get specific alerts by UID
- View cluster, node, and database alerts
- Manage alert settings
- Full JMESPath query support
- Comprehensive mdBook documentation

Author: joshrotenberg

ENTERPRISE_AUDIT.md

@@ -0,0 +1,71 @@
+# Enterprise Command Coverage Audit
+
+## ✅ Implemented Commands (23/31 handlers - 74% coverage)
+
+| Handler | Command File | Status |
+|---------|-------------|---------|
+| actions.rs | actions.rs | ✅ Merged |
+| bdb_groups.rs | bdb_group.rs | ✅ Merged |
+| bdb.rs | database.rs | ✅ Merged |
+| cluster.rs | cluster.rs | ✅ Merged |
+| cm_settings.rs | cm_settings.rs | ✅ Merged |
+| crdb_tasks.rs | crdb_task.rs | ✅ Merged |
+| crdb.rs | crdb.rs | ✅ Merged |
+| diagnostics.rs | diagnostics.rs | ✅ Merged |
+| endpoints.rs | endpoint.rs | ✅ Merged |
+| job_scheduler.rs | job_scheduler.rs | ✅ Merged |
+| jsonschema.rs | jsonschema.rs | 🔄 PR #298 |
+| logs.rs | logs.rs | ✅ Merged |
+| migrations.rs | migration.rs | 🔄 PR #299 |
+| modules.rs | module.rs | ✅ Merged |
+| nodes.rs | node.rs | ✅ Merged |
+| proxies.rs | proxy.rs | ✅ Merged |
+| redis_acls.rs | rbac.rs | ✅ Merged (combined) |
+| roles.rs | rbac.rs | ✅ Merged (combined) |
+| shards.rs | shard.rs | ✅ Merged |
+| stats.rs | stats.rs | ✅ Merged |
+| suffixes.rs | suffix.rs | 🔄 PR #295 |
+| usage_report.rs | usage_report.rs | ✅ Merged |
+| users.rs | rbac.rs | ✅ Merged (combined) |
+
+## ❌ Not Implemented (8/31 handlers)
+
+| Handler | Description | Priority |
+|---------|-------------|----------|
+| alerts.rs | Cluster/node/database alert management | Low |
+| bootstrap.rs | Cluster bootstrap and initialization | Low |
+| debuginfo.rs | Debug information collection | Low |
+| ldap_mappings.rs | LDAP integration mappings | Low (partial in rbac) |
+| license.rs | License management and info | Medium |
+| local.rs | Local node-specific operations | Low |
+| ocsp.rs | OCSP certificate validation | Low |
+| services.rs | Service configuration management | Low |
+
+## Summary
+
+### Progress Today
+- Started with ~50% coverage
+- Added **10 new command modules** in one session
+- Achieved **74% coverage** of all Redis Enterprise handlers
+- Created **7 PRs** with full documentation and tests
+
+### Key Achievements
+- **Complete async operation support** across all create/update/delete operations
+- **Consolidated RBAC** - Combined users, roles, and ACLs into single cohesive module
+- **Full documentation** - Every command has comprehensive mdBook documentation
+- **Test coverage** - All commands have unit tests
+- **Clean code** - All PRs pass linting and clippy checks
+
+### Remaining Work
+The 8 unimplemented handlers are mostly low-priority administrative functions:
+- **Alerts** - Could be useful for monitoring integration
+- **License** - Might be worth adding for license compliance checking
+- **Bootstrap/Local/Services** - Very specialized, rarely used
+- **OCSP/Debug** - Advanced features for specific scenarios
+
+### Recommendation
+The current 74% coverage includes all the critical operational commands. The remaining handlers are specialized administrative functions that most users won't need. Consider implementing:
+1. **license.rs** - For license compliance monitoring
+2. **alerts.rs** - For monitoring integration
+
+The others can be added on-demand if users request them.

crates/redisctl/src/cli.rs

@@ -962,6 +962,9 @@ pub enum EnterpriseCommands {
     /// Action (task) operations
     #[command(subcommand)]
     Action(crate::commands::enterprise::actions::ActionCommands),
+    /// Alert management operations
+    #[command(subcommand)]
+    Alerts(crate::commands::enterprise::alerts::AlertsCommands),
     /// Database group operations
     #[command(subcommand, name = "bdb-group")]
     BdbGroup(crate::commands::enterprise::bdb_group::BdbGroupCommands),