Implemented initial stages on BZ integration

.editorconfig

@@ -7,11 +7,48 @@ indent_style = space
 end_of_line = lf
 
 # ReSharper properties
+resharper_blank_lines_after_block_statements = 0
+resharper_braces_for_ifelse = required_for_multiline
+resharper_braces_redundant = false
+resharper_keep_existing_attribute_arrangement = true
 resharper_wrap_object_and_collection_initializer_style = chop_if_long
 
-[*.{proj, csproj, targets}]
+[*.{proj,csproj,props,targets}]
 indent_size = 4
 insert_final_newline = true
+tab_width = 4
+dotnet_style_operator_placement_when_wrapping = beginning_of_line
+dotnet_style_readonly_field = true:suggestion
+dotnet_style_require_accessibility_modifiers = for_non_interface_members:silent
+dotnet_style_coalesce_expression = true:suggestion
+dotnet_style_null_propagation = true:suggestion
+dotnet_style_prefer_is_null_check_over_reference_equality_method = true:suggestion
+dotnet_style_prefer_auto_properties = true:silent
+dotnet_style_object_initializer = true:suggestion
+dotnet_style_prefer_collection_expression = true:suggestion
+dotnet_style_namespace_match_folder = true:suggestion
+dotnet_style_collection_initializer = true:suggestion
+dotnet_style_prefer_simplified_boolean_expressions = true:suggestion
+dotnet_style_prefer_conditional_expression_over_assignment = true:silent
+dotnet_style_prefer_conditional_expression_over_return = true:silent
+dotnet_style_explicit_tuple_names = true:suggestion
+dotnet_style_prefer_inferred_tuple_names = true:suggestion
+dotnet_style_prefer_inferred_anonymous_type_member_names = true:suggestion
+dotnet_style_prefer_compound_assignment = true:suggestion
+dotnet_style_prefer_simplified_interpolation = true:suggestion
+dotnet_style_allow_multiple_blank_lines_experimental = true:silent
+dotnet_style_allow_statement_immediately_after_block_experimental = true:silent
+dotnet_code_quality_unused_parameters = all:suggestion
+dotnet_style_parentheses_in_arithmetic_binary_operators = always_for_clarity:silent
+dotnet_style_parentheses_in_other_binary_operators = always_for_clarity:silent
+dotnet_style_parentheses_in_relational_binary_operators = always_for_clarity:silent
+dotnet_style_parentheses_in_other_operators = never_if_unnecessary:silent
+dotnet_style_qualification_for_field = false:silent
+dotnet_style_qualification_for_property = false:silent
+dotnet_style_qualification_for_method = false:silent
+dotnet_style_qualification_for_event = false:silent
+dotnet_style_predefined_type_for_locals_parameters_members = true:silent
+dotnet_style_predefined_type_for_member_access = true:silent
 
 [App.config]
 indent_size = 4
@@ -21,6 +58,54 @@ insert_final_newline = true
 indent_size = 4
 insert_final_newline = true
 charset = utf-8
