const pos = this.state.todos.map(todo=>{
  return todo.id;
}).indexOf(todo.id);

ですが、idの配列を作って、todo.idを一致させる理由がよく分かりませんでした。

単に

const pos = todo.id;

としてはまずいのでしょうか？