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

Addget_teams andget_team_members tools#834

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

Merged
mattdholloway merged 29 commits intomainfromteams-tool
Aug 13, 2025
Merged

Conversation

mattdholloway
Copy link
Contributor

@mattdhollowaymattdholloway commentedAug 7, 2025•
edited
Loading

This PR adds two new tools in the context_tools toolset:

  • get_team_members - Get team members

    • org: Organization login (owner) that contains the team. (string, required)
    • team_slug: Team slug (string, required)

  • get_teams - Get teams

    • user: Username to get teams for. If not provided, uses the authenticated user. (string, optional)

Closes:https://github.com/github/copilot-agent-services/issues/270 and#366

@mattdhollowaymattdholloway self-assigned thisAug 7, 2025
@mattdhollowaymattdholloway marked this pull request as ready for reviewAugust 7, 2025 14:07
@mattdhollowaymattdholloway requested a review froma team as acode ownerAugust 7, 2025 14:07
@CopilotCopilotAI review requested due to automatic review settingsAugust 7, 2025 14:07
Copilot

This comment was marked as outdated.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mattdhollowaymattdholloway changed the titleAddget_my_teams toolAddget_teams toolAug 8, 2025
Copilot

This comment was marked as outdated.

Copy link
Contributor

@CopilotCopilotAI 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 PR adds a newget_teams tool to retrieve GitHub team memberships for users across their organizations.

  • IntroducesGetTeams function that uses both REST and GraphQL APIs to fetch team data
  • Adds comprehensive test coverage for the new tool including success, error, and edge cases
  • Updates the default toolset to include the new teams tool in the context tools group

Reviewed Changes

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

Show a summary per file
FileDescription
pkg/github/tools.goRegisters the new GetTeams tool in the context toolset
pkg/github/context_tools.goImplements the GetTeams function with GraphQL querying logic
pkg/github/context_tools_test.goAdds comprehensive test suite for the GetTeams functionality
pkg/github/toolsnaps/get_teams.snapTool snapshot for testing framework
README.mdDocuments the new get_teams tool in the available tools section

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mattdhollowaymattdholloway changed the titleAddget_teams toolAddget_teams andget_team_members toolsAug 11, 2025
Copy link
Contributor

@CopilotCopilotAI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@mattdhollowaymattdholloway merged commit1832210 intomainAug 13, 2025
16 of 17 checks passed
@mattdhollowaymattdholloway deleted the teams-tool branchAugust 13, 2025 12:41
nickytonline pushed a commit to nickytonline/github-mcp-http that referenced this pull requestOct 4, 2025
* add team tool with tests* add to tools* add toolsnaps and docs* Update pkg/github/context_tools.goCo-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>* rewrite to allow providing user* rRename get_my_teams to get_teams and update documentation and tests* remove old snap* rm old comments* update test teams to numbered examples* Update descriptions for allow finding teams of other users* return empty result over custom empty error* fix test expectations for no teams found* flatten teams response to not include Nodes* update description to include clarification about teams you are a member of* fix typo in tool desc* updated description to be more generic for accecss note* amended error handling* Update pkg/github/context_tools.goCo-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>* add additional tool to get team members* update tool desc for get_team_members to include warning about auth* added new scope info* refactor to parse params individually* GetTeams - rename "login" field to "org"---------Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>Co-authored-by: LuluBeatson <lulubeatson@github.com>Co-authored-by: Lulu <59149422+LuluBeatson@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

Copilot code reviewCopilotCopilot left review comments

@IrynaKulakovaIrynaKulakovaIrynaKulakova approved these changes

@LuluBeatsonLuluBeatsonLuluBeatson approved these changes

Assignees

@mattdhollowaymattdholloway

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@mattdholloway@IrynaKulakova@LuluBeatson
[8]ページ先頭
©2009-2025 Movatter.jp