Implemented: mcpd config daemon --help

* set: sets dot-path keys to values
* remove: removes dot-path keys (and their values)
* list: shows all config
* list --available: show all config keys that can be set
* get: gets the current value for a dot-path key

Author: peteski22

cmd/add_test.go

@@ -37,6 +37,10 @@ func (f *fakeConfig) ListServers() []config.ServerEntry {
 	return []config.ServerEntry{f.entry}
 }
 
+func (f *fakeConfig) SaveConfig() error {
+	return nil
+}
+
 type fakeLoader struct {
 	cfg *fakeConfig
 	err error

cmd/config/config.go

@@ -4,6 +4,7 @@ import (
 	"github.com/spf13/cobra"
 
 	"github.com/mozilla-ai/mcpd/v2/cmd/config/args"
+	"github.com/mozilla-ai/mcpd/v2/cmd/config/daemon"
 	"github.com/mozilla-ai/mcpd/v2/cmd/config/env"
 	"github.com/mozilla-ai/mcpd/v2/cmd/config/export"
 	"github.com/mozilla-ai/mcpd/v2/cmd/config/tools"
@@ -21,6 +22,7 @@ func NewConfigCmd(baseCmd *cmd.BaseCmd, opt ...options.CmdOption) (*cobra.Comman
 	// Sub-commands for: mcpd config
 	fns := []func(baseCmd *cmd.BaseCmd, opt ...options.CmdOption) (*cobra.Command, error){
 		args.NewCmd,   // args
+		daemon.NewCmd, // daemon
 		env.NewCmd,    // env
 		tools.NewCmd,  // tools
 		export.NewCmd, // export

cmd/config/daemon/cmd.go

@@ -0,0 +1,35 @@
+package daemon
+
+import (
+	"github.com/spf13/cobra"
+
+	"github.com/mozilla-ai/mcpd/v2/internal/cmd"
+	"github.com/mozilla-ai/mcpd/v2/internal/cmd/options"
+)
+
+func NewCmd(baseCmd *cmd.BaseCmd, opt ...options.CmdOption) (*cobra.Command, error) {
+	cobraCmd := &cobra.Command{
+		Use:   "daemon",
+		Short: "Manages daemon configuration",
+		Long:  "Manages daemon configuration in .mcpd.toml including API settings, CORS, timeouts and intervals",
+	}
+
+	// Sub-commands for: mcpd config daemon
+	fns := []func(baseCmd *cmd.BaseCmd, opt ...options.CmdOption) (*cobra.Command, error){
+		NewSetCmd,    // set
+		NewGetCmd,    // get
+		NewListCmd,   // list
+		NewRemoveCmd, // remove
+		// NewValidateCmd, // validate
+	}
+
+	for _, fn := range fns {
+		tempCmd, err := fn(baseCmd, opt...)
+		if err != nil {
+			return nil, err
+		}
+		cobraCmd.AddCommand(tempCmd)
+	}
+
+	return cobraCmd, nil
+}

cmd/config/daemon/get.go

@@ -0,0 +1,96 @@
+package daemon
+
+import (
+	"fmt"
+	"strings"
+
+	"github.com/spf13/cobra"
+
+	"github.com/mozilla-ai/mcpd/v2/internal/cmd"
+	cmdopts "github.com/mozilla-ai/mcpd/v2/internal/cmd/options"
+	"github.com/mozilla-ai/mcpd/v2/internal/config"
+)
+
+type GetCmd struct {
+	*cmd.BaseCmd
+	cfgLoader config.Loader
+}
+
+func NewGetCmd(baseCmd *cmd.BaseCmd, opt ...cmdopts.CmdOption) (*cobra.Command, error) {
+	opts, err := cmdopts.NewOptions(opt...)
+	if err != nil {
+		return nil, err
+	}
+
+	c := &GetCmd{
+		BaseCmd:   baseCmd,
+		cfgLoader: opts.ConfigLoader,
+	}
+
+	cobraCmd := &cobra.Command{
+		Use:   "get <key>",
+		Short: "Get daemon configuration value",
+		Long: `Get a specific daemon configuration value from .mcpd.toml file using dotted key notation.
+
+Examples:
+  mcpd config daemon get api.addr
+  mcpd config daemon get api.cors.enable
+  mcpd config daemon get mcp.timeout.health`,
+		RunE: c.run,
+		Args: cobra.ExactArgs(1),
+	}
+
+	return cobraCmd, nil
+}
+
+func (c *GetCmd) run(cmd *cobra.Command, args []string) error {
+	cfg, err := c.LoadConfig(c.cfgLoader)
+	if err != nil {
+		return err
+	}
+
+	if cfg.Daemon == nil {
+		return fmt.Errorf("no daemon configuration found")
+	}
+
+	// Split dotted notation into keys for variadic Get
+	key := args[0]
+	keys := strings.Split(key, ".")
+
+	value, err := cfg.Daemon.Get(keys...)
+	if err != nil {
+		return err
+	}
+
+	_, _ = fmt.Fprintln(cmd.OutOrStdout(), c.formatValue(value))
+	return nil
+}
+
+func (c *GetCmd) formatValue(value any) string {
+	switch v := value.(type) {
+	case *config.Duration:
+		if v == nil {
+			return ""
+		}
+		return v.String()
+	case config.Duration:
+		return v.String()
+	case []string:
+		if len(v) == 0 {
+			return "[]"
+		}
+		return strings.Join(v, ",")
+	case *string:
+		if v == nil {
+			return ""
+		}
+		return *v
+	case *bool:
+		if v == nil {
+			return ""
+		}
+		return fmt.Sprintf("%t", *v)
+	default:
+		return fmt.Sprintf("%v", v)
+	}
+}

cmd/config/daemon/list.go

@@ -0,0 +1,141 @@
+package daemon
+
+import (
+	"fmt"
+	"strings"
+
+	"github.com/spf13/cobra"
+
+	"github.com/mozilla-ai/mcpd/v2/internal/cmd"
+	cmdopts "github.com/mozilla-ai/mcpd/v2/internal/cmd/options"
+	"github.com/mozilla-ai/mcpd/v2/internal/config"
+)
+
+type ListCmd struct {
+	*cmd.BaseCmd
+	cfgLoader config.Loader
+	available bool
+}
+
+func NewListCmd(baseCmd *cmd.BaseCmd, opt ...cmdopts.CmdOption) (*cobra.Command, error) {
+	opts, err := cmdopts.NewOptions(opt...)
+	if err != nil {
+		return nil, err
+	}
+
+	c := &ListCmd{
+		BaseCmd:   baseCmd,
+		cfgLoader: opts.ConfigLoader,
+	}
+
+	cobraCmd := &cobra.Command{
+		Use:   "list",
+		Short: "List daemon configuration",
+		Long: `List daemon configuration from .mcpd.toml file.
+
+Examples:
+  mcpd config daemon list                # Show current configuration
+  mcpd config daemon list --available   # Show all available configuration keys`,
+		RunE: c.run,
+		Args: cobra.NoArgs,
+	}
+
+	cobraCmd.Flags().
+		BoolVar(&c.available, "available", false, "Show all available configuration keys with descriptions")
+
+	return cobraCmd, nil
+}
+
+func (c *ListCmd) run(cmd *cobra.Command, args []string) error {
+	if c.available {
+		return c.showAvailableKeys(cmd)
+	}
+
+	cfg, err := c.LoadConfig(c.cfgLoader)
+	if err != nil {
+		return err
+	}
+
+	if cfg.Daemon == nil {
+		_, _ = fmt.Fprintln(cmd.OutOrStdout(), "No daemon configuration found")
+		return nil
+	}
+
+	// Use ConfigGetter to get all configuration
+	allConfig, err := cfg.Daemon.Get()
+	if err != nil {
+		return err
+	}
+
+	return c.showConfig(cmd, allConfig, "daemon")
+}
+
+func (c *ListCmd) showConfig(cmd *cobra.Command, config any, prefix string) error {
+	_, _ = fmt.Fprintf(cmd.OutOrStdout(), "[%s]\n", prefix)
+
+	if configMap, ok := config.(map[string]any); ok {
+		c.printConfigMap(cmd, configMap, prefix, "  ")
+	}
+
+	return nil
+}
+
+func (c *ListCmd) printConfigMap(cmd *cobra.Command, configMap map[string]any, prefix string, indent string) {
+	for key, value := range configMap {
+		switch v := value.(type) {
+		case map[string]any:
+			_, _ = fmt.Fprintf(cmd.OutOrStdout(), "%s[%s.%s]\n", indent, prefix, key)
+			c.printConfigMap(cmd, v, fmt.Sprintf("%s.%s", prefix, key), indent+"  ")
+		case []string:
+			if len(v) > 0 {
+				_, _ = fmt.Fprintf(cmd.OutOrStdout(), "%s%s = %q\n", indent, key, v)
+			}
+		case bool:
+			_, _ = fmt.Fprintf(cmd.OutOrStdout(), "%s%s = %t\n", indent, key, v)
+		case string:
+			_, _ = fmt.Fprintf(cmd.OutOrStdout(), "%s%s = %q\n", indent, key, v)
+		default:
+			_, _ = fmt.Fprintf(cmd.OutOrStdout(), "%s%s = %v\n", indent, key, v)
+		}
+	}
+}
+
+func (c *ListCmd) showAvailableKeys(cmd *cobra.Command) error {
+	// Create a dummy daemon config to get the available keys
+	daemonConfig := &config.DaemonConfig{}
+	keys := daemonConfig.AvailableKeys()
+
+	_, _ = fmt.Fprintln(cmd.OutOrStdout(), "Available daemon configuration keys:")
+	_, _ = fmt.Fprintln(cmd.OutOrStdout(), "")
+
+	// Group keys by top-level section
+	apiKeys := []config.ConfigKey{}
+	mcpKeys := []config.ConfigKey{}
+
+	for _, key := range keys {
+		if strings.HasPrefix(key.Path, "api.") {
+			apiKeys = append(apiKeys, key)
+		} else if strings.HasPrefix(key.Path, "mcp.") {
+			mcpKeys = append(mcpKeys, key)
+		}
+	}
+
+	// Show API keys
+	if len(apiKeys) > 0 {
+		_, _ = fmt.Fprintln(cmd.OutOrStdout(), "API Configuration:")
+		for _, key := range apiKeys {
+			_, _ = fmt.Fprintf(cmd.OutOrStdout(), "  %-30s %-12s %s\n", key.Path, "("+key.Type+")", key.Description)
+		}
+		_, _ = fmt.Fprintln(cmd.OutOrStdout(), "")
+	}
+
+	// Show MCP keys
+	if len(mcpKeys) > 0 {
+		_, _ = fmt.Fprintln(cmd.OutOrStdout(), "MCP Configuration:")
+		for _, key := range mcpKeys {
+			_, _ = fmt.Fprintf(cmd.OutOrStdout(), "  %-30s %-12s %s\n", key.Path, "("+key.Type+")", key.Description)
+		}
+	}
+
+	return nil
+}

cmd/config/daemon/remove.go

@@ -0,0 +1,71 @@
+package daemon
+
+import (
+	"fmt"
+
+	"github.com/spf13/cobra"
+
+	"github.com/mozilla-ai/mcpd/v2/internal/cmd"
+	cmdopts "github.com/mozilla-ai/mcpd/v2/internal/cmd/options"
+	"github.com/mozilla-ai/mcpd/v2/internal/config"
+)
+
+type RemoveCmd struct {
+	*cmd.BaseCmd
+	cfgLoader config.Loader
+}
+
+func NewRemoveCmd(baseCmd *cmd.BaseCmd, opt ...cmdopts.CmdOption) (*cobra.Command, error) {
+	opts, err := cmdopts.NewOptions(opt...)
+	if err != nil {
+		return nil, err
+	}
+
+	c := &RemoveCmd{
+		BaseCmd:   baseCmd,
+		cfgLoader: opts.ConfigLoader,
+	}
+
+	cobraCmd := &cobra.Command{
+		Use:   "remove <key> [key ...]",
+		Short: "Remove daemon configuration values",
+		Long: `Remove specific daemon configuration values from .mcpd.toml file using dotted key notation.
+
+Examples:
+  mcpd config daemon remove api.addr
+  mcpd config daemon remove api.cors.enable api.cors.allow_origins
+  mcpd config daemon remove mcp.timeout.health`,
+		RunE: c.run,
+		Args: cobra.MinimumNArgs(1),
+	}
+
+	return cobraCmd, nil
+}
+
+func (c *RemoveCmd) run(cmd *cobra.Command, args []string) error {
+	cfg, err := c.LoadConfig(c.cfgLoader)
+	if err != nil {
+		return err
+	}
+
+	if cfg.Daemon == nil {
+		return fmt.Errorf("no daemon configuration found")
+	}
+
+	// Remove each specified key by setting to empty value
+	for _, key := range args {
+		_, err := cfg.Daemon.Set(key, "") // Internal use with empty value
+		if err != nil {
+			return err // Already has full path context from domain object
+		}
+
+		_, _ = fmt.Fprintf(cmd.OutOrStdout(), "✓ Removed daemon config '%s'\n", key)
+	}
+
+	// Save config
+	if err := cfg.SaveConfig(); err != nil {
+		return fmt.Errorf("failed to save config: %w", err)
+	}
+
+	return nil
+}