Propagate const T* through argsort/argselect call chain, remove std::remove_const_t workarounds

[Copilot]

• Propagate const T* to xss_argsort, xss_argselect and all related internal functions
• Remove std::remove_const_t and const_cast workarounds from wrapper functions
• Fix clang-format violations (parameter list line-wrapping in std_argselect_withnan, std_argsort_withnan, and function call sites)

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[r-devulap]

@AnkitAhlawat7742 I put an agent to task to resolve #229 (comment). Lets see how it does :)

[r-devulap]

@copilot fix the lint CI failures

[Copilot]

@copilot fix the lint CI failures

Fixed in 37ec892. The failures were clang-format violations in parameter list formatting for std_argselect_withnan, std_argsort_withnan, and two function call sites in the wrapper functions. Applied clang-format -style=file -i to reformat them correctly.

[r-devulap]

lgtm.

[r-devulap]

@seberg mind approving this PR? My approval is apparently not enough :)

[seberg]

Hummm, I guess because you triggered that agent maybe? Which honestly makes some sense.

If this becomes annoying, we can consider removing the "one approval required" rule maybe (I suppose that is active somewhere).

[r-devulap]

If this becomes annoying, we can consider removing the "one approval required" rule maybe (I suppose that is active somewhere).

Might be useful going forward.