@sun-asterisk/sungen 3.2.28 → 3.2.29

package/dist/generators/test-generator/adapters/appium/os-support.js

@@ -0,0 +1,200 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.stripComments = void 0;
+exports.osSupportOf = osSupportOf;
+exports.osSupportOfTemplate = osSupportOfTemplate;
+const fs = __importStar(require("node:fs"));
+const path = __importStar(require("node:path"));
+/**
+ * The per-OS axis of the step-support matrix, DERIVED from each Appium template's own body.
+ *
+ * The Web|Mobile columns answer "does a template exist". They cannot answer the question that
+ * actually bites: a template can ship, compile, and still do nothing on one of the two mobile
+ * runtimes. `hide-keyboard-action` is guarded `if (!driver.isIOS && …)` — on iOS the step compiles,
+ * runs, and silently skips. Nothing in the matrix showed that, so nobody could see iOS drifting
+ * behind Android, which is exactly what the Android-first decision needs to be able to see.
+ *
+ * Derived, never declared. A hand-written per-OS column is the same mistake the spec made when it
+ * claimed capabilities the code did not have: two sources of truth, one of them wrong, no gate to
+ * catch it. Reading the template body means the column cannot lie for longer than a regenerate.
+ *
+ * A template mentioning `driver.isIOS` in a shape this file does not recognise is a THROW, never a
+ * default. Defaulting to 'yes' would let a new one-sided template enter the matrix reported as
+ * working on both — the precise failure the axis exists to prevent.
+ */
+/**
+ * Strip handlebars comments so a body is judged by what it EXECUTES, never by prose ABOUT iOS.
+ * Several templates explain their per-OS reasoning at length in a `{{!-- … --}}` block; counting
+ * that as code would classify half the adapter wrongly.
+ */
+const stripComments = (src) => src.replace(/\{\{!--[\s\S]*?--\}\}/g, '').replace(/\{\{![^}]*?\}\}/g, '');
+exports.stripComments = stripComments;
+/** Worse wins: a template with both a per-OS read and an iOS refusal is an iOS refusal. */
+const RANK = { inherit: 0, yes: 1, noop: 2, refuses: 3 };
+const worse = (a, b) => (RANK[b] > RANK[a] ? b : a);
+/**
+ * The statement (or block) an `if (…)` condition guards.
+ *
+ * Brace-matched rather than regex-matched because the distinction that matters is whether a `throw`
+ * sits INSIDE the OS branch. `swipe-action` throws for a not-found element and implements iOS fully;
+ * a file-level `/throw/` test would read that as an iOS refusal and put a lie in the table.
+ */
+function guardedBody(code, from) {
+    let i = from;
+    while (i < code.length && /\s/.test(code[i]))
+        i++;
+    if (code[i] !== '{') {
+        // Braceless form — `if (driver.isIOS) await x(); else await y();` (double-click-action).
+        const end = code.indexOf(';', i);
+        return code.slice(i, end === -1 ? code.length : end);
+    }
+    let depth = 0;
+    for (let j = i; j < code.length; j++) {
+        if (code[j] === '{')
+            depth++;
+        else if (code[j] === '}' && --depth === 0)
+            return code.slice(i + 1, j);
+    }
+    return code.slice(i); // unbalanced source — the caller's fail-loud path reports it
+}
+const startsWithThrow = (body) => /^\s*throw\b/.test(body);
+/**
+ * Classify one `driver.isIOS` occurrence. Returns null when the shape is unrecognised, which the
+ * caller turns into a throw.
+ */
+function classifyOccurrence(code, at) {
+    const before = code.slice(0, at).replace(/\s+$/, '');
+    const negated = before.endsWith('!');
+    const after = code.slice(at + 'driver.isIOS'.length);
+    // `driver.isIOS ? <ios> : <android>` — a per-OS read, both arms real. The `?` may sit on the
+    // NEXT line: every assertion template formats the ternary across three lines, so a same-line
+    // test would miss twenty templates and report them unrecognised.
+    if (!negated && /^\s*\?/.test(after))
+        return { android: 'yes', ios: 'yes' };
+    // Condition forms. `)` closes the `if (`, so the guarded body starts after it.
+    const closeParen = after.indexOf(')');
+    if (closeParen === -1)
+        return null;
+    // Anything other than whitespace/`&&`-chained conditions between the flag and the `)` means a
+    // shape this file has not been taught to read.
+    const between = after.slice(0, closeParen);
+    if (!/^[\s)]*$/.test(between) && !/^\s*&&/.test(between))
+        return null;
+    const body = guardedBody(after, closeParen + 1);
+    if (negated) {
+        // `!driver.isIOS` guards the ANDROID path.
+        //   with a throw  → Android refuses (the mirror case; none exists today, and if one is added
+        //                   it must not be silently reported as an iOS no-op)
+        //   without       → iOS is skipped with no signal at all: 'noop'
+        return startsWithThrow(body)
+            ? { android: 'refuses', ios: 'yes' }
+            : { android: 'yes', ios: 'noop' };
+    }
+    // `driver.isIOS` guards the iOS path: a leading throw is a refusal, anything else is real code
+    // (the `else` arm carries Android either way).
+    return startsWithThrow(body) ? { android: 'yes', ios: 'refuses' } : { android: 'yes', ios: 'yes' };
+}
+/**
+ * Per-OS support for one Appium template, read off its body.
+ *
+ * @throws when the template branches on `driver.isIOS` in an unrecognised shape — deliberately, so
+ *         the drift gate goes red and a human teaches this file the new shape, rather than the
+ *         matrix quietly claiming the step works on both.
+ */
+function osSupportOf(name, src) {
+    const code = (0, exports.stripComments)(src);
+    if (!code.includes('driver.isIOS'))
+        return { android: 'inherit', ios: 'inherit' };
+    let pair = { android: 'yes', ios: 'yes' };
+    let seen = 0;
+    for (let i = code.indexOf('driver.isIOS'); i !== -1; i = code.indexOf('driver.isIOS', i + 1)) {
+        const one = classifyOccurrence(code, i);
+        if (!one) {
+            throw new Error(`[step-support-matrix] ${name}.hbs branches on driver.isIOS in a shape os-support.ts cannot read ` +
+                `(at offset ${i}). Teach classifyOccurrence() the new shape — do NOT let it default, or the ` +
+                `matrix will report a one-sided template as working on both.\n` +
+                `  context: ${JSON.stringify(code.slice(Math.max(0, i - 40), i + 80))}`);
+        }
+        seen++;
+        pair = { android: worse(pair.android, one.android), ios: worse(pair.ios, one.ios) };
+    }
+    if (!seen)
+        throw new Error(`[step-support-matrix] ${name}.hbs mentions driver.isIOS but no occurrence classified`);
+    return pair;
+}
+/** Appium step templates, the same tree `renderStep` searches. */
+const TEMPLATES_DIR = path.join(__dirname, 'templates', 'steps');
+const cache = new Map();
+/** Locate `<name>.hbs` anywhere under the appium step tree (excluding `partials/`). */
+function findTemplate(dir, file) {
+    for (const e of fs.readdirSync(dir, { withFileTypes: true })) {
+        if (e.isDirectory()) {
+            if (e.name === 'partials')
+                continue;
+            const hit = findTemplate(path.join(dir, e.name), file);
+            if (hit)
+                return hit;
+        }
+        else if (e.name === file) {
+            return path.join(dir, e.name);
+        }
+    }
+    return null;
+}
+/**
+ * Per-OS support for a template BY NAME, memoized — the compile-time `@platform:ios` gate asks this
+ * once per distinct template per run.
+ *
+ * Returns null when the appium adapter ships no such template: that is a different failure (the
+ * step is not a mobile step at all) which the existing template-resolution path already reports, so
+ * answering "unsupported on iOS" here would be a second, misleading message for one cause.
+ */
+function osSupportOfTemplate(name) {
+    if (cache.has(name))
+        return cache.get(name) ?? null;
+    let pair = null;
+    try {
+        const file = findTemplate(TEMPLATES_DIR, `${name}.hbs`);
+        if (file)
+            pair = osSupportOf(name, fs.readFileSync(file, 'utf-8'));
+    }
+    catch {
+        pair = null; // an unreadable tree must not break codegen; the matrix gate reports it instead
+    }
+    cache.set(name, pair);
+    return pair;
+}
+//# sourceMappingURL=os-support.js.map

