Skip to content

feat: hide tools from tools/list - #334

Merged
gregnr merged 3 commits into
mainfrom
feat/hide-tools
Jul 17, 2026
Merged

feat: hide tools from tools/list#334
gregnr merged 3 commits into
mainfrom
feat/hide-tools

Conversation

@gregnr

@gregnr gregnr commented Jul 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Adds a hidden: true flag to tool definitions that will hide the tool from tools/list calls but allow it to still be callable via tools/call. Needed for the expand+contract breaking change pattern.

@gregnr
gregnr requested a review from a team as a code owner July 17, 2026 16:17
@coveralls

coveralls commented Jul 17, 2026 •
edited
Loading

Copy link
Copy Markdown

Coverage Report for CI Build 29612925862

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage increased (+0.007%) to 96.332%

Details

  • Coverage increased (+0.007%) from the base build.
  • Patch coverage: 6 of 6 lines across 2 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.

Coverage Stats

Coverage Status
Relevant Lines: 2884
Covered Lines: 2799
Line Coverage: 97.05%
Relevant Branches: 360
Covered Branches: 326
Branch Coverage: 90.56%
Branches in Coverage %: Yes
Coverage Strength: 44.16 hits per line
💛 - Coveralls

@mattrossman

mattrossman commented Jul 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

We might want to update createToolSchemas() / supabaseMcpToolSchemas to respect this hidden field and/or have a test to make the expectation for how it behaves clear.

Otherwise, this test might fail once we mark a tool as hidden since there will be tools in the schema registry that aren't in tools/list.

@gregnr

gregnr commented Jul 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

@mattrossman I updated the test to actually allow hidden tools to appear in createToolSchemas() (rather than filtering). The reason is that hidden: true is added during the expand phase, which is non-breaking from a semver POV (it just stops clients from discovering the tool at wire-level via tools/list). If we hide the tool from createToolSchemas(), that is breaking from a consumer's POV when our semver hasn't reflected that yet. Instead we make the breaking changes in the contract phase, which fully removes the tool.

I'll merge as is for now but happy to discuss further/change if you disagree with this.

@gregnr
gregnr merged commit d80471a into main Jul 17, 2026
5 checks passed
@supabase-releaser supabase-releaser Bot mentioned this pull request Jul 17, 2026
Merged
barryroodt added a commit that referenced this pull request Jul 21, 2026
@barryroodt
fix: port util.test.ts to v2 SDK client imports
a6733a1 
The hide-tools test added in #334 imports from the v1 monolithic SDK
(@modelcontextprotocol/sdk), which this branch removes. Port it to the
v2 client package and its typed callTool result, matching server.test.ts.
@barryroodt barryroodt mentioned this pull request Jul 21, 2026
Open
@raulb raulb mentioned this pull request Jul 30, 2026
Closed
Rodriguespn pushed a commit that referenced this pull request Aug 10, 2026
@supabase-releaser
chore: release main (#336)
302d2ad 
🤖 I have created a release *beep* *boop*
---


<details><summary>mcp-utils: 0.6.0</summary>

##
[0.6.0](mcp-utils-v0.5.1...mcp-utils-v0.6.0)
(2026-08-10)


### Features

* hide tools from tools/list
([#334](#334))
([d80471a](d80471a))
</details>

<details><summary>mcp-server-supabase: 0.10.0</summary>

##
[0.10.0](mcp-server-supabase-v0.9.0...mcp-server-supabase-v0.10.0)
(2026-08-10)


### ⚠ BREAKING CHANGES

* add query_logs tool for custom log queries
([#333](#333))

### Features

* add --content-api-url flag and SUPABASE_CONTENT_API_URL env var
([#343](#343))
([6fcaaa3](6fcaaa3))
* add query_logs tool for custom log queries
([#333](#333))
([798806b](798806b))
* hide tools from tools/list
([#334](#334))
([d80471a](d80471a))


### Bug Fixes

* hide read-only mode
([#349](#349))
([5cda067](5cda067))
* **pg-meta:** pair composite FK columns positionally to avoid cartesi…
([#317](#317))
([10af00b](10af00b))
* select query_logs dialect via logsDialect
([#357](#357))
([80ff453](80ff453))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: supabase-releaser[bot] <223506987+supabase-releaser[bot]@users.noreply.github.com>
barryroodt added a commit that referenced this pull request Aug 11, 2026
@barryroodt
fix: port util.test.ts to v2 SDK client imports
658074b 
The hide-tools test added in #334 imports from the v1 monolithic SDK
(@modelcontextprotocol/sdk), which this branch removes. Port it to the
v2 client package and its typed callTool result, matching server.test.ts.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mattrossman mattrossman mattrossman 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

@gregnr @coveralls @mattrossman