@gfdelarue/testkit 0.5.0 → 0.6.0

package/docs/test-types.md

@@ -11,12 +11,15 @@ so does any Node-side test that reads its environment from `process.env` or
 [Plain Vitest test files](../README.md#plain-vitest-test-files) for which files
 Testkit runs.
 
-Typed files carry one of three product-level test types in their filename
-suffix:
+A plain Vitest test records data-flow evidence, such as metrics, sequences,
+records, and budgets, with `dataFlow(context)` from
+`@gfdelarue/testkit/node-vitest/data`. Recording evidence gives the test no
+type and no setup. See
+[Record data-flow evidence](../README.md#record-data-flow-evidence).
+
+Typed files carry one product-level test type in their filename suffix:
 
 - `ui`: browser-level UI checks for user-visible pages and interactions.
-- `contract`: Node-backed service/runtime contract checks that need Node APIs but are not data-flow evidence suites.
-- `data`: data-flow and data-contract checks for throughput, payload shape, conversation/request traces, and cost/latency budgets.
 
 Consumers choose the type that describes the product behavior and declare the
 execution engine in each file's `defineFile(...)` metadata. Testkit validates
@@ -24,12 +27,10 @@ that pair against its engine catalog and resolves the exact runner profile.
 
 The current catalog is deliberately explicit:
 
-| Test type          | Engine        | Runner profile         |
-| ------------------ | ------------- | ---------------------- |
-| none, a plain file | `node-vitest` | `node-vitest`          |
-| `data`             | `node-vitest` | `node-vitest-data`     |
-| `contract`         | `node-vitest` | `node-vitest-contract` |
-| `ui`               | `playwright`  | `playwright-ui`        |
+| Test type          | Engine        | Runner profile  |
+| ------------------ | ------------- | --------------- |
+| none, a plain file | `node-vitest` | `node-vitest`   |
+| `ui`               | `playwright`  | `playwright-ui` |
 
 Every typed file must import `defineFile` from the root package and export one
 direct metadata declaration:
@@ -37,7 +38,7 @@ direct metadata declaration:
 ```ts
 import { defineFile } from "@gfdelarue/testkit";
 
-export const testkit = defineFile({ engine: "node-vitest" });
+export const testkit = defineFile({ engine: "playwright" });
 ```
 
 The engine is not inferred from the suffix, imports, or repo configuration. A
@@ -46,9 +47,9 @@ error. The profile is derived from the validated pair and is carried unchanged
 through scheduling, results, and immutable run evidence.
 
 Run selection by `--file`, `--group`, `--type`, or `--suite` narrows the files
-whose engine metadata is read and validated. A `--type` that names test types
-and typed suite selectors select typed files only; `--type all` includes plain
-files. Testkit uses discovered paths, canonical type
+whose engine metadata is read and validated. `--type ui` and typed suite
+selectors select typed files only, `--type plain` selects plain files only, and
+`--type all` selects both. Testkit uses discovered paths, canonical type
 suffixes, and suite names to select candidates; each selected typed file must
 still declare a valid engine. Invalid metadata in an unselected file
 does not block that focused run. Root configuration, service dependencies, and
@@ -56,11 +57,7 @@ group skip, requirement, and isolation selectors remain validated against
 the complete discovered project. `discover`, `doctor`, and `typecheck` validate
 metadata across the whole project.
 
-Canonical filename suffixes:
-
-- `*.ui.testkit.ts`
-- `*.contract.testkit.ts`
-- `*.data.testkit.ts`
+The canonical filename suffix is `*.ui.testkit.ts`.
 
 UI tests may import Playwright's public browser primitives directly from the
 managed native package:
@@ -72,56 +69,9 @@ import { expect, test } from "@playwright/test";
 export const testkit = defineFile({ engine: "playwright" });
 ```
 
-Data suites import data-flow primitives from Testkit:
-
-```ts
-import { defineFile } from "@gfdelarue/testkit";
-import { dataScenario } from "@gfdelarue/testkit/node-vitest/data";
-import { expect } from "vitest";
-
-export const testkit = defineFile({ engine: "node-vitest" });
-```
-
-Contract suites import Node-backed contract primitives from Testkit:
-
-```ts
-import { defineFile } from "@gfdelarue/testkit";
-import { contractScenario } from "@gfdelarue/testkit/node-vitest/contract";
-import { expect } from "vitest";
-
-export const testkit = defineFile({ engine: "node-vitest" });
-```
-
-Use `contract` when the behavior under test is a service/runtime boundary that
-needs Node APIs, fixtures, file IO, or richer protocol checks, but does not need
-Testkit data-flow evidence. Contract files are native Vitest suites:
-`contractScenario(...)` provides the managed Testkit contract context, while
-direct public runner APIs remain available for native behavior such as
-`test.skip(...)` and `test.todo(...)`. Contract suites run as Node processes
-managed by Testkit and should talk to configured service URLs.
-
-Use `data` when the behavior under test is the shape, cost, volume, latency, or
-movement of data through a real product flow. A data suite is not a generic
-Vitest migration target: write `dataScenario(...)` cases that record Testkit
-structured data-flow evidence with the provided data context, such as metrics,
-budgets, traces, fixtures, HTTP payload measurements, cache behavior, LLM usage,
-actors, systems, records, transitions, external calls, and reconciliation
-evidence.
-Data scenarios use Vitest directly for assertions, hooks, and mocking.
-`dataScenario(...)` owns case registration and evidence: lint reports native
-registration imports, and the runtime rejects cases registered any other way.
-Weak bookkeeping such as a single `count(...)`, `event(...)`, or `finding(...)`
-is useful as supporting detail, but it is not enough by itself to make a suite a
-data-flow suite.
-
-Data suites run as Node processes managed by Testkit's `node-vitest` engine and
-should use configured service URLs for data-flow evidence. If a check requires
-service-local loopback, `serviceContextUrl`, DOM rendering, browser automation,
-or raw app-server process access, use a plain Vitest test, `contract`, or `ui` instead.
-
-Use a plain Vitest test file for unit tests and other Node-side checks that need
-no Testkit context, `contract` for typed service or protocol contracts, and `ui`
-when the browser is part of the assertion. Plain Vitest files can exercise HTTP,
+Use a plain Vitest test file for unit tests, other Node-side checks, and
+service or protocol contracts, and `ui` when the browser is part of the
+assertion. Plain Vitest files can exercise HTTP,
 persistence, and service-spanning flows without a type label.
 
 The retired `@gfdelarue/testkit/runtime`, `@gfdelarue/testkit/data`,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gfdelarue/testkit",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "description": "Test orchestration platform for product acceptance and QA workflows",
   "license": "SEE LICENSE IN LICENSE",
   "type": "module",
@@ -50,10 +50,6 @@
       "types": "./dist/types/node-vitest-data.d.ts",
       "default": "./dist/engines/node-vitest/runtime/data/index.js"
     },
-    "./node-vitest/contract": {
-      "types": "./dist/types/node-vitest-contract.d.ts",
-      "default": "./dist/engines/node-vitest/runtime/contract/index.js"
-    },
     "./workspace": {
       "types": "./dist/types/workspace.d.ts",
       "default": "./dist/app/inspect/index.js"
@@ -88,6 +84,7 @@
     "dist/",
     "docs/bug-records.md",
     "docs/engines.md",
+    "docs/release-notes.md",
     "docs/test-types.md",
     "testkit-config-schema.md"
   ],

package/testkit-config-schema.md

@@ -94,12 +94,10 @@ literal values `"node-vitest"` or `"playwright"`.
 The filename suffix determines the test type, and each engine and test type
 pair runs under exactly one runner profile:
 
-| Test type          | Engine        | Profile                |
-| ------------------ | ------------- | ---------------------- |
-| none, a plain file | `node-vitest` | `node-vitest`          |
-| `data`             | `node-vitest` | `node-vitest-data`     |
-| `contract`         | `node-vitest` | `node-vitest-contract` |
-| `ui`               | `playwright`  | `playwright-ui`        |
+| Test type          | Engine        | Profile         |
+| ------------------ | ------------- | --------------- |
+| none, a plain file | `node-vitest` | `node-vitest`   |
+| `ui`               | `playwright`  | `playwright-ui` |
 
 For a typed file, missing metadata, dynamic values, and unsupported
 engine/test-type pairs are discovery errors. There is no default-engine or legacy-runner compatibility
@@ -121,20 +119,29 @@ runtime-policy modules under `src/` just to cooperate with managed runs.
 
 ## Top-Level Fields
 
+The top-level keys are `areas`, `artifacts`, `bugs`, `discovery`, `execution`,
+`fingerprints`, `groups`, `lint`, `redact`, `services`, `toolchains`, and `ui`.
+Testkit refuses a config with any other top-level key. The error names the key
+and the closest known key when one is close.
+
 ### `execution`
 
 Default execution settings for the repo.
 
-- `taskConcurrency`: maximum concurrent test tasks across the run (default `1`)
+- `taskConcurrency`: maximum concurrent test tasks across the run (default `1`).
+  A typed file, a batch of typed files, or one Vitest invocation of plain files
+  is one task. Plain files in one invocation run with Vitest's file
+  parallelism, which the Vitest config's `fileParallelism` and `maxWorkers`
+  control.
 - `fileTimeoutSeconds`
 
-`execution.workers` and `presets.nodeNext({ workers })` remain deprecated aliases
-through 0.2.x and will be removed in 0.3.0. Use `taskConcurrency` in new config.
+`execution.workers` and `presets.nodeNext({ workers })` are deprecated aliases.
+Use `taskConcurrency` in new config.
 If both names are present, their values must match.
 
-`--task-concurrency` overrides the configured value. The CLI alias `--workers`
-also remains available through 0.2.x; contradictory CLI aliases are rejected
-before applying CLI precedence. Contradictory config aliases are rejected even
+`--task-concurrency` overrides the configured value. The deprecated CLI alias
+`--workers` also works; contradictory CLI aliases are rejected before applying
+CLI precedence. Contradictory config aliases are rejected even
 when the CLI supplies an override.
 
 Run documents keep the configured `run.workers` field. Observed
@@ -149,8 +156,9 @@ helpers like `waitFor()` consume this budget automatically.
 Optional `testkit lint` settings. Omit it to run the default rules.
 
 - `rules`: map of rule name to `true`, `false`, or an options object.
-  `engineImports`, `configImports`, `dataSuiteContract`, and `uiSpecShape` default to enabled; `unitTestBoundaries` and
-  `testkitBoundary` default to disabled.
+  `engineImports`, `configImports`, and `uiSpecShape` default to enabled; `unitTestBoundaries` and
+  `testkitBoundary` default to disabled. `dataSuiteContract` was removed in
+  0.6.0, and `testkit lint` rejects a config that sets it.
 - Rule options: `enabled` (default `true`), `include`, `exclude`. `testkitBoundary`
   also accepts `allowed` and `trackedOnly`.
 - `ui`: `maxLines` (default `220`) and `maxTests` (default `8`) for UI suites.
@@ -158,16 +166,11 @@ Optional `testkit lint` settings. Omit it to run the default rules.
 Enabled-rule findings and syntax errors fail the command. There are no severity
 levels. The rules report:
 
-- `engineImports`: a value import the suite's engine can't load, such as Playwright
-  in a contract suite. Type-only imports
+- `engineImports`: a value import the suite's engine can't load, such as Vitest
+  in a UI suite. Type-only imports
   are allowed.
 - `configImports`: `testkit.config.ts` importing a repo-local `__testkit__`
   helper module.
-- `dataSuiteContract`: a data suite that doesn't register its cases through
-  `dataScenario` from `@gfdelarue/testkit/node-vitest/data`, registers them
-  through Vitest's `test`, `it`, `describe`, `suite`, or `bench`, or records
-  only bookkeeping such as `count(...)`, `event(...)`, and `finding(...)`
-  without product data-flow evidence such as `http(...)` or `record(...)`.
 - `uiSpecShape`: a UI suite longer than `ui.maxLines` lines or with more than
   `ui.maxTests` tests.
 - `unitTestBoundaries`: test doubles, module replacement, `process.env` reads,
@@ -277,7 +280,7 @@ runs. The backend falls back to `TESTKIT_BACKEND_BASE_URL` and then
 ### `services`
 
 Map of service name to service config. A service describes what Testkit can
-start. With `groups` set, a service owns no test files.
+start. It owns no test files; [`groups`](#groups) assigns them.
 
 ### `groups`
 
@@ -324,21 +327,21 @@ Files that no group's roots claim belong to the implicit group `root`, which
 uses no service. `root` is reserved. With no `testkit.config.ts`, every file
 belongs to `root`, and `groups: {}` gives every file to `root`.
 
-With `groups` set, Testkit ignores a service's `discovery` and rejects a
-service's `skip`, `requirements`, or `runtimeIsolation` with a message naming
-the group field to use.
+Testkit assigns no files by a service's `discovery`. With `groups` set, it
+rejects a service's `discovery`, `skip`, `requirements`, or `runtimeIsolation`
+with a message naming the group field to use. The `discovery` that
+`database.fixture()` adds by default is the exception.
 
-#### Legacy ownership
+#### Configs without `groups`
 
-A config with `services` and no `groups` key uses legacy ownership. Each
-service is a group of the same name that uses that service. It owns the typed
-`*.testkit.ts` files under its `local.cwd` and `discovery.roots`, and its
-`skip`, `requirements`, and `runtimeIsolation` apply to them. A typed file
-outside every service root belongs to `root`. Legacy ownership never claims a
-plain Vitest test file; those belong to `root` unless declared groups claim
-them. When legacy ownership gives a
-file to a service, Testkit prints a notice on stderr. Legacy ownership will be
-removed; declare `groups` instead.
+Before 0.6.0, a config with `services` and no `groups` key gave each service
+the typed `*.testkit.ts` files under its `local.cwd` and `discovery.roots`. Such
+a config now fails to load when a service's directories hold a typed file. The
+error lists a `groups` entry for each of those services, with its `roots`,
+`uses`, and any `exclude`, so the files keep their services. `status`,
+`destroy`, `cleanup`, and `db schema` still run on such a config. A group also
+owns the plain Vitest files under its roots, which a service never did. When no
+service's directories hold a typed file, every file belongs to `root`.
 
 ### `toolchains`
 
@@ -378,48 +381,12 @@ download mode fails on a musl host before downloading anything.
 - `package.json#packageManager`
 - `package.json#engines.npm`
 
-### `profiles.http`
-
-Named HTTP profiles for Node/Vitest data suites, selected through
-`flow.http({ profile })`.
+### `profiles`
 
-Node/Vitest data profiles may carry `auth`, `headers`, and
-`rawHeaders` functions. `auth.setup` or `auth.setupAsync`
-returns a session bundle of named actors; `headers` maps the bundle and the
-requested actor to request headers.
-
-```ts
-interface SessionBundle {
-  actors: Record<string, { session: { jwt: string } }>;
-  primaryActor: string;
-}
-
-profiles: {
-  http: {
-    default: {
-      auth: {
-        async setupAsync({ env, http }): Promise<SessionBundle> {
-          const login = await http.post(`${env.BASE}/api/v1/auth/login`, body);
-          const jwt = login.cookies?.session?.[0]?.value;
-          if (!jwt) throw new Error("Login response did not set the session cookie");
-          return { actors: { primary: { session: { jwt } } }, primaryActor: "primary" };
-        },
-        headers(setupData): Record<string, string> {
-          const bundle = setupData as SessionBundle | null;
-          return bundle
-            ? { Authorization: `Bearer ${bundle.actors[bundle.primaryActor].session.jwt}` }
-            : {};
-        },
-      },
-    },
-  },
-}
-```
-
-Repository profile setup data is typed as `unknown`. The example gives its
-test-owned setup result a name and uses that type in the corresponding header
-builder. Suite-specific functions can also be passed through
-`flow.http({ profile: "default", auth: { setupAsync, headers } })`.
+Testkit 0.6.0 removed HTTP profiles. A config that sets `profiles` fails to
+load. A test logs in with `fetch` or its own client against the URL from
+`@gfdelarue/testkit/environment`, as
+[Record data-flow evidence](README.md#record-data-flow-evidence) shows.
 
 ## Service Fields
 
@@ -594,10 +561,9 @@ to the matching `{db*:api}` token. `env.values` entries override expanded names.
 ### `envFile` / `envFiles`
 
 Optional extra env files loaded into the declared service/task environment for
-migrations, seeds, services, and every execution engine. Node/Vitest data and contract contexts expose task values through
-immutable `context.env`; and managed Playwright contexts expose them through
-`readPlaywrightContext().env`. Direct managed-Node `process.env.KEY` reads are a
-non-enumerable compatibility overlay, not the consumer-input contract.
+migrations, seeds, services, and every execution engine. Node/Vitest tests read
+task values from `process.env`, and managed Playwright contexts expose them
+through `readPlaywrightContext().env`.
 
 Testkit reads a service's env files only when a selected group uses the service
 or a used service depends on it. Loading the config reads none of them, so a
@@ -766,19 +732,20 @@ Changing the resolved Node or npm version invalidates prepared artifacts, so
 
 ### `skip`
 
-Skip rules for a group's files, set as `groups.<name>.skip`. Under legacy
-ownership a service's `skip` applies to the files it owns. Skipped files remain
+Skip rules for a group's files, set as `groups.<name>.skip`. Skipped files remain
 visible in run artifacts with `status: "skipped"` and the configured reason.
 
 Prefer file-local `export const testkit = defineFile({ ... })` metadata
-for per-file skips and locks. Repo-level `skip` is mainly for broad suite-level
-policy.
+for per-file skips and locks in a typed file. Repo-level `skip` is mainly for
+broad suite-level policy. A plain Vitest file has no file-local metadata, so its
+skips, locks, and isolation go in `skip.files`, `requirements.files`, and
+`runtimeIsolation.files`.
 
 ```ts
 skip: {
   files: [
     {
-      path: "__testkit__/billing/billing.contract.testkit.ts",
+      path: "src/billing/billing.test.ts",
       reason: "Billing is currently unavailable",
     },
   ],
@@ -796,12 +763,13 @@ skip: {
 - `files[].reason`
   - required non-empty explanation recorded in artifacts
 - `suites[].selector`
-  - existing suite selector syntax, including typed selectors like `contract:health`
-    and `ui:billing`
+  - existing suite selector syntax, including typed selectors like `ui:billing`
 - `suites[].reason`
   - required non-empty explanation recorded in artifacts
 
-Use `--ignore-skip-rules` on the CLI to run matched files anyway.
+A run that skips a selected file fails, even when every other file passes, and
+`--allow-empty` doesn't change that. Use `--ignore-skip-rules` on the CLI to run
+skipped files anyway.
 
 ### `runtime`
 
@@ -838,17 +806,16 @@ runtime: {
 
 ### `requirements`
 
-Task lock rules for a group's files, set as `groups.<name>.requirements`. Under
-legacy ownership a service's `requirements` apply to the files it owns.
+Task lock rules for a group's files, set as `groups.<name>.requirements`.
 
 ```ts
 requirements: {
   suites: [
-    { selector: "contract:payments", locks: ["payments-sandbox-webhook"] },
+    { selector: "ui:checkout", locks: ["payments-sandbox-webhook"] },
   ],
   files: [
     {
-      path: "src/api/routes/__testkit__/payout-webhook.contract.testkit.ts",
+      path: "src/api/routes/payout-webhook.test.ts",
       locks: ["payments-sandbox-webhook"],
     },
   ],
@@ -861,7 +828,7 @@ wait for each other. Use a lock for a resource that can't be duplicated, never
 for tests that share rows.
 
 - `suites[].selector`
-  - typed selector like `contract:health` or `ui:billing`
+  - typed selector like `ui:billing`
 - `suites[].locks`
   - named locks required by matching files
 - `files[].path`
@@ -872,15 +839,14 @@ for tests that share rows.
 ### `runtimeIsolation`
 
 What a group's files share with each other in a run, set as
-`groups.<name>.runtimeIsolation`. Under legacy ownership a service's
-`runtimeIsolation` applies to the files it owns. A file can set its own with
+`groups.<name>.runtimeIsolation`. A file can set its own with
 `defineFile({ runtimeIsolation: "resetBefore" })`.
 
 ```ts
 runtimeIsolation: {
   default: "shared",
-  suites: [{ selector: "contract:migrations", policy: "resetBefore" }],
-  files: [{ path: "__testkit__/orders/purge.contract.testkit.ts", policy: "resetBefore" }],
+  suites: [{ selector: "ui:admin", policy: "resetBefore" }],
+  files: [{ path: "src/orders/purge.test.ts", policy: "resetBefore" }],
 }
 ```
 
@@ -957,44 +923,15 @@ Typed suites are discovered from the filesystem:
 - a file belongs to the group with the deepest matching root in
   `groups.<name>.roots`
 - files that no group claims belong to the implicit group `root`
-- under legacy ownership, service groups take their roots from `local.cwd` and
-  `discovery.roots`
-
-Supported suffixes:
 
-- `*.contract.testkit.ts`
-- `*.data.testkit.ts`
-- `*.ui.testkit.ts`
+The supported suffix is `*.ui.testkit.ts`.
 
 Every discovered typed file must also export `defineFile({ engine: "..." })`
 from the root package. The suffix supplies `testType`; the required engine declaration
 supplies `engineId`; and Testkit derives `profileId` from the catalog. None of
 those identities are silently rewritten later in the run.
 
-`*.contract.testkit.ts` is a Node-backed service/runtime contract suite type. It
-declares `engine: "node-vitest"`, uses
-`@gfdelarue/testkit/node-vitest/contract`, runs through
-`node-vitest-contract`, and is the right place for checks that need Node APIs
-but are not data-flow evidence suites. Contract files are native Vitest suites:
-`contractScenario(...)` provides managed Testkit context when needed, while
-direct public Vitest runner APIs remain valid.
-
-`*.data.testkit.ts` is a Testkit-owned data-flow/data-contract suite type. It
-declares `engine: "node-vitest"`, uses
-`@gfdelarue/testkit/node-vitest/data`, records evidence through
-`dataScenario(...)`, and runs through `node-vitest-data`. Data suites may use
-non-registering public Vitest APIs such as `expect`, but `dataScenario(...)`
-owns case registration. Ordinary Node/Vitest assertions, DOM/browser checks,
-and service-local loopback checks belong in the appropriate non-data suite
-type.
-
-Examples:
-
-- `src/api/routes/__testkit__/auth/me.contract.testkit.ts`
-- `src/api/routes/__testkit__/runtime.contract.testkit.ts`
-- `src/api/routes/__testkit__/agent-flow.data.testkit.ts`
-- `src/db/__testkit__/sessions/count-type.contract.testkit.ts`
-- `frontend/__testkit__/navigation/navigation.ui.testkit.ts`
+Example: `frontend/__testkit__/navigation/navigation.ui.testkit.ts`.
 
 `testkit discover` exposes that inventory directly without running suites:
 

package/dist/app/lint/rules/data.js

@@ -1 +0,0 @@
-import{AST_NODE_TYPES as r}from"@typescript-eslint/types";import{binding as p,memberName as w,stringValue as E}from"../policy.js";const T=new Set(["addFinding","actor","budget","bytes","count","duration","event","externalCall","expectMetric","expectSequence","fixture","finding","http","metric","money","ratio","reconcile","record","recordCache","recordHttpRequest","recordLlmUsage","recordRpc","sequence","sequenceBudget","system","transition"]),N=new Set(["actor","bytes","duration","externalCall","http","money","ratio","reconcile","record","recordCache","recordHttpRequest","recordLlmUsage","recordRpc","sequence","system","transition"]),f=new Set(["bench","describe","it","suite","test","BenchmarkRunner","TestRunner","EvaluatedModules"]);export function createDataPolicy({context:o,file:C,report:n}){if(!C.endsWith(".data.testkit.ts"))return{};const d=new Set,h=new Set,v=new Map,g=new Map;let y=!1,b=!1;const u=(e,t,a="vitest-runner-import")=>{n(e,a,t,`Data suites cannot use ${t} for Vitest case registration; declare cases with dataScenario(...)`)},x=e=>{for(const t of o.sourceCode.getAncestors(e).reverse())if(t.type===r.ArrowFunctionExpression||t.type===r.FunctionExpression){const a=g.get(t);if(a)return a}},m=e=>e.startsWith("vitest/")&&e!=="vitest/package.json"||e==="@gfdelarue/testkit/node-vitest/contract",k=e=>{E(e)?.includes("{serviceContextUrl:")&&n(e,"service-namespace-url","serviceContextUrl","Data suites must not use serviceContextUrl placeholders because they run outside the deployed service namespace")};return{ImportDeclaration(e){const t=e.source.value;if(t==="@playwright/test"&&n(e,"browser-runtime-import",t,"Browser/runtime driving belongs in ui suites, not data suites"),(t.startsWith("@testing-library/")||t==="jsdom"||t==="happy-dom")&&n(e,"dom-runtime-import","dom-runtime","DOM/rendering tests belong in ui suites, not data suites"),e.importKind==="type")return;const a=e.specifiers.filter(s=>s.type!==r.ImportSpecifier||s.importKind!=="type");if(!(e.specifiers.length&&!a.length)){m(t)&&u(e,t),t==="@gfdelarue/testkit/node-vitest/data"&&(y=!0);for(const s of a){const i=o.sourceCode.getDeclaredVariables(e).find(l=>l.name===s.local.name);if(t==="vitest"&&s.type===r.ImportNamespaceSpecifier&&i&&h.add(i),s.type!==r.ImportSpecifier)continue;const c=s.imported.type===r.Identifier?s.imported.name:s.imported.value;t==="vitest"&&f.has(c)&&u(s,c),t==="@gfdelarue/testkit/node-vitest/data"&&c==="dataScenario"&&i&&d.add(i)}}},ExportNamedDeclaration(e){if(!e.source||e.exportKind==="type")return;const t=e.specifiers.filter(a=>a.exportKind!=="type");if(!(e.specifiers.length&&!t.length)&&(m(e.source.value)&&u(e,e.source.value),e.source.value==="vitest"))for(const a of t){const s=a.local.type===r.Identifier?a.local.name:a.local.value;f.has(s)&&u(a,s)}},ExportAllDeclaration(e){e.exportKind!=="type"&&m(e.source.value)&&u(e,e.source.value)},MemberExpression(e){if(e.object.type!==r.Identifier)return;const t=p(o,e.object),a=w(e);t&&h.has(t)&&a&&f.has(a)&&u(e,a,"vitest-runner-call")},CallExpression(e){const t=e.callee.type===r.Identifier?p(o,e.callee):null;if(t&&d.has(t)){b=!0;const i=e.arguments.at(-1);if(!i||i.type!==r.ArrowFunctionExpression&&i.type!==r.FunctionExpression){n(e,"missing-scenario-callback","dataScenario","dataScenario(...) must provide a callback that receives the Testkit data context");return}const c=i.params[0]?.type===r.AssignmentPattern?i.params[0].left:i.params.at(0),l=c?.type===r.Identifier?o.sourceCode.getDeclaredVariables(i).find(D=>D.name===c.name):null;if(!l){n(e,"missing-data-context","dataScenario","dataScenario callbacks must accept the Testkit data context and record data evidence");return}const S={callback:i,context:l,contextName:l.name,methods:new Set,assertion:!1};v.set(e,S),g.set(i,S);return}const a=o.sourceCode.getText(e.callee);e.callee.type===r.MemberExpression&&a.includes(".serviceContextUrl")&&n(e,"service-namespace-url","serviceContextUrl","Data suites run in the managed Node.js runner, not inside a service namespace; use baseUrl/public endpoints or move service-namespace checks to contract");const s=x(e);if(s&&(/^(expect|assert)(?:\(|\.)/u.test(a)&&(s.assertion=!0),e.callee.type===r.MemberExpression&&e.callee.object.type===r.Identifier&&p(o,e.callee.object)===s.context)){const i=w(e.callee);i&&T.has(i)&&s.methods.add(i)}},"CallExpression:exit"(e){const t=v.get(e);if(!t)return;const{callback:a,contextName:s,methods:i,assertion:c}=t;i.size?[...i].some(l=>N.has(l))||(n(a,"weak-data-evidence",s,"Data scenarios must record product data-flow evidence such as http, bytes, sequence, record, transition, system, externalCall, recordCache, recordLlmUsage, or reconcile; ordinary Node/Vitest-style checks belong in *.contract.testkit.ts with @gfdelarue/testkit/node-vitest/contract"),c&&n(a,"vitest-shaped-data-suite",s,"This dataScenario looks like an ordinary assertion test with weak data bookkeeping; move Node-backed contract checks to *.contract.testkit.ts or model the product data flow with structured evidence")):n(a,"missing-data-evidence",s,"Data scenarios must record Testkit-owned data-flow evidence with the data context, such as http, bytes, sequence, record, transition, system, externalCall, recordCache, recordLlmUsage, or reconcile")},ThrowStatement(e){const t=x(e);t&&(t.assertion=!0)},Literal:k,TemplateLiteral:k,"Program:exit"(e){y?d.size?b||n(e,"missing-data-scenario","dataScenario","Data suites must declare at least one dataScenario(...)"):n(e,"missing-data-scenario-import","dataScenario","Data suites must import dataScenario from @gfdelarue/testkit/node-vitest/data"):n(e,"missing-data-import","@gfdelarue/testkit/node-vitest/data","Data suites must import dataScenario from @gfdelarue/testkit/node-vitest/data")}}}

package/dist/config/authoring/runtime.js

@@ -1 +0,0 @@
-let o=null;export function registerRepoConfig(n){o=n??null}export function getRepoConfig(){return o}export function clearRepoConfig(){o=null}export function resolveHttpProfile(n){if(!n)return null;const t=getRepoConfig();if(!t)return null;const r=t.profiles?.http?.[n];if(!r)throw new Error(`Unknown testkit HTTP profile "${n}"`);return r}

package/dist/engines/node-vitest/runtime/contract/index.js

@@ -1 +0,0 @@
-import{createHttpMethods as x}from"../../../../config/authoring/http-methods.js";import b from"node:fs";import{parseRuntimeFixtureManifestEntries as g,resolveRuntimeFixtureEntry as N}from"../../../contracts/fixture-manifest.js";import{readNodeVitestRuntimeInput as d}from"../runtime-input.js";import{expect as T,test as C}from"vitest";import{isUnknownRecord as s}from"../../../../kernel/unknown-values.js";export{T as expect};const w=Object.freeze({}),E=Object.freeze({env:w});export function contractScenario(t,e,n){const{options:r,run:i}=M(e,n);C(t,async()=>{await i(O())},r.timeoutMs||r.timeout)}export function readContractContext(){const t=d();if(t===null)return E;if(t.profileId!=="node-vitest-contract")throw new Error("Node/Vitest contract API requires node-vitest-contract runtime input");const e={...t.context,env:t.environment,...Object.hasOwn(t,"setupData")?{setupData:t.setupData}:{}};if(!A(e))throw new Error("Node/Vitest contract runtime context is invalid");return Object.freeze(e)}export function createHttpClient(t={}){const e=(t.baseUrl||readContractContext().baseUrl||"").replace(/\/+$/u,"");if(!e)throw new Error("createHttpClient(...) requires baseUrl or the managed Node/Vitest contract context baseUrl");const n=t.defaultHeaders||{"Content-Type":"application/json"};async function r(i,c,y,h={},a={}){const u=i.toUpperCase()||"GET",f=j(y,{contentTypeJson:a.contentTypeJson!==!1}),o=await fetch(`${e}${l(c)}`,{method:u,headers:{...f.body==null?{}:n,...h},body:f.body,signal:a.signal}),p=await o.text();return{body:p,headers:R(o.headers),json(){return JSON.parse(p)},method:u,path:l(c),status:o.status,url:o.url}}return{baseUrl:e,...x(r),request:r}}function O(){const t=readContractContext();return Object.freeze({...t,fixture(e){return S(e)},http(e={}){return createHttpClient(e)}})}function S(t){const e=U(t),n=b.readFileSync(e.absolutePath);return{buffer(){return Buffer.from(n)},contentType:e.contentType,filename:e.filename,json(){return JSON.parse(n.toString("utf8"))},path:e.path,sha256:e.sha256,sizeBytes:e.sizeBytes,text(r="utf8"){return n.toString(r)}}}function U(t){const e=q(t,"fixture name"),n=z();return N(n.entries,e)}function z(){const t=d();return t===null?{entries:[]}:{entries:g(t.fixtures,"Node/Vitest contract fixture manifest")}}function j(t,e){if(t==null)return{body:void 0};if(typeof t=="string")return{body:t};if(t instanceof Uint8Array){const n=new Uint8Array(t.byteLength);return n.set(t),{body:n}}if(e.contentTypeJson)return{body:JSON.stringify(t)};throw new TypeError("Non-JSON contract request bodies must be strings or Uint8Array values")}function l(t){const e=t||"/";return e.startsWith("/")?e:`/${e}`}function R(t){const e={};for(const[n,r]of t.entries())e[n.toLowerCase()]=r;return e}function M(t,e){if(typeof t=="function")return{options:{},run:t};if(typeof e!="function")throw new Error("contractScenario(name, options?, run) requires a callback");return{options:t,run:e}}function q(t,e){const n=typeof t=="string"?t.trim():"";if(!n)throw new Error(`${e} is required`);return n}function A(t){return!s(t)||t.engineId!=="node-vitest"||t.profileId!=="node-vitest-contract"||!k(t.env)||"runner"in t||t.baseUrl!==void 0&&t.baseUrl!==null&&typeof t.baseUrl!="string"||t.serviceName!==void 0&&typeof t.serviceName!="string"?!1:I(t.task)}function I(t){return s(t)?!("type"in t)&&t.testType==="contract"&&m(t.file)&&J(t.id)&&m(t.suiteName):!1}function m(t){return t===void 0||typeof t=="string"}function J(t){return t===void 0||typeof t=="string"||typeof t=="number"}function k(t){return s(t)&&Object.values(t).every(e=>typeof e=="string")}

package/dist/engines/node-vitest/runtime/data/http.js

@@ -1 +0,0 @@
-import{stringFromUnknown as _}from"./primitives.js";import{createHttpMethods as b}from"../../../../config/authoring/http-methods.js";import{resolveHttpProfile as F}from"../../../../config/authoring/runtime.js";import{readNodeVitestRuntimeInput as G}from"../runtime-input.js";import{isUnknownRecord as l}from"../../../../kernel/unknown-values.js";export function createHttpClient(t={}){const e=(t.baseUrl??readDataContext().baseUrl??"").replace(/\/+$/u,"");if(!e)throw new Error("createHttpClient(...) requires baseUrl or the managed Node/Vitest data context baseUrl");const n=t.recorder||t.trace||null,o=t.defaultHeaders||{"Content-Type":"application/json"},r=t.sessionBundle||null,i=t.defaultActor||r?.primaryActor||null,c=typeof t.getHeaders=="function"?t.getHeaders:null,s=typeof t.getRawHeaders=="function"?t.getRawHeaders:c;async function u(a,f,w,d,E,k={},g={}){O(r,a);const A=w.trim()===""?"GET":w.toUpperCase(),D=`${e}${x(d)}`,H=v(E,{contentTypeJson:g.contentTypeJson!==!1}),$={actor:a&&r?r.actors[a]??null:null,actorName:a,sessionBundle:r},S={...H.body==null?{}:o,...m(f?s:c,$),...k},L=Date.now(),h=await fetch(D,{method:A,headers:S,body:H.body,signal:g.signal}),T=await h.text(),P=Date.now()-L,M=C(h.headers),J={body:T,cookies:V(h.headers),durationMs:P,headers:M,json(){return JSON.parse(T)},method:A,path:x(d),status:h.status,url:D};return n&&n.recordHttpRequest({bodyBytes:it(H.body),durationMs:P,method:A,path:x(d),requestHeaders:S,responseBytes:Buffer.byteLength(T),status:h.status}),J}const p=y(null,!0),R=b(p),U=Object.assign(p,{as:B,get:R.get,post:R.post}),N=y(i,!1);return{...b(N),baseUrl:e,headers(a={}){return{...m(c,{actorName:i,actor:j(r,i),sessionBundle:r}),...a}},rawHeaders(a={}){return{...m(s,{actorName:i,actor:j(r,i),sessionBundle:r}),...a}},raw:U,rawReq:U,request:N,as:I};function y(a,f){return(w,d,E,k,g)=>u(a,f,w,d,E,k,g)}function I(a){O(r,a);const f=y(a,!1),w=B(a);return{...b(f),headers(d={}){return{...m(c,{actorName:a,actor:r?r.actors[a]??null:null,sessionBundle:r}),...d}},rawHeaders(d={}){return{...m(s,{actorName:a,actor:r?r.actors[a]??null:null,sessionBundle:r}),...d}},raw:w,rawReq:w,request:f}}function B(a){const f=y(a,!0);return Object.assign(f,b(f))}}export function readDataContext(){const t=G();if(t===null)return nt;if(t.profileId!=="node-vitest-data")throw new Error("Node/Vitest data API requires node-vitest-data runtime input");return et({...t.context,env:t.environment,...Object.hasOwn(t,"setupData")?{setupData:t.setupData}:{}})}export async function createDataHttpClient(t={}){if(!t.profile)return createHttpClient(t);const e=F(t.profile);if(!e)throw new Error(`Data HTTP profile "${t.profile}" is unavailable because no repo config is registered`);const n=Y(e,t),o=X(n),r=await W(n.auth,o,t.profile),i=createHttpClient({...t,baseUrl:o.BASE,...r?.primaryActor?{defaultActor:r.primaryActor}:{},getHeaders(s){return{...n.auth?.headers?.(r,{actor:s.actorName,env:o})??{},...q(n.headers,r,o,s.actorName)}},getRawHeaders(s){return q(n.rawHeaders,r,o,s.actorName)},...r?{sessionBundle:r}:{}}),c=Q(r,i);return{...i,actor:c.actor,actors:c.actors,env:o}}function K(t){return t??null}function Y(t,e){return{...t,...e,auth:e.auth??K(t.auth),headers:e.headers??t.headers,rawHeaders:e.rawHeaders??t.rawHeaders,env:e.env??t.env}}async function W(t,e,n){const o={env:e,http:Z(),json:ot};let r=null;if(typeof t?.setupAsync=="function"?r=await t.setupAsync(o):typeof t?.setup=="function"&&(r=t.setup(o)),r==null)return null;if(!ft(r))throw new Error(`Data HTTP profile "${n??""}" auth setup must return a session bundle with actors and primaryActor`);return r}function q(t,e,n,o){return typeof t!="function"?{}:t(e,{actor:o,env:n})??{}}function X(t){const n={BASE:readDataContext().baseUrl||process.env.BASE_URL},o={...t.env||n,rawEnv:process.env};if(!o.BASE)throw new Error("Data HTTP profile requires BASE_URL or the managed Node/Vitest data context baseUrl");return{...o,BASE:o.BASE,rawEnv:process.env}}function Q(t,e){const n=Object.keys(t?.actors??{}),o=n.map(s=>{const u=t?.actors[s]??null,p=e.as(s);return{email:u?.email||null,headers:p.headers(),index:u?.actorIndex??0,key:s,name:u?.name||null,organizationKey:u?.organizationKey||null,organizationName:u?.organizationName||null,rawHeaders:p.rawHeaders(),rawReq:p.rawReq,req:p,session:u?.session||null}}),r=new Map(o.map(s=>[s.key,s])),i=t?.primaryActor||null,c=i&&r.get(i)||null;return{actor:c,actors:{names:n,primary:c,get(s){const u=r.get(s);if(!u)throw new Error(`Unknown testkit actor "${s}"`);return u},has(s){return r.has(s)},list(){return o.slice()}}}}function Z(){async function t(e,n,o,r={}){const i=st(r),c=await fetch(n,{method:e,headers:i,body:e==="GET"?void 0:at(o)});return{body:await c.text(),cookies:V(c.headers),headers:C(c.headers),status:c.status}}return{del(e,n,o){return t("DELETE",e,n,o)},get(e,n){return t("GET",e,null,n)},patch(e,n,o){return t("PATCH",e,n,o)},post(e,n,o){return t("POST",e,n,o)},put(e,n,o){return t("PUT",e,n,o)}}}function m(t,e){return typeof t!="function"?{}:t(e)}function O(t,e){if(!(!e||!t)&&!Object.hasOwn(t.actors,e))throw new Error(`Unknown testkit actor "${e}"`)}function x(t){const e=t||"/";return e.startsWith("/")?e:`/${e}`}function v(t,e={}){return t==null?{body:void 0}:typeof t=="string"?{body:t}:t instanceof Uint8Array?{body:Uint8Array.from(t)}:t instanceof URLSearchParams||t instanceof FormData||t instanceof Blob?{body:t}:e.contentTypeJson===!1?{body:z(t)}:{body:JSON.stringify(t)}}function C(t){const e={};for(const[n,o]of t.entries())e[n]=o;return e}function V(t){const e=typeof t.getSetCookie=="function"?t.getSetCookie():tt(t.get("set-cookie")),n={};for(const o of e){const[r]=o.split(";"),i=r.indexOf("=");if(i<=0)continue;const c=r.slice(0,i).trim(),s=r.slice(i+1).trim();n[c]=[{value:s}]}return n}function tt(t){return t?t.split(/,(?=\s*[^;,]+=)/u).map(e=>e.trim()).filter(Boolean):[]}function et(t){if(!l(t))throw new Error("Node/Vitest data context must be an object");if(t.engineId!=="node-vitest")throw new Error('Node/Vitest data context.engineId must be "node-vitest"');if(t.profileId!=="node-vitest-data")throw new Error('Node/Vitest data context.profileId must be "node-vitest-data"');if(!dt(t.env))throw new Error("Node/Vitest data context.env must be a string environment record");if("runner"in t)throw new Error("Node/Vitest data context.runner is not supported");const e={...t,env:t.env};if(t.baseUrl!==void 0&&t.baseUrl!==null&&typeof t.baseUrl!="string")throw new Error("Node/Vitest data context.baseUrl must be a string");if(t.serviceName!==void 0&&typeof t.serviceName!="string")throw new Error("Node/Vitest data context.serviceName must be a string");if(!l(t.task))throw new Error("Node/Vitest data context.task must be an object");{const n=t.task.id;if(n!==void 0&&typeof n!="string"&&typeof n!="number")throw new Error("Node/Vitest data context.task.id is invalid");for(const o of["file","suiteName"])if(t.task[o]!==void 0&&typeof t.task[o]!="string")throw new Error(`Node/Vitest data context.task.${o} must be a string`);if(t.task.testType!=="data")throw new Error('Node/Vitest data context.task.testType must be "data"');if("type"in t.task)throw new Error("Node/Vitest data context.task.type is not supported");e.task={...typeof t.task.file=="string"?{file:t.task.file}:{},...n===void 0?{}:{id:n},...typeof t.task.suiteName=="string"?{suiteName:t.task.suiteName}:{},testType:t.task.testType}}return e.engineId=t.engineId,e.profileId=t.profileId,Object.freeze(e)}const rt=Object.freeze({}),nt=Object.freeze({env:rt});function ot(t){if(!l(t)||typeof t.body!="string")throw new Error("Data HTTP profile response body must be a string");return JSON.parse(t.body)}function st(t){if(t==null)return{};if(!l(t))throw new Error("Data HTTP profile request options must be an object");return t.headers===void 0?{}:ct(t.headers,"Data HTTP profile request headers")}function at(t){if(t!=null)return z(t)}function it(t){return t===void 0?0:typeof t=="string"?Buffer.byteLength(t):t instanceof Blob?t.size:t instanceof URLSearchParams?Buffer.byteLength(t.toString()):t instanceof ArrayBuffer||ArrayBuffer.isView(t)?t.byteLength:0}function j(t,e){return e&&t?.actors?t.actors[e]??null:null}function z(t){return typeof t=="string"||t instanceof Blob||t instanceof FormData||t instanceof URLSearchParams||t instanceof ArrayBuffer?t:ArrayBuffer.isView(t)?Uint8Array.from(new Uint8Array(t.buffer,t.byteOffset,t.byteLength)):_(t)}function ct(t,e){if(!l(t))throw new Error(`${e} must be an object`);const n={};for(const[o,r]of Object.entries(t))n[o]=ut(r,`${e}.${o}`);return n}function ut(t,e){if(typeof t!="string"||t.length===0)throw new Error(`${e} must be a non-empty string`);return t}function ft(t){return l(t)&&l(t.actors)&&(t.primaryActor===null||typeof t.primaryActor=="string")}function dt(t){return l(t)&&Object.values(t).every(e=>typeof e=="string")}

package/dist/engines/node-vitest/runtime/data/runner.js

@@ -1 +0,0 @@
-import{TestRunner as a}from"vitest";import{isDataScenarioCase as o}from"./scenario.js";export default class s extends a{onCollected(t){for(const r of t)for(const e of r.tasks)if(!o(e))throw new Error(`Data suites must register every case with dataScenario: ${e.name}`)}}

package/dist/types/node-vitest-contract.d.ts

@@ -1,75 +0,0 @@
-import { expect } from 'vitest';
-
-export interface HttpMethods<Response, Extra extends unknown[]> {
-	get: (path: string, ...extra: Extra) => Response;
-	delete: (path: string, body?: unknown, ...extra: Extra) => Response;
-	post: (path: string, body?: unknown, ...extra: Extra) => Response;
-	put: (path: string, body?: unknown, ...extra: Extra) => Response;
-	patch: (path: string, body?: unknown, ...extra: Extra) => Response;
-}
-export interface ContractScenarioOptions {
-	timeout?: number;
-	timeoutMs?: number;
-}
-export interface ContractHttpRequestOptions extends Record<string, unknown> {
-	contentTypeJson?: boolean;
-	signal?: AbortSignal;
-}
-export interface ContractHttpResponse {
-	body: string;
-	headers: Record<string, string>;
-	json(): unknown;
-	method: string;
-	path: string;
-	status: number;
-	url: string;
-}
-export interface ContractHttpClientOptions {
-	baseUrl?: string;
-	defaultHeaders?: Record<string, string>;
-}
-export interface ContractHttpClient extends HttpMethods<Promise<ContractHttpResponse>, [
-	headers?: Record<string, string>,
-	options?: ContractHttpRequestOptions
-]> {
-	baseUrl: string;
-	request(method: string, path: string, body?: unknown, headers?: Record<string, string>, options?: ContractHttpRequestOptions): Promise<ContractHttpResponse>;
-}
-export interface ContractFixture {
-	buffer(): Buffer;
-	contentType: string;
-	filename: string;
-	json(): unknown;
-	path: string;
-	sha256: string;
-	sizeBytes: number;
-	text(encoding?: BufferEncoding): string;
-}
-export interface ContractRuntimeContext extends Record<string, unknown> {
-	baseUrl?: string | null;
-	engineId?: "node-vitest";
-	readonly env: Readonly<Record<string, string>>;
-	profileId?: "node-vitest-contract";
-	serviceName?: string;
-	task?: {
-		file?: string;
-		id?: number | string;
-		suiteName?: string;
-		testType: "contract";
-	};
-}
-export interface ContractScenarioContext extends ContractRuntimeContext {
-	fixture(name: string): ContractFixture;
-	http(options?: ContractHttpClientOptions): ContractHttpClient;
-}
-export type ContractScenarioRun = (context: ContractScenarioContext) => unknown;
-export declare function contractScenario(name: string, run: ContractScenarioRun): void;
-export declare function contractScenario(name: string, options: ContractScenarioOptions, run: ContractScenarioRun): void;
-export declare function readContractContext(): ContractRuntimeContext;
-export declare function createHttpClient(options?: Readonly<ContractHttpClientOptions>): ContractHttpClient;
-
-export {
-	expect,
-};
-
-export {};