feat: [#653] Update github.com/urfave/cli/v2 to v3

[almas1992]

📑 Description

Closes goravel/goravel#653

✅ Checks

• Added test cases for my code

[Copilot]

Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.

Files not reviewed (1)

• go.mod: Language not supported

Comments suppressed due to low confidence (1)

console/cli_context.go:218

• Verify that r.instance.Int(key) returns an int64 value as expected in urfave/cli v3; if it returns an int, consider applying an explicit conversion to int64 for consistency in OptionInt64.

return r.instance.Int(key)

[codecov]

Codecov Report

Attention: Patch coverage is 87.03704% with 14 lines in your changes missing coverage. Please review.

Project coverage is 70.18%. Comparing base (9e85faf) to head (b98c632).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
console/cli_context.go	8.33%	11 Missing ⚠️
support/color/color.go	0.00%	2 Missing ⚠️
console/application.go	96.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #994      +/-   ##
==========================================
+ Coverage   70.04%   70.18%   +0.14%     
==========================================
  Files         168      168              
  Lines       11353    11428      +75     
==========================================
+ Hits         7952     8021      +69     
- Misses       3050     3056       +6     
  Partials      351      351              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[hwbrzzl]

The cli code is weird: IntSliceFlag vs []int64

[almas1992]

Yeah, it’s definitely weird. In v3, the original int was replaced with int64, so the flag no longer returns an int. I added a conversion in console/application.go to preserve the original behavior.

[devhaozi]

I think Int* can removed, only keep Int64*.

[hwbrzzl]

Yes, case command.FlagTypeInt and case command.FlagTypeInt64 can use the same flow (in our framework, still distinguish FlagTypeInt and FlagTypeInt64). The same with slice.

[almas1992]

Improved the flag sorting logic based on the feedback in this comment, ensuring consistent output order.

[almas1992]

I’ve added a new function:

func handleNoANSI() {
	if noANSI || env.IsNoANSI() {
		color.Disable()
	} else {
		color.Enable()
	}
}

This ensures that the --no-ansi option doesn’t have side effects between commands.
For example, in the following case:

type Test struct {
}

// Signature The name and signature of the console command.
func (r *Test) Signature() string {
	return "app:test"
}

// Description The console command description.
func (r *Test) Description() string {
	return "Command description"
}

// Extend The console command extend.
func (r *Test) Extend() command.Extend {
	return command.Extend{Category: "app"}
}

// Handle Execute the console command.
func (r *Test) Handle(ctx console.Context) error {
	artisan := facades.Artisan()
	_ = artisan.Call("help --no-ansi build")
	ctx.NewLine()
	_ = artisan.Call("help migrate")
	return nil
}

Result:

[hwbrzzl]

Thanks, LGTM