feat: Add create org invitation tool #1226
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR introduces a new GitHub organization invitation tool that allows creating invitations for users to join an organization through the MCP server. The tool supports inviting users by either GitHub user ID or email address and includes comprehensive validation and error handling.
- Added
CreateOrgInvitationfunction to create organization invitations via GitHub API - Integrated the new tool into the existing toolset framework as a write operation
- Comprehensive test coverage for various scenarios including success cases, validation errors, and API failures
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| pkg/github/tools.go | Integrates the new CreateOrgInvitation tool into the orgs toolset as a write operation |
| pkg/github/orgs.go | Implements the core CreateOrgInvitation function with parameter validation and GitHub API integration |
| pkg/github/orgs_test.go | Provides comprehensive test coverage for the CreateOrgInvitation functionality |
| pkg/github/toolsnaps/create_org_invitation.snap | Contains the tool definition snapshot for testing validation |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
issei-m
force-pushed
the
org
branch
2 times, most recently
from
8966fd0 to
46389d6
Compare
issei-m
changed the title
* Consolidating tools draft * Consolidate pullrequest tools * Prompt tweaks * Fixes
… tools (…" (github#1187) This reverts commit 8a69a5c.
* tighten content filtering logic * add pdf retrieval test case * add application/xml check
* add sort and order params * test adding server instructions for sorting * improve server instructions * reorder
* adding labels tools * fixing ci * update toolsnap * fixing lint * changing comment * fix title
* Add update project item tool * Update docs * Add tests and tool snaps * Remove unnecessary comments * Formatting and fixes * Extract error messages to const * Fix json tag * Rename field * Update params * Update test * Update tool example
* rename tooling * fix double tool
* Improve registry release reliablilty * Remove trigger from docker build pipeline * Remove whitespace and publish from version release * Add whitespave for neater pr
Default to ignore unknown toolsets, with the option to return an error.
* Consolidate pull request review tools * Prompt tweaks + deleting snap * Server instructions change * Add enums * Remove excessive mentions of event parameter * Doc update
* restructure readme for better ghes ghec visibility * adding readme
* Add "trust" option to MCP server configuration for bypassing confirmations * Additional configuration * Add co-author Co-authored-by: Michael Vorburger <[email protected]> --------- Co-authored-by: Michael Vorburger <[email protected]>
* adding better response * adding check for subdomain isolation and return raw resp for better better llm response * adding subdomain for uploads too * remove unnecessary comments * better error message * fix linter
* Add additional toolsets in remote mcp * Add copilot * Fix urls * Move copilot and spaces toolsets to remote-only section --------- Co-authored-by: LuluBeatson <[email protected]>
* docs: add github_support_docs_search to REMOTE server * add copilot_spaces * fix typos * Add support tool to remote toolsets table
* add toolset default to make configuration easier * fix readme * adding transformer to cleanly handle special toolsets * cleaning code * fixing cli message * remove duplicated test * Update internal/ghmcp/server.go Co-authored-by: Copilot <[email protected]> * Update internal/ghmcp/server_test.go Co-authored-by: Copilot <[email protected]> * adding error message for invalid toolsets * fix merge conflict * add better formatting --------- Co-authored-by: Copilot <[email protected]>
* Add gemini manifest * update gemini cli docs * update docker PAT env name * Add trailing `/` for consistency Co-authored-by: Copilot <[email protected]> * Add MCP Server description * Readd remote manual steps * Clarify method names --------- Co-authored-by: Copilot <[email protected]>
Fix environment variable mapping for read-only mode configuration Add SetEnvKeyReplacer to viper configuration to properly map environment variables with underscores to flag names with dashes. This enables the documented GITHUB_READ_ONLY=1 environment variable to work correctly. Without this fix, viper was looking for GITHUB_READ-ONLY (with dash) but the documentation and standard convention use GITHUB_READ_ONLY (with underscore). Fixes issue where read-only mode was not being activated when using GITHUB_READ_ONLY=1 in Docker containers.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I've added only the create-invitation, since that’s the only one I need at the moment.
And this is the first pull-req for this repository, so I'm not sure whether other tools for org should be implemented as well. (if so, I can do that)