feat: optimize artisan check

config/application.go

@@ -20,8 +20,8 @@ type Application struct {
 
 func NewApplication(envPath string) *Application {
 	if !file.Exists(envPath) {
-		color.Redln("Please create .env and initialize it first.")
-		color.Warnln("Run command: \ncp .env.example .env && go run . artisan key:generate")
+		color.Redln("Please create " + envPath + " and initialize it first.")
+		color.Warnln("Example command: \ncp .env.example .env && go run . artisan key:generate")
 		os.Exit(0)
 	}
 
@@ -42,13 +42,13 @@ func NewApplication(envPath string) *Application {
 	if support.Env != support.EnvArtisan {
 		if appKey == nil {
 			color.Redln("Please initialize APP_KEY first.")
-			color.Warnln("Run command: \ngo run . artisan key:generate")
+			color.Warnln("Example command: \ngo run . artisan key:generate")
 			os.Exit(0)
 		}
 
 		if len(appKey.(string)) != 32 {
 			color.Redln("Invalid APP_KEY, please reset it.")
-			color.Warnln("Run command: \ngo run . artisan key:generate")
+			color.Warnln("Example command: \ngo run . artisan key:generate")
 			os.Exit(0)
 		}
 	}

console/application.go

@@ -42,36 +42,43 @@ func (c *Application) Register(commands []console.Command) {
 	}
 }
 
-//Call Run an Artisan console command by name.
+// Call Run an Artisan console command by name.
 func (c *Application) Call(command string) {
 	c.Run(append([]string{os.Args[0], "artisan"}, strings.Split(command, " ")...), false)
 }
 
-//CallAndExit Run an Artisan console command by name and exit.
+// CallAndExit Run an Artisan console command by name and exit.
 func (c *Application) CallAndExit(command string) {
 	c.Run(append([]string{os.Args[0], "artisan"}, strings.Split(command, " ")...), true)
 }
 
-//Run a command. Args come from os.Args.
+// Run a command. Args come from os.Args.
 func (c *Application) Run(args []string, exitIfArtisan bool) {
-	if len(args) >= 2 {
-		if args[1] == "artisan" {
-			if len(args) == 2 {
-				args = append(args, "--help")
-			}
+	artisanIndex := -1
+	for i, arg := range args {
+		if arg == "artisan" {
+			artisanIndex = i
+			break
+		}
+	}
 
-			if args[2] != "-V" && args[2] != "--version" {
-				cliArgs := append([]string{args[0]}, args[2:]...)
-				if err := c.instance.Run(cliArgs); err != nil {
-					panic(err.Error())
-				}
+	if artisanIndex != -1 {
+		// Add --help if no command argument is provided.
+		if artisanIndex+1 == len(args) {
+			args = append(args, "--help")
+		}
+
+		if args[artisanIndex+1] != "-V" && args[artisanIndex+1] != "--version" {
+			cliArgs := append([]string{args[0]}, args[artisanIndex+1:]...)
+			if err := c.instance.Run(cliArgs); err != nil {
+				panic(err.Error())
 			}
+		}
 
-			printResult(args[2])
+		printResult(args[artisanIndex+1])
 
-			if exitIfArtisan {
-				os.Exit(0)
-			}
+		if exitIfArtisan {
+			os.Exit(0)
 		}
 	}
 }

crypt/aes.go

@@ -7,12 +7,12 @@ import (
 	"encoding/base64"
 	"errors"
 	"io"
-	"os"
 
 	"github.com/bytedance/sonic"
 	"github.com/gookit/color"
 
 	"github.com/goravel/framework/contracts/config"
+	"github.com/goravel/framework/support"
 )
 
 type AES struct {
@@ -23,15 +23,14 @@ type AES struct {
 func NewAES(config config.Config) *AES {
 	key := config.GetString("app.key")
 
-	// Don't use AES in artisan key:generate command
-	args := os.Args
-	if len(args) >= 3 && args[1] == "artisan" && args[2] == "key:generate" {
+	// Don't use AES in artisan when the key is empty.
+	if support.Env == support.EnvArtisan && len(key) == 0 {
 		return nil
 	}
 
 	// check key length before using it
 	if len(key) != 16 && len(key) != 24 && len(key) != 32 {
-		color.Redln("[Crypt] Empty or invalid APP_KEY, please reset it.\nRun command:\ngo run . artisan key:generate")
+		color.Redln("[Crypt] Empty or invalid APP_KEY, please reset it.\nExample command:\ngo run . artisan key:generate")
 		return nil
 	}
 	keyBytes := []byte(key)

foundation/application.go

@@ -175,8 +175,11 @@ func setEnv() {
 		support.Env = support.EnvTest
 	}
 	if len(args) >= 2 {
-		if args[1] == "artisan" {
-			support.Env = support.EnvArtisan
+		for _, arg := range args[1:] {
+			if arg == "artisan" {
+				support.Env = support.EnvArtisan
+				break
+			}
 		}
 	}
 

route/gin.go

@@ -12,6 +12,7 @@ import (
 	httpcontract "github.com/goravel/framework/contracts/http"
 	"github.com/goravel/framework/contracts/route"
 	goravelhttp "github.com/goravel/framework/http"
+	"github.com/goravel/framework/support"
 )
 
 type Gin struct {
@@ -120,7 +121,7 @@ func (r *Gin) ServeHTTP(writer http.ResponseWriter, request *http.Request) {
 }
 
 func (r *Gin) outputRoutes() {
-	if r.config.GetBool("app.debug") && !runningInConsole() {
+	if r.config.GetBool("app.debug") && support.Env != support.EnvArtisan {
 		for _, item := range r.instance.Routes() {
 			fmt.Printf("%-10s %s\n", item.Method, colonToBracket(item.Path))
 		}

route/utils.go

@@ -2,7 +2,6 @@ package route
 
 import (
 	"fmt"
-	"os"
 	"regexp"
 	"strings"
 	"time"
@@ -99,9 +98,3 @@ func mergeSlashForPath(path string) string {
 
 	return strings.ReplaceAll(path, "//", "/")
 }
-
-func runningInConsole() bool {
-	args := os.Args
-
-	return len(args) >= 2 && args[1] == "artisan"
-}

support/file/file.go

@@ -50,9 +50,10 @@ func Create(file string, content string) error {
 
 func Exists(file string) bool {
 	_, err := os.Stat(file)
-	if err != nil {
-		return os.IsExist(err)
+	if os.IsNotExist(err) {
+		return false
 	}
+
 	return true
 }