Remove useless comments

Signed-off-by: Djordje Lukic <[email protected]>

Author: rumpl

pkg/tui/components/tool/defaulttool/defaulttool.go

@@ -40,24 +40,20 @@ func New(
 	}
 }
 
-// SetSize implements layout.Model.
 func (c *Component) SetSize(width, height int) tea.Cmd {
 	c.width = width
 	c.height = height
 	return nil
 }
 
-// Init implements layout.Model.
 func (c *Component) Init() tea.Cmd {
 	if c.message.ToolStatus == types.ToolStatusPending || c.message.ToolStatus == types.ToolStatusRunning {
 		return c.spinner.Tick
 	}
 	return nil
 }
 
-// Update implements layout.Model.
 func (c *Component) Update(msg tea.Msg) (layout.Model, tea.Cmd) {
-	// Handle spinner updates
 	if c.message.ToolStatus == types.ToolStatusPending || c.message.ToolStatus == types.ToolStatusRunning {
 		var cmd tea.Cmd
 		c.spinner, cmd = c.spinner.Update(msg)
@@ -67,7 +63,6 @@ func (c *Component) Update(msg tea.Msg) (layout.Model, tea.Cmd) {
 	return c, nil
 }
 
-// View implements layout.Model.
 func (c *Component) View() string {
 	msg := c.message
 	displayName := msg.ToolDefinition.DisplayName()

pkg/tui/components/tool/defaulttool/render.go

@@ -9,7 +9,6 @@ import (
 	"github.com/docker/cagent/pkg/tui/styles"
 )
 
-// renderToolArgs renders tool arguments
 func renderToolArgs(toolCall tools.ToolCall, width int) string {
 	decoder := json.NewDecoder(strings.NewReader(toolCall.Function.Arguments))
 

pkg/tui/components/tool/editfile/editfile.go

@@ -43,24 +43,20 @@ func New(
 	}
 }
 
-// SetSize implements [layout.Model].
 func (c *Component) SetSize(width, height int) tea.Cmd {
 	c.width = width
 	c.height = height
 	return nil
 }
 
-// Init implements [layout.Model].
 func (c *Component) Init() tea.Cmd {
 	if c.message.ToolStatus == types.ToolStatusPending || c.message.ToolStatus == types.ToolStatusRunning {
 		return c.spinner.Tick
 	}
 	return nil
 }
 
-// Update implements [layout.Model].
 func (c *Component) Update(msg tea.Msg) (layout.Model, tea.Cmd) {
-	// Handle spinner updates
 	if c.message.ToolStatus == types.ToolStatusPending || c.message.ToolStatus == types.ToolStatusRunning {
 		var cmd tea.Cmd
 		c.spinner, cmd = c.spinner.Update(msg)
@@ -70,7 +66,6 @@ func (c *Component) Update(msg tea.Msg) (layout.Model, tea.Cmd) {
 	return c, nil
 }
 
-// View implements [layout.Model].
 func (c *Component) View() string {
 	msg := c.message
 	var args builtin.EditFileArgs

pkg/tui/components/tool/editfile/render.go

@@ -37,7 +37,6 @@ type linePair struct {
 	newLineNum int
 }
 
-// renderEditFile renders edit_file tool arguments
 func renderEditFile(toolCall tools.ToolCall, width int, splitView bool, toolStatus types.ToolStatus) string {
 	var args builtin.EditFileArgs
 	if err := json.Unmarshal([]byte(toolCall.Function.Arguments), &args); err != nil {
@@ -65,9 +64,6 @@ func renderEditFile(toolCall tools.ToolCall, width int, splitView bool, toolStat
 	return output.String()
 }
 
-// computeDiff computes a diff between old and new text
-// For confirmation status: reads current file (old) and applies edit to get new content
-// For result status: reads current file (new) and reconstructs old content by reversing the edit
 func computeDiff(path, oldText, newText string, toolStatus types.ToolStatus) []*udiff.Hunk {
 	currentContent, err := os.ReadFile(path)
 	if err != nil {
@@ -130,7 +126,6 @@ func normalizeDiff(diff []*udiff.Hunk) []*udiff.Hunk {
 	return diff
 }
 
-// syntaxHighlight applies syntax highlighting to code and returns styled text
 func syntaxHighlight(code, filePath string) []chromaToken {
 	lexer := lexers.Match(filePath)
 	if lexer == nil {
@@ -178,7 +173,6 @@ func chromaToLipgloss(tokenType chroma.TokenType, style *chroma.Style) lipgloss.
 	return lipStyle
 }
 
-// renderDiffWithSyntaxHighlight renders a unified diff view
 func renderDiffWithSyntaxHighlight(diff []*udiff.Hunk, filePath string, width int) string {
 	var output strings.Builder
 	contentWidth := width - lineNumWidth
@@ -201,7 +195,6 @@ func renderDiffWithSyntaxHighlight(diff []*udiff.Hunk, filePath string, width in
 	return strings.TrimSuffix(output.String(), "\n")
 }
 
-// renderSplitDiffWithSyntaxHighlight renders a split diff view with old/new side-by-side
 func renderSplitDiffWithSyntaxHighlight(diff []*udiff.Hunk, filePath string, width int) string {
 	// Fall back to unified diff if terminal is too narrow
 	separator := styles.SeparatorStyle.Render(" │ ")
@@ -229,7 +222,6 @@ func renderSplitDiffWithSyntaxHighlight(diff []*udiff.Hunk, filePath string, wid
 	return strings.TrimSuffix(output.String(), "\n")
 }
 
-// getDisplayLineNumber returns the appropriate line number and updates counters
 func getDisplayLineNumber(line *udiff.Line, oldLineNum, newLineNum *int) int {
 	switch line.Kind {
 	case udiff.Delete:
@@ -249,7 +241,6 @@ func getDisplayLineNumber(line *udiff.Line, oldLineNum, newLineNum *int) int {
 	return 0
 }
 
-// prepareContent normalizes content for display
 func prepareContent(content string, maxWidth int) string {
 	content = strings.ReplaceAll(content, "\t", strings.Repeat(" ", tabWidth))
 	content = strings.TrimRight(content, "\n")
@@ -259,7 +250,6 @@ func prepareContent(content string, maxWidth int) string {
 	return content
 }
 
-// renderLine renders a line with syntax highlighting and appropriate styling
 func renderLine(content string, kind udiff.OpKind, filePath string, width int) string {
 	tokens := syntaxHighlight(content, filePath)
 	lineStyle := getLineStyle(kind)
@@ -269,7 +259,6 @@ func renderLine(content string, kind udiff.OpKind, filePath string, width int) s
 	return padToWidth(rendered, width, lineStyle)
 }
 
-// renderSplitSide renders one side of a split diff
 func renderSplitSide(line *udiff.Line, lineNum int, filePath string, width int) string {
 	lineNumStr := formatLineNum(line, lineNum)
 
@@ -284,7 +273,6 @@ func renderSplitSide(line *udiff.Line, lineNum int, filePath string, width int)
 	return styles.LineNumberStyle.Render(lineNumStr) + styledContent
 }
 
-// renderTokensWithStyle applies consistent styling to tokens
 func renderTokensWithStyle(tokens []chromaToken, lineStyle lipgloss.Style) string {
 	var output strings.Builder
 
@@ -296,7 +284,6 @@ func renderTokensWithStyle(tokens []chromaToken, lineStyle lipgloss.Style) strin
 	return output.String()
 }
 
-// padToWidth adds padding to reach the desired width
 func padToWidth(content string, width int, style lipgloss.Style) string {
 	currentWidth := ansi.StringWidth(content)
 	if paddingNeeded := width - currentWidth; paddingNeeded > 0 {
@@ -306,7 +293,6 @@ func padToWidth(content string, width int, style lipgloss.Style) string {
 	return content
 }
 
-// ensureWidth ensures a line has consistent width
 func ensureWidth(line string, width int) string {
 	if lineWidth := ansi.StringWidth(line); lineWidth < width {
 		padding := styles.DiffUnchangedStyle.Render(strings.Repeat(" ", width-lineWidth))
@@ -315,7 +301,6 @@ func ensureWidth(line string, width int) string {
 	return line
 }
 
-// getLineStyle returns the style for a diff line type
 func getLineStyle(kind udiff.OpKind) lipgloss.Style {
 	switch kind {
 	case udiff.Delete:
@@ -327,15 +312,13 @@ func getLineStyle(kind udiff.OpKind) lipgloss.Style {
 	}
 }
 
-// formatLineNum formats a line number or returns empty space
 func formatLineNum(line *udiff.Line, lineNum int) string {
 	if line == nil {
 		return strings.Repeat(" ", lineNumWidth)
 	}
 	return fmt.Sprintf("%4d ", lineNum)
 }
 
-// pairDiffLines pairs old and new lines for split view rendering
 func pairDiffLines(lines []udiff.Line, fromLine, toLine int) []linePair {
 	var pairs []linePair
 	oldLineNum, newLineNum := fromLine, toLine

pkg/tui/components/tool/factory.go

@@ -22,14 +22,12 @@ type Factory struct {
 	registry *Registry
 }
 
-// NewFactory creates a new component factory with the given registry.
 func NewFactory(registry *Registry) *Factory {
 	return &Factory{
 		registry: registry,
 	}
 }
 
-// Create creates a tool component for the given message.
 func (f *Factory) Create(
 	msg *types.Message,
 	renderer *glamour.TermRenderer,

pkg/tui/components/tool/readfile/readfile.go

@@ -40,24 +40,20 @@ func New(
 	}
 }
 
-// SetSize implements layout.Model.
 func (c *Component) SetSize(width, height int) tea.Cmd {
 	c.width = width
 	c.height = height
 	return nil
 }
 
-// Init implements layout.Model.
 func (c *Component) Init() tea.Cmd {
 	if c.message.ToolStatus == types.ToolStatusPending || c.message.ToolStatus == types.ToolStatusRunning {
 		return c.spinner.Tick
 	}
 	return nil
 }
 
-// Update implements layout.Model.
 func (c *Component) Update(msg tea.Msg) (layout.Model, tea.Cmd) {
-	// Handle spinner updates
 	if c.message.ToolStatus == types.ToolStatusPending || c.message.ToolStatus == types.ToolStatusRunning {
 		var cmd tea.Cmd
 		c.spinner, cmd = c.spinner.Update(msg)
@@ -67,7 +63,6 @@ func (c *Component) Update(msg tea.Msg) (layout.Model, tea.Cmd) {
 	return c, nil
 }
 
-// View implements layout.Model.
 func (c *Component) View() string {
 	msg := c.message
 	var args builtin.ReadFileArgs

pkg/tui/components/tool/registry.go

@@ -23,23 +23,18 @@ type Registry struct {
 	builders map[string]ComponentBuilder
 }
 
-// NewRegistry creates a new component registry.
 func NewRegistry() *Registry {
 	return &Registry{
 		builders: make(map[string]ComponentBuilder),
 	}
 }
 
-// Register adds a component builder for a tool name.
-// If a builder already exists for this tool name, it will be replaced.
 func (r *Registry) Register(toolName string, builder ComponentBuilder) {
 	r.mu.Lock()
 	defer r.mu.Unlock()
 	r.builders[toolName] = builder
 }
 
-// Get retrieves a component builder for a tool name.
-// Returns the builder and true if found, nil and false otherwise.
 func (r *Registry) Get(toolName string) (ComponentBuilder, bool) {
 	r.mu.RLock()
 	defer r.mu.RUnlock()

pkg/tui/components/tool/todotool/component.go

@@ -42,22 +42,19 @@ func New(
 	}
 }
 
-// SetSize implements [layout.Model].
 func (c *Component) SetSize(width, height int) tea.Cmd {
 	c.width = width
 	c.height = height
 	return nil
 }
 
-// Init implements [layout.Model].
 func (c *Component) Init() tea.Cmd {
 	if c.message.ToolStatus == types.ToolStatusPending || c.message.ToolStatus == types.ToolStatusRunning {
 		return c.spinner.Tick
 	}
 	return nil
 }
 
-// Update implements [layout.Model].
 func (c *Component) Update(msg tea.Msg) (layout.Model, tea.Cmd) {
 	if c.message.ToolStatus == types.ToolStatusPending || c.message.ToolStatus == types.ToolStatusRunning {
 		var cmd tea.Cmd
@@ -68,7 +65,6 @@ func (c *Component) Update(msg tea.Msg) (layout.Model, tea.Cmd) {
 	return c, nil
 }
 
-// View implements [layout.Model].
 func (c *Component) View() string {
 	msg := c.message
 	toolName := msg.ToolCall.Function.Name
@@ -88,7 +84,6 @@ func (c *Component) View() string {
 	}
 }
 
-// renderCreate renders the create_todo operation
 func (c *Component) renderCreate() string {
 	msg := c.message
 	displayName := msg.ToolDefinition.DisplayName()
@@ -117,7 +112,6 @@ func (c *Component) renderCreate() string {
 	return styles.BaseStyle.PaddingLeft(2).PaddingTop(1).Render(content + resultContent)
 }
 
-// renderCreateMultiple renders the create_todos operation
 func (c *Component) renderCreateMultiple() string {
 	msg := c.message
 	displayName := msg.ToolDefinition.DisplayName()
@@ -148,7 +142,6 @@ func (c *Component) renderCreateMultiple() string {
 	return styles.BaseStyle.PaddingLeft(2).PaddingTop(1).Render(content + resultContent)
 }
 
-// renderList renders the list_todos operation
 func (c *Component) renderList() string {
 	msg := c.message
 	displayName := msg.ToolDefinition.DisplayName()
@@ -186,7 +179,6 @@ func (c *Component) renderList() string {
 	return styles.BaseStyle.PaddingLeft(2).PaddingTop(1).Render(content)
 }
 
-// renderUpdate renders the update_todo operation
 func (c *Component) renderUpdate() string {
 	msg := c.message
 	displayName := msg.ToolDefinition.DisplayName()
@@ -218,7 +210,6 @@ func (c *Component) renderUpdate() string {
 	return styles.BaseStyle.PaddingLeft(2).PaddingTop(1).Render(content + resultContent)
 }
 
-// renderDefault provides a fallback rendering for unknown tool types
 func (c *Component) renderDefault() string {
 	msg := c.message
 	displayName := msg.ToolDefinition.DisplayName()

pkg/tui/components/tool/todotool/sidebar.go

@@ -25,12 +25,10 @@ func NewSidebarComponent(manager *service.TodoManager) *SidebarComponent {
 	}
 }
 
-// SetSize sets the component width
 func (c *SidebarComponent) SetSize(width int) {
 	c.width = width
 }
 
-// SetTodos sets the todo from a tool call, handles create_todo, create_todos, update_todo
 func (c *SidebarComponent) SetTodos(toolCall tools.ToolCall) error {
 	params, err := parseTodoArgs(toolCall)
 	if err != nil {

pkg/tui/components/tool/todotool/todotool.go

@@ -6,7 +6,6 @@ import (
 	"github.com/docker/cagent/pkg/tui/styles"
 )
 
-// renderTodoIcon returns the icon and style for a todo status
 func renderTodoIcon(status string) (string, lipgloss.Style) {
 	switch status {
 	case "pending":