@lambdatest/smartui-cli 4.1.38-beta.1 → 4.1.39

package/dist/index.cjs

@@ -297,6 +297,7 @@ var constants_default = {
   MOBILE_ORIENTATION_PORTRAIT: "portrait",
   MOBILE_ORIENTATION_LANDSCAPE: "landscape",
   // build status
+  BUILD_RUNNING: "running",
   BUILD_COMPLETE: "completed",
   BUILD_ERROR: "error",
   // CI
@@ -1920,7 +1921,7 @@ function startPdfPolling(ctx) {
     attempts++;
     try {
       const response = yield ctx.client.fetchPdfResults(ctx);
-      if (response.screenshots && ((_a = response.build) == null ? void 0 : _a.build_status) === constants_default.BUILD_COMPLETE) {
+      if (response.screenshots && ((_a = response.build) == null ? void 0 : _a.build_status) !== constants_default.BUILD_RUNNING) {
         clearInterval(interval);
         const pdfGroups = groupScreenshotsByPdf(response.screenshots);
         const pdfsWithMismatches = countPdfsWithMismatches(pdfGroups);
@@ -2086,12 +2087,13 @@ function listenToSmartUISSE(baseURL, accessToken, ctx, onEvent) {
     const url = `${baseURL}/api/v1/sse/smartui`;
     const abortController = new AbortController();
     try {
+      const cookieKey = baseURL === "https://server-events.lambdatest.com" ? "accessToken" : "stageAccessToken";
       const response = yield fetch(url, {
         method: "GET",
         headers: {
           "Accept": "text/event-stream",
           "Cache-Control": "no-cache",
-          "Cookie": `stageAccessToken=Basic ${accessToken}`
+          "Cookie": `${cookieKey}=Basic ${accessToken}`
         },
         signal: abortController.signal
       });
@@ -2269,7 +2271,7 @@ var server_default = (ctx) => __async(void 0, null, function* () {
           capsBuildId = (cachedCapabilities == null ? void 0 : cachedCapabilities.buildId) || "";
         } else {
           try {
-            let fetchedCapabilitiesResp = yield ctx.client.getSmartUICapabilities(sessionId, ctx.config, ctx.git, ctx.log, ctx.isStartExec);
+            let fetchedCapabilitiesResp = yield ctx.client.getSmartUICapabilities(sessionId, ctx.config, ctx.git, ctx.log, ctx.isStartExec, ctx.options.baselineBuild);
             capsBuildId = (fetchedCapabilitiesResp == null ? void 0 : fetchedCapabilitiesResp.buildId) || "";
             ctx.log.debug(`fetch caps for sessionId: ${sessionId} are ${JSON.stringify(fetchedCapabilitiesResp)}`);
             if (capsBuildId) {
@@ -2397,7 +2399,8 @@ var server_default = (ctx) => __async(void 0, null, function* () {
     let replyBody;
     try {
       ctx.log.debug(`request.query : ${JSON.stringify(request.query)}`);
-      const { contextId, pollTimeout, snapshotName } = request.query;
+      const { contextId, pollTimeout, snapshotName: rawSnapshotName } = request.query;
+      const snapshotName = rawSnapshotName == null ? void 0 : rawSnapshotName.trim();
       if (!contextId || !snapshotName) {
         throw new Error("contextId and snapshotName are required parameters");
       }
@@ -2650,7 +2653,7 @@ var authExec_default = (ctx) => {
 };
 
 // package.json
-var version = "4.1.38-beta.1";
+var version = "4.1.39";
 var package_default = {
   name: "@lambdatest/smartui-cli",
   version,
@@ -2961,7 +2964,7 @@ var httpClient = class {
       }
     }, log2);
   }
-  getSmartUICapabilities(sessionId, config, git, log2, isStartExec) {
+  getSmartUICapabilities(sessionId, config, git, log2, isStartExec, baselineBuild) {
     return this.request({
       url: "/sessions/capabilities",
       method: "GET",
@@ -2971,7 +2974,8 @@ var httpClient = class {
       data: {
         git,
         config,
-        isStartExec
+        isStartExec,
+        baselineBuild
       },
       headers: {
         projectToken: "",
@@ -3328,8 +3332,13 @@ var httpClient = class {
   }
   getSnapshotStatus(buildId, snapshotName, snapshotUuid, ctx) {
     return this.request({
-      url: `/snapshot/status?buildId=${buildId}&snapshotName=${snapshotName}&snapshotUUID=${snapshotUuid}`,
+      url: `/snapshot/status`,
       method: "GET",
+      params: {
+        buildId,
+        snapshotName,
+        snapshotUUID: snapshotUuid
+      },
       headers: {
         "Content-Type": "application/json"
       }
@@ -3719,7 +3728,8 @@ var createBuildExec_default = (ctx) => {
               name: resp.data.buildName,
               url: resp.data.buildURL,
               baseline: resp.data.baseline,
-              useKafkaFlow: resp.data.useKafkaFlow || false
+              useKafkaFlow: resp.data.useKafkaFlow || false,
+              checkPendingRequests: resp.data.checkPendingRequests || false
             };
             process.env.SMARTUI_BUILD_ID = resp.data.buildId;
             process.env.SMARTUI_BUILD_NAME = resp.data.buildName;
@@ -4005,8 +4015,20 @@ var globalCache = new NodeCache__default.default({ stdTTL: 3600, checkperiod: 60
 var MAX_RESOURCE_SIZE = 15 * 1024 ** 2;
 var ALLOWED_RESOURCES = ["document", "stylesheet", "image", "media", "font", "other"];
 var ALLOWED_STATUSES = [200, 201];
-var REQUEST_TIMEOUT = 18e5;
+var REQUEST_TIMEOUT = 18e4;
 var MIN_VIEWPORT_HEIGHT = 1080;
+var MAX_WAIT_FOR_REQUEST_CALL = 3e4;
+var normalizeSameSite = (value) => {
+  if (!value)
+    return "Lax";
+  const normalized = value.trim().toLowerCase();
+  const mapping = {
+    "lax": "Lax",
+    "strict": "Strict",
+    "none": "None"
+  };
+  return mapping[normalized] || value;
+};
 function prepareSnapshot(snapshot, ctx) {
   return __async(this, null, function* () {
     let processedOptions = {};
@@ -4229,7 +4251,8 @@ function processSnapshot(snapshot, ctx) {
           ctx.log.debug(`Skipping invalid custom cookie: missing required fields (name, value, or domain)`);
           return false;
         }
-        if (cookie.sameSite && !["Strict", "Lax", "None"].includes(cookie.sameSite)) {
+        const sameSiteValue = normalizeSameSite(cookie.sameSite);
+        if (!["Strict", "Lax", "None"].includes(sameSiteValue)) {
           ctx.log.debug(`Skipping invalid custom cookie: invalid sameSite value '${cookie.sameSite}'`);
           return false;
         }
@@ -4241,7 +4264,7 @@ function processSnapshot(snapshot, ctx) {
         path: cookie.path || "/",
         httpOnly: cookie.httpOnly || false,
         secure: cookie.secure || false,
-        sameSite: cookie.sameSite || "Lax"
+        sameSite: normalizeSameSite(cookie.sameSite)
       }));
       if (validCustomCookies.length > 0) {
         try {
@@ -4265,6 +4288,7 @@ function processSnapshot(snapshot, ctx) {
         };
       }
     }
+    const pendingRequests = /* @__PURE__ */ new Set();
     yield page.route("**/*", (route, request) => __async(this, null, function* () {
       var _a2;
       const requestUrl = request.url();
@@ -4314,8 +4338,14 @@ function processSnapshot(snapshot, ctx) {
           body = globalCache.get(requestUrl).body;
         } else {
           ctx.log.debug(`Resource not found in cache or global cache ${requestUrl} fetching from server`);
+          if (ctx.build.checkPendingRequests) {
+            pendingRequests.add(requestUrl);
+          }
           response = yield page.request.fetch(request, requestOptions);
           body = yield response.body();
+          if (ctx.build.checkPendingRequests) {
+            pendingRequests.delete(requestUrl);
+          }
         }
         if (!body) {
           ctx.log.debug(`Handling request ${requestUrl}
@@ -4347,8 +4377,14 @@ function processSnapshot(snapshot, ctx) {
           }
           let responseOfRetry, bodyOfRetry;
           ctx.log.debug(`Resource had a disallowed status ${requestUrl} fetching from server again`);
+          if (ctx.build.checkPendingRequests) {
+            pendingRequests.add(requestUrl);
+          }
           responseOfRetry = yield page.request.fetch(request, requestOptions);
           bodyOfRetry = yield responseOfRetry.body();
+          if (ctx.build.checkPendingRequests) {
+            pendingRequests.delete(requestUrl);
+          }
           if (responseOfRetry && responseOfRetry.status() && ALLOWED_STATUSES.includes(responseOfRetry.status())) {
             ctx.log.debug(`Handling request after retry ${requestUrl}
  - content-type ${responseOfRetry.headers()["content-type"]}`);
@@ -4356,6 +4392,12 @@ function processSnapshot(snapshot, ctx) {
               body: bodyOfRetry.toString("base64"),
               type: responseOfRetry.headers()["content-type"]
             };
+            if (ctx.config.useGlobalCache) {
+              globalCache.set(requestUrl, {
+                body: bodyOfRetry.toString("base64"),
+                type: responseOfRetry.headers()["content-type"]
+              });
+            }
             route.fulfill({
               status: responseOfRetry.status(),
               headers: responseOfRetry.headers(),
@@ -4732,6 +4774,25 @@ function processSnapshot(snapshot, ctx) {
       processedOptions.selectDOM = options == null ? void 0 : options.selectDOM;
       ctx.log.debug(`Processed options: ${JSON.stringify(processedOptions)}`);
     }
+    const checkPending = () => __async(this, null, function* () {
+      let startTime = Date.now();
+      ctx.log.debug(`${pendingRequests.size} Pending requests before wait for ${snapshot.name}: ${Array.from(pendingRequests)}`);
+      while (pendingRequests.size > 0) {
+        const elapsedTime = Date.now() - startTime;
+        if (elapsedTime >= MAX_WAIT_FOR_REQUEST_CALL) {
+          ctx.log.debug(`Timeout reached (${MAX_WAIT_FOR_REQUEST_CALL / 1e3}s). Stopping wait for pending requests.`);
+          ctx.log.debug(`${pendingRequests.size} Pending requests after wait for ${snapshot.name}: ${Array.from(pendingRequests)}`);
+          break;
+        }
+        yield new Promise((resolve) => setTimeout(resolve, 1e3));
+      }
+      if (pendingRequests.size === 0) {
+        ctx.log.debug(`No pending requests for ${snapshot.name}.`);
+      }
+    });
+    if (ctx.build.checkPendingRequests) {
+      yield checkPending();
+    }
     let hasBrowserErrors = false;
     for (let browser in discoveryErrors.browsers) {
       if (discoveryErrors.browsers[browser]) {
@@ -5465,7 +5526,8 @@ var createBuild_default = (ctx) => {
             name: resp.data.buildName,
             url: resp.data.buildURL,
             baseline: resp.data.baseline,
-            useKafkaFlow: resp.data.useKafkaFlow || false
+            useKafkaFlow: resp.data.useKafkaFlow || false,
+            checkPendingRequests: resp.data.checkPendingRequests || false
           };
           process.env.SMARTUI_BUILD_ID = resp.data.buildId;
           process.env.SMARTUI_BUILD_NAME = resp.data.buildName;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lambdatest/smartui-cli",
-  "version": "4.1.38-beta.1",
+  "version": "4.1.39",
   "description": "A command line interface (CLI) to run SmartUI tests on LambdaTest",
   "files": [
     "dist/**/*"