The comment indicates this is "FIXED" but the function name suggests this still demonstrates the problematic pattern. The comment should clarify that this function shows the corrected approach, or the function should be renamed to better reflect its purpose.
* Example function that demonstrates the corrected approach
* This function avoids the unnecessary spread operation
*/
export function sortConnectorsWithoutUnnecessarySpread(
connectors: Connector[],
): Connector[] {
// No unnecessary spread operation; sortConnectorsByExplorerWallet already creates a copy
Originally posted by @copilot-pull-request-reviewer in #38 (comment)