@sun-asterisk/sungen 3.2.28 → 3.2.30

package/dist/generators/test-generator/adapters/playwright/templates/steps/actions/click-at-position-action.hbs

@@ -0,0 +1,23 @@
+{{#if clickPosition}}
+{{!-- Positioned click: coordinates are relative to the element's PADDING box (clientWidth/Height),
+      the same box `click({ position })` measures from — boundingBox() would be the border box and
+      drift by the border width. Read once, checked, then clicked, so an offset past the real size
+      fails naming that size instead of landing on a neighbour. --}}
+{ const el = {{> locator}};
+const __label = {{{label}}};
+await el.waitFor({ state: 'visible' }); // the box is read once — not before the element has laid out
+{{!-- padding box = bounding rect minus borders: clientWidth/Height would read 0 on an inline element (a link). --}}
+const box = await el.evaluate((n) => {
+  const r = n.getBoundingClientRect(); const cs = getComputedStyle(n); const b = (p: string) => parseFloat(cs.getPropertyValue(p)) || 0;
+  return { w: r.width - b('border-left-width') - b('border-right-width'), h: r.height - b('border-top-width') - b('border-bottom-width') };
+});
+if (box.w <= 0 || box.h <= 0) throw new Error('[sungen] ' + __label + ': element has no layout box (hidden, detached or zero-sized)');
+const x = {{#if (eq clickPosition.xUnit '%')}}box.w * {{clickPosition.xRatio}}{{else}}{{clickPosition.x}}{{/if}};
+const y = {{#if (eq clickPosition.yUnit '%')}}box.h * {{clickPosition.yRatio}}{{else}}{{clickPosition.y}}{{/if}};
+if (x >= box.w || y >= box.h) throw new Error('[sungen] ' + __label + ': click position (' + Math.round(x) + ', ' + Math.round(y) + ') lies outside the element box ' + Math.round(box.w) + 'x' + Math.round(box.h));
+await el.click({ position: { x, y } });
+}
+{{else}}
+{{!-- Unreachable through the compiler (SG-E026 replaces the step) — kept so a stray render fails loud. --}}
+throw new Error('[sungen] click position could not be parsed');
+{{/if}}

package/dist/generators/test-generator/adapters/playwright/templates/steps/actions/expand-action.hbs

@@ -5,20 +5,20 @@ await el.click();
 {{else if expanded.class}}
 {{!-- State signalled by a class on a descendant (e.g. an accordion chevron icon). --}}
 {{#if (eq direction 'expand')}}
-if (!(await el.locator('.{{expanded.class}}').count())) { await el.click(); }
-await expect(el.locator('.{{expanded.class}}')).toBeVisible();
+if (!(await el.locator('.{{escapeQuotes expanded.class}}').count())) { await el.click(); }
+await expect(el.locator('.{{escapeQuotes expanded.class}}')).toBeVisible();
 {{else}}
-if (await el.locator('.{{expanded.class}}').count()) { await el.click(); }
-await expect(el.locator('.{{expanded.class}}')).toHaveCount(0);
+if (await el.locator('.{{escapeQuotes expanded.class}}').count()) { await el.click(); }
+await expect(el.locator('.{{escapeQuotes expanded.class}}')).toHaveCount(0);
 {{/if}}
 {{else if expanded.attribute}}
 {{!-- Explicit boolean attribute ('true'/'false'), custom or aria-expanded. --}}
 {{#if (eq direction 'expand')}}
-if (await el.getAttribute('{{expanded.attribute}}') !== 'true') { await el.click(); }
-await expect(el).toHaveAttribute('{{expanded.attribute}}', 'true');
+if (await el.getAttribute('{{escapeQuotes expanded.attribute}}') !== 'true') { await el.click(); }
+await expect(el).toHaveAttribute('{{escapeQuotes expanded.attribute}}', 'true');
 {{else}}
-if (await el.getAttribute('{{expanded.attribute}}') !== 'false') { await el.click(); }
-await expect(el).toHaveAttribute('{{expanded.attribute}}', 'false');
+if (await el.getAttribute('{{escapeQuotes expanded.attribute}}') !== 'false') { await el.click(); }
+await expect(el).toHaveAttribute('{{escapeQuotes expanded.attribute}}', 'false');
 {{/if}}
 {{else}}
 {{!-- Default (zero-config): drive by aria-expanded, but assert it ONLY when the element exposes it. --}}

package/dist/generators/test-generator/adapters/playwright/templates/steps/actions/fill-editor-action.hbs

@@ -1,3 +1,8 @@
-const editorLocator = {{> locator}};
+{{!-- Block-scoped, like the appium counterpart: `const editorLocator` at statement level made a
+      SECOND editor fill in the same scenario a duplicate declaration, and Playwright fails the
+      WHOLE file with one SyntaxError — every scenario in it never runs. Unreachable while
+      `contenteditable:` was dead and nothing set `inputMethod` (#712); a form with two rich-text
+      fields hits it immediately once the field works. SG-E022 is the gate that caught it. --}}
+{ const editorLocator = {{> locator}};
 await editorLocator.click();
-await editorLocator.pressSequentially('{{escapeQuotes fillValue}}');
+await editorLocator.pressSequentially('{{escapeQuotes fillValue}}'); }

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

@@ -1,2 +1,7 @@
-{ const tableRow = {{> locator}}.getByRole('row').nth({{{rowIndexExpr}}});
-await {{{actionLocator}}}.{{action}}(); }
+{{!-- 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}}});
+{{!-- `check`/`uncheck` must not emit a bare .check() here either (#675) — the row control is just
+     as likely to be an aria-pressed toggle as a standalone one. `click` stays a plain click. --}}
+{{#if (eq action 'click')}}await {{{actionLocator}}}.click();{{else}}await setCheckedState({{{actionLocator}}}, {{#if (eq action 'check')}}true{{else}}false{{/if}}, '{{escapeQuotes elementName}}'{{> checked-opts cls=actionCheckedClass attr=actionCheckedAttribute}});{{/if}} }

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

@@ -1,2 +1,7 @@
-{ const tableRow = {{> locator}}.getByRole('row').filter({ hasText: '{{escapeQuotes filterValue}}' });
-await {{{actionLocator}}}.{{action}}(); }
+{{!-- 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}}' });
+{{!-- `check`/`uncheck` must not emit a bare .check() here either (#675) — the row control is just
+     as likely to be an aria-pressed toggle as a standalone one. `click` stays a plain click. --}}
+{{#if (eq action 'click')}}await {{{actionLocator}}}.click();{{else}}await setCheckedState({{{actionLocator}}}, {{#if (eq action 'check')}}true{{else}}false{{/if}}, '{{escapeQuotes elementName}}'{{> checked-opts cls=actionCheckedClass attr=actionCheckedAttribute}});{{/if}} }

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

@@ -0,0 +1,5 @@
+{{!-- The paginator is located from `page`, NOT from inside the table locator: antd/MUI/Radix all
+      render the control as a SIBLING of the <table>, so scoping it to the table would resolve 0
+      elements (issue #712). No implicit wait — the round-trip belongs to
+      `User wait for [X] table to refresh`. A disabled control fails loud: click waits for enabled. --}}
+await page.locator('{{escapeQuotes paginationSelector}}').click();

package/dist/generators/test-generator/adapters/playwright/templates/steps/actions/uncheck-action.hbs

@@ -1 +1,3 @@
-await {{> locator}}.uncheck();
+{{!-- Inverse of check-action.hbs — see there and specs/checked-state.ts for why the control is
+      inspected at run time rather than driven by a bare .uncheck() (#675). --}}
+await setCheckedState({{> locator}}, false, '{{escapeQuotes selectorRef}}'{{> checked-opts cls=checkedClass attr=checkedAttribute}});

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

@@ -1 +1 @@
-await page.getByText('{{escapeQuotes selectorValue}}').filter({ hasText: {{> has-text-value dataValue=dataValue exact=exact legacyTail=(eq selectorValue "")}} }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}.{{action}}();
+await page.getByText('{{escapeQuotes selectorValue}}').filter({ hasText: {{> has-text-value dataValue=dataValue exact=exact legacyTail=(eq selectorValue "")}} }){{> locator-nth}}.{{action}}();

package/dist/generators/test-generator/adapters/playwright/templates/steps/actions/wait-for-page.hbs

@@ -1 +1,18 @@
-await page.waitForURL(/{{pathRegex}}/);
+{{!-- `wait for [T] page` is a WAIT, not an oracle — but a bare `page.waitForURL(/…/)` resolves on
+      the FIRST URL sample that matches, which for a client-side router is the URL it pushed before
+      the route guard had answered. Every step after the wait then ran against a page mid-redirect,
+      so their failures pointed at the wrong thing (#677). The unanchored regex also matched a
+      neighbour: `/vi/search` resolved on `/vi/search-results`.
+
+      `waitForPage` is the third caller of the settle-and-hold core in specs/url-assert.ts, after
+      the `see [T] page` / `is on [T] page` pair #662 routed there (partials/url-assertion.hbs):
+      arrive, let the page settle, HOLD the URL, and fail naming where the user actually ended up.
+
+      Where it deliberately DIFFERS from the Then-side `expectPage`: the query is not constrained.
+      A bare page ASSERTION means a bare page; a wait for a results page legitimately lands on
+      `?q=…`, and reddening there would assert something the step never said. The helper's docstring
+      carries the full argument.
+
+      `hostname` rides separately and only for an absolute page value: it is read off `u.hostname`,
+      which never appears in `u.pathname`, so one concatenated regex cannot carry both. --}}
+await waitForPage(page, { pathname: /^{{pathRegex}}\/?$/{{#if hostRegex}}, hostname: /^{{hostRegex}}$/{{/if}} });

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

@@ -1,7 +1,35 @@
-{
-    const __items_{{stepCounter}} = await {{> locator}}.allInnerTexts();
-    expect(__items_{{stepCounter}}.length, 'Expected at least one [{{selectorRef}}]').toBeGreaterThan(0);
+{{!-- All-contain: the assertion has to SURVIVE whatever async action precedes it.
+
+      `locator.allInnerTexts()` is a one-shot DOM read — no auto-wait, no retry — and wrapping its
+      RESULT in expect() adds no waiting either, because by then it is a plain array of strings.
+      Playwright's auto-waiting only applies when the LOCATOR is handed to expect(). Placed after a
+      search click or a filter change, the old shape asserted against the PRE-action DOM and failed
+      deterministically, naming real-looking values that read as a product defect rather than as a
+      generator bug (#672). A `toBeGreaterThan(0)` guard does not help: the stale list is non-empty,
+      so the length check passes and the CONTENT check is what races.
+
+      Playwright has no locator-level "every match contains X" — `filter({ hasNotText })` matches
+      case-insensitively on normalized whitespace, which is not the `toContain` semantics this step
+      promises — so the wait is expressed with expect.poll, which retries for `expect.timeout` from
+      playwright.config.ts. Two details carry the fix:
+
+      - The loop asserts on the snapshot the poll SETTLED on. Re-reading the locator afterwards
+        reopens the very race being closed — a late render pass between the poll settling and the
+        re-read swapped the rows out again, which is how the reporter's first patch passed in
+        isolation and failed in the full suite.
+      - The poll's own rejection is swallowed: on its own it can only report
+        `toBe(true) received false`. The per-item expect() below names expected vs received, which
+        is the message a QA has to act on. --~}}{
+    const __expected_{{stepCounter}} = '{{escapeQuotes expectedText}}';
+    let __items_{{stepCounter}}: string[] = [];
+    try {
+      await expect.poll(async () => {
+        __items_{{stepCounter}} = await {{> locator}}.allInnerTexts();
+        return __items_{{stepCounter}}.length > 0 && __items_{{stepCounter}}.every((__s) => __s.includes(__expected_{{stepCounter}}));
+      }).toBe(true);
+    } catch { /* the assertions below report expected-vs-received on the settled snapshot */ }
+    expect(__items_{{stepCounter}}.length, 'Expected at least one [{{escapeQuotes selectorRef}}]').toBeGreaterThan(0);
     for (const __t_{{stepCounter}} of __items_{{stepCounter}}) {
-      expect(__t_{{stepCounter}}).toContain('{{expectedText}}');
+      expect(__t_{{stepCounter}}).toContain(__expected_{{stepCounter}});
     }
   }

package/dist/generators/test-generator/adapters/playwright/templates/steps/assertions/all-contain-element.hbs

@@ -1,7 +1,10 @@
-{
+{{!-- The count() below is a one-shot read, same class as #672: after a search or filter click the
+      container list may not have rendered yet, and a bare `toBeGreaterThan(0)` on a stale 0 fails
+      instantly with no retry. `not.toHaveCount(0)` IS locator-based, so it auto-waits for the list
+      to appear; count() then reads a list that is already known to exist. --~}}{
     const {{containerVar}} = {{> locator}};
+    await expect({{containerVar}}, 'Expected at least one [{{escapeQuotes selectorRef}}]').not.toHaveCount(0);
     const __n_{{stepCounter}} = await {{containerVar}}.count();
-    expect(__n_{{stepCounter}}, 'Expected at least one [{{escapeQuotes selectorRef}}]').toBeGreaterThan(0);
     for (let {{containerIndexVar}} = 0; {{containerIndexVar}} < __n_{{stepCounter}}; {{containerIndexVar}}++) {
       await expect({{#with child}}{{> locator parentLocator=(add (add (add ../containerVar ".nth(") ../containerIndexVar) ")")}}{{/with}}, 'Every [{{escapeQuotes selectorRef}}] must contain [{{escapeQuotes childRef}}]').toBeVisible();
     }

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

@@ -1 +1,5 @@
-await expect({{> locator}}).toBeChecked();
+{{!-- `expect(locator).toBeChecked()` throws the same "Not a checkbox or radio button" (#675) on a
+      `<button aria-pressed>` as the action side did, so driving a toggle worked while ASSERTING it
+      did not. expectCheckedState reads state the same way setCheckedState drives it — an action and
+      the assertion after it can never disagree about what the control reports. --}}
+await expectCheckedState({{> locator}}, true, '{{escapeQuotes selectorRef}}'{{> checked-opts cls=checkedClass attr=checkedAttribute}});

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

@@ -1,3 +1,14 @@
-await expect({{> locator}}.getByRole('columnheader', { name: '{{columnName}}', includeHidden: true })).toBeVisible();
+{{!-- Rooted at `page`, like the two lines below it. This line used to render `{{> locator}}` from
+      step data that carries no strategy/value at all, so it emitted `page.locator('')` — an empty
+      CSS selector, which Playwright refuses to parse. The step could not run.
+
+      Block-scoped: the column-index const is named after the [Reference], so two steps naming the
+      same column — or two non-ASCII references, which sanitize down to the same identifier — would
+      redeclare it in one scenario and the spec would not compile. The braces make each step's
+      binding its own.
+
+      Not dialog/frame scoped: this step form carries no selector entry, so there is nothing for
+      `pageRoot` to read. A column assertion inside a modal needs the row-scoped form. --~}}{ await expect(page.getByRole('columnheader', { name: '{{columnName}}', includeHidden: true })).toBeVisible();
 const {{columnIndexVar}} = (await page.getByRole('columnheader', { includeHidden: true }).filter({ visible: true }).allTextContents()).findIndex(h => h.includes('{{columnName}}'));
-await expect(page.getByRole('row', { includeHidden: true }).filter({ visible: true }).nth({{rowNth}}).getByRole('cell', { includeHidden: true }).filter({ visible: true }).nth({{columnIndexVar}})).toHaveText('{{dataValue}}');
+expect({{columnIndexVar}}, 'Column "{{escapeQuotes columnName}}" is not among the visible headers').toBeGreaterThanOrEqual(0);
+await expect(page.getByRole('row', { includeHidden: true }).filter({ visible: true }).nth({{rowNth}}).getByRole('cell', { includeHidden: true }).filter({ visible: true }).nth({{columnIndexVar}})).toHaveText('{{dataValue}}'); }

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

@@ -1 +1 @@
-await expect({{> locator}}).toContainText('{{expectedText}}');
+await expect({{> locator}}).toContainText('{{escapeQuotes expectedText}}');

package/dist/generators/test-generator/adapters/playwright/templates/steps/assertions/disabled-with-role-variable-assertion.hbs

@@ -1,5 +1,5 @@
 {{#if name}}
-await expect(page.getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeDisabled();
+await expect(page.getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{> locator-nth}}).toBeDisabled();
 {{else}}
-await expect(page.getByRole('{{role}}'{{#if exact}}, { exact: true }{{/if}}).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeDisabled();
+await expect(page.getByRole('{{role}}'{{#if exact}}, { exact: true }{{/if}}).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{> locator-nth}}).toBeDisabled();
 {{/if}}

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

@@ -1 +1 @@
-await expect(page.getByText('{{escapeQuotes selectorRef}}').filter({ hasText: {{#if (eq selectorValue "")}}/.*{{escapeRegex dataValue}}$/{{else}}'{{escapeQuotes dataValue}}'{{/if}} }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeDisabled();
+await expect(page.getByText('{{escapeQuotes selectorRef}}').filter({ hasText: {{#if (eq selectorValue "")}}/.*{{escapeRegex dataValue}}$/{{else}}'{{escapeQuotes dataValue}}'{{/if}} }){{> locator-nth}}).toBeDisabled();

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

@@ -1 +1 @@
-await expect({{> locator}}).toHaveText('{{expectedText}}');
+await expect({{> locator}}).toHaveText('{{escapeQuotes expectedText}}');

package/dist/generators/test-generator/adapters/playwright/templates/steps/assertions/hidden-with-role-variable-assertion.hbs

@@ -1,5 +1,5 @@
 {{#if name}}
-await expect(page.getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeHidden();
+await expect(page.getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{> locator-nth}}).toBeHidden();
 {{else}}
-await expect(page.getByRole('{{role}}'{{#if exact}}, { exact: true }{{/if}}).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeHidden();
+await expect(page.getByRole('{{role}}'{{#if exact}}, { exact: true }{{/if}}).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{> locator-nth}}).toBeHidden();
 {{/if}}

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

@@ -1 +1 @@
-await expect(page.getByText('{{escapeQuotes selectorRef}}').filter({ hasText: {{#if (eq selectorValue "")}}/.*{{escapeRegex dataValue}}$/{{else}}'{{escapeQuotes dataValue}}'{{/if}} }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeHidden();
+await expect(page.getByText('{{escapeQuotes selectorRef}}').filter({ hasText: {{#if (eq selectorValue "")}}/.*{{escapeRegex dataValue}}$/{{else}}'{{escapeQuotes dataValue}}'{{/if}} }){{> locator-nth}}).toBeHidden();

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

@@ -1 +1,2 @@
-await expect({{> locator}}).toBeChecked({ checked: false });
+{{!-- Inverse of checked-assertion.hbs — see there (#675). --}}
+await expectCheckedState({{> locator}}, false, '{{escapeQuotes selectorRef}}'{{> checked-opts cls=checkedClass attr=checkedAttribute}});

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

@@ -1,7 +1,16 @@
-{{#if (or (eq role 'checkbox') (eq role 'radio') (eq role 'switch'))}}
+{{#if (or checkedClass checkedAttribute)}}
+{{!-- A DECLARED signal (issue #730) short-circuits the whole ARIA probe below: the author
+     reached for `selected:`/`checked:` precisely because this component publishes no aria-selected,
+     no aria-current and no checked state — a Tailwind card/tab picker. Probing for them first would
+     just spend the budget confirming they are absent. --}}
+await expectCheckedState({{> locator}}, true, '{{escapeQuotes selectorRef}}'{{> checked-opts cls=checkedClass attr=checkedAttribute}});
+{{else if (or (eq role 'checkbox') (eq role 'radio') (eq role 'switch'))}}
 {{!-- Form control: "selected" is the checked state, never aria-selected. A native checkbox/radio
      exposes no aria-selected attribute — its state lives in the `checked` property. --}}
-await expect({{> locator}}).toBeChecked();
+{{!-- "selected" on a form control IS the checked state — routed through expectCheckedState so an
+     aria-pressed toggle does not throw "Not a checkbox or radio button" (#675), and so the selector
+     that declares role: switch while the element publishes aria-pressed still reads correctly. --}}
+await expectCheckedState({{> locator}}, true, '{{escapeQuotes selectorRef}}');
 {{else}}
 {{!-- Other/unknown role (tab, option, or a text/testid/css selector): probe at runtime. An ARIA
      single-select widget exposes aria-selected → assert it; anything without it falls back to the
@@ -9,8 +18,17 @@ 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();
+{{!-- Last resort, same as the -with-filter twin: neither aria-selected nor aria-current, so read the
+     checked/pressed state through the #675-safe path rather than toBeChecked(). --}}
+await expectCheckedState(el, true, '{{escapeQuotes selectorRef}}');
 }
 }
 {{/if}}

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

@@ -1,13 +1,31 @@
 {{!-- Filter contract shared with the visible/hidden/contains siblings (has-text-value):
       substring by default, `exact: true` in the selector YAML anchors. --}}
 {{#if selectedProbe}}
+{{#if (or checkedClass checkedAttribute)}}
+{{!-- Declared class signal (issue #730) — short-circuits the ARIA probe, exactly as the unfiltered
+     twin selected-assertion.hbs does. Without this branch the filtered form stayed broken while the
+     unfiltered one worked, which is the inconsistency #730's second field report ran into. --}}
+await expectCheckedState({{> locator-base}}.filter({ hasText: {{> has-text-value dataValue=dataValue exact=exact}} }){{> locator-nth}}, true, '{{escapeQuotes selectorRef}}'{{> checked-opts cls=checkedClass attr=checkedAttribute}});
+{{else}}
 { 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();
+{{!-- Last resort: the element publishes neither aria-selected nor aria-current. toBeChecked() used
+     to sit here and threw "Not a checkbox or radio button" on an aria-pressed toggle (#675). --}}
+await expectCheckedState(el, true, '{{escapeQuotes selectorRef}}');
 }
 }
+{{/if}}
+{{else}}{{#if isCheckedAssertion}}
+await expectCheckedState({{> locator-base}}.filter({ hasText: {{> has-text-value dataValue=dataValue exact=exact}} }){{> locator-nth}}, {{checkedWant}}, '{{escapeQuotes selectorRef}}'{{> checked-opts cls=checkedClass attr=checkedAttribute}});
 {{else}}
 await expect({{> locator-base}}.filter({ hasText: {{> has-text-value dataValue=dataValue exact=exact}} }){{> locator-nth}}).{{{stateAssertion}}};
-{{/if}}
+{{/if}}{{/if}}

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/assertions/visible-with-role-variable-assertion.hbs

@@ -1,9 +1,9 @@
 {{#if name}}
 {{#if dataValue}}
-await expect(page.getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeVisible();
+await expect(page.getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{> locator-nth}}).toBeVisible();
 {{else}}
-await expect(page.getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeVisible();
+await expect(page.getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }){{> locator-nth}}).toBeVisible();
 {{/if}}
 {{else}}
-await expect(page.getByRole('{{role}}'{{#if exact}}, { exact: true }{{/if}}).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeVisible();
+await expect(page.getByRole('{{role}}'{{#if exact}}, { exact: true }{{/if}}).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{> locator-nth}}).toBeVisible();
 {{/if}}

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

@@ -1 +1 @@
-await expect(page.getByText('{{escapeQuotes value}}'{{#if exact}}, { exact: true }{{/if}}){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeVisible();
+await expect(page.getByText('{{escapeQuotes value}}'{{#if exact}}, { exact: true }{{/if}}){{> locator-nth}}).toBeVisible();

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

@@ -1 +1 @@
-await expect(page.getByText('{{escapeQuotes selectorValue}}').filter({ hasText: /.*{{escapeRegex dataValue}}$/ }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}).toBeVisible();
+await expect(page.getByText('{{escapeQuotes selectorValue}}').filter({ hasText: /.*{{escapeRegex dataValue}}$/ }){{> locator-nth}}).toBeVisible();

package/dist/generators/test-generator/adapters/playwright/templates/steps/partials/checked-opts.hbs

@@ -0,0 +1,8 @@
+{{!-- The `checked:`/`selected:` signal from selectors.yaml, appended as the options argument of
+      setCheckedState/expectCheckedState (issue #730). One partial so all nine call sites carry it
+      identically, and so a site that declares nothing stays BYTE-IDENTICAL to before — the whole
+      argument disappears rather than becoming an empty object. Takes `cls`/`attr` so a caller can
+      pass a signal resolved from a DIFFERENT entry than the one it spread (table-action-in-row
+      spreads the TABLE but acts on the row's own control). A selector declares one or the other;
+      the resolver refuses both, so the two branches never collide here. --~}}
+{{~#if cls}}, { checkedClass: '{{escapeQuotes cls}}' }{{else if attr}}, { checkedAttribute: '{{escapeQuotes attr}}' }{{/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-nth.hbs

@@ -1 +1 @@
-{{#if (gte nth 0)}}.nth({{nth}}){{/if}}
+{{#if (gte nth 0)}}.nth({{nth}}){{else if (eq nth -1)}}.last(){{/if}}

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}}')

package/dist/generators/test-generator/adapters/playwright/templates/steps/partials/locator.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}}{{> locator-nth}}{{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}}{{> locator-nth}}{{else}}{{#switch strategy~}}
   {{~#case 'testid'}}{{> locator-strategies/testid}}{{/case~}}
   {{~#case 'role'}}{{> locator-strategies/role}}{{/case~}}

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

@@ -98,6 +98,16 @@ test.describe.serial('{{featureName}}', () => {
   });
 {{/if}}
 
+{{#if beforeAll}}
+{{beforeAll}}
+
+{{/if}}
+{{#if afterAll}}
+  // User @afterAll runs BEFORE the built-in close below (Playwright runs afterAll hooks in registration
+  // order), so the shared page is still open for its cleanup steps.
+{{afterAll}}
+
+{{/if}}
   test.afterAll(async () => {
     await page.close();
     await context.close();
@@ -112,18 +122,10 @@ test.describe.serial('{{featureName}}', () => {
 {{#if screenshotOnFailure}}
   test.use({ screenshotOnFailure: true });
 
-{{/if}}
-{{#if beforeAll}}
-{{beforeAll}}
-
 {{/if}}
 {{#if afterEach}}
 {{afterEach}}
 
-{{/if}}
-{{#if afterAll}}
-{{afterAll}}
-
 {{/if}}
 {{#if authGroups}}
 {{#each authGroups}}