feat: added opt out option of backup migrations - fix #56 (#57)

openhotel · Feb 2, 2025 · 43ba2d8 · 43ba2d8
1 parent 98fd247
commit 43ba2d8
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 7 deletions.
diff --git a/src/modules/db.ts b/src/modules/db.ts
@@ -12,12 +12,16 @@ import type {
 import { join } from "@std/path";
 import { getUtilDate } from "../utils/date.utils.ts";
 
-export const getDb = (
-  { pathname, backupsPathname }: DbProps = {
-    pathname: "./database",
-    backupsPathname: "./database-backups",
+export const getDb = ({
+  pathname = "./database",
+  backups: {
+    pathname: backupsPathname = "./database-backups",
+    onMigration: backupOnMigration = true,
+  } = {
+    pathname: "./database-backups",
+    onMigration: true,
   },
-): DbMutable => {
+}: DbProps): DbMutable => {
   let db: Deno.Kv;
 
   const $migrations: DbMigrationsMutable = ((): DbMigrationsMutable => {
@@ -51,7 +55,7 @@ export const getDb = (
           console.log(`- Migration ${migration.id} was already applied!`);
           continue;
         }
-        await backup(`_migration_${migration.id}`);
+        if (backupOnMigration) await backup(`_migration_${migration.id}`);
 
         try {
           await migration.up(dbMutables);

diff --git a/src/types/db.types.ts b/src/types/db.types.ts
@@ -1,6 +1,9 @@
 export type DbProps = {
   pathname?: string;
-  backupsPathname?: string;
+  backups?: {
+    pathname?: string;
+    onMigration?: boolean;
+  };
 };
 
 export type Migration = {