zapier-platform icon indicating copy to clipboard operation
zapier-platform copied to clipboard

ZapierPromise does not use scoped prototype functionality

Open codebycaleb opened this issue 3 years ago • 4 comments

Bug Description

ZapierPromise does not adhere to bluebird's scoped prototypes recommendations regarding requireing bluebird.

Reproduction Steps

  1. Create an app that uses bluebird for promises
  2. Create a Zap using the app and test it via the editor

Errors will be logged that cite bluebird's documentation about not invoking the Promise constructor directly.

The code should work without errors.

Version Info

  • Version info: core: 11.2.0
  • App id: 154066

codebycaleb avatar Feb 02 '22 12:02 codebycaleb

@codebycaleb do you have a screenshot of how dire the error messages look? Trying to gauge severity here. And does this only affect CLI apps which separately use bluebird promises? If so, seems fairly narrow. We were just talking about pulling the bluebird stuff out, but it's esoteric and working just fine, so we're hesitant to touch it if we can avoid it.

xavdid avatar Feb 02 '22 22:02 xavdid

Yep! It's not mine though. 😅 Here's an obfuscated version:

And does this only affect CLI apps which separately use bluebird promises?

Yeah I think so! This app had a dependency for bluebird@^3.7.2. Definitely agreed on the narrow point - this is the first report I've ever seen and as far as I know this has essentially always been the case. :)

codebycaleb avatar Feb 02 '22 22:02 codebycaleb

ok, cool! Good to know.

Yeah, it's the first I've heard of it too. We'll keep this open in case other people have the same issue, but it doesn't strike me as a priority.

xavdid avatar Feb 02 '22 23:02 xavdid

Just noting I have very similar symptoms but I am not using bluebird in my app directly, it's just imported as a dependency of zapier-platform-core

[email protected] /Users/dpovey/src/ppcsamurai/zapier-adpulse.io
└─┬ [email protected]
  └── [email protected]

When I try to test my Zap I get about 20% of the time the error:

image

And then when activating my Zap about a similar percentage: image

dpovey avatar Feb 04 '22 07:02 dpovey