package/dist/generators/test-generator/adapters/appium/os-support.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"os-support.js","sourceRoot":"","sources":["../../../../../src/generators/test-generator/adapters/appium/os-support.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAkHA,kCAqBC;AA6BD,kDAWC;AA/KD,4CAA8B;AAC9B,gDAAkC;AAElC;;;;;;;;;;;;;;;;GAgBG;AAEH;;;;GAIG;AACI,MAAM,aAAa,GAAG,CAAC,GAAW,EAAU,EAAE,CACnD,GAAG,CAAC,OAAO,CAAC,wBAAwB,EAAE,EAAE,CAAC,CAAC,OAAO,CAAC,kBAAkB,EAAE,EAAE,CAAC,CAAC;AAD/D,QAAA,aAAa,iBACkD;AAa5E,2FAA2F;AAC3F,MAAM,IAAI,GAA8B,EAAE,OAAO,EAAE,CAAC,EAAE,GAAG,EAAE,CAAC,EAAE,IAAI,EAAE,CAAC,EAAE,OAAO,EAAE,CAAC,EAAE,CAAC;AACpF,MAAM,KAAK,GAAG,CAAC,CAAY,EAAE,CAAY,EAAa,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;AAErF;;;;;;GAMG;AACH,SAAS,WAAW,CAAC,IAAY,EAAE,IAAY;IAC7C,IAAI,CAAC,GAAG,IAAI,CAAC;IACb,OAAO,CAAC,GAAG,IAAI,CAAC,MAAM,IAAI,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAAE,CAAC,EAAE,CAAC;IAClD,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;QACpB,yFAAyF;QACzF,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC,CAAC;QACjC,OAAO,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC;IACvD,CAAC;IACD,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACrC,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,GAAG;YAAE,KAAK,EAAE,CAAC;aACxB,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,GAAG,IAAI,EAAE,KAAK,KAAK,CAAC;YAAE,OAAO,IAAI,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC;IACzE,CAAC;IACD,OAAO,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,6DAA6D;AACrF,CAAC;AAED,MAAM,eAAe,GAAG,CAAC,IAAY,EAAW,EAAE,CAAC,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAE5E;;;GAGG;AACH,SAAS,kBAAkB,CAAC,IAAY,EAAE,EAAU;IAClD,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,OAAO,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;IACrD,MAAM,OAAO,GAAG,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC;IACrC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,EAAE,GAAG,cAAc,CAAC,MAAM,CAAC,CAAC;IAErD,6FAA6F;IAC7F,6FAA6F;IAC7F,iEAAiE;IACjE,IAAI,CAAC,OAAO,IAAI,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC;QAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,EAAE,KAAK,EAAE,CAAC;IAE5E,+EAA+E;IAC/E,MAAM,UAAU,GAAG,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;IACtC,IAAI,UAAU,KAAK,CAAC,CAAC;QAAE,OAAO,IAAI,CAAC;IACnC,8FAA8F;IAC9F,+CAA+C;IAC/C,MAAM,OAAO,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,UAAU,CAAC,CAAC;IAC3C,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC;QAAE,OAAO,IAAI,CAAC;IAEtE,MAAM,IAAI,GAAG,WAAW,CAAC,KAAK,EAAE,UAAU,GAAG,CAAC,CAAC,CAAC;IAEhD,IAAI,OAAO,EAAE,CAAC;QACZ,2CAA2C;QAC3C,6FAA6F;QAC7F,sEAAsE;QACtE,iEAAiE;QACjE,OAAO,eAAe,CAAC,IAAI,CAAC;YAC1B,CAAC,CAAC,EAAE,OAAO,EAAE,SAAS,EAAE,GAAG,EAAE,KAAK,EAAE;YACpC,CAAC,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,CAAC;IACtC,CAAC;IACD,+FAA+F;IAC/F,+CAA+C;IAC/C,OAAO,eAAe,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,EAAE,KAAK,EAAE,CAAC;AACrG,CAAC;AAED;;;;;;GAMG;AACH,SAAgB,WAAW,CAAC,IAAY,EAAE,GAAW;IACnD,MAAM,IAAI,GAAG,IAAA,qBAAa,EAAC,GAAG,CAAC,CAAC;IAChC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,cAAc,CAAC;QAAE,OAAO,EAAE,OAAO,EAAE,SAAS,EAAE,GAAG,EAAE,SAAS,EAAE,CAAC;IAElF,IAAI,IAAI,GAAW,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,EAAE,KAAK,EAAE,CAAC;IAClD,IAAI,IAAI,GAAG,CAAC,CAAC;IACb,KAAK,IAAI,CAAC,GAAG,IAAI,CAAC,OAAO,CAAC,cAAc,CAAC,EAAE,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,GAAG,IAAI,CAAC,OAAO,CAAC,cAAc,EAAE,CAAC,GAAG,CAAC,CAAC,EAAE,CAAC;QAC7F,MAAM,GAAG,GAAG,kBAAkB,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;QACxC,IAAI,CAAC,GAAG,EAAE,CAAC;YACT,MAAM,IAAI,KAAK,CACb,yBAAyB,IAAI,qEAAqE;gBAClG,cAAc,CAAC,8EAA8E;gBAC7F,+DAA+D;gBAC/D,cAAc,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,CACxE,CAAC;QACJ,CAAC;QACD,IAAI,EAAE,CAAC;QACP,IAAI,GAAG,EAAE,OAAO,EAAE,KAAK,CAAC,IAAI,CAAC,OAAO,EAAE,GAAG,CAAC,OAAO,CAAC,EAAE,GAAG,EAAE,KAAK,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC;IACtF,CAAC;IACD,IAAI,CAAC,IAAI;QAAE,MAAM,IAAI,KAAK,CAAC,yBAAyB,IAAI,yDAAyD,CAAC,CAAC;IACnH,OAAO,IAAI,CAAC;AACd,CAAC;AAED,kEAAkE;AAClE,MAAM,aAAa,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,WAAW,EAAE,OAAO,CAAC,CAAC;AAEjE,MAAM,KAAK,GAAG,IAAI,GAAG,EAAyB,CAAC;AAE/C,uFAAuF;AACvF,SAAS,YAAY,CAAC,GAAW,EAAE,IAAY;IAC7C,KAAK,MAAM,CAAC,IAAI,EAAE,CAAC,WAAW,CAAC,GAAG,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC,EAAE,CAAC;QAC7D,IAAI,CAAC,CAAC,WAAW,EAAE,EAAE,CAAC;YACpB,IAAI,CAAC,CAAC,IAAI,KAAK,UAAU;gBAAE,SAAS;YACpC,MAAM,GAAG,GAAG,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,CAAC,EAAE,IAAI,CAAC,CAAC;YACvD,IAAI,GAAG;gBAAE,OAAO,GAAG,CAAC;QACtB,CAAC;aAAM,IAAI,CAAC,CAAC,IAAI,KAAK,IAAI,EAAE,CAAC;YAC3B,OAAO,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC;QAChC,CAAC;IACH,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;;;;GAOG;AACH,SAAgB,mBAAmB,CAAC,IAAY;IAC9C,IAAI,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC;QAAE,OAAO,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC;IACpD,IAAI,IAAI,GAAkB,IAAI,CAAC;IAC/B,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,YAAY,CAAC,aAAa,EAAE,GAAG,IAAI,MAAM,CAAC,CAAC;QACxD,IAAI,IAAI;YAAE,IAAI,GAAG,WAAW,CAAC,IAAI,EAAE,EAAE,CAAC,YAAY,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC,CAAC;IACrE,CAAC;IAAC,MAAM,CAAC;QACP,IAAI,GAAG,IAAI,CAAC,CAAC,gFAAgF;IAC/F,CAAC;IACD,KAAK,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;IACtB,OAAO,IAAI,CAAC;AACd,CAAC"}

