const passwordLength = document.getElementById('password-length');

など addEventListener() の外で定義するイメージでしたが、今回の場合、中でしか使わないことが確定しているから中で定義しているようなイメージを持てばいいのでしょうか。
中でしか使わないものを外で定義しないことによって読込速度などに変化が生じるからでしょうか。