chore: task fmt (#1098)

Amolith created 1 month ago

Change summary

internal/config/load.go               | 14 +++++++-------
internal/llm/provider/anthropic.go    |  2 +-
internal/llm/provider/openai.go       |  2 +-
internal/llm/provider/provider.go     |  2 +-
internal/shell/shell.go               |  2 +-
internal/tui/exp/diffview/diffview.go |  6 +++---
internal/tui/exp/diffview/split.go    |  2 +-
internal/tui/exp/list/list.go         |  2 +-
8 files changed, 16 insertions(+), 16 deletions(-)

Detailed changes

internal/config/load.go

@@ -381,7 +381,7 @@ func (c *Config) applyLSPDefaults() {
 func (c *Config) defaultModelSelection(knownProviders []catwalk.Provider) (largeModel SelectedModel, smallModel SelectedModel, err error) {
 	if len(knownProviders) == 0 && c.Providers.Len() == 0 {
 		err = fmt.Errorf("no providers configured, please configure at least one provider")
-		return
+		return largeModel, smallModel, err
 	}
 
 	// Use the first provider enabled based on the known providers order
@@ -394,7 +394,7 @@ func (c *Config) defaultModelSelection(knownProviders []catwalk.Provider) (large
 		defaultLargeModel := c.GetModel(string(p.ID), p.DefaultLargeModelID)
 		if defaultLargeModel == nil {
 			err = fmt.Errorf("default large model %s not found for provider %s", p.DefaultLargeModelID, p.ID)
-			return
+			return largeModel, smallModel, err
 		}
 		largeModel = SelectedModel{
 			Provider:        string(p.ID),
@@ -406,7 +406,7 @@ func (c *Config) defaultModelSelection(knownProviders []catwalk.Provider) (large
 		defaultSmallModel := c.GetModel(string(p.ID), p.DefaultSmallModelID)
 		if defaultSmallModel == nil {
 			err = fmt.Errorf("default small model %s not found for provider %s", p.DefaultSmallModelID, p.ID)
-			return
+			return largeModel, smallModel, err
 		}
 		smallModel = SelectedModel{
 			Provider:        string(p.ID),
@@ -414,7 +414,7 @@ func (c *Config) defaultModelSelection(knownProviders []catwalk.Provider) (large
 			MaxTokens:       defaultSmallModel.DefaultMaxTokens,
 			ReasoningEffort: defaultSmallModel.DefaultReasoningEffort,
 		}
-		return
+		return largeModel, smallModel, err
 	}
 
 	enabledProviders := c.EnabledProviders()
@@ -424,13 +424,13 @@ func (c *Config) defaultModelSelection(knownProviders []catwalk.Provider) (large
 
 	if len(enabledProviders) == 0 {
 		err = fmt.Errorf("no providers configured, please configure at least one provider")
-		return
+		return largeModel, smallModel, err
 	}
 
 	providerConfig := enabledProviders[0]
 	if len(providerConfig.Models) == 0 {
 		err = fmt.Errorf("provider %s has no models configured", providerConfig.ID)
-		return
+		return largeModel, smallModel, err
 	}
 	defaultLargeModel := c.GetModel(providerConfig.ID, providerConfig.Models[0].ID)
 	largeModel = SelectedModel{
@@ -444,7 +444,7 @@ func (c *Config) defaultModelSelection(knownProviders []catwalk.Provider) (large
 		Model:     defaultSmallModel.ID,
 		MaxTokens: defaultSmallModel.DefaultMaxTokens,
 	}
-	return
+	return largeModel, smallModel, err
 }
 
 func (c *Config) configureSelectedModels(knownProviders []catwalk.Provider) error {

internal/llm/provider/anthropic.go

@@ -175,7 +175,7 @@ func (a *anthropicClient) convertMessages(messages []message.Message) (anthropic
 			anthropicMessages = append(anthropicMessages, anthropic.NewUserMessage(results...))
 		}
 	}
-	return
+	return anthropicMessages
 }
 
 func (a *anthropicClient) convertTools(tools []tools.BaseTool) []anthropic.ToolUnionParam {

internal/llm/provider/openai.go

@@ -178,7 +178,7 @@ func (o *openaiClient) convertMessages(messages []message.Message) (openaiMessag
 		}
 	}
 
-	return
+	return openaiMessages
 }
 
 func (o *openaiClient) convertTools(tools []tools.BaseTool) []openai.ChatCompletionToolParam {

internal/llm/provider/provider.go

@@ -98,7 +98,7 @@ func (p *baseProvider[C]) cleanMessages(messages []message.Message) (cleaned []m
 		}
 		cleaned = append(cleaned, msg)
 	}
-	return
+	return cleaned
 }
 
 func (p *baseProvider[C]) SendMessages(ctx context.Context, messages []message.Message, tools []tools.BaseTool) (*ProviderResponse, error) {

internal/shell/shell.go

@@ -207,7 +207,7 @@ func splitArgsFlags(parts []string) (args []string, flags []string) {
 			args = append(args, part)
 		}
 	}
-	return
+	return args, flags
 }
 
 func (s *Shell) blockHandler() func(next interp.ExecHandlerFunc) interp.ExecHandlerFunc {

internal/tui/exp/diffview/diffview.go

@@ -408,7 +408,7 @@ func (dv *DiffView) renderUnified() string {
 		content = ansi.GraphemeWidth.Cut(content, dv.xOffset, len(content))
 		content = ansi.Truncate(content, dv.codeWidth, "…")
 		leadingEllipsis = dv.xOffset > 0 && strings.TrimSpace(content) != ""
-		return
+		return content, leadingEllipsis
 	}
 
 outer:
@@ -531,7 +531,7 @@ func (dv *DiffView) renderSplit() string {
 		content = ansi.GraphemeWidth.Cut(content, dv.xOffset, len(content))
 		content = ansi.Truncate(content, dv.codeWidth, "…")
 		leadingEllipsis = dv.xOffset > 0 && strings.TrimSpace(content) != ""
-		return
+		return content, leadingEllipsis
 	}
 
 outer:
@@ -716,7 +716,7 @@ func (dv *DiffView) hunkShownLines(h *udiff.Hunk) (before, after int) {
 			before++
 		}
 	}
-	return
+	return before, after
 }
 
 func (dv *DiffView) lineStyleForType(t udiff.OpKind) LineStyle {

internal/tui/exp/diffview/split.go

@@ -69,5 +69,5 @@ func hunkToSplit(h *udiff.Hunk) (sh splitHunk) {
 		sh.lines = append(sh.lines, &sl)
 	}
 
-	return
+	return sh
 }

internal/tui/exp/list/list.go

@@ -1372,7 +1372,7 @@ func (l *list[T]) findWordBoundaries(col, line int) (startCol, endCol int) {
 	if startCol == -1 {
 		return 0, 0
 	}
-	return
+	return startCol, endCol
 }
 
 func (l *list[T]) findParagraphBoundaries(line int) (startLine, endLine int, found bool) {