From 2a52094766c8235481b6f4935e4d5536c3dfd4b0 Mon Sep 17 00:00:00 2001 From: Nawaz Dhandala Date: Sun, 1 Mar 2026 14:02:07 +0000 Subject: [PATCH] refactor: Improve code formatting and comments in SyntheticMonitor execution logic --- .../Utils/Monitors/MonitorTypes/SyntheticMonitor.ts | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/Probe/Utils/Monitors/MonitorTypes/SyntheticMonitor.ts b/Probe/Utils/Monitors/MonitorTypes/SyntheticMonitor.ts index 6ee7bb1b01..d70dea8cf7 100644 --- a/Probe/Utils/Monitors/MonitorTypes/SyntheticMonitor.ts +++ b/Probe/Utils/Monitors/MonitorTypes/SyntheticMonitor.ts @@ -62,14 +62,15 @@ export default class SyntheticMonitor { `Running Synthetic Monitor: ${options?.monitorId?.toString()}, Screen Size: ${screenSizeType}, Browser: ${browserType}`, ); - const retryResult: ExecuteWithRetryResult = - await this.executeWithRetry({ + const retryResult: ExecuteWithRetryResult = await this.executeWithRetry( + { script: options.script, browserType: browserType, screenSizeType: screenSizeType, retryCountOnError: options.retryCountOnError || 0, monitorId: options.monitorId, - }); + }, + ); if (retryResult.response) { retryResult.response.browserType = browserType; @@ -91,8 +92,10 @@ export default class SyntheticMonitor { */ if (totalExecutions > 0 && results.length === 0) { if (dedupSkips > 0 && infraSkips === 0) { - // All skips were due to deduplication — another worker is already - // processing this monitor, which is normal and expected. + /* + * All skips were due to deduplication — another worker is already + * processing this monitor, which is normal and expected. + */ logger.debug( `Synthetic Monitor ${options?.monitorId?.toString()}: all ${totalExecutions} executions skipped (already being processed by another worker), skipping this check cycle`, );