fix: Fix Oracle CLI examples failing in GitHub Actions

- Fix BaseCliCommand::buildConfigFromEnv() to check PDODB_SERVICE_NAME/PDODB_SID for Oracle instead of PDODB_DATABASE
- Add PDODB_SERVICE_NAME and PDODB_SID to environment variables list in buildConfigFromEnv()
- Add setEnvFromConfig() helper function to set environment variables from config array
- Update Oracle examples (04-07) to use setEnvFromConfig() for CLI commands
- Fix 04-dump-restore.php to set SQLite PDODB_PATH before calling setEnvFromConfig()
- Fix 01-ddl.php to handle foreign key constraints for MySQL/MariaDB

This fixes the issue where Oracle examples failed in GitHub Actions because
environment variables were set but buildConfigFromEnv() required PDODB_DATABASE
which Oracle doesn't use (it uses PDODB_SERVICE_NAME or PDODB_SID instead).

Author: tommyknocker

examples/11-schema/01-ddl.php

@@ -21,7 +21,14 @@
 $schema = $db->schema();
 
 // Drop table if exists (cleanup)
+// For MySQL/MariaDB, disable foreign key checks to avoid constraint violations
+if ($driver === 'mysql' || $driver === 'mariadb') {
+    $db->rawQuery("SET FOREIGN_KEY_CHECKS=0");
+}
 $schema->dropTableIfExists('users');
+if ($driver === 'mysql' || $driver === 'mariadb') {
+    $db->rawQuery("SET FOREIGN_KEY_CHECKS=1");
+}
 
 // Example 1: Create table with ColumnSchema fluent API
 echo "1. Creating table with ColumnSchema fluent API:\n";

examples/11-schema/04-dump-restore.php

@@ -28,16 +28,18 @@
     $dbPath = sys_get_temp_dir() . '/pdodb_dump_example_' . uniqid() . '.sqlite';
     $db = new PdoDb('sqlite', ['path' => $dbPath]);
     // Ensure CLI commands use the same SQLite database file
+    putenv('PDODB_DRIVER=sqlite');
     putenv('PDODB_PATH=' . $dbPath);
 } else {
     $db = createExampleDb();
+    // Set environment variables from config for CLI commands
+    // This ensures BaseCliCommand can access the same database configuration
+    $config = getExampleConfig();
+    setEnvFromConfig($config);
 }
 $driver = $db->connection?->getDriverName() ?? 'sqlite';
 $schema = $db->schema();
 
-// Set driver and non-interactive mode for CLI commands
-// This ensures BaseCliCommand loads config from examples/config.{driver}.php or environment variables
-putenv('PDODB_DRIVER=' . $driver);
 putenv('PDODB_NON_INTERACTIVE=1');
 
 // Create CLI application

examples/11-schema/05-monitoring.php

@@ -30,6 +30,10 @@
 // Create database connection
 $db = createPdoDbWithErrorHandling($driver, $config);
 
+// Set environment variables from config for CLI commands
+setEnvFromConfig($config);
+putenv('PDODB_NON_INTERACTIVE=1');
+
 // Create a test table
 $schema = $db->schema();
 $schema->dropTableIfExists('monitor_test');

examples/11-schema/06-cache-management.php

@@ -30,7 +30,8 @@
 // Create cache instance
 $cache = new ArrayCache();
 
-// Set cache environment variables for CLI commands to use the same cache
+// Set environment variables from config for CLI commands
+setEnvFromConfig($config);
 putenv('PDODB_CACHE_ENABLED=true');
 putenv('PDODB_CACHE_TYPE=array');
 putenv('PDODB_NON_INTERACTIVE=1');

examples/11-schema/07-repository-service-generation.php

@@ -33,30 +33,10 @@
 echo "================================================\n\n";
 echo "Driver: {$driver}\n\n";
 
-// Set non-interactive mode for CLI commands
+// Set environment variables from config for CLI commands
+setEnvFromConfig($config);
 putenv('PDODB_NON_INTERACTIVE=1');
 
