@atlaspack/runtime-browser-hmr 2.14.5-dev.1c70d50f9.99 → 2.14.5-dev.69

package/CHANGELOG.md

@@ -1,39 +1,5 @@
 # @atlaspack/runtime-browser-hmr
 
-## 2.14.14
-
-### Patch Changes
-
-- Updated dependencies []:
-  - @atlaspack/utils@2.15.2
-  - @atlaspack/plugin@2.14.14
-
-## 2.14.13
-
-### Patch Changes
-
-- [#633](https://github.com/atlassian-labs/atlaspack/pull/633) [`26aa9c5`](https://github.com/atlassian-labs/atlaspack/commit/26aa9c599d2be45ce1438a74c5fa22f39b9b554b) Thanks [@sbhuiyan-atlassian](https://github.com/sbhuiyan-atlassian)! - Ported various HMR changes from Parcel
-
-- Updated dependencies []:
-  - @atlaspack/utils@2.15.1
-  - @atlaspack/plugin@2.14.13
-
-## 2.14.12
-
-### Patch Changes
-
-- Updated dependencies [[`e39c6cf`](https://github.com/atlassian-labs/atlaspack/commit/e39c6cf05f7e95ce5420dbcea66f401b1cbd397c)]:
-  - @atlaspack/utils@2.15.0
-  - @atlaspack/plugin@2.14.12
-
-## 2.14.11
-
-### Patch Changes
-
-- Updated dependencies []:
-  - @atlaspack/utils@2.14.11
-  - @atlaspack/plugin@2.14.11
-
 ## 2.14.10
 
 ### Patch Changes

package/lib/loaders/hmr-runtime.js

@@ -67,7 +67,7 @@ function Module(moduleName) {
 }
 module.bundle.Module = Module;
 module.bundle.hotData = {};
-var checkedAssets /*: {|[string]: boolean|} */, disposedAssets /*: {|[string]: boolean|} */, assetsToDispose /*: Array<[ParcelRequire, string]> */, assetsToAccept /*: Array<[ParcelRequire, string]> */;
+var checkedAssets /*: {|[string]: boolean|} */, assetsToDispose /*: Array<[ParcelRequire, string]> */, assetsToAccept /*: Array<[ParcelRequire, string]> */;
 
 function getHostname() {
   return HMR_HOST || (location.protocol.indexOf('http') === 0 ? location.hostname : 'localhost');
@@ -111,7 +111,6 @@ if ((!parent || !parent.isParcelRequire) && typeof WebSocket !== 'undefined') {
   // $FlowFixMe
   ws.onmessage = async function (event /*: {data: string, ...} */) {
     checkedAssets = {} /*: {|[string]: boolean|} */;
-    disposedAssets = {} /*: {|[string]: boolean|} */;
     assetsToAccept = [];
     assetsToDispose = [];
     var data /*: HMRMessage */ = JSON.parse(event.data);
@@ -136,10 +135,19 @@ if ((!parent || !parent.isParcelRequire) && typeof WebSocket !== 'undefined') {
           window.dispatchEvent(new CustomEvent('parcelhmraccept'));
         }
         await hmrApplyUpdates(assets);
-        hmrDisposeQueue();
+
+        // Dispose all old assets.
+        let processedAssets = {} /*: {|[string]: boolean|} */;
+        for (let i = 0; i < assetsToDispose.length; i++) {
+          let id = assetsToDispose[i][1];
+          if (!processedAssets[id]) {
+            hmrDispose(assetsToDispose[i][0], id);
+            processedAssets[id] = true;
+          }
+        }
 
         // Run accept callbacks. This will also re-execute other disposed assets in topological order.
-        let processedAssets = {};
+        processedAssets = {};
         for (let i = 0; i < assetsToAccept.length; i++) {
           let id = assetsToAccept[i][1];
           if (!processedAssets[id]) {
@@ -377,11 +385,7 @@ function hmrApply(bundle /*: ParcelRequire */, asset /*:  HMRAsset */) {
       // $FlowFixMe
       let fn = global.parcelHotUpdate[asset.id];
       modules[asset.id] = [fn, deps];
-    }
-
-    // Always traverse to the parent bundle, even if we already replaced the asset in this bundle.
-    // This is required in case modules are duplicated. We need to ensure all instances have the updated code.
-    if (bundle.parent) {
+    } else if (bundle.parent) {
       hmrApply(bundle.parent, asset);
     }
   }
@@ -465,17 +469,6 @@ function hmrAcceptCheckOne(bundle /*: ParcelRequire */, id /*: string */, depsBy
     return true;
   }
 }
-function hmrDisposeQueue() {
-  // Dispose all old assets.
-  for (let i = 0; i < assetsToDispose.length; i++) {
-    let id = assetsToDispose[i][1];
-    if (!disposedAssets[id]) {
-      hmrDispose(assetsToDispose[i][0], id);
-      disposedAssets[id] = true;
-    }
-  }
-  assetsToDispose = [];
-}
 function hmrDispose(bundle /*: ParcelRequire */, id /*: string */) {
   var cached = bundle.cache[id];
   bundle.hotData[id] = {};
@@ -496,23 +489,18 @@ function hmrAccept(bundle /*: ParcelRequire */, id /*: string */) {
   // Run the accept callbacks in the new version of the module.
   var cached = bundle.cache[id];
   if (cached && cached.hot && cached.hot._acceptCallbacks.length) {
-    let assetsToAlsoAccept = [];
     cached.hot._acceptCallbacks.forEach(function (cb) {
-      let additionalAssets = cb(function () {
+      var assetsToAlsoAccept = cb(function () {
         return getParents(module.bundle.root, id);
       });
-      if (Array.isArray(additionalAssets) && additionalAssets.length) {
-        assetsToAlsoAccept.push(...additionalAssets);
+      if (assetsToAlsoAccept && assetsToAccept.length) {
+        assetsToAlsoAccept.forEach(function (a) {
+          hmrDispose(a[0], a[1]);
+        });
+
+        // $FlowFixMe[method-unbinding]
+        assetsToAccept.push.apply(assetsToAccept, assetsToAlsoAccept);
       }
     });
-    if (assetsToAlsoAccept.length > 0) {
-      let handled = assetsToAlsoAccept.every(function (a) {
-        return hmrAcceptCheck(a[0], a[1]);
-      });
-      if (!handled) {
-        return fullReload();
-      }
-      hmrDisposeQueue();
-    }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atlaspack/runtime-browser-hmr",
-  "version": "2.14.5-dev.1c70d50f9.99+1c70d50f9",
+  "version": "2.14.5-dev.69+67cb517ae",
   "license": "(MIT OR Apache-2.0)",
   "publishConfig": {
     "access": "public"
@@ -15,9 +15,9 @@
     "node": ">= 16.0.0"
   },
   "dependencies": {
-    "@atlaspack/plugin": "2.14.5-dev.1c70d50f9.99+1c70d50f9",
-    "@atlaspack/utils": "2.14.5-dev.1c70d50f9.99+1c70d50f9"
+    "@atlaspack/plugin": "2.14.5-dev.69+67cb517ae",
+    "@atlaspack/utils": "2.14.5-dev.69+67cb517ae"
   },
   "type": "commonjs",
-  "gitHead": "1c70d50f914cb662515b0b61053e51a06f3af234"
+  "gitHead": "67cb517ae793046fb5a0d2ef02ba74510fefccf3"
 }

package/src/loaders/hmr-runtime.js

@@ -72,7 +72,6 @@ module.bundle.Module = Module;
 module.bundle.hotData = {};
 
 var checkedAssets /*: {|[string]: boolean|} */,
-  disposedAssets /*: {|[string]: boolean|} */,
   assetsToDispose /*: Array<[ParcelRequire, string]> */,
   assetsToAccept /*: Array<[ParcelRequire, string]> */;
 
@@ -135,7 +134,6 @@ if ((!parent || !parent.isParcelRequire) && typeof WebSocket !== 'undefined') {
   // $FlowFixMe
   ws.onmessage = async function (event /*: {data: string, ...} */) {
     checkedAssets = ({} /*: {|[string]: boolean|} */);
-    disposedAssets = ({} /*: {|[string]: boolean|} */);
     assetsToAccept = [];
     assetsToDispose = [];
 
@@ -175,10 +173,19 @@ if ((!parent || !parent.isParcelRequire) && typeof WebSocket !== 'undefined') {
 
         await hmrApplyUpdates(assets);
 
-        hmrDisposeQueue();
+        // Dispose all old assets.
+        let processedAssets = ({} /*: {|[string]: boolean|} */);
+        for (let i = 0; i < assetsToDispose.length; i++) {
+          let id = assetsToDispose[i][1];
+
+          if (!processedAssets[id]) {
+            hmrDispose(assetsToDispose[i][0], id);
+            processedAssets[id] = true;
+          }
+        }
 
         // Run accept callbacks. This will also re-execute other disposed assets in topological order.
-        let processedAssets = {};
+        processedAssets = {};
         for (let i = 0; i < assetsToAccept.length; i++) {
           let id = assetsToAccept[i][1];
 
@@ -486,11 +493,7 @@ function hmrApply(bundle /*: ParcelRequire */, asset /*:  HMRAsset */) {
       // $FlowFixMe
       let fn = global.parcelHotUpdate[asset.id];
       modules[asset.id] = [fn, deps];
-    }
-
-    // Always traverse to the parent bundle, even if we already replaced the asset in this bundle.
-    // This is required in case modules are duplicated. We need to ensure all instances have the updated code.
-    if (bundle.parent) {
+    } else if (bundle.parent) {
       hmrApply(bundle.parent, asset);
     }
   }
@@ -594,20 +597,6 @@ function hmrAcceptCheckOne(
   }
 }
 
-function hmrDisposeQueue() {
-  // Dispose all old assets.
-  for (let i = 0; i < assetsToDispose.length; i++) {
-    let id = assetsToDispose[i][1];
-
-    if (!disposedAssets[id]) {
-      hmrDispose(assetsToDispose[i][0], id);
-      disposedAssets[id] = true;
-    }
-  }
-
-  assetsToDispose = [];
-}
-
 function hmrDispose(bundle /*: ParcelRequire */, id /*: string */) {
   var cached = bundle.cache[id];
   bundle.hotData[id] = {};
@@ -631,26 +620,18 @@ function hmrAccept(bundle /*: ParcelRequire */, id /*: string */) {
   // Run the accept callbacks in the new version of the module.
   var cached = bundle.cache[id];
   if (cached && cached.hot && cached.hot._acceptCallbacks.length) {
-    let assetsToAlsoAccept = [];
     cached.hot._acceptCallbacks.forEach(function (cb) {
-      let additionalAssets = cb(function () {
+      var assetsToAlsoAccept = cb(function () {
         return getParents(module.bundle.root, id);
       });
-      if (Array.isArray(additionalAssets) && additionalAssets.length) {
-        assetsToAlsoAccept.push(...additionalAssets);
-      }
-    });
-
-    if (assetsToAlsoAccept.length > 0) {
-      let handled = assetsToAlsoAccept.every(function (a) {
-        return hmrAcceptCheck(a[0], a[1]);
-      });
+      if (assetsToAlsoAccept && assetsToAccept.length) {
+        assetsToAlsoAccept.forEach(function (a) {
+          hmrDispose(a[0], a[1]);
+        });
 
-      if (!handled) {
-        return fullReload();
+        // $FlowFixMe[method-unbinding]
+        assetsToAccept.push.apply(assetsToAccept, assetsToAlsoAccept);
       }
-
-      hmrDisposeQueue();
-    }
+    });
   }
 }