Skip to content

fix: make server instructions intent-based instead of naming tools - #372

Merged
gregnr merged 1 commit into
mainfrom
fix/intent-based-server-instructions
Aug 21, 2026
Merged

fix: make server instructions intent-based instead of naming tools#372
gregnr merged 1 commit into
mainfrom
fix/intent-based-server-instructions

Conversation

@claude

@claude claude Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Requested by Greg Richardson

The server instructions named get_logs (hidden from tools/list on hosted since #333) and get_publishable_api_key (should have been get_publishable_keys); they now describe capabilities instead of tool names, so they stay correct across platforms. Tool descriptions are out of scope.

@coveralls

coveralls commented Aug 21, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 32520995041

Coverage remained the same at 96.52%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 3036
Covered Lines: 2951
Line Coverage: 97.2%
Relevant Branches: 384
Covered Branches: 350
Branch Coverage: 91.15%
Branches in Coverage %: Yes
Coverage Strength: 50.92 hits per line

💛 - Coveralls

The server-level MCP `instructions` string named literal tool names, two
of which are wrong on hosted Supabase:

- `get_logs` is hidden from `tools/list` on platforms that implement
  `queryLogs` (#333), so hosted clients were told to use a tool absent
  from their own tool list.
- `get_publishable_api_key` has never existed; the registered tool is
  `get_publishable_keys`.

Describe the capability instead of the tool. The guidance now stays
correct across hosted, self-hosted and CLI platforms, and survives
future tool renames.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CGspmz4goqhVVtfr5iPKZG
@claude
claude Bot force-pushed the fix/intent-based-server-instructions branch from 2f60565 to 4f9369e Compare August 21, 2026 19:55
@claude
claude Bot marked this pull request as ready for review August 21, 2026 20:19
@claude
claude Bot requested a review from a team as a code owner August 21, 2026 20:19
@gregnr
gregnr merged commit fc54ea2 into main Aug 21, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants