simplify isUrlKey

kerobbi · kerobbi · commit c960f498da70 · 2026-02-18T17:04:45.000Z
diff --git a/pkg/response/optimize.go b/pkg/response/optimize.go
@@ -274,11 +274,10 @@ func trimArrayFields(items []any, fields []string) []any {
 
 // isURLKey matches "url", "*_url", and their dot-prefixed variants.
 func isURLKey(key string) bool {
-	base := key
-	if idx := strings.LastIndex(base, "."); idx >= 0 {
-		base = base[idx+1:]
+	if idx := strings.LastIndex(key, "."); idx >= 0 {
+		key = key[idx+1:]
 	}
-	return base == "url" || strings.HasSuffix(base, "_url")
+	return key == "url" || strings.HasSuffix(key, "_url")
 }
 
 func isZeroValue(v any) bool {

Original file line number	Diff line number	Diff line change
`@@ -274,11 +274,10 @@ func trimArrayFields(items []any, fields []string) []any {`
`274`	`274`
`275`	`275`	`// isURLKey matches "url", "*_url", and their dot-prefixed variants.`
`276`	`276`	`func isURLKey(key string) bool {`
`277`		`- base := key`
`278`		`- if idx := strings.LastIndex(base, "."); idx >= 0 {`
`279`		`- base = base[idx+1:]`
	`277`	`+ if idx := strings.LastIndex(key, "."); idx >= 0 {`
	`278`	`+ key = key[idx+1:]`
`280`	`279`	`}`
`281`		`- return base == "url" \|\| strings.HasSuffix(base, "_url")`
	`280`	`+ return key == "url" \|\| strings.HasSuffix(key, "_url")`
`282`	`281`	`}`
`283`	`282`
`284`	`283`	`func isZeroValue(v any) bool {`