clean up string builders

silverwind · silverwind · commit 3408302589eb · 2025-11-01T11:40:36.000+01:00
diff --git a/modules/auth/password/password.go b/modules/auth/password/password.go
@@ -61,25 +61,23 @@ func NewComplexity() {
 }
 
 func setupComplexity(values []string) {
+	var validCharsSb strings.Builder
 	if len(values) != 1 || values[0] != "off" {
-		var validCharsSb strings.Builder
 		for _, val := range values {
 			if complexity, ok := charComplexities[val]; ok {
 				validCharsSb.WriteString(complexity.ValidChars)
 				requiredList = append(requiredList, complexity)
 			}
 		}
-		validChars += validCharsSb.String()
 		if len(requiredList) == 0 {
 			// No valid character classes found; use all classes as default
-			var validCharsSb strings.Builder
 			for _, complexity := range charComplexities {
 				validCharsSb.WriteString(complexity.ValidChars)
 				requiredList = append(requiredList, complexity)
 			}
-			validChars += validCharsSb.String()
 		}
 	}
+	validChars = validCharsSb.String()
 	if validChars == "" {
 		// No complexities to check; provide a sensible default for password generation
 		validChars = charComplexities["lower"].ValidChars + charComplexities["upper"].ValidChars + charComplexities["digit"].ValidChars
diff --git a/modules/git/foreachref/format.go b/modules/git/foreachref/format.go
@@ -75,11 +75,9 @@ func (f Format) Parser(r io.Reader) *Parser {
 // hexEscaped produces hex-escpaed characters from a string. For example, "\n\0"
 // would turn into "%0a%00".
 func (f Format) hexEscaped(delim []byte) string {
-	escaped := ""
 	var escapedSb strings.Builder
 	for i := range delim {
 		escapedSb.WriteString("%" + hex.EncodeToString([]byte{delim[i]}))
 	}
-	escaped += escapedSb.String()
-	return escaped
+	return escapedSb.String()
 }
diff --git a/modules/git/notes_nogogit.go b/modules/git/notes_nogogit.go
@@ -26,8 +26,6 @@ func GetNote(ctx context.Context, repo *Repository, commitID string, note *Note)
 		return err
 	}
 
-	path := ""
-
 	tree := &notes.Tree
 	log.Trace("Found tree with ID %q while searching for git note corresponding to the commit %q", tree.ID, commitID)
 
@@ -53,7 +51,7 @@ func GetNote(ctx context.Context, repo *Repository, commitID string, note *Note)
 			return err
 		}
 	}
-	path += pathSb.String()
+	path := pathSb.String()
 
 	blob := entry.Blob()
 	dataRc, err := blob.DataAsync()
