feat(cli/extension): add enable and disable subcommands (#200)

Adding disable (and its counterpart enable) also gives users a non-destructive way to temporarily turn off an extension without losing its config entry.

Author: indaco

README.md

@@ -50,12 +50,12 @@ sley manages [SemVer 2.0.0](https://semver.org/) versions using a simple `.versi
 
 ## Features
 
-- **Simple** — One `.version` file, one source of truth
-- **Language-agnostic** — Works with Go, Node, Python, Rust, or any stack
-- **Plugin system** — Extend with built-in or custom plugins
-- **Git integration** — Auto-tag releases, generate changelogs
-- **Monorepo support** — Manage multiple modules independently
-- **CI/CD ready** — Designed for automation pipelines
+- **Simple** - One `.version` file, one source of truth
+- **Language-agnostic** - Works with Go, Node, Python, Rust, or any stack
+- **Plugin system** - Extend with built-in or custom plugins
+- **Git integration** - Auto-tag releases, generate changelogs
+- **Monorepo support** - Manage multiple modules independently
+- **CI/CD ready** - Designed for automation pipelines
 
 ## Installation
 

contrib/extensions/README.md

@@ -44,8 +44,14 @@ sley extension install --path ./contrib/extensions/github-version-sync
 # List installed extensions
 sley extension list
 
-# Remove an extension
-sley extension remove --name commit-validator
+# Disable an extension (temporarily turn off without uninstalling)
+sley extension disable --name commit-validator
+
+# Re-enable a disabled extension
+sley extension enable --name commit-validator
+
+# Uninstall an extension (remove from config, optionally delete files)
+sley extension uninstall --name commit-validator
 ```
 
 ## Documentation

internal/commands/extension/disable.go

@@ -0,0 +1,51 @@
+package extension
+
+import (
+	"context"
+	"fmt"
+	"strings"
+
+	"github.com/indaco/sley/internal/extensionmgr"
+	"github.com/indaco/sley/internal/printer"
+	"github.com/urfave/cli/v3"
+)
+
+// disableCmd returns the "disable" subcommand.
+func disableCmd() *cli.Command {
+	return &cli.Command{
+		Name:  "disable",
+		Usage: "Disable a registered extension",
+		Flags: []cli.Flag{
+			&cli.StringFlag{
+				Name:  "name",
+				Usage: "Name of the extension to disable",
+			},
+		},
+		Action: func(ctx context.Context, cmd *cli.Command) error {
+			return runExtensionDisable(cmd)
+		},
+	}
+}
+
+// runExtensionDisable disables a registered extension by setting enabled: false
+// in the configuration file. It uses a surgical YAML replacement so that
+// comments and formatting are preserved.
+func runExtensionDisable(cmd *cli.Command) error {
+	extensionName := cmd.String("name")
+	if extensionName == "" {
+		return fmt.Errorf("please provide an extension name to disable")
+	}
+
+	updater := extensionmgr.NewDefaultConfigUpdater(&extensionmgr.DefaultYAMLMarshaler{})
+	if err := updater.SetExtensionEnabled(configFilePath, extensionName, false); err != nil {
+		if strings.Contains(err.Error(), "not found in configuration") {
+			printer.PrintWarning(fmt.Sprintf("extension %q not found", extensionName))
+			return nil
+		}
+		printer.PrintError(fmt.Sprintf("failed to disable extension: %v", err))
+		return nil
+	}
+
+	printer.PrintSuccess(fmt.Sprintf("Extension %q disabled.", extensionName))
+	return nil
+}

internal/commands/extension/enable.go

@@ -0,0 +1,51 @@
+package extension
+
+import (
+	"context"
+	"fmt"
+	"strings"
+
+	"github.com/indaco/sley/internal/extensionmgr"
+	"github.com/indaco/sley/internal/printer"
+	"github.com/urfave/cli/v3"
+)
+
+// enableCmd returns the "enable" subcommand.
+func enableCmd() *cli.Command {
+	return &cli.Command{
+		Name:  "enable",
+		Usage: "Enable a registered extension",
+		Flags: []cli.Flag{
+			&cli.StringFlag{
+				Name:  "name",
+				Usage: "Name of the extension to enable",
+			},
+		},
+		Action: func(ctx context.Context, cmd *cli.Command) error {
+			return runExtensionEnable(cmd)
+		},
+	}
+}
+
+// runExtensionEnable enables a registered extension by setting enabled: true
+// in the configuration file. It uses a surgical YAML replacement so that
+// comments and formatting are preserved.
+func runExtensionEnable(cmd *cli.Command) error {
+	extensionName := cmd.String("name")
+	if extensionName == "" {
+		return fmt.Errorf("please provide an extension name to enable")
+	}
+
+	updater := extensionmgr.NewDefaultConfigUpdater(&extensionmgr.DefaultYAMLMarshaler{})
+	if err := updater.SetExtensionEnabled(configFilePath, extensionName, true); err != nil {
+		if strings.Contains(err.Error(), "not found in configuration") {
+			printer.PrintWarning(fmt.Sprintf("extension %q not found", extensionName))
+			return nil
+		}
+		printer.PrintError(fmt.Sprintf("failed to enable extension: %v", err))
+		return nil
+	}
+
+	printer.PrintSuccess(fmt.Sprintf("Extension %q enabled.", extensionName))
+	return nil
+}

internal/commands/extension/extensioncmd.go

@@ -11,8 +11,10 @@ func Run() *cli.Command {
 		Usage: "Manage extensions for sley",
 		Commands: []*cli.Command{
 			installCmd(),
-			listCmd(),
 			uninstallCmd(),
+			enableCmd(),
+			disableCmd(),
+			listCmd(),
 		},
 	}
 }

internal/commands/extension/extensioncmd_test.go

@@ -601,6 +601,172 @@ func TestExtensionUninstallCmd_WriteConfigError(t *testing.T) {
 	}
 }
 
+/* ------------------------------------------------------------------------- */
+/* EXTENSION ENABLE COMMAND                                                  */
+/* ------------------------------------------------------------------------- */
+
+func TestExtensionEnableCmd_Success(t *testing.T) {
+	tmpDir := t.TempDir()
+	configPath := filepath.Join(tmpDir, ".sley.yaml")
+
+	content := `extensions:
+  - name: mock-extension
+    path: /some/path
+    enabled: false`
+	if err := os.WriteFile(configPath, []byte(content), 0644); err != nil {
+		t.Fatalf("failed to write config file: %v", err)
+	}
+
+	cfg := &config.Config{Path: configPath}
+	appCli := testutils.BuildCLIForTests(cfg.Path, []*cli.Command{Run()})
+
+	output, err := testutils.CaptureStdout(func() {
+		testutils.RunCLITest(t, appCli, []string{
+			"sley", "extension", "enable", "--name", "mock-extension",
+		}, tmpDir)
+	})
+	if err != nil {
+		t.Fatalf("CLI run failed: %v", err)
+	}
+
+	expected := `Extension "mock-extension" enabled.`
+	if !strings.Contains(output, expected) {
+		t.Errorf("expected output to contain %q, got:\n%s", expected, output)
+	}
+
+	// Verify the extension is now enabled in the config
+	data, readErr := os.ReadFile(configPath)
+	if readErr != nil {
+		t.Fatalf("failed to read config: %v", readErr)
+	}
+	if !strings.Contains(string(data), "enabled: true") {
+		t.Errorf("expected config to contain 'enabled: true', got:\n%s", string(data))
+	}
+}
+
+func TestExtensionEnableCmd_MissingName(t *testing.T) {
+	if os.Getenv("TEST_EXTENSION_ENABLE_MISSING_NAME") == "1" {
+		tmp := t.TempDir()
+		configPath := filepath.Join(tmp, ".sley.yaml")
+		content := `extensions:
+  - name: mock-extension
+    path: /some/path
+    enabled: false`
+		if err := os.WriteFile(configPath, []byte(content), 0644); err != nil {
+			fmt.Fprintln(os.Stderr, "failed to write config:", err)
+			os.Exit(1)
+		}
+
+		cfg := &config.Config{Path: configPath}
+		appCli := testutils.BuildCLIForTests(cfg.Path, []*cli.Command{Run()})
+
+		err := appCli.Run(context.Background(), []string{
+			"sley", "extension", "enable",
+		})
+		if err != nil {
+			fmt.Fprintln(os.Stderr, err)
+			os.Exit(1)
+		}
+		os.Exit(0)
+	}
+
+	cmd := exec.Command(os.Args[0], "-test.run=TestExtensionEnableCmd_MissingName")
+	cmd.Env = append(os.Environ(), "TEST_EXTENSION_ENABLE_MISSING_NAME=1")
+	output, err := cmd.CombinedOutput()
+
+	if err == nil {
+		t.Fatal("expected non-zero exit status")
+	}
+
+	expected := "please provide an extension name to enable"
+	if !strings.Contains(string(output), expected) {
+		t.Errorf("expected output to contain %q, got:\n%s", expected, output)
+	}
+}
+
+/* ------------------------------------------------------------------------- */
+/* EXTENSION DISABLE COMMAND                                                 */
+/* ------------------------------------------------------------------------- */
+
+func TestExtensionDisableCmd_Success(t *testing.T) {
+	tmpDir := t.TempDir()
+	configPath := filepath.Join(tmpDir, ".sley.yaml")
+
+	content := `extensions:
+  - name: mock-extension
+    path: /some/path
+    enabled: true`
+	if err := os.WriteFile(configPath, []byte(content), 0644); err != nil {
+		t.Fatalf("failed to write config file: %v", err)
+	}
+
+	cfg := &config.Config{Path: configPath}
+	appCli := testutils.BuildCLIForTests(cfg.Path, []*cli.Command{Run()})
+
+	output, err := testutils.CaptureStdout(func() {
+		testutils.RunCLITest(t, appCli, []string{
+			"sley", "extension", "disable", "--name", "mock-extension",
+		}, tmpDir)
+	})
+	if err != nil {
+		t.Fatalf("CLI run failed: %v", err)
+	}
+
+	expected := `Extension "mock-extension" disabled.`
+	if !strings.Contains(output, expected) {
+		t.Errorf("expected output to contain %q, got:\n%s", expected, output)
+	}
+
+	// Verify the extension is now disabled in the config
+	data, readErr := os.ReadFile(configPath)
+	if readErr != nil {
+		t.Fatalf("failed to read config: %v", readErr)
+	}
+	if !strings.Contains(string(data), "enabled: false") {
+		t.Errorf("expected config to contain 'enabled: false', got:\n%s", string(data))
+	}
+}
+
+func TestExtensionDisableCmd_MissingName(t *testing.T) {
+	if os.Getenv("TEST_EXTENSION_DISABLE_MISSING_NAME") == "1" {
+		tmp := t.TempDir()
+		configPath := filepath.Join(tmp, ".sley.yaml")
+		content := `extensions:
+  - name: mock-extension
+    path: /some/path
+    enabled: true`
+		if err := os.WriteFile(configPath, []byte(content), 0644); err != nil {
+			fmt.Fprintln(os.Stderr, "failed to write config:", err)
+			os.Exit(1)
+		}
+
+		cfg := &config.Config{Path: configPath}
+		appCli := testutils.BuildCLIForTests(cfg.Path, []*cli.Command{Run()})
+
+		err := appCli.Run(context.Background(), []string{
+			"sley", "extension", "disable",
+		})
+		if err != nil {
+			fmt.Fprintln(os.Stderr, err)
+			os.Exit(1)
+		}
+		os.Exit(0)
+	}
+
+	cmd := exec.Command(os.Args[0], "-test.run=TestExtensionDisableCmd_MissingName")
+	cmd.Env = append(os.Environ(), "TEST_EXTENSION_DISABLE_MISSING_NAME=1")
+	output, err := cmd.CombinedOutput()
+
+	if err == nil {
+		t.Fatal("expected non-zero exit status")
+	}
+
+	expected := "please provide an extension name to disable"
+	if !strings.Contains(string(output), expected) {
+		t.Errorf("expected output to contain %q, got:\n%s", expected, output)
+	}
+}
+
 /* ------------------------------------------------------------------------- */
 /* EXTENSION INSTALL COMMAND - ADDITIONAL TESTS                              */
 /* ------------------------------------------------------------------------- */

internal/extensionmgr/interfaces.go

@@ -14,6 +14,7 @@ type YAMLMarshaler interface {
 type ConfigUpdater interface {
 	AddExtension(path string, extension config.ExtensionConfig) error
 	RemoveExtension(path string, extensionName string) error
+	SetExtensionEnabled(path string, extensionName string, enabled bool) error
 }
 
 // ManifestLoader handles loading extension manifests

internal/extensionmgr/mocks.go

@@ -22,8 +22,9 @@ func (m *MockYAMLMarshaler) Marshal(v any) ([]byte, error) {
 
 // MockConfigUpdater is a mock implementation of ConfigUpdater for testing
 type MockConfigUpdater struct {
-	AddExtensionFunc    func(path string, extension config.ExtensionConfig) error
-	RemoveExtensionFunc func(path string, extensionName string) error
+	AddExtensionFunc        func(path string, extension config.ExtensionConfig) error
+	RemoveExtensionFunc     func(path string, extensionName string) error
+	SetExtensionEnabledFunc func(path string, extensionName string, enabled bool) error
 }
 
 func (m *MockConfigUpdater) AddExtension(path string, extension config.ExtensionConfig) error {
@@ -40,6 +41,13 @@ func (m *MockConfigUpdater) RemoveExtension(path string, extensionName string) e
 	return nil
 }
 
+func (m *MockConfigUpdater) SetExtensionEnabled(path string, extensionName string, enabled bool) error {
+	if m.SetExtensionEnabledFunc != nil {
+		return m.SetExtensionEnabledFunc(path, extensionName, enabled)
+	}
+	return nil
+}
+
 // MockManifestLoader is a mock implementation of ManifestLoader for testing
 type MockManifestLoader struct {
 	LoadFunc func(path string) (*extensions.ExtensionManifest, error)