package/dist/generators/test-generator/adapters/appium/templates/steps/actions/alert-fill-action.hbs

@@ -1,2 +1,9 @@
-{{!-- Filling a native prompt is iOS-specific and app-dependent; Android rarely has fillable alerts.
-      Requested value: '{{fillValue}}'. Handle manually if your alert has a text field. --}}
+{{!-- Filling a native prompt has no Appium implementation here. This file used to be comment-ONLY,
+      which made it the one member of the web-only set that failed SILENTLY: the comment is stripped,
+      so the step emitted nothing, ran, and did nothing — a scenario could fill a prompt and pass
+      having never touched it. Every other web-only step on this adapter refuses by name, and now so
+      does this one. `alert-fill` also declares `platforms: ['web']`, so the refusal normally arrives
+      at COMPILE time (SG-W020); this body is the second line of defence for any future pattern that
+      reaches the template without declaring its platforms.
+      Requested value: '{{fillValue}}' --}}
+throw new Error("[sungen] filling a native prompt is not supported on the appium adapter. Accept or dismiss the alert instead, drive the app's own text field as a normal element, or tag this scenario @manual:M10.");

package/dist/generators/test-generator/adapters/appium/templates/steps/actions/capture-variable.hbs

@@ -6,5 +6,28 @@
   const __sources = driver.isIOS
     ? {{#if (eq capture 'inputValue')}}[await __el.getAttribute('value'), await __el.getAttribute('label')]{{else}}[await __el.getAttribute('label'), await __el.getAttribute('value')]{{/if}}
     : [await __el.getText(), await __el.getAttribute('content-desc')];
-  testData.set('{{escapeQuotes varName}}', String(__sources.find(s => s && String(s).trim() !== '') ?? '').trim());
+  let __captured = String(__sources.find(s => s && String(s).trim() !== '') ?? '').trim();
+{{#if (eq capture 'inputValue')}}
+  {{!-- iOS placeholder guard. Some WebDriverAgent versions echo an empty UITextField's
+        placeholder back as its @value (the XCUITest driver documents @value as
+        "a complex attribute whose calculation algorithm depends on the actual element type"
+        and promises nothing for an empty field). Unguarded, an empty field captures its
+        PLACEHOLDER into testData under the author's variable name — and because a capture
+        feeds later steps, that corruption propagates silently instead of failing loudly.
+        Unlike the assertion path there is no expected value to gate on, so this check is
+        unconditional on iOS: one extra round-trip buys a captured value that is not a lie. --}}
+  if (driver.isIOS && __captured !== '') {
+    let __placeholder = null;
+    try { __placeholder = await __el.getAttribute('placeholderValue'); } catch { /* attribute unsupported on this node */ }
+    if (__placeholder && String(__placeholder).trim() === __captured) {
+      {{!-- Leave a breadcrumb. Unlike the assertion path — which throws at the field in
+            question — a discarded capture succeeds silently and the empty string travels to
+            whatever step later consumes it, possibly in another file. That downstream failure
+            carries nothing linking it back here, so say it out loud now. --}}
+      console.warn(`  ! [sungen] {{escapeQuotes varName}} read back as its own placeholder ("${__captured}") — captured empty instead`);
+      __captured = '';
+    }
+  }
+{{/if}}
+  testData.set('{{escapeQuotes varName}}', __captured);
 }

package/dist/generators/test-generator/adapters/appium/templates/steps/actions/hover-action.hbs

@@ -1,2 +1,6 @@
-{{!-- No hover on touch devices — no-op. (Hover-revealed content is normally already visible on mobile;
-      if a tap is actually required, use `User tap [X]` instead.) --}}
+{{!-- No hover on a touch screen. This template used to hold only this comment, so the step compiled
+      to an empty line: the scenario claimed to exercise a hover, exercised nothing, and passed.
+      `hover-element` now declares platforms: ['web'], so a mobile project is refused at SG-W020
+      before reaching here — this throw is the second source agreeing with the first, the treatment
+      alert-fill-action already has. --}}
+throw new Error("[sungen] hover is not available on the appium adapter — a touch screen has no hover state. Content a hover reveals is normally already visible on mobile; if the interaction is genuinely required, use `User tap [X]`, otherwise tag the scenario @manual:M10.");

package/dist/generators/test-generator/adapters/appium/templates/steps/actions/hover-element-with-text.hbs

@@ -1,3 +1,4 @@
-{{!-- No hover on touch devices — no-op, mirrors hover-action.hbs. (A text-filtered hover target
-      is normally already visible on mobile; if a tap on the matching element is actually
-      required, use `User tap [X] item with {{value}}` instead.) --}}
+{{!-- No hover on a touch screen; mirrors hover-action.hbs. Text-filtering the target changes which
+      element is picked, not whether hovering it means anything. Braces are avoided in the message
+      below on purpose: handlebars would interpolate a literal mustache away to an empty string. --}}
+throw new Error("[sungen] hover is not available on the appium adapter — a touch screen has no hover state. To act on the item matching a value, tap it instead (`User tap [X] item with` your data ref); if the hover itself is the thing under test, tag the scenario @manual:M10.");

package/dist/generators/test-generator/adapters/appium/templates/steps/actions/unknown-element-action.hbs

@@ -1,2 +1,11 @@
+{{#if (eq action "hover")}}
+{{!-- This template is reached by a MIXED pattern that serves click, double-click AND hover, so it
+      cannot be declared web-only without deleting tap from mobile. It also ignored `action` and
+      always clicked — which turned `hover [Target X] with {{ref}}` into a TAP on mobile. That is
+      worse than the silent no-op the pure hover patterns had: a tap has side effects a hover never
+      had, so the scenario changed app state while claiming to hover. Refuse instead. --}}
+throw new Error("[sungen] hover is not available on the appium adapter — a touch screen has no hover state, and this step would otherwise have tapped the element instead. Use `User tap` if acting on it is what you meant, or tag the scenario @manual:M10.");
+{{else}}
 {{!-- Fallback for an unrecognized action: tap an element by text / content-desc. --}}
 await $('//*[contains(@content-desc,"{{escapeQuotes dataValue}}") or contains(@text,"{{escapeQuotes dataValue}}")]').click();
+{{/if}}

package/dist/generators/test-generator/adapters/appium/templates/steps/assertions/have-value-assertion.hbs

@@ -1 +1,34 @@
-await expect({{> locator}}).toHaveValue('{{escapeQuotes dataValue}}');
+{{!-- Input-value assertion (mobile). The web template asserts toHaveValue(), which
+      expect-webdriverio implements as toHaveElementProperty('value') → GET
+      element/:id/property/value. That is a DOM command: UiAutomator2 does not implement
+      getElementProperty and answers 404 for every call, so the assertion used to fail
+      whatever the field held. Read the native value instead, exactly as
+      actions/capture-variable.hbs does for `capture: 'inputValue'`:
+        · iOS  — @value first, then @label (this is the input's OWN value, not its label)
+        · Android — getText() (GET element/:id/text), then @content-desc
+      `probeNative` is deliberately ignored: it drives a DOM tagName probe for ambiguous
+      dropdown/select/combobox targets on web, and a native tree has no tagName to probe. --}}
+{
+  const __el = await {{> locator}};
+  await __el.waitForDisplayed();
+  const __expected = String('{{escapeQuotes dataValue}}').trim();
+  const __sources = driver.isIOS
+    ? [await __el.getAttribute('value'), await __el.getAttribute('label')]
+    : [await __el.getText(), await __el.getAttribute('content-desc')];
+  let __actual = String(__sources.find(s => s && String(s).trim() !== '') ?? '').trim();
+  {{!-- iOS only, and only when we EXPECT empty: some WebDriverAgent versions echo an empty
+        UITextField's placeholder back as its @value. The XCUITest driver documents @value as
+        "a complex attribute whose calculation algorithm depends on the actual element type"
+        and promises nothing for an empty field, so the behaviour varies by WDA version.
+        @placeholderValue is a supported XCUITest attribute; an exact match means the field is
+        empty and we simply read its placeholder. Gated on __expected === '' so a non-empty
+        assertion pays no extra round-trip and its behaviour is bit-for-bit unchanged. --}}
+  if (driver.isIOS && __expected === '' && __actual !== '') {
+    let __placeholder = null;
+    try { __placeholder = await __el.getAttribute('placeholderValue'); } catch { /* attribute unsupported on this node */ }
+    if (__placeholder && String(__placeholder).trim() === __actual) __actual = '';
+  }
+  if (__actual !== __expected) {
+    throw new Error(`[sungen] Expected value "${__expected}" but element reads "${__actual}"`);
+  }
+}

package/dist/generators/test-generator/adapters/appium/templates/steps/assertions/label-value-assertion.hbs

@@ -1,12 +1,20 @@
-{{!-- Label-value assertion (mobile): locate a visible text node carrying the value.
-      Android renders text into @text; iOS (RN/Flutter) into @label/@value. The web variant
-      matches "label: value" as one string — native trees keep label and value as separate
-      nodes, so only the value is matched here. The expected value must stay a standalone
-      quoted literal so the runtime-data transform can swap it for testData.get(...). --}}
+{{!-- Label-value assertion (mobile), NO selectors.yaml entry: locate a visible node carrying
+      the value anywhere on screen. The web variant matches "label: value" as one string —
+      native trees keep label and value as separate nodes, so only the value is matched here.
+      Declare an entry for the ref to assert a specific element instead (#666).
+
+      Android publishes text into @text (native/RN) or @content-desc (Flutter Semantics), and
+      which one is populated is a property of the toolkit, not of the widget. This used to be
+      `UiSelector().text(...)`, which cannot read @content-desc at all and so missed on every
+      Flutter screen; UiSelector has no OR, hence the xpath. iOS scatters it across
+      label/value, which the predicate already ORs.
+
+      The expected value must stay a standalone quoted literal so the runtime-data transform
+      can swap it for testData.get(...). --}}
 {
   const __expected = String('{{escapeQuotes dataValue}}').trim();
   const __sel = driver.isIOS
     ? `-ios predicate string:label == "${__expected}" OR value == "${__expected}"`
-    : `android=new UiSelector().text("${__expected}")`;
+    : `//*[@text="${__expected}" or @content-desc="${__expected}"]`;
   await expect($(__sel)).toBeDisplayed();
 }

package/dist/generators/test-generator/adapters/appium/templates/steps/gestures/go-back-action.hbs

@@ -0,0 +1,21 @@
+{{!-- The OS/hardware back gesture. NOT the in-app Back button, and the difference is the whole
+      point: Android's system back does NOT run the screen's own handler unless the app wires it
+      (onBackPressed / BackHandler), so a defect on this path is invisible to a `tap [Back]`
+      scenario. That is exactly why `viewpoint-ledger.ts` treats the in-app control as a wrong
+      substitution for the "OS/hardware back gesture" theme (#626) — and why this template has to
+      exist for that theme to have any answer other than @manual.
+
+      Android: KEYCODE_BACK through the execute alias rather than `driver.pressKeyCode(4)`, for the
+      same reason `location-services-action` uses `mobile: isGpsEnabled` — WDIO v9 dropped the
+      legacy JSONWP wrappers, while a driver extension call cannot be removed from under us.
+
+      iOS: there IS no hardware back. `driver.back()` is WebDriverAgent's NAVIGATION back (it drives
+      the nav-bar back affordance), not the user's edge swipe-back gesture. Both OSes are therefore
+      implemented — the matrix reads ✅/✅ and the semantic difference is stated here and in the
+      matrix note, rather than dressed up as a refusal it is not. --}}
+if (driver.isIOS) {
+  // WDA navigation-back — NOT the user's swipe-back gesture; iOS exposes no hardware back key.
+  await driver.back();
+} else {
+  await driver.execute('mobile: pressKey', { keycode: 4 }); // KEYCODE_BACK — the real hardware back
+}

package/dist/generators/test-generator/adapters/appium/templates/steps/gestures/set-geolocation-action.hbs

@@ -1,2 +1,18 @@
+{{#if missing}}
+{{!-- No coordinates in the step and no data reference to read. Refuse by name: this used to default
+      to latitude 0 / longitude 0, which is Null Island — a real place, so the test passed there. --}}
+throw new Error("[sungen] cannot set location: no coordinates and no data reference in \"{{escapeQuotes missing}}\". Write the numbers inline (`User set location to 10.7769, 106.7009`), or point at a test-data key holding \"<latitude> <longitude>\".");
+{{else}}
+{{#if dataRef}}
+{{!-- Coordinates read from test-data AT RUN TIME, so one compiled spec serves every environment. --}}
+await (async () => {
+  const __raw = String(testData.get('{{dataRef}}'));
+  const __nums = __raw.match(/-?\d+(?:\.\d+)?/g) || [];
+  if (__nums.length < 2) throw new Error(`[sungen] test-data '{{dataRef}}' = "${__raw}" — setting a location needs two numbers, "<latitude> <longitude>"; found ${__nums.length}.`);
+  await driver.setGeoLocation({ latitude: Number(__nums[0]), longitude: Number(__nums[1]), altitude: 0 });
+})();
+{{else}}
 {{!-- Mock the device GPS location. --}}
 await driver.setGeoLocation({ latitude: {{latitude}}, longitude: {{longitude}}, altitude: 0 });
+{{/if}}
+{{/if}}

package/dist/generators/test-generator/adapters/appium/templates/test-file.hbs

@@ -1,6 +1,14 @@
 {{#if platform}}
 // sungen:platform={{platform}}
 {{/if}}
+{{#if screenshotOnFailure}}
+// @screenshot:on-failure — used to be web-only and silently dropped on mobile.
+// wdio.conf.ts's afterTest hook already captures screenshot + page source + device log for EVERY
+// mobile failure, so the tag is not what turns evidence on. What it declares is that this feature's
+// evidence is ESSENTIAL: it survives SUNGEN_MOBILE_EVIDENCE=off, which otherwise suppresses capture
+// for the run. Set at module load, so it covers every scenario in this spec.
+(globalThis as Record<string, unknown>).__sungenForceEvidence = true;
+{{/if}}
 {{#if runtimeData}}
 
 const testData = TestDataLoader.load('{{screenName}}', '{{featureFileName}}');

package/dist/generators/test-generator/adapters/playwright/templates/steps/actions/table-action-in-row-nth.hbs

@@ -1,2 +1,5 @@
-{ const tableRow = {{> locator}}.getByRole('row').nth({{{rowIndexExpr}}});
+{{!-- Row universe: `rows:` on the table entry (what EVERY row matches — a <div> list has no
+     role=row, so the ARIA chain below is 0 elements and this assertion would be vacuous, issue #688)
+     wins; otherwise the historical ARIA chain, byte-identical. --}}
+{ const tableRow = {{> locator}}{{#if rows}}.locator('{{escapeQuotes rows}}'){{else}}.getByRole('row'){{/if}}.nth({{{rowIndexExpr}}});
 await {{{actionLocator}}}.{{action}}(); }

package/dist/generators/test-generator/adapters/playwright/templates/steps/actions/table-action-in-row.hbs

@@ -1,2 +1,5 @@
-{ const tableRow = {{> locator}}.getByRole('row').filter({ hasText: '{{escapeQuotes filterValue}}' });
+{{!-- Row universe: `rows:` on the table entry (what EVERY row matches — a <div> list has no
+     role=row, so the ARIA chain below is 0 elements and this assertion would be vacuous, issue #688)
+     wins; otherwise the historical ARIA chain, byte-identical. --}}
+{ const tableRow = {{> locator}}{{#if rows}}.locator('{{escapeQuotes rows}}'){{else}}.getByRole('row'){{/if}}.filter({ hasText: '{{escapeQuotes filterValue}}' });
 await {{{actionLocator}}}.{{action}}(); }

package/dist/generators/test-generator/adapters/playwright/templates/steps/assertions/selected-assertion.hbs

@@ -9,6 +9,13 @@ await expect({{> locator}}).toBeChecked();
 { const el = {{> locator}};
 if ((await el.getAttribute('aria-selected')) !== null) {
 await expect(el).toHaveAttribute('aria-selected', 'true');
+} else if ((await el.getAttribute('aria-current')) !== null) {
+{{!-- aria-current marks the CURRENT item of a navigation set (WAI-ARIA 1.1): a Next.js tab strip
+     of plain <a> links carries aria-current="page" and has neither aria-selected nor a checked
+     state, so toBeChecked() threw "Not a checkbox or radio button" on a correct step. Any value
+     but "false" ("page", "step", "true", …) means current. Probed AFTER aria-selected: a real
+     role=tab owns the stronger signal. --}}
+await expect(el).toHaveAttribute('aria-current', /^(?!false$)./);
 } else {
 await expect(el).toBeChecked();
 }

package/dist/generators/test-generator/adapters/playwright/templates/steps/assertions/state-with-filter-assertion.hbs

@@ -4,6 +4,13 @@
 { const el = {{> locator-base}}.filter({ hasText: {{> has-text-value dataValue=dataValue exact=exact}} }){{> locator-nth}};
 if ((await el.getAttribute('aria-selected')) !== null) {
 await expect(el).toHaveAttribute('aria-selected', 'true');
+} else if ((await el.getAttribute('aria-current')) !== null) {
+{{!-- aria-current marks the CURRENT item of a navigation set (WAI-ARIA 1.1): a Next.js tab strip
+     of plain <a> links carries aria-current="page" and has neither aria-selected nor a checked
+     state, so toBeChecked() threw "Not a checkbox or radio button" on a correct step. Any value
+     but "false" ("page", "step", "true", …) means current. Probed AFTER aria-selected: a real
+     role=tab owns the stronger signal. --}}
+await expect(el).toHaveAttribute('aria-current', /^(?!false$)./);
 } else {
 await expect(el).toBeChecked();
 }

package/dist/generators/test-generator/adapters/playwright/templates/steps/assertions/table-empty.hbs

@@ -1 +1,4 @@
-await expect({{> locator}}.locator('tbody').getByRole('row', { includeHidden: true }).filter({ visible: true })).toHaveCount(0);
+{{!-- Row universe: `rows:` on the table entry (what EVERY row matches — a <div> list has no
+     role=row, so the ARIA chain below is 0 elements and this assertion would be vacuous, issue #688)
+     wins; otherwise the historical ARIA chain, byte-identical. --}}
+await expect({{> locator}}{{#if rows}}.locator('{{escapeQuotes rows}}'){{else}}.locator('tbody').getByRole('row', { includeHidden: true }){{/if}}.filter({ visible: true })).toHaveCount(0);

package/dist/generators/test-generator/adapters/playwright/templates/steps/assertions/table-match-data.hbs

@@ -1,6 +1,9 @@
 {
   const table = {{> locator}};
-  const rows = table.locator('tbody').getByRole('row', { includeHidden: true }).filter({ visible: true });
+{{!-- Row universe: `rows:` on the table entry (what EVERY row matches — a <div> list has no
+     role=row, so the ARIA chain below is 0 elements and this assertion would be vacuous, issue #688)
+     wins; otherwise the historical ARIA chain, byte-identical. --}}
+  const rows = table{{#if rows}}.locator('{{escapeQuotes rows}}'){{else}}.locator('tbody').getByRole('row', { includeHidden: true }){{/if}}.filter({ visible: true });
 {{#each assertions}}
   {{{this}}}
 {{/each~}}

package/dist/generators/test-generator/adapters/playwright/templates/steps/assertions/table-row-count.hbs

@@ -1 +1,4 @@
-await expect({{> locator}}.locator('tbody').getByRole('row', { includeHidden: true }).filter({ visible: true })).toHaveCount({{expectedCount}});
+{{!-- Row universe: `rows:` on the table entry (what EVERY row matches — a <div> list has no
+     role=row, so the ARIA chain below is 0 elements and this assertion would be vacuous, issue #688)
+     wins; otherwise the historical ARIA chain, byte-identical. --}}
+await expect({{> locator}}{{#if rows}}.locator('{{escapeQuotes rows}}'){{else}}.locator('tbody').getByRole('row', { includeHidden: true }){{/if}}.filter({ visible: true })).toHaveCount({{expectedCount}});

package/dist/generators/test-generator/adapters/playwright/templates/steps/assertions/table-row-exists.hbs

@@ -1,17 +1,25 @@
 {{~#if rowSelector}}
+{{!-- Declared row LABEL = one positioned row. The toHaveCount(1) guard is what turns "I used the
+     row label as the row SHAPE" (div.job-row, issue #688) into a named error instead of a
+     strict-mode message pointing at the wrong line. --}}
 {{~#if isGiven}}
-const tableRow = {{> locator}}.locator('{{rowSelector}}');
+const tableRow = {{> locator}}.locator('{{escapeQuotes rowSelector}}');
+await expect(tableRow, 'row label "[{{escapeQuotes rowLabel}}]" must position exactly ONE row (its selectors.yaml entry is relative to the table) — if this selector describes what EVERY row looks like, declare it as `rows:` on the TABLE entry instead').toHaveCount(1);
 await tableRow.filter({ hasText: '{{escapeQuotes filterValue}}' }).waitFor();
 {{~else}}
-const tableRow = {{> locator}}.locator('{{rowSelector}}');
+const tableRow = {{> locator}}.locator('{{escapeQuotes rowSelector}}');
+await expect(tableRow, 'row label "[{{escapeQuotes rowLabel}}]" must position exactly ONE row (its selectors.yaml entry is relative to the table) — if this selector describes what EVERY row looks like, declare it as `rows:` on the TABLE entry instead').toHaveCount(1);
 await expect(tableRow).toContainText('{{escapeQuotes filterValue}}');
 {{~/if}}
 {{~else}}
+{{!-- Row universe: `rows:` on the table entry (what EVERY row matches — a <div> list has no
+     role=row, so the ARIA chain below is 0 elements and this assertion would be vacuous, issue #688)
+     wins; otherwise the historical ARIA chain, byte-identical. --}}
 {{~#if isGiven}}
-const tableRow = {{> locator}}.getByRole('row', { includeHidden: true }).filter({ hasText: '{{escapeQuotes filterValue}}' }).filter({ visible: true });
+const tableRow = {{> locator}}{{#if rows}}.locator('{{escapeQuotes rows}}'){{else}}.getByRole('row', { includeHidden: true }){{/if}}.filter({ hasText: '{{escapeQuotes filterValue}}' }).filter({ visible: true });
 await tableRow.waitFor();
 {{~else}}
-const tableRow = {{> locator}}.getByRole('row', { includeHidden: true }).filter({ hasText: '{{escapeQuotes filterValue}}' }).filter({ visible: true });
+const tableRow = {{> locator}}{{#if rows}}.locator('{{escapeQuotes rows}}'){{else}}.getByRole('row', { includeHidden: true }){{/if}}.filter({ hasText: '{{escapeQuotes filterValue}}' }).filter({ visible: true });
 await expect(tableRow).toBeVisible();
 {{~/if}}
 {{~/if}}

package/dist/generators/test-generator/adapters/playwright/templates/steps/assertions/table-row-not-exists.hbs

@@ -1,5 +1,17 @@
+{{~#if rowSelector}}
+{{!-- Declared row LABEL = one positioned row (same contract as the exists twin). The row must not
+     hold the value. Block-scoped: this step does not enter row scope, so it must not redeclare
+     the function-level tableRow the exists step binds. --}}
+{ const tableRow = {{> locator}}.locator('{{escapeQuotes rowSelector}}');
+await expect(tableRow, 'row label "[{{escapeQuotes rowLabel}}]" must position exactly ONE row (its selectors.yaml entry is relative to the table) — if this selector describes what EVERY row looks like, declare it as `rows:` on the TABLE entry instead').toHaveCount(1);
+await expect(tableRow).not.toContainText('{{escapeQuotes filterValue}}'); }
+{{~else}}
+{{!-- Row universe: `rows:` on the table entry (what EVERY row matches — a <div> list has no
+     role=row, so the ARIA chain below is 0 elements and this assertion would be vacuous, issue #688)
+     wins; otherwise the historical ARIA chain, byte-identical. --}}
 {{~#if isGiven}}
-await {{> locator}}.getByRole('row', { includeHidden: true }).filter({ hasText: '{{escapeQuotes filterValue}}' }).filter({ visible: true }).waitFor({ state: 'hidden' });
+await {{> locator}}{{#if rows}}.locator('{{escapeQuotes rows}}'){{else}}.getByRole('row', { includeHidden: true }){{/if}}.filter({ hasText: '{{escapeQuotes filterValue}}' }).filter({ visible: true }).waitFor({ state: 'hidden' });
 {{~else}}
-await expect({{> locator}}.getByRole('row', { includeHidden: true }).filter({ hasText: '{{escapeQuotes filterValue}}' }).filter({ visible: true })).toHaveCount(0);
+await expect({{> locator}}{{#if rows}}.locator('{{escapeQuotes rows}}'){{else}}.getByRole('row', { includeHidden: true }){{/if}}.filter({ hasText: '{{escapeQuotes filterValue}}' }).filter({ visible: true })).toHaveCount(0);
+{{~/if}}
 {{~/if}}

package/dist/generators/test-generator/adapters/playwright/templates/steps/partials/locator-base.hbs

@@ -1,22 +1,22 @@
 {{#if parentLocator}}{{parentLocator}}.{{#switch strategy~}}
-  {{~#case 'testid'}}getByTestId('{{value}}'){{/case~}}
+  {{~#case 'testid'}}getByTestId('{{escapeQuotes value}}'){{/case~}}
   {{~#case 'role'}}{{#if name}}getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }){{else}}getByRole('{{role}}'{{#if exact}}, { exact: true }{{/if}}){{/if}}{{/case~}}
   {{~#case 'placeholder'}}getByPlaceholder('{{escapeQuotes value}}'{{#if exact}}, { exact: true }{{/if}}){{/case~}}
   {{~#case 'label'}}getByLabel('{{escapeQuotes value}}'{{#if exact}}, { exact: true }{{/if}}){{/case~}}
   {{~#case 'text'}}getByText('{{escapeQuotes value}}'{{#if exact}}, { exact: true }{{/if}}){{/case~}}
-  {{~#case 'locator'}}locator('{{value}}'){{/case~}}
-  {{~#case 'id'}}locator('{{value}}'){{/case~}}
-  {{~#default}}locator('{{value}}'){{/default~}}
+  {{~#case 'locator'}}locator('{{escapeQuotes value}}'){{/case~}}
+  {{~#case 'id'}}locator('{{escapeQuotes value}}'){{/case~}}
+  {{~#default}}locator('{{escapeQuotes value}}'){{/default~}}
 {{~/switch}}{{else~}}
 {{#if scope}}{{#if (eq scope "dialog")}}{{pageRoot}}.{{else}}{{pageRoot}}.getByLabel('{{escapeQuotes scope}}').{{/if}}{{#switch strategy~}}
-  {{~#case 'testid'}}getByTestId('{{value}}'){{/case~}}
+  {{~#case 'testid'}}getByTestId('{{escapeQuotes value}}'){{/case~}}
   {{~#case 'role'}}{{#if name}}getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }){{else}}getByRole('{{role}}'{{#if exact}}, { exact: true }{{/if}}){{/if}}{{/case~}}
   {{~#case 'placeholder'}}getByPlaceholder('{{escapeQuotes value}}'{{#if exact}}, { exact: true }{{/if}}){{/case~}}
   {{~#case 'label'}}getByLabel('{{escapeQuotes value}}'{{#if exact}}, { exact: true }{{/if}}){{/case~}}
   {{~#case 'text'}}getByText('{{escapeQuotes value}}'{{#if exact}}, { exact: true }{{/if}}){{/case~}}
-  {{~#case 'locator'}}locator('{{value}}'){{/case~}}
-  {{~#case 'id'}}locator('{{value}}'){{/case~}}
-  {{~#default}}locator('{{value}}'){{/default~}}
+  {{~#case 'locator'}}locator('{{escapeQuotes value}}'){{/case~}}
+  {{~#case 'id'}}locator('{{escapeQuotes value}}'){{/case~}}
+  {{~#default}}locator('{{escapeQuotes value}}'){{/default~}}
 {{~/switch}}{{else}}{{#switch strategy~}}
   {{~#case 'testid'}}{{> locator-strategies/testid}}{{/case~}}
   {{~#case 'role'}}{{> locator-strategies/role}}{{/case~}}

package/dist/generators/test-generator/adapters/playwright/templates/steps/partials/locator-strategies/id.hbs

@@ -1 +1 @@
-page.locator('{{selector}}')
+page.locator('{{escapeQuotes selector}}')

package/dist/generators/test-generator/adapters/playwright/templates/steps/partials/locator-strategies/testid.hbs

@@ -1 +1 @@
-{{pageRoot}}.getByTestId('{{value}}')
+{{pageRoot}}.getByTestId('{{escapeQuotes value}}')