Stage 3 / 4 reviews

[js-choi]

As per the TC39 process document.

• @ljharb
• @nicolo-ribaudo

Editors (tc39/ecma262#3581):

• @michaelficarra
• @bakkot
• @syg

[nicolo-ribaudo]

I opened #19 with an observation/idea, but apart from that the spec looks good. ✔️

There is a minor editorial bug: the Abstract Closure should capture thisArg.

[js-choi]

Since #20 has been merged, I have marked your review as completed, @nicolo-ribaudo. Thank you again.

[ljharb]

For my review:

• I'd love to see the %Array.prototype.values% simplification upstreamed to Array.from in 262 as a PR, both as an improvement and also to minimize from/fromAsync diffs
• after the spec PR lands, it'd be great to figure out if there's a way to use one or more common AOs to maximally share steps between from and fromAsync

Otherwise, LGTM.

[js-choi]

@nicolo-ribaudo, @ljharb:
The spec has had significant changes due to the reversion of #20 in #27, so that its behavior would better match for await and AsyncIterator.prototype.toArray (see
#19 (comment)). Hopefully we’ve got everything set now.

I’d like to request another review of the proposal. I would like to try advancing this proposal to Stage 3 at the 2022-09 plenary meeting. Thank you again; please let me know here if you may not be able to adequately review by 2022-09-01. Thank you sincerely for your time.

[js-choi]

@ljharb, @nicolo-ribaudo: This is another ping before next week’s plenary meeting to re-review the proposal spec within the next few days. Let us know once you re-review—or if you anticipate that you won’t be able to properly review it by then. Thank you!

[nicolo-ribaudo]

I will do this on Friday, sorry for being late.

[ljharb]

Spec seems good to me.

[nicolo-ribaudo]

I will do this on Friday, sorry for being late.

I forgot to reply before plenary, but I re-reviewed the proposal spec and it still looks good.

(Also, I like how you extended AsyncBlockStart because I need the same for another proposal!)

[js-choi]

We got Stage 3, conditional on editor approval.

@syg, @michaelficarra, @bakkot: Please let us know when you have been able to review the spec.

(The typo that @waldemarhorwat found during the plenary has already been fixed.)

[mgaudet]

This is already shipping in Safari Technical Preview, despite being only 'conditionally' stage 3. Firefox has an implementation and is ready to start shipping to nightly when this is officially stage 3.

Any updates on editorial review @syg, @bakkot or @michaelficarra ?