code quality

Author: KSemenenko

SDK/dotnet/src/ManagedCode.Tps/Internal/TpsContentCompiler.cs

@@ -5,7 +5,7 @@ namespace ManagedCode.Tps.Internal;
 
 internal sealed class TpsContentCompiler
 {
-    public ContentCompilationResult Compile(
+    public static ContentCompilationResult Compile(
         string rawText,
         int startOffset,
         InheritedFormattingState inherited,

SDK/dotnet/src/ManagedCode.Tps/Internal/TpsParser.cs

@@ -7,7 +7,7 @@ namespace ManagedCode.Tps.Internal;
 
 internal sealed partial class TpsParser
 {
-    public DocumentAnalysis Parse(string source)
+    public static DocumentAnalysis Parse(string source)
     {
         var normalized = TpsSupport.NormalizeLineEndings(source);
         var lineStarts = TpsSupport.CreateLineStarts(normalized);

SDK/dotnet/src/ManagedCode.Tps/ManagedCode.Tps.csproj

@@ -4,9 +4,14 @@
     <TargetFramework>net10.0</TargetFramework>
     <ImplicitUsings>enable</ImplicitUsings>
     <Nullable>enable</Nullable>
+    
     <AssemblyName>ManagedCode.Tps</AssemblyName>
     <RootNamespace>ManagedCode.Tps</RootNamespace>
     <PackageId>ManagedCode.Tps</PackageId>
+
+    <EnableNETAnalyzers>true</EnableNETAnalyzers>
+    <CodeAnalysisTreatWarningsAsErrors>true</CodeAnalysisTreatWarningsAsErrors>
+    <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
   </PropertyGroup>
 
 </Project>

SDK/dotnet/src/ManagedCode.Tps/TpsPlayer.cs

@@ -33,13 +33,13 @@ public PlayerState GetState(int elapsedMs)
         var clampedElapsed = Math.Clamp(elapsedMs, 0, Script.TotalDurationMs);
         var currentWord = FindCurrentWord(clampedElapsed);
         var currentSegment = currentWord is null
-            ? Script.Segments.FirstOrDefault()
+            ? Script.Segments.Count > 0 ? Script.Segments[0] : null
             : _segmentById.GetValueOrDefault(currentWord.SegmentId);
         var currentBlock = currentWord is null
-            ? currentSegment?.Blocks.FirstOrDefault()
+            ? currentSegment is not null && currentSegment.Blocks.Count > 0 ? currentSegment.Blocks[0] : null
             : _blockById.GetValueOrDefault(currentWord.BlockId);
         var currentPhrase = currentWord is null
-            ? currentBlock?.Phrases.FirstOrDefault()
+            ? currentBlock is not null && currentBlock.Phrases.Count > 0 ? currentBlock.Phrases[0] : null
             : _phraseById.GetValueOrDefault(currentWord.PhraseId);
         var currentWordIndex = currentWord?.Index ?? -1;
         var previousWord = currentWordIndex > 0 ? Script.Words[currentWordIndex - 1] : null;

SDK/dotnet/src/ManagedCode.Tps/TpsRuntime.cs

@@ -7,7 +7,7 @@ public static class TpsRuntime
 {
     public static TpsValidationResult Validate(string source)
     {
-        var analysis = new TpsParser().Parse(source);
+        var analysis = TpsParser.Parse(source);
         CompileAnalysis(analysis);
         return new TpsValidationResult
         {
@@ -18,7 +18,7 @@ public static TpsValidationResult Validate(string source)
 
     public static TpsParseResult Parse(string source)
     {
-        var analysis = new TpsParser().Parse(source);
+        var analysis = TpsParser.Parse(source);
         CompileAnalysis(analysis);
         return new TpsParseResult
         {
@@ -30,7 +30,7 @@ public static TpsParseResult Parse(string source)
 
     public static TpsCompilationResult Compile(string source)
     {
-        var analysis = new TpsParser().Parse(source);
+        var analysis = TpsParser.Parse(source);
         var script = CompileAnalysis(analysis);
         return new TpsCompilationResult
         {
@@ -144,7 +144,7 @@ private static BlockCandidate CompileBlock(BlockDefinition definition, Inherited
             inherited.SpeedOffsets,
             blockArchetype);
 
-        var content = new TpsContentCompiler().Compile(
+        var content = TpsContentCompiler.Compile(
             definition.Content?.Text ?? string.Empty,
             definition.Content?.StartOffset ?? 0,
             blockInherited,