@sun-asterisk/sungen 2.4.6 → 2.5.0

package/dist/orchestrator/templates/ai-instructions/github-skill-sungen-tc-generation.md

@@ -112,20 +112,59 @@ Given User is on [Screen] page
 And User wait for [Page Title] heading is visible
 ```
 
+## Cleanup & Hooks
+
+### Auto-assign `@cleanup:*` tags based on screen sections
+
+After identifying screen sections, add appropriate `@cleanup:*` feature-level tags. These activate base.ts fixtures that auto-clean state between tests.
+
+| Screen has | Add tag | Why |
+|---|---|---|
+| Modal / Dialog / Drawer | `@cleanup:overlay` | Dismiss leftover overlays between tests |
+| Form & Inputs / Search / Filter | `@cleanup:forms` | Clear form fields, reset selects |
+| Long scrollable content | `@cleanup:scroll` | Scroll to top for consistent assertions |
+| Auth tokens / session data in tests | `@cleanup:storage` | Clear sessionStorage |
+| CI/CD or debug-heavy screens | `@screenshot:on-failure` | Auto-capture screenshot on test failure |
+
+**Always add `@cleanup:overlay`** if ANY section opens a dialog (Create/Add, Update/Edit, Delete confirmation). Most CRUD screens need it.
+
+**Always add `@cleanup:forms`** if the screen has inline search, filter dropdowns, or editable forms that persist between tests.
+
+### When to add `@afterEach` hook scenario
+
+Only when `@cleanup:*` tags aren't enough — feature-specific cleanup logic:
+- Reset a dropdown filter to default value (not just clear)
+- Navigate away from a sub-tab back to the main tab
+- Close a specific sidebar panel
+
+```gherkin
+@afterEach
+Scenario: Reset filters to default
+  When User select [Status Filter] dropdown with {{default_status}}
+```
+
+### `@beforeAll` / `@afterAll` — optional, low priority
+
+For one-time setup/teardown. Most screens don't need these.
+
 ## Output Format
 
 **Feature file** — `qa/screens/<screen>/features/<screen>.feature`
 
-**Never use `Background:`.** Use `@steps` + `@extend` for shared setup (see `sungen-gherkin-syntax` skill).
+`Background` is valid for simple shared setup (navigate to page). Use `@steps`/`@extend` for complex flows with scope (dialog, frame).
 
 ```gherkin
 @auth:role
+@cleanup:overlay
+@cleanup:forms
 Feature: <Screen> Screen
 
+  Background:
+    Given User is on [Screen] page
+
   # Shared setup — NO priority tag on @steps
   @steps:open_form
   Scenario: Open form
-    Given User is on [Screen] page
     When User click [Create] button
     Then User see [Form] dialog
 
@@ -171,6 +210,18 @@ Feature: <Screen> Screen
 
 **Naming**: `VP-<CATEGORY>-<NNN>` prefix. Scenario name must use the **same element type** as the steps — e.g., if the step uses `dialog`, write "dialog opens" not "modal opens".
 
-**Test data** — `qa/screens/<screen>/test-data/<screen>.yaml`, grouped by section.
+**Test data** — `qa/screens/<screen>/test-data/<screen>.yaml`, grouped by section. Data is loaded **at runtime** — keys become runtime lookups, not hardcoded strings. The same compiled test works across environments.
+
+**Environment-specific data**: For values that differ per environment (credentials, URLs, test users), create `<screen>.<env>.yaml` alongside the base file. Users run `SUNGEN_ENV=staging npx playwright test` to merge overrides. Structure env YAML with the same keys, only including values that change:
+
+```yaml
+# login.yaml (base)
+valid_email: admin@dev.example.com
+valid_password: DevPass123
+
+# login.staging.yaml (override for staging)
+valid_email: admin@staging.example.com
+valid_password: StagingPass456
+```
 
 **Do NOT generate**: `selectors.yaml` (created during run-test), Playwright code (sungen compiles).

package/dist/orchestrator/templates/specs-base.d.ts

@@ -1,4 +1,15 @@
 import { expect } from '@playwright/test';
-declare const test: import("@playwright/test").TestType<import("@playwright/test").PlaywrightTestArgs & import("@playwright/test").PlaywrightTestOptions, import("@playwright/test").PlaywrightWorkerArgs & import("@playwright/test").PlaywrightWorkerOptions>;
+type CleanupConfig = {
+    overlay?: boolean;
+    forms?: boolean;
+    scroll?: boolean;
+    storage?: boolean;
+};
+declare const test: import("@playwright/test").TestType<import("@playwright/test").PlaywrightTestArgs & import("@playwright/test").PlaywrightTestOptions & {
+    autoCleanup: CleanupConfig;
+    screenshotOnFailure: boolean;
+    _autoCleanup: void;
+    _autoScreenshot: void;
+}, import("@playwright/test").PlaywrightWorkerArgs & import("@playwright/test").PlaywrightWorkerOptions>;
 export { test, expect };
 //# sourceMappingURL=specs-base.d.ts.map

