Meteor 1.8.2-beta.17, ReferenceError: meteorBabelHelpers is not defined

doronrk · 2019-09-09T23:28:45.649Z

I’m using Meteor 1.8.2-beta.17, and running into the same issue described here. It seems this is a result of @benjamn’s PR, Move meteorBabelHelpers to modules package, but more likely the fault of the peerlibrary/meteor-subscription-scope package which has a for-in loop in coffeescript.

Issues related to babel/transpilation/legacy-bundles are a little over my head at the moment, but what is the proper way to solve this?

I’m using an up-to-date version of Chrome if that’s relevant: Version 76.0.3809.132 (Official Build) (64-bit)

Thanks for your help!

update: I just put up this PR which solves the problem locally for me, but perhaps it isn’t the most correct solution. @mitar maybe you can help out here.

coagmano · 2019-09-10T01:11:38.534Z

I suspect your app may be loading a cached version of the package which is compiled slightly differently to what’s expected after the commit you’ve linked.

Try removing the cache from .meteor/packages/peerlibrary_meteor-subscription-scope/ and performing a meteor reset to force it to re-compile?

doronrk · 2019-09-10T03:43:52.759Z

Hmm. Thanks for your suggestion! Unfortunately I still get the same error after taking those steps. Let me know if there’s any other information that would be useful.

I suppose my confusion stems from the fact that on the one hand, I’m not using a legacy browser, so it makes sense that after the meteor PR meteorBabelHelpers is not defined, but on the other hand, peerlibrary/meteor-subscription-scope seems to require meteorBabelHelpers since it throws an error on line 25 of client.coffee:

for field of res when field.lastIndexOf('_sub_', 0) is 0

with Uncaught ReferenceError: meteorBabelHelpers is not defined

Perhaps benjamn’s discussion on this issue here can provide more insight:

The meteorBabelHelpers namespace is used for only one thing: meteorBabelHelpers.sanitizeForInLoop , which is used to sanitize objects looped over by for - in loops so that those loops behave the same way in all browsers.

So I’m guessing you’ve got a .coffee file with a for - in loop enclosed by backticks.

Given this analysis, although you could still solve this by depending on babel-runtime , the coffeescript package should really api.imply the same runtime environment that ecmascript provides.

There aren’t any back-ticks in this case, but the two scenarios do seem suspiciously similar.

coagmano · 2019-09-10T07:23:09.102Z

hmm, I wonder if it’s because peerlibrary:subscription-scope has api.versionsFrom('METEOR@1.4.4.5');

If you make a local fork and update the versionsFrom, does it help?

Otherwise it might be the coffeescript compiler ?

doronrk · 2019-09-10T19:56:57.134Z

Thanks again for the suggestion @coagmano - still no luck unfortunately. The only thing that’s worked so far so converting the loop.

coagmano · 2019-09-10T23:24:31.268Z

That’s unfortunate. Seems like such a strange error

doronrk · 2019-09-29T19:33:43.423Z

For posterity:

github.com/peerlibrary/meteor-subscription-scope

Convert for..of object loop to for..in range loop in client.coffee

by doronrk on 12:16AM - 10 Sep 19 UTC

1 commits changed 1 files with 5 additions and 2 deletions.

github.com/meteor/meteor

Release 1.8.2

by benjamn on 04:40PM - 10 Apr 19 UTC

263 commits changed 280 files with 7319 additions and 4637 deletions.