Merge pull request #1 from springdom/claude/gifted-lovelace

feat: alert tags, notes, silences, notifications & integrations

Author: springdom

README.md

@@ -18,6 +18,10 @@ Open-source alert management and incident response platform. Ingest alerts from
 - **Full audit trail** — Every incident action (created, alert added, severity changed, acknowledged, resolved) is recorded as a timeline event.
 - **Auto-resolve** — When all alerts in an incident resolve, the incident auto-resolves.
 - **Dark ops-console dashboard** — Real-time React UI with incident/alert views, severity badges, detail panels, and one-click ack/resolve.
+- **Alert tagging** — Categorize alerts with free-form tags. Add/remove tags from the detail panel or via API. Stored as JSONB with GIN index.
+- **Investigation notes** — Add timestamped notes to alerts during investigation. Supports author attribution and full CRUD.
+- **Silence windows** — Suppress alerts during maintenance windows with service/severity matchers.
+- **Notification channels** — Configure Slack and email notification channels with severity/service filters.
 
 ## Architecture
 
@@ -70,7 +74,8 @@ curl -X POST http://localhost:8000/api/v1/webhooks/generic \
     "severity": "critical",
     "service": "payment-api",
     "host": "web-01",
-    "description": "CPU usage above 95% for 10 minutes"
+    "description": "CPU usage above 95% for 10 minutes",
+    "tags": ["production", "us-east-1"]
   }'
 
 # Prometheus Alertmanager
@@ -162,10 +167,21 @@ curl -X POST http://localhost:8000/api/v1/webhooks/generic \
 | `GET` | `/api/v1/alerts/{id}` | Get alert by ID |
 | `POST` | `/api/v1/alerts/{id}/acknowledge` | Acknowledge alert |
 | `POST` | `/api/v1/alerts/{id}/resolve` | Resolve alert |
+| `PUT` | `/api/v1/alerts/{id}/tags` | Replace all tags |
+| `POST` | `/api/v1/alerts/{id}/tags/{tag}` | Add a tag |
+| `DELETE` | `/api/v1/alerts/{id}/tags/{tag}` | Remove a tag |
+| `GET` | `/api/v1/alerts/{id}/notes` | List notes |
+| `POST` | `/api/v1/alerts/{id}/notes` | Add a note |
+| `PUT` | `/api/v1/alerts/notes/{id}` | Update a note |
+| `DELETE` | `/api/v1/alerts/notes/{id}` | Delete a note |
 | `GET` | `/api/v1/incidents` | List incidents (filterable) |
 | `GET` | `/api/v1/incidents/{id}` | Get incident with alerts + events |
 | `POST` | `/api/v1/incidents/{id}/acknowledge` | Acknowledge incident + all alerts |
 | `POST` | `/api/v1/incidents/{id}/resolve` | Resolve incident + all alerts |
+| `GET` | `/api/v1/silences` | List silence windows |
+| `POST` | `/api/v1/silences` | Create silence window |
+| `GET` | `/api/v1/notifications/channels` | List notification channels |
+| `POST` | `/api/v1/notifications/channels` | Create notification channel |
 
 ## Configuration
 
@@ -213,14 +229,16 @@ cd frontend && npm install && npm run dev
 
 ## Roadmap
 
-- [ ] Grafana normalizer
-- [ ] Datadog normalizer
+- [x] Grafana normalizer
+- [x] Datadog normalizer
+- [x] Notification channels (Slack, email)
+- [x] Silence / maintenance windows
+- [x] Alert tagging and investigation notes
+- [x] Metrics and SLA tracking (MTTA, MTTR)
 - [ ] On-call scheduling and escalation policies
-- [ ] Notification channels (Slack, email, PagerDuty bridge)
-- [ ] Silence / maintenance windows
 - [ ] RBAC and multi-tenancy
 - [ ] Topology-aware correlation (service dependency graph)
-- [ ] Metrics and SLA tracking (MTTA, MTTR)
+- [ ] PagerDuty bridge
 
 ## License
 

backend/api/routes/alerts.py

@@ -9,13 +9,67 @@
 from backend.schemas import (
     AlertAckRequest,
     AlertListResponse,
+    AlertNoteCreate,
+    AlertNoteListResponse,
+    AlertNoteResponse,
+    AlertNoteUpdate,
     AlertResponse,
+    AlertTagsUpdate,
+)
+from backend.services import (
+    acknowledge_alert,
+    add_alert_tag,
+    create_alert_note,
+    delete_alert_note,
+    get_alert_notes,
+    get_alerts,
+    remove_alert_tag,
+    resolve_alert,
+    update_alert_note,
+    update_alert_tags,
 )
-from backend.services import acknowledge_alert, get_alerts, resolve_alert
 
 router = APIRouter(prefix="/alerts", tags=["alerts"])
 
 
