Skip to content

Conversation

@peppescg
Copy link
Collaborator

Screenshot 2025-11-20 at 20 55 04

Copilot AI review requested due to automatic review settings November 20, 2025 20:02
@peppescg peppescg changed the title Issues/36 feat(card): create server card Nov 20, 2025
Copy link

Copilot AI left a 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 ServerCard component with server information display and URL copy functionality
  • Introduces reusable Card and Badge UI 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

@peppescg peppescg merged commit ba0e405 into main Nov 21, 2025
15 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.

3 participants