[BUG] No Playwright class typed definition anymore in @next

Context:

• Playwright Version: 0.11.1-next.1584947279674 (March 23, 2020)
• Operating System: Linux

Code Snippet

import { Playwright } from 'playwright-core/lib/server/playwright'

const pw = new Playwright({
  browsers: ['webkit', 'chromium', 'firefox'],
});

Describe the bug

In version 0.11.1 and some of the recent next releases it was working. Now the types are missing.

Version 0.11.1 example: https://unpkg.com/playwright-core@0.11.1/lib/server/playwright.d.ts

I think it’s related to #1166.

cc @JoelEinbinder

[JoelEinbinder]

https://github.com/mxschmitt/try-playwright/blob/6b112eec4b2fdd2e831b9bf7d3052ecf6119bb55/backend/src/playwright.ts#L86-L101

Hmm, yep its a kinda hacky approach to overwrite the methods.

The main issue here is that you want multiple playwright objects in the same node process, with their own prototype hacks on it. If you are ok with the same hacks everywhere, then requiring playwright-core directly will work. I don’t think this will ever work cleanly though.

Edit: Another use case is on Try Playwright for that the usage in the Monaco editor. there I also just have used the previous class Playwright as a playwright interface.

Instead try typeof import('playwright-core').

Also I found a issue in playwright-video: https://github.com/qawolf/playwright-video/blob/9a7912f6fb04aa86ff5c4be757ef5dccfddd0dcb/src/ScreencastFrameCollector.ts#L4 that the type could not be resolved.

This just needs to use the CDPSession type from the root of playwright instead.

Closing this issue for now as I consider it not a bug. Feel free to keep up the discussion here about internal types, or file new issues for specific things that might be missing/broken.

[JoelEinbinder]

Is it possible for playwright-core to expose the types for power users so we don’t have to manually define our own declaration files?

I really don’t want to support people hacking on our internal code. I’d much rather support your use cases in the public api.