Merge branch 'dev' into chris/refactor-contract-updates

.changeset/improve_migration_out_after_foreignkey_check_fails.md

@@ -0,0 +1,5 @@
+---
+default: patch
+---
+
+# Improve migration out after foreignkey check fails

stores/sql/sqlite/common.go

@@ -44,11 +44,29 @@ func applyMigration(ctx context.Context, db *sql.DB, fn func(tx sql.Tx) (bool, e
 		} else if !migrated {
 			return nil
 		}
+
 		// perform foreign key integrity check
-		if err := tx.QueryRow(ctx, "PRAGMA foreign_key_check").Scan(); !errors.Is(err, dsql.ErrNoRows) {
-			return fmt.Errorf("foreign key constraints are not satisfied")
+		rows, err := tx.Query(ctx, "PRAGMA foreign_key_check")
+		if err != nil {
+			return err
+		}
+		defer rows.Close()
+
+		// check if there are any foreign key constraint violations
+		var errs []error
+		var tableName, foreignKey string
+		var rowID int
+		for rows.Next() {
+			if err := rows.Scan(&tableName, &rowID, &foreignKey); err != nil {
+				return fmt.Errorf("failed to scan foreign key check result: %w", err)
+			}
+			errs = append(errs, fmt.Errorf("foreign key constraint violation in table '%s': row %d, foreign key %s", tableName, rowID, foreignKey))
+		}
+
+		if err := rows.Err(); err != nil {
+			return fmt.Errorf("error iterating foreign key check results: %w", err)
 		}
-		return nil
+		return errors.Join(errs...)
 	})
 }