Optimize user list updates for quit/part/kick events

client/js/socket-events/msg.js

@@ -85,12 +85,8 @@ socket.on("msg", function(data) {
 		channel.moreHistoryAvailable = true;
 	}
 
-	if ((data.msg.type === "message" || data.msg.type === "action") && channel.type === "channel") {
+	if (channel.type === "channel") {
-		const user = channel.users.find((u) => u.nick === data.msg.from.nick);
+		updateUserList(channel, data.msg);
-
-		if (user) {
-			user.lastMessage = new Date(data.msg.time).getTime() || Date.now();
-		}
 	}
 });
 
@@ -169,3 +165,25 @@ function notifyMessage(targetId, channel, activeChannel, msg) {
 		}
 	}
 }
+
+function updateUserList(channel, msg) {
+	if (msg.type === "message" || msg.type === "action") {
+		const user = channel.users.find((u) => u.nick === msg.from.nick);
+
+		if (user) {
+			user.lastMessage = new Date(msg.time).getTime() || Date.now();
+		}
+	} else if (msg.type === "quit" || msg.type === "part") {
+		const idx = channel.users.findIndex((u) => u.nick === msg.from.nick);
+
+		if (idx > -1) {
+			channel.users.splice(idx, 1);
+		}
+	} else if (msg.type === "kick") {
+		const idx = channel.users.findIndex((u) => u.nick === msg.target.nick);
+
+		if (idx > -1) {
+			channel.users.splice(idx, 1);
+		}
+	}
+}

src/plugins/irc-events/kick.js

@@ -35,9 +35,5 @@ module.exports = function(irc, network) {
 		} else {
 			chan.removeUser(msg.target);
 		}
-
-		client.emit("users", {
-			chan: chan.id,
-		});
 	});
 };

src/plugins/irc-events/part.js

@@ -33,9 +33,6 @@ module.exports = function(irc, network) {
 			});
 		} else {
 			chan.removeUser(user);
-			client.emit("users", {
-				chan: chan.id,
-			});
 		}
 	});
 };

src/plugins/irc-events/quit.js

@@ -23,9 +23,6 @@ module.exports = function(irc, network) {
 			chan.pushMessage(client, msg);
 
 			chan.removeUser(user);
-			client.emit("users", {
-				chan: chan.id,
-			});
 		});
 
 		// If user with the nick we are trying to keep has quit, try to get this nick