@lobb-js/core 0.26.0 → 0.26.2

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lobb-js/core",
   "license": "UNLICENSED",
-  "version": "0.26.0",
+  "version": "0.26.2",
   "type": "module",
   "publishConfig": {
     "access": "public"

package/src/database/DatabaseSyncManager.ts

@@ -35,11 +35,27 @@ export class DatabaseSyncManager {
     // Startup tasks run first — applies any idempotent DB fixes before schema reading begins.
     await runCoreDbSetup();
 
+    const dbSchemaDiff = await this.getDbDifferences(specificColleciton);
+
+    // Pass 1: additive ops only (add-collection, add-field) — always safe, no data loss possible.
+    // Running these before migrations means any new collection defined in config already exists
+    // when migration code tries to seed it.
+    const additiveOps = dbSchemaDiff.filter(
+      (op) => op.op === "add-collection" || op.op === "add-field",
+    );
+    await this.applyDbSchemaDiff(additiveOps, forceSync);
+
+    // Migrations run after additive schema changes so they can safely insert into new collections.
     const migrationsManager = new MigrationsManager();
     await migrationsManager.init();
 
-    const dbSchemaDiff = await this.getDbDifferences(specificColleciton);
-    await this.applyDbSchemaDiff(dbSchemaDiff, forceSync);
+    // Pass 2: everything else (remove-collection, remove-field, alter-field, indexes).
+    // Destructive ops run after migrations so a migration can clear data before applyDbSchemaDiff
+    // would otherwise refuse to drop a column/table that still has rows.
+    const remainingOps = dbSchemaDiff.filter(
+      (op) => op.op !== "add-collection" && op.op !== "add-field",
+    );
+    await this.applyDbSchemaDiff(remainingOps, forceSync);
   }
 
   private async applyDbSchemaDiff(ops: SchemaDiffOp[], forceSync: boolean) {

package/src/database/drivers/pgDriver/PGDriver.ts

@@ -38,13 +38,43 @@ export class PGDriver extends DatabaseDriver {
 
   public async createConnection() {
     const default_pool_connections = (cpus().length * 2) + 4;
+    const { host, port } = this.databaseConfig;
 
-    // Ensure the database exists before connecting
-    await this.ensureDatabaseExists();
+    try {
+      // Ensure the database exists before connecting
+      await this.ensureDatabaseExists();
+    } catch (err: any) {
+      const addr = `${host}:${port}`;
+      if (err.code === "ESERVFAIL" || err.code === "ENOTFOUND" || err.code === "EAI_AGAIN") {
+        throw new LobbError({
+          code: "INTERNAL_SERVER_ERROR",
+          message: `Cannot connect to database: failed to resolve host "${host}". Make sure the database server is running and reachable at ${addr}.`,
+        });
+      }
+      if (err.code === "ECONNREFUSED") {
+        throw new LobbError({
+          code: "INTERNAL_SERVER_ERROR",
+          message: `Cannot connect to database: connection refused at ${addr}. Make sure the database server is running.`,
+        });
+      }
+      if (err.code === "ETIMEDOUT") {
+        throw new LobbError({
+          code: "INTERNAL_SERVER_ERROR",
+          message: `Cannot connect to database: connection timed out at ${addr}. Make sure the database server is reachable.`,
+        });
+      }
+      if ((err as any).code === "28P01") {
+        throw new LobbError({
+          code: "INTERNAL_SERVER_ERROR",
+          message: `Cannot connect to database: authentication failed for user "${this.databaseConfig.username}" at ${addr}. Check your database credentials.`,
+        });
+      }
+      throw err;
+    }
 
     // Now connect to the actual target database
     this.pool = new Pool({
-      host: this.databaseConfig.host,
+      host,
       port: this.databaseConfig.port,
       user: this.databaseConfig.username,
       password: this.databaseConfig.password,