Skip to content

Conversation

JoannaaKL
Copy link
Contributor

@JoannaaKL JoannaaKL commented Sep 26, 2025

This pr adds:

  • get_project_field tool for user and org
  • get_project_item tool for user and org
  • list_project_items tool for user and org

Updates:

  • projectNumber tool param to project_number

After this PR is merged last chunk of project tools to be added are:

  • add_project_item
  • delete_project_item
  • update_project_item

It's worth noting that we are returning minimal project v2 and not the full response.
Implements #44

@JoannaaKL JoannaaKL marked this pull request as ready for review September 26, 2025 13:07
@JoannaaKL JoannaaKL requested a review from a team as a code owner September 26, 2025 13:07
@Copilot Copilot AI review requested due to automatic review settings September 26, 2025 13:07
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds a new get_project_field tool for retrieving specific project fields from GitHub projects for both users and organizations. This PR implements issue #44 by creating functionality to fetch individual project field details by field ID.

  • Implements GetProjectField function with support for user and org projects
  • Adds comprehensive test coverage for the new tool including error cases
  • Updates existing list_project_fields tool to use correct numeric type for project numbers

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pkg/github/projects.go Adds GetProjectField function and fixes projectNumber type in ListProjectFields
pkg/github/projects_test.go Adds comprehensive test cases for GetProjectField tool
pkg/github/toolsnaps/get_project_field.snap Tool snapshot for new get_project_field tool
pkg/github/toolsnaps/list_project_fields.snap Updated snapshot reflecting projectNumber type change
README.md Updated documentation to reflect projectNumber type change

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@JoannaaKL JoannaaKL changed the title Add get project fields tool Add get project field tool Sep 26, 2025
@JoannaaKL JoannaaKL marked this pull request as draft September 26, 2025 14:12
@JoannaaKL JoannaaKL changed the title Add get project field tool Add get_project_field tool Sep 26, 2025
@JoannaaKL JoannaaKL marked this pull request as ready for review September 26, 2025 15:21
@JoannaaKL JoannaaKL changed the title Add get_project_field tool Add tool for project fields and items Sep 29, 2025
@JoannaaKL JoannaaKL requested a review from kerobbi September 29, 2025 12:32
kerobbi
kerobbi previously approved these changes Sep 29, 2025
Copy link
Contributor

@kerobbi kerobbi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a non blocking nit, everything else looks solid!

@JoannaaKL JoannaaKL merged commit 0885601 into main Sep 29, 2025
16 checks passed
@JoannaaKL JoannaaKL deleted the add-discussion-tools branch September 29, 2025 13:43
nickytonline pushed a commit to nickytonline/github-mcp-http that referenced this pull request Oct 4, 2025
* Add get project fields tool

* Data types

* Docs

* Update projectNumber's type

* Add list_project_items tool

* Add get_project_item tool

* Return minimal project

* Remove unused per_page
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.

2 participants