@activate-core/test-forge 0.18.5 → 0.18.7

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 ||`,

package/dist/core/run-generate.js

@@ -598,18 +598,23 @@ export async function runGenerate(docsDir, useAI = true, force = false, only, ar
         // AI can't fall back to an uninstalled convention (e.g. supertest).
         const allowedModules = installedModules(testDir);
         const transport = transportHint(testDir, framework);
-        // Playwright is the one framework whose import is mandatory in every spec
-        // (jest's describe/it/expect are ambient, so a jest area generates fine
-        // either way). If it is missing, stop before spending a single AI call.
-        if (framework === 'playwright' && !allowedModules.has('@playwright/test')) {
+        // The area's OWN framework is always allowed, installed or not. Refusing
+        // it confuses two different things: output that reached for a module the
+        // project does not use (genuinely wrong — still refused), and output that
+        // is perfectly correct while the project has simply not installed the
+        // runner it is configured to use. The second is an environment gap, and
+        // throwing away a correct test for it loses real work — the test is
+        // written, and the run says the framework must be installed to run it.
+        const frameworkPkg = framework === 'playwright' ? '@playwright/test' : 'jest';
+        if (!allowedModules.has(frameworkPkg)) {
+            allowedModules.add(frameworkPkg);
             const blocked = blockedAreas.get(journey.area) ?? {
-                pkg: '@playwright/test',
+                pkg: frameworkPkg,
                 dir: areaDir,
                 count: 0,
             };
             blocked.count += 1;
             blockedAreas.set(journey.area, blocked);
-            continue;
         }
         // F10 — selector grounding (Playwright areas only): the real test-hooks
         // in this area's components, so a generated UI test can't invent one.
@@ -838,17 +843,17 @@ export async function runGenerate(docsDir, useAI = true, force = false, only, ar
         });
     }
     for (const [area, b] of blockedAreas) {
+        const browsers = b.pkg === '@playwright/test' ? ' && npx playwright install' : '';
         results.push({
-            journeyName: `${b.count} ${area} journey(s) not attempted — ${b.pkg} is not installed`,
+            journeyName: `${area}: tests written, but ${b.pkg} is not installed`,
             testDir: '',
             testFiles: [],
             seedPath: '',
             status: 'skipped',
             seedStatus: 'none',
             warnings: [
-                `every ${area} spec must import ${b.pkg}, so without it the dependency gate ` +
-                    `refuses each generated file and leaves a deferred stub — hours of AI calls for nothing. ` +
-                    `Install it, then re-run: npm i -D ${b.pkg} && npx playwright install`,
+                `${b.count} ${area} journey(s) were generated normally and are complete. They cannot RUN ` +
+                    `until the runner is installed: npm i -D ${b.pkg}${browsers}`,
             ],
         });
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@activate-core/test-forge",
-  "version": "0.18.5",
+  "version": "0.18.7",
   "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": {