- Notifications
You must be signed in to change notification settings - Fork3.1k
Convert tomodelcontextprotocol/go-sdk#1428
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
omgitsads wants to merge66 commits intomainChoose a base branch fromomgitsads/go-sdk
base:main
Could not load branches
Branch not found:{{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline, and old review comments may become outdated.
+11,083 −8,514
Open
Changes fromall commits
Commits
Show all changes
66 commits Select commitHold shift + click to select a range
d69b616 Move some utilities to go SDK
omgitsads8084335 Remove commented out tool
omgitsads5a92f9c Add intermediate structs for toolsets
omgitsads385dd8d Move to go-sdk IOTransport
omgitsads28bc3f4 Update context tools and tests to use ToolHandlerFor with typed argum…
omgitsads42ada5f comment out broken tools for now, we'll tackle them one by one
omgitsadsafc455d fix schema issues
omgitsadsb2f07e5 Get tests fully working
omgitsads11a154c Add sdk migration agent
omgitsads5483a5b Lets try actually making the changes in the subagent
omgitsadscabc7b6 Be explicit about the files to use
omgitsads99f2d8d Update about toolsnaps
omgitsads6fa137b Add info to allow this to be parallelized
omgitsads2b323c2 Dont use the request var
omgitsads6ce7550 Take any ToolHandlerFor
omgitsads5b4c6df More info about moved files
omgitsads655bcca move files rather than commenting out
omgitsadsc946ada Move files back, use build tags
omgitsads3620126 Update licenses
omgitsads1d257b1 Update agent to use build tags
omgitsads1e7a3f0 Merge branch 'main' into omgitsads/go-sdk
omgitsads9f25ebe Remove dupe import from merge conflict
omgitsadsfaa90d2 fix linter issues
omgitsadsb1ac345 Don't assert without a testing.T
omgitsads1286e06 just return the tool & handler
omgitsads102181d use lowercase strings for the jsonschema types
omgitsads3f03753 Update cmd/github-mcp-server/generate_docs.go
omgitsads407a974 Add Close method to IOLogger to close underlying reader and writer
omgitsads5ab610c Merge branch 'omgitsads/go-sdk' of https://github.com/github/github-m…
omgitsadscd77c13 Dont bubble up an error for getClient
omgitsads6c07546 Migrate gists toolset to modelcontextprotocol/go-sdk (#1431)
Copilota405519 Migrate code-scanning toolset to modelcontextprotocol/go-sdk (#1430)
Copiloteab4876 Merge branch 'omgitsads/go-sdk' of https://github.com/github/github-m…
omgitsadsc06ace3 Migrate security_advisories toolset to modelcontextprotocol/go-sdk (#…
Copilot66e6ad5 Migrate secret_scanning toolset to modelcontextprotocol/go-sdk (#1436)
Copilotef60ef7 Merge branch 'omgitsads/go-sdk' of https://github.com/github/github-m…
omgitsadseaf411c Migrate git toolset to modelcontextprotocol/go-sdk (#1432)
Copilot9bf905b Migrate labels toolset to modelcontextprotocol/go-sdk (#1433)
Copilot726c683 Migrate issues toolset to modelcontextprotocol/go-sdk (#1440)
Copilot1b769a5 Migrate dependabot toolset to modelcontextprotocol/go-sdk (#1429)
Copilot0a19bf4 Migrate Repository Resources to the Go SDK (#1457)
omgitsads948fe76 Fix handling of multi path resources (#1458)
omgitsads42b5533 Migrate dynamic toolset to modelcontextprotocol/go-sdk (#1450)
Copilot4042338 Migrate discussions toolset to modelcontextprotocol/go-sdk (#1448)
Copilotee72841 Migrate notifications toolset to modelcontextprotocol/go-sdk (#1449)
Copilot17aaf6b Migrate `pull_requests` toolset to Go SDK (#1466)
LuluBeatson7fbf80d Migrate actions toolset to modelcontextprotocol/go-sdk (#1446)
Copilot9e40d53 Migrate 4 Search Tools to Go SDK (#1468)
LuluBeatson77ac1a7 Migrate projects toolset to modelcontextprotocol/go-sdk (#1475)
Copilotba40198 Enable empty experiments toolset in Go SDK (#1477)
omgitsads001e0a1 Migrate repos toolset to modelcontextprotocol/go-sdk (#1445)
Copilot77771a2 Update documentation
omgitsads782cadc re-enable get_label
omgitsads71223b2 Update copilot instructions to reference modelcontextprotocol/go-sdk
Copilote76ae99 Fix linter issues
omgitsads7c1340b Merge branch 'omgitsads/go-sdk' of https://github.com/github/github-m…
omgitsads5bc4bb6 Merge branch 'main' into omgitsads/go-sdk
omgitsadsa05df2e Revert change in merge for agent
omgitsads61d606b Migrate e2e tests to modelcontextprotocol/go-sdk
SamMorrowDrumse3aeae3 Fix e2e test bugs in TestDirectoryDeletion and TestPullRequestReviewC…
SamMorrowDrums5b7b3cd Add side parameter to FILE-level review comment in e2e test
SamMorrowDrumsbf8e9b4 Add TODO comments for e2e test improvements
SamMorrowDrumsa18fd3a Remove unused mark3labs/mcp-go dependency
SamMorrowDrums609e6f6 remove go:build ignore in search_utils_test.go
LuluBeatson7e19170 Allow overriding the options passed to NewServer
omgitsads8f5488e Merge branch 'main' into omgitsads/go-sdk
omgitsadsFile filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
There are no files selected for viewing
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
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
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
Oops, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
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.