-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Explore turning on recording mode as soon as dev tools panel is shown #229
Comments
I just came here to raise a similar issue; but I was wondering if it would make sense to have a similar mode as the Chrome DevTools 'Performance' tab, where it has a 'reload and start recording' button. I haven't looked at the current method of patching fetch too deeply; but I also wonder if there are better 'higher level' extension API's that can be used, rather than monkeypatching From memory, I believe it's not possible to get the response body with I haven't deeply validated this, but here's what ChatGPT suggested for it:
|
Copying the behaviour of the Chrome DevTools seems like a good idea UX wise. I like it. Using The Really the main thing that needs to keep working is streaming support. |
I believe it's referring to the chrome devtools protocol: Specifically:
Not sure which part emits this; but the description on it certainly reads in a potentially stream friendly way:
It might just be part of this:
And even the description of response on this says 'bytes received so far':
So sounds like there may be some bits worth PoC'ing with/exploring in there at the very least. |
Huh, yeah that seems like it might just work. |
Problem
#195 added some safety to reduce the risk that the extension blows up any page (by patching fetch potentially incorrectly9), but does so by adding a recording mode. You have to press it after loading your page to start seeing RSC chunk data coming in. This makes it hard to see prefetching responses.
Proposed solution
The extension could start recording automatically as soon as it is shown to the user. See https://twitter.com/nickemccurdy/status/1690111061142155264
The text was updated successfully, but these errors were encountered: