Skip to content

chore: further cleanup, removed duplicate files, tests for get_alert_investigation_report, missing camelCase for AI Investigation report output - #24

Merged
samuelmatos-s1 merged 1 commit into
mainfrom
chore/small-adjustments
Jul 1, 2026
Merged

chore: further cleanup, removed duplicate files, tests for get_alert_investigation_report, missing camelCase for AI Investigation report output#24
samuelmatos-s1 merged 1 commit into
mainfrom
chore/small-adjustments

Conversation

@samuelmatos-s1

Copy link
Copy Markdown
Contributor

No description provided.

…investigation_report, missing camelCase for AI Investigation report output
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

AI Analysis

Here are some suggestions to consider:

  • No blocking issues spotted; the new investigation-report tests (tests/unit/tools/test_alerts_investigation_report.py:1) together with the client coverage (tests/unit/libs/alerts/test_client.py:757) exercise success/null/error paths, AIInvestigation now emits camelCase JSON (src/purple_mcp/libs/alerts/models.py:609), and the standalone fields-validation tests (tests/unit/tools/test_fields_validation.py:1) thoroughly cover the helper, so this branch is already well exercised.

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 July 1, 2026 16:49
@samuelmatos-s1
samuelmatos-s1 merged commit 7ac1eb3 into main Jul 1, 2026
17 checks passed
@samuelmatos-s1
samuelmatos-s1 deleted the chore/small-adjustments branch July 1, 2026 16:49
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