Contributing Guide

Nitrox-Cracked-Mod/
├── Nitrox.Launcher/          # Desktop application (Avalonia UI)
├── NitroxPatcher/            # Harmony patches and game injection
│   ├── Patches/
│   │   ├── Dynamic/          # Multiplayer-only patches
│   │   └── Persistent/       # Always-active patches
│   └── Main.cs               # Entry point
├── NitroxClient/             # Client-side multiplayer logic
│   ├── Communication/        # Networking layer
│   ├── GameLogic/            # Client game systems
│   └── MonoBehaviours/       # Unity components
├── Nitrox.Server.Subnautica/ # Dedicated server
│   └── Models/               # Server-side systems
├── Nitrox.Model/             # Shared core library
│   ├── Packets/              # Packet base classes
│   └── DataStructures/       # Common data types
├── Nitrox.Model.Subnautica/  # Subnautica-specific models
│   └── Packets/              # Game-specific packets
└── Nitrox.Test/              # Unit and integration tests

# Debug build
dotnet build -c Debug

# Release build
dotnet build -c Release

// ✅ Good: Clear, concise, follows conventions
public class PlayerMovementHandler
{
    private readonly IPacketSender packetSender;
    
    public PlayerMovementHandler(IPacketSender packetSender)
    {
        this.packetSender = packetSender;
    }
    
    public void SendMovementUpdate(Vector3 position, Quaternion rotation)
    {
        packetSender.Send(new PlayerMovement(position, rotation));
    }
}

// ❌ Bad: Over-commented, verbose
public class PlayerMovementHandler
{
    // This is the packet sender that we use to send packets
    private readonly IPacketSender packetSender;
    
    /// <summary>
    /// Constructor that initializes the packet sender
    /// </summary>
    /// <param name="packetSender">The packet sender to use</param>
    public PlayerMovementHandler(IPacketSender packetSender)
    {
        // Set the packet sender field to the provided value
        this.packetSender = packetSender;
    }
    
    // This method sends a movement update to the server
    public void SendMovementUpdate(Vector3 position, Quaternion rotation)
    {
        // Create a new player movement packet
        PlayerMovement packet = new PlayerMovement(position, rotation);
        // Send the packet using the packet sender
        packetSender.Send(packet);
    }
}

namespace NitroxPatcher.Patches.Dynamic;

/// <summary>
/// Synchronizes base piece deconstruction across all players.
/// Uses transpiler to inject packet sending at precise points in the deconstruction flow.
/// </summary>
public sealed partial class BaseDeconstructable_Deconstruct_Patch : NitroxPatch, IDynamicPatch
{
    // Implementation...
}

## Summary
Adds synchronization for storage container contents when items are added/removed.

Fixes #123

## Changes
- Added StorageContainerSync packet
- Patched ItemsContainer.AddItem and RemoveItem
- Server broadcasts changes to other players

## Testing
Tested with multiple players adding/removing items from lockers simultaneously.

## Introduction
I noticed that when I was playing with my friend, the storage containers...
[47 more paragraphs explaining the entire history of the bug and your thought process]

// ❌ AI-generated code with excessive comments and unnecessary complexity

/// <summary>
/// This interface represents a factory that creates instances of player objects
/// in the multiplayer environment. It provides a abstraction layer for player
/// creation that allows for dependency injection and testability.
/// </summary>
public interface IPlayerFactory
{
    /// <summary>
    /// Creates a new player instance with the specified parameters.
    /// </summary>
    /// <param name="id">The unique identifier for the player</param>
    /// <param name="name">The display name of the player</param>
    /// <returns>A new instance of the Player class</returns>
    Player CreatePlayer(ushort id, string name);
}

// Just do this instead:
public interface IPlayerFactory
{
    Player CreatePlayer(ushort id, string name);
}

dotnet test Nitrox.Test/Nitrox.Test.csproj

[TestClass]
public class PacketSerializationTests
{
    [TestMethod]
    public void ChatMessage_SerializesAndDeserializes()
    {
        // Arrange
        var original = new ChatMessage(123, "Hello world");
        
        // Act
        byte[] serialized = original.Serialize();
        Packet deserialized = Packet.Deserialize(serialized);
        
        // Assert
        Assert.IsInstanceOfType(deserialized, typeof(ChatMessage));
        var chatMsg = (ChatMessage)deserialized;
        Assert.AreEqual(123, chatMsg.PlayerId);
        Assert.AreEqual("Hello world", chatMsg.Text);
    }
}

Log.Debug("Detailed debug information");
Log.Info("General information");
Log.Warn("Warning message");
Log.Error("Error occurred");
Log.Error(exception, "Error with exception");