Oltre al fatto che è sempre meglio evitare eval quando si può... se vuoi mantenere il trattino così ti dovrebbe funzionare:

codice:
function passaidcg(id,indice){
  document.edit["idcg" + indice].value = id;
}
function passaidsi(id,indice,idtipoci){
  document.edit["idsi" + idtipoci + "-" + indice].value = id;
}