Skip to content

Commit 622b3db

Browse files
chore(mcp): remove unused running tool state (#44082)
Co-authored-by: microsoft-playwright-automation[bot] <203992400+microsoft-playwright-automation[bot]@users.noreply.github.com> Co-authored-by: Yury Semikhatsky <yurys@chromium.org>
1 parent c59cf20 commit 622b3db

2 files changed

Lines changed: 0 additions & 12 deletions

File tree

‎packages/playwright-core/src/tools/backend/browserBackend.ts‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,6 @@ export class BrowserBackend extends EventEmitter<{ disconnected: [], dynamictool
126126
const cwd = rawArguments._meta?.cwd;
127127
const raw = !!rawArguments._meta?.raw;
128128
const response = new Response(context, name, parsedArguments, { relativeTo: cwd, raw, json, commandName });
129-
context.setRunningTool(name);
130129
let responseObject: mcpServer.CallToolResult & { isClose?: boolean };
131130
try {
132131
await handle(parsedArguments, response);
@@ -138,8 +137,6 @@ export class BrowserBackend extends EventEmitter<{ disconnected: [], dynamictool
138137
const message = context.config.skillMode && error instanceof Error ? error.message : String(error);
139138
const messages = [message, ...context.drainPendingUnhandledRejections().filter(reason => reason !== error).map(formatRejectionReason)];
140139
responseObject = formatError(messages.join('\n\n'));
141-
} finally {
142-
context.setRunningTool(undefined);
143140
}
144141
if (this._disconnected || responseObject.isClose) {
145142
delete responseObject.isClose;

‎packages/playwright-core/src/tools/backend/context.ts‎

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,6 @@ export class Context {
139139
private _webmcpToolsSignature = '';
140140
private _redactUrlParamsRegex: RegExp | undefined;
141141

142-
private _runningToolName: string | undefined;
143142
private _pendingUnhandledRejections: unknown[] = [];
144143
private _unhandledRejectionListeners = new Set<(reason: unknown) => void>();
145144
private _onUnhandledRejection = (reason: unknown) => {
@@ -434,14 +433,6 @@ export class Context {
434433
return toRemove.length;
435434
}
436435

437-
isRunningTool() {
438-
return this._runningToolName !== undefined;
439-
}
440-
441-
setRunningTool(name: string | undefined) {
442-
this._runningToolName = name;
443-
}
444-
445436
private async _setupRequestInterception(context: playwrightTypes.BrowserContext, disposables: Disposable[]) {
446437
if (this.config.network?.allowedOrigins?.length) {
447438
disposables.push(await context.route('**', route => route.abort('blockedbyclient')));

0 commit comments

Comments
 (0)