@sun-asterisk/sungen 3.2.9 → 3.2.10-beta.1

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

@@ -48,9 +48,27 @@ export class TestDataLoader {
     if (value === undefined || value === null) {
       throw new Error(`Test data key not found: ${key}`);
     }
+    // A projected array (e.g. `{{q.rows[*].name}}`) serializes as a JSON list so the full
+    // ordered set is preserved for the assert-compare — String([...]) would flatten it to a
+    // lossy comma-join that can't be distinguished from a scalar containing commas.
+    if (Array.isArray(value)) return this.interpolate(JSON.stringify(value));
     return this.interpolate(String(value));
   }
 
+  /**
+   * Resolve a key to its RAW, uncoerced value (array/object/number kept as-is) — for binding
+   * DB query parameters, where type-strict stores (MySQL numeric columns, MongoDB) require the
+   * native type. get() stringifies for Gherkin text; raw() must not. Throws on missing, mirroring
+   * get()'s guard.
+   */
+  raw(key: string): any {
+    const value = this.resolve(key);
+    if (value === undefined || value === null) {
+      throw new Error(`Test data key not found: ${key}`);
+    }
+    return value;
+  }
+
   /**
    * Substitute embedded `{{ref}}` cross-references inside a resolved string value.
    * Each ref is looked up via resolve() (flat key, @cases row column, or dotted path),
@@ -78,6 +96,7 @@ export class TestDataLoader {
    *      `q.count` / `q.length`  → number of rows
    *      `q.first.col` / `q.last.col` / `q[2].col` → a specific row's column
    *      `q.col`                → shorthand for the first row's column
+   *      `q.rows[*].col`        → the `col` of EVERY row, projected to an ordered array
    */
   private resolve(key: string): any {
     // 1. Exact flat key — captured vars (set()) live under a literal, possibly dotted, key.
@@ -88,9 +107,26 @@ export class TestDataLoader {
       return this.data[key];
     }
     // 2. Structured path: head from the row (cases) or shared data, then walk segments.
-    const tokens = String(key).replace(/\[(\d+)\]/g, '.$1').split('.');
-    let cur: any = (this.row && tokens[0] in this.row) ? this.row[tokens[0]] : this.data[tokens[0]];
-    for (let i = 1; i < tokens.length && cur != null; i++) cur = TestDataLoader.step(cur, tokens[i]);
+    //    `[n]` → `.n` (numeric index); `[*]` → `.*` (project the rest of the path over each element).
+    const tokens = String(key).replace(/\[(\d+)\]/g, '.$1').replace(/\[\*\]/g, '.*').split('.');
+    const cur: any = (this.row && tokens[0] in this.row) ? this.row[tokens[0]] : this.data[tokens[0]];
+    return TestDataLoader.walk(cur, tokens, 1);
+  }
+
+  /**
+   * Walk the remaining path tokens from `i`. A `*` token on an array projects: the rest of the
+   * path is mapped over every element and the collected array returned (recurses for nested `[*]`).
+   * Non-`*` tokens use the linear single-step navigation, so numeric/first/last/object access is
+   * unchanged.
+   */
+  private static walk(cur: any, tokens: string[], i: number): any {
+    for (; i < tokens.length && cur != null; i++) {
+      if (tokens[i] === '*' && Array.isArray(cur)) {
+        const rest = tokens.slice(i + 1);
+        return cur.map((el) => TestDataLoader.walk(el, rest, 0));
+      }
+      cur = TestDataLoader.step(cur, tokens[i]);
+    }
     return cur;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sun-asterisk/sungen",
-  "version": "3.2.9",
+  "version": "3.2.10-beta.1",
   "description": "Deterministic E2E Test Compiler - Gherkin + Selectors → Playwright tests",
   "main": "src/index.ts",
   "types": "src/index.ts",
@@ -12,7 +12,7 @@
     "copy-templates": "mkdir -p dist/generators/test-generator/adapters/playwright/templates/steps && mkdir -p dist/generators/test-generator/templates && mkdir -p dist/orchestrator/templates && mkdir -p dist/dashboard/templates && cp -r src/generators/test-generator/adapters/playwright/templates/*.hbs dist/generators/test-generator/adapters/playwright/templates/ 2>/dev/null || true && cp -r src/generators/test-generator/adapters/playwright/templates/steps dist/generators/test-generator/adapters/playwright/templates/ && mkdir -p dist/generators/test-generator/adapters/appium/templates/steps && cp -r src/generators/test-generator/adapters/appium/templates/*.hbs dist/generators/test-generator/adapters/appium/templates/ 2>/dev/null || true && cp -r src/generators/test-generator/adapters/appium/templates/steps dist/generators/test-generator/adapters/appium/templates/ && cp src/generators/test-generator/templates/*.hbs dist/generators/test-generator/templates/ 2>/dev/null || true && cp -r src/orchestrator/templates/* dist/orchestrator/templates/ && cp src/dashboard/templates/index.html dist/dashboard/templates/index.html && mkdir -p dist/harness/catalog && cp src/harness/catalog/*.yaml dist/harness/catalog/",
     "build:dashboard": "cd ../../dashboard && npm install --silent && npm run build && cd - && cp ../../dashboard/dist/index.html src/dashboard/templates/index.html",
     "dev": "tsx src/cli/index.ts",
-    "test": "tsx tests/golden/run.ts && tsx tests/audit/run.ts && tsx tests/ingest/run.ts && tsx tests/eval/run.ts && tsx tests/exporter/run.ts && tsx tests/exporter/feature-parser-category.run.ts && tsx tests/exporter/api-detail-sheet.run.ts && tsx tests/exporter/overview-sheet.run.ts && tsx tests/exporter/delivery-divider-tcid.run.ts && tsx tests/exporter/note-expected-actual.run.ts && tsx tests/exporter/test-data-vars.run.ts && tsx tests/exporter/mobile-app-id.run.ts && tsx tests/exporter/api-testcase-cells.run.ts && tsx tests/exporter/delivery-cases-result.run.ts && tsx tests/exporter/precondition-auth-role.run.ts && tsx tests/exporter/report-builder-parity.run.ts && tsx tests/exporter/delivery-preflight-testdata.run.ts && tsx tests/dashboard/api-flows-discovery.run.ts && tsx tests/dashboard/status-results-fallback.run.ts && tsx tests/codegen/verb-ref-masking.run.ts && tsx tests/codegen/table-aria-hidden-locators.run.ts && tsx tests/api-runtime/base-path-url-join.run.ts && tsx tests/api-runtime/file-upload-multipart.run.ts && tsx tests/api-runtime/schema-assertion.run.ts && tsx tests/runtime/test-data-cross-ref.run.ts && tsx tests/capabilities/run.ts && tsx tests/openapi/run.ts && tsx tests/api-field-coverage/run.ts && tsx tests/packaging/run.ts && tsx tests/generate-hint/run.ts && tsx tests/template-assertion/run.ts && tsx tests/journey/run.ts && tsx tests/harness/serial-cascade.run.ts && tsx tests/ai-skills/cross-assistant-orphan.run.ts && tsx tests/init/codex-mcp-config.run.ts && tsx src/orchestrator/ai-skills/golden-skills.test.ts && tsx tests/db-runtime/sql-placeholder-rewrite.run.ts && tsx tests/db-runtime/fallback-flow-codified.run.ts && tsx tests/db-runtime/mysql-integration.opt-in.run.ts",
+    "test": "tsx tests/golden/run.ts && tsx tests/audit/run.ts && tsx tests/ingest/run.ts && tsx tests/eval/run.ts && tsx tests/exporter/run.ts && tsx tests/exporter/feature-parser-category.run.ts && tsx tests/exporter/api-detail-sheet.run.ts && tsx tests/exporter/overview-sheet.run.ts && tsx tests/exporter/delivery-divider-tcid.run.ts && tsx tests/exporter/note-expected-actual.run.ts && tsx tests/exporter/test-data-vars.run.ts && tsx tests/exporter/mobile-app-id.run.ts && tsx tests/exporter/api-testcase-cells.run.ts && tsx tests/exporter/delivery-cases-result.run.ts && tsx tests/exporter/precondition-auth-role.run.ts && tsx tests/exporter/report-builder-parity.run.ts && tsx tests/exporter/delivery-preflight-testdata.run.ts && tsx tests/exporter/delivery-preflight-selectors.run.ts && tsx tests/dashboard/api-flows-discovery.run.ts && tsx tests/dashboard/status-results-fallback.run.ts && tsx tests/codegen/verb-ref-masking.run.ts && tsx tests/codegen/table-aria-hidden-locators.run.ts && tsx tests/api-runtime/base-path-url-join.run.ts && tsx tests/api-runtime/file-upload-multipart.run.ts && tsx tests/api-runtime/schema-assertion.run.ts && tsx tests/runtime/test-data-cross-ref.run.ts && tsx tests/runtime/test-data-array-projection.run.ts &&tsx tests/capabilities/run.ts && tsx tests/openapi/run.ts && tsx tests/api-field-coverage/run.ts && tsx tests/packaging/run.ts && tsx tests/generate-hint/run.ts && tsx tests/template-assertion/run.ts && tsx tests/journey/run.ts && tsx tests/harness/serial-cascade.run.ts && tsx tests/ai-skills/cross-assistant-orphan.run.ts && tsx tests/init/codex-mcp-config.run.ts && tsx src/orchestrator/ai-skills/golden-skills.test.ts && tsx tests/db-runtime/sql-placeholder-rewrite.run.ts && tsx tests/db-runtime/fallback-flow-codified.run.ts && tsx tests/db-runtime/cosmos-engine.run.ts && tsx tests/db-runtime/mongodb-engine.run.ts && tsx tests/db-runtime/dynamodb-engine.run.ts && tsx tests/db-runtime/mysql-integration.opt-in.run.ts && tsx tests/db-runtime/mongodb-integration.opt-in.run.ts && tsx tests/db-runtime/dynamodb-integration.opt-in.run.ts",
     "test:update": "tsx tests/golden/run.ts --update && tsx tests/audit/run.ts --update && tsx tests/ingest/run.ts --update",
     "prepublishOnly": "npm run build:dashboard && npm run build"
   },
@@ -39,7 +39,7 @@
     "@babel/types": "^7.28.5",
     "@cucumber/gherkin": "^37.0.0",
     "@cucumber/messages": "^31.0.0",
-    "@sungen/driver-ui": "3.2.9",
+    "@sungen/driver-ui": "3.2.10-beta.1",
     "chalk": "^5.6.2",
     "commander": "^14.0.2",
     "dotenv": "^17.2.3",
@@ -65,6 +65,9 @@
     "LICENSE"
   ],
   "devDependencies": {
-    "mysql2": "^3"
+    "mysql2": "^3",
+    "mongodb": "^7",
+    "@aws-sdk/client-dynamodb": "^3",
+    "@aws-sdk/lib-dynamodb": "^3"
   }
 }

package/src/capabilities/discover.ts

@@ -27,7 +27,7 @@ import { LOCAL_DRIVERS, registerCoreCapability } from './builtins';
  * move to their packages (R5.5/R5.6) they join this list and leave `LOCAL_DRIVERS`; eventually this
  * becomes a scan of installed `@sungen/driver-*` packages (R5.7).
  */
-const EXTERNAL_DRIVERS = ['@sungen/driver-ui', '@sungen/driver-db', '@sungen/driver-api', '@sungen/driver-mobile'];
+const EXTERNAL_DRIVERS = ['@sungen/driver-ui', '@sungen/driver-db', '@sungen/driver-api', '@sungen/driver-mobile', '@sungen/driver-data-factory'];
 
 /**
  * Bundled (non-opt-in) drivers: shipped as a core dependency (`drivers.yaml` `bundled: true`), so the

package/src/capabilities/registry.ts

@@ -62,6 +62,14 @@ export interface CapabilityDescriptor {
    */
   cliCommands?: Array<(program: unknown) => void>;
 
+  /**
+   * App-profile presets this capability ships (spec: sungen_app_profile_spec.md §3).
+   * Keyed by short name (`radix`, `antd`, `native`); each value is an AppProfile-shaped
+   * plain object that `qa/app.yaml` can pull in via `extends:`. UI knowledge stays in
+   * the UI driver — core only merges data.
+   */
+  appProfilePresets?: Record<string, unknown>;
+
   // --- Declared for later R-steps; not consumed yet (kept here so the SPI shape is visible). ---
   // adapter?: string;                  // codegen adapter id (existing adapterRegistry)
 }
@@ -97,6 +105,11 @@ export class CapabilityRegistry {
     return this.all().flatMap((c) => c.patterns ?? []);
   }
 
+  /** All app-profile presets contributed by registered capabilities (later registration wins on key collision). */
+  appProfilePresets(): Record<string, unknown> {
+    return Object.assign({}, ...this.all().map((c) => c.appProfilePresets ?? {}));
+  }
+
   /** All registered sensors, optionally filtered by kind ('advisory' | 'gate'). */
   sensors(kind?: Sensor['kind']): Sensor[] {
     const all = this.all().flatMap((c) => c.sensors ?? []);

package/src/cli/commands/delivery.ts

@@ -355,11 +355,19 @@ function featureReferencesTestData(featureFile: string): boolean {
 
 function checkSelectorsHasEntries(selectorsFile: string, screen: string): boolean {
   if (!fs.existsSync(selectorsFile)) return false;
-  const content = fs.readFileSync(selectorsFile, 'utf-8');
-  // Very small check: YAML has more than just a single `<screen>: page` entry
-  const topLevelKeys = (content.match(/^[a-zA-Z_][^\s:]*:\s*$/gm) || []).length +
-    (content.match(/^[a-zA-Z_][^\s:]*:\s/gm) || []).length;
-  return topLevelKeys > 1;
+  // Parse the YAML and count real top-level keys. The old regex heuristic required
+  // key names without whitespace, but sungen's own selector-key convention produces
+  // multi-word keys ("email address:", "contact points table:") — a file whose keys
+  // are ALL multi-word counted as 0 entries and falsely blocked the export.
+  try {
+    const parsed = parseYaml(fs.readFileSync(selectorsFile, 'utf-8'));
+    if (!parsed || typeof parsed !== 'object' || Array.isArray(parsed)) return false;
+    // More than just the single `<screen>: page` entry.
+    return Object.keys(parsed as Record<string, unknown>).length > 1;
+  } catch {
+    // Malformed YAML — surface as "missing entries" so preflight points at the file.
+    return false;
+  }
 }
 
 export function hasBlockingMissing(p: PreflightCheck): boolean {

package/src/cli/commands/generate.ts

@@ -99,6 +99,7 @@ export function registerGenerateCommand(program: Command): void {
     .option('--all', 'Generate tests for all screens, flows, and API areas')
     .option('--framework <name>', 'Override the platform driver (else read from qa/capabilities.yaml)')
     .option('--inline-data', 'Hardcode test data at compile time instead of runtime loading')
+    .option('--strict', 'Exit non-zero when compile diagnostics are found (unmatched steps, dropped {{values}})')
     .action(async (options) => {
       try {
         const screenName = options.screen;
@@ -192,6 +193,20 @@ export function registerGenerateCommand(program: Command): void {
 
         console.log(`\n${results.length} test file(s) generated.`);
 
+        // Compile diagnostics (SG-W001 unmatched step, SG-W002 dropped {{value}}, ...).
+        // Warning-only by default; --strict turns any diagnostic into exit 1.
+        const diagnostics = generator.getDiagnostics();
+        if (diagnostics.count > 0) {
+          console.log(`\n⚠️  Compile diagnostics (${diagnostics.count}):`);
+          for (const line of diagnostics.formatLines().slice(0, 20)) console.log(`   ${line}`);
+          if (diagnostics.count > 20) console.log(`   … and ${diagnostics.count - 20} more`);
+          console.log(`   Unmatched steps compile to no-op // TODO comments — the test runs weaker than it reads.`);
+          if (options.strict) {
+            console.error(`\n--strict: failing generation on ${diagnostics.count} diagnostic(s).`);
+            process.exit(1);
+          }
+        }
+
         // Security S0 — warn (never block) if test-data looks to hold a real secret.
         const scanDirs: string[] = [];
         if (screenName) scanDirs.push(path.join(process.cwd(), 'qa', 'screens', screenName));

package/src/exporters/spec-parser.ts

@@ -20,6 +20,10 @@ function extractTestBlock(content: string, startIdx: number): {
   //   test('title', { tag: [...] }, async ({ page }) => {
   // Backreference \1 lets the inner title contain the opposite quote type
   // (e.g. test('Footer "X" link', ...) — common when scenarios cite UI labels).
+  // `\\.` consumes any backslash-escaped char FIRST, so the wrapping quote may also appear
+  // ESCAPED inside the title (codegen emits `row\'s` for an apostrophe in a scenario name).
+  // Without it the title capture stopped at `\'` and the whole test() silently vanished from
+  // the parse — the scenario was then mislabelled "Not compiled"/N/A although it ran and passed.
   // Quote char may be ' " or ` — the backtick form is a data-driven (@cases) title
   // like `VP-… — ${__row.__label}`.
   // The options object uses `\{.*?\}` (non-greedy, not `[^}]*`) so a tag value containing a
@@ -27,13 +31,14 @@ function extractTestBlock(content: string, startIdx: number): {
   // single-line, so `.*?` anchored on `}, async` is safe.
   // Accept both Playwright `test(...)` and mobile/Appium mocha `it(...)` (issue #391). The leading
   // `\b` prevents `it(` from matching the tail of a larger word like `submit(` / `edit(`.
-  const testRegex = /\b(?:test|it)\s*\(\s*(['"`])((?:(?!\1).)+)\1\s*,\s*(?:\{.*?\}\s*,\s*)?async\s*\([^)]*\)\s*=>\s*\{/g;
+  const testRegex = /\b(?:test|it)\s*\(\s*(['"`])((?:\\.|(?!\1).)+)\1\s*,\s*(?:\{.*?\}\s*,\s*)?async\s*\([^)]*\)\s*=>\s*\{/g;
   testRegex.lastIndex = startIdx;
   const match = testRegex.exec(content);
   if (!match) return null;
 
-  // [1] = outer quote char, [2] = title
-  const title = match[2];
+  // [1] = outer quote char, [2] = title. Unescape `\'`/`\"`/backtick/backslash: the results.json
+  // title is the RUNTIME string (`row's`), so the escaped source form would never match it.
+  const title = unescapeQuoted(match[2]);
   const bodyStart = match.index + match[0].length;
 
   // Find matching closing brace (accounting for nested braces)
@@ -54,6 +59,11 @@ function extractTestBlock(content: string, startIdx: number): {
   return { title, body, endIdx };
 }
 
+/** Unescape backslash-escaped quote chars (and backslashes) captured from a quoted JS string. */
+function unescapeQuoted(s: string): string {
+  return s.replace(/\\(['"`\\])/g, '$1');
+}
+
 /**
  * Find describe('name', ...) blocks and return their names + body ranges.
  */
@@ -61,10 +71,12 @@ function extractDescribeBlocks(content: string): Array<{ name: string; bodyStart
   const result: Array<{ name: string; bodyStart: number; bodyEnd: number }> = [];
   // Accept Playwright `test.describe(...)` / `test.describe.serial(...)` AND mobile/Appium mocha
   // `describe(...)` (issue #391). The optional `test.` prefix keeps web specs matching unchanged.
-  const regex = /(?:test\.)?describe(?:\.serial)?\s*\(\s*['"]([^'"]+)['"]\s*,\s*\(\s*\)\s*=>\s*\{/g;
+  // Same escaped-quote handling as the test-title regex: a feature name with an apostrophe
+  // (describe('Manager\'s dashboard')) must not drop the whole describe block from the parse.
+  const regex = /(?:test\.)?describe(?:\.serial)?\s*\(\s*(['"])((?:\\.|(?!\1).)+)\1\s*,\s*\(\s*\)\s*=>\s*\{/g;
   let match: RegExpExecArray | null;
   while ((match = regex.exec(content)) !== null) {
-    const name = match[1];
+    const name = unescapeQuoted(match[2]);
     const bodyStart = match.index + match[0].length;
     let depth = 1;
     let i = bodyStart;

package/src/generators/test-generator/adapters/playwright/templates/steps/actions/alert-accept-action.hbs

@@ -1 +1 @@
-page.on('dialog', dialog => dialog.accept());
+page.once('dialog', dialog => dialog.accept());

package/src/generators/test-generator/adapters/playwright/templates/steps/actions/alert-dismiss-action.hbs

@@ -1 +1 @@
-page.on('dialog', dialog => dialog.dismiss());
+page.once('dialog', dialog => dialog.dismiss());

package/src/generators/test-generator/adapters/playwright/templates/steps/actions/alert-fill-action.hbs

@@ -1 +1 @@
-page.on('dialog', dialog => dialog.accept('{{escapeQuotes fillValue}}'));
+page.once('dialog', dialog => dialog.accept('{{escapeQuotes fillValue}}'));

package/src/generators/test-generator/adapters/playwright/templates/steps/actions/capture-row-column.hbs

@@ -0,0 +1,2 @@
+{ const tableRow = {{> locator}}.getByRole('row').nth({{{rowIndexExpr}}});
+testData.set('{{varName}}', ((await {{{cellExpr}}}.textContent()) ?? '').trim()); }

package/src/generators/test-generator/adapters/playwright/templates/steps/actions/click-element-with-text.hbs

@@ -1 +1 @@
-await {{> locator-base}}.filter({ hasText: {{#if (and (eq strategy "text") (eq value ""))}}/.*{{escapeRegex dataValue}}$/{{else}}'{{escapeQuotes dataValue}}'{{/if}} }){{> locator-nth}}.click();
+await {{> locator-base}}.filter({ hasText: {{> has-text-value dataValue=dataValue exact=exact legacyTail=(and (eq strategy "text") (eq value ""))}} }){{> locator-nth}}.click();

package/src/generators/test-generator/adapters/playwright/templates/steps/actions/click-with-alert-action.hbs

@@ -0,0 +1,2 @@
+page.once('dialog', dialog => dialog.{{dialogAction}}());
+await {{> locator}}.click();

package/src/generators/test-generator/adapters/playwright/templates/steps/actions/recipe-sequence.hbs

@@ -0,0 +1,3 @@
+{{#each steps}}
+{{> (lookup this 'partial') this}}
+{{/each}}

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

@@ -0,0 +1,2 @@
+{ const tableRow = {{> locator}}.getByRole('row').nth({{{rowIndexExpr}}});
+await {{{actionLocator}}}.{{action}}(); }

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

@@ -1 +1 @@
-await page.getByText('{{escapeQuotes selectorValue}}').filter({ hasText: {{#if (eq selectorValue "")}}/.*{{escapeRegex dataValue}}$/{{else}}'{{escapeQuotes dataValue}}'{{/if}} }){{#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 "")}} }){{#if (gte nth 0)}}.nth({{nth}}){{/if}}.{{action}}();

package/src/generators/test-generator/adapters/playwright/templates/steps/navigation/route-assertion.hbs

@@ -1 +1,4 @@
 await expect(page).toHaveURL(/{{escapeRegex path}}/);
+{{#if query}}
+expect(page.url()).toContain('{{escapeQuotes query}}');
+{{/if}}

package/src/generators/test-generator/adapters/playwright/templates/steps/navigation/wait-for-element.hbs

@@ -1 +1 @@
-await {{> locator}}.waitFor({ state: '{{state}}' });
+{{#if (eq state "enabled")}}await expect({{> locator}}).toBeEnabled();{{else if (eq state "disabled")}}await expect({{> locator}}).toBeDisabled();{{else}}await {{> locator}}.waitFor({ state: '{{state}}' });{{/if}}

package/src/generators/test-generator/adapters/playwright/templates/steps/navigation/wait-table-refresh.hbs

@@ -0,0 +1,3 @@
+{ const refreshIndicator = page.locator('{{escapeQuotes indicator}}');
+  await refreshIndicator.first().waitFor({ state: 'visible', timeout: 1500 }).catch(() => {});
+  await refreshIndicator.first().waitFor({ state: 'hidden' }); }

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/has-text-value.hbs

@@ -0,0 +1 @@
+{{#if exact}}/^{{escapeRegex dataValue}}$/{{else if legacyTail}}/.*{{escapeRegex dataValue}}$/{{else}}'{{escapeQuotes dataValue}}'{{/if}}

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

@@ -8,7 +8,7 @@
   {{~#case 'id'}}locator('{{value}}'){{/case~}}
   {{~#default}}locator('{{value}}'){{/default~}}
 {{~/switch}}{{else~}}
-{{#if scope}}{{pageRoot}}.getByLabel('{{escapeQuotes scope}}').{{#switch strategy~}}
+{{#if scope}}{{#if (eq scope "dialog")}}{{pageRoot}}.{{else}}{{pageRoot}}.getByLabel('{{escapeQuotes scope}}').{{/if}}{{#switch strategy~}}
   {{~#case 'testid'}}getByTestId('{{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~}}

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

@@ -8,7 +8,7 @@
   {{~#case 'id'}}locator('{{value}}'){{/case~}}
   {{~#default}}locator('{{value}}'){{/default~}}
 {{~/switch}}{{> locator-nth}}{{else~}}
-{{#if scope}}{{pageRoot}}.getByLabel('{{escapeQuotes scope}}').{{#switch strategy~}}
+{{#if scope}}{{#if (eq scope "dialog")}}{{pageRoot}}.{{else}}{{pageRoot}}.getByLabel('{{escapeQuotes scope}}').{{/if}}{{#switch strategy~}}
   {{~#case 'testid'}}getByTestId('{{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~}}

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-blur.hbs

@@ -0,0 +1 @@
+await {{{targetExpr}}}.evaluate((el) => (el instanceof HTMLElement) && el.blur());

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-click-text.hbs

@@ -0,0 +1,3 @@
+{ const __opt = {{{targetExpr}}};
+await __opt.scrollIntoViewIfNeeded().catch(() => {});
+await __opt.click(); }

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-click.hbs

@@ -0,0 +1 @@
+await {{{targetExpr}}}.click();

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-expect-text.hbs

@@ -0,0 +1 @@
+await expect({{{targetExpr}}}).toContainText({{{valueExpr}}});

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-expect-value.hbs

@@ -0,0 +1 @@
+await expect({{{targetExpr}}}).toHaveValue({{{valueExpr}}});

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-for-each.hbs

@@ -0,0 +1,5 @@
+for (const item of String({{{sourceExpr}}}).split('{{split}}').map((s) => s.trim())) {
+{{#each body}}
+{{> (lookup this 'partial') this}}
+{{/each}}
+}

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-hover.hbs

@@ -0,0 +1 @@
+await {{{targetExpr}}}.hover();

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-press.hbs

@@ -0,0 +1 @@
+await page.keyboard.press('{{key}}');

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-scroll-into-view.hbs

@@ -0,0 +1 @@
+await {{{targetExpr}}}.scrollIntoViewIfNeeded();

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-type.hbs

@@ -0,0 +1 @@
+await page.keyboard.type({{{valueExpr}}});

package/src/generators/test-generator/adapters/playwright/templates/steps/partials/recipe-wait-for.hbs

@@ -0,0 +1 @@
+await {{{targetExpr}}}.first().waitFor({ state: '{{state}}' });

package/src/generators/test-generator/app-profile.ts

@@ -0,0 +1,185 @@
+import * as fs from 'fs';
+import * as path from 'path';
+import yaml from 'yaml';
+
+/**
+ * App Profile (`qa/app.yaml`) — the per-project BEHAVIOR layer (spec:
+ * docs/spec/sungen_app_profile_spec.md). Three kinds of knowledge live in three files:
+ *   INTENT   → .feature + test-data      (portable across projects)
+ *   IDENTITY → selectors.yaml            (per-screen: WHERE an element is)
+ *   BEHAVIOR → qa/app.yaml               (per-project: HOW this app's UI behaves)
+ *
+ * The file describes the app under test, never the tool (no timeouts/retries/sleeps —
+ * those belong to sungen.config.yaml). One file per project; presets merge in via
+ * `extends:`; per-element `variant:`/`scope:` in selectors.yaml wins over the profile.
+ * No file → the built-in defaults below → pre-profile behavior stays intact.
+ */
+
+export interface AppProfile {
+  version?: number;
+  extends?: string | string[];
+  app?: { ui_library?: string; framework?: string; locales?: string[]; base_locale?: string };
+  widgets?: Record<string, string | { recipe?: string; variant?: string }>;
+  recipes?: Record<string, unknown>;
+  layers?: { portal_roles?: string[] };
+  feedback?: {
+    toast?: { role?: string; named_by?: 'content' | 'label'; stacking?: string };
+    loading?: { indicator?: string };
+    field_error?: { locate?: string; indicator?: string };
+    confirm?: string;
+  };
+  conventions?: Record<string, unknown>;
+  detection?: Record<string, unknown>;
+}
+
+export interface ProfileWarning {
+  code: 'SG-W005' | 'SG-W006';
+  message: string;
+}
+
+export interface ResolvedAppProfile {
+  /** ARIA roles rendered via portal — never wrapped in dialog/landmark scope (issue #465). */
+  portalRoles: string[];
+  /** Roles whose accessible name derives from CONTENT — defaulting name from the Gherkin
+   *  reference can never match (issue #467). */
+  contentNamedRoles: string[];
+  /** widget kind → variant/recipe name, normalized (`select: radix-select`). Per-element
+   *  `variant:` in selectors.yaml overrides this. */
+  widgetVariants: Record<string, string>;
+  /** recipe name → { intent → primitive list } — built-in variant recipes merged under
+   *  project-defined ones (project wins). */
+  recipes: Record<string, Record<string, unknown[]>>;
+  /** Refresh/loading signal for `wait for [T] table to refresh` (feedback.loading.indicator). */
+  loadingIndicator: string;
+  /** Positional-row conventions (`row {{n}}` steps): first row number + whether the header
+   *  counts as a row in getByRole('row') indexing. */
+  tableRowBase: number;
+  tableHeaderIsRow: boolean;
+  /** The merged profile (presets ← project), or null when qa/app.yaml does not exist. */
+  raw: AppProfile | null;
+  warnings: ProfileWarning[];
+}
+
+/** Portal-rendered option layers (Radix/MUI/Headless UI/antd portal to document.body). */
+export const DEFAULT_PORTAL_ROLES = ['option', 'listbox', 'menu', 'menuitem', 'tooltip'];
+
+/** ARIA name-from-content roles (per spec): a `{ name }` filter invented from the Gherkin
+ *  reference text can never match these. */
+export const DEFAULT_CONTENT_NAMED_ROLES = ['status', 'alert', 'log', 'marquee', 'timer'];
+
+/** Default refresh signal — apps that set aria-busy while loading need zero config. */
+export const DEFAULT_LOADING_INDICATOR = '[aria-busy="true"]';
+
+const KNOWN_TOP_KEYS = ['version', 'extends', 'app', 'widgets', 'recipes', 'layers', 'feedback', 'conventions', 'detection'];
+
+/** Reserved preset key: always merged as the base layer (built-in widget recipes ship here). */
+const BUILTIN_PRESET_KEY = 'builtin';
+
+export function defaultAppProfile(presets: Record<string, unknown> = {}): ResolvedAppProfile {
+  return finalizeProfile((presets[BUILTIN_PRESET_KEY] as AppProfile) ?? {}, null, []);
+}
+
+/** Derive the compiler-consumed slices from a merged profile document. */
+function finalizeProfile(merged: AppProfile, raw: AppProfile | null, warnings: ProfileWarning[]): ResolvedAppProfile {
+  const portalRoles = merged.layers?.portal_roles ?? DEFAULT_PORTAL_ROLES;
+  const contentNamed = new Set(DEFAULT_CONTENT_NAMED_ROLES);
+  if (merged.feedback?.toast?.named_by === 'content' && merged.feedback.toast.role) {
+    contentNamed.add(merged.feedback.toast.role);
+  }
+  const widgetVariants: Record<string, string> = {};
+  for (const [kind, v] of Object.entries(merged.widgets ?? {})) {
+    const name = typeof v === 'string' ? v : (v?.recipe ?? v?.variant);
+    if (name) widgetVariants[kind] = name;
+  }
+  const conventions = merged.conventions as { table?: { row_base?: number; header_is_row?: boolean } } | undefined;
+  return {
+    portalRoles,
+    contentNamedRoles: [...contentNamed],
+    widgetVariants,
+    recipes: (merged.recipes ?? {}) as Record<string, Record<string, unknown[]>>,
+    loadingIndicator: merged.feedback?.loading?.indicator ?? DEFAULT_LOADING_INDICATOR,
+    tableRowBase: conventions?.table?.row_base ?? 1,
+    tableHeaderIsRow: conventions?.table?.header_is_row ?? false,
+    raw,
+    warnings,
+  };
+}
+
+/**
+ * Load and resolve `qa/app.yaml`: merge the `extends` preset chain (project wins),
+ * validate top-level keys, and derive the compiler-consumed slices. Never throws on
+ * a bad profile — problems surface as SG-W005/W006 diagnostics, and each slice falls
+ * back to its built-in default.
+ */
+export function loadAppProfile(cwd: string, presets: Record<string, unknown> = {}): ResolvedAppProfile {
+  const warnings: ProfileWarning[] = [];
+  const qaDir = path.join(cwd, 'qa');
+
+  // Per-environment profiles are unsupported BY DESIGN — widget behavior must not differ
+  // between environments (test-data/locale overlays already cover what legitimately does).
+  if (fs.existsSync(qaDir)) {
+    for (const name of fs.readdirSync(qaDir)) {
+      if (/^app\.[^.]+\.ya?ml$/.test(name)) {
+        warnings.push({
+          code: 'SG-W006',
+          message: `qa/${name}: per-environment app profiles are unsupported by design — describe the app once in qa/app.yaml`,
+        });
+      }
+    }
+  }
+
+  const file = path.join(qaDir, 'app.yaml');
+  if (!fs.existsSync(file)) {
+    return { ...defaultAppProfile(presets), warnings };
+  }
+
+  let doc: AppProfile;
+  try {
+    doc = (yaml.parse(fs.readFileSync(file, 'utf-8')) ?? {}) as AppProfile;
+  } catch (e) {
+    warnings.push({ code: 'SG-W005', message: `qa/app.yaml: unparseable YAML (${e instanceof Error ? e.message : e}) — using built-in defaults` });
+    return { ...defaultAppProfile(presets), warnings };
+  }
+
+  for (const key of Object.keys(doc)) {
+    if (!KNOWN_TOP_KEYS.includes(key)) {
+      warnings.push({ code: 'SG-W005', message: `qa/app.yaml: unknown key "${key}" (known: ${KNOWN_TOP_KEYS.join(', ')})` });
+    }
+  }
+
+  // extends: preset chain, merged in order, project file last (narrow beats broad).
+  // The reserved `builtin` preset (built-in widget recipes from driver-ui) is always
+  // the base layer, so per-element `variant: radix-select` works without any extends.
+  const extendNames = doc.extends ? (Array.isArray(doc.extends) ? doc.extends : [doc.extends]) : [];
+  const availableNames = Object.keys(presets).filter((k) => k !== BUILTIN_PRESET_KEY);
+  const chain: AppProfile[] = presets[BUILTIN_PRESET_KEY] ? [presets[BUILTIN_PRESET_KEY] as AppProfile] : [];
+  for (const name of extendNames) {
+    const key = String(name).replace(/^@sungen\/app-/, '');
+    const preset = presets[key] as AppProfile | undefined;
+    if (!preset || key === BUILTIN_PRESET_KEY) {
+      warnings.push({
+        code: 'SG-W005',
+        message: `qa/app.yaml: unknown preset "${name}" in extends (available: ${availableNames.join(', ') || 'none — is the driver that ships it installed?'})`,
+      });
+    } else {
+      chain.push(preset);
+    }
+  }
+  const merged = [...chain, doc].reduce<AppProfile>((acc, layer) => deepMerge(acc, layer) as AppProfile, {});
+
+  return finalizeProfile(merged, merged, warnings);
+}
+
+/** Plain-object deep merge; arrays and scalars replace (a preset's list is a whole answer). */
+function deepMerge(base: unknown, layer: unknown): unknown {
+  if (!isPlainObject(base) || !isPlainObject(layer)) return layer === undefined ? base : layer;
+  const out: Record<string, unknown> = { ...base };
+  for (const [k, v] of Object.entries(layer)) {
+    out[k] = k in out ? deepMerge(out[k], v) : v;
+  }
+  return out;
+}
+
+function isPlainObject(v: unknown): v is Record<string, unknown> {
+  return typeof v === 'object' && v !== null && !Array.isArray(v);
+}

package/src/generators/test-generator/code-generator.ts

@@ -4,6 +4,7 @@ import { ParsedFeature, ParsedScenario, ParsedStep } from '../gherkin-parser';
 import { StepMapper } from './step-mapper';
 import { TestGeneratorAdapter, adapterRegistry } from './adapters';
 import { transformToRuntimeData } from './utils/runtime-data-transformer';
+import { DiagnosticsCollector } from './diagnostics';
 import { capabilityRegistry } from '../../capabilities/registry';
 import { discoverAndRegisterCapabilities } from '../../capabilities/discover';
 import { readCapabilities } from '../../harness/capability';
@@ -227,10 +228,13 @@ export class CodeGenerator {
   // Steps registry built per feature during generateTestCode(); used by countSteps()
   private stepsRegistry = new Map<string, ParsedScenario>();
 
+  private diagnostics = new DiagnosticsCollector();
+
   constructor(options: { useAI?: boolean; verbose?: boolean; framework?: string; baseURL?: string; screenName?: string; runtimeData?: boolean; flowMode?: boolean } = {}) {
     this.options = options;
     this.screenName = options.screenName;
     this.stepMapper = new StepMapper(options);
+    this.stepMapper.setDiagnostics(this.diagnostics);
     // Get adapter from registry. Platform comes from the caller (resolved from
     // qa/capabilities.yaml); default `web` = the bundled Playwright adapter
     // (back-compat baseline). External platforms (e.g. mobile) are loaded by the
@@ -243,6 +247,11 @@ export class CodeGenerator {
     }
   }
 
+  /** Compile diagnostics collected across all generateTest calls (SG-W001/W002...) */
+  getDiagnostics(): DiagnosticsCollector {
+    return this.diagnostics;
+  }
+
   /**
    * Generate test file from parsed feature
    */