fix(livechat): ensure transaction is aborted only if in progress and simplify room closing check

- Added a check to abort the transaction only if it is currently in progress, improving error handling.
- Simplified the condition for throwing an error when closing a room by using optional chaining for better readability.

Author: ggazzo

apps/meteor/app/livechat/server/lib/closeRoom.ts

@@ -42,7 +42,9 @@ export async function closeRoom(params: CloseRoomParams, attempts = 2): Promise<
 		removedInquiryObj = removedInquiry;
 	} catch (e) {
 		logger.error({ err: e, msg: 'Failed to close room', afterAttempts: attempts });
-		await session.abortTransaction();
+		if (session.inTransaction()) {
+			await session.abortTransaction();
+		}
 		// Dont propagate transaction errors
 		if (shouldRetryTransaction(e)) {
 			if (attempts > 0) {
@@ -186,7 +188,7 @@ async function doCloseRoom(
 	}
 
 	const updatedRoom = await LivechatRooms.closeRoomById(rid, closeData, { session });
-	if (!params.forceClose && (!updatedRoom || updatedRoom.modifiedCount !== 1)) {
+	if (!params.forceClose && updatedRoom?.modifiedCount !== 1) {
 		throw new Error('Error closing room');
 	}