imdone-core 2.5.1 → 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.1",
+  "version": "2.5.3",
   "description": "imdone-core",
   "repository": {
     "type": "git",
@@ -93,14 +93,13 @@
     "remove-markdown": "^0.3.0",
     "sanitize-filename": "^1.6.3",
     "sift": "^17.1.3",
-    "simple-git": "^3.19.1",
     "uniqid": "^5.4.0",
     "xregexp": "^5.1.2"
   },
   "devDependencies": {
     "@types/node": "^22.13.13",
-    "@vitest/coverage-istanbul": "^4.1.9",
-    "@vitest/coverage-v8": "^4.1.9",
+    "@vitest/coverage-istanbul": "^4.1.11",
+    "@vitest/coverage-v8": "^4.1.11",
     "chai": "^5.2.0",
     "colors": "^1.4.0",
     "dependency-cruiser": "^16.10.0",
@@ -112,7 +111,7 @@
     "json-stringify-safe": "~5.0.0",
     "prettier": "^2.5.1",
     "typescript": "^5.8.2",
-    "vitest": "^4.1.9"
+    "vitest": "^4.1.11"
   },
   "allowScripts": {
     "core-js-pure@3.42.0": true,