Refactor vehicle passenger handling and dismount event logic

- Remove unnecessary semicolons
- Simplify dismount event emission
- Ensure consistent null and array checks for vehicle and passenger entities

lib/plugins/entities.js

@@ -842,22 +842,22 @@ function inject (bot) {
     if (vehicle && !vehicle.passengers) {
       vehicle.passengers = []
     }
-    
+
     for (const passengerEntity of passengerEntities) {
-      if (!passengerEntity) continue; // Skip if passenger entity is null
-      
+      if (!passengerEntity) continue // Skip if passenger entity is null
+
       if (!passengerEntity.passengers) {
         passengerEntity.passengers = []
       }
-      
+
       const originalVehicle = passengerEntity.vehicle
       if (originalVehicle && originalVehicle.passengers && Array.isArray(originalVehicle.passengers)) {
         const index = originalVehicle.passengers.indexOf(passengerEntity)
         if (index !== -1) {
           originalVehicle.passengers.splice(index, 1) // Fixed: use splice directly instead of reassignment
         }
       }
-      
+
       passengerEntity.vehicle = vehicle
       if (vehicle && vehicle.passengers && Array.isArray(vehicle.passengers)) {
         vehicle.passengers.push(passengerEntity)
@@ -878,11 +878,11 @@ function inject (bot) {
 
   // dismounting when the vehicle is gone
   bot._client.on('entityGone', (entity) => {
-    if (!entity) return;
-    
+    if (!entity) return
+
     if (bot.vehicle === entity) {
       bot.vehicle = null
-      bot.emit('dismount', (entity))
+      bot.emit('dismount', entity)
     }
     if (entity.passengers && Array.isArray(entity.passengers)) {
       for (const passenger of entity.passengers) {