@activate-core/test-forge 0.18.6 → 0.18.8

package/dist/cli/index.js

@@ -92,6 +92,8 @@ function printUsage() {
     console.log('  testforge env               Write .env.testing.example from docs');
     console.log('  testforge run [--area a] [--journey j]  Run the generated tests (Jest/Playwright)');
     console.log('  testforge run --env-file <file>       Run against another server (e.g. .env.dev / .env.staging)');
+    console.log('  testforge run --headed | --ui | --debug   Watch the browser / open the Playwright UI');
+    console.log('  testforge run -- <flags>              Pass anything else straight to the runner');
 }
 async function main() {
     if (command === 'update') {
@@ -125,10 +127,19 @@ async function main() {
         runEnvTemplate('docs', '.env.testing.example', '.', generatedScanDirs());
     }
     else if (command === 'run') {
+        // Anything after `--`, plus the runner flags people reach for most, goes
+        // straight to the test runner. Watching a UI suite is an ordinary thing to
+        // want and should not mean rebuilding the command by hand.
+        const dashIdx = process.argv.indexOf('--');
+        const passThrough = [
+            ...['--headed', '--ui', '--debug'].filter((f) => process.argv.includes(f)),
+            ...(dashIdx === -1 ? [] : process.argv.slice(dashIdx + 1)),
+        ];
         const code = runTests({
             area: flagValue('--area'),
             journey: flagValue('--journey'),
             envFile: flagValue('--env-file'),
+            passThrough,
         });
         process.exitCode = code;
     }

package/dist/core/ai-generator.js

@@ -225,6 +225,18 @@ export function buildPrompt(journey, map, testPath, focus, harnessPath, transpor
         `    obvious controller, and it may set the field under a different`,
         `    name. Only when no code path anywhere produces the state is the`,
         `    scenario imaginary — then drop it and say what you searched.`,
+        `    AND IT MUST SATISFY THE SCHEMA, NOT YOUR IDEA OF IT. A row is`,
+        `    written by a database, which refuses it outright if a NOT NULL`,
+        `    column with no default is missing, or if an id is of the wrong`,
+        `    type. Take both from the schema itself. In particular, DO NOT`,
+        `    ASSUME ONE ID TYPE COVERS THE PROJECT: a users table keyed by uuid`,
+        `    commonly sits beside tables keyed by integer or bigint, and a uuid`,
+        `    supplied for one of those is rejected by the database — every test`,
+        `    that touches that fixture then fails for a reason that has nothing`,
+        `    to do with the product. The same applies to a query: an array cast`,
+        `    (\`= ANY($1::uuid[])\`) must name the column's real type. When you`,
+        `    invent a fixture id, make it of the column's type, and prefer`,
+        `    letting the database assign it and reading it back.`,
         `(9) A TIMEOUT IS NOT A BUG REPORT — AND IT IS USUALLY YOU WAITING FOR`,
         `    THE WRONG THING. Before you accept a wait, ask what event actually`,
         `    arrives. A page whose job is to REDIRECT never fires a load event on`,

package/dist/core/grounding.js

@@ -62,6 +62,17 @@ function buildGroundingPrompt(journey, map, srcDir) {
         `    returns, and the status codes the handler can emit.`,
         `  DATA — the models/collections written or read, and the fields that`,
         `    matter to this journey (including enum values where they exist).`,
+        `    For each table a test must SEED, record two things exactly as the`,
+        `    schema declares them, never as the ORM or another table suggests:`,
+        `      * every column that is NOT NULL and has NO default — a seed that`,
+        `        omits one is rejected outright, and the row never exists;`,
+        `      * the TYPE of the primary key and of every foreign key a test`,
+        `        will supply or query by. Do not assume one id type covers the`,
+        `        schema: a users table keyed by uuid sits happily beside tables`,
+        `        keyed by integer or bigint, and a uuid handed to one of those`,
+        `        is refused by the database, not merely unmatched.`,
+        `    Read this from the migrations, the schema dump, or the models'`,
+        `    declarations — whichever the project actually keeps.`,
         `  EXTERNALS — any third-party client the code calls, the module name`,
         `    a test would have to stand in for, and the env var its base URL or`,
         `    key comes from.`,

package/dist/core/harness.js

@@ -54,6 +54,17 @@ export const HARNESS_CONTRACT = [
     `                                  whose default is ''), write WITHOUT update`,
     `                                  validators (runValidators:false) rather`,
     `                                  than failing the whole seed.`,
+    `                                  READ THE SCHEMA BEFORE WRITING A SEED.`,
+    `                                  A relational database refuses a row that`,
+    `                                  omits a NOT NULL column with no default,`,
+    `                                  so every such column must be supplied —`,
+    `                                  take the list from the migrations or the`,
+    `                                  schema itself, not from the fields the`,
+    `                                  journey happens to mention. And take each`,
+    `                                  id's TYPE from the schema too: one table`,
+    `                                  keyed by uuid does not make them all uuid,`,
+    `                                  and an id of the wrong type is rejected`,
+    `                                  outright rather than simply not matching.`,
     `  - TEST_IDS: object              the stable well-known test ids/handles.`,
     `                                  EVERY id MUST be env-overridable: resolve`,
     `                                  each as process.env.TEST_<NAME>_ID ||`,
@@ -304,6 +315,12 @@ export function playwrightConfig(testDirRelToPackage, globalSetupRelToPackage, c
         `  testMatch: /.*\\.(test|spec)\\.[jt]s$/,`,
         `  globalSetup: require.resolve('${globalSetupRelToPackage}'),`,
         `  timeout: 60000,`,
+        `  // Serial, like the Jest config: these suites share ONE test database`,
+        `  // and seed/reset it, so parallel workers race on the same rows and the`,
+        `  // same run gives different answers twice. It also keeps --headed to a`,
+        `  // single browser window instead of one per worker.`,
+        `  fullyParallel: false,`,
+        `  workers: 1,`,
         `  // Console line output. The last reporter auto-publishes the run to the`,
         `  // TestForge dashboard (needs TESTFORGE_KEY in env + project/dashboardUrl`,
         `  // in testforge.config.json).`,

package/dist/core/run-tests.js

@@ -59,7 +59,25 @@ export function runTests(opts = {}) {
         if (!cmd) {
             if (framework === 'playwright') {
                 const target = opts.journey ? `${rel}/${opts.journey}` : rel;
-                cmd = `npx playwright test ${target}`;
+                // Name the config explicitly. It is written beside the AREA's tests
+                // (tests/Feature/playwright.config.js for tests/Feature/Web), not at
+                // the package root, so Playwright's own discovery finds nothing and
+                // silently falls back to defaults — no globalSetup, no reporters, and
+                // therefore no run published to the dashboard. Walk up from the test
+                // directory to the package root and take the first config found.
+                let cfgFlag = '';
+                for (let dir = rel;; dir = dirname(dir)) {
+                    const hit = ['playwright.config.js', 'playwright.config.ts']
+                        .map((f) => (dir === '.' ? f : join(dir, f)))
+                        .find((f) => existsSync(join(root, f)));
+                    if (hit) {
+                        cfgFlag = ` --config=${hit}`;
+                        break;
+                    }
+                    if (dir === '.' || dir === dirname(dir))
+                        break;
+                }
+                cmd = `npx playwright test${cfgFlag} ${target}`;
             }
             else {
                 cmd = opts.journey
@@ -67,6 +85,8 @@ export function runTests(opts = {}) {
                     : `npx jest ${rel}`;
             }
         }
+        if (opts.passThrough?.length)
+            cmd += ` ${opts.passThrough.join(' ')}`;
         ranAnything = true;
         console.log(`\n=== ${area} (${framework}) ===`);
         console.log(`$ ${cmd}   [cwd: ${root}]`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@activate-core/test-forge",
-  "version": "0.18.6",
+  "version": "0.18.8",
   "description": "AI-powered QA automation. Scans a project, drafts journey docs, and generates seeded Jest/Playwright tests. Works in any project, with any AI CLI.",
   "type": "module",
   "bin": {