fix: migrations can run on pristine database

server/lib/migrationSystem.js

@@ -3,12 +3,16 @@ import migrations from './migrations.js'
 import { Meta } from './mongo.js'
 
 const migrationStorage = {
-  async initializeStorage() {
+  async initialize() {
     await migrations.get(0).up()
     return (await Meta.findOne({ _id: 'database-version' })).value
   },
   async getVersion() {
-    return (await Meta.findOne({ _id: 'database-version' })).value
+    const databaseVersion = await Meta.findOne({ _id: 'database-version' })
+    if (!databaseVersion) {
+      return null;
+    }
+    return databaseVersion.value
   },
   async setVersion(version) {
     const databaseVersion = await Meta.findOne({ _id: 'database-version' })

server/lib/migrations.js

@@ -66,25 +66,15 @@ migrations.set(2, {
 migrations.set(3, {
   name: 'note-types',
   async up() {
-    return new Promise(async (resolve, reject) => {
-      mongoose.connection.db
-        .listCollections({ name: 'links' })
-        .toArray((error, collectionNames) => {
-          if (error) {
-            reject(error)
-          }
+    const collectionNames = await mongoose.connection.db
+      .listCollections({ name: 'links' })
+      .toArray()
 
-          if (collectionNames.length) {
-            const linkCollection = mongoose.connection.db.collection('links')
-            linkCollection
-              .updateMany({}, { $set: { type: 'Link' } })
-              .then(() => linkCollection.rename('notes'))
-              .then(() => resolve())
-          } else {
-            resolve()
-          }
-        })
-    })
+    if (collectionNames.length) {
+      const linkCollection = mongoose.connection.db.collection('links')
+      await linkCollection.updateMany({}, { $set: { type: 'Link' } })
+      await linkCollection.rename('notes')
+    }
   },
   async down() {
     return Text.find()