+csharp_space_around_binary_operators = before_and_after
+csharp_indent_labels = one_less_than_current
+csharp_style_expression_bodied_methods = false:silent
+csharp_style_expression_bodied_constructors = false:silent
+csharp_style_expression_bodied_operators = false:silent
+csharp_style_expression_bodied_properties = true:silent
+csharp_style_expression_bodied_indexers = true:silent
+csharp_style_expression_bodied_accessors = true:silent
+csharp_style_expression_bodied_lambdas = true:silent
+csharp_style_expression_bodied_local_functions = false:silent
+csharp_style_prefer_switch_expression = true:suggestion
+csharp_style_prefer_pattern_matching = true:silent
+csharp_style_pattern_matching_over_is_with_cast_check = true:suggestion
+csharp_style_pattern_matching_over_as_with_null_check = true:suggestion
+csharp_style_prefer_not_pattern = true:suggestion
+csharp_style_prefer_extended_property_pattern = true:suggestion
+csharp_prefer_simple_using_statement = true:suggestion
+csharp_prefer_braces = true:silent
+csharp_style_namespace_declarations = file_scoped:suggestion
+csharp_style_prefer_method_group_conversion = true:silent
+csharp_style_prefer_top_level_statements = true:silent
+csharp_style_prefer_primary_constructors = true:suggestion
+csharp_prefer_static_local_function = true:suggestion
+csharp_style_prefer_readonly_struct = true:suggestion
+csharp_style_prefer_readonly_struct_member = true:suggestion
+csharp_style_throw_expression = true:suggestion
+csharp_style_prefer_null_check_over_type_check = true:suggestion
+csharp_prefer_simple_default_expression = true:suggestion
+csharp_style_prefer_local_over_anonymous_function = true:suggestion
+csharp_style_prefer_index_operator = true:suggestion
+csharp_style_prefer_range_operator = true:suggestion
+csharp_style_implicit_object_creation_when_type_is_apparent = true:suggestion
+csharp_style_prefer_tuple_swap = true:suggestion
+csharp_style_prefer_utf8_string_literals = true:suggestion
+csharp_style_inlined_variable_declaration = true:suggestion
+csharp_style_deconstructed_variable_declaration = true:suggestion
+csharp_style_unused_value_assignment_preference = discard_variable:suggestion
+csharp_style_unused_value_expression_statement_preference = discard_variable:silent
+csharp_style_allow_embedded_statements_on_same_line_experimental = true:silent
+csharp_style_allow_blank_lines_between_consecutive_braces_experimental = true:silent
+csharp_style_allow_blank_line_after_colon_in_constructor_initializer_experimental = true:silent
+csharp_style_allow_blank_line_after_token_in_conditional_expression_experimental = true:silent
+csharp_style_allow_blank_line_after_token_in_arrow_expression_clause_experimental = true:silent
+csharp_style_var_for_built_in_types = false:silent
+csharp_style_var_when_type_is_apparent = false:silent
+csharp_style_var_elsewhere = false:silent
+csharp_style_conditional_delegate_call = true:suggestion
+csharp_using_directive_placement = outside_namespace:silent
 
 [*.g.cs]
 generated_code = true

.github/ISSUE_TEMPLATE/bug_report.yaml

@@ -24,7 +24,7 @@ body:
     id: os_type
     attributes:
       label: Which OS are you playing on?
