Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add: GitHub alert aggregator + optimized message format + GitHub tool for sre assistant #22

Merged
merged 8 commits into from
Nov 29, 2024

Conversation

schobele
Copy link
Collaborator

This pull request introduces several enhancements and new features to the src/aggregator module, focusing on integrating GitHub context aggregation, improving error handling, and updating the context analysis process. The most important changes include adding a new GitHub aggregator, updating the context analysis prompts, and modifying the structure of job logs.

GitHub Aggregation Integration:

  • Added a new githubAggregator to fetch context from GitHub repositories, including recent commits, pull requests, and releases. (src/aggregator/ContextAggregator.ts, src/aggregator/github-aggregator.ts, src/aggregator/github-aggregator.spec.ts) [1] [2] [3] [4]

Context Analysis Improvements:

  • Updated the context analysis prompts to provide more detailed and structured information, including a new format for presenting context and guidelines for generating summaries. (src/aggregator/chains.ts) [1] [2]

Error Handling Enhancements:

  • Improved error handling in the aggregate method of CheckContextAggregator to log errors and return an empty array if context fetching fails. (src/aggregator/ContextAggregator.ts)

Job Log Structure Update:

  • Modified the structure of jobLog in ErrorMessage to separate logs into setup, request, and teardown phases. (src/checkly/models.ts) [1] [2]

Miscellaneous Updates:

  • Updated test data in checkly-aggregator.spec.ts to reflect new check names and IDs. (src/aggregator/checkly-aggregator.spec.ts)
  • Added a name property and logging to the checklyAggregator for better traceability. (src/aggregator/checkly-aggregator.ts)

These changes collectively enhance the functionality and robustness of the context aggregation and analysis processes, particularly with the integration of GitHub data.

@schobele schobele merged commit adc07ab into main Nov 29, 2024
1 check passed
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