drop legacy iterator helpers modules

zloirock · Oct 8, 2024 · bf737c4 · bf737c4
1 parent 4bb841e
commit bf737c4
Show file tree

Hide file tree

Showing 30 changed files with 16 additions and 95 deletions.
diff --git a/packages/core-js-compat/src/data.mjs b/packages/core-js-compat/src/data.mjs
@@ -2084,36 +2084,10 @@ export const data = {
   },
   'esnext.function.metadata': {
   },
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.constructor': null,
   'esnext.iterator.dispose': {
   },
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.drop': null,
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.every': null,
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.filter': null,
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.find': null,
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.flat-map': null,
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.for-each': null,
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.from': null,
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.map': null,
   'esnext.iterator.range': {
   },
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.reduce': null,
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.some': null,
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.take': null,
-  // TODO: Remove from `core-js@4`
-  'esnext.iterator.to-array': null,
   'esnext.iterator.to-async': {
   },
   'esnext.json.is-raw-json': {
@@ -2428,20 +2402,7 @@ export const data = {
 };
 
 export const renamed = new Map([
-  // TODO: Clean in `core-js@4`
-  ['esnext.iterator.constructor', 'es.iterator.constructor'],
-  ['esnext.iterator.drop', 'es.iterator.drop'],
-  ['esnext.iterator.every', 'es.iterator.every'],
-  ['esnext.iterator.filter', 'es.iterator.filter'],
-  ['esnext.iterator.find', 'es.iterator.find'],
-  ['esnext.iterator.flat-map', 'es.iterator.flat-map'],
-  ['esnext.iterator.for-each', 'es.iterator.for-each'],
-  ['esnext.iterator.from', 'es.iterator.from'],
-  ['esnext.iterator.map', 'es.iterator.map'],
-  ['esnext.iterator.reduce', 'es.iterator.reduce'],
-  ['esnext.iterator.some', 'es.iterator.some'],
-  ['esnext.iterator.take', 'es.iterator.take'],
-  ['esnext.iterator.to-array', 'es.iterator.to-array'],
+  // empty
 ]);
 
 for (const [old, nw] of renamed) data[old] = data[nw];

diff --git a/packages/core-js/modules/es.iterator.drop.js b/packages/core-js/modules/es.iterator.drop.js
@@ -25,7 +25,7 @@ var IteratorProxy = createIteratorProxy(function () {
 
 // `Iterator.prototype.drop` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true, forced: IS_PURE }, {
   drop: function drop(limit) {
     anObject(this);

diff --git a/packages/core-js/modules/es.iterator.every.js b/packages/core-js/modules/es.iterator.every.js
@@ -7,7 +7,7 @@ var getIteratorDirect = require('../internals/get-iterator-direct');
 
 // `Iterator.prototype.every` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true }, {
   every: function every(predicate) {
     anObject(this);

diff --git a/packages/core-js/modules/es.iterator.filter.js b/packages/core-js/modules/es.iterator.filter.js
@@ -24,7 +24,7 @@ var IteratorProxy = createIteratorProxy(function () {
 
 // `Iterator.prototype.filter` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true, forced: IS_PURE }, {
   filter: function filter(predicate) {
     anObject(this);

diff --git a/packages/core-js/modules/es.iterator.find.js b/packages/core-js/modules/es.iterator.find.js
@@ -7,7 +7,7 @@ var getIteratorDirect = require('../internals/get-iterator-direct');
 
 // `Iterator.prototype.find` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true }, {
   find: function find(predicate) {
     anObject(this);

diff --git a/packages/core-js/modules/es.iterator.flat-map.js b/packages/core-js/modules/es.iterator.flat-map.js
@@ -33,7 +33,7 @@ var IteratorProxy = createIteratorProxy(function () {
 
 // `Iterator.prototype.flatMap` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true, forced: IS_PURE }, {
   flatMap: function flatMap(mapper) {
     anObject(this);

diff --git a/packages/core-js/modules/es.iterator.for-each.js b/packages/core-js/modules/es.iterator.for-each.js
@@ -7,7 +7,7 @@ var getIteratorDirect = require('../internals/get-iterator-direct');
 
 // `Iterator.prototype.forEach` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true }, {
   forEach: function forEach(fn) {
     anObject(this);

diff --git a/packages/core-js/modules/es.iterator.from.js b/packages/core-js/modules/es.iterator.from.js
@@ -14,7 +14,7 @@ var IteratorProxy = createIteratorProxy(function () {
 
 // `Iterator.from` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', stat: true, forced: IS_PURE }, {
   from: function from(O) {
     var iteratorRecord = getIteratorFlattenable(typeof O == 'string' ? toObject(O) : O, true);

diff --git a/packages/core-js/modules/es.iterator.map.js b/packages/core-js/modules/es.iterator.map.js
@@ -17,7 +17,7 @@ var IteratorProxy = createIteratorProxy(function () {
 
 // `Iterator.prototype.map` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true, forced: IS_PURE }, {
   map: function map(mapper) {
     anObject(this);

diff --git a/packages/core-js/modules/es.iterator.reduce.js b/packages/core-js/modules/es.iterator.reduce.js
@@ -9,7 +9,7 @@ var $TypeError = TypeError;
 
 // `Iterator.prototype.reduce` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true }, {
   reduce: function reduce(reducer /* , initialValue */) {
     anObject(this);

diff --git a/packages/core-js/modules/es.iterator.some.js b/packages/core-js/modules/es.iterator.some.js
@@ -7,7 +7,7 @@ var getIteratorDirect = require('../internals/get-iterator-direct');
 
 // `Iterator.prototype.some` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true }, {
   some: function some(predicate) {
     anObject(this);

diff --git a/packages/core-js/modules/es.iterator.take.js b/packages/core-js/modules/es.iterator.take.js
@@ -22,7 +22,7 @@ var IteratorProxy = createIteratorProxy(function () {
 
 // `Iterator.prototype.take` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true, forced: IS_PURE }, {
   take: function take(limit) {
     anObject(this);

diff --git a/packages/core-js/modules/es.iterator.to-array.js b/packages/core-js/modules/es.iterator.to-array.js
@@ -8,7 +8,7 @@ var push = [].push;
 
 // `Iterator.prototype.toArray` method
 // https://github.com/tc39/proposal-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true }, {
   toArray: function toArray() {
     var result = [];

diff --git a/packages/core-js/modules/esnext.iterator.constructor.js b/packages/core-js/modules/esnext.iterator.constructor.js
diff --git a/packages/core-js/modules/esnext.iterator.drop.js b/packages/core-js/modules/esnext.iterator.drop.js
diff --git a/packages/core-js/modules/esnext.iterator.every.js b/packages/core-js/modules/esnext.iterator.every.js
diff --git a/packages/core-js/modules/esnext.iterator.filter.js b/packages/core-js/modules/esnext.iterator.filter.js
diff --git a/packages/core-js/modules/esnext.iterator.find.js b/packages/core-js/modules/esnext.iterator.find.js
diff --git a/packages/core-js/modules/esnext.iterator.flat-map.js b/packages/core-js/modules/esnext.iterator.flat-map.js
diff --git a/packages/core-js/modules/esnext.iterator.for-each.js b/packages/core-js/modules/esnext.iterator.for-each.js
diff --git a/packages/core-js/modules/esnext.iterator.from.js b/packages/core-js/modules/esnext.iterator.from.js
diff --git a/packages/core-js/modules/esnext.iterator.map.js b/packages/core-js/modules/esnext.iterator.map.js
diff --git a/packages/core-js/modules/esnext.iterator.range.js b/packages/core-js/modules/esnext.iterator.range.js
@@ -97,7 +97,7 @@ addGetter('step');
 
 // `Iterator.range` method
 // https://github.com/tc39/proposal-Number.range
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', stat: true, forced: true }, {
   range: function range(start, end, option) {
     if (typeof start == 'number') return new $NumericRangeIterator(start, end, option, 'number', 0, 1);

diff --git a/packages/core-js/modules/esnext.iterator.reduce.js b/packages/core-js/modules/esnext.iterator.reduce.js
diff --git a/packages/core-js/modules/esnext.iterator.some.js b/packages/core-js/modules/esnext.iterator.some.js
diff --git a/packages/core-js/modules/esnext.iterator.take.js b/packages/core-js/modules/esnext.iterator.take.js
diff --git a/packages/core-js/modules/esnext.iterator.to-array.js b/packages/core-js/modules/esnext.iterator.to-array.js
diff --git a/packages/core-js/modules/esnext.iterator.to-async.js b/packages/core-js/modules/esnext.iterator.to-async.js
@@ -8,7 +8,7 @@ var IS_PURE = require('../internals/is-pure');
 
 // `Iterator.prototype.toAsync` method
 // https://github.com/tc39/proposal-async-iterator-helpers
-// dependency: esnext.iterator.constructor
+// dependency: es.iterator.constructor
 $({ target: 'Iterator', proto: true, real: true, forced: IS_PURE }, {
   toAsync: function toAsync() {
     return new WrapAsyncIterator(getIteratorDirect(new AsyncFromSyncIterator(getIteratorDirect(anObject(this)))));

diff --git a/scripts/build-entries/entries-definitions.mjs b/scripts/build-entries/entries-definitions.mjs
@@ -3649,7 +3649,6 @@ export const proposals = {
     link: 'https://github.com/tc39/proposal-Number.range',
     stage: 2,
     modules: [
-      'es.iterator.constructor',
       'esnext.iterator.range',
     ],
   },

diff --git a/tests/compat-tools/compat.mjs b/tests/compat-tools/compat.mjs
@@ -126,7 +126,7 @@ deepEqual(compat({
 const { list: inverted1 } = compat({ targets: { esmodules: true }, inverse: true });
 
 ok(inverted1.includes('es.symbol.iterator'), 'inverse #1');
-ok(!inverted1.includes('esnext.iterator.from'), 'inverse #2');
+ok(!inverted1.includes('esnext.iterator.range'), 'inverse #2');
 // TODO: enable after adding new stabilized aliases in `core-js@4`
 // ok(!inverted1.includes('esnext.array.at'), 'inverse #3');