Skip to content

feat: tool for retrieving agentic investigation reports#23

Merged
samuelmatos-s1 merged 1 commit into
mainfrom
feat/investigation-report
Jun 26, 2026
Merged

feat: tool for retrieving agentic investigation reports#23
samuelmatos-s1 merged 1 commit into
mainfrom
feat/investigation-report

Conversation

@samuelmatos-s1

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown

AI Analysis

Here are some suggestions to consider:

  • src/purple_mcp/tools/alerts.py:859 and src/purple_mcp/libs/alerts/client.py:550 add the agentic investigation-report path, but there are no unit tests covering the new client method or the MCP tool. Other alert helpers are exercised in tests/unit/libs/alerts/test_client.py:630 and tests/unit/test_alerts_tools.py:244, so please add parallel coverage for success/empty/error flows (and any validation trimming you plan) so this new surface meets the project’s comprehensive-testing expectations.
  • README.md:228 is still listing the pre-existing alert endpoints, so please document get_alert_investigation_report alongside the others so users know the feature exists and the available tools page stays accurate.

These suggestions are generated by gpt-5.1-codex-mini as an experimental capability. It may make mistakes. You do not need to follow all these suggestions.

@samuelmatos-s1
samuelmatos-s1 marked this pull request as ready for review June 26, 2026 13:56
@samuelmatos-s1
samuelmatos-s1 merged commit 07d4992 into main Jun 26, 2026
17 checks passed
@samuelmatos-s1
samuelmatos-s1 deleted the feat/investigation-report branch June 26, 2026 13:57
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.

3 participants