Revert "docs: clarify release deployment process and enable manual deploy trigger"

This reverts commit c40374c.

Author: tadasant

.github/workflows/deploy.yml

@@ -4,7 +4,6 @@ on:
   push:
     branches:
       - main
-  workflow_dispatch:
 
 permissions:
   contents: read

cmd/publisher/commands/init.go

@@ -308,7 +308,7 @@ func createServerJSON(
 			URL:    repoURL,
 			Source: repoSource,
 		},
-		Version:  version,
+		Version: version,
 		Packages: []model.Package{pkg},
 	}
 }

cmd/publisher/commands/logout.go

@@ -13,7 +13,7 @@ func LogoutCommand() error {
 	}
 
 	tokenPath := filepath.Join(homeDir, TokenFileName)
-
+	
 	// Check if token file exists
 	if _, err := os.Stat(tokenPath); os.IsNotExist(err) {
 		_, _ = fmt.Fprintln(os.Stdout, "Not logged in")
@@ -30,7 +30,7 @@ func LogoutCommand() error {
 		".mcpregistry_github_token",
 		".mcpregistry_registry_token",
 	}
-
+	
 	for _, file := range legacyFiles {
 		path := filepath.Join(homeDir, file)
 		if _, err := os.Stat(path); err == nil {
@@ -40,4 +40,4 @@ func LogoutCommand() error {
 
 	_, _ = fmt.Fprintln(os.Stdout, "✓ Successfully logged out")
 	return nil
-}
+}

cmd/publisher/main.go

@@ -67,4 +67,4 @@ func printUsage() {
 	_, _ = fmt.Fprintln(os.Stdout, "  publish       Publish server.json to the registry")
 	_, _ = fmt.Fprintln(os.Stdout)
 	_, _ = fmt.Fprintln(os.Stdout, "Use 'mcp-publisher <command> --help' for more information about a command.")
-}
+}

deploy/pkg/k8s/ingress.go

@@ -51,7 +51,7 @@ func SetupIngressController(ctx *pulumi.Context, cluster *providers.ProviderInfo
 		Values: pulumi.Map{
 			"controller": pulumi.Map{
 				"service": pulumi.Map{
-					"type":        serviceType,
+					"type": serviceType,
 					"annotations": pulumi.Map{},
 				},
 				"config": pulumi.Map{

deploy/pkg/providers/gcp/provider.go

@@ -22,19 +22,19 @@ type Provider struct{}
 // createGCPProvider creates a GCP provider with explicit credentials if configured
 func createGCPProvider(ctx *pulumi.Context, name string) (*gcp.Provider, error) {
 	gcpConf := config.New(ctx, "gcp")
-
+	
 	// Get project ID from config
 	projectID := gcpConf.Get("project")
 	if projectID == "" {
 		return nil, fmt.Errorf("GCP project ID not configured. Set gcp:project")
 	}
-
+	
 	// Get region from config or use default
 	region := gcpConf.Get("region")
 	if region == "" {
 		region = "us-central1"
 	}
-
+	
 	// Get credentials from config (base64 encoded service account JSON)
 	credentials := gcpConf.Get("credentials")
 	if credentials != "" {
@@ -45,7 +45,7 @@ func createGCPProvider(ctx *pulumi.Context, name string) (*gcp.Provider, error)
 		}
 		credentials = string(decodedCreds)
 	}
-
+	
 	// Create a GCP provider with explicit credentials if provided
 	if credentials != "" {
 		return gcp.NewProvider(ctx, name, &gcp.ProviderArgs{
@@ -54,7 +54,7 @@ func createGCPProvider(ctx *pulumi.Context, name string) (*gcp.Provider, error)
 			Credentials: pulumi.String(credentials),
 		})
 	}
-
+	
 	return nil, nil
 }
 

docs/guides/contributing/releasing.md

@@ -16,34 +16,10 @@ The release workflow will automatically:
 
 ## After Release
 
-The release workflow will automatically:
-- Build and publish Docker images:
+- Docker images will be available at:
   - `ghcr.io/modelcontextprotocol/registry:latest` - Latest stable release
   - `ghcr.io/modelcontextprotocol/registry:vX.Y.Z` - Specific release version
-- Binaries will be available on the GitHub release page
-
-**Important:** Creating a release does **not** automatically deploy to production. Production deployments are triggered by pushes to the `main` branch.
-
-## Deploying the Release to Production
-
-After creating a release, you need to ensure production pulls the new `:latest` Docker image. You have two options:
-
-1. **Recommended:** Merge a commit to `main` (e.g., a changelog update or version bump). This will trigger the deployment workflow which will:
-   - Build and push a new `:main` Docker image
-   - Deploy to staging
-   - Deploy to production (which uses `:latest` and will pull the new image)
-
-2. **Manual restart (requires kubectl access):**
-   ```bash
-   kubectl rollout restart deployment/mcp-registry -n default
-   ```
-
-## Verifying the Deployment
-
-After deployment completes, verify the changes are live:
-- Check the deployment workflow succeeded: https://github.com/modelcontextprotocol/registry/actions/workflows/deploy.yml
-- Test the API endpoint to confirm schema changes are applied
-- Check migrations ran successfully in the pod logs
+- Binaries can be downloaded from the GitHub release page
 
 ## Docker Image Tags
 

internal/api/handlers/v0/auth/http_test.go

@@ -387,6 +387,7 @@ func TestDefaultHTTPKeyFetcher(t *testing.T) {
 	}
 }
 
+
 func TestHTTPAuthHandler_Permissions(t *testing.T) {
 	cfg := &config.Config{
 		JWTPrivateKey: "0123456789abcdef0123456789abcdef0123456789abcdef0123456789abcdef",
@@ -737,3 +738,4 @@ func TestHTTPvsDNS_PermissionDifferences(t *testing.T) {
 		})
 	}
 }
+

internal/api/handlers/v0/edit.go

@@ -97,8 +97,8 @@ func RegisterEditEndpoints(api huma.API, registry service.RegistryService, cfg *
 
 			// Prevent undeleting servers - once deleted, they stay deleted
 			if currentServer.Meta.Official != nil &&
-				currentServer.Meta.Official.Status == model.StatusDeleted &&
-				newStatus != model.StatusDeleted {
+			   currentServer.Meta.Official.Status == model.StatusDeleted &&
+			   newStatus != model.StatusDeleted {
 				return nil, huma.Error400BadRequest("Cannot change status of deleted server. Deleted servers cannot be undeleted.")
 			}
 

internal/api/handlers/v0/edit_test.go

@@ -639,4 +639,4 @@ func TestEditServerEndpointEdgeCases(t *testing.T) {
 // Helper function
 func stringPtr(s string) *string {
 	return &s
-}
+}