fix spinners + interactive

Signed-off-by: razzle <harry@razzle.cloud>

src/pkg/packager/filters/deploy.go

@@ -161,6 +161,11 @@ func (f *deploymentFilter) Apply(pkg types.ZarfPackage) ([]types.ZarfComponent,
 			} else {
 				component := groupedComponents[groupKey][0]
 
+				if isRequired(component) {
+					selectedComponents = append(selectedComponents, component)
+					continue
+				}
+
 				if f.isInteractive {
 					selected, err := interactive.SelectOptionalComponent(component)
 					if err != nil {
@@ -172,7 +177,7 @@ func (f *deploymentFilter) Apply(pkg types.ZarfPackage) ([]types.ZarfComponent,
 					}
 				}
 
-				if component.Default || isRequired(component) {
+				if component.Default {
 					selectedComponents = append(selectedComponents, component)
 					continue
 				}

src/pkg/packager/mirror.go

@@ -17,9 +17,6 @@ import (
 
 // Mirror pulls resources from a package (images, git repositories, etc) and pushes them to remotes in the air gap without deploying them
 func (p *Packager) Mirror() (err error) {
-	spinner := message.NewProgressSpinner("Mirroring Zarf package %s", p.cfg.PkgOpts.PackageSource)
-	defer spinner.Stop()
-
 	filter := filters.Combine(
 		filters.ByLocalOS(runtime.GOOS),
 		filters.BySelectState(p.cfg.PkgOpts.OptionalComponents),
@@ -42,11 +39,6 @@ func (p *Packager) Mirror() (err error) {
 		return fmt.Errorf("mirror cancelled")
 	}
 
-	p.cfg.Pkg.Components, err = filter.Apply(p.cfg.Pkg)
-	if err != nil {
-		return err
-	}
-
 	p.cfg.State = &types.ZarfState{
 		RegistryInfo: p.cfg.InitOpts.RegistryInfo,
 		GitServer:    p.cfg.InitOpts.GitServer,

src/pkg/packager/sources/tarball.go

@@ -87,6 +87,7 @@ func (s *TarballSource) LoadPackage(dst *layout.PackagePaths, filter filters.Com
 		return pkg, nil, err
 	}
 	if filter != nil {
+		spinner.Stop()
 		pkg.Components, err = filter.Apply(pkg)
 		if err != nil {
 			return pkg, nil, err