fix: rename kubernetes-default provider ID to kubernetes

e2e/deploy_test.go

@@ -38,7 +38,7 @@ var agentDeployTargets = []deployTarget{
 	},
 	{
 		name:     "kubernetes",
-		deplArgs: []string{"--provider-id", "kubernetes-default", "--namespace", "default"},
+		deplArgs: []string{"--provider-id", "kubernetes", "--namespace", "default"},
 		cleanup: func(t *testing.T, agentName string) {
 			ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second)
 			defer cancel()
@@ -65,7 +65,7 @@ var mcpDeployTargets = []deployTarget{
 	},
 	{
 		name:     "kubernetes",
-		deplArgs: []string{"--provider-id", "kubernetes-default", "--namespace", "default"},
+		deplArgs: []string{"--provider-id", "kubernetes", "--namespace", "default"},
 		cleanup: func(t *testing.T, mcpName string) {
 			ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second)
 			defer cancel()

internal/cli/agent/deploy.go

@@ -17,7 +17,7 @@ var DeployCmd = &cobra.Command{
 Example:
   arctl agent deploy my-agent --version latest
   arctl agent deploy my-agent --version 1.2.3
-  arctl agent deploy my-agent --version latest --provider-id kubernetes-default`,
+  arctl agent deploy my-agent --version latest --provider-id kubernetes`,
 	Args: cobra.ExactArgs(1),
 	RunE: runDeploy,
 }

internal/registry/api/handlers/v0/deployment_adapters.go

@@ -88,7 +88,7 @@ func (a *kubernetesDeploymentAdapter) Deploy(ctx context.Context, req *models.De
 	}
 	providerID := req.ProviderID
 	if providerID == "" {
-		providerID = "kubernetes-default"
+		providerID = "kubernetes"
 	}
 	env := req.Env
 	if env == nil {

internal/registry/api/handlers/v0/providers_test.go

@@ -161,7 +161,7 @@ func TestListProviders_WithData(t *testing.T) {
 	k8sAdapter := &fakeProviderAdapter{
 		platform: "kubernetes",
 		providers: map[string]*models.Provider{
-			"kubernetes-default": {ID: "kubernetes-default", Name: "Kubernetes Default", Platform: "kubernetes"},
+			"kubernetes": {ID: "kubernetes", Name: "Kubernetes Default", Platform: "kubernetes"},
 		},
 	}
 	v0.RegisterProvidersEndpoints(api, "/v0", fake, v0.PlatformExtensions{
@@ -178,7 +178,7 @@ func TestListProviders_WithData(t *testing.T) {
 	assert.Equal(t, http.StatusOK, w.Code)
 	assert.Contains(t, w.Body.String(), `"id":"local"`)
 	assert.Contains(t, w.Body.String(), `"platform":"local"`)
-	assert.Contains(t, w.Body.String(), `"id":"kubernetes-default"`)
+	assert.Contains(t, w.Body.String(), `"id":"kubernetes"`)
 	assert.Contains(t, w.Body.String(), `"platform":"kubernetes"`)
 }
 

internal/registry/database/migrations/010_rename_kubernetes_default_provider.sql

@@ -0,0 +1,14 @@
+-- Rename the kubernetes-default provider ID to kubernetes for consistency.
+
+-- Insert the new provider if it doesn't exist yet.
+INSERT INTO providers (id, name, platform, config)
+VALUES ('kubernetes', 'Kubernetes', 'kubernetes', '{}'::jsonb)
+ON CONFLICT (id) DO NOTHING;
+
+-- Migrate any deployments referencing the old ID.
+UPDATE deployments
+SET provider_id = 'kubernetes'
+WHERE provider_id = 'kubernetes-default';
+
+-- Remove the old provider.
+DELETE FROM providers WHERE id = 'kubernetes-default';

internal/registry/service/registry_service.go

@@ -31,7 +31,7 @@ const (
 	maxServerVersionsPerServer = 10000
 
 	localProviderID      = "local"
-	kubernetesProviderID = "kubernetes-default"
+	kubernetesProviderID = "kubernetes"
 	platformLocal        = "local"
 	platformKubernetes   = "kubernetes"
 	resourceTypeMCP      = "mcp"