@glance-apps/sync 1.1.0 → 1.1.2

package/CHANGELOG.md

@@ -7,6 +7,28 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.1.2] - 2026-06-04
+
+### Fixed
+
+- **412 retry — local DB mutated before upload**: `applyPayload` was called before
+  `upload()` in `doCycle()`. If the upload returned a 412 Precondition Failed (concurrent
+  write from another device), the retry cycle re-merged already-dirtied local state with
+  the freshly downloaded remote, producing inconsistent data (e.g. completion events
+  referencing chore sync IDs that no longer exist after the second merge). `applyPayload`
+  is now deferred until after a successful upload (204), leaving the local DB untouched
+  if the upload fails so the retry starts from a clean state.
+
+## [1.1.1] - 2026-06-03
+
+### Fixed
+
+- **WebDAV — 405 on MKCOL treated as directory-already-exists**: `mkcolWithParents`
+  now handles HTTP 405 (Method Not Allowed) the same as 409/404, triggering the
+  parent-directory creation fallback. Some WebDAV servers (e.g. certain nginx
+  configurations) return 405 rather than 409 when MKCOL is called on a path that
+  already exists as a collection.
+
 ## [1.1.0] - 2026-05-25
 
 ### Added
@@ -97,7 +119,10 @@ entire sync infrastructure can be shared across the GLANCE app family.
 - TypeScript declarations in `types/index.d.ts`.
 - WebDAV CORS proxy handler in `api/webdav-proxy.js`.
 
-[Unreleased]: https://github.com/glance-apps/glance-sync/compare/v1.0.3...HEAD
+[Unreleased]: https://github.com/glance-apps/glance-sync/compare/v1.1.2...HEAD
+[1.1.2]: https://github.com/glance-apps/glance-sync/compare/v1.1.1...v1.1.2
+[1.1.1]: https://github.com/glance-apps/glance-sync/compare/v1.1.0...v1.1.1
+[1.1.0]: https://github.com/glance-apps/glance-sync/compare/v1.0.3...v1.1.0
 [1.0.3]: https://github.com/glance-apps/glance-sync/compare/v1.0.2...v1.0.3
 [1.0.2]: https://github.com/glance-apps/glance-sync/compare/v1.0.1...v1.0.2
 [1.0.1]: https://github.com/glance-apps/glance-sync/compare/v1.0.0...v1.0.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@glance-apps/sync",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "license": "MIT",
   "type": "module",
   "main": "src/index.js",

package/src/engine.js

@@ -346,34 +346,36 @@ export const createSyncEngine = (config) => {
       const mergeResult = mergePayloads(localData, remote.data);
       const { data: mergedData, localChanged, remoteChanged } = mergeResult;
 
-      if (localChanged) {
-        let mayApply = true;
-        if (validateApplyPayload) {
-          const check = await validateApplyPayload({ ...remote, data: mergedData });
-          if (!check?.valid) {
-            // eslint-disable-next-line no-console
-            console.error(`[${appId}] applyRemoteData aborted:`, check?.reason || 'validation failed');
-            mayApply = false;
-          }
-        }
-        if (mayApply) {
-          await applyPayload(mergedData, { allowEmpty: !!remote.lastModified });
-          localStorage.setItem(KEY_LOCAL_MOD, new Date().toISOString());
-        }
-      }
-
       if (remoteChanged || localChanged) {
         const mergedPayload = {
           version: ENVELOPE_DATA_VERSION,
           lastModified: new Date().toISOString(),
           data: mergedData,
         };
+        // Upload first — if this throws (e.g. 412 PRECONDITION_FAILED), the
+        // local DB must remain untouched so the retry starts from clean state.
         await upload({
           prebuiltPayload: mergedPayload,
           skipLockCheck: true,
           etag: etag || overrideEtag || null,
         });
 
+        if (localChanged) {
+          let mayApply = true;
+          if (validateApplyPayload) {
+            const check = await validateApplyPayload({ ...remote, data: mergedData });
+            if (!check?.valid) {
+              // eslint-disable-next-line no-console
+              console.error(`[${appId}] applyRemoteData aborted:`, check?.reason || 'validation failed');
+              mayApply = false;
+            }
+          }
+          if (mayApply) {
+            await applyPayload(mergedData, { allowEmpty: !!remote.lastModified });
+            localStorage.setItem(KEY_LOCAL_MOD, new Date().toISOString());
+          }
+        }
+
         if (hasNeverSynced && localChanged) {
           onFirstSyncReload?.();
           return true;

package/src/providers.js

@@ -102,7 +102,7 @@ async function parseDownloadResponse(res, engineConfig) {
 
 async function mkcolWithParents(dirUrl, authHeaders, wf) {
   const res = await wf('MKCOL', dirUrl, authHeaders);
-  if (res.status === 409 || res.status === 404) {
+  if (res.status === 409 || res.status === 404 || res.status === 405) {
     // Derive parent by stripping the last path segment via URL() to avoid
     // producing malformed strings on root-level inputs.
     let parent;