feat: Add --noconfirm for auto build

Author: Hayao0819

CLAUDE.md

@@ -51,6 +51,7 @@ Build requires root privileges. Output goes to `alteriso/out/`, work directory i
 ### Injectable Mechanism
 
 AlterISO leverages archiso's `_run_once()` function which sources `profiledef.sh`. Generated `profiledef.sh` includes:
+
 - Pre/post hooks: `pre_<function>()`, `post_<function>()`
 - Override hooks: `override_<function>()`
 - Example: `post__make_packages()`, `pre__make_customize_airootfs()`
@@ -69,6 +70,7 @@ modules/<name>/
 ### Profile Configuration
 
 `profiledef.json` fields:
+
 - `arch` (required): Target architecture (currently "x86_64")
 - `modules` (required): List of modules to load in order
 - `os_name`, `kernel_name`, `username`, `cow_spacesize`: Optional config

alteriso/src/internal/archiso/injects/loader.sh

@@ -32,13 +32,13 @@ __alteriso_injected_list() {
         fi
     done < <(compgen -v)
 
-	local _a _a_org
-	while read -r _a; do
-		_a_org=$(echo "${_a}" | sed -E 's/^(pre|post|override)_(.+)$/\2/')
-		if printf '%s\n' "${_funcs[@]}" | grep -qx -- "$_a_org"; then
-			_msg_info "Injected as array: $_a"
-		fi
-	done < <(printf '%s\n' "${_arrays[@]}" | grep -E -- '^(pre|post|override)_(.+)$' | sort -u)
+    local _a _a_org
+    while read -r _a; do
+        _a_org=$(echo "${_a}" | sed -E 's/^(pre|post|override)_(.+)$/\2/')
+        if printf '%s\n' "${_funcs[@]}" | grep -qx -- "$_a_org"; then
+            _msg_info "Injected as array: $_a"
+        fi
+    done < <(printf '%s\n' "${_arrays[@]}" | grep -E -- '^(pre|post|override)_(.+)$' | sort -u)
 }
 
 __alteriso_cleanup() {
@@ -197,16 +197,17 @@ __alteriso_show_config() {
     _kernel_name=$(__alteriso_profiledef_kernelname)
     _arch=$(__alteriso_profiledef_arch)
 
-    echo "[alteriso] INFO:              Architecture:   $_arch"
-    echo "[alteriso] INFO:                  Username:   $_username"
-    echo "[alteriso] INFO:               Kernel Name:   $_kernel_name"
-    echo "[alteriso] INFO:         Profile Directory:   $__alteriso_profile_dir"
+    echo "[alteriso] INFO:              Architecture:   $_arch" >&2
+    echo "[alteriso] INFO:                  Username:   $_username" >&2
+    echo "[alteriso] INFO:               Kernel Name:   $_kernel_name" >&2
+    echo "[alteriso] INFO:         Profile Directory:   $__alteriso_profile_dir" >&2
 
-    echo -n "[alteriso] Are you sure to continue? (y/N): "
+    echo -n "[alteriso] Are you sure to continue? (y/N): " >&2
     read -r answer
-    if [[ "$answer" != "y" && "$answer" != "Y" ]]; then
+
+    if [[ "${answer,,}" = "y" ]]; then
         echo "[alteriso] Aborted."
-        exit 0
+        exit 1
     fi
 }
 

alteriso/src/internal/archiso/injects/profiledef.sh.in

@@ -1,27 +1,28 @@
 #!/usr/bin/env bash
 # shellcheck disable=all
 
+# Initial loader
 {{ .LoaderContent }}
 
 __alteriso_loadfile "$__alteriso_profile_dir/injecter.sh"
 
-{{ range $index, $module := .EmbedScripts }}
+# Function defines
+{{- range $index, $module := .EmbedScripts }}
 {{ inject_script $module }}
 {{ end }}
-
-{{- range $func, $codes := .Injects -}}
+# Inject Array
+{{ range $func, $codes := .Injects -}}
     {{- $func }}{{ "+=(" }}
         {{- range $index, $code := $codes -}}
             {{- $code }}
         {{- end -}}
     {{ ")" }}
 {{ end -}}
-
+# Variable defines
 {{ .ProfileDefContent }}
-
+# Validate alteriso profiles
 __alteriso_validate
 __alteriso_validate_profile
-
 {{ if .RequireInjectable -}}
 __alteriso_require_injectable
 {{- end }}
@@ -32,4 +33,6 @@ if [[ "$__alteriso_compatible_mode" = "y" ]]; then
     return 0
 fi
 __alteriso_injected_list
+{{- if not .NoConfirm }}
 __alteriso_show_config
+{{- end }}

alteriso/src/internal/archiso/profile_def.go

@@ -74,12 +74,14 @@ func (p *Profile) ProfileDefSh() ([]byte, error) {
 		EmbedScripts      []string
 		Injects           map[string][]string
 		RequireInjectable bool
+		NoConfirm         bool
 	}{
 		LoaderContent:     string(loaderContent),
 		ProfileDefContent: string(profileDef),
 		EmbedScripts:      embedScripts,
 		Injects:           injects,
 		RequireInjectable: p.Config.RequireInjectable,
+		NoConfirm:         p.NoConfirm,
 	}
 
 	profileDefContent, err := profileDefTemplate()

alteriso/src/internal/archiso/profile_new.go

@@ -16,6 +16,13 @@ type profileOption func(*Profile) error
 // 	}
 // }
 
+func WithNoConfirm(f bool) profileOption {
+	return func(p *Profile) error {
+		p.NoConfirm = f
+		return nil
+	}
+}
+
 func WithModulesPath(dir string) profileOption {
 	return func(p *Profile) error {
 		p.ModulesPath = dir

alteriso/src/internal/archiso/profile_struct.go

@@ -6,6 +6,7 @@ type Profile struct {
 	Path            string
 	BootloadersPath string
 	ModulesPath     string
+	NoConfirm       bool
 }
 
 type ProfileDef struct {

alteriso/src/internal/cmd/profile/generate.go

@@ -3,7 +3,6 @@ package profile
 import (
 	"log/slog"
 	"os"
-	"path"
 
 	"github.com/FascodeNet/alterlinux/src/internal/archiso"
 	"github.com/FascodeNet/alterlinux/src/internal/errors"
@@ -38,27 +37,27 @@ func generateCmd() *cobra.Command {
 
 			bootloadersPath := cmd.Parent().PersistentFlags().Lookup("bootloaders").Value.String()
 			modulesPath := cmd.Parent().PersistentFlags().Lookup("modules").Value.String()
+			noConfirm := cmd.Parent().PersistentFlags().Lookup("noconfirm").Changed
 
 			profile, err := archiso.NewProfile(configDir,
 				archiso.WithModulesPath(modulesPath),
 				archiso.WithbootloadersPath(bootloadersPath),
+				archiso.WithNoConfirm(noConfirm),
 				// archiso.WithPacmanCacheDir(pacmanCacheDir),
 			)
 			if err != nil {
 				return err
 			}
 
-			// dstDir := path.Join(outDir, configName)
-			dstDir := path.Join(outDir)
-			if futils.Exists(dstDir) {
+			if futils.Exists(outDir) {
 				return errors.New("output directory already exists")
 			}
 
-			if err := profile.GenArchisoProfile(dstDir); err != nil {
+			if err := profile.GenArchisoProfile(outDir); err != nil {
 				return errors.Wrap(err)
 			}
 
-			slog.Info("Generated archiso profile", "dir", dstDir)
+			slog.Info("Generated archiso profile", "dir", outDir)
 
 			return nil
 		},

alteriso/src/internal/cmd/profile/root.go

@@ -10,6 +10,7 @@ var profileReg = cobrautils.Registory{}
 func Cmd() *cobra.Command {
 	bootloadersPath := "/usr/share/alteriso/bootloaders"
 	modulesPath := "/usr/share/alteriso/modules/"
+	noconfirm := false
 
 	cmd := &cobra.Command{
 		Use:   "profile",
@@ -18,6 +19,7 @@ func Cmd() *cobra.Command {
 
 	cmd.PersistentFlags().StringVarP(&modulesPath, "modules", "", modulesPath, "Path to modules config dir")
 	cmd.PersistentFlags().StringVarP(&bootloadersPath, "bootloaders", "", bootloadersPath, "Path to bootloaders config dir")
+	cmd.PersistentFlags().BoolVarP(&noconfirm, "noconfirm", "", noconfirm, "No check the settings before building")
 
 	profileReg.Bind(cmd)
 	return cmd