Skip to content

Commit 824d800

Browse files
committed
fix: resolve lint errors flagged by CI
- Remove unused context parameter from navigatePage handler - Use import type for PredefinedNetworkConditions in WaitForHelper
1 parent 2b57f00 commit 824d800

3 files changed

Lines changed: 7 additions & 5 deletions

File tree

src/WaitForHelper.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
import {logger} from './logger.js';
88
import type {Page, Protocol, CdpPage} from './third_party/index.js';
9-
import {PredefinedNetworkConditions} from './third_party/index.js';
9+
import type {PredefinedNetworkConditions} from './third_party/index.js';
1010

1111
export class WaitForHelper {
1212
#abortController = new AbortController();

src/tools/pages.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ export const navigatePage = definePageTool({
166166
),
167167
...timeoutSchema,
168168
},
169-
handler: async (request, response, context) => {
169+
handler: async (request, response) => {
170170
const page = request.page;
171171
const options = {
172172
timeout: request.params.timeout,

src/tools/script.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -77,9 +77,11 @@ Example with arguments: \`(el) => {
7777
}
7878

7979
const worker = await getWebWorker(context, serviceWorkerId);
80-
await context.getSelectedMcpPage().waitForEventsAfterAction(async () => {
81-
await performEvaluation(worker, fnString, [], response);
82-
});
80+
await context
81+
.getSelectedMcpPage()
82+
.waitForEventsAfterAction(async () => {
83+
await performEvaluation(worker, fnString, [], response);
84+
});
8385
return;
8486
}
8587

0 commit comments

Comments
 (0)