What he is saying is that the function is mutating the original signature array.
You should make a copy of the signature array. You can do that with const aryintSignature_Copy = [...signature];

for sake of learning, why is this the worst solution?

Why this is the top rated solution? We polluting argument here. It is the worst one.