Usando una expresión regular en split()
sería un enfoque totalmente distinto y reduce un poco el código:
const num = 24589246483;2458936;
const res = `${num}`.split(/([13579])/g).slice(0,-1).join("-").replace(/--/g, '-');
console.log(res);
Iterando, yo usaría [...'${num}']
para pasar de numero a array y luego %2
para cada número:
const num = 24589;2458936;
const res = [...`${num}`].map((e,i,a)=>(e%2)?(i<a.length -1)?`-${e}-`:`-${e}`:e).join("").replace(/--/g, '-');
console.log(res);