Only log keycheck header once

Author: openbullet

RuriLib.Tests/Models/Blocks/Custom/KeycheckBlockInstanceTests.cs

@@ -192,7 +192,7 @@ public void ToSyntax_NormalBlock_OutputScript()
         block.Label = "My Label";
         block.Keychains = CreateKeychains();
 
-        var expected = $"if (CheckCondition(data, myString.AsString(), StrComparison.Contains, \"abc\") || CheckCondition(data, 3F, NumComparison.GreaterThan, 1.5F)){_nl} {{ data.STATUS = \"SUCCESS\"; }}{_nl}else if (CheckCondition(data, myList.AsList(), ListComparison.Contains, \"abc\") && CheckCondition(data, myDict.AsDict(), DictComparison.HasKey, \"abc\")){_nl}  {{ data.STATUS = \"FAIL\"; return; }}{_nl}else if (myBool.AsBool()){_nl}  {{ data.STATUS = \"BAN\"; return; }}{_nl}if (CheckGlobalBanKeys(data)) {{ data.STATUS = \"BAN\"; return; }}{_nl}if (CheckGlobalRetryKeys(data)) {{ data.STATUS = \"RETRY\"; return; }}{_nl}";
+        var expected = $"data.Logger.LogHeader(\"CheckCondition\");{_nl}if (CheckCondition(data, myString.AsString(), StrComparison.Contains, \"abc\") || CheckCondition(data, 3F, NumComparison.GreaterThan, 1.5F)){_nl} {{ data.STATUS = \"SUCCESS\"; }}{_nl}else if (CheckCondition(data, myList.AsList(), ListComparison.Contains, \"abc\") && CheckCondition(data, myDict.AsDict(), DictComparison.HasKey, \"abc\")){_nl}  {{ data.STATUS = \"FAIL\"; return; }}{_nl}else if (myBool.AsBool()){_nl}  {{ data.STATUS = \"BAN\"; return; }}{_nl}if (CheckGlobalBanKeys(data)) {{ data.STATUS = \"BAN\"; return; }}{_nl}if (CheckGlobalRetryKeys(data)) {{ data.STATUS = \"RETRY\"; return; }}{_nl}";
         Assert.Equal(NormalizeSnippet(expected), RenderSyntax(block, new ConfigSettings()));
     }
 
@@ -205,14 +205,15 @@ public void ToSyntax_NoKeychains_OutputScript()
         banIfNoMatch.InputMode = SettingInputMode.Variable;
         banIfNoMatch.InputVariableName = "myBool";
 
