refactor: comment on registry_types

Author: peppescg

lint

@@ -0,0 +1,12 @@
+# https://taskfile.dev
+
+version: '3'
+
+vars:
+  GREETING: Hello, World!
+
+tasks:
+  default:
+    cmds:
+      - echo "{{.GREETING}}"
+    silent: true

pkg/api/v1/workload_types.go

@@ -69,9 +69,9 @@ type updateRequest struct {
 	Group string `json:"group,omitempty"`
 
 	// Remote server specific fields
-	URL         string            `json:"url,omitempty"`
-	OAuthConfig remoteOAuthConfig `json:"oauth_config,omitempty"`
-	Headers     []*registry.Header   `json:"headers,omitempty"`
+	URL         string             `json:"url,omitempty"`
+	OAuthConfig remoteOAuthConfig  `json:"oauth_config,omitempty"`
+	Headers     []*registry.Header `json:"headers,omitempty"`
 }
 
 // toolOverride represents a tool override

pkg/auth/remote/config.go

@@ -56,20 +56,20 @@ func (r *Config) UnmarshalJSON(data []byte) error {
 	if _, isOld := raw["ClientID"]; isOld {
 		// Unmarshal using old PascalCase format
 		var oldFormat struct {
-			ClientID         string            `json:"ClientID,omitempty"`
-			ClientSecret     string            `json:"ClientSecret,omitempty"`
-			ClientSecretFile string            `json:"ClientSecretFile,omitempty"`
-			Scopes           []string          `json:"Scopes,omitempty"`
-			SkipBrowser      bool              `json:"SkipBrowser,omitempty"`
-			Timeout          time.Duration     `json:"Timeout,omitempty"`
-			CallbackPort     int               `json:"CallbackPort,omitempty"`
-			UsePKCE          bool              `json:"UsePKCE,omitempty"`
-			Issuer           string            `json:"Issuer,omitempty"`
-			AuthorizeURL     string            `json:"AuthorizeURL,omitempty"`
-			TokenURL         string            `json:"TokenURL,omitempty"`
-			Headers          []*registry.Header   `json:"Headers,omitempty"`
-			EnvVars          []*registry.EnvVar   `json:"EnvVars,omitempty"`
-			OAuthParams      map[string]string `json:"OAuthParams,omitempty"`
+			ClientID         string             `json:"ClientID,omitempty"`
+			ClientSecret     string             `json:"ClientSecret,omitempty"`
+			ClientSecretFile string             `json:"ClientSecretFile,omitempty"`
+			Scopes           []string           `json:"Scopes,omitempty"`
+			SkipBrowser      bool               `json:"SkipBrowser,omitempty"`
+			Timeout          time.Duration      `json:"Timeout,omitempty"`
+			CallbackPort     int                `json:"CallbackPort,omitempty"`
+			UsePKCE          bool               `json:"UsePKCE,omitempty"`
+			Issuer           string             `json:"Issuer,omitempty"`
+			AuthorizeURL     string             `json:"AuthorizeURL,omitempty"`
+			TokenURL         string             `json:"TokenURL,omitempty"`
+			Headers          []*registry.Header `json:"Headers,omitempty"`
+			EnvVars          []*registry.EnvVar `json:"EnvVars,omitempty"`
+			OAuthParams      map[string]string  `json:"OAuthParams,omitempty"`
 		}
 
 		if err := json.Unmarshal(data, &oldFormat); err != nil {

pkg/container/verifier/verifier.go

@@ -13,7 +13,7 @@ import (
 
 	"github.com/stacklok/toolhive/pkg/container/images"
 	"github.com/stacklok/toolhive/pkg/logger"
-	 "github.com/stacklok/toolhive/pkg/registry/registry"
+	"github.com/stacklok/toolhive/pkg/registry/registry"
 )
 
 const (

pkg/mcp/server/search_registry.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 
 	"github.com/mark3labs/mcp-go/mcp"
+
 	types "github.com/stacklok/toolhive/pkg/registry/registry"
 )
 

pkg/registry/converters/converters_fixture_test.go

@@ -7,9 +7,10 @@ import (
 	"testing"
 
 	upstream "github.com/modelcontextprotocol/registry/pkg/api/v0"
-	types "github.com/stacklok/toolhive/pkg/registry/registry"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
+
+	types "github.com/stacklok/toolhive/pkg/registry/registry"
 )
 
 // TestConverters_Fixtures validates converter functions using JSON fixture files

pkg/registry/converters/converters_test.go

@@ -6,9 +6,10 @@ import (
 
 	upstream "github.com/modelcontextprotocol/registry/pkg/api/v0"
 	"github.com/modelcontextprotocol/registry/pkg/model"
-	types "github.com/stacklok/toolhive/pkg/registry/registry"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
+
+	types "github.com/stacklok/toolhive/pkg/registry/registry"
 )
 
 // Test Helpers

pkg/registry/converters/envvar_extraction_test.go

@@ -4,9 +4,10 @@ import (
 	"testing"
 
 	"github.com/modelcontextprotocol/registry/pkg/model"
-	types "github.com/stacklok/toolhive/pkg/registry/registry"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
+
+	types "github.com/stacklok/toolhive/pkg/registry/registry"
 )
 
 // Test extracting environment variables from runtime arguments (-e flags)

pkg/registry/converters/integration_test.go

@@ -9,8 +9,9 @@ import (
 	"testing"
 
 	upstream "github.com/modelcontextprotocol/registry/pkg/api/v0"
-	types "github.com/stacklok/toolhive/pkg/registry/registry"
 	"github.com/stretchr/testify/require"
+
+	types "github.com/stacklok/toolhive/pkg/registry/registry"
 )
 
 // ToolHiveRegistry represents the structure of registry.json

pkg/registry/converters/registry_converters_test.go

@@ -6,9 +6,10 @@ import (
 
 	upstreamv0 "github.com/modelcontextprotocol/registry/pkg/api/v0"
 	"github.com/modelcontextprotocol/registry/pkg/model"
-	types "github.com/stacklok/toolhive/pkg/registry/registry"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
+
+	types "github.com/stacklok/toolhive/pkg/registry/registry"
 )
 
 func TestNewUpstreamRegistryFromToolhiveRegistry(t *testing.T) {