Fixes #115 and #118 - Proper SetInfected packet body

src/Impostor.Api/Games/IGame.cs

@@ -4,6 +4,7 @@
 using Impostor.Api.Innersloth;
 using Impostor.Api.Net;
 using Impostor.Api.Net.Inner;
+using Impostor.Api.Net.Inner.Objects;
 using Impostor.Api.Net.Messages;
 
 namespace Impostor.Api.Games
@@ -49,9 +50,16 @@ public interface IGame
         ///     Syncs the internal <see cref="GameOptionsData"/> to all players.
         ///     Necessary to do if you modified it, otherwise it won't be used.
         /// </summary>
-        /// <returns></returns>
+        /// <returns>A <see cref="ValueTask"/> representing the asynchronous operation.</returns>
         ValueTask SyncSettingsAsync();
 
+        /// <summary>
+        ///     Sets the specified list as Impostor on all connected players.
+        /// </summary>
+        /// <param name="players">List of players to be Impostor.</param>
+        /// <returns>A <see cref="ValueTask"/> representing the asynchronous operation.</returns>
+        ValueTask SetInfectedAsync(IEnumerable<IInnerPlayerControl> players);
+
         /// <summary>
         ///     Send the message to all players.
         /// </summary>
@@ -77,4 +85,4 @@ public interface IGame
         /// <returns>A <see cref="ValueTask"/> representing the asynchronous operation.</returns>
         ValueTask SendToAsync(IMessageWriter writer, int id);
     }
-}
+}

src/Impostor.Api/Net/IClientPlayer.cs

@@ -25,7 +25,7 @@ public interface IClientPlayer
         /// </summary>
         LimboStates Limbo { get; set; }
 
-        IInnerPlayerControl Character { get; }
+        IInnerPlayerControl? Character { get; }
 
         public bool IsHost { get; }
 
@@ -40,4 +40,4 @@ public interface IClientPlayer
 
         ValueTask BanAsync();
     }
-}
+}

src/Impostor.Api/Net/Inner/Objects/IInnerPlayerControl.cs

@@ -6,6 +6,11 @@ namespace Impostor.Api.Net.Inner.Objects
 {
     public interface IInnerPlayerControl : IInnerNetObject
     {
+        /// <summary>
+        ///     Gets the <see cref="PlayerId"/> assigned by the client of the host of the game.
+        /// </summary>
+        byte PlayerId { get; }
+
         /// <summary>
         ///     Gets the <see cref="IInnerPlayerPhysics"/> of the <see cref="IInnerPlayerControl"/>.
         ///     Contains vent logic.
@@ -100,13 +105,6 @@ public interface IInnerPlayerControl : IInnerNetObject
         /// <returns>Task that must be awaited.</returns>
         ValueTask SendChatToPlayerAsync(string text, IInnerPlayerControl? player = null);
 
-        /// <summary>
-        ///     Sets the current to infected (impostor) <see cref="IInnerPlayerControl"/>.
-        ///     Visible to all players.
-        /// </summary>
-        /// <returns>Task that must be awaited.</returns>
-        ValueTask SetInfectedAsync();
-
         /// <summary>
         ///     Sets the current to be murdered <see cref="IInnerPlayerControl"/>.
         ///     Visible to all players.

src/Impostor.Server/Net/Inner/Objects/InnerGameData.cs

@@ -43,8 +43,7 @@ public InnerGameData(ILogger<InnerGameData> logger, Game game, IServiceProvider
             return _allPlayers.TryGetValue(id, out var player) ? player : null;
         }
 
-        public override ValueTask HandleRpc(ClientPlayer sender, ClientPlayer? target, RpcCalls call,
-            IMessageReader reader)
+        public override ValueTask HandleRpc(ClientPlayer sender, ClientPlayer? target, RpcCalls call, IMessageReader reader)
         {
             switch (call)
             {

src/Impostor.Server/Net/Inner/Objects/InnerPlayerControl.Api.cs

@@ -103,12 +103,5 @@ public async ValueTask SetMurderedAsync()
             writer.Write((byte)NetId);
             await _game.FinishRpcAsync(writer);
         }
-
-        public async ValueTask SetInfectedAsync()
-        {
-            var writer = _game.StartRpc(NetId, RpcCalls.SetInfected);
-            writer.Write((byte)NetId);
-            await _game.FinishRpcAsync(writer);
-        }
     }
 }

src/Impostor.Server/Net/State/ClientPlayer.Api.cs

@@ -13,6 +13,6 @@ internal partial class ClientPlayer
         IGame IClientPlayer.Game => Game;
 
         /// <inheritdoc />
-        IInnerPlayerControl IClientPlayer.Character => Character;
+        IInnerPlayerControl? IClientPlayer.Character => Character;
     }
-}
+}

src/Impostor.Server/Net/State/ClientPlayer.cs

@@ -30,7 +30,7 @@ public ClientPlayer(ILogger<ClientPlayer> logger, ClientBase client, Game game)
         /// <inheritdoc />
         public LimboStates Limbo { get; set; }
 
-        public InnerPlayerControl Character { get; internal set; }
+        public InnerPlayerControl? Character { get; internal set; }
 
         public bool IsHost => Game?.Host == this;
 

src/Impostor.Server/Net/State/Game.Api.cs

@@ -1,10 +1,13 @@
-using System.IO;
+using System.Collections.Generic;
+using System.IO;
 using System.Net;
 using System.Threading.Tasks;
+using Impostor.Api;
 using Impostor.Api.Games;
 using Impostor.Api.Innersloth;
 using Impostor.Api.Net;
 using Impostor.Api.Net.Inner;
+using Impostor.Api.Net.Inner.Objects;
 using Impostor.Server.Net.Inner;
 
 namespace Impostor.Server.Net.State
@@ -22,6 +25,11 @@ public void BanIp(IPAddress ipAddress)
 
         public async ValueTask SyncSettingsAsync()
         {
+            if (Host.Character == null)
+            {
+                throw new ImpostorException("Attempted to set infected when the host was not spawned.");
+            }
+
             using (var writer = StartRpc(Host.Character.NetId, RpcCalls.SyncSettings))
             {
                 // Someone will probably forget to do this, so we include it here.
@@ -38,5 +46,25 @@ await using (var writerBin = new BinaryWriter(memory))
                 await FinishRpcAsync(writer);
             }
         }
+
+        public async ValueTask SetInfectedAsync(IEnumerable<IInnerPlayerControl> players)
+        {
+            if (Host.Character == null)
+            {
+                throw new ImpostorException("Attempted to set infected when the host was not spawned.");
+            }
+
+            using (var writer = StartRpc(Host.Character.NetId, RpcCalls.SetInfected))
+            {
+                writer.Write((byte)Host.Character.NetId);
+
+                foreach (var player in players)
+                {
+                    writer.Write((byte)player.PlayerId);
+                }
+
+                await FinishRpcAsync(writer);
+            }
+        }
     }
-}
+}