From 12dfcdf25aae3b204247b8912461dfe7e44e10ef Mon Sep 17 00:00:00 2001
From: Christian Rocha <christian@rocha.is>
Date: Thu, 19 Mar 2026 12:58:03 -0400
Subject: [PATCH] docs(mcp/docker): comments

---
 internal/config/docker_mcp.go | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/internal/config/docker_mcp.go b/internal/config/docker_mcp.go
index 0bac24dff2aedb4fd27023f604e1978b7f936852..fda70e36f2734164c3bf926532b0b85b25e3ef34 100644
--- a/internal/config/docker_mcp.go
+++ b/internal/config/docker_mcp.go
@@ -35,6 +35,8 @@ func IsDockerMCPAvailable() bool {
 	return err == nil
 }
 
+// DockerMCPAvailabilityCached returns the cached Docker MCP availability and
+// whether the cached value is still fresh.
 func DockerMCPAvailabilityCached() (available bool, known bool) {
 	dockerMCPAvailabilityCache.mu.Lock()
 	defer dockerMCPAvailabilityCache.mu.Unlock()
@@ -48,6 +50,7 @@ func DockerMCPAvailabilityCached() (available bool, known bool) {
 	return dockerMCPAvailabilityCache.available, true
 }
 
+// RefreshDockerMCPAvailability refreshes and caches Docker MCP availability.
 func RefreshDockerMCPAvailability() bool {
 	available := IsDockerMCPAvailable()
 	dockerMCPAvailabilityCache.mu.Lock()
@@ -67,6 +70,7 @@ func (c *Config) IsDockerMCPEnabled() bool {
 	return exists
 }
 
+// DockerMCPConfig returns the default Docker MCP stdio configuration.
 func DockerMCPConfig() MCPConfig {
 	return MCPConfig{
 		Type:     MCPStdio,
@@ -76,6 +80,8 @@ func DockerMCPConfig() MCPConfig {
 	}
 }
 
+// PrepareDockerMCPConfig validates Docker MCP availability and stages the
+// Docker MCP configuration in memory.
 func (s *ConfigStore) PrepareDockerMCPConfig() (MCPConfig, error) {
 	if !IsDockerMCPAvailable() {
 		return MCPConfig{}, fmt.Errorf("docker mcp is not available, please ensure docker is installed and 'docker mcp version' succeeds")
@@ -89,6 +95,8 @@ func (s *ConfigStore) PrepareDockerMCPConfig() (MCPConfig, error) {
 	return mcpConfig, nil
 }
 
+// PersistDockerMCPConfig persists a previously prepared Docker MCP
+// configuration to the global config file.
 func (s *ConfigStore) PersistDockerMCPConfig(mcpConfig MCPConfig) error {
 	if err := s.SetConfigField(ScopeGlobal, "mcp."+DockerMCPName, mcpConfig); err != nil {
 		return fmt.Errorf("failed to persist docker mcp configuration: %w", err)