Why is my spectator script not working every EVEN time its clicked?

The MouseButton1Click connections are stacking. Every time the player clicks the "Spectate" button, a new connection is made, and so the same code is run multiple time for each click. I believe the code is actually "working", it's just that it cycles back to the same player when there is an even number of connections! We can change the code to clean up the connections when spectating ends:

camera = workspace.CurrentCamera
getPlayerInfo = game.ReplicatedStorage.Remotes.GetPlayerRole
gameLobby = script.Parent.Parent.GameLobby
components = script.Parent.Parent.SpectateComponents
getPlayers = game.ReplicatedStorage.Remotes.GetPlayers
local index

function createSpectateList()
    local players = getPlayers:InvokeServer()
    local spectateList = {}

    for _, plr in pairs(players) do
        if getPlayerInfo:InvokeServer(plr) ~= "" and plr.Name ~= game.Players.LocalPlayer.Name then
            table.insert(spectateList, plr.Name)
        end
    end

    return spectateList
end

script.Parent.MouseButton1Click:Connect(function()
    index = 1
    local players = createSpectateList()
    components.SpectateUser.Visible = true
    components.Right.Visible = true
    components.Left.Visible = true
    script.Parent.Parent.LeaveSpectate.Visible = true

    gameLobby.Visible = false
    script.Parent.Visible = false
    components.SpectateUser.Text = players[index]
    camera.CameraSubject = game.Players:FindFirstChild(players[index]).Character:WaitForChild("Humanoid")

    local connections = {} -- Add a table of connections

    connections.right = components.Right.MouseButton1Click:Connect(function()
        if index ~= #players then index = index + 1 else index = 1 end
        camera.CameraSubject = game.Players:FindFirstChild(players[index]).Character:WaitForChild("Humanoid")
        components.SpectateUser.Text = players[index]
    end)

    connections.left = components.Left.MouseButton1Click:Connect(function()
        if index ~= 1 then index = index - 1 else index = #players end
        camera.CameraSubject = game.Players:FindFirstChild(players[index]).Character:WaitForChild("Humanoid")
        components.SpectateUser.Text = players[index]
    end)

    connections.leave = script.Parent.Parent.LeaveSpectate.MouseButton1Click:Connect(function()
        index = 1
        camera.CameraSubject = game.Players.LocalPlayer.Character:WaitForChild("Humanoid")
        components.SpectateUser.Visible = false
        components.Right.Visible = false
        components.Left.Visible = false
        script.Parent.Parent.LeaveSpectate.Visible = false
        script.Parent.Parent.GameLobby.Visible = true
        script.Parent.Visible = true

        -- Clean up the connections
        for _, conn in pairs(connections) do
            conn:Disconnect()
        end
    end)
end)