The external `Connection` type regressed in https://github.com/microsoft/playwright-mcp/pull/490/files#diff-a6be0583428e46844273df76939f02077073da3075716fc57d291a5f2463eaf5, where the `connect()` function was removed but not from the types. I've changed the code so we import from there, similar to how we do it for `config.d.ts`, so this shouldn't happen again.
47 lines
1.8 KiB
TypeScript
47 lines
1.8 KiB
TypeScript
/**
|
|
* Copyright (c) Microsoft Corporation.
|
|
*
|
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
|
* you may not use this file except in compliance with the License.
|
|
* You may obtain a copy of the License at
|
|
*
|
|
* http://www.apache.org/licenses/LICENSE-2.0
|
|
*
|
|
* Unless required by applicable law or agreed to in writing, software
|
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
* See the License for the specific language governing permissions and
|
|
* limitations under the License.
|
|
*/
|
|
|
|
import { createConnection as createConnectionImpl } from './connection.js';
|
|
import type { Connection } from '../index.js';
|
|
import { resolveConfig } from './config.js';
|
|
import { contextFactory } from './browserContextFactory.js';
|
|
|
|
import type { Config } from '../config.js';
|
|
import type { BrowserContext } from 'playwright';
|
|
import type { BrowserContextFactory } from './browserContextFactory.js';
|
|
|
|
export async function createConnection(userConfig: Config = {}, contextGetter?: () => Promise<BrowserContext>): Promise<Connection> {
|
|
const config = await resolveConfig(userConfig);
|
|
const factory = contextGetter ? new SimpleBrowserContextFactory(contextGetter) : contextFactory(config.browser);
|
|
return createConnectionImpl(config, factory);
|
|
}
|
|
|
|
class SimpleBrowserContextFactory implements BrowserContextFactory {
|
|
private readonly _contextGetter: () => Promise<BrowserContext>;
|
|
|
|
constructor(contextGetter: () => Promise<BrowserContext>) {
|
|
this._contextGetter = contextGetter;
|
|
}
|
|
|
|
async createContext(): Promise<{ browserContext: BrowserContext, close: () => Promise<void> }> {
|
|
const browserContext = await this._contextGetter();
|
|
return {
|
|
browserContext,
|
|
close: () => browserContext.close()
|
|
};
|
|
}
|
|
}
|