Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

projects: serialization & pagination updates#1390

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

Open
tmelliottjr wants to merge1 commit intomain
base:main
Choose a base branch
Loading
fromtmelliottjr/update-projects-tools

Conversation

@tmelliottjr
Copy link
Contributor

This pull request updates the GitHub Projects API documentation, tool schemas, and unit tests to improve support for pagination, advanced filtering, and response formatting. The changes ensure more accurate descriptions, tighter parameter validation, and consistent output structures for project-related endpoints.

@tmelliottjrtmelliottjr requested a review froma team as acode ownerNovember 11, 2025 15:20
CopilotAI review requested due to automatic review settingsNovember 11, 2025 15:20
@tmelliottjrtmelliottjrforce-pushed thetmelliottjr/update-projects-tools branch frome65f52e toda66054CompareNovember 11, 2025 15:21
Copilot finished reviewing on behalf oftmelliottjrNovember 11, 2025 15:22
Copy link
Contributor

CopilotAI 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

This pull request updates the GitHub Projects API integration to enhance pagination support, improve filtering capabilities, and standardize response formats. The changes involve restructuring API responses to include pagination metadata alongside results, refining parameter descriptions, and removing an unused prompt component.

Key changes:

  • Added cursor-based pagination withafter andbefore parameters, and wrapped responses in objects containing both data andpageInfo
  • Refined parameter descriptions and constraints (e.g., max 50 items per page) for better API clarity
  • Removed theManageProjectItemsPrompt function and its registration from the toolset

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
FileDescription
pkg/github/tools.goRemoved registration of ManageProjectItemsPrompt from project toolset
pkg/github/projects_test.goUpdated tests to handle new response structure with pageInfo, fixed field names, and refined error message assertions
pkg/github/projects.goAdded pagination helpers, updated endpoints to return structured responses with pageInfo, removed ManageProjectItemsPrompt function, added new types for fields and pagination
pkg/github/toolsnaps/*.snapUpdated tool schemas to include new pagination parameters and refined descriptions
README.mdUpdated documentation to reflect new pagination parameters and improved descriptions

💡Add Copilot custom instructions for smarter, more guided reviews.Learn how to get started.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

Copilot code reviewCopilotCopilot left review comments

@JoannaaKLJoannaaKLJoannaaKL approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tmelliottjr@JoannaaKL

[8]ページ先頭

©2009-2025 Movatter.jp