- Notifications
You must be signed in to change notification settings - Fork1.4k
Add Dependabot Alert Tools#631
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This PR introduces a new Dependabot toolset that allows fetching and listing Dependabot alerts in GitHub repositories.
- Registers a
dependabottoolset in the main tool group with two read-only tools. - Implements
get_dependabot_alertandlist_dependabot_alertshandlers with full request/response logic. - Adds tests and updates documentation to expose the new tools.
Reviewed Changes
Copilot reviewed 5 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/github/tools.go | Registered thedependabot toolset and addedToStringPtr |
| pkg/github/dependabot.go | Added implementations for the two Dependabot alert tools |
| pkg/github/dependabot_test.go | Created unit tests covering success and error scenarios |
| docs/remote-server.md | Updated toolsets table to include Dependabot |
| README.md | Listed Dependabot tools and detailed their parameters |
Files not reviewed (2)
- pkg/github/toolsnaps/get_dependabot_alert.snap: Language not supported
- pkg/github/toolsnaps/list_dependabot_alerts.snap: Language not supported
23f6f3a intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Add new
dependabottoolset containing:get_dependabot_alertlist_dependabot_alertsExample prompts:
Note:
For filtering alerts when listing them, I've included state (default open) and severity enums. But they could have instead been comma-separated strings.
Closes: