Tim Fish

Results 398 comments of Tim Fish

> The v8 inspector docs are quite light on what exactly these functions do Not only what the functions do, but how all the commands should be used together! You...

I'll be back looking at this on Monday evening on my way back from vacation. The e2e test failures in CI suggested that the delay was required but I'll double...

> I just tried this branch in my local with and without delaying releaseObject and I get variables for both While we get local variables without the delay in releasing,...

Plot twist... While `setImmediate` is enough for Node v20, for v22 it releases too early and local variables are missing 😬

> Honestly, that is equal parts interesting and terrifying. Feels like we're relying on undefined behaviour somehow Yeah, feels like this entire debug API is prone to race conditions. >...

I think you're right, the simplest solution is to delay before checking for local variables and we can include a timeout.

> Both exceptions will save in the LRU in the same key so only one will persist, and then the delay will finish and it will be attached to just...

> I wonder if we could generate+inject a unique uuid on exception creation or exception throw that could then be used to match? From the debugger we can modify almost...

> I wonder if we could write a unique ID property the error object So I don't think we can't do this at the point when the debugger is initially...

This suggests that `http.url` has been deprecated and will be replaced by `url.full`: https://github.com/open-telemetry/semantic-conventions/blob/3c16c802e8ae8849ae0cf31eac02c3cabf64e4dd/model/registry/deprecated/http.yaml#L25-L30