Skip to content

Migrate MCP Server to .NET and add CI workflows#34

Merged
MWG-Logan merged 3 commits intodevfrom
ci-cd
Nov 14, 2025
Merged

Migrate MCP Server to .NET and add CI workflows#34
MWG-Logan merged 3 commits intodevfrom
ci-cd

Conversation

@MWG-Logan
Copy link
Owner

Updated copilot-instructions.md to reflect the transition to .NET 10, C# 14, and Azure Functions isolated worker model. Added detailed sections on the technology stack, storage schema, Azure Functions endpoints, validation workflow, and troubleshooting.

Introduced dev-publish-function.yml and prod-publish-function.yml GitHub Actions workflows for automated deployment to Azure Function Apps in development and production environments. Configured workflows to build the .NET project and deploy using OIDC authentication.

Replaced Node.js/npm setup instructions with .NET-specific commands and documented the new project structure, including dependency injection setup and critical file locations.

Updated `copilot-instructions.md` to reflect the transition to .NET 10, C# 14, and Azure Functions isolated worker model. Added detailed sections on the technology stack, storage schema, Azure Functions endpoints, validation workflow, and troubleshooting.

Introduced `dev-publish-function.yml` and `prod-publish-function.yml` GitHub Actions workflows for automated deployment to Azure Function Apps in development and production environments. Configured workflows to build the .NET project and deploy using OIDC authentication.

Replaced Node.js/npm setup instructions with .NET-specific commands and documented the new project structure, including dependency injection setup and critical file locations.
Copilot AI review requested due to automatic review settings November 14, 2025 19:49
Copy link
Contributor

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 migrates the Central Memory MCP Server from Node.js/TypeScript to .NET 10/C# 14 with Azure Functions isolated worker model. The changes include comprehensive documentation updates and CI/CD workflow configurations.

  • Complete technology stack transition from npm/TypeScript to dotnet/C# with updated build commands, validation workflows, and project structure
  • Introduction of automated CI/CD pipelines for dev and prod environments using GitHub Actions with OIDC authentication
  • Documentation of new storage schema, MCP tool endpoints, dependency injection, and troubleshooting procedures

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File Description
.github/workflows/prod-publish-function.yml New GitHub Actions workflow for deploying to production Azure Function App using .NET 10 and OIDC authentication
.github/workflows/dev-publish-function.yml New GitHub Actions workflow for deploying to development Azure Function App using .NET 10 and OIDC authentication
.github/copilot-instructions.md Complete rewrite documenting .NET 10 migration, updated commands, project structure, storage schema, and validation procedures

Added `AZURE_CLIENT_ID`, `AZURE_TENANT_ID`, and `AZURE_SUBSCRIPTION_ID` secrets to configuration comments in `dev-publish-function.yml` and `prod-publish-function.yml` for improved authentication setup.

Updated `AZURE_FUNCTIONAPP_PACKAGE_PATH` to `CentralMemoryMcp.Functions` to specify the correct project path for the function app.

Improved readability of environment variable definitions by reformatting `AZURE_FUNCTIONAPP_NAME` and `DOTNET_VERSION`.
Updated `README.md` and `ARCHITECTURE.md` to use consistent
bullet point styles (`-` instead of `–`) for better readability
and alignment with Markdown conventions. These changes enhance
the visual presentation without altering the content or
functionality described in the documents.
@MWG-Logan MWG-Logan merged commit 496874d into dev Nov 14, 2025
1 check failed
@MWG-Logan MWG-Logan deleted the ci-cd branch November 14, 2025 19:59
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.

2 participants