package/dist/orchestrator/templates/specs-base.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"specs-base.d.ts","sourceRoot":"","sources":["../../../src/orchestrator/templates/specs-base.ts"],"names":[],"mappings":"AAAA,OAAO,EAAgB,MAAM,EAAE,MAAM,kBAAkB,CAAC;AAQxD,QAAA,MAAM,IAAI,6OA8DR,CAAC;AAEH,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC"}
+{"version":3,"file":"specs-base.d.ts","sourceRoot":"","sources":["../../../src/orchestrator/templates/specs-base.ts"],"names":[],"mappings":"AAAA,OAAO,EAAgB,MAAM,EAAE,MAAM,kBAAkB,CAAC;AAGxD,KAAK,aAAa,GAAG;IACnB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB,CAAC;AAOF,QAAA,MAAM,IAAI;iBACK,aAAa;yBACL,OAAO;kBACd,IAAI;qBACD,IAAI;wGA6GrB,CAAC;AAEH,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC"}

package/dist/orchestrator/templates/specs-base.js

@@ -8,6 +8,8 @@ Object.defineProperty(exports, "expect", { enumerable: true, get: function () {
 const contextCache = new Map();
 const GOTO_PATCHED = Symbol('goto-patched');
 const test = test_1.test.extend({
+    autoCleanup: [{}, { option: true }],
+    screenshotOnFailure: [false, { option: true }],
     page: async ({ browser, storageState }, use) => {
         if (storageState) {
             const cacheKey = typeof storageState === 'string' ? storageState : JSON.stringify(storageState);
@@ -26,11 +28,6 @@ const test = test_1.test.extend({
                     try {
                         const currentPath = new URL(page.url()).pathname;
                         if (currentPath === url || currentPath === url + '/') {
-                            // Dismiss any open overlays (dropdowns, dialogs) from previous test
-                            await page.keyboard.press('Escape').catch(() => { });
-                            await page.locator('body').click({ position: { x: 1, y: 1 }, force: true }).catch(() => { });
-                            // Safety check: if a fixed-position overlay (modal/dialog) is still present, full reload
-                            // eslint-disable-next-line no-eval -- runs in browser context via Playwright
                             const hasOverlay = await page.evaluate(`(() => {
                 const el = document.elementFromPoint(window.innerWidth / 2, window.innerHeight / 2);
                 if (!el) return false;
@@ -65,6 +62,51 @@ const test = test_1.test.extend({
             await context.close();
         }
     },
+    // Auto-cleanup fixture: runs teardown after each test based on @cleanup:* tags
+    _autoCleanup: [async ({ page, autoCleanup }, use) => {
+            await use();
+            if (autoCleanup.overlay) {
+                await page.keyboard.press('Escape').catch(() => { });
+                await page.locator('body').click({ position: { x: 1, y: 1 }, force: true }).catch(() => { });
+                // Dismiss persistent fixed overlays (modals, dialogs)
+                const hasOverlay = await page.evaluate(`(() => {
+        const el = document.elementFromPoint(window.innerWidth / 2, window.innerHeight / 2);
+        if (!el) return false;
+        let current = el;
+        while (current && current !== document.body) {
+          if (getComputedStyle(current).position === 'fixed') return true;
+          current = current.parentElement;
+        }
+        return false;
+      })()`).catch(() => false);
+                if (hasOverlay) {
+                    await page.keyboard.press('Escape').catch(() => { });
+                }
+            }
+            if (autoCleanup.forms) {
+                await page.evaluate(`(() => {
+        document.querySelectorAll('input:not([type=hidden]):not([type=submit])').forEach(el => { el.value = ''; });
+        document.querySelectorAll('textarea').forEach(el => { el.value = ''; });
+        document.querySelectorAll('select').forEach(el => { el.selectedIndex = 0; });
+      })()`).catch(() => { });
+            }
+            if (autoCleanup.scroll) {
+                await page.evaluate('window.scrollTo(0, 0)').catch(() => { });
+            }
+            if (autoCleanup.storage) {
+                await page.evaluate('sessionStorage.clear()').catch(() => { });
+            }
+        }, { auto: true }],
+    // Auto-screenshot fixture: captures screenshot on test failure
+    _autoScreenshot: [async ({ page, screenshotOnFailure }, use, testInfo) => {
+            await use();
+            if (screenshotOnFailure && testInfo.status !== testInfo.expectedStatus) {
+                await testInfo.attach('screenshot', {
+                    body: await page.screenshot(),
+                    contentType: 'image/png',
+                });
+            }
+        }, { auto: true }],
 });
 exports.test = test;
 //# sourceMappingURL=specs-base.js.map

package/dist/orchestrator/templates/specs-base.js.map

@@ -1 +1 @@
-{"version":3,"file":"specs-base.js","sourceRoot":"","sources":["../../../src/orchestrator/templates/specs-base.ts"],"names":[],"mappings":";;;AAAA,2CAAwD;AAwEzC,uFAxEQ,aAAM,OAwER;AArErB,yEAAyE;AACzE,4DAA4D;AAC5D,MAAM,YAAY,GAAG,IAAI,GAAG,EAAmD,CAAC;AAChF,MAAM,YAAY,GAAG,MAAM,CAAC,cAAc,CAAC,CAAC;AAE5C,MAAM,IAAI,GAAG,WAAI,CAAC,MAAM,CAAC;IACvB,IAAI,EAAE,KAAK,EAAE,EAAE,OAAO,EAAE,YAAY,EAAE,EAAE,GAAG,EAAE,EAAE;QAC7C,IAAI,YAAY,EAAE,CAAC;YACjB,MAAM,QAAQ,GAAG,OAAO,YAAY,KAAK,QAAQ,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;YAEhG,IAAI,MAAM,GAAG,YAAY,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;YACxC,IAAI,CAAC,MAAM,EAAE,CAAC;gBACZ,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,UAAU,CAAC,EAAE,YAAY,EAAE,CAAC,CAAC;gBAC3D,MAAM,IAAI,GAAG,MAAM,OAAO,CAAC,OAAO,EAAE,CAAC;gBACrC,MAAM,GAAG,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC;gBAC3B,YAAY,CAAC,GAAG,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;YACrC,CAAC;YAED,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC;YAEzB,iEAAiE;YACjE,IAAI,CAAE,IAAY,CAAC,YAAY,CAAC,EAAE,CAAC;gBACjC,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBAC1C,IAAI,CAAC,IAAI,GAAG,KAAK,WAAW,GAAW,EAAE,OAAa;oBACpD,IAAI,CAAC;wBACH,MAAM,WAAW,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,QAAQ,CAAC;wBACjD,IAAI,WAAW,KAAK,GAAG,IAAI,WAAW,KAAK,GAAG,GAAG,GAAG,EAAE,CAAC;4BACrD,oEAAoE;4BACpE,MAAM,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;4BACpD,MAAM,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,KAAK,CAAC,EAAE,QAAQ,EAAE,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;4BAE5F,yFAAyF;4BACzF,6EAA6E;4BAC7E,MAAM,UAAU,GAAG,MAAM,IAAI,CAAC,QAAQ,CAAC;;;;;;;;;mBASlC,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;4BAEzB,IAAI,UAAU,EAAE,CAAC;gCACf,OAAO,YAAY,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;4BACpC,CAAC;4BACD,OAAO,IAAW,CAAC;wBACrB,CAAC;oBACH,CAAC;oBAAC,MAAM,CAAC;wBACP,+CAA+C;oBACjD,CAAC;oBACD,OAAO,YAAY,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;gBACpC,CAAC,CAAC;gBACD,IAAY,CAAC,YAAY,CAAC,GAAG,IAAI,CAAC;YACrC,CAAC;YAED,MAAM,GAAG,CAAC,IAAI,CAAC,CAAC;QAClB,CAAC;aAAM,CAAC;YACN,+DAA+D;YAC/D,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,UAAU,EAAE,CAAC;YAC3C,MAAM,IAAI,GAAG,MAAM,OAAO,CAAC,OAAO,EAAE,CAAC;YACrC,MAAM,GAAG,CAAC,IAAI,CAAC,CAAC;YAChB,MAAM,IAAI,CAAC,KAAK,EAAE,CAAC;YACnB,MAAM,OAAO,CAAC,KAAK,EAAE,CAAC;QACxB,CAAC;IACH,CAAC;CACF,CAAC,CAAC;AAEM,oBAAI"}
+{"version":3,"file":"specs-base.js","sourceRoot":"","sources":["../../../src/orchestrator/templates/specs-base.ts"],"names":[],"mappings":";;;AAAA,2CAAwD;AAkIzC,uFAlIQ,aAAM,OAkIR;AAxHrB,yEAAyE;AACzE,4DAA4D;AAC5D,MAAM,YAAY,GAAG,IAAI,GAAG,EAAmD,CAAC;AAChF,MAAM,YAAY,GAAG,MAAM,CAAC,cAAc,CAAC,CAAC;AAE5C,MAAM,IAAI,GAAG,WAAI,CAAC,MAAM,CAKrB;IACD,WAAW,EAAE,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IACnC,mBAAmB,EAAE,CAAC,KAAK,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IAE9C,IAAI,EAAE,KAAK,EAAE,EAAE,OAAO,EAAE,YAAY,EAAE,EAAE,GAAG,EAAE,EAAE;QAC7C,IAAI,YAAY,EAAE,CAAC;YACjB,MAAM,QAAQ,GAAG,OAAO,YAAY,KAAK,QAAQ,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;YAEhG,IAAI,MAAM,GAAG,YAAY,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;YACxC,IAAI,CAAC,MAAM,EAAE,CAAC;gBACZ,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,UAAU,CAAC,EAAE,YAAY,EAAE,CAAC,CAAC;gBAC3D,MAAM,IAAI,GAAG,MAAM,OAAO,CAAC,OAAO,EAAE,CAAC;gBACrC,MAAM,GAAG,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC;gBAC3B,YAAY,CAAC,GAAG,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;YACrC,CAAC;YAED,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC;YAEzB,iEAAiE;YACjE,IAAI,CAAE,IAAY,CAAC,YAAY,CAAC,EAAE,CAAC;gBACjC,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBAC1C,IAAI,CAAC,IAAI,GAAG,KAAK,WAAW,GAAW,EAAE,OAAa;oBACpD,IAAI,CAAC;wBACH,MAAM,WAAW,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,QAAQ,CAAC;wBACjD,IAAI,WAAW,KAAK,GAAG,IAAI,WAAW,KAAK,GAAG,GAAG,GAAG,EAAE,CAAC;4BACrD,MAAM,UAAU,GAAG,MAAM,IAAI,CAAC,QAAQ,CAAC;;;;;;;;;mBASlC,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;4BAEzB,IAAI,UAAU,EAAE,CAAC;gCACf,OAAO,YAAY,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;4BACpC,CAAC;4BACD,OAAO,IAAW,CAAC;wBACrB,CAAC;oBACH,CAAC;oBAAC,MAAM,CAAC;wBACP,+CAA+C;oBACjD,CAAC;oBACD,OAAO,YAAY,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;gBACpC,CAAC,CAAC;gBACD,IAAY,CAAC,YAAY,CAAC,GAAG,IAAI,CAAC;YACrC,CAAC;YAED,MAAM,GAAG,CAAC,IAAI,CAAC,CAAC;QAClB,CAAC;aAAM,CAAC;YACN,+DAA+D;YAC/D,MAAM,OAAO,GAAG,MAAM,OAAO,CAAC,UAAU,EAAE,CAAC;YAC3C,MAAM,IAAI,GAAG,MAAM,OAAO,CAAC,OAAO,EAAE,CAAC;YACrC,MAAM,GAAG,CAAC,IAAI,CAAC,CAAC;YAChB,MAAM,IAAI,CAAC,KAAK,EAAE,CAAC;YACnB,MAAM,OAAO,CAAC,KAAK,EAAE,CAAC;QACxB,CAAC;IACH,CAAC;IAED,+EAA+E;IAC/E,YAAY,EAAE,CAAC,KAAK,EAAE,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,GAAG,EAAE,EAAE;YAClD,MAAM,GAAG,EAAE,CAAC;YAEZ,IAAI,WAAW,CAAC,OAAO,EAAE,CAAC;gBACxB,MAAM,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;gBACpD,MAAM,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,KAAK,CAAC,EAAE,QAAQ,EAAE,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;gBAC5F,sDAAsD;gBACtD,MAAM,UAAU,GAAG,MAAM,IAAI,CAAC,QAAQ,CAAC;;;;;;;;;WASlC,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,KAAK,CAAC,CAAC;gBAC1B,IAAI,UAAU,EAAE,CAAC;oBACf,MAAM,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;gBACtD,CAAC;YACH,CAAC;YACD,IAAI,WAAW,CAAC,KAAK,EAAE,CAAC;gBACtB,MAAM,IAAI,CAAC,QAAQ,CAAC;;;;WAIf,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;YACzB,CAAC;YACD,IAAI,WAAW,CAAC,MAAM,EAAE,CAAC;gBACvB,MAAM,IAAI,CAAC,QAAQ,CAAC,uBAAuB,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;YAC/D,CAAC;YACD,IAAI,WAAW,CAAC,OAAO,EAAE,CAAC;gBACxB,MAAM,IAAI,CAAC,QAAQ,CAAC,wBAAwB,CAAC,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;YAChE,CAAC;QACH,CAAC,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC;IAElB,+DAA+D;IAC/D,eAAe,EAAE,CAAC,KAAK,EAAE,EAAE,IAAI,EAAE,mBAAmB,EAAE,EAAE,GAAG,EAAE,QAAQ,EAAE,EAAE;YACvE,MAAM,GAAG,EAAE,CAAC;YAEZ,IAAI,mBAAmB,IAAI,QAAQ,CAAC,MAAM,KAAK,QAAQ,CAAC,cAAc,EAAE,CAAC;gBACvE,MAAM,QAAQ,CAAC,MAAM,CAAC,YAAY,EAAE;oBAClC,IAAI,EAAE,MAAM,IAAI,CAAC,UAAU,EAAE;oBAC7B,WAAW,EAAE,WAAW;iBACzB,CAAC,CAAC;YACL,CAAC;QACH,CAAC,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC;CACnB,CAAC,CAAC;AAEM,oBAAI"}

package/dist/orchestrator/templates/specs-base.ts

@@ -1,12 +1,27 @@
 import { test as base, expect } from '@playwright/test';
 import type { BrowserContext, Page } from '@playwright/test';
 
+type CleanupConfig = {
+  overlay?: boolean;
+  forms?: boolean;
+  scroll?: boolean;
+  storage?: boolean;
+};
+
 // Share one context per storageState — avoids creating multiple sessions
 // that trigger server rate limiting or session invalidation
 const contextCache = new Map<string, { context: BrowserContext; page: Page }>();
 const GOTO_PATCHED = Symbol('goto-patched');
 
-const test = base.extend({
+const test = base.extend<{
+  autoCleanup: CleanupConfig;
+  screenshotOnFailure: boolean;
+  _autoCleanup: void;
+  _autoScreenshot: void;
+}>({
+  autoCleanup: [{}, { option: true }],
+  screenshotOnFailure: [false, { option: true }],
+
   page: async ({ browser, storageState }, use) => {
     if (storageState) {
       const cacheKey = typeof storageState === 'string' ? storageState : JSON.stringify(storageState);
@@ -28,12 +43,6 @@ const test = base.extend({
           try {
             const currentPath = new URL(page.url()).pathname;
             if (currentPath === url || currentPath === url + '/') {
-              // Dismiss any open overlays (dropdowns, dialogs) from previous test
-              await page.keyboard.press('Escape').catch(() => {});
-              await page.locator('body').click({ position: { x: 1, y: 1 }, force: true }).catch(() => {});
-
-              // Safety check: if a fixed-position overlay (modal/dialog) is still present, full reload
-              // eslint-disable-next-line no-eval -- runs in browser context via Playwright
               const hasOverlay = await page.evaluate(`(() => {
                 const el = document.elementFromPoint(window.innerWidth / 2, window.innerHeight / 2);
                 if (!el) return false;
@@ -68,6 +77,55 @@ const test = base.extend({
       await context.close();
     }
   },
+
+  // Auto-cleanup fixture: runs teardown after each test based on @cleanup:* tags
+  _autoCleanup: [async ({ page, autoCleanup }, use) => {
+    await use();
+
+    if (autoCleanup.overlay) {
+      await page.keyboard.press('Escape').catch(() => {});
+      await page.locator('body').click({ position: { x: 1, y: 1 }, force: true }).catch(() => {});
+      // Dismiss persistent fixed overlays (modals, dialogs)
+      const hasOverlay = await page.evaluate(`(() => {
+        const el = document.elementFromPoint(window.innerWidth / 2, window.innerHeight / 2);
+        if (!el) return false;
+        let current = el;
+        while (current && current !== document.body) {
+          if (getComputedStyle(current).position === 'fixed') return true;
+          current = current.parentElement;
+        }
+        return false;
+      })()`).catch(() => false);
+      if (hasOverlay) {
+        await page.keyboard.press('Escape').catch(() => {});
+      }
+    }
+    if (autoCleanup.forms) {
+      await page.evaluate(`(() => {
+        document.querySelectorAll('input:not([type=hidden]):not([type=submit])').forEach(el => { el.value = ''; });
+        document.querySelectorAll('textarea').forEach(el => { el.value = ''; });
+        document.querySelectorAll('select').forEach(el => { el.selectedIndex = 0; });
+      })()`).catch(() => {});
+    }
+    if (autoCleanup.scroll) {
+      await page.evaluate('window.scrollTo(0, 0)').catch(() => {});
+    }
+    if (autoCleanup.storage) {
+      await page.evaluate('sessionStorage.clear()').catch(() => {});
+    }
+  }, { auto: true }],
+
+  // Auto-screenshot fixture: captures screenshot on test failure
+  _autoScreenshot: [async ({ page, screenshotOnFailure }, use, testInfo) => {
+    await use();
+
+    if (screenshotOnFailure && testInfo.status !== testInfo.expectedStatus) {
+      await testInfo.attach('screenshot', {
+        body: await page.screenshot(),
+        contentType: 'image/png',
+      });
+    }
+  }, { auto: true }],
 });
 
 export { test, expect };

package/dist/orchestrator/templates/specs-test-data.d.ts

@@ -0,0 +1,14 @@
+export declare class TestDataLoader {
+    private data;
+    private constructor();
+    /**
+     * Load test data for a screen/feature combination.
+     *
+     * Priority (later wins):
+     * 1. {feature}.yaml — base data
+     * 2. {feature}.{SUNGEN_ENV}.yaml — environment-specific (if SUNGEN_ENV set)
+     */
+    static load(screenName: string, featureName: string): TestDataLoader;
+    get(key: string): string;
+}
+//# sourceMappingURL=specs-test-data.d.ts.map

package/dist/orchestrator/templates/specs-test-data.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"specs-test-data.d.ts","sourceRoot":"","sources":["../../../src/orchestrator/templates/specs-test-data.ts"],"names":[],"mappings":"AAIA,qBAAa,cAAc;IACzB,OAAO,CAAC,IAAI,CAAsB;IAElC,OAAO;IAIP;;;;;;OAMG;IACH,MAAM,CAAC,IAAI,CAAC,UAAU,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,cAAc;IAcpE,GAAG,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM;CAczB"}

package/dist/orchestrator/templates/specs-test-data.js

@@ -0,0 +1,100 @@
+"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;
+    };
+})();
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.TestDataLoader = void 0;
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+const yaml_1 = __importDefault(require("yaml"));
+class TestDataLoader {
+    constructor(data) {
+        this.data = data;
+    }
+    /**
+     * Load test data for a screen/feature combination.
+     *
+     * Priority (later wins):
+     * 1. {feature}.yaml — base data
+     * 2. {feature}.{SUNGEN_ENV}.yaml — environment-specific (if SUNGEN_ENV set)
+     */
+    static load(screenName, featureName) {
+        const baseDir = path.join(process.cwd(), 'qa', 'screens', screenName, 'test-data');
+        const env = process.env.SUNGEN_ENV;
+        let data = loadYamlSync(path.join(baseDir, `${featureName}.yaml`)) || {};
+        if (env) {
+            const envData = loadYamlSync(path.join(baseDir, `${featureName}.${env}.yaml`));
+            if (envData)
+                data = deepMerge(data, envData);
+        }
+        return new TestDataLoader(data);
+    }
+    get(key) {
+        const parts = key.split('.');
+        let current = this.data;
+        for (const part of parts) {
+            if (current == null || typeof current !== 'object') {
+                throw new Error(`Test data key not found: ${key} (failed at '${part}')`);
+            }
+            current = current[part];
+        }
+        if (current === undefined || current === null) {
+            throw new Error(`Test data key not found: ${key}`);
+        }
+        return String(current);
+    }
+}
+exports.TestDataLoader = TestDataLoader;
+function loadYamlSync(filePath) {
+    if (!fs.existsSync(filePath))
+        return null;
+    const content = fs.readFileSync(filePath, 'utf-8');
+    return yaml_1.default.parse(content) || null;
+}
+function deepMerge(base, override) {
+    const result = { ...base };
+    for (const [key, value] of Object.entries(override)) {
+        if (value && typeof value === 'object' && !Array.isArray(value) &&
+            result[key] && typeof result[key] === 'object' && !Array.isArray(result[key])) {
+            result[key] = deepMerge(result[key], value);
+        }
+        else {
+            result[key] = value;
+        }
+    }
+    return result;
+}
+//# sourceMappingURL=specs-test-data.js.map

package/dist/orchestrator/templates/specs-test-data.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"specs-test-data.js","sourceRoot":"","sources":["../../../src/orchestrator/templates/specs-test-data.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA,uCAAyB;AACzB,2CAA6B;AAC7B,gDAAwB;AAExB,MAAa,cAAc;IAGzB,YAAoB,IAAyB;QAC3C,IAAI,CAAC,IAAI,GAAG,IAAI,CAAC;IACnB,CAAC;IAED;;;;;;OAMG;IACH,MAAM,CAAC,IAAI,CAAC,UAAkB,EAAE,WAAmB;QACjD,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,IAAI,EAAE,SAAS,EAAE,UAAU,EAAE,WAAW,CAAC,CAAC;QACnF,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,CAAC,UAAU,CAAC;QAEnC,IAAI,IAAI,GAAG,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,GAAG,WAAW,OAAO,CAAC,CAAC,IAAI,EAAE,CAAC;QAEzE,IAAI,GAAG,EAAE,CAAC;YACR,MAAM,OAAO,GAAG,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,GAAG,WAAW,IAAI,GAAG,OAAO,CAAC,CAAC,CAAC;YAC/E,IAAI,OAAO;gBAAE,IAAI,GAAG,SAAS,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;QAC/C,CAAC;QAED,OAAO,IAAI,cAAc,CAAC,IAAI,CAAC,CAAC;IAClC,CAAC;IAED,GAAG,CAAC,GAAW;QACb,MAAM,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC7B,IAAI,OAAO,GAAQ,IAAI,CAAC,IAAI,CAAC;QAC7B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,IAAI,OAAO,IAAI,IAAI,IAAI,OAAO,OAAO,KAAK,QAAQ,EAAE,CAAC;gBACnD,MAAM,IAAI,KAAK,CAAC,4BAA4B,GAAG,gBAAgB,IAAI,IAAI,CAAC,CAAC;YAC3E,CAAC;YACD,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;QAC1B,CAAC;QACD,IAAI,OAAO,KAAK,SAAS,IAAI,OAAO,KAAK,IAAI,EAAE,CAAC;YAC9C,MAAM,IAAI,KAAK,CAAC,4BAA4B,GAAG,EAAE,CAAC,CAAC;QACrD,CAAC;QACD,OAAO,MAAM,CAAC,OAAO,CAAC,CAAC;IACzB,CAAC;CACF;AA1CD,wCA0CC;AAED,SAAS,YAAY,CAAC,QAAgB;IACpC,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,QAAQ,CAAC;QAAE,OAAO,IAAI,CAAC;IAC1C,MAAM,OAAO,GAAG,EAAE,CAAC,YAAY,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAC;IACnD,OAAO,cAAI,CAAC,KAAK,CAAC,OAAO,CAAC,IAAI,IAAI,CAAC;AACrC,CAAC;AAED,SAAS,SAAS,CAAC,IAAyB,EAAE,QAA6B;IACzE,MAAM,MAAM,GAAG,EAAE,GAAG,IAAI,EAAE,CAAC;IAC3B,KAAK,MAAM,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAC;QACpD,IAAI,KAAK,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC;YAC3D,MAAM,CAAC,GAAG,CAAC,IAAI,OAAO,MAAM,CAAC,GAAG,CAAC,KAAK,QAAQ,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,EAAE,CAAC;YAClF,MAAM,CAAC,GAAG,CAAC,GAAG,SAAS,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE,KAAK,CAAC,CAAC;QAC9C,CAAC;aAAM,CAAC;YACN,MAAM,CAAC,GAAG,CAAC,GAAG,KAAK,CAAC;QACtB,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC"}

package/dist/orchestrator/templates/specs-test-data.ts

@@ -0,0 +1,66 @@
+import * as fs from 'fs';
+import * as path from 'path';
+import yaml from 'yaml';
+
+export class TestDataLoader {
+  private data: Record<string, any>;
+
+  private constructor(data: Record<string, any>) {
+    this.data = data;
+  }
+
+  /**
+   * Load test data for a screen/feature combination.
+   *
+   * Priority (later wins):
+   * 1. {feature}.yaml — base data
+   * 2. {feature}.{SUNGEN_ENV}.yaml — environment-specific (if SUNGEN_ENV set)
+   */
+  static load(screenName: string, featureName: string): TestDataLoader {
+    const baseDir = path.join(process.cwd(), 'qa', 'screens', screenName, 'test-data');
+    const env = process.env.SUNGEN_ENV;
+
+    let data = loadYamlSync(path.join(baseDir, `${featureName}.yaml`)) || {};
+
+    if (env) {
+      const envData = loadYamlSync(path.join(baseDir, `${featureName}.${env}.yaml`));
+      if (envData) data = deepMerge(data, envData);
+    }
+
+    return new TestDataLoader(data);
+  }
+
+  get(key: string): string {
+    const parts = key.split('.');
+    let current: any = this.data;
+    for (const part of parts) {
+      if (current == null || typeof current !== 'object') {
+        throw new Error(`Test data key not found: ${key} (failed at '${part}')`);
+      }
+      current = current[part];
+    }
+    if (current === undefined || current === null) {
+      throw new Error(`Test data key not found: ${key}`);
+    }
+    return String(current);
+  }
+}
+
+function loadYamlSync(filePath: string): Record<string, any> | null {
+  if (!fs.existsSync(filePath)) return null;
+  const content = fs.readFileSync(filePath, 'utf-8');
+  return yaml.parse(content) || null;
+}
+
+function deepMerge(base: Record<string, any>, override: Record<string, any>): Record<string, any> {
+  const result = { ...base };
+  for (const [key, value] of Object.entries(override)) {
+    if (value && typeof value === 'object' && !Array.isArray(value) &&
+        result[key] && typeof result[key] === 'object' && !Array.isArray(result[key])) {
+      result[key] = deepMerge(result[key], value);
+    } else {
+      result[key] = value;
+    }
+  }
+  return result;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sun-asterisk/sungen",
-  "version": "2.4.6",
+  "version": "2.5.0",
   "description": "Deterministic E2E Test Compiler - Gherkin + Selectors → Playwright tests",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/cli/commands/generate.ts

@@ -58,6 +58,7 @@ export function registerGenerateCommand(program: Command): void {
     .option('-s, --screen <name>', 'Generate tests for a specific screen')
     .option('--all', 'Generate tests for all screens')
     .option('--framework <name>', 'Test framework (default: playwright)', 'playwright')
+    .option('--inline-data', 'Hardcode test data at compile time instead of runtime loading')
     .action(async (options) => {
       try {
         const screenName = options.screen;
@@ -89,6 +90,7 @@ export function registerGenerateCommand(program: Command): void {
           framework: options.framework || 'playwright',
           screenName,
           verbose: program.opts().verbose,
+          runtimeData: !options.inlineData,
         });
 
         const results = await generator.generateAllTests(

package/src/cli/index.ts

@@ -18,7 +18,7 @@ async function main() {
   program
     .name('sungen')
     .description('Deterministic E2E Test Compiler — Gherkin + Selectors → Playwright')
-    .version('2.4.6');
+    .version('2.5.0');
 
   // Global options
   program

package/src/generators/gherkin-parser/index.ts

@@ -34,6 +34,7 @@ export interface ParsedScenario {
   steps: ParsedStep[];
   stepsName?: string;   // set when scenario has @steps:<name> — marks it as a reusable block
   extendsName?: string; // set when scenario has @extend:<name> — merges base steps inline
+  hookType?: 'beforeAll' | 'afterEach' | 'afterAll'; // set when scenario has @beforeAll/@afterEach/@afterAll
 }
 
 export interface ParsedFeature {
@@ -120,12 +121,15 @@ export class GherkinParser {
         const tags = scenario.tags.map((tag) => tag.name);
         const stepsName = tags.find(t => t.startsWith('@steps:'))?.replace('@steps:', '') || undefined;
         const extendsName = tags.find(t => t.startsWith('@extend:'))?.replace('@extend:', '') || undefined;
+        const hookTag = tags.find(t => /^@(beforeAll|afterEach|afterAll)$/.test(t));
+        const hookType = hookTag?.replace('@', '') as ParsedScenario['hookType'];
         return {
           name: scenario.name,
           tags,
           steps: scenario.steps.map((step) => this.parseStep(step)),
           stepsName,
           extendsName,
+          hookType,
         };
       });
   }

package/src/generators/test-generator/adapters/adapter-interface.ts

@@ -13,6 +13,14 @@ export interface TestFileData {
   featureName: string;
   featureDescription?: string;
   background?: string;
+  beforeAll?: string;
+  afterEach?: string;
+  afterAll?: string;
+  cleanupConfig?: string; // Pre-formatted autoCleanup config from @cleanup:* tags
+  screenshotOnFailure?: boolean; // @screenshot:on-failure tag
+  runtimeData?: boolean; // --runtime-data flag: testData.get() instead of hardcoded values
+  screenName?: string; // Screen name for TestDataLoader.load()
+  featureFileName?: string; // Feature file name for TestDataLoader.load()
   scenarios: string[];
   authGroups?: AuthGroup[]; // Grouped by auth role for nested describes
   singleAuthRole?: string;  // Auth role when all scenarios share the same role
@@ -49,8 +57,11 @@ export interface TestGeneratorAdapter {
   // Template rendering methods
   renderTestFile(data: TestFileData): string;
   renderScenario(data: ScenarioData): string;
-  renderImports(): string;
+  renderImports(options?: { runtimeData?: boolean }): string;
   renderBeforeEach(data: { steps: Array<{ comment?: string; code: string }> }): string;
+  renderBeforeAll(data: { steps: Array<{ comment?: string; code: string }> }): string;
+  renderAfterEach(data: { steps: Array<{ comment?: string; code: string }> }): string;
+  renderAfterAll(data: { steps: Array<{ comment?: string; code: string }> }): string;
 
   // Step rendering
   renderStep(templateName: string, data: any): string;

package/src/generators/test-generator/adapters/playwright/playwright-adapter.ts

@@ -26,14 +26,26 @@ export class PlaywrightAdapter implements TestGeneratorAdapter {
     return this.templateEngine.renderScenario(data);
   }
 
-  renderImports(): string {
-    return this.templateEngine.renderImports();
+  renderImports(options?: { runtimeData?: boolean }): string {
+    return this.templateEngine.renderImports(options);
   }
 
   renderBeforeEach(data: { steps: Array<{ comment?: string; code: string }> }): string {
     return this.templateEngine.renderBeforeEach(data);
   }
 
+  renderBeforeAll(data: { steps: Array<{ comment?: string; code: string }> }): string {
+    return this.templateEngine.renderBeforeAll(data);
+  }
+
+  renderAfterEach(data: { steps: Array<{ comment?: string; code: string }> }): string {
+    return this.templateEngine.renderAfterEach(data);
+  }
+
+  renderAfterAll(data: { steps: Array<{ comment?: string; code: string }> }): string {
+    return this.templateEngine.renderAfterAll(data);
+  }
+
   renderStep(templateName: string, data: any): string {
     return this.templateEngine.renderStep(templateName, data);
   }

package/src/generators/test-generator/adapters/playwright/templates/after-all.hbs

@@ -0,0 +1,8 @@
+  test.afterAll(async ({ request }) => {
+{{#each steps}}
+{{#if comment}}
+    // {{comment}}
+{{/if}}
+{{code}}
+{{/each}}
+  });

package/src/generators/test-generator/adapters/playwright/templates/after-each.hbs

@@ -0,0 +1,8 @@
+  test.afterEach(async ({ page }) => {
+{{#each steps}}
+{{#if comment}}
+    // {{comment}}
+{{/if}}
+{{code}}
+{{/each}}
+  });

package/src/generators/test-generator/adapters/playwright/templates/before-all.hbs

@@ -0,0 +1,8 @@
+  test.beforeAll(async ({ request }) => {
+{{#each steps}}
+{{#if comment}}
+    // {{comment}}
+{{/if}}
+{{code}}
+{{/each}}
+  });

package/src/generators/test-generator/adapters/playwright/templates/imports.hbs

@@ -1,4 +1,7 @@
 import { test, expect } from '../base';
+{{#if runtimeData}}
+import { TestDataLoader } from '../test-data';
+{{/if}}
 
 // This file is auto-generated from Gherkin feature files
 // DO NOT EDIT MANUALLY - changes will be overwritten