Merge branch 'main' into feature/config-volumes-remove

Author: javiermtorres

.github/workflows/golangci-lint.yaml

@@ -25,6 +25,6 @@ jobs:
           go-version-file: go.mod
 
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@4afd733a84b1f43292c63897423277bb7f4313a9 # 8.0.0
+        uses: golangci/golangci-lint-action@1e7e51e771db61008b38414a730f564565cf7c20 # v9.2.0
         with:
-          version: v2.4.0
+          version: v2.10.1

README.md

@@ -135,7 +135,7 @@ Plugin SDKs are built using the [mcpd plugin Protocol Buffers specification](htt
 ## 💻 Development
 
 If you are developing `mcpd`, you will need:
-- [**Go**](https://go.dev/doc/install) (v1.25.6+ recommended)
+- [**Go**](https://go.dev/doc/install) (v1.26.0+ recommended)
 
 Build local code:
 ```bash

cmd/config/args/remove.go

@@ -88,7 +88,10 @@ func (c *RemoveCmd) run(cmd *cobra.Command, args []string) error {
 
 	if _, err := fmt.Fprintf(
 		cmd.OutOrStdout(),
-		"✓ Arguments removed for server '%s' (operation: %s): %v\n", serverName, string(res), slices.Collect(maps.Keys(argMap)),
+		"✓ Arguments removed for server '%s' (operation: %s): %v\n",
+		serverName,
+		string(res),
+		slices.Collect(maps.Keys(argMap)),
 	); err != nil {
 		return err
 	}

cmd/config/args/set.go

@@ -99,7 +99,11 @@ func (c *SetCmd) run(cmd *cobra.Command, args []string) error {
 
 	if _, err := fmt.Fprintf(
 		cmd.OutOrStdout(),
-		"✓ Startup arguments %s for server '%s' (operation: %s): %v\n", operation, serverName, string(res), normalizedArgs,
+		"✓ Startup arguments %s for server '%s' (operation: %s): %v\n",
+		operation,
+		serverName,
+		string(res),
+		normalizedArgs,
 	); err != nil {
 		return err
 	}

cmd/config/env/remove.go

@@ -77,7 +77,10 @@ func (c *RemoveCmd) run(cmd *cobra.Command, args []string) error {
 
 	if _, err := fmt.Fprintf(
 		cmd.OutOrStdout(),
-		"✓ Environment variables removed for server '%s' (operation: %s): %v\n", serverName, string(res), slices.Collect(maps.Keys(envMap)),
+		"✓ Environment variables removed for server '%s' (operation: %s): %v\n",
+		serverName,
+		string(res),
+		slices.Collect(maps.Keys(envMap)),
 	); err != nil {
 		return err
 	}

cmd/config/env/set.go

@@ -87,7 +87,10 @@ func (c *SetCmd) run(cmd *cobra.Command, args []string) error {
 
 	if _, err := fmt.Fprintf(
 		cmd.OutOrStdout(),
-		"✓ Environment variables set for server '%s' (operation: %s): %v\n", serverName, string(res), slices.Collect(maps.Keys(envMap)),
+		"✓ Environment variables set for server '%s' (operation: %s): %v\n",
+		serverName,
+		string(res),
+		slices.Collect(maps.Keys(envMap)),
 	); err != nil {
 		return err
 	}

cmd/daemon.go

@@ -1060,43 +1060,43 @@ func (c *DaemonCmd) formatConfigInfo(addr string) string {
 
 	// API configuration (only show if non-default).
 	if addr != "" && addr != "0.0.0.0:8090" {
-		info.WriteString(fmt.Sprintf("  API address:\t%s\n", addr))
+		fmt.Fprintf(&info, "  API address:\t%s\n", addr)
 	}
 
 	// CORS configuration.
 	if c.config.cors.enable {
-		info.WriteString(fmt.Sprintf("  CORS enabled:\t%v (origins: %s)\n",
+		fmt.Fprintf(&info, "  CORS enabled:\t%v (origins: %s)\n",
 			c.config.cors.enable,
-			strings.Join(c.config.cors.origins, ", ")))
+			strings.Join(c.config.cors.origins, ", "))
 
 		if len(c.config.cors.methods) > 0 {
-			info.WriteString(fmt.Sprintf("  CORS methods:\t%s\n",
-				strings.Join(c.config.cors.methods, ", ")))
+			fmt.Fprintf(&info, "  CORS methods:\t%s\n",
+				strings.Join(c.config.cors.methods, ", "))
 		}
 
 		if c.config.cors.credentials {
 			info.WriteString("  CORS credentials:\ttrue\n")
 		}
 
 		if c.config.cors.maxAge != "" {
-			info.WriteString(fmt.Sprintf("  CORS max age:\t%s\n", c.config.cors.maxAge))
+			fmt.Fprintf(&info, "  CORS max age:\t%s\n", c.config.cors.maxAge)
 		}
 	}
 
 	// Timeout configuration.
 	if v := c.config.timeout.apiShutdown; v != "" && v != daemon.DefaultAPIShutdownTimeout().String() {
-		info.WriteString(fmt.Sprintf("  API shutdown timeout:\t%s\n", v))
+		fmt.Fprintf(&info, "  API shutdown timeout:\t%s\n", v)
 	}
 	if v := c.config.timeout.mcpInit; v != "" && v != daemon.DefaultClientInitTimeout().String() {
-		info.WriteString(fmt.Sprintf("  MCP init timeout:\t%s\n", v))
+		fmt.Fprintf(&info, "  MCP init timeout:\t%s\n", v)
 	}
 
 	if v := c.config.timeout.healthCheck; v != "" && v != daemon.DefaultHealthCheckTimeout().String() {
-		info.WriteString(fmt.Sprintf("  MCP health check timeout:\t%s\n", v))
+		fmt.Fprintf(&info, "  MCP health check timeout:\t%s\n", v)
 	}
 
 	if v := c.config.interval.healthCheck; v != "" && v != daemon.DefaultHealthCheckInterval().String() {
-		info.WriteString(fmt.Sprintf("  MCP health check interval:\t%s\n", v))
+		fmt.Fprintf(&info, "  MCP health check interval:\t%s\n", v)
 	}
 
 	return info.String()

docs/requirements.md

@@ -5,7 +5,7 @@ To use `mcpd`, ensure the following tools are installed:
 | Tool           | Purpose                                                     | URL                                                                                                                                    |
 |----------------|-------------------------------------------------------------|----------------------------------------------------------------------------------------------------------------------------------------|
 | `Docker`       | For running MCP servers as Docker containers                | [https://www.docker.com/products/docker-desktop/](https://www.docker.com/products/docker-desktop/)                                     |
-| `Go >= 1.25.6` | Required for building `mcpd` and running tests              | [https://go.dev/doc/install](https://go.dev/doc/install)                                                                               |
+| `Go >= 1.26.0` | Required for building `mcpd` and running tests              | [https://go.dev/doc/install](https://go.dev/doc/install)                                                                               |
 | `uv`           | For running `uvx` Python packages in `mcpd`, and local docs | [https://docs.astral.sh/uv/getting-started/installation/](https://docs.astral.sh/uv/getting-started/installation/)                     |
 | `npx`          | For running JavaScript/TypeScript packages in `mcpd`        | [https://docs.npmjs.com/downloading-and-installing-node-js-and-npm](https://docs.npmjs.com/downloading-and-installing-node-js-and-npm) |
 

go.mod

@@ -1,6 +1,6 @@
 module github.com/mozilla-ai/mcpd
 
-go 1.25.6
+go 1.26.0
 
 require (
 	github.com/BurntSushi/toml v1.5.0

internal/config/daemon_config.go

@@ -262,7 +262,10 @@ func (a *APIConfigSection) Validate() error {
 		if *a.Addr == "" {
 			validationErrors = append(validationErrors, fmt.Errorf("API address cannot be empty"))
 		} else if !isValidAddr(*a.Addr) {
-			validationErrors = append(validationErrors, fmt.Errorf("API address \"%s\" appears to be invalid (expected format: host:port)", *a.Addr))
+			validationErrors = append(
+				validationErrors,
+				fmt.Errorf("API address \"%s\" appears to be invalid (expected format: host:port)", *a.Addr),
+			)
 		}
 	}