Use PromiseResolve and PerformPromiseThen directly

[petamoriken]

The current draft uses methods like %Promise.all% directly and is not resistant to rewriting Promise.resolve or Promise.prototype.then.

const originalThen = Promise.prototype.then;
Promise.prototype.then = function () { console.log("bang!"); originalThen.apply(this, arguments) };

// bang!
await.all [new Promise((r) => r("foo"))];

I would like to suggest using PromiseResolve and PerformPromiseThen instead like the await operator, so that it can be worked with in Node.js and Deno's internal code.

FYI: denoland/deno#16196

[zloirock]

Passed to await.all promises can be third-party and PerformPromiseThen will not work with them - all such features work by thenable protocol.

[ljharb]

Indeed it should be as robust as await.

[ljharb]

However, it’s far too early for anyone to implement this proposal - until stage 3 - and the spec text won’t be finalized until stage 2 - so there’s time to adjust it.