Allow remainingDays to be null - it is not always reported.

DataDefinitions/ThargoidWar.cs

@@ -10,7 +10,7 @@ public class ThargoidWar
         public FactionState CurrentState { get; set; }
 
         [PublicAPI( "The number of remaining days in the current phase of the Thargoid war." )]
-        public int remainingDays { get; set; }
+        public int? remainingDays { get; set; }
 
         [PublicAPI( "The next phase of the Thargoid war, if success is not achieved, as a localizable object." )]
         public FactionState FailureState { get; set; }

EDDI/ChangeLog.md

@@ -12,6 +12,9 @@ Full details of the variables available for each noted event, and VoiceAttack in
     * Revised speech service to make rendering even more thread-safe when transitioning between the `Play()` function and normal speech (#2549).
     * Revised speech service to retry using speech stripped of SSML tags when speech fails using the System.Speech synthesizer.
     * Worked around a journal bug when `HullHealth` is not written to the player journal `Loadout` event. (#2552)
+  * Speech Responder
+    * Changes to existing scripts
+      * `System state report` updated to allow thargoid war `remainingDays` to be null (since for some states it is not present in the journal output).
 
 ## 4.0.3-b3
   * Core

Tests/JournalMonitorTests.cs

@@ -1070,7 +1070,7 @@ public void TestJumpedEventThargoidProbing ()
             Assert.AreEqual( FactionState.ThargoidControlled, @event.ThargoidWar.FailureState );
             Assert.IsTrue( @event.ThargoidWar.succeeded );
             Assert.AreEqual( 100M, @event.ThargoidWar.progress );
-            Assert.AreEqual( 0, @event.ThargoidWar.remainingDays );
+            Assert.AreEqual( null, @event.ThargoidWar.remainingDays );
             Assert.IsNull( @event.ThargoidWar.remainingPorts );
         }