refactor(extensions): enhance modularity and update dependencies across multiple extensions

- Restructured various extensions to improve modularity and maintainability, including updates to configuration and core functionalities.
- Added new helper functions and improved existing ones for better clarity and usability.
- Updated dependencies in `go.mod` and `go.sum` files across multiple extensions to ensure compatibility with the latest versions.
- Enhanced documentation and comments for better understanding and guidance on extension usage.

Signed-off-by: Rex Raphael <rex.raphael@outlook.com>

Author: juicycleff

.golangci.yml

@@ -204,6 +204,11 @@ linters:
       - linters:
           - staticcheck
         text: "SA9003:"
+      # Exclude SA5011 from test files (nil pointer checks are intentional in tests)
+      - path: _test\.go
+        linters:
+          - staticcheck
+        text: "SA5011"
       # Exclude `lll` issues for long lines with `go:generate`.
       - linters:
           - lll

app_impl.go

@@ -148,13 +148,11 @@ func newApp(config AppConfig) *app {
 					)
 				}
 			}
-		} else {
+		} else if logger != nil {
 			// Auto-discovery failed, but that's okay - we'll create a default manager
-			if logger != nil {
-				logger.Debug("config auto-discovery did not find files, using empty config",
-					F("error", err.Error()),
-				)
-			}
+			logger.Debug("config auto-discovery did not find files, using empty config",
+				F("error", err.Error()),
+			)
 		}
 	}
 

cli/color.go

@@ -72,7 +72,7 @@ func isTerminal(w io.Writer) bool {
 
 // ConfigureColors configures the global color settings.
 func ConfigureColors(config ColorConfig) {
-	if config.NoColor {
+	if config.NoColor { //nolint:gocritic // ifElseChain: priority-based config clearer with if-else
 		color.NoColor = true
 	} else if config.ForceColor {
 		color.NoColor = false

cli/context.go

@@ -10,7 +10,7 @@ import (
 
 // commandContext implements the CommandContext interface.
 type commandContext struct {
-	ctx    context.Context
+	ctx    context.Context //nolint:containedctx // context needed for command lifecycle and cancellation
 	cmd    Command
 	args   []string
 	flags  map[string]*flagValue

cli/flag.go

@@ -202,15 +202,15 @@ func WithValidator(validator func(any) error) FlagOption {
 }
 
 // ValidateRange validates that an int flag is within a range.
-func ValidateRange(min, max int) FlagOption {
+func ValidateRange(minVal, maxVal int) FlagOption {
 	return WithValidator(func(value any) error {
 		v, ok := value.(int)
 		if !ok {
 			return errors.New("expected int value")
 		}
 
-		if v < min || v > max {
-			return fmt.Errorf("value must be between %d and %d", min, max)
+		if v < minVal || v > maxVal {
+			return fmt.Errorf("value must be between %d and %d", minVal, maxVal)
 		}
 
 		return nil

cli/prompt.go

@@ -255,7 +255,7 @@ func interactiveSelect(question string, options []string, multi bool) ([]string,
 
 	fmt.Fprint(os.Stdout, "\r\n") // CR+LF after instructions
 	fmt.Fprint(os.Stdout, "\r\n") // Empty line before options
-	os.Stdout.Sync()               // Flush header before rendering options
+	os.Stdout.Sync()              // Flush header before rendering options
 
 	for {
 		// Render options
@@ -283,8 +283,8 @@ func interactiveSelect(question string, options []string, multi bool) ([]string,
 
 		case 27: // ESC sequence
 			// Arrow keys send ESC [ A/B/C/D
-			if n >= 3 && buf[1] == '[' {
-				switch buf[2] {
+			if n >= 3 && buf[1] == '[' { //nolint:gosec // G602: bounds checked with n >= 3
+				switch buf[2] { //nolint:gosec // G602: bounds checked with n >= 3
 				case 'A': // Up arrow
 					if cursor > 0 {
 						cursor--
@@ -375,9 +375,9 @@ func renderOptions(options []string, cursor int, selected map[int]bool, multi bo
 		prefix := "  "
 		optText := opt
 
-		if i == cursor {
+		if i == cursor { //nolint:gocritic // ifElseChain: UI state logic clearer with nested if-else
 			// Highlighted item
-			if multi && selected[i] {
+			if multi && selected[i] { //nolint:gocritic // ifElseChain: checkbox state clearer with if-else
 				prefix = Green("▸ [✓]")
 			} else if multi {
 				prefix = Green("▸ [ ]")
@@ -443,7 +443,6 @@ func setRawMode() (*term.State, error) {
 // restoreMode restores the terminal to normal mode.
 func restoreMode(oldState *term.State) {
 	if oldState != nil {
-		//nolint:errcheck // Terminal restore errors are not critical
 		_ = term.Restore(int(os.Stdin.Fd()), oldState)
 	}
 }

cmd/forge/go.mod

@@ -88,8 +88,8 @@ require (
 	github.com/xdg-go/scram v1.1.2 // indirect
 	github.com/xdg-go/stringprep v1.0.4 // indirect
 	github.com/xraph/confy v0.0.3 // indirect
-	github.com/xraph/go-utils v0.0.10 // indirect
-	github.com/xraph/vessel v0.0.1 // indirect
+	github.com/xraph/go-utils v0.0.11 // indirect
+	github.com/xraph/vessel v0.0.3 // indirect
 	github.com/youmark/pkcs8 v0.0.0-20240726163527-a2c0da244d78 // indirect
 	go.mongodb.org/mongo-driver v1.17.4 // indirect
 	go.uber.org/multierr v1.11.0 // indirect

cmd/forge/go.sum

@@ -55,8 +55,8 @@ require (
 	github.com/uptrace/bunrouter v1.0.23 // indirect
 	github.com/x448/float16 v0.8.4 // indirect
 	github.com/xraph/confy v0.0.3 // indirect
-	github.com/xraph/go-utils v0.0.10 // indirect
-	github.com/xraph/vessel v0.0.1 // indirect
+	github.com/xraph/go-utils v0.0.11 // indirect
+	github.com/xraph/vessel v0.0.3 // indirect
 	go.uber.org/multierr v1.11.0 // indirect
 	go.uber.org/zap v1.27.1 // indirect
 	go.yaml.in/yaml/v2 v2.4.3 // indirect