-// Export database config to environment for CLI commands
-putenv('PDODB_DRIVER=' . $driver);
-if (isset($config['host'])) {
-    putenv('PDODB_HOST=' . $config['host']);
-}
-if (isset($config['port'])) {
-    putenv('PDODB_PORT=' . (string)$config['port']);
-}
-if (isset($config['database'])) {
-    putenv('PDODB_DATABASE=' . $config['database']);
-}
-if (isset($config['username'])) {
-    putenv('PDODB_USERNAME=' . $config['username']);
-}
-if (isset($config['password'])) {
-    putenv('PDODB_PASSWORD=' . $config['password']);
-}
-if (isset($config['path'])) {
-    putenv('PDODB_PATH=' . $config['path']);
-}
-
 // Create database connection
 $db = createPdoDbWithErrorHandling($driver, $config);
 

examples/helpers.php

@@ -245,4 +245,51 @@ function getCurrentDriver(PdoDb $db): string
     return $connection->getDriverName();
 }
 
+/**
+ * Set environment variables from database configuration for CLI commands
+ * 
+ * This function ensures that CLI commands can access the same database configuration
+ * that was used to create the PdoDb instance. It reads the configuration array
+ * and sets all necessary PDODB_* environment variables.
+ * 
+ * @param array<string, mixed> $config Database configuration array
+ */
+function setEnvFromConfig(array $config): void
+{
+    if (isset($config['driver'])) {
+        putenv('PDODB_DRIVER=' . $config['driver']);
+    }
+    if (isset($config['host'])) {
+        putenv('PDODB_HOST=' . $config['host']);
+    }
+    if (isset($config['port'])) {
+        putenv('PDODB_PORT=' . (string)$config['port']);
+    }
+    if (isset($config['username'])) {
+        putenv('PDODB_USERNAME=' . $config['username']);
+    }
+    if (isset($config['password'])) {
+        putenv('PDODB_PASSWORD=' . $config['password']);
+    }
+    if (isset($config['database'])) {
+        putenv('PDODB_DATABASE=' . $config['database']);
+    }
+    if (isset($config['dbname'])) {
+        putenv('PDODB_DATABASE=' . $config['dbname']);
+    }
+    if (isset($config['charset'])) {
+        putenv('PDODB_CHARSET=' . $config['charset']);
+    }
+    if (isset($config['path'])) {
+        putenv('PDODB_PATH=' . $config['path']);
+    }
+    // Oracle-specific
+    if (isset($config['service_name'])) {
+        putenv('PDODB_SERVICE_NAME=' . $config['service_name']);
+    }
+    if (isset($config['sid'])) {
+        putenv('PDODB_SID=' . $config['sid']);
+    }
+}
+
 

src/cli/BaseCliCommand.php

@@ -219,7 +219,7 @@ protected static function buildConfigFromEnv(string $driver): ?array
             $envVars = [];
             // Always use getenv() to ensure we get variables set via putenv()
             // $_ENV may not be updated when putenv() is called
-            $envVarNames = ['PDODB_HOST', 'PDODB_PORT', 'PDODB_DATABASE', 'PDODB_USERNAME', 'PDODB_PASSWORD', 'PDODB_CHARSET', 'PDODB_PATH', 'PDODB_DRIVER'];
+            $envVarNames = ['PDODB_HOST', 'PDODB_PORT', 'PDODB_DATABASE', 'PDODB_USERNAME', 'PDODB_PASSWORD', 'PDODB_CHARSET', 'PDODB_PATH', 'PDODB_DRIVER', 'PDODB_SERVICE_NAME', 'PDODB_SID'];
             foreach ($envVarNames as $envVar) {
                 $envValue = getenv($envVar);
                 if ($envValue !== false) {
@@ -228,7 +228,12 @@ protected static function buildConfigFromEnv(string $driver): ?array
             }
 
             // Check required variables
-            if (!isset($envVars['PDODB_DATABASE']) || !isset($envVars['PDODB_USERNAME'])) {
+            if ($driver === 'oci') {
+                // Oracle requires username and either SERVICE_NAME or SID
+                if (!isset($envVars['PDODB_USERNAME']) || (!isset($envVars['PDODB_SERVICE_NAME']) && !isset($envVars['PDODB_SID']))) {
+                    return null;
+                }
+            } elseif (!isset($envVars['PDODB_DATABASE']) || !isset($envVars['PDODB_USERNAME'])) {
                 // SQLite doesn't require username/database
                 if ($driver !== 'sqlite') {
                     return null;