chore: modernize code to Go 1.26 idioms

ai/conversation.go

@@ -411,8 +411,7 @@ func (r *conversation) runMiddlewarePipeline(ctx context.Context, prompt contrac
 	next := destination
 	activeMiddlewares := slices.Clone(r.middlewares)
 
-	for i := len(activeMiddlewares) - 1; i >= 0; i-- {
-		middleware := activeMiddlewares[i]
+	for _, middleware := range slices.Backward(activeMiddlewares) {
 		nextHandler := next
 		next = contractsai.Next(func(ctx context.Context, prompt contractsai.AgentPrompt) (contractsai.AgentResponse, error) {
 			return middleware.Handle(ctx, prompt, nextHandler)

ai/failover.go

@@ -3,7 +3,7 @@ package ai
 import (
 	"context"
 	"regexp"
-	"sort"
+	"slices"
 	"strings"
 
 	contractsai "github.com/goravel/framework/contracts/ai"
@@ -53,7 +53,7 @@ func newFailoverProvider(providers []resolvedProvider) contractsai.Provider {
 		return providers[0].provider
 	}
 
-	return &failoverProvider{providers: append([]resolvedProvider(nil), providers...)}
+	return &failoverProvider{providers: slices.Clone(providers)}
 }
 
 func (e *failoverError) Error() string {
@@ -215,7 +215,7 @@ func NewFailoverRules(provider string, patterns map[contractsai.FailoverReason][
 			reasons = append(reasons, string(reason))
 		}
 	}
-	sort.Strings(reasons)
+	slices.Sort(reasons)
 
 	var rules []failoverRule
 	for _, reasonValue := range reasons {

ai/media_storage.go

@@ -3,6 +3,7 @@ package ai
 import (
 	pathpkg "path"
 	"regexp"
+	"slices"
 	"strings"
 
 	contractsfilesystem "github.com/goravel/framework/contracts/filesystem"
@@ -69,13 +70,7 @@ func storeContent(content []byte, name, path, disk string, nameRequired error) (
 }
 
 func hasParentPathSegment(path string) bool {
-	for _, segment := range strings.Split(path, "/") {
-		if segment == ".." {
-			return true
-		}
-	}
-
-	return false
+	return slices.Contains(strings.Split(path, "/"), "..")
 }
 
 func isAbsoluteStoragePath(path string) bool {

ai/response.go

@@ -3,6 +3,7 @@ package ai
 import (
 	"bytes"
 	"context"
+	"slices"
 	"strings"
 
 	contractsai "github.com/goravel/framework/contracts/ai"
@@ -251,5 +252,5 @@ func cloneTranscriptionSegments(segments []contractsai.TranscriptionSegment) []c
 		return nil
 	}
 
-	return append([]contractsai.TranscriptionSegment(nil), segments...)
+	return slices.Clone(segments)
 }

cache/memory_test.go

@@ -78,14 +78,12 @@ func (s *MemoryTestSuite) TestDecrementWithConcurrent() {
 		failed atomic.Int64
 	)
 	for range 1000 {
-		wg.Add(1)
-		go func() {
-			defer wg.Done()
+		wg.Go(func() {
 			// Assertions belong on the test goroutine, and err is shared.
 			if _, err := s.memory.Decrement("decrement_concurrent", 1); err != nil {
 				failed.Add(1)
 			}
-		}()
+		})
 	}
 
 	wg.Wait()
@@ -126,22 +124,18 @@ func (s *MemoryTestSuite) TestFlushWhileReading() {
 
 	var wg sync.WaitGroup
 	for range 50 {
-		wg.Add(1)
-		go func() {
-			defer wg.Done()
+		wg.Go(func() {
 			for range 500 {
 				s.memory.Get("test-flush-while-reading")
 			}
-		}()
+		})
 	}
 	for range 10 {
-		wg.Add(1)
-		go func() {
-			defer wg.Done()
+		wg.Go(func() {
 			for range 200 {
 				s.memory.Flush()
 			}
-		}()
+		})
 	}
 	wg.Wait()
 
@@ -229,17 +223,15 @@ func (s *MemoryTestSuite) TestAddWithConcurrent() {
 			start = make(chan struct{})
 		)
 		for range 8 {
-			wg.Add(1)
-			go func() {
-				defer wg.Done()
+			wg.Go(func() {
 				// Contend on the same expired item: a goroutine released after
 				// a winner has replaced it leaves through the "not expired"
 				// branch, which proves nothing about the swap.
 				<-start
 				if s.memory.Add(key, "goravel", NoExpiration) {
 					added.Add(1)
 				}
-			}()
+			})
 		}
 		close(start)
 		wg.Wait()
@@ -383,14 +375,12 @@ func (s *MemoryTestSuite) TestIncrementWithConcurrent() {
 		failed atomic.Int64
 	)
 	for range 1000 {
-		wg.Add(1)
-		go func() {
-			defer wg.Done()
+		wg.Go(func() {
 			// Assertions belong on the test goroutine, and err is shared.
 			if _, err := s.memory.Increment("increment_concurrent", 1); err != nil {
 				failed.Add(1)
 			}
-		}()
+		})
 	}
 
 	wg.Wait()

console/application_test.go

@@ -44,12 +44,10 @@ func TestRun_Concurrent(t *testing.T) {
 
 	var wg sync.WaitGroup
 	for i := 0; i < 100; i++ {
-		wg.Add(1)
-		go func() {
+		wg.Go(func() {
 			_ = cliApp.Call("test1")
 			_ = cliApp.Call("test2")
-			wg.Done()
-		}()
+		})
 	}
 
 	wg.Wait()
@@ -698,7 +696,7 @@ type filterTestCommand struct {
 	signature string
 }
 
-func (f *filterTestCommand) Signature() string  { return f.signature }
+func (f *filterTestCommand) Signature() string   { return f.signature }
 func (f *filterTestCommand) Description() string { return f.signature }
 func (f *filterTestCommand) Extend() command.Extend {
 	return command.Extend{}

console/cli_context.go

@@ -204,7 +204,7 @@ func (r *CliContext) NewLine(times ...int) {
 	if len(times) > 0 && times[0] > 0 {
 		numLines = times[0]
 	}
-	for i := 0; i < numLines; i++ {
+	for range numLines {
 		color.Default().Println()
 	}
 }

console/cli_context_test.go

@@ -16,7 +16,6 @@ import (
 	"github.com/goravel/framework/contracts/console"
 	"github.com/goravel/framework/contracts/console/command"
 	"github.com/goravel/framework/support/color"
-	"github.com/goravel/framework/support/convert"
 )
 
 func TestArgumentString(t *testing.T) {
@@ -2094,10 +2093,10 @@ func TestTable(t *testing.T) {
 		{
 			name: "compact style using boolean flags",
 			option: console.TableOption{
-				BorderTop:    convert.Pointer(false),
-				BorderBottom: convert.Pointer(false),
-				BorderLeft:   convert.Pointer(false),
-				BorderRight:  convert.Pointer(false),
+				BorderTop:    new(false),
+				BorderBottom: new(false),
+				BorderLeft:   new(false),
+				BorderRight:  new(false),
 			},
 			contains: []string{"ID", "Name", "│"},
 		},

console/cli_helper.go

@@ -1,10 +1,12 @@
 package console
 
 import (
+	"cmp"
 	"context"
 	"fmt"
 	"io"
 	"os"
+	"slices"
 	"sort"
 	"strings"
 	"text/tabwriter"
@@ -162,11 +164,8 @@ func findAlternatives(name string, collection []string) (result []string) {
 			sortedAlternatives = append(sortedAlternatives, scoredItem{item, score})
 		}
 	}
-	sort.Slice(sortedAlternatives, func(i, j int) bool {
-		if sortedAlternatives[i].score == sortedAlternatives[j].score {
-			return sortedAlternatives[i].name < sortedAlternatives[j].name
-		}
-		return sortedAlternatives[i].score < sortedAlternatives[j].score
+	slices.SortFunc(sortedAlternatives, func(a, b scoredItem) int {
+		return cmp.Or(cmp.Compare(a.score, b.score), strings.Compare(a.name, b.name))
 	})
 	for _, item := range sortedAlternatives {
 		result = append(result, item.name)
@@ -186,8 +185,8 @@ func getFlagDefaultText(flag cli.DocGenerationFlag) string {
 
 func getFlagName(flag cli.Flag) string {
 	names := flag.Names()
-	sort.Slice(names, func(i, j int) bool {
-		return len(names[i]) < len(names[j])
+	slices.SortFunc(names, func(a, b string) int {
+		return cmp.Compare(len(a), len(b))
 	})
 	prefixed := cli.FlagNamePrefixer(names, "")
 	// If there is no short name, add some padding to align flag name.
@@ -384,10 +383,17 @@ func sortVisibleFlags(cmd *cli.Command) []cli.Flag {
 	for i := range globalFlags {
 		globalFlagNames[getFlagName(globalFlags[i])] = struct{}{}
 	}
-	sort.Slice(flags, func(i, j int) bool {
-		_, isGlobalI := globalFlagNames[getFlagName(flags[i])]
-		_, isGlobalJ := globalFlagNames[getFlagName(flags[j])]
-		return !isGlobalI && isGlobalJ
+	// Stable sort keeps the by-name order from above inside each group.
+	slices.SortStableFunc(flags, func(a, b cli.Flag) int {
+		_, isGlobalA := globalFlagNames[getFlagName(a)]
+		_, isGlobalB := globalFlagNames[getFlagName(b)]
+		switch {
+		case !isGlobalA && isGlobalB:
+			return -1
+		case isGlobalA && !isGlobalB:
+			return 1
+		}
+		return 0
 	})
 
 	return flags

console/service_provider_test.go

@@ -1,7 +1,7 @@
 package console
 
 import (
-	"sort"
+	"slices"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -50,7 +50,7 @@ func signaturesOf(commands []contractsconsole.Command) []string {
 	for _, c := range commands {
 		sigs = append(sigs, c.Signature())
 	}
-	sort.Strings(sigs)
+	slices.Sort(sigs)
 	return sigs
 }
 
@@ -107,7 +107,7 @@ func TestServiceProviderBootAllowlistApplies(t *testing.T) {
 	artisan.EXPECT().Register(mock.MatchedBy(func(commands []contractsconsole.Command) bool {
 		got := signaturesOf(commands)
 		want := []string{"list", "make:command"}
-		sort.Strings(want)
+		slices.Sort(want)
 		if len(got) != len(want) {
 			return false
 		}

console/style.go

@@ -7,7 +7,6 @@ import (
 	"github.com/pterm/pterm"
 
 	"github.com/goravel/framework/contracts/console"
-	"github.com/goravel/framework/support/convert"
 )
 
 var (
@@ -40,13 +39,13 @@ var (
 
 		StyleFunc: DefaultTableStyleFunc,
 
-		BorderTop:    convert.Pointer(true),
-		BorderBottom: convert.Pointer(true),
-		BorderLeft:   convert.Pointer(true),
-		BorderRight:  convert.Pointer(true),
-		BorderHeader: convert.Pointer(true),
-		BorderColumn: convert.Pointer(true),
-		BorderRow:    convert.Pointer(false),
+		BorderTop:    new(true),
+		BorderBottom: new(true),
+		BorderLeft:   new(true),
+		BorderRight:  new(true),
+		BorderHeader: new(true),
+		BorderColumn: new(true),
+		BorderRow:    new(false),
 	}
 
 	GlobalHuhTheme = func() *huh.Theme {

contracts/translation/translator.go

@@ -27,6 +27,9 @@ type Option struct {
 	Locale   string
 }
 
+// Bool returns a pointer to the value.
+//
+// Deprecated: Use new(value) instead, Bool will be removed in a future version.
 func Bool(value bool) *bool {
 	return &value
 }

crypt/aes_test.go

@@ -79,9 +79,7 @@ func Benchmark_EncryptString(b *testing.B) {
 		b.Fatal(err)
 	}
 
-	b.ResetTimer()
-
-	for i := 0; i < b.N; i++ {
+	for b.Loop() {
 		_, err := aes.EncryptString("Goravel")
 		if err != nil {
 			b.Error(err)
@@ -101,9 +99,8 @@ func Benchmark_DecryptString(b *testing.B) {
 	if err != nil {
 		b.Error(err)
 	}
-	b.ResetTimer()
 
-	for i := 0; i < b.N; i++ {
+	for b.Loop() {
 		_, err := aes.DecryptString(payload)
 		if err != nil {
 			b.Error(err)