-
Notifications
You must be signed in to change notification settings - Fork 0
feat(card): create server card #71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
peppescg
commented
Nov 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements a server card component for displaying MCP (Model Context Protocol) servers in the catalog, along with supporting UI components (Card and Badge). The changes replace placeholder fixture data with realistic server examples and add basic test coverage for the new component.
- Adds
ServerCardcomponent with server information display and URL copy functionality - Introduces reusable
CardandBadgeUI components following shadcn/ui patterns - Updates mock fixture data to use realistic MCP server examples
Reviewed Changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/components/server-card.tsx | New component that displays MCP server information with copy URL functionality |
| src/components/server-card.test.tsx | Basic test coverage for ServerCard rendering |
| src/components/ui/card.tsx | New Card UI component with sub-components (Header, Content, Footer, etc.) |
| src/components/ui/badge.tsx | New Badge UI component with variant support |
| src/mocks/fixtures/registry_v0_1_servers/get.ts | Replaces placeholder data with realistic MCP server examples |
| package.json | Adds @radix-ui/react-slot dependency for Badge component |
| pnpm-lock.yaml | Updates lockfile with new dependency |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported