fix(cli/capture): remove global Logger, inject via dependency injection (issue #585)

Fixes #585 — removes the package-level global var Logger *log.ZapLogger
from cli/cmd/root.go and replaces all usages across the capture package
with proper dependency injection. Each sub-command gets its own named
child logger for contextual logging.

Changes:
- cli/cmd/root.go: removed global Logger; added NewLogger() factory
- cli/cmd/capture/capture.go: creates logger once, passes to subcommands
- cli/cmd/capture/create.go: logger injected via constructor;
  named child logger .Named("capture-create")
- cli/cmd/capture/delete.go: logger injected;
  named child logger .Named("capture-delete")
- cli/cmd/capture/download.go: logger injected into DownloadService;
  named child logger .Named("capture-download")

Closes #585

Signed-off-by: Claude user <claudeuser@Sunithas-MacBook-Pro.local>

Author: Claude user

cli/cmd/capture/capture.go

@@ -1,11 +1,11 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT license.
-
 package capture
 
 import (
 	"time"
 
+	retinacmd "github.com/microsoft/retina/cli/cmd"
 	"github.com/spf13/cobra"
 	"k8s.io/cli-runtime/pkg/genericclioptions"
 	"k8s.io/client-go/kubernetes"
@@ -57,11 +57,9 @@ func NewCommand(kubeClient kubernetes.Interface) *cobra.Command {
 	opts.ConfigFlags = *genericclioptions.NewConfigFlags(true)
 	opts.AddFlags(capture.PersistentFlags())
 	capture.PersistentFlags().StringVar(opts.Name, "name", DefaultName, "The name of the Retina Capture")
-
-	capture.AddCommand(NewCreateSubCommand(kubeClient))
-	capture.AddCommand(NewDeleteSubCommand(kubeClient))
-	capture.AddCommand(NewDownloadSubCommand())
+	capture.AddCommand(NewCreateSubCommand(kubeClient, retinacmd.NewLogger().Named("capture-create")))
+	capture.AddCommand(NewDeleteSubCommand(kubeClient, retinacmd.NewLogger().Named("capture-delete")))
+	capture.AddCommand(NewDownloadSubCommand(retinacmd.NewLogger().Named("capture-download")))
 	capture.AddCommand(NewListSubCommand())
-
 	return capture
 }