refactor: add config status enum for adding validation in the future (#71)

* feat: add config status for future validation

* refactor: use status enum and switch

Author: sushichan044

cmd/ajisai/root.go

@@ -53,7 +53,8 @@ func Run(args []string) error {
 					return config.StoreNotFoundInContext(ctx), nil
 				}
 
-				return ctx, fmt.Errorf("failed to load configuration from %s: %w", cfgPath, err)
+				// TODO: add validation and distinguish between validation errors and other errors.
+				return config.StoreValidationErrorInContext(ctx, err), nil
 			}
 
 			return config.StoreInContext(ctx, loadedCfg), nil
@@ -114,8 +115,14 @@ func doApply(c context.Context, _ *cli.Command) error {
 		return fmt.Errorf("failed to retrieve config from context: %w", err)
 	}
 
-	if cfgCtx.NotFound {
+	switch cfgCtx.Status {
+	case config.StatusValid:
+		// No action needed.
+		// But we include it for use exhaustive linter.
+	case config.StatusNotFound:
 		return errors.New("apply command requires an existing config file")
+	case config.StatusValidationFailed:
+		return cfgCtx.ValidationError
 	}
 
 	cfg := cfgCtx.Config
@@ -145,8 +152,14 @@ func doClean(c context.Context, cmd *cli.Command) error {
 		return fmt.Errorf("failed to retrieve config from context: %w", err)
 	}
 
-	if cfgCtx.NotFound {
+	switch cfgCtx.Status {
+	case config.StatusValid:
+		// No action needed.
+		// But we include it for use exhaustive linter.
+	case config.StatusNotFound:
 		return errors.New("clean command requires an existing config file")
+	case config.StatusValidationFailed:
+		return cfgCtx.ValidationError
 	}
 
 	cfg := cfgCtx.Config

internal/config/context.go

@@ -8,26 +8,54 @@ import (
 type (
 	contextKey string
 
+	// Status represents the status of a configuration.
+	Status int
+
 	Context struct {
 		Config *Config
 
-		NotFound bool
+		// Status represents the state of the configuration.
+		Status Status
+
+		// ValidationError holds the validation error if Status is StatusValidationFailed.
+		ValidationError error
 	}
 )
 
 const (
 	configContextKey contextKey = "config"
 )
 
+const (
+	// StatusValid indicates the configuration is valid and ready to use.
+	StatusValid Status = iota
+
+	// StatusNotFound indicates the configuration file was not found.
+	StatusNotFound
+
+	// StatusValidationFailed indicates the configuration failed validation.
+	StatusValidationFailed
+
+	// Add more status types here as needed...
+)
+
 func StoreInContext(ctx context.Context, cfg *Config) context.Context {
 	return context.WithValue(ctx, configContextKey, &Context{
 		Config: cfg,
+		Status: StatusValid,
 	})
 }
 
 func StoreNotFoundInContext(ctx context.Context) context.Context {
 	return context.WithValue(ctx, configContextKey, &Context{
-		NotFound: true,
+		Status: StatusNotFound,
+	})
+}
+
+func StoreValidationErrorInContext(ctx context.Context, validationError error) context.Context {
+	return context.WithValue(ctx, configContextKey, &Context{
+		Status:          StatusValidationFailed,
+		ValidationError: validationError,
 	})
 }