+# ─── Note routes (must be before /{alert_id} to avoid UUID collision) ──
+
+
+@router.put(
+    "/notes/{note_id}",
+    response_model=AlertNoteResponse,
+    summary="Update a note",
+)
+async def edit_note(
+    note_id: uuid.UUID,
+    body: AlertNoteUpdate,
+    db: AsyncSession = Depends(get_db),
+) -> AlertNoteResponse:
+    """Update the text of an existing note."""
+    note = await update_alert_note(db, str(note_id), body.text)
+    if not note:
+        raise HTTPException(status_code=404, detail="Note not found")
+    return AlertNoteResponse.model_validate(note)
+
+
+@router.delete(
+    "/notes/{note_id}",
+    status_code=204,
+    summary="Delete a note",
+)
+async def remove_note(
+    note_id: uuid.UUID,
+    db: AsyncSession = Depends(get_db),
+) -> None:
+    """Delete a note."""
+    deleted = await delete_alert_note(db, str(note_id))
+    if not deleted:
+        raise HTTPException(status_code=404, detail="Note not found")
+
+
+# ─── Alert list & detail ──────────────────────────────────
+
+
 @router.get(
     "",
     response_model=AlertListResponse,
@@ -67,6 +121,9 @@ async def get_alert(
     return AlertResponse.model_validate(alert)
 
 
+# ─── Alert actions ─────────────────────────────────────────
+
+
 @router.post(
     "/{alert_id}/acknowledge",
     response_model=AlertResponse,
@@ -103,3 +160,96 @@ async def resolve(
         raise HTTPException(status_code=404, detail="Alert not found")
 
     return AlertResponse.model_validate(alert)
+
+
+# ─── Tags ──────────────────────────────────────────────────
+
+
+@router.put(
+    "/{alert_id}/tags",
+    response_model=AlertResponse,
+    summary="Set alert tags",
+)
+async def set_tags(
+    alert_id: uuid.UUID,
+    body: AlertTagsUpdate,
+    db: AsyncSession = Depends(get_db),
+) -> AlertResponse:
+    """Replace all tags on an alert."""
+    alert = await update_alert_tags(db, str(alert_id), body.tags)
+    if not alert:
+        raise HTTPException(status_code=404, detail="Alert not found")
+    return AlertResponse.model_validate(alert)
+
+
+@router.post(
+    "/{alert_id}/tags/{tag}",
+    response_model=AlertResponse,
+    summary="Add a tag to an alert",
+)
+async def add_tag(
+    alert_id: uuid.UUID,
+    tag: str,
+    db: AsyncSession = Depends(get_db),
+) -> AlertResponse:
+    """Add a single tag to an alert."""
+    alert = await add_alert_tag(db, str(alert_id), tag)
+    if not alert:
+        raise HTTPException(status_code=404, detail="Alert not found")
+    return AlertResponse.model_validate(alert)
+
+
+@router.delete(
+    "/{alert_id}/tags/{tag}",
+    response_model=AlertResponse,
+    summary="Remove a tag from an alert",
+)
+async def delete_tag(
+    alert_id: uuid.UUID,
+    tag: str,
+    db: AsyncSession = Depends(get_db),
+) -> AlertResponse:
+    """Remove a single tag from an alert."""
+    alert = await remove_alert_tag(db, str(alert_id), tag)
+    if not alert:
+        raise HTTPException(status_code=404, detail="Alert not found")
+    return AlertResponse.model_validate(alert)
+
+
+# ─── Notes ─────────────────────────────────────────────────
+
+
+@router.get(
+    "/{alert_id}/notes",
+    response_model=AlertNoteListResponse,
+    summary="List alert notes",
+)
+async def list_notes(
+    alert_id: uuid.UUID,
+    db: AsyncSession = Depends(get_db),
+) -> AlertNoteListResponse:
+    """Get all notes for an alert."""
+    notes = await get_alert_notes(db, str(alert_id))
+    return AlertNoteListResponse(
+        notes=[AlertNoteResponse.model_validate(n) for n in notes],
+        total=len(notes),
+    )
+
+
+@router.post(
+    "/{alert_id}/notes",
+    response_model=AlertNoteResponse,
+    status_code=201,
+    summary="Add a note to an alert",
+)
+async def add_note(
+    alert_id: uuid.UUID,
+    body: AlertNoteCreate,
+    db: AsyncSession = Depends(get_db),
+) -> AlertNoteResponse:
+    """Add a timestamped note to an alert."""
+    try:
+        note = await create_alert_note(db, str(alert_id), body.text, body.author)
+    except ValueError:
+        raise HTTPException(status_code=404, detail="Alert not found")
+    return AlertNoteResponse.model_validate(note)