@dev-blinq/cucumber_client 1.0.1457-dev → 1.0.1457-stage

package/bin/client/recorderv3/step_runner.js

@@ -67,13 +67,12 @@ export class BVTStepRunner {
           resolve();
         }
       } else {
-        console.warn(`No paused command found for cmdId: ${cmdId}`);
-        socketLogger.error(`No paused command found for cmdId: ${cmdId}`);
+        socketLogger.error(`No paused command found for cmdId: ${cmdId}`, undefined, "BVTStepRunner.resumeExecution");
       }
     }
   }
 
-  async copyCodetoTempFolder({ step, parametersMap, tempFolderPath }) {
+  async copyCodetoTempFolder({ step, parametersMap, tempFolderPath, AICode }) {
     if (!fs.existsSync(tempFolderPath)) {
       fs.mkdirSync(tempFolderPath);
     }
@@ -84,6 +83,18 @@ export class BVTStepRunner {
       overwrite: true,
       recursive: true,
     });
+
+    // If AICode is provided, save it as well
+    if (AICode) {
+      for (const { mjsFileContent, mjsFile } of AICode) {
+        const mjsPath = path
+          .normalize(mjsFile)
+          .split(path.sep)
+          .filter((part) => part !== "features")
+          .join(path.sep);
+        writeFileSync(path.join(tempFolderPath, mjsPath), mjsFileContent);
+      }
+    }
   }
 
   async writeTempFeatureFile({ step, parametersMap, tempFolderPath, tags }) {
@@ -250,7 +261,7 @@ export class BVTStepRunner {
     return { result, info };
   }
 
-  async runStep({ step, parametersMap, envPath, tags, config }, bvtContext, options) {
+  async runStep({ step, parametersMap, envPath, tags, config, AICode }, bvtContext, options) {
     // Create a new AbortController for this specific step execution
     this.#currentStepController = new AbortController();
     const { signal } = this.#currentStepController;
@@ -284,13 +295,27 @@ export class BVTStepRunner {
           return cId;
         };
       }
+      if (bvtContext.api) {
+        bvtContext.api.getCmdId = () => {
+          if (cmdIDs.length === 0) {
+            cmdIDs = (step.commands || []).map((cmd) => cmd.cmdId ?? cmd.id);
+          }
+          const cId = cmdIDs.shift();
+          this.sendExecutionStatus({
+            type: "cmdExecutionStart",
+            cmdId: cId,
+          });
+          this.#lastAttemptedCmdId = cId;
+          return cId;
+        };
+      }
 
       const __temp_features_FolderName = "__temp_features" + Math.random().toString(36).substring(2, 7);
       const tempFolderPath = path.join(this.projectDir, __temp_features_FolderName);
       process.env.tempFeaturesFolderPath = __temp_features_FolderName;
       process.env.TESTCASE_REPORT_FOLDER_PATH = tempFolderPath;
 
-      await this.copyCodetoTempFolder({ step, parametersMap, tempFolderPath });
+      await this.copyCodetoTempFolder({ step, parametersMap, tempFolderPath, AICode });
 
       // Write abort wrapper code with this step's signal
       await this.writeWrapperCode(tempFolderPath, signal);
@@ -306,7 +331,10 @@ export class BVTStepRunner {
         });
       }
 
-      if (!step.isImplemented && step.commands.length > 0) {
+      if (
+        step.isUtilStep ||
+        ((!(step.isImplemented && !step.shouldOverride) || step.shouldMakeStepTextUnique) && step.commands.length > 0)
+      ) {
         const pageName = generatePageName(step.startFrame?.url ?? "default");
         const stepDefinitionFolderPath = path.join(tempFolderPath, "step_definitions");
         if (!existsSync(stepDefinitionFolderPath)) {
@@ -321,6 +349,7 @@ export class BVTStepRunner {
           projectDir: this.projectDir,
           stepsDefinitions,
           parametersMap,
+          logger: socketLogger,
         });
         if (codePage) {
           await codePage.save(stepDefsFilePath);

package/bin/client/recorderv3/step_utils.js

@@ -1,7 +1,6 @@
-import { existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from "fs";
-import path from "path";
-import url from "url";
-import logger from "../../logger.js";
+import { existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import path from "node:path";
+import url from "node:url";
 import { CodePage, getAiConfig } from "../code_gen/page_reflection.js";
 import { generateCode, generatePageName } from "../code_gen/playwright_codeget.js";
 import { invertCodeToCommand } from "../code_gen/code_inversion.js";
@@ -9,8 +8,9 @@ import { Step } from "../cucumber/feature.js";
 import { locateDefinitionPath, StepsDefinitions } from "../cucumber/steps_definitions.js";
 import { Recording } from "../recording.js";
 import { generateApiCode } from "../code_gen/api_codegen.js";
-import { tmpdir } from "os";
-import { createHash } from "crypto";
+import { tmpdir } from "node:os";
+import { createHash } from "node:crypto";
+import { getErrorMessage } from "../utils/socket_logger.js";
 
 const __dirname = path.dirname(url.fileURLToPath(import.meta.url));
 
@@ -48,7 +48,7 @@ const replaceLastOccurence = (str, search, replacement) => {
   return str.substring(0, lastIndex) + replacement + str.substring(lastIndex + search.length);
 };
 
-const _toRecordingStep = (cmd) => {
+export const _toRecordingStep = (cmd, stepText) => {
   switch (cmd.type) {
     case "hover_element": {
       return {
@@ -250,6 +250,7 @@ const _toRecordingStep = (cmd) => {
         },
         parameters: [cmd.selectedField, cmd.value],
         lastKnownUrlPath: cmd.lastKnownUrlPath,
+        valueInStepText: stepText && typeof stepText === "string" ? stepText.includes(`"${cmd.value}"`) : false,
       };
     }
     case "conditional_wait": {
@@ -261,6 +262,7 @@ const _toRecordingStep = (cmd) => {
         },
         parameters: [cmd.timeout, cmd.selectedField, cmd.value],
         lastKnownUrlPath: cmd.lastKnownUrlPath,
+        valueInStepText: stepText && typeof stepText === "string" ? stepText.includes(`"${cmd.value}"`) : false,
       };
     }
     case "navigate": {
@@ -296,6 +298,8 @@ const _toRecordingStep = (cmd) => {
         type: "verify_page_snapshot",
         parameters: [cmd.value],
         selectors: cmd.selectors,
+        data: cmd.data,
+        valueInStepText: stepText && typeof stepText === "string" ? stepText.includes(`"${cmd.value}"`) : false,
       };
     }
     default: {
@@ -332,45 +336,62 @@ const _parameterizeLocators = (locators, replacementFromValue, replacementToValu
   }
   return locators;
 };
-const parameterizeLocators = ({ cmd, locs, isValueVariable, isTextVariable, parametersMap }) => {
+const parameterizeLocators = ({ cmd, locs, isValueVariable, isTargetValueVariable, parametersMap }) => {
   if (isValueVariable) {
     const variable = cmd.value.slice(1, -1);
-    const val = parametersMap[variable];
-    const replacementFromValue = val.trim();
-    const replacementToValue = `{${variable}}`;
-    locs = _parameterizeLocators(locs, replacementFromValue, replacementToValue);
-  }
-  if (isTextVariable) {
-    const variable = cmd.text.slice(1, -1);
-    const val = parametersMap[variable];
-    const replacementFromValue = val.trim();
-    const replacementToValue = `{${variable}}`;
-    locs = _parameterizeLocators(locs, replacementFromValue, replacementToValue);
+    // const val = parametersMap[variable];
+    if (typeof cmd.text === "string") {
+      const replacementFromValue = cmd.text.trim().replace(/\s+/g, " ") ?? ""; // val.trim();
+      if (replacementFromValue.length > 0) {
+        const replacementToValue = `{${variable}}`;
+        locs = _parameterizeLocators(locs, replacementFromValue, replacementToValue);
+      }
+    }
+  }
+  if (isTargetValueVariable) {
+    const variable = cmd.targetValue.slice(1, -1);
+    // const val = parametersMap[variable];
+    if (typeof cmd.targetText === "string") {
+      const replacementFromValue = cmd.targetText.trim().replace(/\s+/g, " ") ?? ""; // val.trim();
+      if (replacementFromValue.length > 0) {
+        const replacementToValue = `{${variable}}`;
+        locs = _parameterizeLocators(locs, replacementFromValue, replacementToValue);
+      }
+    }
   }
   return locs;
 };
 
 //TODO: IMPORTAN
-export const toRecordingStep = (cmd, parametersMap) => {
+export const toRecordingStep = (cmd, parametersMap, stepText) => {
   if (cmd.type === "api") {
     return {
       type: "api",
       value: cmd.value,
     };
   }
-  const step = _toRecordingStep(cmd);
+  const step = _toRecordingStep(cmd, stepText);
   const cmdID = {
     cmdId: cmd.id,
+    options: cmd.options ?? null,
   };
   Object.assign(step, cmdID);
 
   const locatorsObject = JSON.parse(JSON.stringify(cmd.locators ?? null));
 
   if (!locatorsObject) return step;
+
+  const element_name = cmd?.locators?.element_name ?? `${cmd.label} ${cmd.role ?? "Text"}`;
+  locatorsObject.element_name = element_name;
+
   const isValueVariable = isVariable(cmd.value);
-  const isTextVariable = isVariable(cmd.text);
+  const isTargetValueVariable = isVariable(cmd.targetValue);
+  const allStrategyLocators = JSON.parse(JSON.stringify(cmd?.allStrategyLocators ?? null));
+  step.locators = locatorsObject;
+  step.allStrategyLocators = allStrategyLocators;
+  step.isLocatorsAssigned = true;
 
-  if (!isValueVariable && !isTextVariable) {
+  if (!isValueVariable && !isTargetValueVariable) {
     return step;
   }
 
@@ -378,7 +399,6 @@ export const toRecordingStep = (cmd, parametersMap) => {
     step.dataSource = "parameters";
     step.dataKey = convertToIdentifier(cmd.value.slice(1, -1));
   }
-  const allStrategyLocators = JSON.parse(JSON.stringify(cmd?.allStrategyLocators ?? null));
 
   if (!allStrategyLocators) {
     let locs = locatorsObject.locators;
@@ -386,7 +406,7 @@ export const toRecordingStep = (cmd, parametersMap) => {
       cmd,
       locs,
       isValueVariable,
-      isTextVariable,
+      isTargetValueVariable,
       parametersMap,
     });
     locatorsObject.locators = locs;
@@ -405,7 +425,7 @@ export const toRecordingStep = (cmd, parametersMap) => {
       cmd,
       locs: locators,
       isValueVariable,
-      isTextVariable,
+      isTargetValueVariable,
       parametersMap,
     });
   }
@@ -442,7 +462,7 @@ export function getCodePage(stepDefsFilePath) {
 export function getCucumberStep({ step }) {
   const cucumberStep = new Step();
   cucumberStep.loadFromJson({
-    text: step.text,
+    text: step.renamedText ? step.renamedText : step.text,
     keyword: step.keyword,
     keywordType: step.keywordType,
     parameters: [],
@@ -460,12 +480,16 @@ export function getCucumberStep({ step }) {
 
 function makeStepTextUnique(step, stepsDefinitions) {
   // const utilsFilePath = path.join("features", "step_definitions", "utils.mjs");
-  let stepText = step.text;
+  let stepText = step.renamedText ? step.renamedText : step.text;
   let stepIndex = 1;
   // console.log("makeStepTextUnique", step.text);
   let stepDef = stepsDefinitions.findMatchingStep(stepText);
   // console.log({ stepDef });
-  if (stepDef && stepDef?.file.endsWith("utils.mjs")) {
+  if (
+    stepDef &&
+    (stepDef?.file.endsWith("utils.mjs") || stepDef?.file.endsWith("default_page.mjs")) &&
+    !step.shouldMakeStepTextUnique
+  ) {
     return true;
   }
   while (stepDef) {
@@ -476,7 +500,18 @@ function makeStepTextUnique(step, stepsDefinitions) {
   step.text = stepText;
 }
 
-export async function saveRecording({ step, cucumberStep, codePage, projectDir, stepsDefinitions, parametersMap }) {
+export async function saveRecording({
+  step,
+  cucumberStep,
+  codePage,
+  projectDir,
+  stepsDefinitions,
+  parametersMap,
+  logger,
+}) {
+  if (step.commands && Array.isArray(step.commands)) {
+    step.commands = step.commands.map((cmd) => toRecordingStep(cmd, parametersMap, step.text));
+  }
   let routesPath = path.join(tmpdir(), "blinq_temp_routes");
 
   if (process.env.TEMP_RUN === "true") {
@@ -484,21 +519,19 @@ export async function saveRecording({ step, cucumberStep, codePage, projectDir,
       rmSync(routesPath, { recursive: true });
     }
     mkdirSync(routesPath, { recursive: true });
-    saveRoutes({ step, folderPath: routesPath });
+    saveRoutes({ step, folderPath: routesPath }, logger);
   } else {
     if (existsSync(routesPath)) {
-      // remove the folder
       try {
         rmSync(routesPath, { recursive: true });
-        console.log("Removed temp_routes_folder:", routesPath);
       } catch (error) {
-        console.error("Error removing temp_routes folder", error);
+        logger.error(`Error removing temp routes folder: ${getErrorMessage(error)}`);
       }
       routesPath = path.join(projectDir, "data", "routes");
       if (!existsSync(routesPath)) {
         mkdirSync(routesPath, { recursive: true });
       }
-      saveRoutes({ step, folderPath: routesPath });
+      saveRoutes({ step, folderPath: routesPath }, logger);
     }
   }
 
@@ -506,49 +539,63 @@ export async function saveRecording({ step, cucumberStep, codePage, projectDir,
     return;
   }
 
+  let isUtilStep = false;
+  let isChangedUtilStepName = false;
+
   if (step.isImplemented && step.shouldOverride) {
-    let stepDef = stepsDefinitions.findMatchingStep(step.text);
+    const stepDef = stepsDefinitions.findMatchingStep(step.text);
     codePage = getCodePage(stepDef.file);
   } else {
-    const isUtilStep = makeStepTextUnique(step, stepsDefinitions);
+    isUtilStep = makeStepTextUnique(step, stepsDefinitions);
 
     if (isUtilStep) {
-      return;
+      isChangedUtilStepName =
+        step.renamedText && stepsDefinitions.findMatchingStep(step.renamedText)?.file.endsWith("default_page.mjs");
+
+      if (!isChangedUtilStepName) {
+        const isUtilStep = stepsDefinitions.findMatchingStep(step.text)?.file.endsWith("utils.mjs");
+        if (!step.renamedText || step.renamedText === step.text || isUtilStep) {
+          return;
+        }
+        step.text = step.text.trim();
+        const { functionName } = stepsDefinitions.findMatchingStep(step.renamedText);
+        step.renamedText = functionName;
+        const newImportLine = `import { ${functionName} } from "./utils.mjs";\n`;
+
+        if (!codePage.fileContent.includes(newImportLine)) {
+          codePage.fileContent = newImportLine + codePage.fileContent;
+        }
+      }
     }
   }
 
+  const renamedUtil = step.renamedText && isUtilStep;
+
   routesPath = path.join(tmpdir(), "blinq_temp_routes");
   if (process.env.TEMP_RUN === "true") {
-    console.log("Save routes in temp folder for running:", routesPath);
     if (existsSync(routesPath)) {
-      console.log("Removing existing temp_routes_folder:", routesPath);
       rmSync(routesPath, { recursive: true });
     }
     mkdirSync(routesPath, { recursive: true });
-    console.log("Created temp_routes_folder:", routesPath);
-    saveRoutes({ step, folderPath: routesPath });
+    saveRoutes({ step, folderPath: routesPath }, logger);
   } else {
-    console.log("Saving routes in project directory:", projectDir);
     if (existsSync(routesPath)) {
-      // remove the folder
       try {
         rmSync(routesPath, { recursive: true });
-        console.log("Removed temp_routes_folder:", routesPath);
       } catch (error) {
-        console.error("Error removing temp_routes folder", error);
+        logger.error(`Error removing temp routes folder: ${getErrorMessage(error)}`);
       }
     }
     routesPath = path.join(projectDir, "data", "routes");
-    console.log("Saving routes to:", routesPath);
     if (!existsSync(routesPath)) {
       mkdirSync(routesPath, { recursive: true });
     }
-    saveRoutes({ step, folderPath: routesPath });
+    saveRoutes({ step, folderPath: routesPath }, logger);
   }
 
   cucumberStep.text = step.text;
   const recording = new Recording();
-  step.commands = step.commands.map((cmd) => toRecordingStep(cmd, parametersMap));
+
   const steps = step.commands;
 
   recording.loadFromObject({ steps, step: cucumberStep });
@@ -599,7 +646,7 @@ export async function saveRecording({ step, cucumberStep, codePage, projectDir,
       stepsDefinitions
     );
 
-    if (!step.isImplemented) {
+    if (!renamedUtil && !(step.isImplemented && step.shouldOverride)) {
       stepsDefinitions.addStep({
         name: step.text,
         file: result.codePage.sourceFileName,
@@ -611,13 +658,16 @@ export async function saveRecording({ step, cucumberStep, codePage, projectDir,
     return result.codePage;
   } else {
     const generateCodeResult = generateCode(recording, codePage, userData, projectDir, methodName);
-    if (generateCodeResult.noCode === true) {
-      logger.log("No code generated for step: " + step.text);
+    console.log("Generated code for step:", step.text);
+    if (!isUtilStep && generateCodeResult.noCode === true) {
       return generateCodeResult.page;
     }
     codePage = generateCodeResult.page;
     methodName = generateCodeResult.methodName;
-    codePage.insertElements(generateCodeResult.elements);
+
+    if (!renamedUtil) {
+      codePage.insertElements(generateCodeResult.elements);
+    }
 
     const description = cucumberStep.text;
     let path = null;
@@ -630,15 +680,43 @@ export async function saveRecording({ step, cucumberStep, codePage, projectDir,
         protect = true;
       }
     }
-    const infraResult = codePage.addInfraCommand(
-      methodName,
-      description,
-      cucumberStep.getVariablesList(),
-      generateCodeResult.codeLines,
-      protect,
-      "recorder",
-      path
-    );
+
+    if (renamedUtil) {
+      if (isChangedUtilStepName) {
+        const newFileContent = codePage.fileContent
+          .replace(`async function ${step.renamedText}`, `async function ${methodName}`)
+          .replace(`Then("${step.renamedText}"`, `// Then("${step.renamedText}"`)
+          .replace(`When("${step.renamedText}"`, `// When("${step.renamedText}"`)
+          .replace(`Given("${step.renamedText}"`, `// Given("${step.renamedText}"`);
+
+        codePage._init();
+        codePage.generateModel(newFileContent);
+      } else {
+        codePage.addInfraCommandUtil(
+          methodName,
+          description,
+          cucumberStep.parameters,
+          generateCodeResult.codeLines,
+          step.renamedText,
+          step.text,
+          parametersMap,
+          protect,
+          "recorder",
+          path
+        );
+      }
+    } else {
+      codePage.addInfraCommand(
+        methodName,
+        description,
+        cucumberStep.getVariablesList(),
+        generateCodeResult.codeLines,
+        protect,
+        "recorder",
+        path
+      );
+    }
+
     const keyword = (cucumberStep.keywordAlias ?? cucumberStep.keyword).trim();
     const stepResult = codePage.addCucumberStep(
       keyword,
@@ -648,7 +726,7 @@ export async function saveRecording({ step, cucumberStep, codePage, projectDir,
       step.finalTimeout
     );
 
-    if (!step.isImplemented) {
+    if (!renamedUtil && !(step.isImplemented && step.shouldOverride)) {
       stepsDefinitions.addStep({
         name: step.text,
         file: codePage.sourceFileName,
@@ -657,6 +735,7 @@ export async function saveRecording({ step, cucumberStep, codePage, projectDir,
     }
 
     codePage.removeUnusedElements();
+    codePage.mergeSimilarElements();
     cucumberStep.methodName = methodName;
     if (generateCodeResult.locatorsMetadata) {
       codePage.addLocatorsMetadata(generateCodeResult.locatorsMetadata);
@@ -700,7 +779,7 @@ export const getCommandsForImplementedStep = (stepName, stepsDefinitions, stepPa
     const file = step?.file;
     const locatorsJson = getLocatorsJson(file);
     if (!step) {
-      throw new Error("Step definition not found" + stepName);
+      throw new Error(`Step definition not found: ${stepName}`);
     }
     isImplemented = true;
     const { codeCommands, codePage, elements, parametersNames, error } =
@@ -708,26 +787,35 @@ export const getCommandsForImplementedStep = (stepName, stepsDefinitions, stepPa
     if (error) {
       throw new Error(error);
     }
+    isUtilStep = codePage.sourceFileName.endsWith("utils.mjs") || codePage.sourceFileName.endsWith("default_page.mjs");
 
     if (parametersNames.length !== stepParams.length) {
       // console.log("Parameters mismatch", parametersNames, stepParams);
       throw new Error("Parameters mismatch");
     }
-    for (let i = 0; i < parametersNames.length; i++) {
-      stepParams[i].argumentName = parametersNames[i];
-    }
 
-    isUtilStep = codePage.sourceFileName.endsWith("utils.mjs");
-    for (const { code } of codeCommands) {
-      const command = invertCodeToCommand(code, elements, stepParams, stepsDefinitions, codePage, stepName)[0];
-      if (command === undefined || command.type === null) continue;
-      if (command.element) {
-        const key = command.element.key;
-        if (key && locatorsJson[key]) {
-          command.allStrategyLocators = locatorsJson[key];
+    const pattern = step.name;
+    if (isUtilStep && pattern === "Verify the file {string} exists") {
+      commands.push({
+        type: "verify_file_exists",
+        parameters: [stepParams[0].text],
+      });
+    } else {
+      for (let i = 0; i < parametersNames.length; i++) {
+        stepParams[i].argumentName = parametersNames[i];
+      }
+
+      for (const { code } of codeCommands) {
+        const command = invertCodeToCommand(code, elements, stepParams, stepsDefinitions, codePage, stepName)[0];
+        if (command === undefined || command.type === null) continue;
+        if (command.element) {
+          const key = command.element.key;
+          if (key && locatorsJson[key]) {
+            command.allStrategyLocators = locatorsJson[key];
+          }
         }
+        commands.push(command);
       }
-      commands.push(command);
     }
   } catch (error) {
     console.error(error);
@@ -775,7 +863,7 @@ export async function executeStep({ stepsDefinitions, cucumberStep, context, cod
   }
 }
 
-export async function updateStepDefinitions({ scenario, featureName, projectDir }) {
+export async function updateStepDefinitions({ scenario, featureName, projectDir, logger }) {
   // set the candidate step definition file name
   // set the utils file path
   const utilsFilePath = path.join(projectDir, "features", "step_definitions", "utils.mjs");
@@ -806,42 +894,37 @@ export async function updateStepDefinitions({ scenario, featureName, projectDir
         step.isImplementedWhileRecording = true;
       }
     }
-    if ((step.isImplemented && !step.shouldOverride) || step.commands.length === 0) {
+    if (!step.isUtilStep && ((step.isImplemented && !step.shouldOverride) || step.commands.length === 0)) {
       let routesPath = path.join(tmpdir(), `blinq_temp_routes`);
       if (process.env.TEMP_RUN === "true") {
-        console.log("Save routes in temp folder for running:", routesPath);
         if (existsSync(routesPath)) {
-          console.log("Removing existing temp_routes_folder:", routesPath);
           routesPath = path.join(tmpdir(), `blinq_temp_routes`);
           rmSync(routesPath, { recursive: true });
         }
         mkdirSync(routesPath, { recursive: true });
-        console.log("Created temp_routes_folder:", routesPath);
-        saveRoutes({ step, folderPath: routesPath });
+        saveRoutes({ step, folderPath: routesPath }, logger);
       } else {
-        console.log("Saving routes in project directory:", projectDir);
         if (existsSync(routesPath)) {
-          // remove the folder
           try {
             rmSync(routesPath, { recursive: true });
-            console.log("Removed temp_routes_folder:", routesPath);
           } catch (error) {
-            console.error("Error removing temp_routes folder", error);
+            logger.error(`Error removing temp routes folder: ${getErrorMessage(error)}`);
           }
         }
         routesPath = path.join(projectDir, "data", "routes");
-        console.log("Saving routes to:", routesPath);
         if (!existsSync(routesPath)) {
           mkdirSync(routesPath, { recursive: true });
         }
-        saveRoutes({ step, folderPath: routesPath });
+        saveRoutes({ step, folderPath: routesPath }, logger);
+      }
+      if (step.commands && Array.isArray(step.commands)) {
+        step.commands = step.commands.map((cmd) => toRecordingStep(cmd, scenario.parametersMap));
       }
       continue;
     }
     const cucumberStep = getCucumberStep({ step });
     const pageName = generatePageName(step.startFrame?.url ?? "default");
     const stepDefsFilePath = locateDefinitionPath(featureFolder, pageName);
-    // path.join(stepDefinitionFolderPath, pageName + "_page.mjs");
     let codePage = getCodePage(stepDefsFilePath);
     codePage = await saveRecording({
       step,
@@ -862,7 +945,7 @@ export async function updateStepDefinitions({ scenario, featureName, projectDir
   writeFileSync(utilsFilePath, utilsContent, "utf8");
 }
 
-export function saveRoutes({ step, folderPath }) {
+export function saveRoutes({ step, folderPath }, logger) {
   const routeItems = step.routeItems;
   if (!routeItems || routeItems.length === 0) {
     return;
@@ -885,21 +968,18 @@ export function saveRoutes({ step, folderPath }) {
     };
   });
 
-  const routesFilePath = path.join(folderPath, stepNameHash + ".json");
-  console.log("Routes file path:", routesFilePath);
+  const routesFilePath = path.join(folderPath, `${stepNameHash}.json`);
   const routesData = {
     template,
     routes: routeItemsWithFilters,
   };
-  console.log("Routes data to save:", routesData);
 
   if (!existsSync(folderPath)) {
     mkdirSync(folderPath, { recursive: true });
   }
   try {
     writeFileSync(routesFilePath, JSON.stringify(routesData, null, 2), "utf8");
-    console.log("Saved routes to", routesFilePath);
   } catch (error) {
-    console.error("Failed to save routes to", routesFilePath, "Error:", error);
+    logger.error(`Error saving routes to ${routesFilePath}: ${getErrorMessage(error)}`);
   }
 }