fix: properly implement /join

irc/handler/handler.go

@@ -73,6 +73,20 @@ func handleJoin(msg parser.IRCMessage, client *irc.Client) {
 	channel := msg.Parameters[0]
 
 	joinMsg := fmt.Sprintf("== %s has joined", nick)
+
+	if nick == client.Nickname {
+		client.Channels = append(client.Channels, irc.Channel{
+			Name:  channel,
+			Users: make(map[string]irc.User),
+		})
+		client.ActiveChannelIndex = len(client.Channels) - 1
+		client.ActiveChannel = channel
+		client.Channels[client.ActiveChannelIndex].History += joinMsg + irc.CRLF
+		if _, exists := client.Channels[client.ActiveChannelIndex].Users[nick]; !exists {
+			client.Channels[client.ActiveChannelIndex].Users[nick] = irc.User{}
+		}
+		client.Tea.Send(cmds.UpdateTabBar())
+	} else {
 		for i, c := range client.Channels {
 			if c.Name == channel {
 				client.Channels[i].History += joinMsg + irc.CRLF
@@ -81,6 +95,7 @@ func handleJoin(msg parser.IRCMessage, client *irc.Client) {
 				}
 			}
 		}
+	}
 
 	if channel == client.ActiveChannel {
 		client.Tea.Send(cmds.SwitchChannels())

irc/handler/slash.go

@@ -26,35 +26,21 @@ import (
 )
 
 func handleSlashJoin(params []string, client *irc.Client) tea.Cmd {
-	// TODO: only add channel to client slice when
-	// we get a JOIN command from the server with our nickname
-	// TODO: handle comma separated channels
-	var cmdsToProcess []tea.Cmd
-
 	channel := ""
 	if len(params) > 0 {
 		channel = params[0]
 	}
 
-	client.ActiveChannel = channel
 	for i, c := range client.Channels {
 		if c.Name == channel {
 			client.ActiveChannelIndex = i
+			client.ActiveChannel = channel
 			break
-		} else if i == len(client.Channels)-1 {
-			client.Channels = append(client.Channels, irc.Channel{
-				Name:  channel,
-				Users: make(map[string]irc.User),
-			})
-			client.ActiveChannelIndex = len(client.Channels) - 1
-			cmdsToProcess = append(cmdsToProcess, cmds.UpdateTabBar)
 		}
 	}
 	client.SendCommand(commands.JOIN, params...)
 
-	cmdsToProcess = append(cmdsToProcess, cmds.SwitchChannels)
+	return cmds.SwitchChannels
-
-	return tea.Batch(cmdsToProcess...)
 }
 
 func HandleSlashCommand(msg string, client *irc.Client) tea.Cmd {