stacklok / toolhive-studio / 24335811921 / 1

70%

main: 70%

push

github

feat: show/hide custom MCP servers by permission (#1974)

* test: add failing tests for custom MCP permission feature (TDD)

Tests define expected behavior for show/hide custom MCP servers by
permission. All tests fail intentionally — coder agent will implement
the feature to make them pass.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* test: rename permission key to non-registry-servers

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* feat: hide custom MCP server options based on non-registry-servers permission

Add NON_REGISTRY_SERVERS permission key and use it to conditionally
show/hide the "Add MCP server" dropdown and adjust empty state text
on the group route page.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: rename permission key to custom-mcp-servers and remove redundant tests

Rename NON_REGISTRY_SERVERS → CUSTOM_MCP_SERVERS across all usages.
Remove custom-mcp-servers-permission and grid-cards-mcp-server tests
that were not providing value.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: rename permission key to CUSTOM_MCP_SERVERS with non_registry_servers value

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* feat: add browse registry button in empty state when custom MCP is disabled

Show a direct "Browse registry" link instead of the dropdown when the
custom MCP servers permission is off. Also remove the dropdown menu
test that was testing permission gating inside the component (gating
happens at the route level).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* test: verify dropdown is hidden when custom MCP servers permission is disabled

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

3490 of 5800 branches covered (60.17%)

5585 of 8515 relevant lines covered (65.59%)

121.26 hits per line