@dev-blinq/cucumber_client 1.0.1743-dev → 1.0.1745-dev

package/bin/client/code_cleanup/utils.js

@@ -14,6 +14,40 @@ import socketLogger, { getErrorMessage } from "../utils/socket_logger.js";
 
 const STEP_KEYWORDS = new Set(["Given", "When", "Then"]);
 
+/**
+ * @typedef {Object} StepDefinitionParseError
+ * @property {"STEP_DEFINITION_PARSE_ERROR"} code
+ * @property {string} message
+ * @property {string} filePath
+ * @property {number | null} line
+ * @property {number | null} column
+ * @property {Error} cause
+ */
+
+/**
+ * Create a structured parse error for step definition files.
+ * @param {string} filePath
+ * @param {unknown} error
+ * @returns {Error & StepDefinitionParseError}
+ */
+export function createStepDefinitionParseError(filePath, error) {
+  const err = error instanceof Error ? error : new Error(String(error));
+  const loc = /** @type {any} */ (error)?.loc || /** @type {any} */ (error)?.location;
+  const line = loc?.line ?? null;
+  const column = loc?.column ?? null;
+  const message = err.message || "Failed to parse step definition file";
+
+  const parseError = new Error(
+    `Step definition parse failed in ${filePath}${line ? `:${line}${column !== null ? `:${column}` : ""}` : ""}: ${message}`
+  );
+  /** @type {Error & StepDefinitionParseError} */ (parseError).code = "STEP_DEFINITION_PARSE_ERROR";
+  /** @type {Error & StepDefinitionParseError} */ (parseError).filePath = filePath;
+  /** @type {Error & StepDefinitionParseError} */ (parseError).line = line;
+  /** @type {Error & StepDefinitionParseError} */ (parseError).column = column;
+  /** @type {Error & StepDefinitionParseError} */ (parseError).cause = err;
+  return /** @type {Error & StepDefinitionParseError} */ (parseError);
+}
+
 /**
  * Parse a step definitions file and return its AST.
  * @param {string} filePath
@@ -21,12 +55,15 @@ const STEP_KEYWORDS = new Set(["Given", "When", "Then"]);
  */
 export async function parse(filePath) {
   const code = await fs.readFile(filePath, "utf-8");
-  const ast = babelParse(code, {
-    sourceType: "module",
-
-    // plugins: ["jsx", "typescript"],
-  });
-  return ast;
+  try {
+    const ast = babelParse(code, {
+      sourceType: "module",
+      // plugins: ["jsx", "typescript"],
+    });
+    return ast;
+  } catch (error) {
+    throw createStepDefinitionParseError(filePath, error);
+  }
 }
 
 /**

package/bin/client/recorderv3/bvt_recorder.js

@@ -41,7 +41,7 @@ async function evaluate(frame, script) {
         await evaluate(childFrame, script);
     }
 }
-async function findNestedFrameSelector(frame, obj = {}) {
+async function findNestedFrameSelector(frame, obj) {
     try {
         const parent = frame.parentFrame();
         if (!parent)
@@ -49,10 +49,10 @@ async function findNestedFrameSelector(frame, obj = {}) {
         const frameElement = await frame.frameElement();
         if (!frameElement)
             return;
-        const selectors = await frameElement.evaluate((element) => {
+        const selectors = await parent.evaluate((element) => {
             const recorder = window.__bvt_Recorder;
             return recorder.locatorGenerator.getElementLocators(element, { excludeText: true }).locators;
-        });
+        }, frameElement);
         return findNestedFrameSelector(parent, { children: obj, selectors });
     }
     catch (e) {
@@ -773,11 +773,11 @@ export class BVTRecorder {
             const contextElement = document.querySelector(`[data-blinq-context-id="${contextId}"]`);
             const el = document.querySelector(`[data-blinq-id="${id}"]`);
             if (!recorder || !el) {
-                return { locators: [], allStrategyLocators: [] };
+                return { locators: [] };
             }
             if (contextElement && recorder.locatorGenerator.toContextLocators) {
                 const result = recorder.locatorGenerator.toContextLocators(el, contextElement);
-                return result ?? { locators: [], allStrategyLocators: [] };
+                return result ?? { locators: [] };
             }
             const isRecordingText = mode === "recordingText";
             return recorder.locatorGenerator.getElementLocators(el, {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dev-blinq/cucumber_client",
-  "version": "1.0.1743-dev",
+  "version": "1.0.1745-dev",
   "description": " ",
   "main": "bin/index.js",
   "types": "bin/index.d.ts",