- Notifications
You must be signed in to change notification settings - Fork2.7k
Add tools to add, update and delete project items#1152
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 adds write operations for GitHub project items, enabling users to create, update, and delete project items programmatically. The implementation follows the existing pattern for read-only operations and includes comprehensive test coverage.
- Add three new write tools: AddProjectItem, DeleteProjectItem, and UpdateProjectItem
- Each tool supports both user and organization project contexts with proper parameter validation
- Comprehensive test coverage with various success and error scenarios
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/github/tools.go | Registers the new write tools in the toolset group |
| pkg/github/projects.go | Implements the three new project item manipulation functions with proper error handling |
| pkg/github/projects_test.go | Adds comprehensive test coverage for AddProjectItem including edge cases |
| pkg/github/toolsnaps/add_project_item.snap | Generated snapshot for tool schema validation |
| README.md | Updates documentation with the new tool descriptions and parameters |
Tip: Customize your code reviews with copilot-instructions.md.Create the file orlearn how to get started.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
b52fc1d tof4bbd39Compare| <summary>Projects</summary> | ||
| -**add_project_item** - Add project item | ||
| -`item_id`: The numeric ID of the issue or pull request to add to the project. (number, required) |
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.
item_id here means pr id below specifically something else, would this benefit from a new distinct name, or below(not the issue or pull request ID).
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.
It can mean either an issue id or a pr id, depending on the item type
6793b9d intogithub:mainUh oh!
There was an error while loading.Please reload this page.
* Add add_project_item tool* Add tools to update and delete project items* Update pkg/github/projects.goCo-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>* Update pkg/github/projects.goCo-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>* Add tests* Lint the code* Fix req params---------Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Uh oh!
There was an error while loading.Please reload this page.
This PR adds:
Last chunk of work to add tools wrapping Projects V2https://docs.github.com/en/rest/projects/items?apiVersion=2022-11-28
Implements#44