fix: use copy() for remaining ToolResults loop (S1001)

Patel230 · Patel230 · commit 8eca2ca08b6f · 2026-05-28T21:09:23.000+05:30
diff --git a/cmd/chat_commands.go b/cmd/chat_commands.go
@@ -842,9 +842,7 @@ func (m *chatModel) handleCommand(text string) (tea.Model, tea.Cmd) {
 			}
 			if len(sm.ToolResults) > 0 {
 				em.ToolResults = make([]client.ToolResult, len(sm.ToolResults))
-				for i, tr := range sm.ToolResults {
-					em.ToolResults[i] = tr
-				}
+				copy(em.ToolResults, sm.ToolResults)
 			}
 			msgs = append(msgs, em)
 			if sm.Role == "user" || sm.Role == "assistant" {

Original file line number	Diff line number	Diff line change
`@@ -842,9 +842,7 @@ func (m *chatModel) handleCommand(text string) (tea.Model, tea.Cmd) {`
`842`	`842`	`}`
`843`	`843`	`if len(sm.ToolResults) > 0 {`
`844`	`844`	`em.ToolResults = make([]client.ToolResult, len(sm.ToolResults))`
`845`		`- for i, tr := range sm.ToolResults {`
`846`		`- em.ToolResults[i] = tr`
`847`		`- }`
	`845`	`+ copy(em.ToolResults, sm.ToolResults)`
`848`	`846`	`}`
`849`	`847`	`msgs = append(msgs, em)`
`850`	`848`	`if sm.Role == "user" \|\| sm.Role == "assistant" {`