fix: prevent waitForKey from blocking in non-interactive mode

Add non-interactive mode check to waitForKey() method to prevent infinite
loops when readKey() returns null in test/CI environments. This fixes
timeouts and blocking input issues that were occurring in tests.

Note: PHPUnit version temporarily downgraded to 11.5.

Author: tommyknocker

composer.json

@@ -127,6 +127,7 @@
     "ext-pdo": "*",
     "psr/log": "^3.0",
     "psr/simple-cache": "^3.0",
+
     "psr/event-dispatcher": "^1.0"
   },
   "suggest": {
@@ -145,7 +146,7 @@
     "ext-pdo_oci": "Required for Oracle database support"
   },
   "require-dev": {
-    "phpunit/phpunit": "^12.0",
+    "phpunit/phpunit": "^11.5.0",
     "monolog/monolog": "^3.9",
     "symfony/cache": "^7.3",
     "phpstan/phpstan": "^2.0",

src/ai/mcp/tools/AnalyzeQueryTool.php

@@ -65,6 +65,7 @@ public function execute(array $arguments): string|array
         try {
             // Get EXPLAIN plan for better AI analysis
             $explainAnalysis = null;
+
             try {
                 $connection = $this->db->connection;
                 $dialect = $connection->getDialect();

src/cli/commands/AiCommand.php

@@ -70,6 +70,7 @@ protected function analyze(): int
             $pdo = $connection->getPdo();
 
             $explainAnalysis = null;
+
             try {
                 $explainResults = $dialect->executeExplain($pdo, $sql, []);
                 $queryBuilder = $db->find();

src/cli/ui/InputHandler.php

@@ -245,6 +245,23 @@ protected static function parseEscapeSequence(string $seq): string
      */
     public static function waitForKey(array $allowedKeys = []): string
     {
+        // Check for non-interactive mode first (fastest check)
+        $nonInteractive = getenv('PDODB_NON_INTERACTIVE') !== false
+            || getenv('PHPUNIT') !== false;
+
+        // Only check stream_isatty if not already in non-interactive mode (expensive check)
+        if (!$nonInteractive) {
+            $nonInteractive = !stream_isatty(STDIN);
+        }
+
+        if ($nonInteractive) {
+            // In non-interactive mode, return empty string or first allowed key
+            if (!empty($allowedKeys)) {
+                return $allowedKeys[0];
+            }
+            return '';
+        }
+
         while (true) {
             $key = self::readKey(1000000); // 1 second timeout, but will loop