Commit Graph

20 Commits

Author SHA1 Message Date
Sam Morrow 82c493056e fix: include empty properties in get_me schema for OpenAI compatibility
Build and Test Go Project / build (ubuntu-latest) (push) Has been cancelled
Build and Test Go Project / build (windows-latest) (push) Has been cancelled
CodeQL / Analyze (go) (push) Has been cancelled
CodeQL / Analyze (actions) (push) Has been cancelled
Docker / build (push) Has been cancelled
Build and Test Go Project / build (macos-latest) (push) Has been cancelled
GoReleaser Release / release (push) Has been cancelled
License Check / license-check (push) Has been cancelled
Publish to MCP Registry / publish (push) Has been cancelled
OpenAI strict mode requires the `properties` field to be present in
object schemas, even when empty. The jsonschema library uses omitempty
which causes empty maps to be omitted during JSON serialization.

This change uses json.RawMessage to bypass the library's serialization
and explicitly include `"properties": {}` in the output.

Fixes #1548
2025-12-08 11:15:21 +01:00
Adam Holt f197a9fd2a Add Tool Handler shim to RegisterFunc (#1536)
* Add tool handler shim to use server.AddTool instead of mcp.AddTool

---

Co-authored-by: Ksenia Bobrova <almaleksia@github.com>
Co-authored-by: Sam Morrow <SamMorrowDrums@github.com>
2025-12-04 15:56:24 +01:00
Adam Holt 60aef5d2e3 Convert to modelcontextprotocol/go-sdk (#1428)
Move from `mark3labs/mcp-go` to `modelcontextprotocol/go-sdk`.

This is mostly focused on updating tool schema and tool handler signatures, along with any associated internal changes related to those changes.

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: omgitsads <4619+omgitsads@users.noreply.github.com>
Co-authored-by: LuluBeatson <lulubeatson@github.com>
Co-authored-by: Lulu <59149422+LuluBeatson@users.noreply.github.com>
Co-authored-by: SamMorrowDrums <4811358+SamMorrowDrums@users.noreply.github.com>
Co-authored-by: Sam Morrow <info@sam-morrow.com>
2025-12-01 14:30:59 +01:00
Matt Holloway 1832210139 Add get_teams and get_team_members tools (#834)
* add team tool with tests

* add to tools

* add toolsnaps and docs

* Update pkg/github/context_tools.go

Co-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.go

Co-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>
2025-08-13 13:41:04 +01:00
Tony Truong 0cf70ebdb5 Remove redundant param for get_me and update contribution guide (#649)
* remove reason param for get_me

* updating toolsnap

* update contributing

* updating tool get_me

* add small changes

* update snapshots
2025-07-07 18:07:48 +02:00
LuluBeatson e43fca195b embed optional UserDetails in MinimalUser 2025-07-04 14:58:38 +02:00
LuluBeatson 1d057c975d refactor: user get methods to avoid nil checks 2025-07-04 14:58:38 +02:00
LuluBeatson 37d1ed6fd8 return MinimalUser 2025-07-04 14:58:38 +02:00
LuluBeatson 6c0453a9c1 omit site_admin from get_me output 2025-07-04 14:58:38 +02:00
Sam Morrow f90ff16f2e move to new approach and update testing 2025-06-25 08:12:38 +02:00
Adam Holt 36e6add0dd Return concrete error types for API errors 2025-06-25 08:12:38 +02:00
Lulu cbcf29f6d1 Export ToBoolPtr and RequiredParam (#495)
* ToBoolPtr, RequiredParam

* lint: type assertion in RequiredParam

* cap docstring
2025-06-09 15:11:38 +02:00
William Martin 023f59d5cc Use typed tool handler for get_me tool 2025-05-28 15:56:54 +02:00
Sam Morrow 50043c32de enact second piece of copilot feedback 2025-05-19 17:36:24 +02:00
Sam Morrow b7f4890a65 Update context_tools.go
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
2025-05-19 17:36:24 +02:00
Sam Morrow a3bf5587d8 alter the description of get-me to reduce redundant repeat calls 2025-05-19 17:36:24 +02:00
Toby Padilla 22d080beec fix: update get_me reason arg description 2025-05-16 00:51:56 +02:00
dependabot[bot] 705b61b814 build(deps): bump github.com/mark3labs/mcp-go from 0.25.0 to 0.27.0 (#397)
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: William Martin <williammartin@github.com>
2025-05-13 13:26:36 +02:00
Sam Morrow a58937c0b3 feat: provide tool annotations 2025-04-23 13:14:01 +02:00
Sam Morrow ff3036d596 feat: partition tools by product/feature 2025-04-15 01:11:53 +02:00