.github/workflows/node.js.yml

@@ -16,7 +16,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [12.x, 14.x, 16.x, 18.x]
+        node-version: [12.x, 14.x, 16.x, 18.x, 20.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:

package.json

@@ -16,7 +16,7 @@
     "desugaring",
     "ES6"
   ],
-  "version": "0.14.9",
+  "version": "0.14.10",
   "homepage": "http://github.com/facebook/regenerator",
   "repository": {
     "type": "git",
@@ -34,9 +34,9 @@
     "commoner": "^0.10.8",
     "private": "^0.1.8",
     "recast": "^0.21.5",
-    "regenerator-preset": "^0.14.0",
-    "regenerator-runtime": "^0.13.10",
-    "regenerator-transform": "^0.15.0",
+    "regenerator-preset": "^0.14.1",
+    "regenerator-runtime": "^0.13.11",
+    "regenerator-transform": "^0.15.2",
     "through": "^2.3.8"
   },
   "devDependencies": {

packages/preset/package.json

@@ -2,7 +2,7 @@
   "name": "regenerator-preset",
   "author": "Ben Newman <bn@cs.stanford.edu>",
   "description": "Babel preset for easy use of regenerator-transform.",
-  "version": "0.14.0",
+  "version": "0.14.1",
   "main": "index.js",
   "keywords": [
     "regenerator",

packages/runtime/package.json

@@ -2,7 +2,7 @@
   "name": "regenerator-runtime",
   "author": "Ben Newman <bn@cs.stanford.edu>",
   "description": "Runtime for Regenerator-compiled generator and async functions.",
-  "version": "0.13.11",
+  "version": "0.14.0",
   "main": "runtime.js",
   "keywords": [
     "regenerator",

packages/runtime/runtime.js

@@ -487,7 +487,7 @@ var runtime = (function (exports) {
   };
 
   function values(iterable) {
-    if (iterable) {
+    if (iterable || iterable === "") {
       var iteratorMethod = iterable[iteratorSymbol];
       if (iteratorMethod) {
         return iteratorMethod.call(iterable);
@@ -517,8 +517,7 @@ var runtime = (function (exports) {
       }
     }
 
-    // Return an iterator with no values.
-    return { next: doneResult };
+    throw new TypeError(typeof iterable + " is not iterable");
   }
   exports.values = values;
 

packages/transform/package.json

@@ -2,7 +2,7 @@
   "name": "regenerator-transform",
   "author": "Ben Newman <bn@cs.stanford.edu>",
   "description": "Explode async and generator functions into a state machine.",
-  "version": "0.15.0",
+  "version": "0.15.2",
   "main": "lib/index.js",
   "keywords": [
     "regenerator",

packages/transform/src/emit.js

@@ -57,8 +57,13 @@ exports.Emitter = Emitter;
 // the amazingly convenient benefit of allowing the exact value of the
 // location to be determined at any time, even after generating code that
 // refers to the location.
+// We use 'Number.MAX_VALUE' to mark uninitialized location. We can safely do
+// so because no code can realistically have about 1.8e+308 locations before
+// hitting memory limit of the machine it's running on. For comparison, the
+// estimated number of atoms in the observable universe is around 1e+80.
+const PENDING_LOCATION = Number.MAX_VALUE;
 Ep.loc = function() {
-  const l = util.getTypes().numericLiteral(-1)
+  const l = util.getTypes().numericLiteral(PENDING_LOCATION)
   this.insertedLocs.add(l);
   return l;
 }
@@ -76,7 +81,7 @@ Ep.getContextId = function() {
 Ep.mark = function(loc) {
   util.getTypes().assertLiteral(loc);
   let index = this.listing.length;
-  if (loc.value === -1) {
+  if (loc.value === PENDING_LOCATION) {
     loc.value = index;
   } else {
     // Locations can be marked redundantly, but their values cannot change
@@ -644,7 +649,7 @@ Ep.explodeStatement = function(path, labelId) {
     );
 
     self.mark(after);
-    if (defaultLoc.value === -1) {
+    if (defaultLoc.value === PENDING_LOCATION) {
       self.mark(defaultLoc);
       assert.strictEqual(after.value, defaultLoc.value);
     }
@@ -885,7 +890,7 @@ Ep.updateContextPrevLoc = function(loc) {
   if (loc) {
     t.assertLiteral(loc);
 
-    if (loc.value === -1) {
+    if (loc.value === PENDING_LOCATION) {
       // If an uninitialized location literal was passed in, set its value
       // to the current this.listing.length.
       loc.value = this.listing.length;

test/tests.es6.js

@@ -1504,6 +1504,28 @@ describe("delegated yield", function() {
     check(gen(iterator), [], "1foo");
   });
 
+  it("should work with empty string", function() {
+    function* f() {
+      yield* "";
+    };
+
+    assert.deepEqual(f().next(), { value: undefined, done: true });
+  });
+
+  it("should throw if not iterable", function() {
+    function* f(x) {
+      yield* x;
+    };
+
+    assert.throws(() => f(undefined).next(), TypeError);
+    assert.throws(() => f(null).next(), TypeError);
+    assert.throws(() => f(false).next(), TypeError);
+    assert.throws(() => f(true).next(), TypeError);
+    assert.throws(() => f(0).next(), TypeError);
+    assert.throws(() => f(1).next(), TypeError);
+    assert.throws(() => f({}).next(), TypeError);
+  });
+
   it("should throw if the delegated iterable's iterator doesn't have .next", function() {
     var it = function* () {
       yield* {