@html-next/vertical-collection 4.0.0 → 4.0.2

package/CHANGELOG.md

@@ -2,6 +2,28 @@ Changelog
 =========
 
 
+
+
+## v4.0.2 (2022-11-17)
+
+#### :bug: Bug Fix
+* [#396](https://github.com/html-next/vertical-collection/pull/396) Fix build for ember-source 4.0+ ([@wagenet](https://github.com/wagenet))
+
+#### Committers: 1
+- Peter Wagenet ([@wagenet](https://github.com/wagenet))
+
+## v4.0.1 (2022-11-07)
+
+#### :memo: Documentation
+* [#381](https://github.com/html-next/vertical-collection/pull/381) Fix README ([@runspired](https://github.com/runspired))
+
+#### Committers: 4
+- Alex Navasardyan ([@twokul](https://github.com/twokul))
+- Chris Thoburn ([@runspired](https://github.com/runspired))
+- Matthew Beale ([@mixonic](https://github.com/mixonic))
+- [@Atrue](https://github.com/Atrue)
+
+
 ## v4.0.0 (2022-09-12)
 
 * Drops support for Ember < 3.12-LTS.

package/README.md

@@ -88,7 +88,7 @@ It works via a scrollable div or scrollable body.
 
 ### Svelte Everything
 
-If it can be trimmer, smoke-and-mirrors likes to trim it.
+If it can be trimmer, vertical-collection likes to trim it.
 
 ## Status
 

package/addon/components/vertical-collection/component.js

@@ -327,6 +327,7 @@ const VerticalCollection = Component.extend({
     _radar.bufferSize = this.bufferSize;
 
     _radar.scheduleUpdate(true);
+    this._clearScheduledActions();
 
     return _radar.virtualComponents;
   }),
@@ -335,6 +336,12 @@ const VerticalCollection = Component.extend({
     return scheduler.schedule(queueName, job, this.token);
   },
 
+  _clearScheduledActions() {
+    clearTimeout(this._nextSendActions);
+    this._nextSendActions = null;
+    this._scheduledActions.length = 0;
+  },
+
   _scheduleSendAction(action, index) {
     this._scheduledActions.push([action, index]);
 

package/index.js

@@ -19,16 +19,16 @@ module.exports = {
     return '';
   },
 
-  // Borrowed from ember-cli-babel
-  _emberVersionRequiresModulesAPIPolyfill() {
-    let checker = this.checker.for('ember-source', 'npm');
+    // Borrowed from ember-cli-babel
+    _emberVersionRequiresModulesAPIPolyfill() {
+      let checker = this.checker.for('ember-source', 'npm');
 
-    if (!checker.exists()) {
-      return true;
-    }
+      if (!checker.exists()) {
+        return true;
+      }
 
-    return checker.lt('3.27.0-alpha.1');
-  },
+      return checker.lt('3.27.0-alpha.1');
+    },
 
   treeForAddon(tree) {
     let babel = this.addons.find((addon) => addon.name === 'ember-cli-babel');
@@ -41,38 +41,34 @@ module.exports = {
       destDir: '@html-next/vertical-collection'
     });
 
-    // When compiling with `compileModules: false`, ember-cli-babel defaults to
-    // using the modules polyfill, since it assumes we are concatenating the
-    // output script using `app.import` without an AMD wrapper.
-    //
-    // This does not apply to us, since we are compiling the `-private` modules
-    // into a single AMD module (via rollup below), which can in fact have
-    // external dependencies.
-    //
-    // We can opt-out of this with `disableEmberModulesAPIPolyfill: true`. In
-    // Ember versions with "real modules", that is what we want in order to
-    // avoid the Ember global deprecation (or just completely not working in
-    // 4.0+).
-    //
-    // It seems like the intent may have been that we should be able to set
-    // this to `true` unconditionally, and `ember-cli-babel` will ignore this
-    // setting if the Ember verion requires the modules API polyfill. However,
-    // presumably due to a bug, ember-cli-babel actually checks for this value
-    // first and return out of the function early if its value is truthy. This
-    // means that if we set this to true unconditionally, then we would have
-    // disabled the modules polyfill for Ember versions that needs it, which
-    // would be incorrect. Therefore, we have to duplicate the detection logic
-    // here in order to set this value appropriately.
-    //
-    // Ideally, we should just stop trying to rollup the -private modules and
-    // let the modern build pipeline optimizes things for us, then none of this
-    // would have been necessary.
     let privateTree = babel.transpileTree(withPrivate, {
       babel: this.options.babel,
       'ember-cli-babel': {
+        // we leave our output as valid ES
+        // for the consuming app's config to transpile as desired
+        // so we don't want to compileModules to amd here
         compileModules: false,
-        disableEmberModulesAPIPolyfill:
-          !this._emberVersionRequiresModulesAPIPolyfill(),
+
+        disableEmberModulesAPIPolyfill: !this._emberVersionRequiresModulesAPIPolyfill(),
+
+        // TODO for the embroider world we want to leave our -private module
+        // as an es module and only transpile the few things we genuinely care about.
+        // ideally this would occur as a pre-publish step so that consuming apps would
+        // just see a `-private.js` file and not pay any additional costs.
+        // CURRENTLY we transpile the -private module fully acccording to the
+        // consuming app's config, so we must leave these enabled.
+        disablePresetEnv: false,
+        disableDebugTooling: false,
+        disableDecoratorTransforms: false,
+        enableTypeScriptTransform: true,
+
+        throwUnlessParallelizable: true,
+
+        // consuming app will take care of this if needed,
+        // we don't need to also include
+        includePolyfill: false,
+
+        extensions: ['js', 'ts'],
       },
     });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@html-next/vertical-collection",
-  "version": "4.0.0",
+  "version": "4.0.2",
   "description": "infinite-scroll, done right. done.",
   "keywords": [
     "occlusion",