-        var expected = $"if (myBool.AsBool()){_nl}  {{ data.STATUS = \"BAN\"; return; }}{_nl}if (CheckGlobalBanKeys(data)) {{ data.STATUS = \"BAN\"; return; }}{_nl}if (CheckGlobalRetryKeys(data)) {{ data.STATUS = \"RETRY\"; return; }}{_nl}";
+        var expected = $"data.Logger.LogHeader(\"CheckCondition\");{_nl}if (myBool.AsBool()){_nl}  {{ data.STATUS = \"BAN\"; return; }}{_nl}if (CheckGlobalBanKeys(data)) {{ data.STATUS = \"BAN\"; return; }}{_nl}if (CheckGlobalRetryKeys(data)) {{ data.STATUS = \"RETRY\"; return; }}{_nl}";
         Assert.Equal(NormalizeSnippet(expected), RenderSyntax(block, new ConfigSettings()));
     }
 
     [Fact]
     public void ToSyntax_TracksExpectedStatusFlowVariants()
     {
         AssertSyntax(CreateVariableBanBlock(), new ConfigSettings(),
+            "data.Logger.LogHeader(\"CheckCondition\")",
             "if (CheckCondition(data, ObjectExtensions.DynamicAsString(globals.myString), StrComparison.Contains, \"abc\") || CheckCondition(data, 3F, NumComparison.GreaterThan, 1.5F))",
             "data.STATUS = \"SUCCESS\";",
             "else if (CheckCondition(data, ObjectExtensions.DynamicAsList(globals.myList), ListComparison.Contains, \"abc\") && CheckCondition(data, ObjectExtensions.DynamicAsDict(input.myDict), DictComparison.HasKey, \"abc\"))",

RuriLib/Blocks/Conditions/Methods.cs

@@ -22,8 +22,6 @@ public static class Methods
     /// </summary>
     public static bool CheckCondition(BotData data, bool leftTerm, BoolComparison comparison, bool rightTerm)
     {
-        data.Logger.LogHeader();
-
         var result = RuriLib.Functions.Conditions.Conditions.Check(leftTerm, comparison, rightTerm);
 
         if (result)
@@ -39,8 +37,6 @@ public static bool CheckCondition(BotData data, bool leftTerm, BoolComparison co
     /// </summary>
     public static bool CheckCondition(BotData data, string leftTerm, StrComparison comparison, string rightTerm)
     {
-        data.Logger.LogHeader();
-
         var result = RuriLib.Functions.Conditions.Conditions.Check(leftTerm, comparison, rightTerm);
 
         if (result)
@@ -56,8 +52,6 @@ public static bool CheckCondition(BotData data, string leftTerm, StrComparison c
     /// </summary>
     public static bool CheckCondition(BotData data, List<string> leftTerm, ListComparison comparison, string rightTerm)
     {
-        data.Logger.LogHeader();
-
         var result = RuriLib.Functions.Conditions.Conditions.Check(leftTerm, comparison, rightTerm);
 
         if (result)
@@ -73,8 +67,6 @@ public static bool CheckCondition(BotData data, List<string> leftTerm, ListCompa
     /// </summary>
     public static bool CheckCondition(BotData data, int leftTerm, NumComparison comparison, int rightTerm)
     {
-        data.Logger.LogHeader();
-
         var result = RuriLib.Functions.Conditions.Conditions.Check(leftTerm, comparison, rightTerm);
 
         if (result)
@@ -90,8 +82,6 @@ public static bool CheckCondition(BotData data, int leftTerm, NumComparison comp
     /// </summary>
     public static bool CheckCondition(BotData data, float leftTerm, NumComparison comparison, float rightTerm)
     {
-        data.Logger.LogHeader();
-
         var result = RuriLib.Functions.Conditions.Conditions.Check(leftTerm, comparison, rightTerm);
 
         if (result)
@@ -107,8 +97,6 @@ public static bool CheckCondition(BotData data, float leftTerm, NumComparison co
     /// </summary>
     public static bool CheckCondition(BotData data, Dictionary<string, string> leftTerm, DictComparison comparison, string rightTerm)
     {
-        data.Logger.LogHeader();
-
         var result = RuriLib.Functions.Conditions.Conditions.Check(leftTerm, comparison, rightTerm);
 
         if (result)
@@ -124,8 +112,6 @@ public static bool CheckCondition(BotData data, Dictionary<string, string> leftT
     /// </summary>
     public static bool CheckGlobalBanKeys(BotData data)
     {
-        data.Logger.LogHeader();
-
         var result = data.Providers.ProxySettings.ContainsBanKey(data.SOURCE, out var matchedKey);
 
         if (result)
@@ -141,8 +127,6 @@ public static bool CheckGlobalBanKeys(BotData data)
     /// </summary>
     public static bool CheckGlobalRetryKeys(BotData data)
     {
-        data.Logger.LogHeader();
-
         var result = data.Providers.ProxySettings.ContainsRetryKey(data.SOURCE, out var matchedKey);
 
         if (result)

RuriLib/Models/Blocks/Custom/KeycheckBlockInstance.cs

@@ -153,7 +153,13 @@ public override IEnumerable<StatementSyntax> ToSyntax(BlockSyntaxGenerationConte
     {
         ArgumentNullException.ThrowIfNull(context);
 
-        var statements = new List<StatementSyntax>();
+        var statements = new List<StatementSyntax>
+        {
+            BlockSyntaxFactory.CreateMemberInvocation(
+                Expr("data.Logger"),
+                "LogHeader",
+                Arg(Lit("CheckCondition"))).Stmt()
+        };
         var banIfNoMatch = Settings["banIfNoMatch"];
         var nonEmpty = Keychains.Where(kc => kc.Keys.Count > 0).ToList();
         var continueBan = context.Settings.GeneralSettings.ContinueStatuses.Contains("BAN");