fix(ui): prefer human-readable server title in registry cards - #1573
Open
RuyaaCapital-admin wants to merge 1 commit into
Open
fix(ui): prefer human-readable server title in registry cards#1573RuyaaCapital-admin wants to merge 1 commit into
RuyaaCapital-admin wants to merge 1 commit into
Conversation
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
server.titleas the primary heading in Registry cardsserver.namevisible as a smaller secondary identity when the values differserver.nameheading whentitleis absent or identicalWhy
The Registry schema defines
titleas a human-readable display name, but collapsed cards currently headline only the reverse-DNSname. That makes browsing harder for users who do not recognize programmatic MCP identifiers.Generic example:
The technical identity remains visible for namespace verification. All user-provided values continue to pass through the existing
escapeHtml(...)helper, and long values use the existing Tailwind wrapping utilities.Scope
This is presentation-only and backward-compatible. It does not change schemas, API responses, search behavior, publishing, authentication, or protocol behavior.
The API
searchparameter currently matches onlyserver_name; title and description search are not included here. Icon rendering is also intentionally out of scope and is already tracked by #784.Testing
git diff --checkLocal
make checkwas unavailable because this Windows environment does not have Go, Make, Docker, or a WSL distribution installed. The upstream PR CI is configured to run lint, schema validation, build, vulnerability scanning, and all tests.