James M Snell comments

Results 646 comments of


                                            James M Snell

Implement v8::Exception::CaptureStackTrace for DOMException

Looks like it'll be v8 12.6 before the upstream change will be available here.

Node.js standards-positions

Yeah, that obviously works as well and is largely what I expected, but wanted to make sure to give y'all a heads up on it in case you saw this...

Implements the web platform standard reportError API

> ... what happens here with Tail Workers? I think that still needs to be determined. The reporting is the same as the existing `logUncaughtException(...)` but I don't believe that...

Implements the web platform standard reportError API

@irvinebroque ... ok, the PR has been updated to publish the reported error to the worker tracer as an exception. Unfortunately we can't actually have a test to verify it...

V8 patch for v8::Date::ToUTCString and v8::Date::Parse

I will be contributing these changes directly to v8. Once that lands there, we can either float that version of the patch until we pick it up in the regular...

V8 patch for v8::Date::ToUTCString and v8::Date::Parse

https://chromium-review.googlesource.com/c/v8/v8/+/5444005

V8 patch for v8::Date::ToUTCString and v8::Date::Parse

Happy days... the v8 change has been accepted. Will just be waiting on us to receive the v8 update that includes the new APIs and this PR can be updated...

V8 patch for v8::Date::ToUTCString and v8::Date::Parse

Looks like it'll be v8 12.6 before the upstream change will land

Add ReadableStreamBYOBReader.read(view, { min })

Woo! super happy to see this land. Thanks for the work all.

Proposal: ReadableStreamBodyReader

What are folks' thoughts about advancing this proposal forward? I think for the `FormData` issue we could actually just omit `formData()` from this proposal.