Bring MCP server into the repository #238
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR integrates the MCP (Model Context Protocol) server functionality into the awesome-copilot repository, establishing a comprehensive infrastructure for serving Copilot customizations. The implementation includes a complete .NET-based MCP server with search and content loading capabilities, containerization support, and deployment configurations for both local development and Azure hosting.
Key changes include:
- Implementation of a .NET MCP server with search and load tools for awesome-copilot content
- Full containerization and Azure deployment infrastructure using Bicep
- Metadata extraction and processing system for chatmodes, instructions, and prompts
- Development tooling integration with Aspire for local development
Reviewed Changes
Copilot reviewed 39 out of 39 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| mcp-server/ | Complete MCP server implementation with .NET project structure, models, services, and tools |
| mcp-server/infra/ | Azure infrastructure-as-code using Bicep for Container Apps deployment |
| .github/workflows/docker-image.yml | CI/CD pipeline for building and publishing Docker images |
| .aspire/settings.json | Aspire configuration for local development |
| .vscode/settings.json | VS Code configuration updates for Python environment |
| @@ -0,0 +1,343 @@ | |||
| #!/usr/bin/env node | |||
There was a problem hiding this comment.
Missing README update: The new file should be added to the README.md.
Copilot generated this review using guidance from repository custom instructions.
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
|
||
| // Read the JSON schema to understand the structure | ||
| const schemaPath = path.join(__dirname, "frontmatter-schema.json"); | ||
| const schema = JSON.parse(fs.readFileSync(schemaPath, "utf8")); |
Check notice
Code scanning / CodeQL
Unused variable, import, function or class Note
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 2 days ago
To fix the unused variable issue, we should remove the declaration and initialization of the schema variable on line 25. This also eliminates the unnecessary parsing of the file, which improves script startup time and readability. Specifically, remove the following code line:
const schema = JSON.parse(fs.readFileSync(schemaPath, "utf8"));This change does not require any changes elsewhere, as the variable is not referenced anywhere else.
| @@ -22,7 +22,6 @@ | ||
|
|
||
| // Read the JSON schema to understand the structure | ||
| const schemaPath = path.join(__dirname, "frontmatter-schema.json"); | ||
| const schema = JSON.parse(fs.readFileSync(schemaPath, "utf8")); | ||
|
|
||
| // Define the directories to process | ||
| const directories = { |
CI/CD and Containerization
.github/workflows/docker-image.yml).Dockerfilefor building the MCP server using .NET 10 Native AOT, including multi-stage build and runtime setup (mcp-server/Dockerfile)..dockerignoreand.gitignorefiles to exclude unnecessary files and directories from Docker builds and git tracking (mcp-server/.dockerignore,mcp-server/.gitignore). [1] [2]Azure Deployment and Infrastructure
mcp-server/infra/main.bicep,mcp-server/azure.yaml). [1] [2]mcp-server/infra/abbreviations.json).Project Structure and Developer Tools
mcp-server/AwesomeCopilot.slnx,mcp-server/NuGet.config). [1] [2].aspire/settings.json,.vscode/settings.json). [1] [2]Documentation and Schema Validation
mcp-server/README.md).mcp-server/frontmatter-schema.json).