refactor(runtime): Implement hook using runtime

cmd/hook.go

@@ -1,12 +1,11 @@
 package cmd
 
 import (
-	"context"
 	"fmt"
 
 	"github.com/spf13/cobra"
 	"github.com/windsorcli/cli/pkg/di"
-	"github.com/windsorcli/cli/pkg/pipelines"
+	"github.com/windsorcli/cli/pkg/runtime"
 )
 
 var hookCmd = &cobra.Command{
@@ -19,21 +18,14 @@ var hookCmd = &cobra.Command{
 		// Get shared dependency injector from context
 		injector := cmd.Context().Value(injectorKey).(di.Injector)
 
-		// Create execution context with shell type
-		ctx := context.WithValue(cmd.Context(), "shellType", args[0])
-		if verbose {
-			ctx = context.WithValue(ctx, "verbose", true)
-		}
+		// Create Runtime and execute hook installation
+		err := runtime.NewRuntime(injector).
+			LoadShell().
+			InstallHook(args[0]).
+			Do()
 
-		// Set up the hook pipeline
-		pipeline, err := pipelines.WithPipeline(injector, ctx, "hookPipeline")
 		if err != nil {
-			return fmt.Errorf("failed to set up hook pipeline: %w", err)
-		}
-
-		// Execute the pipeline
-		if err := pipeline.Execute(ctx); err != nil {
-			return fmt.Errorf("Error executing hook pipeline: %w", err)
+			return fmt.Errorf("Error installing hook: %w", err)
 		}
 
 		return nil

pkg/pipelines/pipeline.go

@@ -55,7 +55,6 @@ var pipelineConstructors = map[string]PipelineConstructor{
 	"initPipeline":     func() Pipeline { return NewInitPipeline() },
 	"execPipeline":     func() Pipeline { return NewExecPipeline() },
 	"contextPipeline":  func() Pipeline { return NewContextPipeline() },
-	"hookPipeline":     func() Pipeline { return NewHookPipeline() },
 	"checkPipeline":    func() Pipeline { return NewCheckPipeline() },
 	"upPipeline":       func() Pipeline { return NewUpPipeline() },
 	"downPipeline":     func() Pipeline { return NewDownPipeline() },

pkg/pipelines/pipeline_test.go

@@ -528,7 +528,6 @@ func TestWithPipeline(t *testing.T) {
 			{"InitPipeline", "initPipeline"},
 			{"ExecPipeline", "execPipeline"},
 			{"ContextPipeline", "contextPipeline"},
-			{"HookPipeline", "hookPipeline"},
 			{"CheckPipeline", "checkPipeline"},
 			{"UpPipeline", "upPipeline"},
 			{"DownPipeline", "downPipeline"},
@@ -728,7 +727,6 @@ func TestWithPipeline(t *testing.T) {
 			"initPipeline",
 			"execPipeline",
 			"contextPipeline",
-			"hookPipeline",
 			"checkPipeline",
 			"basePipeline",
 		}