zevairouter 1.0.140 → 1.0.142

package/src/app/(dashboard)/dashboard/providers/components/ConnectionsCard.js

@@ -354,8 +354,8 @@ export default function ConnectionsCard({ providerId, isOAuth }) {
     setConnections(next);
     try {
       await Promise.all([
-        fetch(`/api/providers/${next[i1].id}`, { method: "PUT", headers: { "Content-Type": "application/json" }, body: JSON.stringify({ priority: i1 }) }),
-        fetch(`/api/providers/${next[i2].id}`, { method: "PUT", headers: { "Content-Type": "application/json" }, body: JSON.stringify({ priority: i2 }) }),
+        fetch(`/api/providers/${next[i1].id}`, { method: "PUT", headers: { "Content-Type": "application/json" }, body: JSON.stringify({ priority: i1 + 1 }) }),
+        fetch(`/api/providers/${next[i2].id}`, { method: "PUT", headers: { "Content-Type": "application/json" }, body: JSON.stringify({ priority: i2 + 1 }) }),
       ]);
     } catch { await fetch_(); }
   };

package/src/lib/db/repos/connectionsRepo.js

@@ -76,13 +76,25 @@ export async function getProviderConnectionById(id) {
 }
 
 // Internal sync reorder — must be called INSIDE a transaction
-function reorderInTx(db, providerId) {
+function reorderInTx(db, providerId, movedId = null) {
   const list = db.all(`SELECT * FROM providerConnections WHERE provider = ?`, [providerId]).map(rowToConn);
   list.sort((a, b) => {
     const pDiff = (a.priority || 0) - (b.priority || 0);
     if (pDiff !== 0) return pDiff;
-    return new Date(b.updatedAt || 0) - new Date(a.updatedAt || 0);
+    return a.id.localeCompare(b.id);
   });
+
+  if (movedId) {
+    const movedIndex = list.findIndex((c) => c.id === movedId);
+    if (movedIndex !== -1) {
+      const [moved] = list.splice(movedIndex, 1);
+      const movedPriority = Number(moved.priority);
+      const requestedIndex = Number.isFinite(movedPriority) ? Math.ceil(movedPriority) - 1 : list.length;
+      const targetIndex = Math.max(0, Math.min(requestedIndex, list.length));
+      list.splice(targetIndex, 0, moved);
+    }
+  }
+
   list.forEach((c, i) => {
     db.run(`UPDATE providerConnections SET priority = ? WHERE id = ?`, [i + 1, c.id]);
   });
@@ -114,7 +126,12 @@ export async function createProviderConnection(data) {
     if (existing) {
       const merged = { ...existing, ...data, updatedAt: now };
       upsert(db, merged);
-      result = merged;
+      if (data.priority !== undefined) {
+        reorderInTx(db, existing.provider, existing.id);
+        result = rowToConn(db.get(`SELECT * FROM providerConnections WHERE id = ?`, [existing.id]));
+      } else {
+        result = merged;
+      }
       return;
     }
 
@@ -146,8 +163,8 @@ export async function createProviderConnection(data) {
     if (data.email !== undefined) conn.email = data.email;
 
     upsert(db, conn);
-    reorderInTx(db, data.provider);
-    result = conn;
+    reorderInTx(db, data.provider, conn.id);
+    result = rowToConn(db.get(`SELECT * FROM providerConnections WHERE id = ?`, [conn.id]));
   });
 
   return result;
@@ -163,8 +180,12 @@ export async function updateProviderConnection(id, data) {
     const existing = rowToConn(row);
     const merged = { ...existing, ...data, updatedAt: new Date().toISOString() };
     upsert(db, merged);
-    if (data.priority !== undefined) reorderInTx(db, existing.provider);
-    result = merged;
+    if (data.priority !== undefined) {
+      reorderInTx(db, existing.provider, id);
+      result = rowToConn(db.get(`SELECT * FROM providerConnections WHERE id = ?`, [id]));
+    } else {
+      result = merged;
+    }
   });
   return result;
 }