fix: Bot private message command processing

- Bot now correctly processes private messages without "/" prefix
- Added ExecuteWithoutPrefix method to ClientCommandCollection
- Private messages (e.g., "exec calculate.ps1") now work properly
- Added comprehensive unit tests for command processing

This fixes the issue where commands like "/msg CalculatorBot exec calculate.ps1 '7 + 4'"
were received but not executed by the bot.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: stho32

Source/LocalNetAppChat/LocalNetAppChat.Bot/Program.cs

@@ -88,15 +88,16 @@ public static async Task Main(string[] args)
                     {
                         output.WriteLine(message);
 
-                        if (CommandMessageTokenizer.IsCommandMessage(message.Message.Text)) 
+                        if (IsAPrivateMessage(message))
                         {
-                            if (IsAPrivateMessage(message))
-                            {
-                                Result<string> result = privateClientCommands.Execute(message.Message.Text);
-                                await SendResultBack(lnacClient, message.Message.Name, result);
-                                continue;
-                            }
+                            // Private messages don't need the "/" prefix
+                            Result<string> result = privateClientCommands.ExecuteWithoutPrefix(message.Message.Text);
+                            await SendResultBack(lnacClient, message.Message.Name, result);
+                            continue;
+                        }
 
+                        if (CommandMessageTokenizer.IsCommandMessage(message.Message.Text)) 
+                        {
                             if (publicClientCommands.IsAKnownCommand(message.Message.Text))
                             {
                                 await SendResultBack(

Source/LocalNetAppChat/LocalNetAppChat.Domain.Tests/Bots/ClientCommands/ClientCommandCollectionTests.cs

@@ -0,0 +1,119 @@
+using LocalNetAppChat.Domain.Bots.ClientCommands;
+using NUnit.Framework;
+
+namespace LocalNetAppChat.Domain.Tests.Bots.ClientCommands
+{
+    [TestFixture]
+    public class ClientCommandCollectionTests
+    {
+        private class TestCommand : IClientCommand
+        {
+            private readonly string _keyword;
+            private readonly string _response;
+
+            public TestCommand(string keyword, string response)
+            {
+                _keyword = keyword;
+                _response = response;
+            }
+
+            public bool IsReponsibleFor(string keyword)
+            {
+                return keyword == _keyword;
+            }
+
+            public string Execute(string arguments)
+            {
+                return $"{_response} {arguments}".Trim();
+            }
+        }
+
+        [Test]
+        public void Execute_WithSlashPrefix_ExecutesCommand()
+        {
+            // Arrange
+            var collection = new ClientCommandCollection();
+            collection.Add(new TestCommand("test", "executed"));
+
+            // Act
+            var result = collection.Execute("/test arg1 arg2");
+
+            // Assert
+            Assert.IsTrue(result.IsSuccess);
+            Assert.AreEqual("executed arg1 arg2", result.Value);
+        }
+
+        [Test]
+        public void Execute_WithoutSlashPrefix_ReturnsFailure()
+        {
+            // Arrange
+            var collection = new ClientCommandCollection();
+            collection.Add(new TestCommand("test", "executed"));
+
+            // Act
+            var result = collection.Execute("test arg1 arg2");
+
+            // Assert
+            Assert.IsFalse(result.IsSuccess);
+            Assert.AreEqual("Invalid Command!", result.Error);
+        }
+
+        [Test]
+        public void ExecuteWithoutPrefix_WithoutSlashPrefix_ExecutesCommand()
+        {
+            // Arrange
+            var collection = new ClientCommandCollection();
+            collection.Add(new TestCommand("exec", "executed"));
+
+            // Act
+            var result = collection.ExecuteWithoutPrefix("exec calculate.ps1 \"7 + 4\"");
+
+            // Assert
+            Assert.IsTrue(result.IsSuccess);
+            Assert.AreEqual("executed calculate.ps1 \"7 + 4\"", result.Value);
+        }
+
+        [Test]
+        public void ExecuteWithoutPrefix_UnknownCommand_ReturnsFailure()
+        {
+            // Arrange
+            var collection = new ClientCommandCollection();
+            collection.Add(new TestCommand("exec", "executed"));
+
+            // Act
+            var result = collection.ExecuteWithoutPrefix("unknown arg1 arg2");
+
+            // Assert
+            Assert.IsFalse(result.IsSuccess);
+            Assert.AreEqual("Unknown command: unknown", result.Error);
+        }
+
+        [Test]
+        public void IsAKnownCommand_WithSlashPrefix_ReturnsTrue()
+        {
+            // Arrange
+            var collection = new ClientCommandCollection();
+            collection.Add(new TestCommand("test", "executed"));
+
+            // Act
+            var result = collection.IsAKnownCommand("/test");
+
+            // Assert
+            Assert.IsTrue(result);
+        }
+
+        [Test]
+        public void IsAKnownCommand_WithoutSlashPrefix_ReturnsFalse()
+        {
+            // Arrange
+            var collection = new ClientCommandCollection();
+            collection.Add(new TestCommand("test", "executed"));
+
+            // Act
+            var result = collection.IsAKnownCommand("test");
+
+            // Assert
+            Assert.IsFalse(result);
+        }
+    }
+}

Source/LocalNetAppChat/LocalNetAppChat.Domain/Bots/ClientCommands/ClientCommandCollection.cs

@@ -30,6 +30,22 @@ public Result<string> Execute(string command)
             return Result<string>.Failure("Invalid commmand.");
         }
 
+        public Result<string> ExecuteWithoutPrefix(string command)
+        {
+            var rest = command;
+            var keyWord = CommandMessageTokenizer.GetToken(ref rest);
+
+            foreach (var clientCommand in _clientCommands)
+            {
+                if (clientCommand.IsReponsibleFor(keyWord))
+                {
+                    return Result<string>.Success(clientCommand.Execute(rest));
+                }
+            }
+
+            return Result<string>.Failure($"Unknown command: {keyWord}");
+        }
+
         public bool IsAKnownCommand(string command)
         {
             if (!CommandMessageTokenizer.IsCommandMessage(command))

docs/scenarios/math-bots/quickstart-math-bots.md

@@ -21,19 +21,14 @@ LocalNetAppChat.Server --port 5000 --key "demo"
 
 **Terminal 2 - Emitter (Generator):**
 ```bash
-LocalNetAppChat.ConsoleClient emitter \
-  --server localhost --port 5000 --key "demo" \
-  --clientName "MathGenerator" \
-  --command "python -u python/generator-simple.py"
+LocalNetAppChat.ConsoleClient emitter --server localhost --port 5000 --key "demo" --clientName "MathGenerator" --command "python -u python/generator-simple.py"
 ```
 
 **Wichtig**: Das `-u` Flag ist essentiell! Ohne dieses Flag puffert Python die Ausgabe und der Emitter sendet nichts.
 
 **Terminal 3 - Beobachter:**
 ```bash
-LocalNetAppChat.ConsoleClient listener \
-  --server localhost --port 5000 --key "demo" \
-  --clientName "Observer"
+LocalNetAppChat.ConsoleClient listener --server localhost --port 5000 --key "demo" --clientName "Observer"
 ```
 
 ### Was Sie sehen werden
@@ -66,20 +61,17 @@ LocalNetAppChat.Server --port 5000 --key "demo"
 
 **Terminal 2 - Calculator Bot:**
 ```bash
-LocalNetAppChat.Bot --server localhost --port 5000 --key "demo" \
-  --clientName "CalculatorBot" --scriptspath "./scripts"
+LocalNetAppChat.Bot --server localhost --port 5000 --key "demo" --clientName "CalculatorBot" --scriptspath "./scripts"
 ```
 
 **Terminal 3 - Observer:**
 ```bash
-LocalNetAppChat.ConsoleClient listener --server localhost --port 5000 --key "demo" \
-  --clientName "Observer"
+LocalNetAppChat.ConsoleClient listener --server localhost --port 5000 --key "demo" --clientName "Observer"
 ```
 
 **Terminal 4 - Chat (Controller):**
 ```bash
-LocalNetAppChat.ConsoleClient chat --server localhost --port 5000 --key "demo" \
-  --clientName "Teacher"
+LocalNetAppChat.ConsoleClient chat --server localhost --port 5000 --key "demo" --clientName "Teacher"
 ```
 
 ### Bots verwenden