Skip to content

Conversation

lanej
Copy link

@lanej lanej commented Sep 28, 2025

Summary

Add list_incident_alerts tool to retrieve detailed alert context for incidents, enabling better troubleshooting and incident analysis.

Changes

  • New Alert Models: Alert, AlertBody, AlertStatus, AlertSeverity with proper PagerDuty API response handling
  • New Tool: list_incident_alerts - retrieve alerts associated with specific incidents
  • Enhanced References: Added IntegrationReference for alert-integration relationships
  • Comprehensive Tests: Unit tests for models and tools, evaluation test framework
  • Clean Integration: Follows existing patterns, proper tool registration

Features

  • Rich alert context including metrics, thresholds, and monitoring links
  • Proper handling of complex PagerDuty API response structures
  • Consistent with existing tool patterns and safety model
  • Comprehensive test coverage with real API response validation

Test Results

  • ✅ All 161+ tests passing
  • ✅ Real API integration verified
  • ✅ MCP protocol compliance confirmed

API Scope

This implementation focuses on the working /incidents/{id}/alerts endpoint accessible with user tokens. Standalone alert search endpoints were excluded due to API access limitations.

🤖 Generated with Claude Code

Add list_incident_alerts tool to retrieve detailed alert context for incidents:
- Create Alert, AlertBody, AlertStatus, AlertSeverity data models
- Add IntegrationReference to handle alert-integration relationships
- Implement proper PagerDuty API response parsing
- Add comprehensive unit and evaluation tests
- Update tool registration and model exports
- Version bump to 0.1.5-dev+alerts

This enables better incident troubleshooting by providing rich alert context
including metrics, thresholds, and monitoring tool links.
@lanej lanej force-pushed the feature/incident-alerts branch from 1261006 to f99343d Compare September 29, 2025 22:13
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.

1 participant