fix: unbiased Fisher-Yates in bogoSort; do not mutate RGB input

[zeeshanzzz788]

Description

BogoSort (Sorts/BogoSort.js)
The previous shuffle used Math.random() * i and swapped with index i - 1, which is a biased shuffle (some permutations never occur). Replaced with standard Fisher–Yates over [0, i].

RgbHslConversion (Conversions/RgbHslConversion.js)
rgbToHsl aliased the input array (let colorHsl = colorRgb) and wrote HSL values into it, destroying the caller’s RGB data. It now copies with .slice() first.

Verification

const input = [24, 98, 118]
const result = rgbToHsl(input)
// input remains [24, 98, 118]; result is [193, 66, 28]; input !== result
bogoSort([3,1,2]) // -> [1,2,3]

Fixes #1867
Fixes #1907

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.91%. Comparing base (5c39e87) to head (21221c4).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1912   +/-   ##
=======================================
  Coverage   85.91%   85.91%           
=======================================
  Files         379      379           
  Lines       19778    19778           
  Branches     3016     3015    -1     
=======================================
  Hits        16993    16993           
  Misses       2785     2785           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.