imdone-core 2.5.2 → 2.5.3

package/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.5.3
+
+- Reorder configured lists atomically with `repository.moveList`, preserving every list field and unrelated configuration, rolling back failed saves, and emitting `list.modified` only after persistence succeeds.
+
 ## 2.3.5
 
 - Incorporate newly added watched files into repository state before emitting `file.update`, so live board consumers can render the new task without a reload or restart.

package/lib/repository.js

@@ -1129,13 +1129,26 @@ export class Repository extends Emitter {
   // order:-190
   // -->
   async moveList (name, pos) {
-    var list = this.getList(name)
-    if (list) {
-      _remove(this.getLists(), { name: name })
-      this.getLists().splice(pos, 0, list)
+    const lists = this.getLists()
+    const currentPosition = lists.findIndex((list) => list.name === name)
+    if (currentPosition < 0) throw new Error(`List not found: ${name}`)
+    if (!Number.isInteger(pos)) throw new TypeError('List position must be an integer')
+    if (pos < 0 || pos >= lists.length) throw new RangeError(`List position out of range: ${pos}`)
+    if (currentPosition === pos) return
+
+    const previousLists = lists.map((list) => ({ ...list }))
+    const [list] = lists.splice(currentPosition, 1)
+    lists.splice(pos, 0, list)
+    this.config.lists = lists
+
+    try {
       await this.saveConfig()
-      this.emit('list.modified', name)
+    } catch (error) {
+      this.config.lists = previousLists
+      throw error
     }
+
+    this.emit('list.modified', name)
   }
 
   // READY Refactor toggleListIgnore to use async/await

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "imdone-core",
-  "version": "2.5.2",
+  "version": "2.5.3",
   "description": "imdone-core",
   "repository": {
     "type": "git",