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
feat: initial release #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is much easier to grok/understand! I'm super excited that you're building this out without coupling to frameworks/etc, and that it's so little code 💯
Let's make sure we've got tests which will "fail nicely" if the private stuff we're dependent on stops working as expected. Otherwise LGTM!
const panel = await extensionPanelTarget.page() | ||
|
||
// The extension panel should be the first embedded frame of the targeted page | ||
const [panelFrame] = await panel.frames() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we validate that it's the "correct" frame?
There's a few more steps we probably want to take (license, where to publish) but this is the initial code for getting devtools contexts from within puppeteer.
Reviewer checks
Required fields, to be filled out by PR reviewer(s)