Skip to content

feat: add retrohunt and alert search tools to security rules MCP#229

Open
mihirvala08 wants to merge 3 commits intomainfrom
feature/adv-rule-operations
Open

feat: add retrohunt and alert search tools to security rules MCP#229
mihirvala08 wants to merge 3 commits intomainfrom
feature/adv-rule-operations

Conversation

@mihirvala08
Copy link
Collaborator

Advanced Security Rules Tools for Chronicle SIEM

Summary

Added Advanced Security Rules Management tools to SecOps MCP for Chronicle SIEM, enabling AI-assisted retrohunts, rule alert searches, and historical threat hunting operations.

What

Added 3 Chronicle SIEM Advanced Security Rules tools:

  • create_retrohunt - Create retrohunt operations to run detection rules against historical data for threat hunting campaigns
  • get_retrohunt - Retrieve status and results of retrohunt operations with progress tracking
  • search_rule_alerts - Search alerts generated by detection rules across time ranges with pagination support

Changelog

  • server/secops/secops_mcp/tools/security_rules.py: Added create_retrohunt, get_retrohunt, and search_rule_alerts tools with comprehensive documentation and error handling.
  • server/secops/tests/test_adv_rules_mcp.py: Integration tests with end-to-end retrohunt workflow and alert search tests with pagination support

@mihirvala08 mihirvala08 marked this pull request as ready for review February 5, 2026 09:48
@dandye
Copy link
Collaborator

dandye commented Feb 6, 2026

Integration Test Evidence

Screenshot 2026-02-06 at 8 27 07 AM Screenshot 2026-02-06 at 8 27 53 AM Screenshot 2026-02-06 at 9 02 34 AM Screenshot 2026-02-06 at 9 05 37 AM

The group continues to evolve tactics and primarily targets financial institutions, cryptocurrency exchanges, and organizations in sectors of strategic interest to North Korea.
```

<<<<<<< HEAD
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mihirvala08 There is a stray merge conflict marker here and on line 700. Please delete these before merging.

Copy link
Collaborator

@dandye dandye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left integration test evidence in the comments. 100% works as intended. Thank you, @mihirvala08 !

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.

2 participants