diff --git a/modules/setting/config.go b/modules/setting/config.go
@@ -24,13 +24,11 @@ type OpenWithEditorApp struct {
 type OpenWithEditorAppsType []OpenWithEditorApp
 
 func (t OpenWithEditorAppsType) ToTextareaString() string {
-	ret := ""
 	var retSb strings.Builder
 	for _, app := range t {
 		retSb.WriteString(app.DisplayName + " = " + app.OpenURL + "\n")
 	}
-	ret += retSb.String()
-	return ret
+	return retSb.String()
 }
 
 func DefaultOpenWithEditorApps() OpenWithEditorAppsType {
diff --git a/modules/setting/config_env.go b/modules/setting/config_env.go
@@ -77,8 +77,8 @@ func decodeEnvSectionKey(encoded string) (ok bool, section, key string) {
 		}
 		last = unescapeIdx[1]
 	}
-	key += keySb.String()
-	section += sectionSb.String()
+	key = keySb.String()
+	section = sectionSb.String()
 	remaining := encoded[last:]
 	if !inKey {
 		if splitter := strings.Index(remaining, "__"); splitter > -1 {
diff --git a/modules/templates/util_render.go b/modules/templates/util_render.go
@@ -249,8 +249,8 @@ func (ut *RenderUtils) MarkdownToHtml(input string) template.HTML { //nolint:rev
 func (ut *RenderUtils) RenderLabels(labels []*issues_model.Label, repoLink string, issue *issues_model.Issue) template.HTML {
 	isPullRequest := issue != nil && issue.IsPull
 	baseLink := fmt.Sprintf("%s/%s", repoLink, util.Iif(isPullRequest, "pulls", "issues"))
-	htmlCode := `<span class="labels-list">`
 	var htmlCodeSb strings.Builder
+	htmlCodeSb.WriteString(`<span class="labels-list">`)
 	for _, label := range labels {
 		// Protect against nil value in labels - shouldn't happen but would cause a panic if so
 		if label == nil {
@@ -259,9 +259,8 @@ func (ut *RenderUtils) RenderLabels(labels []*issues_model.Label, repoLink strin
 		link := fmt.Sprintf("%s?labels=%d", baseLink, label.ID)
 		htmlCodeSb.WriteString(string(ut.RenderLabelWithLink(label, template.URL(link))))
 	}
-	htmlCode += htmlCodeSb.String()
-	htmlCode += "</span>"
-	return template.HTML(htmlCode)
+	htmlCodeSb.WriteString("</span>")
+	return template.HTML(htmlCodeSb.String())
 }
 
 func (ut *RenderUtils) RenderThemeItem(info *webtheme.ThemeMetaInfo, iconSize int) template.HTML {
diff --git a/modules/timeutil/since.go b/modules/timeutil/since.go
@@ -95,7 +95,7 @@ func timeSincePro(then, now time.Time, lang translation.Locale) string {
 		return lang.TrString("tool.now")
 	}
 
-	var timeStr, diffStr string
+	var diffStr string
 	var timeStrSb strings.Builder
 	for {
 		if diff == 0 {
@@ -105,6 +105,5 @@ func timeSincePro(then, now time.Time, lang translation.Locale) string {
 		diff, diffStr = computeTimeDiffFloor(diff, lang)
 		timeStrSb.WriteString(", " + diffStr)
 	}
-	timeStr += timeStrSb.String()
-	return strings.TrimPrefix(timeStr, ", ")
+	return strings.TrimPrefix(timeStrSb.String(), ", ")
 }
diff --git a/routers/api/packages/rubygems/rubygems.go b/routers/api/packages/rubygems/rubygems.go
@@ -433,17 +433,16 @@ func makePackageVersionDependency(ctx *context.Context, version *packages_model.
 }
 
 func makePackageInfo(ctx *context.Context, versions []*packages_model.PackageVersion, c *cache.EphemeralCache) (string, error) {
-	ret := "---\n"
 	var retSb strings.Builder
+	retSb.WriteString("---\n")
 	for _, v := range versions {
 		dep, err := makePackageVersionDependency(ctx, v, c)
 		if err != nil {
 			return "", err
 		}
 		retSb.WriteString(dep + "\n")
 	}
-	ret += retSb.String()
-	return ret, nil
+	return retSb.String(), nil
 }
 
 func makeGemFullFileName(gemName, version, platform string) string {
diff --git a/routers/web/auth/oauth2_provider.go b/routers/web/auth/oauth2_provider.go
@@ -179,13 +179,11 @@ func AuthorizeOAuth(ctx *context.Context) {
 	errs := binding.Errors{}
 	errs = form.Validate(ctx.Req, errs)
 	if len(errs) > 0 {
-		errstring := ""
 		var errstringSb strings.Builder
 		for _, e := range errs {
 			errstringSb.WriteString(e.Error() + "\n")
 		}
-		errstring += errstringSb.String()
-		ctx.ServerError("AuthorizeOAuth: Validate: ", fmt.Errorf("errors occurred during validation: %s", errstring))
+		ctx.ServerError("AuthorizeOAuth: Validate: ", fmt.Errorf("errors occurred during validation: %s", errstringSb.String()))
 		return
 	}
 
diff --git a/routers/web/user/setting/keys.go b/routers/web/user/setting/keys.go
@@ -132,13 +132,12 @@ func KeysPost(ctx *context.Context) {
 			}
 			return
 		}
-		keyIDs := ""
 		var keyIDsSb strings.Builder
 		for _, key := range keys {
 			keyIDsSb.WriteString(key.KeyID)
 			keyIDsSb.WriteString(", ")
 		}
-		keyIDs += keyIDsSb.String()
+		keyIDs := keyIDsSb.String()
 		if len(keyIDs) > 0 {
 			keyIDs = keyIDs[:len(keyIDs)-2]
 		}
diff --git a/services/webhook/dingtalk.go b/services/webhook/dingtalk.go
@@ -72,7 +72,6 @@ func (dc dingtalkConvertor) Push(p *api.PushPayload) (DingtalkPayload, error) {
 
 	title := fmt.Sprintf("[%s:%s] %s", p.Repo.FullName, branchName, commitDesc)
 
-	var text string
 	// for each commit, generate attachment text
 	var textSb strings.Builder
 	for i, commit := range p.Commits {
@@ -87,9 +86,8 @@ func (dc dingtalkConvertor) Push(p *api.PushPayload) (DingtalkPayload, error) {
 			textSb.WriteString("\r\n")
 		}
 	}
-	text += textSb.String()
 
-	return createDingtalkPayload(title, text, linkText, titleLink), nil
+	return createDingtalkPayload(title, textSb.String(), linkText, titleLink), nil
 }
 
 // Issue implements PayloadConvertor Issue method
diff --git a/services/webhook/discord.go b/services/webhook/discord.go
@@ -159,7 +159,6 @@ func (d discordConvertor) Push(p *api.PushPayload) (DiscordPayload, error) {
 
 	title := fmt.Sprintf("[%s:%s] %s", p.Repo.FullName, branchName, commitDesc)
 
-	var text string
 	// for each commit, generate attachment text
 	var textSb strings.Builder
 	for i, commit := range p.Commits {
@@ -176,9 +175,8 @@ func (d discordConvertor) Push(p *api.PushPayload) (DiscordPayload, error) {
 			textSb.WriteString("\n")
 		}
 	}
-	text += textSb.String()
 
-	return d.createPayload(p.Sender, title, text, titleLink, greenColor), nil
+	return d.createPayload(p.Sender, title, textSb.String(), titleLink, greenColor), nil
 }
 
 // Issue implements PayloadConvertor Issue method
diff --git a/services/webhook/feishu.go b/services/webhook/feishu.go
@@ -76,9 +76,10 @@ func (fc feishuConvertor) Push(p *api.PushPayload) (FeishuPayload, error) {
 		commitDesc string
 	)
 
-	text := fmt.Sprintf("[%s:%s] %s\r\n", p.Repo.FullName, branchName, commitDesc)
-	// for each commit, generate attachment text
 	var textSb strings.Builder
+	textSb.WriteString(fmt.Sprintf("[%s:%s] %s\r\n", p.Repo.FullName, branchName, commitDesc))
+
+	// for each commit, generate attachment text
 	for i, commit := range p.Commits {
 		var authorName string
 		if commit.Author != nil {
@@ -91,9 +92,8 @@ func (fc feishuConvertor) Push(p *api.PushPayload) (FeishuPayload, error) {
 			textSb.WriteString("\r\n")
 		}
 	}
-	text += textSb.String()
 
-	return newFeishuTextPayload(text), nil
+	return newFeishuTextPayload(textSb.String()), nil
 }
 
 // Issue implements PayloadConvertor Issue method
diff --git a/services/webhook/matrix.go b/services/webhook/matrix.go
@@ -173,20 +173,20 @@ func (m matrixConvertor) Push(p *api.PushPayload) (MatrixPayload, error) {
 
 	repoLink := htmlLinkFormatter(p.Repo.HTMLURL, p.Repo.FullName)
 	branchLink := MatrixLinkToRef(p.Repo.HTMLURL, p.Ref)
-	text := fmt.Sprintf("[%s] %s pushed %s to %s:<br>", repoLink, p.Pusher.UserName, commitDesc, branchLink)
 
-	// for each commit, generate a new line text
 	var textSb strings.Builder
+	textSb.WriteString(fmt.Sprintf("[%s] %s pushed %s to %s:<br>", repoLink, p.Pusher.UserName, commitDesc, branchLink))
+
+	// for each commit, generate a new line text
 	for i, commit := range p.Commits {
 		textSb.WriteString(fmt.Sprintf("%s: %s - %s", htmlLinkFormatter(commit.URL, commit.ID[:7]), commit.Message, commit.Author.Name))
 		// add linebreak to each commit but the last
 		if i < len(p.Commits)-1 {
 			textSb.WriteString("<br>")
 		}
 	}
-	text += textSb.String()
 
-	return m.newPayload(text, p.Commits...)
+	return m.newPayload(textSb.String(), p.Commits...)
 }
 
 // PullRequest implements payloadConvertor PullRequest method
diff --git a/services/webhook/msteams.go b/services/webhook/msteams.go
@@ -131,7 +131,6 @@ func (m msteamsConvertor) Push(p *api.PushPayload) (MSTeamsPayload, error) {
 
 	title := fmt.Sprintf("[%s:%s] %s", p.Repo.FullName, branchName, commitDesc)
 
-	var text string
 	// for each commit, generate attachment text
 	var textSb strings.Builder
 	for i, commit := range p.Commits {
@@ -142,13 +141,12 @@ func (m msteamsConvertor) Push(p *api.PushPayload) (MSTeamsPayload, error) {
 			textSb.WriteString("\n\n")
 		}
 	}
-	text += textSb.String()
 
 	return createMSTeamsPayload(
 		p.Repo,
 		p.Sender,
 		title,
-		text,
+		textSb.String(),
 		titleLink,
 		greenColor,
 		&MSTeamsFact{"Commit count:", strconv.Itoa(p.TotalCommits)},
diff --git a/services/webhook/slack.go b/services/webhook/slack.go
@@ -208,7 +208,6 @@ func (s slackConvertor) Push(p *api.PushPayload) (SlackPayload, error) {
 	branchLink := SlackLinkToRef(p.Repo.HTMLURL, p.Ref)
 	text := fmt.Sprintf("[%s:%s] %s pushed by %s", repoLink, branchLink, commitString, p.Pusher.UserName)
 
-	var attachmentText string
 	// for each commit, generate attachment text
 	var attachmentTextSb strings.Builder
 	for i, commit := range p.Commits {
@@ -218,13 +217,12 @@ func (s slackConvertor) Push(p *api.PushPayload) (SlackPayload, error) {
 			attachmentTextSb.WriteString("\n")
 		}
 	}
-	attachmentText += attachmentTextSb.String()
 
 	return s.createPayload(text, []SlackAttachment{{
 		Color:     s.Color,
 		Title:     p.Repo.HTMLURL,
 		TitleLink: p.Repo.HTMLURL,
-		Text:      attachmentText,
+		Text:      attachmentTextSb.String(),
 	}}), nil
 }
 
diff --git a/services/webhook/telegram.go b/services/webhook/telegram.go
@@ -94,16 +94,14 @@ func (t telegramConvertor) Push(p *api.PushPayload) (TelegramPayload, error) {
 	}
 	title := fmt.Sprintf(`[%s:%s] %s`, htmlLinkFormatter(p.Repo.HTMLURL, p.Repo.FullName), htmlLinkFormatter(titleLink, branchName), html.EscapeString(commitDesc))
 
-	var htmlCommits string
 	var htmlCommitsSb strings.Builder
 	for _, commit := range p.Commits {
 		htmlCommitsSb.WriteString(fmt.Sprintf("\n[%s] %s", htmlLinkFormatter(commit.URL, commit.ID[:7]), html.EscapeString(strings.TrimRight(commit.Message, "\r\n"))))
 		if commit.Author != nil {
 			htmlCommitsSb.WriteString(" - " + html.EscapeString(commit.Author.Name))
 		}
 	}
-	htmlCommits += htmlCommitsSb.String()
-	return createTelegramPayloadHTML(title + htmlCommits), nil
+	return createTelegramPayloadHTML(title + htmlCommitsSb.String()), nil
 }
 
 // Issue implements PayloadConvertor Issue method
diff --git a/services/webhook/wechatwork.go b/services/webhook/wechatwork.go
@@ -77,7 +77,6 @@ func (wc wechatworkConvertor) Push(p *api.PushPayload) (WechatworkPayload, error
 
 	title := fmt.Sprintf("# %s:%s <font color=\"warning\">  %s  </font>", p.Repo.FullName, branchName, commitDesc)
 
-	var text string
 	// for each commit, generate attachment text
 	var textSb strings.Builder
 	for i, commit := range p.Commits {
@@ -95,8 +94,7 @@ func (wc wechatworkConvertor) Push(p *api.PushPayload) (WechatworkPayload, error
 			textSb.WriteString("\n")
 		}
 	}
-	text += textSb.String()
-	return newWechatworkMarkdownPayload(title + "\r\n\r\n" + text), nil
+	return newWechatworkMarkdownPayload(title + "\r\n\r\n" + textSb.String()), nil
 }
 
 // Issue implements PayloadConvertor Issue method

Original file line number	Diff line number	Diff line change
`@@ -61,25 +61,23 @@ func NewComplexity() {`
`61`	`61`	`}`
`62`	`62`
`63`	`63`	`func setupComplexity(values []string) {`
	`64`	`+ var validCharsSb strings.Builder`
`64`	`65`	`if len(values) != 1 \|\| values[0] != "off" {`
`65`		`- var validCharsSb strings.Builder`
`66`	`66`	`for _, val := range values {`
`67`	`67`	`if complexity, ok := charComplexities[val]; ok {`
`68`	`68`	`validCharsSb.WriteString(complexity.ValidChars)`
`69`	`69`	`requiredList = append(requiredList, complexity)`
`70`	`70`	`}`
`71`	`71`	`}`
`72`		`- validChars += validCharsSb.String()`
`73`	`72`	`if len(requiredList) == 0 {`
`74`	`73`	`// No valid character classes found; use all classes as default`
`75`		`- var validCharsSb strings.Builder`
`76`	`74`	`for _, complexity := range charComplexities {`
`77`	`75`	`validCharsSb.WriteString(complexity.ValidChars)`
`78`	`76`	`requiredList = append(requiredList, complexity)`
`79`	`77`	`}`
`80`		`- validChars += validCharsSb.String()`
`81`	`78`	`}`
`82`	`79`	`}`
	`80`	`+ validChars = validCharsSb.String()`
`83`	`81`	`if validChars == "" {`
`84`	`82`	`// No complexities to check; provide a sensible default for password generation`
`85`	`83`	`validChars = charComplexities["lower"].ValidChars + charComplexities["upper"].ValidChars + charComplexities["digit"].ValidChars`
Original file line number	Diff line number	Diff line change
`@@ -75,11 +75,9 @@ func (f Format) Parser(r io.Reader) *Parser {`
`75`	`75`	`// hexEscaped produces hex-escpaed characters from a string. For example, "\n\0"`
`76`	`76`	`// would turn into "%0a%00".`
`77`	`77`	`func (f Format) hexEscaped(delim []byte) string {`
`78`		`- escaped := ""`
`79`	`78`	`var escapedSb strings.Builder`
`80`	`79`	`for i := range delim {`
`81`	`80`	`escapedSb.WriteString("%" + hex.EncodeToString([]byte{delim[i]}))`
`82`	`81`	`}`
`83`		`- escaped += escapedSb.String()`
`84`		`- return escaped`
	`82`	`+ return escapedSb.String()`
`85`	`83`	`}`
Original file line number	Diff line number	Diff line change
`@@ -26,8 +26,6 @@ func GetNote(ctx context.Context, repo Repository, commitID string, note Note)`
`26`	`26`	`return err`
`27`	`27`	`}`
`28`	`28`
`29`		`- path := ""`
`30`		`-`
`31`	`29`	`tree := &notes.Tree`
`32`	`30`	`log.Trace("Found tree with ID %q while searching for git note corresponding to the commit %q", tree.ID, commitID)`
`33`	`31`
`@@ -53,7 +51,7 @@ func GetNote(ctx context.Context, repo Repository, commitID string, note Note)`
`53`	`51`	`return err`
`54`	`52`	`}`
`55`	`53`	`}`
`56`		`- path += pathSb.String()`
	`54`	`+ path := pathSb.String()`
`57`	`55`
`58`	`56`	`blob := entry.Blob()`
`59`	`57`	`dataRc, err := blob.DataAsync()`
Original file line number	Diff line number	Diff line change
`@@ -24,13 +24,11 @@ type OpenWithEditorApp struct {`
`24`	`24`	`type OpenWithEditorAppsType []OpenWithEditorApp`
`25`	`25`
`26`	`26`	`func (t OpenWithEditorAppsType) ToTextareaString() string {`
`27`		`- ret := ""`
`28`	`27`	`var retSb strings.Builder`
`29`	`28`	`for _, app := range t {`
`30`	`29`	`retSb.WriteString(app.DisplayName + " = " + app.OpenURL + "\n")`
`31`	`30`	`}`
`32`		`- ret += retSb.String()`
`33`		`- return ret`
	`31`	`+ return retSb.String()`
`34`	`32`	`}`
`35`	`33`
`36`	`34`	`func DefaultOpenWithEditorApps() OpenWithEditorAppsType {`
Original file line number	Diff line number	Diff line change
`@@ -77,8 +77,8 @@ func decodeEnvSectionKey(encoded string) (ok bool, section, key string) {`
`77`	`77`	`}`
`78`	`78`	`last = unescapeIdx[1]`
`79`	`79`	`}`
`80`		`- key += keySb.String()`
`81`		`- section += sectionSb.String()`
	`80`	`+ key = keySb.String()`
	`81`	`+ section = sectionSb.String()`
`82`	`82`	`remaining := encoded[last:]`
`83`	`83`	`if !inKey {`
`84`	`84`	`if splitter := strings.Index(remaining, "__"); splitter > -1 {`
Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ func timeSincePro(then, now time.Time, lang translation.Locale) string {`
`95`	`95`	`return lang.TrString("tool.now")`
`96`	`96`	`}`
`97`	`97`
`98`		`- var timeStr, diffStr string`
	`98`	`+ var diffStr string`
`99`	`99`	`var timeStrSb strings.Builder`
`100`	`100`	`for {`
`101`	`101`	`if diff == 0 {`
`@@ -105,6 +105,5 @@ func timeSincePro(then, now time.Time, lang translation.Locale) string {`
`105`	`105`	`diff, diffStr = computeTimeDiffFloor(diff, lang)`
`106`	`106`	`timeStrSb.WriteString(", " + diffStr)`
`107`	`107`	`}`
`108`		`- timeStr += timeStrSb.String()`
`109`		`- return strings.TrimPrefix(timeStr, ", ")`
	`108`	`+ return strings.TrimPrefix(timeStrSb.String(), ", ")`
`110`	`109`	`}`
Original file line number	Diff line number	Diff line change
`@@ -433,17 +433,16 @@ func makePackageVersionDependency(ctx context.Context, version packages_model.`
`433`	`433`	`}`
`434`	`434`
`435`	`435`	`func makePackageInfo(ctx context.Context, versions []packages_model.PackageVersion, c *cache.EphemeralCache) (string, error) {`
`436`		`- ret := "---\n"`
`437`	`436`	`var retSb strings.Builder`
	`437`	`+ retSb.WriteString("---\n")`
`438`	`438`	`for _, v := range versions {`
`439`	`439`	`dep, err := makePackageVersionDependency(ctx, v, c)`
`440`	`440`	`if err != nil {`
`441`	`441`	`return "", err`
`442`	`442`	`}`
`443`	`443`	`retSb.WriteString(dep + "\n")`
`444`	`444`	`}`
`445`		`- ret += retSb.String()`
`446`		`- return ret, nil`
	`445`	`+ return retSb.String(), nil`
`447`	`446`	`}`
`448`	`447`
`449`	`448`	`func makeGemFullFileName(gemName, version, platform string) string {`
Original file line number	Diff line number	Diff line change
`@@ -179,13 +179,11 @@ func AuthorizeOAuth(ctx *context.Context) {`
`179`	`179`	`errs := binding.Errors{}`
`180`	`180`	`errs = form.Validate(ctx.Req, errs)`
`181`	`181`	`if len(errs) > 0 {`
`182`		`- errstring := ""`
`183`	`182`	`var errstringSb strings.Builder`
`184`	`183`	`for _, e := range errs {`
`185`	`184`	`errstringSb.WriteString(e.Error() + "\n")`
`186`	`185`	`}`
`187`		`- errstring += errstringSb.String()`
`188`		`- ctx.ServerError("AuthorizeOAuth: Validate: ", fmt.Errorf("errors occurred during validation: %s", errstring))`
	`186`	`+ ctx.ServerError("AuthorizeOAuth: Validate: ", fmt.Errorf("errors occurred during validation: %s", errstringSb.String()))`
`189`	`187`	`return`
`190`	`188`	`}`
`191`	`189`
Original file line number	Diff line number	Diff line change
`@@ -132,13 +132,12 @@ func KeysPost(ctx *context.Context) {`
`132`	`132`	`}`
`133`	`133`	`return`
`134`	`134`	`}`
`135`		`- keyIDs := ""`
`136`	`135`	`var keyIDsSb strings.Builder`
`137`	`136`	`for _, key := range keys {`
`138`	`137`	`keyIDsSb.WriteString(key.KeyID)`
`139`	`138`	`keyIDsSb.WriteString(", ")`
`140`	`139`	`}`
`141`		`- keyIDs += keyIDsSb.String()`
	`140`	`+ keyIDs := keyIDsSb.String()`
`142`	`141`	`if len(keyIDs) > 0 {`
`143`	`142`	`keyIDs = keyIDs[:len(keyIDs)-2]`
`144`	`143`	`}`