@shepai/cli 1.112.0 → 1.112.1

package/dist/packages/core/src/infrastructure/persistence/sqlite/sqlite-migration-storage.d.ts

@@ -27,6 +27,12 @@ export declare class SQLiteMigrationStorage implements UmzugStorage<Database.Dat
      *   PRAGMA user_version tracking and seeds the umzug_migrations table accordingly.
      */
     constructor(db: Database.Database, legacyMigrationNames?: string[]);
+    /**
+     * Mapping of legacy migration names to the tables they create.
+     * Used by verifyBootstrappedMigrations to detect migrations that were
+     * marked as executed by the bootstrap seeder but never actually ran.
+     */
+    private static readonly MIGRATION_TABLE_MAP;
     /**
      * Returns the names of all executed migrations, sorted lexicographically.
      */
@@ -55,5 +61,12 @@ export declare class SQLiteMigrationStorage implements UmzugStorage<Database.Dat
      * user_version using the provided legacy migration names.
      */
     private bootstrapLegacyMigrations;
+    /**
+     * Checks that each bootstrapped migration's expected table actually exists.
+     * Only checks migrations that were seeded (present in seededNames).
+     * If a table is missing, removes the migration record from umzug_migrations
+     * so that umzug will re-run it on the next up() call.
+     */
+    private verifyBootstrappedMigrations;
 }
 //# sourceMappingURL=sqlite-migration-storage.d.ts.map

package/dist/packages/core/src/infrastructure/persistence/sqlite/sqlite-migration-storage.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"sqlite-migration-storage.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/persistence/sqlite/sqlite-migration-storage.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAC3C,OAAO,KAAK,EAAE,YAAY,EAAE,eAAe,EAAE,MAAM,OAAO,CAAC;AAE3D;;;;;GAKG;AACH,qBAAa,sBAAuB,YAAW,YAAY,CAAC,QAAQ,CAAC,QAAQ,CAAC;IAC5E,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAoB;IAEvC;;;;;;OAMG;gBACS,EAAE,EAAE,QAAQ,CAAC,QAAQ,EAAE,oBAAoB,CAAC,EAAE,MAAM,EAAE;IAQlE;;OAEG;IACG,QAAQ,CAAC,KAAK,EAAE,IAAI,CAAC,eAAe,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,SAAS,CAAC,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC;IAO7F;;OAEG;IACG,YAAY,CAAC,MAAM,EAAE,eAAe,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;IAM7E;;OAEG;IACG,cAAc,CAAC,MAAM,EAAE,eAAe,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;IAI/E;;OAEG;IACH,OAAO,CAAC,WAAW;IASnB;;;;;;;;;;OAUG;IACH,OAAO,CAAC,yBAAyB;CA+BlC"}
+{"version":3,"file":"sqlite-migration-storage.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/persistence/sqlite/sqlite-migration-storage.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAC3C,OAAO,KAAK,EAAE,YAAY,EAAE,eAAe,EAAE,MAAM,OAAO,CAAC;AAE3D;;;;;GAKG;AACH,qBAAa,sBAAuB,YAAW,YAAY,CAAC,QAAQ,CAAC,QAAQ,CAAC;IAC5E,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAoB;IAEvC;;;;;;OAMG;gBACS,EAAE,EAAE,QAAQ,CAAC,QAAQ,EAAE,oBAAoB,CAAC,EAAE,MAAM,EAAE;IAQlE;;;;OAIG;IACH,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,mBAAmB,CAOzC;IAEF;;OAEG;IACG,QAAQ,CAAC,KAAK,EAAE,IAAI,CAAC,eAAe,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,SAAS,CAAC,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC;IAO7F;;OAEG;IACG,YAAY,CAAC,MAAM,EAAE,eAAe,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;IAM7E;;OAEG;IACG,cAAc,CAAC,MAAM,EAAE,eAAe,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;IAI/E;;OAEG;IACH,OAAO,CAAC,WAAW;IASnB;;;;;;;;;;OAUG;IACH,OAAO,CAAC,yBAAyB;IAuCjC;;;;;OAKG;IACH,OAAO,CAAC,4BAA4B;CAiBrC"}

package/dist/packages/core/src/infrastructure/persistence/sqlite/sqlite-migration-storage.js

@@ -31,6 +31,19 @@ export class SQLiteMigrationStorage {
             this.bootstrapLegacyMigrations(legacyMigrationNames);
         }
     }
+    /**
+     * Mapping of legacy migration names to the tables they create.
+     * Used by verifyBootstrappedMigrations to detect migrations that were
+     * marked as executed by the bootstrap seeder but never actually ran.
+     */
+    static MIGRATION_TABLE_MAP = {
+        '001-create-settings-table': 'settings',
+        '003-create-agent-runs': 'agent_runs',
+        '004-create-features': 'features',
+        '008-add-approval-gates-and-phase-timings': 'phase_timings',
+        '015-create-repositories-and-backfill': 'repositories',
+        '033-create-pr-sync-lock': 'pr_sync_lock',
+    };
     /**
      * Returns the names of all executed migrations, sorted lexicographically.
      */
@@ -94,5 +107,29 @@ export class SQLiteMigrationStorage {
             }
         });
         seedAll();
+        // After seeding, verify that expected schema artifacts actually exist.
+        // If a migration was inserted into the legacy list after the DB was already
+        // at a higher user_version, its DDL never ran but the seeder marked it done.
+        const seededCount = Math.min(userVersion, legacyMigrationNames.length);
+        const seededNames = legacyMigrationNames.slice(0, seededCount);
+        this.verifyBootstrappedMigrations(seededNames);
+    }
+    /**
+     * Checks that each bootstrapped migration's expected table actually exists.
+     * Only checks migrations that were seeded (present in seededNames).
+     * If a table is missing, removes the migration record from umzug_migrations
+     * so that umzug will re-run it on the next up() call.
+     */
+    verifyBootstrappedMigrations(seededNames) {
+        const seededSet = new Set(seededNames);
+        const tableCheck = this.db.prepare("SELECT COUNT(*) as count FROM sqlite_master WHERE type='table' AND name=?");
+        for (const [migrationName, tableName] of Object.entries(SQLiteMigrationStorage.MIGRATION_TABLE_MAP)) {
+            if (!seededSet.has(migrationName))
+                continue;
+            const result = tableCheck.get(tableName);
+            if (result.count === 0) {
+                this.db.prepare('DELETE FROM umzug_migrations WHERE name = ?').run(migrationName);
+            }
+        }
     }
 }