refactor: eliminate code duplication in chat tools argument formatting

[Copilot]

jscpd reported code duplication in pkg/cli/cmd/chat/tools.go between getToolArgs and formatToolArguments - both functions implemented identical logic for sorting map keys and formatting arguments as key=value pairs.

Changes

• Extracted formatArgsMap helper: Consolidates shared map-to-string conversion logic with sorted keys for deterministic output
• Refactored getToolArgs: Now calls formatArgsMap and wraps result in parentheses
• Refactored formatToolArguments: Now calls formatArgsMap directly

// Before: 8 lines of duplicate sorting/formatting logic in each function

// After: Single shared helper
func formatArgsMap(args map[string]any) string {
    if len(args) == 0 {
        return ""
    }
    keys := make([]string, 0, len(args))
    for k := range args {
        keys = append(keys, k)
    }
    slices.Sort(keys)
    parts := make([]string, 0, len(keys))
    for _, k := range keys {
        parts = append(parts, fmt.Sprintf("%s=%v", k, args[k]))
    }
    return strings.Join(parts, ", ")
}

Reduces jscpd duplication from 0.01% (over 0% threshold) to 0%.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.