Merge pull request #9 from MaestroError/chat-keys-management

Fix chat clear command

Author: MaestroError

README.md

@@ -491,7 +491,7 @@ public function lastMessage(): ?MessageInterface;
 /**
  * Get all chat keys associated with this agent class
  */
-public function getChatKeys(): array
+public function getChatKeys(): array;
 ```
 
 ### Tools

src/Commands/AgentChatClearCommand.php

@@ -31,7 +31,9 @@ public function handle()
         if (!empty($chatKeys)) {
             // Clear each chat history
             foreach ($chatKeys as $key) {
-                $agent = $agentClass::for($key);
+                // Create new chat history with save_chat_keys disabled
+                $agent->createChatHistory($key);
+                // Clear messages
                 $agent->clear();
             }
         }

tests/Commands/AgentChatClearCommandTest.php

@@ -72,13 +72,17 @@ protected function onInitialize()
     $agent->message('Hello')->respond();
 
     // Verify chat history exists
-    expect($agent->getChatKeys())->not->toBeEmpty();
+    $chatKeys = $agent->getChatKeys();
+    expect($chatKeys)->toHaveCount(1);
+    expect($chatKeys)->toContain('TestAgent_gpt-4-mini_test_key');
 
     // Clear the history
     $this->artisan('agent:chat:clear', ['agent' => 'TestAgent'])
         ->assertSuccessful()
         ->expectsOutput('Successfully cleared chat history for agent: TestAgent');
 
-    // Verify chat history is cleared
-    expect(\App\AiAgents\TestAgent::for('test_key')->chatHistory()->getMessages())->toBeEmpty();
+    // Verify all chat histories are cleared but keys remain
+    $agent = \App\AiAgents\TestAgent::for('test_key');
+    expect($agent->chatHistory()->getMessages())->toBeEmpty();
+    expect($agent->getChatKeys())->toHaveCount(1);
 });