@sun-asterisk/sungen 3.2.32 → 3.2.34

package/dist/generators/test-generator/adapters/adapter-interface.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"adapter-interface.d.ts","sourceRoot":"","sources":["../../../../src/generators/test-generator/adapters/adapter-interface.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,MAAM,WAAW,SAAS;IACxB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,SAAS,EAAE,MAAM,EAAE,CAAC;CACrB;AAED;;;;;;GAMG;AACH,MAAM,WAAW,QAAQ;IACvB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,sGAAsG;AACtG,MAAM,WAAW,cAAc;IAC7B,KAAK,EAAE,QAAQ,EAAE,CAAC;IAClB,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AAED,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,EAAE,MAAM,CAAC;IACpB,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,mBAAmB,CAAC,EAAE,OAAO,CAAC;IAC9B,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,OAAO,CAAC,EAAE;QAAE,OAAO,CAAC,EAAE,OAAO,CAAC;QAAC,KAAK,CAAC,EAAE,OAAO,CAAC;QAAC,MAAM,CAAC,EAAE,OAAO,CAAC;QAAC,OAAO,CAAC,EAAE,OAAO,CAAA;KAAE,CAAC;IACtF,eAAe,CAAC,EAAE,KAAK,CAAC;QAAE,OAAO,CAAC,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAC5D,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,UAAU,CAAC,EAAE,SAAS,EAAE,CAAC;IACzB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,SAAS,CAAC,EAAE,MAAM,CAAC;IAMnB,kBAAkB,CAAC,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,OAAO,CAAA;KAAE,CAAC,CAAC;CAC5E;AAED,MAAM,WAAW,YAAY;IAC3B,YAAY,EAAE,MAAM,CAAC;IACrB,KAAK,EAAE,KAAK,CAAC;QAAE,OAAO,CAAC,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IACjD,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,gBAAgB;IAC/B,YAAY,EAAE,MAAM,CAAC;IACrB,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;CAC3B;AAED,MAAM,WAAW,iBAAiB;IAChC,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;IACd,GAAG,CAAC,EAAE,MAAM,CAAC;CACd;AAED;;;GAGG;AAEH,iGAAiG;AACjG,MAAM,WAAW,oBAAoB;IACnC,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,kBAAkB,CAAC,EAAE,OAAO,CAAC;IAC7B,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B,mBAAmB,CAAC,EAAE,MAAM,CAAC;CAC9B;AAED,MAAM,WAAW,oBAAoB;IACnC,QAAQ,CAAC,aAAa,EAAE,MAAM,CAAC;IAC/B,QAAQ,CAAC,aAAa,EAAE,MAAM,CAAC;IAC/B,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IAG9B,cAAc,CAAC,IAAI,EAAE,YAAY,GAAG,MAAM,CAAC;IAC3C,cAAc,CAAC,IAAI,EAAE,YAAY,GAAG,MAAM,CAAC;IAC3C,aAAa,CAAC,OAAO,CAAC,EAAE,oBAAoB,GAAG,MAAM,CAAC;IACtD,gBAAgB,CAAC,IAAI,EAAE;QAAE,KAAK,EAAE,KAAK,CAAC;YAAE,OAAO,CAAC,EAAE,MAAM,CAAC;YAAC,IAAI,EAAE,MAAM,CAAA;SAAE,CAAC,CAAA;KAAE,GAAG,MAAM,CAAC;IACrF,eAAe,CAAC,IAAI,EAAE,cAAc,GAAG,MAAM,CAAC;IAC9C,eAAe,CAAC,IAAI,EAAE,cAAc,GAAG,MAAM,CAAC;IAC9C,cAAc,CAAC,IAAI,EAAE,cAAc,GAAG,MAAM,CAAC;IAG7C,UAAU,CAAC,YAAY,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,GAAG,MAAM,CAAC;CACrD"}
+{"version":3,"file":"adapter-interface.d.ts","sourceRoot":"","sources":["../../../../src/generators/test-generator/adapters/adapter-interface.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,MAAM,WAAW,SAAS;IACxB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB;4FACwF;IACxF,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED;;;;;;GAMG;AACH,MAAM,WAAW,QAAQ;IACvB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,sGAAsG;AACtG,MAAM,WAAW,cAAc;IAC7B,KAAK,EAAE,QAAQ,EAAE,CAAC;IAClB,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AAED,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,EAAE,MAAM,CAAC;IACpB,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,mBAAmB,CAAC,EAAE,OAAO,CAAC;IAC9B,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,OAAO,CAAC,EAAE;QAAE,OAAO,CAAC,EAAE,OAAO,CAAC;QAAC,KAAK,CAAC,EAAE,OAAO,CAAC;QAAC,MAAM,CAAC,EAAE,OAAO,CAAC;QAAC,OAAO,CAAC,EAAE,OAAO,CAAA;KAAE,CAAC;IACtF,eAAe,CAAC,EAAE,KAAK,CAAC;QAAE,OAAO,CAAC,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAC5D,SAAS,EAAE,MAAM,EAAE,CAAC;IACpB,UAAU,CAAC,EAAE,SAAS,EAAE,CAAC;IACzB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,SAAS,CAAC,EAAE,MAAM,CAAC;IAMnB,kBAAkB,CAAC,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,OAAO,CAAA;KAAE,CAAC,CAAC;CAC5E;AAED,MAAM,WAAW,YAAY;IAC3B,YAAY,EAAE,MAAM,CAAC;IACrB,KAAK,EAAE,KAAK,CAAC;QAAE,OAAO,CAAC,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IACjD,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,MAAM,WAAW,gBAAgB;IAC/B,YAAY,EAAE,MAAM,CAAC;IACrB,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;CAC3B;AAED,MAAM,WAAW,iBAAiB;IAChC,OAAO,EAAE,MAAM,CAAC;IAChB,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;IACd,GAAG,CAAC,EAAE,MAAM,CAAC;CACd;AAED;;;GAGG;AAEH,iGAAiG;AACjG,MAAM,WAAW,oBAAoB;IACnC,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,kBAAkB,CAAC,EAAE,OAAO,CAAC;IAC7B,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,SAAS,CAAC,EAAE,OAAO,CAAC;IACpB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B,mBAAmB,CAAC,EAAE,MAAM,CAAC;CAC9B;AAED,MAAM,WAAW,oBAAoB;IACnC,QAAQ,CAAC,aAAa,EAAE,MAAM,CAAC;IAC/B,QAAQ,CAAC,aAAa,EAAE,MAAM,CAAC;IAC/B,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IAG9B,cAAc,CAAC,IAAI,EAAE,YAAY,GAAG,MAAM,CAAC;IAC3C,cAAc,CAAC,IAAI,EAAE,YAAY,GAAG,MAAM,CAAC;IAC3C,aAAa,CAAC,OAAO,CAAC,EAAE,oBAAoB,GAAG,MAAM,CAAC;IACtD,gBAAgB,CAAC,IAAI,EAAE;QAAE,KAAK,EAAE,KAAK,CAAC;YAAE,OAAO,CAAC,EAAE,MAAM,CAAC;YAAC,IAAI,EAAE,MAAM,CAAA;SAAE,CAAC,CAAA;KAAE,GAAG,MAAM,CAAC;IACrF,eAAe,CAAC,IAAI,EAAE,cAAc,GAAG,MAAM,CAAC;IAC9C,eAAe,CAAC,IAAI,EAAE,cAAc,GAAG,MAAM,CAAC;IAC9C,cAAc,CAAC,IAAI,EAAE,cAAc,GAAG,MAAM,CAAC;IAG7C,UAAU,CAAC,YAAY,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,GAAG,MAAM,CAAC;CACrD"}

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

@@ -2,8 +2,8 @@
 // Persistence oracles (dismissals, pins) hinge on what survives a reinstall.
 await (async () => {
   const __pkg = __resolveAppId();
-  const __caps = (driver.requestedCapabilities || {}) as Record<string, string>;
-  const __appPath = __caps['appium:app'] || __caps['app'];
+  // Same reader the reset tiers use — one place decides what "the session has a build file" means.
+  const __appPath = __appBuildPath();
   if (!__appPath) {
     throw new Error('reinstall the app: the session must install from a build file — set APP_APK (Android) / IOS_APP (iOS) so the appium:app capability is present.');
   }

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

@@ -9,12 +9,18 @@
       Flutter screen; UiSelector has no OR, hence the xpath. iOS scatters it across
       label/value, which the predicate already ORs.
 
+      SCOPE (#747): an `in [Parent] <type>` clause chains this under the parent element. The android
+      xpath must go RELATIVE (`.//`) to do so — a child xpath beginning with `//` is ABSOLUTE in
+      Appium even when chained, so keeping it would emit code that LOOKS scoped and searches the whole
+      screen, which is the same silent widening the scope clause exists to prevent. The iOS predicate
+      carries no such trap and chains as written.
+
       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}"`
-    : `//*[@text="${__expected}" or @content-desc="${__expected}"]`;
-  await expect($(__sel)).toBeDisplayed();
+    : `{{#if parentLocator}}.{{/if}}//*[@text="${__expected}" or @content-desc="${__expected}"]`;
+  await expect({{#if parentLocator}}{{parentLocator}}.$(__sel){{else}}$(__sel){{/if}}).toBeDisplayed();
 }

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

@@ -1,13 +1,17 @@
 {{!-- Role variable, visible (mobile best-effort): ARIA role has no native-mobile analog — reuse
       the role→accessibility-id mapping in appium-selector.hbs (`~name`) when a static name
       exists; the resolver already folds dataValue into name when there is none, so this is only
-      the fallback for a caller that omits both. --}}
+      the fallback for a caller that omits both.
+
+      SCOPE (#747): an `in [Parent] <type>` clause chains this under the parent element. Both
+      branches locate by accessibility id / predicate, neither of which carries the absolute-xpath
+      trap the label-value twin has to work around. --}}
 {{#if name}}
-await expect($('~{{escapeQuotes name}}')).toBeDisplayed();
+await expect({{#if parentLocator}}{{parentLocator}}.$('~{{escapeQuotes name}}'){{else}}$('~{{escapeQuotes name}}'){{/if}}).toBeDisplayed();
 {{else}}
 {
   const __expected = String('{{escapeQuotes dataValue}}').trim();
   const __sel = driver.isIOS ? `-ios predicate string:label == "${__expected}" OR value == "${__expected}"` : `android=new UiSelector().text("${__expected}")`;
-  await expect($(__sel)).toBeDisplayed();
+  await expect({{#if parentLocator}}{{parentLocator}}.$(__sel){{else}}$(__sel){{/if}}).toBeDisplayed();
 }
 {{/if}}

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

@@ -502,9 +502,36 @@ describe('{{featureName}}', () => {
   // lose the escalation between them — SG-W024 already reports that shape when tiers are involved.)
   let __escalateAfter: string | null = null;
 
+  // `@known-defect` marks a scenario that is EXPECTED to be red — it asserts a live defect. The
+  // scaffolded /sungen:run-test command tells authors exactly that ("expected-red scenarios stay red;
+  // never 'fix' them"), so the ladder honours it here: such a failure is the suite working as
+  // designed, not evidence of leftover state. Arming escalation off it would make the NEXT scenario
+  // pay a harder reset for nothing — on every run, forever, because the defect is known.
+  //
+  // Read off the TITLE because `@known-defect` is a pass-through tag: appium appends it there as the
+  // `[@a @b]` suffix (which is what keeps it greppable via `--grep`), and that is how the field
+  // report shows it. Anchored to that trailing group, so a scenario merely NAMED `… known-defect …`
+  // cannot exempt itself — the same reason __tierFor refuses to strip a trailing `[Selector]` ref.
+  //
+  // Matched as a whole TAG TOKEN, never as a substring. `@known-defect-list` is a DIFFERENT tag, and
+  // reading it as this one would silently stop the next scenario's real dirty-state failure from
+  // getting the escalated reset it needs — with no log line at all, because "never armed" is not the
+  // "capped" path and prints nothing. The `:<ref>` parameter form is accepted because that is what
+  // the rest of the vocabulary does (`@manual:M10`, `@reset:data`).
+  //
+  // Caveat, inherited rather than introduced: a @cases scenario builds its title at runtime as
+  // `name — <row label>`, so a row LABEL that itself ends in `[@…]` would be read as the tag group.
+  // Same trailing-bracket ambiguity __tierFor accepts for a `[Selector]` ref.
+  function __isExpectedFail(title: string): boolean {
+    const tagGroup = title.match(/ \[@[^\]]*\]$/);
+    if (!tagGroup) return false;
+    return tagGroup[0].slice(2, -1).split(/\s+/)
+      .some((t) => t === '@known-defect' || t.startsWith('@known-defect:'));
+  }
+
   // Registered BEFORE any user @afterEach block so a throwing user hook cannot skip the bookkeeping.
   afterEach(async function () {
-    __escalateAfter = this.currentTest?.state === 'failed' ? (this.currentTest?.title ?? '') : null;
+    __escalateAfter = this.currentTest?.state === 'failed' && !__isExpectedFail(this.currentTest?.title ?? '') ? (this.currentTest?.title ?? '') : null;
   });
 
 {{#if scenarioResetTiers}}
@@ -583,9 +610,31 @@ describe('{{featureName}}', () => {
     await driver.execute('mobile: clearApp', { appId: pkg });
   }
 
-  async function __reinstallApp(pkg: string): Promise<void> {
+  /** The app's build file, when the session was given one. Absent when the app was attached BY PACKAGE. */
+  function __appBuildPath(): string | undefined {
     const caps = (driver.requestedCapabilities || {}) as Record<string, string>;
-    const appPath = caps['appium:app'] || caps['app'];
+    return caps['appium:app'] || caps['app'];
+  }
+
+  /**
+   * The strongest tier THIS SESSION can actually carry out.
+   *
+   * Reinstalling needs the app's build file, and `data` on iOS IS a reinstall (no `mobile: clearApp`
+   * outside UiAutomator2). Attaching the app by package — the common local setup — leaves no build
+   * file, so those tiers cannot run at all.
+   *
+   * This exists so ESCALATION can cap itself. Escalation is a recovery the runner CHOSE; nothing
+   * asked for it. Escalating into a tier that throws turns one failed scenario into a dead spec file,
+   * because the throw lands in `beforeEach` and mocha then skips every remaining test in the file.
+   * A DECLARED tier is never capped — that is an unmet contract and must still throw by name.
+   */
+  function __performableTier(): string {
+    if (__appBuildPath()) return 'full';
+    return driver.isIOS ? 'soft' : 'data';
+  }
+
+  async function __reinstallApp(pkg: string): Promise<void> {
+    const appPath = __appBuildPath();
     if (!appPath) {
       throw new Error(
         `reset tier needs the app's build file to reinstall — set APP_APK (Android) / IOS_APP (iOS) so the appium:app capability is present, or drop to @reset:soft.`,
@@ -637,9 +686,24 @@ describe('{{featureName}}', () => {
     // Same title ⟹ this is a retry of the scenario that just failed: keep its declared tier so a real
     // failure fails both attempts (see __escalateAfter above).
     const __escalate = __escalateAfter !== null && __escalateAfter !== (this.currentTest?.title ?? '');
-    const __tier = __escalate ? __ESCALATE[__base] : __base;
+    // Escalation caps at what this session can perform, with the DECLARED tier as the FLOOR.
+    // The cap stops a tier NOBODY ASKED FOR from throwing in this hook and taking the rest of the
+    // spec file with it. The floor stops the cap from silently WEAKENING what the feature declared:
+    // a declared tier this session cannot perform still reaches __resetApp and still throws by name,
+    // because that one IS an unmet contract.
+    const __want = __escalate ? __ESCALATE[__base] : __base;
+    const __tier = __escalate
+      ? __TIER_ORDER[Math.max(
+          __TIER_ORDER.indexOf(__base),
+          Math.min(__TIER_ORDER.indexOf(__want), __TIER_ORDER.indexOf(__performableTier())),
+        )]
+      : __want;
     if (__tier !== __base) {
       console.log(`  \u2191 reset escalated ${__base} \u2192 ${__tier} (previous scenario failed: "${__escalateAfter}")`);
+    } else if (__want !== __base) {
+      // Said per occurrence, not once per run: this names the scenario that did NOT get the stronger
+      // reset it was owed, which is a fact about THIS scenario, not just about the session.
+      console.warn(`  ! reset could not escalate ${__base} \u2192 ${__want} — this session has no app build file to reinstall from (the app was attached by package). Staying at ${__base}; set APP_APK (Android) / IOS_APP (iOS) to enable the stronger reset.`);
     }
     await __resetApp(__tier);
 {{#if backgroundSteps}}

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

@@ -1,2 +1,2 @@
 await {{> locator}}.click();
-await page.getByText('{{selectValue}}', { exact: true }).first().click();
+{{> pick-option}}

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

@@ -9,7 +9,7 @@ if ((await el.evaluate((n) => n.tagName.toLowerCase())) === 'select') {
 await el.selectOption('{{selectValue}}');
 } else {
 await el.click();
-await page.getByText('{{selectValue}}', { exact: true }).first().click();
+{{> pick-option}}
 }
 }
 {{~else~}}

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

@@ -1 +1,7 @@
-await expect(page.getByText(/{{#if label}}{{escapeRegex label}}\s*{{else}}(?:^|\s){{/if}}{{escapeRegex dataValue}}$/)).toBeVisible();
+{{!-- Caption+value read: the ref names on-screen TEXT ("学科 -"), not an element, so the locator is
+      built inline from the caption rather than through the `locator` partial. That is why the scope
+      clause needs its own slot here — the partial's `parentLocator` handling never runs (#747).
+      Unscoped, `… in [Education Card 6] card` compiled away to nothing: on a screen repeating ten
+      学歴・職歴 cards the assertion strict-mode-failed where the caption repeats, and where it
+      happened to be unique elsewhere it passed having read a DIFFERENT card than the sentence names. --}}
+await expect({{#if parentLocator}}{{parentLocator}}{{else}}page{{/if}}.getByText(/{{#if label}}{{escapeRegex label}}\s*{{else}}(?:^|\s){{/if}}{{escapeRegex dataValue}}$/)).toBeVisible();

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

@@ -1,9 +1,13 @@
+{{!-- `getByRole` is built inline from role/name rather than through the `locator` partial, so the
+      scope clause needs its own slot here exactly as in label-value-assertion — the label branch's
+      twin (#747). `see [Logo] image with {{alt}} in [Product Card 3] card` searched every card on
+      the page before this. --}}
 {{#if name}}
 {{#if dataValue}}
-await expect(page.getByRole('{{role}}', { name: '{{escapeQuotes name}}'{{#if exact}}, exact: true{{/if}} }).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{> locator-nth}}).toBeVisible();
+await expect({{#if parentLocator}}{{parentLocator}}{{else}}page{{/if}}.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}} }){{> locator-nth}}).toBeVisible();
+await expect({{#if parentLocator}}{{parentLocator}}{{else}}page{{/if}}.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}}$/ }){{> locator-nth}}).toBeVisible();
-{{/if}}
+await expect({{#if parentLocator}}{{parentLocator}}{{else}}page{{/if}}.getByRole('{{role}}'{{#if exact}}, { exact: true }{{/if}}).filter({ hasText: /^{{escapeRegex dataValue}}$/ }){{> locator-nth}}).toBeVisible();
+{{/if}}

package/dist/generators/test-generator/adapters/playwright/templates/steps/partials/pick-option.hbs

@@ -0,0 +1,13 @@
+{{!-- #847 — pick the value in an OPENED custom listbox (Radix/shadcn, MUI, Headless UI).
+      The popup portals to the end of <body>, so a page-wide getByText(…).first() resolved to
+      any EARLIER element with the same text (a badge showing the current choice, a result
+      card) — and the open popup blocks clicks outside it, so the step timed out every time,
+      looking like a race. The ARIA option lives only in the open listbox: wait briefly for it
+      (a count() right after opening would race the popup animation), and fall back to the old
+      text match only for a widget that exposes no listbox roles. --}}
+{ const __opt = page.getByRole('option', { name: '{{selectValue}}', exact: true }).first();
+if (await __opt.waitFor({ state: 'visible', timeout: 3000 }).then(() => true, () => false)) {
+await __opt.click();
+} else {
+await page.getByText('{{selectValue}}', { exact: true }).first().click();
+} }

package/dist/generators/test-generator/adapters/playwright/templates/steps/partials/url-assertion.hbs

@@ -23,8 +23,12 @@
       `https://admin.example.com/jobs` asserted the PATH on any host, and the public site's /jobs
       satisfied it.
 
+      `ignoreQuery` comes from the page ENTRY's `query: any`, not from the step, because it is a fact
+      about the page rather than about one assertion (#748). It relaxes only the bare-page rule; the
+      declared-value half below is untouched, so the two compose rather than override each other.
+
       The declared-value half is gated on `dataRef`, not on `query`. The step wrote `with {{v}}`
       either way; gating on the resolved VALUE meant an empty test-data entry silently dropped the
       whole check and fell back to the bare-page rule, which is the one case specs/url-assert.ts
       exists to throw on. --}}
-await expectPage(page, { pathname: /^{{pathRegex}}\/?$/{{#if hostRegex}}, hostname: /^{{hostRegex}}$/{{/if}}{{#if dataRef}}, dataRef: '{{escapeQuotes dataRef}}', declared: '{{escapeQuotes query}}'{{/if}} });
+await expectPage(page, { pathname: /^{{pathRegex}}\/?$/{{#if hostRegex}}, hostname: /^{{hostRegex}}$/{{/if}}{{#if ignoreQuery}}, ignoreQuery: true{{/if}}{{#if dataRef}}, dataRef: '{{escapeQuotes dataRef}}', declared: '{{escapeQuotes query}}'{{/if}} });

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

@@ -46,6 +46,10 @@ test.describe('{{featureName}}', () => {
   test.describe('{{authRole}}', () => {
     test.use({ storageState: 'specs/.auth/{{authRole}}.json' });
 
+{{#if afterEach}}
+{{indent afterEach 2}}
+
+{{/if}}
 {{#each scenarios}}
 {{indent this 2}}
 
@@ -63,6 +67,18 @@ test.describe('{{featureName}}', () => {
 {{#each scenarios}}
 {{indent this 2}}
 
+{{/each}}
+  });
+
+{{else if afterEach}}
+{{!-- The un-tagged group next to a `@no-auth` one (#815): an ANONYMOUS describe scopes the hook to
+     these tests without adding a title segment, so test titles (and delivery fingerprints) stay put. --}}
+  test.describe(() => {
+{{indent afterEach 2}}
+
+{{#each scenarios}}
+{{indent this 2}}
+
 {{/each}}
   });
 

package/dist/generators/test-generator/code-generator.d.ts

@@ -139,7 +139,8 @@ export declare class CodeGenerator {
      * a name in the committed .env.qa.example. An undeclared ref means the test throws
      * "environment variable not set" the moment the key is used — surface it at compile
      * time instead. Only the `${QA_*}` shape is checked: any other `${...}` text is
-     * ordinary literal data by design (never resolved, never warned about).
+     * ordinary literal data by design (never resolved, never warned about). A `${QA_*}`
+     * that appears only inside a `#` comment is prose, not a reference, and is ignored.
      */
     private reportedEnvRefs;
     private checkEnvSecretRefs;

package/dist/generators/test-generator/code-generator.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"code-generator.d.ts","sourceRoot":"","sources":["../../../src/generators/test-generator/code-generator.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,aAAa,EAA8B,MAAM,mBAAmB,CAAC;AAI9E,OAAO,EAAE,oBAAoB,EAAE,MAAM,eAAe,CAAC;AAsQrD;;GAEG;AACH,MAAM,WAAW,aAAa;IAC5B,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,qBAAa,aAAa;IACxB,OAAO,CAAC,UAAU,CAAa;IAC/B,OAAO,CAAC,OAAO,CAAuB;IACtC,OAAO,CAAC,UAAU,CAAC,CAAS;IAC5B,OAAO,CAAC,OAAO,CAAM;IAErB,OAAO,CAAC,eAAe,CAAc;IAErC,OAAO,CAAC,aAAa,CAAqC;IAE1D,OAAO,CAAC,WAAW,CAA8B;gBAErC,OAAO,GAAE;QAAE,KAAK,CAAC,EAAE,OAAO,CAAC;QAAC,OAAO,CAAC,EAAE,OAAO,CAAC;QAAC,SAAS,CAAC,EAAE,MAAM,CAAC;QAAC,OAAO,CAAC,EAAE,MAAM,CAAC;QAAC,UAAU,CAAC,EAAE,MAAM,CAAC;QAAC,WAAW,CAAC,EAAE,OAAO,CAAC;QAAC,QAAQ,CAAC,EAAE,OAAO,CAAA;KAAO;IAiBtK,oFAAoF;IACpF,cAAc,IAAI,oBAAoB;IAItC;;OAEG;IACG,YAAY,CAChB,OAAO,EAAE,aAAa,EACtB,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC,aAAa,CAAC;IAqGzB;;;;OAIG;IACH,OAAO,CAAC,eAAe;IAavB;;;OAGG;IACH,OAAO,CAAC,UAAU;IAclB;;OAEG;IACH,SAAS,CAAC,aAAa,EAAE,aAAa,GAAG,IAAI;IAS7C;;OAEG;IACH;;;;;OAKG;IACH,OAAO,CAAC,mBAAmB;IA0C3B;;;;OAIG;IACH,OAAO,CAAC,uBAAuB;IAoB/B;;;;;OAKG;IACH,OAAO,CAAC,mBAAmB;IAY3B,cAAc,CAAC,SAAS,EAAE,MAAM,GAAG,IAAI;YAczB,gBAAgB;YAiahB,uBAAuB;YAYvB,kBAAkB;IAoBhC;;;;;;;;;;;;;;;;OAgBG;YACW,YAAY;IA+C1B;;;;;OAKG;IACH,OAAO,CAAC,wBAAwB;IAShC;;;;;;;OAOG;IACH,OAAO,CAAC,mBAAmB;IAgC3B;;;OAGG;IACH,OAAO,CAAC,oBAAoB;IAM5B;;;;OAIG;IACH,QAAQ,CAAC,SAAS,wBAA+B;YAEnC,gBAAgB;IA6H9B;;OAEG;IACH,OAAO,CAAC,UAAU;IAKlB;;;OAGG;IACH,OAAO,CAAC,qBAAqB;IAS7B;;OAEG;IACH;;;;;;;OAOG;IACH,OAAO,CAAC,eAAe,CAAqB;IAC5C,OAAO,CAAC,kBAAkB;IAyBpB,gBAAgB,CACpB,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,MAAM,EACjB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,aAAa,EAAE,CAAC;CA+D5B"}
+{"version":3,"file":"code-generator.d.ts","sourceRoot":"","sources":["../../../src/generators/test-generator/code-generator.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,aAAa,EAA8B,MAAM,mBAAmB,CAAC;AAI9E,OAAO,EAAE,oBAAoB,EAAE,MAAM,eAAe,CAAC;AAyTrD;;GAEG;AACH,MAAM,WAAW,aAAa;IAC5B,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,qBAAa,aAAa;IACxB,OAAO,CAAC,UAAU,CAAa;IAC/B,OAAO,CAAC,OAAO,CAAuB;IACtC,OAAO,CAAC,UAAU,CAAC,CAAS;IAC5B,OAAO,CAAC,OAAO,CAAM;IAErB,OAAO,CAAC,eAAe,CAAc;IAErC,OAAO,CAAC,aAAa,CAAqC;IAE1D,OAAO,CAAC,WAAW,CAA8B;gBAErC,OAAO,GAAE;QAAE,KAAK,CAAC,EAAE,OAAO,CAAC;QAAC,OAAO,CAAC,EAAE,OAAO,CAAC;QAAC,SAAS,CAAC,EAAE,MAAM,CAAC;QAAC,OAAO,CAAC,EAAE,MAAM,CAAC;QAAC,UAAU,CAAC,EAAE,MAAM,CAAC;QAAC,WAAW,CAAC,EAAE,OAAO,CAAC;QAAC,QAAQ,CAAC,EAAE,OAAO,CAAA;KAAO;IAiBtK,oFAAoF;IACpF,cAAc,IAAI,oBAAoB;IAItC;;OAEG;IACG,YAAY,CAChB,OAAO,EAAE,aAAa,EACtB,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC,aAAa,CAAC;IAqGzB;;;;OAIG;IACH,OAAO,CAAC,eAAe;IAavB;;;OAGG;IACH,OAAO,CAAC,UAAU;IAclB;;OAEG;IACH,SAAS,CAAC,aAAa,EAAE,aAAa,GAAG,IAAI;IAS7C;;OAEG;IACH;;;;;OAKG;IACH,OAAO,CAAC,mBAAmB;IA0C3B;;;;OAIG;IACH,OAAO,CAAC,uBAAuB;IAoB/B;;;;;OAKG;IACH,OAAO,CAAC,mBAAmB;IAY3B,cAAc,CAAC,SAAS,EAAE,MAAM,GAAG,IAAI;YAczB,gBAAgB;YAqbhB,uBAAuB;YAYvB,kBAAkB;IAoBhC;;;;;;;;;;;;;;;;OAgBG;YACW,YAAY;IA+C1B;;;;;OAKG;IACH,OAAO,CAAC,wBAAwB;IAShC;;;;;;;OAOG;IACH,OAAO,CAAC,mBAAmB;IAgC3B;;;OAGG;IACH,OAAO,CAAC,oBAAoB;IAM5B;;;;OAIG;IACH,QAAQ,CAAC,SAAS,wBAA+B;YAEnC,gBAAgB;IA6H9B;;OAEG;IACH,OAAO,CAAC,UAAU;IAKlB;;;OAGG;IACH,OAAO,CAAC,qBAAqB;IAS7B;;OAEG;IACH;;;;;;;;OAQG;IACH,OAAO,CAAC,eAAe,CAAqB;IAC5C,OAAO,CAAC,kBAAkB;IAyBpB,gBAAgB,CACpB,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,MAAM,EACjB,YAAY,EAAE,MAAM,EAAE,GACrB,OAAO,CAAC,aAAa,EAAE,CAAC;CA+D5B"}

package/dist/generators/test-generator/code-generator.js

@@ -15,6 +15,7 @@ const registry_1 = require("../../capabilities/registry");
 const discover_1 = require("../../capabilities/discover");
 const capability_1 = require("../../harness/capability");
 const scenario_merger_1 = require("../../exporters/scenario-merger");
+const manual_hybrid_oracle_1 = require("../../exporters/manual-hybrid-oracle");
 const spec_syntax_check_1 = require("./utils/spec-syntax-check");
 /**
  * Filter base scenario steps for @extend: only keep Given→When steps.
@@ -193,7 +194,7 @@ function getAuthMode(scenarioTags, featureTags) {
  * Grouping key for the anonymous bucket. A literal that no `@auth:<role>` can produce, so an
  * imaginative role name can never land in the same group as `@no-auth`.
  */
-const NO_AUTH_GROUP_KEY = 'no-auth';
+const NO_AUTH_GROUP_KEY = '\0no-auth';
 /**
  * True when the scenario has at least one step in the Then bucket —
  * And/But inherit the preceding primary keyword (Given/When/Then).
@@ -243,6 +244,69 @@ function warnMultipleAuthTags(tags, context) {
             `  Ignoring: ${authTags.slice(1).join(', ')}`);
     }
 }
+/**
+ * Drop the YAML comment from ONE line: a `#` at line start or preceded by whitespace,
+ * and not inside a quoted scalar. Line-level only — see `yamlWithoutComments` for the
+ * block-scalar case, which this function cannot see from a single line.
+ */
+function stripYamlComment(line) {
+    let quote = null;
+    for (let i = 0; i < line.length; i++) {
+        const c = line[i];
+        if (quote) {
+            if (quote === '"' && c === '\\') {
+                i++;
+                continue;
+            }
+            if (c === quote)
+                quote = null;
+            continue;
+        }
+        if (c === '"' || c === "'") {
+            quote = c;
+            continue;
+        }
+        if (c === '#' && (i === 0 || /\s/.test(line[i - 1])))
+            return line.slice(0, i);
+    }
+    return line;
+}
+/**
+ * The YAML text a `${QA_*}` scan may look at: comments removed, block scalars kept whole.
+ *
+ * SG-W009's reference scan used to run over the whole raw file, so a `${QA_*}` that only ever
+ * appeared in a `#` comment counted as a live reference — including the one sungen's own
+ * scaffolded "HOW TO FILL VALUES" block writes into every test-data file. The declared side has
+ * always been line-wise; this brings the reference side to the same footing.
+ *
+ * Inside a `|`/`>` block scalar every character is literal content, `#` included, so those lines
+ * pass through untouched. Stripping them would send the check the other way — a real, undeclared
+ * `${QA_*}` written after a `#` in a block scalar would go unreported, and the run would only fail
+ * later with "environment variable not set". A missed warning is worse than a noisy one.
+ */
+function yamlWithoutComments(text) {
+    const out = [];
+    let blockIndent = null;
+    for (const line of text.split('\n')) {
+        if (blockIndent !== null) {
+            const indent = line.search(/\S/);
+            // blank lines and anything indented deeper than the key stay inside the scalar
+            if (indent === -1 || indent > blockIndent) {
+                out.push(line);
+                continue;
+            }
+            blockIndent = null;
+        }
+        const block = /^(\s*)[^#\s][^:]*:\s*[|>][+-]?\d*[+-]?\s*$/.exec(line);
+        if (block) {
+            blockIndent = block[1].length;
+            out.push(line);
+            continue;
+        }
+        out.push(stripYamlComment(line));
+    }
+    return out.join('\n');
+}
 class CodeGenerator {
     constructor(options = {}) {
         // Screen/flow name for the CURRENT feature, in catalog-resolution form (`flows/<x>` for flows).
@@ -265,7 +329,8 @@ class CodeGenerator {
          * a name in the committed .env.qa.example. An undeclared ref means the test throws
          * "environment variable not set" the moment the key is used — surface it at compile
          * time instead. Only the `${QA_*}` shape is checked: any other `${...}` text is
-         * ordinary literal data by design (never resolved, never warned about).
+         * ordinary literal data by design (never resolved, never warned about). A `${QA_*}`
+         * that appears only inside a `#` comment is prose, not a reference, and is ignored.
          */
         this.reportedEnvRefs = new Set();
         this.options = options;
@@ -689,10 +754,16 @@ class CodeGenerator {
                 //    labeled `Observable:`/`Oracle:`/`Expected:` line (parseManualComments — the
                 //    SAME parser delivery uses). Field finding (student-site, 87 scenarios): prose
                 //    paragraphs and rationale-only comments exported EMPTY Steps + Expected cells.
+                //  • HAS a Then, but every assertion is a bare presence placeholder while the real oracle
+                //    sits in the comments (#810) — the same banned hybrid wearing a Then: the deliverable
+                //    reads "See [X] table" and a tester ticking it Passed records a check nobody made.
                 const isStructured = scenario.steps.length > 0;
                 if (isStructured && !hasThenStep(scenario)) {
                     this.diagnostics.add('SG-W008', `@manual scenario "${scenario.name}" has Gherkin steps but no Then — a When without Then is invalid Gherkin. Add the Then step, or remove the steps and write the full procedure (action steps, then "Observable:"/"Oracle:" lines) in its \`# Tester verifies:\` block`, { feature: feature.name, step: scenario.name });
                 }
+                else if (isStructured && (0, manual_hybrid_oracle_1.isPresencePlaceholderHybrid)(scenario.steps, scenario.comments ?? [])) {
+                    this.diagnostics.add('SG-W008', `@manual scenario "${scenario.name}" asserts only presence (e.g. "User see [X] table") while its real oracle sits in the \`# Tester verifies:\` comment — the deliverable's Expected would not say what the case checks. Write the oracle as the Then step (e.g. \`table match data:\`), or remove the steps and write the full numbered procedure ("1. <action>" … "N. Observable: <what>")`, { feature: feature.name, step: scenario.name });
+                }
                 else if (!isStructured) {
                     const parsed = (0, scenario_merger_1.parseManualComments)(scenario.comments ?? []);
                     if (!parsed.steps.length || !parsed.expected.length) {
@@ -765,7 +836,7 @@ class CodeGenerator {
             }
             authGroupMap.get(key).push(code);
         }
-        const authGroups = groupOrder.map(key => ({
+        const authGroups = groupOrder.map((key) => ({
             authRole: key === NO_AUTH_GROUP_KEY ? undefined : (key || undefined),
             noAuth: key === NO_AUTH_GROUP_KEY,
             scenarios: authGroupMap.get(key),
@@ -799,6 +870,20 @@ class CodeGenerator {
                 `into one feature file per auth role.`);
         }
         const scenarios = renderedScenarios.map(s => s.code);
+        // #815 — a feature-level `@afterEach` used to sit in the outermost describe, so it also ran after
+        // every `@no-auth` test, on an EMPTY session: a hook that needs the signed-in page (`Given User is
+        // on [probe] page` → redirected to /login) failed an anonymous test whose own steps passed. When
+        // the feature mixes `@no-auth` with any other group, the hook moves INTO each other group and the
+        // no-auth group runs without it. A feature that is anonymous throughout keeps the hook — it was
+        // written for those tests.
+        // (A no-auth group beside another one means ≥2 groups, so `renderAuthBlocks` is already true.)
+        const afterEachInGroups = !!afterEach
+            && authGroups.some(g => g.noAuth) && authGroups.some(g => !g.noAuth);
+        if (afterEachInGroups) {
+            for (const g of authGroups)
+                if (!g.noAuth)
+                    g.afterEach = afterEach;
+        }
         // For single group, extract the auth role to put test.use at describe level. Keyed on
         // `renderAuthBlocks`, not `needsGrouping`: a lone `@no-auth` group now renders its own block, so
         // hoisting anything to the feature describe would double up. Its authRole is undefined anyway.
@@ -889,7 +974,7 @@ class CodeGenerator {
             featureDescription: feature.description,
             background,
             beforeAll,
-            afterEach,
+            afterEach: afterEachInGroups ? undefined : afterEach,
             afterAll,
             cleanupConfig,
             screenshotOnFailure,
@@ -1184,7 +1269,7 @@ class CodeGenerator {
             return;
         const referenced = new Set();
         for (const f of fs_1.default.readdirSync(testDataDir).filter((x) => x.endsWith('.yaml') || x.endsWith('.yml'))) {
-            const text = fs_1.default.readFileSync(path_1.default.join(testDataDir, f), 'utf-8');
+            const text = yamlWithoutComments(fs_1.default.readFileSync(path_1.default.join(testDataDir, f), 'utf-8'));
             for (const m of text.matchAll(/\$\{(QA_[A-Z0-9_]+)\}/g))
                 referenced.add(m[1]);
         }