Improve reset command logic

- Remove sessions from protected tables (true nuclear option)
- Rename protectedTables to trackingTables for clarity
- Clear seed records (cake_seeds) along with migration records
- Rename clearMigrationRecords to clearTrackingRecords

Author: dereuromark

src/Command/ResetCommand.php

@@ -37,15 +37,17 @@ class ResetCommand extends Command
     use EventDispatcherTrait;
 
     /**
-     * Tables that should never be dropped.
+     * Migration/seed tracking tables that should not be dropped.
+     *
+     * These tables are kept (structure preserved) but their contents
+     * are cleared so migrations can run fresh.
      *
      * @var array<string>
      */
-    protected array $protectedTables = [
+    protected array $trackingTables = [
         'cake_migrations',
         'cake_seeds',
         'phinxlog',
-        'sessions',
     ];
 
     /**
@@ -163,7 +165,7 @@ public function execute(Arguments $args, ConsoleIo $io): ?int
             $this->dropTables($connection, $tablesToDrop, $io);
             /** @var string|null $plugin */
             $plugin = $args->getOption('plugin');
-            $this->clearMigrationRecords($connection, $plugin, $io);
+            $this->clearTrackingRecords($connection, $plugin, $io);
         } else {
             $io->info('DRY-RUN: Would drop ' . count($tablesToDrop) . ' table(s).');
         }
@@ -194,11 +196,11 @@ protected function getTablesToDrop(Connection $connection): array
         $schema = $connection->getDriver()->schemaDialect();
         $tables = $schema->listTables();
 
-        // Filter out protected tables
+        // Filter out migration/seed tracking tables
         $tablesToDrop = [];
         foreach ($tables as $table) {
-            // Skip migration tracking tables
-            if (in_array($table, $this->protectedTables, true)) {
+            // Skip migration/seed tracking tables (we clear their contents instead)
+            if (in_array($table, $this->trackingTables, true)) {
                 continue;
             }
             // Skip plugin phinxlog tables
@@ -316,18 +318,18 @@ protected function setForeignKeyChecks(Connection $connection, bool $enable): vo
     }
 
     /**
-     * Clear migration records from the tracking table.
+     * Clear migration and seed records from tracking tables.
      *
      * @param \Cake\Database\Connection $connection Database connection
      * @param string|null $plugin Plugin name
      * @param \Cake\Console\ConsoleIo $io Console IO
      * @return void
      */
-    protected function clearMigrationRecords(Connection $connection, ?string $plugin, ConsoleIo $io): void
+    protected function clearTrackingRecords(Connection $connection, ?string $plugin, ConsoleIo $io): void
     {
         $schema = $connection->getDriver()->schemaDialect();
 
-        // Clear unified table if exists
+        // Clear unified migrations table if exists
         if ($schema->hasTable('cake_migrations')) {
             $query = $connection->deleteQuery()->delete('cake_migrations');
             if ($plugin !== null) {
@@ -345,6 +347,14 @@ protected function clearMigrationRecords(Connection $connection, ?string $plugin
                 ->execute();
             $io->verbose("Cleared migration records from {$legacyTable}");
         }
+
+        // Clear seed tracking table if exists
+        if ($schema->hasTable('cake_seeds')) {
+            $connection->deleteQuery()
+                ->delete('cake_seeds')
+                ->execute();
+            $io->verbose('Cleared seed records from cake_seeds');
+        }
     }
 
     /**