From 95eeb6a51c83b974f44b0535cf17d6de324f5d53 Mon Sep 17 00:00:00 2001
From: Guillaume Chau <alphadelta.fg@gmail.com>
Date: Sun, 12 Jan 2020 13:07:33 +0100
Subject: [PATCH] fix: improved error messages

---
 packages/@nodepack/db-migrator/src/lib/Migrator.ts  | 4 ++--
 packages/@nodepack/env-migrator/src/lib/Migrator.ts | 6 +++---
 packages/@nodepack/service/src/commands/dev.js      | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/packages/@nodepack/db-migrator/src/lib/Migrator.ts b/packages/@nodepack/db-migrator/src/lib/Migrator.ts
index 701ddee6..de3b18cc 100644
--- a/packages/@nodepack/db-migrator/src/lib/Migrator.ts
+++ b/packages/@nodepack/db-migrator/src/lib/Migrator.ts
@@ -25,7 +25,7 @@ export class Migrator extends EnvMigrator {
       const data = await this.context.readDbMigrationRecords()
       this.fileMigrationRecords = data.files
     } catch (e) {
-      consola.error('Could not read migration records. Error:', e.stack)
+      consola.error('Could not read migration records. Error:', e.stack || e)
       this.fileMigrationRecords = []
     }
   }
@@ -43,7 +43,7 @@ export class Migrator extends EnvMigrator {
         plugins: [],
       })
     } catch (e) {
-      consola.error('Could not write migration records. Error:', e.stack)
+      consola.error('Could not write migration records. Error:', e.stack || e)
     }
   }
 }
diff --git a/packages/@nodepack/env-migrator/src/lib/Migrator.ts b/packages/@nodepack/env-migrator/src/lib/Migrator.ts
index 1df05787..154bbdfa 100644
--- a/packages/@nodepack/env-migrator/src/lib/Migrator.ts
+++ b/packages/@nodepack/env-migrator/src/lib/Migrator.ts
@@ -83,7 +83,7 @@ export class Migrator {
           })
           count++
         } catch (e) {
-          consola.error(`Migration failed for ${module.file}\n${e.stack || e.message}`)
+          consola.error(`Migration failed for ${module.file}:`, e.stack || e)
           await this.writeMigrationRecords()
           process.exit(1)
         }
@@ -118,7 +118,7 @@ export class Migrator {
       const data = await readConfigFile(this.cwd, FILE_ENV_MIGRATIONS_RECORDS)
       this.fileMigrationRecords = data.files
     } catch (e) {
-      consola.error('Could not read migration records. Error:', e.stack)
+      consola.error('Could not read migration records. Error:', e.stack || e)
       this.fileMigrationRecords = []
     }
   }
@@ -133,7 +133,7 @@ export class Migrator {
         plugins: [],
       })
     } catch (e) {
-      consola.error('Could not write migration records. Error:', e.stack)
+      consola.error('Could not write migration records. Error:', e.stack || e)
     }
   }
 }
diff --git a/packages/@nodepack/service/src/commands/dev.js b/packages/@nodepack/service/src/commands/dev.js
index 922b2ae0..baa1cc17 100644
--- a/packages/@nodepack/service/src/commands/dev.js
+++ b/packages/@nodepack/service/src/commands/dev.js
@@ -152,7 +152,7 @@ module.exports = (api, options) => {
         child.kill('SIGINT')
         return true
       } catch (e) {
-        consola.error(`Couldn't terminate process ${child.pid}: ${e}`)
+        consola.error(`Couldn't terminate process ${child.pid}:`, e.stack || e)
       }
     }