feat(patching): introduce combat phase extension for card model

- Added a new extension method to determine if a card model's owner is in the "Play" turn phase, improving clarity and reusability.
- Refactored existing logic in `ModCardHandOutlinePatchHelper` to utilize the new extension method for better readability and maintainability.
- Enhanced the overall structure of combat phase checks within the patching logic.

Author: BAKAOLC

Scaffolding/Cards/HandOutline/Patches/ModCardHandOutlinePatchHelper.cs

@@ -3,6 +3,7 @@
 using MegaCrit.Sts2.Core.Combat;
 using MegaCrit.Sts2.Core.Models;
 using MegaCrit.Sts2.Core.Nodes.Cards.Holders;
+using STS2RitsuLib.Scaffolding.Combat;
 
 namespace STS2RitsuLib.Scaffolding.Cards.HandOutline.Patches
 {
@@ -30,7 +31,7 @@ internal static void ApplyHighlight(NHandCardHolder holder, CardModel model, Mod
             if (CombatManager.Instance is not { IsInProgress: true })
                 return;
 
-            var inPlayPhase = model.Owner?.PlayerCombatState?.Phase == PlayerTurnPhase.Play;
+            var inPlayPhase = model.IsOwnerPlayPhase();
             var shouldGlowRed = inPlayPhase && model.ShouldGlowRed;
             var shouldGlowGold = inPlayPhase && model.CanPlay() && model.ShouldGlowGold;
             var vanillaShow = model.CanPlay() || shouldGlowRed || shouldGlowGold;
@@ -52,7 +53,7 @@ internal static void ApplyFlash(NHandCardHolder holder, CardModel model, ModCard
                 !GodotObject.IsInstanceValid(flash))
                 return;
 
-            var inPlayPhase = model.Owner?.PlayerCombatState?.Phase == PlayerTurnPhase.Play;
+            var inPlayPhase = model.IsOwnerPlayPhase();
             var shouldGlowRed = inPlayPhase && model.ShouldGlowRed;
             var shouldGlowGold = inPlayPhase && model.CanPlay() && model.ShouldGlowGold;
             var vanillaShow = model.CanPlay() || shouldGlowRed || shouldGlowGold;

Scaffolding/Combat/CombatTurnPhaseExtensions.cs

@@ -0,0 +1,25 @@
+using MegaCrit.Sts2.Core.Combat;
+using MegaCrit.Sts2.Core.Models;
+
+namespace STS2RitsuLib.Scaffolding.Combat
+{
+    /// <summary>
+    ///     Compatibility helpers for querying the current player turn phase.
+    /// </summary>
+    public static class CombatTurnPhaseExtensions
+    {
+        /// <summary>
+        ///     Returns whether <paramref name="model" />'s owner is currently in the "Play" turn phase.
+        /// </summary>
+        public static bool IsOwnerPlayPhase(this CardModel model)
+        {
+            ArgumentNullException.ThrowIfNull(model);
+
+#if !STS2_AT_LEAST_0_104_0
+            return CombatManager.Instance.IsPlayPhase;
+#else
+            return model.Owner?.PlayerCombatState?.Phase == PlayerTurnPhase.Play;
+#endif
+        }
+    }
+}