feat: add --help global option and fix pdodb dump behavior

- Add --help to global options list in Application::showHelp()
- Add handling for global --help option in Application::run()
- Fix pdodb dump to show help when no arguments provided (consistent
  with other commands)
- Allow pdodb dump to dump entire database when dump-specific options
  are provided (--schema-only, --data-only, --output)
- Add tests for new dump command behavior:
  - testDumpCommandShowsHelpWhenNoArguments()
  - testDumpCommandDumpsDatabaseWhenOutputOptionProvided()

Author: tommyknocker

src/cli/Application.php

@@ -117,6 +117,12 @@ public function run(array $argv): int
             return 0;
         }
 
+        // Check for global --help option
+        if ($argv[0] === '--help') {
+            $this->showHelp();
+            return 0;
+        }
+
         $commandName = $argv[0];
         $command = $this->getCommand($commandName);
 
@@ -185,6 +191,7 @@ protected function showHelp(): void
         }
 
         echo "\nGlobal options:\n";
+        echo "  --help               Show help message\n";
         echo "  --connection=<name>  Use a named connection from config/db.php\n";
         echo "  --config=<path>      Path to db.php configuration file\n";
         echo "  --env=<path>         Path to .env file\n";

src/cli/commands/DumpCommand.php

@@ -28,15 +28,15 @@ public function __construct()
      */
     public function execute(): int
     {
-        // Show help only if explicitly requested via option or argument
+        // Show help if explicitly requested via option or argument
         if ($this->getOption('help', false)) {
             $this->showHelp();
             return 0;
         }
 
         $firstArg = $this->getArgument(0);
 
-        if ($firstArg === 'help') {
+        if ($firstArg === '--help' || $firstArg === 'help') {
             $this->showHelp();
             return 0;
         }
@@ -45,7 +45,19 @@ public function execute(): int
             return $this->restore();
         }
 
-        // Default: dump (first arg is table name if provided, null = entire database)
+        // If no arguments, check if dump-specific options are provided
+        // If yes, dump entire database; if no, show help
+        if ($firstArg === null) {
+            $hasDumpOptions = $this->getOption('schema-only', false) !== false
+                || $this->getOption('data-only', false) !== false
+                || $this->getOption('output') !== null;
+            if (!$hasDumpOptions) {
+                $this->showHelp();
+                return 0;
+            }
+        }
+
+        // First arg is table name (dump specific table), or null for entire database
         return $this->dump();
     }
 

tests/shared/DumpCommandCliTests.php

@@ -143,4 +143,38 @@ public function testDumpCommandHelp(): void
         $this->assertStringContainsString('Database Dump and Restore', $output);
         $this->assertStringContainsString('Usage:', $output);
     }
+
+    public function testDumpCommandShowsHelpWhenNoArguments(): void
+    {
+        $app = new Application();
+        ob_start();
+        $exitCode = $app->run(['pdodb', 'dump']);
+        $output = ob_get_clean();
+
+        $this->assertEquals(0, $exitCode);
+        $this->assertStringContainsString('Database Dump and Restore', $output);
+        $this->assertStringContainsString('Usage:', $output);
+    }
+
+    public function testDumpCommandDumpsDatabaseWhenOutputOptionProvided(): void
+    {
+        $dumpFile = sys_get_temp_dir() . '/pdodb_cli_test_' . uniqid() . '.sql';
+        $app = new Application();
+        putenv('PDODB_DRIVER=sqlite');
+        putenv('PDODB_PATH=' . self::$dbPath);
+        putenv('PDODB_NON_INTERACTIVE=1');
+
+        ob_start();
+        $exitCode = $app->run(['pdodb', 'dump', '--output=' . $dumpFile]);
+        $output = ob_get_clean();
+
+        $this->assertEquals(0, $exitCode);
+        $this->assertFileExists($dumpFile);
+        $sql = file_get_contents($dumpFile);
+        $this->assertStringContainsString('CREATE TABLE', $sql);
+        $this->assertStringContainsString('users', $sql);
+        $this->assertStringContainsString('posts', $sql);
+
+        unlink($dumpFile);
+    }
 }