fix: replace manual loop with copy() builtin

rubensantoniorosa2704 · rubensantoniorosa2704 · commit a99d993675a0 · 2025-10-31T21:14:33.000-03:00
Replaces manual slice copy loop with built-in copy() function
to fix S1001 linter warning: should use copy(to, from) instead of a loop.
diff --git a/internal/codegen/golang/result.go b/internal/codegen/golang/result.go
@@ -141,9 +141,7 @@ func newGoEmbed(embed *plugin.Identifier, structs []Struct, defaultSchema string
 		}
 
 		fields := make([]Field, len(s.Fields))
-		for i, f := range s.Fields {
-			fields[i] = f
-		}
+		copy(fields, s.Fields)
 
 		return &goEmbed{
 			modelType: s.Name,

Original file line number	Diff line number	Diff line change
`@@ -141,9 +141,7 @@ func newGoEmbed(embed *plugin.Identifier, structs []Struct, defaultSchema string`
`141`	`141`	`}`
`142`	`142`
`143`	`143`	`fields := make([]Field, len(s.Fields))`
`144`		`- for i, f := range s.Fields {`
`145`		`- fields[i] = f`
`146`		`- }`
	`144`	`+ copy(fields, s.Fields)`
`147`	`145`
`148`	`146`	`return &goEmbed{`
`149`	`147`	`modelType: s.Name,`