From 35bd41a131356f4b2beb72c4ec908bf07a4e2503 Mon Sep 17 00:00:00 2001 From: Andrey Velichkevich Date: Wed, 13 Aug 2025 11:25:44 +0100 Subject: [PATCH] fix(ci): Fix dev-local target in the Makefile (#259) Hi Folks, thanks for the great work on MCP Registry! While running it locally, I noticed a bug with the `make dev-local` command. ``` $ make dev-local go run ./cmd/registry/main.go # command-line-arguments cmd/registry/main.go:29:36: undefined: Version cmd/registry/main.go:30:34: undefined: GitCommit ``` ## Motivation and Context Command for running MCP Registry locally is broken. ## How Has This Been Tested? ## Breaking Changes ## Types of changes - [x] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] Documentation update ## Checklist - [x] I have read the [MCP Documentation](https://modelcontextprotocol.io) - [x] My code follows the repository's style guidelines - [x] New and existing tests pass locally - [x] I have added appropriate error handling - [x] I have added or updated documentation as needed ## Additional context cc @domdomegg @sridharavinash @tadasant Signed-off-by: Andrey Velichkevich Co-authored-by: adam jones --- Makefile | 2 +- scripts/test_endpoints.sh | 2 +- scripts/test_publish.sh | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 0eb15b8..bc786cf 100644 --- a/Makefile +++ b/Makefile @@ -58,7 +58,7 @@ dev-compose: ## Start development environment with Docker Compose (builds image docker compose up --build dev-local: ## Run registry locally (requires MongoDB) - go run cmd/registry/main.go + go run ./cmd/registry # Cleanup clean: ## Clean build artifacts and coverage files diff --git a/scripts/test_endpoints.sh b/scripts/test_endpoints.sh index 2efd2f3..364ccdd 100755 --- a/scripts/test_endpoints.sh +++ b/scripts/test_endpoints.sh @@ -8,7 +8,7 @@ echo "==================================================" echo "This script expects the MCP Registry server to be running locally." echo "Please ensure the server is started using one of the following methods:" echo " • Docker Compose: docker compose up" -echo " • Direct execution: go run cmd/registry/main.go" +echo " • Direct execution: go run ./cmd/registry" echo " • Built binary: ./build/registry" echo "==================================================" echo "" diff --git a/scripts/test_publish.sh b/scripts/test_publish.sh index 4a37367..feb2440 100755 --- a/scripts/test_publish.sh +++ b/scripts/test_publish.sh @@ -8,7 +8,7 @@ echo "==================================================" echo "This script expects the MCP Registry server to be running locally." echo "Please ensure the server is started using one of the following methods:" echo " • Docker Compose: docker compose up" -echo " • Direct execution: go run cmd/registry/main.go" +echo " • Direct execution: go run ./cmd/registry" echo " • Built binary: ./build/registry" echo "" echo "REQUIRED: Set the BEARER_TOKEN environment variable with a valid GitHub token"