-      description: Please select your Operation System from the dropdown.
+      description: Please select your Operating System from the dropdown.
       options:
         - Windows
         - Linux (currently not supported, don't file any reports if your are using this)
@@ -55,7 +55,7 @@ body:
     id: steps_to_reproduce
     attributes:
       label: Describe the steps you took to encounter the bug.
-      description: How did it happened? Please provide steps to reproduce the bug if possible. Did the bug disappear with slightly altered steps?
+      description: How did it happen? Please provide steps to reproduce the bug if possible. Did the bug disappear with slightly altered steps?
       placeholder: Tell us what you did! 
     validations:
       required: false
@@ -65,7 +65,7 @@ body:
       label: Relevant log output
       description: |
         Please upload your game logs if possible. The logs are located next to the launcher in a folder called "Nitrox Logs".
-        Upload the most recent "game-XXXXXXXX.log" and if you hosted the server also "server-XXXXXXXX.log" to [Pastbin](https://pastebin.com/).
+        Upload the most recent "game-XXXXXXXX.log" and if you hosted the server also "server-XXXXXXXX.log" to [Pastebin](https://pastebin.com/).
         After you created a paste for both files post their links into this box.
       placeholder: |
         For example:

CONTRIBUTING.md

@@ -1,56 +1,3 @@
 # Contributing
 
-## Code
-We do not have a formatting guideline *yet*, and expect you to write code with common sense in mind. Consistency is key, so look at the surrounding code and adhere to the same standards. Always check the diff before you commit (and more importantly: before filing a PR); many irrelevant changes usually mean your autoformat settings are off. Luckily, our project now includes a `.editorconfig` that automatically sets all the built-in formatting/language-rules. This is however only supported in Visual Studio 2017.
-A large list of discussion items has been compiled [here](https://github.com/SubnauticaNitrox/Nitrox/issues/36), which hopefully make it into a guideline soon™.
-
-### Formatting
-- Use UpperCamelCase for methods, classes, properties and public fields.
-- Use lowerCamelCase for local variables, private fields and function parameters.
-- Remove unused `using` statements (Visual Studio has a shortcut to do this, as well as sort them).
-- Put opening braces (`{`) on their own line.
-- Encapsulate everything in braces, even single-line statements.
-- Use `Optional<T>` for explicit nullable types.
-- Use strict access modifiers where possible, and mark immutable variables wherever appropriate (`const`, `readonly` or `static readonly`, depending on the usecase).
-- Remove `private set;` from properties if these are only assigned in the constructor.
-- Assign simple data types (that do not require arguments) at declaration rather than in the constructor (such as `private readonly List<int> someList = new List<int>();`, instead of putting the `new` part in the constructor).
-
-Again, consistency is key! If you're unsure, look at the rest of the code!
-
-### Safe guidelines
-When retrieving data from the game, use `Validate.NotNull` or similar where possible. Subnautica is still heavily under development, and things *will* change. Defensive coding standards help identify and fix these changes more easily.
-To reduce redundant if sturctures, there are several helper functions that combine a certain retrieval function with a `Validation` check - it's recommended to use these. For example `GameObject.RequireComponent` and `GuidHelper.RequireObjectFrom`. In case of `Optional<T>` return types, these are unwrapped for you as well, saving the redundant checks *and* unwrapping.
-
-### Help your fellow developers!
-If code (especially patches that touch game internals) is/are not immediately clear, add a comment explaining the situation. This goes a long way, as you're basically presenting an overview of the research you did in the game code. And it helps others to figure out what's supposed to happen, in case the game changes and the patch is not functioning properly anymore.
-
-### General OO(P) concepts
-... such as "low coupling" and "high coherency", which basically means that you should try to reduce interaction between different pieces of code (classes) as much as possible (low coupling), and make sure that these pieces of code (classes) do the thing they are meant to do, and nothing else (high coherency). This goes a long way in code maintainability; less spaghetti code is always better :wink:
-Also in this case, keep common sense in mind: if you're constantly accessing a lot of fields and methods of another class, consider tucking it away in a method in that class.
-
-## Git workflow
-### Git help
-<sub><sup>Skip this section is if you are familiar with git.</sup></sub>
-Git is there to help you develop and share code efficiently, though it may seem daunting at first. In order to use it effectively, we recommend you to read some tutorials so that you are familiar with the concept of commits, the way these are linked together, branches, and remotes.
-
-When cloning a repository, git adds the clone url as a remote named `origin`. In this guide, we assume that's your fork. To keep your repository updated, add this official repo to your remotes:
-```bash
-# SSH:
-git remote add upstream git@github.com:SubnauticaNitrox/Nitrox.git
-# HTTPS:
-git remote add upstream https://github.com/SubnauticaNitrox/Nitrox.git
-```
-
-We recommend to keep your master branch up to date with the offical master branch. This makes it easier to base feature branches on.
-To go even further, you configure your master branch to pull from `SubnauticaNitrox`, and push to your own fork:
-```bash
-git config branch.master.remote upstream
-git config branch.master.pushRemote origin
-```
-
-### Filing a PR
-When filing a PR, we obviously expect the code to compile, run with no errors, and merge without conflicts.
-To prevent these, and ensure that your code is compatible with the most recent 'version',
-merge master into your branch, or rebase your branch on top of master. Even if git(hub) says your code can be merged without conflicts, there might be structural changes (renamings, moved files, refactors, etc), causing the final result to fail compilation, or break at runtime.
-
-It is not desired to remove code just because "it doesn't work", or "causes exceptions in the log". If that's the case, try to fix it (recommended to file the changes in a separate PR), or notify the other Nitrox devs (by creating an issue on github, for example). All code is there for a reason - and someone else spent time creating it.
+Please have a look at our [documentation site](https://subnauticanitrox.github.io/Documentation/) for contribution guidelines & information on how to get started.

Directory.Build.props

@@ -1,7 +1,7 @@
 <Project>
     <!-- Set default properties for all projects (can be overridden per project) -->
     <PropertyGroup>
-        <LangVersion>11</LangVersion>
+        <LangVersion>12.0</LangVersion>
         <Version>1.8.0.0</Version>
         <NitroxProject>false</NitroxProject>
         <TestLibrary>false</TestLibrary>
@@ -12,16 +12,18 @@
         <BuildGenDllDir>$(BuildGenDir)publicized_assemblies\</BuildGenDllDir>
         <AllowUnsafeBlocks>true</AllowUnsafeBlocks>
         <Prefer32Bit>false</Prefer32Bit>
+        <DebugType>embedded</DebugType>
         <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
         <ImplicitUsings>disable</ImplicitUsings>
         <PathMap>$(MSBuildProjectDirectory)=$(MSBuildProjectName)</PathMap>
         <EnableNETAnalyzers>true</EnableNETAnalyzers>
+        <PolySharpUsePublicAccessibilityForGeneratedTypes>true</PolySharpUsePublicAccessibilityForGeneratedTypes>
     </PropertyGroup>
 
     <PropertyGroup Condition="$([System.Text.RegularExpressions.Regex]::IsMatch($(MSBuildProjectName), '^Nitrox.*$'))">
         <NitroxProject>true</NitroxProject>
     </PropertyGroup>
-    <PropertyGroup Condition="'$(NitroxProject)' == 'true' and '$(MSBuildProjectName)' != 'Nitrox.Analyzers'">
+    <PropertyGroup Condition="'$(NitroxProject)' == 'true'">
         <NitroxLibrary>true</NitroxLibrary>
     </PropertyGroup>
     <PropertyGroup Condition="'$(NitroxLibrary)' == 'true' and '$(MSBuildProjectName)' != 'NitroxModel' and '$(MSBuildProjectName)' != 'NitroxServer' and '$(MSBuildProjectName)' != 'Nitrox.BuildTool'">
@@ -30,27 +32,27 @@
     <PropertyGroup Condition="'$(MSBuildProjectName)' == 'Nitrox.Test'">
         <TestLibrary>true</TestLibrary>
     </PropertyGroup>
+    <PropertyGroup Condition=" '$(Configuration)' == 'BelowZero' ">
+        <DefineConstants>DEBUG;BELOWZERO</DefineConstants>
+        <OutputPath>bin\BelowZero\</OutputPath>
+    </PropertyGroup>
+
 
     <!-- Shared dependencies for all Nitrox.* projects -->
     <Choose>
         <When Condition="'$(NitroxProject)' == 'true'">
             <ItemGroup>
-                <PackageReference Include="JetBrains.Annotations" Version="2022.3.1">
+                <PackageReference Include="JetBrains.Annotations" Version="2023.3.0">
                     <Aliases>JB</Aliases>
                 </PackageReference>
-                <PackageReference Include="IsExternalInit" Version="1.0.3">
-                    <PrivateAssets>all</PrivateAssets>
-                    <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
-                </PackageReference>
-            </ItemGroup>
-        </When>
-    </Choose>
 
-    <!-- Include our analyzer and code gen library to all Nitrox projects -->
-    <Choose>
-        <When Condition="'$(NitroxLibrary)' == 'true'">
-            <ItemGroup>
-                <ProjectReference Include="..\Nitrox.Analyzers\Nitrox.Analyzers.csproj" OutputItemType="Analyzer" ReferenceOutputAssembly="false" />
+              <PackageReference Include="PolySharp" Version="1.14.1">
+                <PrivateAssets>all</PrivateAssets>
+                <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
+              </PackageReference>
+
+              <PackageReference Include="System.Memory" Version="4.5.5" />
+              <PackageReference Include="System.ValueTuple" Version="4.5.0" />
             </ItemGroup>
         </When>
     </Choose>
@@ -67,6 +69,10 @@
                 <ProjectReference Include="$(SolutionDir)NitroxModel\NitroxModel.csproj">
                     <Name>NitroxModel</Name>
                 </ProjectReference>
+                <PackageReference Include="Nitrox.Analyzers" Version="*">
+                    <PrivateAssets>all</PrivateAssets>
+                    <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+                </PackageReference>
             </ItemGroup>
         </When>
     </Choose>

Directory.Build.targets

@@ -4,6 +4,10 @@
         <When Condition="'$(UnityModLibrary)' == 'true' or '$(TestLibrary)' == 'true'">
             <!-- UnityEngine libraries to include -->
             <ItemGroup>
+                <Reference Include="FMODUnity">
+                    <HintPath>$(GameManagedDir)\FMODUnity.dll</HintPath>
+                    <Private>$(TestLibrary)</Private>
+                </Reference>
                 <Reference Include="Newtonsoft.Json">
                     <HintPath>$(GameManagedDir)\Newtonsoft.Json.dll</HintPath>
                     <Private>$(TestLibrary)</Private>