From 477c121f72d1a10b925fb018ccd8067a47445660 Mon Sep 17 00:00:00 2001
From: alogindtractor <251821224+A-Log-In-D-Tractor@users.noreply.github.com>
Date: Thu, 5 Feb 2026 21:26:11 -0800
Subject: [PATCH] Refactor MinesCommand regex patterns and messages (#62)

---
 .../Commands/Kasino/MinesCommand.cs           | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/KfChatDotNetBot/Commands/Kasino/MinesCommand.cs b/KfChatDotNetBot/Commands/Kasino/MinesCommand.cs
index 4ad3712..ee929ce 100644
--- a/KfChatDotNetBot/Commands/Kasino/MinesCommand.cs
+++ b/KfChatDotNetBot/Commands/Kasino/MinesCommand.cs
@@ -12,14 +12,14 @@ namespace KfChatDotNetBot.Commands.Kasino;
 public class MinesCommand : ICommand
 {
     public List<Regex> Patterns => [
-        //attempting to continue a game below here
-        new Regex(@"^mines (?<betString>.+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase),                                                       
-        new Regex(@"^mines (?<picks>\d+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase),                                                               
         //attempting to start a game below here
+        new Regex(@"^mines (?<bet>\d+\.\d+) (?<size>\d+) (?<mines>\d+) (?<picks>\d+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase), 
+        new Regex(@"^mines (?<bet>\d+) (?<size>\d+) (?<mines>\d+) (?<picks>\d+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase),  
         new Regex(@"^mines (?<bet>\d+\.\d+) (?<size>\d+) (?<mines>\d+) (?<betString>.+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase),        
-        new Regex(@"^mines (?<bet>\d+) (?<size>\d+) (?<mines>\d+) (?<betString>.+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase),                          
-        new Regex(@"^mines (?<bet>\d+\.\d+) (?<size>\d+) (?<mines>\d+) (?<picks>\d+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase),           
-        new Regex(@"^mines (?<bet>\d+) (?<size>\d+) (?<mines>\d+) (?<picks>\d+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase),                                 
+        new Regex(@"^mines (?<bet>\d+) (?<size>\d+) (?<mines>\d+) (?<betString>.+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase),        
+        //attempting to continue a game below here
+        new Regex(@"^mines (?<picks>\d+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase), 
+        new Regex(@"^mines (?<betString>.+) (?<cashout>cashout|)$", RegexOptions.IgnoreCase),                              
         //cashout
         new Regex(@"^mines (?<cashout>cashout)$", RegexOptions.IgnoreCase),                                                                     
         //refresh
@@ -73,7 +73,7 @@ public class MinesCommand : ICommand
             if (arguments.TryGetValue("refresh", out var refresh))
             {
                 await botInstance.SendChatMessageAsync(
-                    $"{user.FormatUsername()}, you don't have a game running. !mines <bet> <board size> <number of mines> <picks> to play simple mines. !mines <bet> <board size> <number of mines> <betString> for advanced mines. Tool: {ToolUrl}",
+                    $"{user.FormatUsername()}, you tried to refresh but don't have a game running. !mines <bet> <board size> <number of mines> <picks> to play simple mines. !mines <bet> <board size> <number of mines> <betString> for advanced mines. Tool: {ToolUrl}",
                     true, autoDeleteAfter: cleanupDelay);
                 return;
             }
@@ -81,7 +81,7 @@ public class MinesCommand : ICommand
             if (!arguments.TryGetValue("bet", out var bet))
             {
                 await botInstance.SendChatMessageAsync(
-                    $"{user.FormatUsername()}, not enough arguments. !mines <bet> <board size> <number of mines> <picks> to play simple mines. !mines <bet> <board size> <number of mines> <betString> for advanced mines. Tool: {ToolUrl}",
+                    $"{user.FormatUsername()}, not enough arguments(bet+). !mines <bet> <board size> <number of mines> <picks> to play simple mines. !mines <bet> <board size> <number of mines> <betString> for advanced mines. Tool: {ToolUrl}",
                     true, autoDeleteAfter: cleanupDelay);
                 return;
             }
@@ -95,7 +95,7 @@ public class MinesCommand : ICommand
             if (!arguments.TryGetValue("size", out var size) || !arguments.TryGetValue("mines", out var mines))
             {
                 await botInstance.SendChatMessageAsync(
-                    $"{user.FormatUsername()}, not enough arguments. !mines <bet> <board size> <number of mines> <picks> to play simple mines. !mines <bet> <board size> <number of mines> <betString> for advanced mines. Tool: {ToolUrl}",
+                    $"{user.FormatUsername()}, not enough arguments(mines and or size+). !mines <bet> <board size> <number of mines> <picks> to play simple mines. !mines <bet> <board size> <number of mines> <betString> for advanced mines. Tool: {ToolUrl}",
                     true, autoDeleteAfter: cleanupDelay);
                 return;
             }
@@ -123,7 +123,7 @@ public class MinesCommand : ICommand
             else //if they didn't put anything
             {
                 await botInstance.SendChatMessageAsync(
-                    $"{user.FormatUsername()}, not enough arguments. !mines <bet> <board size> <number of mines> <picks> to play simple mines. !mines <bet> <board size> <number of mines> <betString> for advanced mines. Tool: {ToolUrl}",
+                    $"{user.FormatUsername()}, not enough arguments(picks or betstring). !mines <bet> <board size> <number of mines> <picks> to play simple mines. !mines <bet> <board size> <number of mines> <betString> for advanced mines. Tool: {ToolUrl}",
                     true, autoDeleteAfter: cleanupDelay);
                 return;
             }