@ripplo/testing 0.4.3 → 0.4.5

package/dist/actions.d.ts

@@ -192,9 +192,9 @@ interface HandleDialogOptions {
     readonly uiOnly?: boolean;
 }
 /**
- * Respond to a native browser dialog (`alert`, `confirm`, `prompt`). Set
- * `uiOnly: true` only when the dialog has no backend side effect — if it
- * triggers a mutation, wire an observer and keep `uiOnly` off.
+ * Register a one-shot listener for the next native dialog (`alert`, `confirm`,
+ * `prompt`). Pure client-side — the backend mutation, if any, comes from the
+ * triggering action; observer assertions belong on that step, not here.
  */
 declare function handleDialog({ action, promptText, uiOnly }: HandleDialogOptions): UnlabeledStep<{
     action: "accept" | "dismiss";

package/dist/assert.d.ts

@@ -1,4 +1,4 @@
-import { O as ObserverHandle, a as ObserverInput, P as Primitive, b as ObserverBudgetTier } from './types-2i1Sm7_J.js';
+import { O as ObserverHandle, a as ObserverInput, P as Primitive, b as ObserverBudgetTier } from './types-16SB7zjP.js';
 import { Variable, StaticStringRef, VariableRef } from './control.js';
 import { U as UnlabeledStep } from './step-De52hTLd.js';
 import { CheckLocator, AnyLocator } from './locators.js';

package/dist/{builder-mhjAdyNW.d.ts → builder-BMjy83Iy.d.ts}

@@ -1,4 +1,4 @@
-import { P as Primitive, O as ObserverHandle, h as Precondition, m as PreconditionData, j as TestDefinition, f as ObserverDefinition, l as PreconditionDefinition, U as UnimplementedItems, e as ObserverContext, g as ObserverOutcome, S as SetupContext, k as TestValue, T as TeardownContext } from './types-2i1Sm7_J.js';
+import { P as Primitive, O as ObserverHandle, h as Precondition, m as PreconditionData, j as TestDefinition, f as ObserverDefinition, l as PreconditionDefinition, U as UnimplementedItems, e as ObserverContext, g as ObserverOutcome, S as SetupContext, k as TestValue, T as TeardownContext } from './types-16SB7zjP.js';
 import { ObserverBudget } from '@ripplo/spec';
 import { S as Step } from './step-De52hTLd.js';
 

package/dist/{chunk-GWSEDWEF.js → chunk-DL3HLCD7.js}

@@ -62,6 +62,7 @@ function compileTest(def, preconditionDefs) {
     preconditions: resolvedPreconditions,
     requiresKeys: { ...def.requiresKeys },
     slug,
+    sourcePath: def.sourcePath,
     spec,
     warnings
   };

package/dist/compiler.d.ts

@@ -1,6 +1,6 @@
 import { Observer, Precondition, WorkflowSpec } from '@ripplo/spec';
-import { d as RipploBuilder } from './builder-mhjAdyNW.js';
-import './types-2i1Sm7_J.js';
+import { d as RipploBuilder } from './builder-BMjy83Iy.js';
+import './types-16SB7zjP.js';
 import './step-De52hTLd.js';
 
 interface CompileResult {
@@ -18,6 +18,7 @@ interface CompiledTest {
     readonly preconditions: ReadonlyArray<string>;
     readonly requiresKeys: Record<string, string>;
     readonly slug: string;
+    readonly sourcePath: string | undefined;
     readonly spec: WorkflowSpec;
     readonly warnings: ReadonlyArray<string>;
 }

package/dist/compiler.js

@@ -1,6 +1,6 @@
 import {
   compile
-} from "./chunk-GWSEDWEF.js";
+} from "./chunk-DL3HLCD7.js";
 import "./chunk-MGATMMCZ.js";
 import "./chunk-4MGIQFAJ.js";
 export {

package/dist/elysia.d.ts

@@ -1,7 +1,7 @@
 import { Elysia } from 'elysia';
-import { R as RipploEngine } from './engine-B6MM4AEw.js';
-import './builder-mhjAdyNW.js';
-import './types-2i1Sm7_J.js';
+import { R as RipploEngine } from './engine-DMOkJdjd.js';
+import './builder-BMjy83Iy.js';
+import './types-16SB7zjP.js';
 import './step-De52hTLd.js';
 import '@ripplo/spec';
 

package/dist/{engine-B6MM4AEw.d.ts → engine-DMOkJdjd.d.ts}

@@ -1,5 +1,5 @@
-import { c as PreconditionRegistry, a as ObserverRegistry, O as ObserverImplFn, P as PreconditionImpl, e as RipploInstance } from './builder-mhjAdyNW.js';
-import { P as Primitive, g as ObserverOutcome, C as CookieEntry, f as ObserverDefinition, l as PreconditionDefinition, U as UnimplementedItems, O as ObserverHandle, h as Precondition } from './types-2i1Sm7_J.js';
+import { c as PreconditionRegistry, a as ObserverRegistry, O as ObserverImplFn, P as PreconditionImpl, e as RipploInstance } from './builder-BMjy83Iy.js';
+import { P as Primitive, g as ObserverOutcome, C as CookieEntry, f as ObserverDefinition, l as PreconditionDefinition, U as UnimplementedItems, O as ObserverHandle, h as Precondition } from './types-16SB7zjP.js';
 
 interface EngineResult {
     readonly cookies: ReadonlyArray<CookieEntry>;

package/dist/express.d.ts

@@ -1,7 +1,7 @@
 import { Router } from 'express';
-import { R as RipploEngine } from './engine-B6MM4AEw.js';
-import './builder-mhjAdyNW.js';
-import './types-2i1Sm7_J.js';
+import { R as RipploEngine } from './engine-DMOkJdjd.js';
+import './builder-BMjy83Iy.js';
+import './types-16SB7zjP.js';
 import './step-De52hTLd.js';
 import '@ripplo/spec';
 

package/dist/fastify.d.ts

@@ -1,7 +1,7 @@
 import { FastifyInstance } from 'fastify';
-import { R as RipploEngine } from './engine-B6MM4AEw.js';
-import './builder-mhjAdyNW.js';
-import './types-2i1Sm7_J.js';
+import { R as RipploEngine } from './engine-DMOkJdjd.js';
+import './builder-BMjy83Iy.js';
+import './types-16SB7zjP.js';
 import './step-De52hTLd.js';
 import '@ripplo/spec';
 

package/dist/hono.d.ts

@@ -1,7 +1,7 @@
 import { Hono } from 'hono';
-import { R as RipploEngine } from './engine-B6MM4AEw.js';
-import './builder-mhjAdyNW.js';
-import './types-2i1Sm7_J.js';
+import { R as RipploEngine } from './engine-DMOkJdjd.js';
+import './builder-BMjy83Iy.js';
+import './types-16SB7zjP.js';
 import './step-De52hTLd.js';
 import '@ripplo/spec';
 

package/dist/index.d.ts

@@ -1,9 +1,9 @@
-export { C as CoverageRegistry, O as ObserverImplFn, a as ObserverRegistry, P as PreconditionImpl, b as PreconditionRecord, c as PreconditionRegistry, R as ResolveDeps, d as RipploBuilder, e as RipploInstance, f as RipploRegistries, g as createRipplo, o as observer, p as precondition, t as test } from './builder-mhjAdyNW.js';
+export { C as CoverageRegistry, O as ObserverImplFn, a as ObserverRegistry, P as PreconditionImpl, b as PreconditionRecord, c as PreconditionRegistry, R as ResolveDeps, d as RipploBuilder, e as RipploInstance, f as RipploRegistries, g as createRipplo, o as observer, p as precondition, t as test } from './builder-BMjy83Iy.js';
 import { CompileResult } from './compiler.js';
 export { CompiledTest, compile } from './compiler.js';
-export { E as EngineImpls, a as EngineResult, b as ExecuteBatchOptions, N as NotImplemented, O as ObserverImplFnFor, P as PreconditionImplFor, R as RipploEngine, c as createEngine, n as notImplemented } from './engine-B6MM4AEw.js';
-import { C as CookieEntry } from './types-2i1Sm7_J.js';
-export { c as CookieOptions, D as DEFAULT_IGNORE_PATHS, d as DEFAULT_WATCH_PATHS, e as ObserverContext, f as ObserverDefinition, O as ObserverHandle, a as ObserverInput, g as ObserverOutcome, h as Precondition, i as PreconditionDeps, P as Primitive, S as SetupContext, T as TeardownContext, j as TestDefinition, k as TestValue } from './types-2i1Sm7_J.js';
+export { E as EngineImpls, a as EngineResult, b as ExecuteBatchOptions, N as NotImplemented, O as ObserverImplFnFor, P as PreconditionImplFor, R as RipploEngine, c as createEngine, n as notImplemented } from './engine-DMOkJdjd.js';
+import { C as CookieEntry } from './types-16SB7zjP.js';
+export { c as CookieOptions, D as DEFAULT_IGNORE_PATHS, d as DEFAULT_WATCH_PATHS, e as ObserverContext, f as ObserverDefinition, O as ObserverHandle, a as ObserverInput, g as ObserverOutcome, h as Precondition, i as PreconditionDeps, P as Primitive, S as SetupContext, T as TeardownContext, j as TestDefinition, k as TestValue } from './types-16SB7zjP.js';
 export { D as DslNodeInput } from './step-De52hTLd.js';
 import '@ripplo/spec';
 

package/dist/index.js

@@ -14,7 +14,7 @@ import {
 } from "./chunk-YQAEOH5W.js";
 import {
   compile
-} from "./chunk-GWSEDWEF.js";
+} from "./chunk-DL3HLCD7.js";
 import "./chunk-MGATMMCZ.js";
 import {
   buildSetCookieHeader,
@@ -103,7 +103,21 @@ function observer(name) {
 }
 function test(id, options) {
   validateTestId(id);
-  return buildTestName(id, options?.uiOnly);
+  const sourcePath = captureTestSourcePath();
+  return buildTestName({ id, sourcePath, uiOnly: options?.uiOnly });
+}
+var TESTS_ANCHOR_PATTERN = /[/\\]\.ripplo[/\\]tests[/\\]([^):]+?)(?::\d+:\d+\)?)?$/;
+function captureTestSourcePath() {
+  const stack = new Error("capture").stack;
+  if (stack == null) {
+    return void 0;
+  }
+  const match = stack.split("\n").map((line) => TESTS_ANCHOR_PATTERN.exec(line)).find((m) => m != null);
+  const captured = match?.[1];
+  if (captured == null) {
+    return void 0;
+  }
+  return captured.replaceAll("\\", "/");
 }
 function createRipplo(registries) {
   const { observers, preconditions, tests } = registries;
@@ -223,15 +237,20 @@ function buildPreconditionWithDeps({
     })
   });
 }
-function buildTestName(id, uiOnly) {
+function buildTestName({ id, sourcePath, uiOnly }) {
   return chainable({
-    name: (displayName) => buildTestRequires({ id, name: displayName, uiOnly })
+    name: (displayName) => buildTestRequires({ id, name: displayName, sourcePath, uiOnly })
   });
 }
 function castOutcome(value) {
   return value;
 }
-function buildTestRequires({ id, name, uiOnly }) {
+function buildTestRequires({
+  id,
+  name,
+  sourcePath,
+  uiOnly
+}) {
   let description = "";
   const self = chainable({
     description(text) {
@@ -250,6 +269,7 @@ function buildTestRequires({ id, name, uiOnly }) {
           name,
           reqNames,
           requiresKeys,
+          sourcePath,
           uiOnly,
           getDescription: () => description
         })
@@ -264,6 +284,7 @@ function buildTestOutcome({
   name,
   reqNames,
   requiresKeys,
+  sourcePath,
   uiOnly
 }) {
   const description = getDescription();
@@ -276,6 +297,7 @@ function buildTestOutcome({
         name,
         reqNames,
         requiresKeys,
+        sourcePath,
         uiOnly
       });
     }
@@ -288,6 +310,7 @@ function buildTestStartsAt({
   name,
   reqNames,
   requiresKeys,
+  sourcePath,
   uiOnly
 }) {
   return chainable({
@@ -300,6 +323,7 @@ function buildTestStartsAt({
       name,
       requires: [...reqNames],
       requiresKeys,
+      sourcePath,
       startsAtFn: void 0,
       stepsFn: void 0,
       uiOnly
@@ -312,6 +336,7 @@ function buildTestStartsAt({
         name,
         reqNames,
         requiresKeys,
+        sourcePath,
         startsAtFn: fn,
         uiOnly
       });
@@ -325,6 +350,7 @@ function buildTestSteps({
   name,
   reqNames,
   requiresKeys,
+  sourcePath,
   startsAtFn,
   uiOnly
 }) {
@@ -339,6 +365,7 @@ function buildTestSteps({
         name,
         requires: [...reqNames],
         requiresKeys,
+        sourcePath,
         startsAtFn,
         stepsFn,
         uiOnly
@@ -450,18 +477,21 @@ function noDuplicateLabels(nodes, _test, report) {
 }
 function assertAfterAction(nodes, _test, report) {
   let consecutiveActions = 0;
-  nodes.forEach((node) => {
+  nodes.forEach((node, index) => {
+    if (index === 0 && node.type === "goto") {
+      return;
+    }
     if (isAssertionNode(node)) {
       consecutiveActions = 0;
-    } else {
-      consecutiveActions++;
-      if (consecutiveActions === 3) {
-        report({
-          message: "3+ consecutive actions without an assertion \u2014 add verification between actions",
-          rule: "assert-after-action",
-          step: node.label ?? node.id
-        });
-      }
+      return;
+    }
+    consecutiveActions++;
+    if (consecutiveActions === 3) {
+      report({
+        message: "3+ consecutive actions without an assertion \u2014 add verification between actions",
+        rule: "assert-after-action",
+        step: node.label ?? node.id
+      });
     }
   });
 }
@@ -691,9 +721,6 @@ function isLikelyBackendMutation(node) {
   if (node.type === "upload") {
     return true;
   }
-  if (node.type === "handleDialog" && node.action === "accept") {
-    return true;
-  }
   if (node.type !== "click") {
     return false;
   }

package/dist/koa.d.ts

@@ -1,7 +1,7 @@
 import { Middleware } from 'koa';
-import { R as RipploEngine } from './engine-B6MM4AEw.js';
-import './builder-mhjAdyNW.js';
-import './types-2i1Sm7_J.js';
+import { R as RipploEngine } from './engine-DMOkJdjd.js';
+import './builder-BMjy83Iy.js';
+import './types-16SB7zjP.js';
 import './step-De52hTLd.js';
 import '@ripplo/spec';
 

package/dist/lockfile.d.ts

@@ -1,8 +1,8 @@
 import { Codec } from '@ripplo/spec';
 import { z } from 'zod';
 import { CompileResult } from './compiler.js';
-import './builder-mhjAdyNW.js';
-import './types-2i1Sm7_J.js';
+import './builder-BMjy83Iy.js';
+import './types-16SB7zjP.js';
 import './step-De52hTLd.js';
 
 declare const LOCKFILE_RELATIVE_PATH = ".ripplo/ripplo.lock";
@@ -27,6 +27,7 @@ declare const lockfileBodySchema: z.ZodObject<{
         preconditions: z.ZodArray<z.ZodString>;
         requiresKeys: z.ZodRecord<z.ZodString, z.ZodString>;
         slug: z.ZodString;
+        sourcePath: z.ZodOptional<z.ZodString>;
         spec: z.ZodObject<{
             entryNode: z.ZodString;
             nodes: z.ZodRecord<z.ZodString, z.ZodDiscriminatedUnion<[z.ZodObject<{

package/dist/lockfile.js

@@ -509,6 +509,7 @@ var compiledTestSchema = z10.object({
   preconditions: z10.array(z10.string().max(200)).max(1e3),
   requiresKeys: requiresKeysSchema,
   slug: z10.string().max(200),
+  sourcePath: z10.string().max(500).optional(),
   spec: workflowSpecSchema
 });
 var lockfileBodySchema = z10.object({
@@ -528,6 +529,7 @@ function compileResultToLockfile(result) {
       preconditions: [...test.preconditions],
       requiresKeys: { ...test.requiresKeys },
       slug: test.slug,
+      sourcePath: test.sourcePath,
       spec: test.spec
     }))
   };

package/dist/nestjs.d.ts

@@ -1,7 +1,7 @@
 import { DynamicModule } from '@nestjs/common';
-import { R as RipploEngine } from './engine-B6MM4AEw.js';
-import './builder-mhjAdyNW.js';
-import './types-2i1Sm7_J.js';
+import { R as RipploEngine } from './engine-DMOkJdjd.js';
+import './builder-BMjy83Iy.js';
+import './types-16SB7zjP.js';
 import './step-De52hTLd.js';
 import '@ripplo/spec';
 

package/dist/nextjs.d.ts

@@ -1,6 +1,6 @@
-import { R as RipploEngine } from './engine-B6MM4AEw.js';
-import './builder-mhjAdyNW.js';
-import './types-2i1Sm7_J.js';
+import { R as RipploEngine } from './engine-DMOkJdjd.js';
+import './builder-BMjy83Iy.js';
+import './types-16SB7zjP.js';
 import './step-De52hTLd.js';
 import '@ripplo/spec';
 

package/dist/{types-2i1Sm7_J.d.ts → types-16SB7zjP.d.ts}

@@ -60,6 +60,7 @@ interface TestDefinition {
     readonly name: string;
     readonly requires: ReadonlyArray<string>;
     readonly requiresKeys: Record<string, string>;
+    readonly sourcePath: string | undefined;
     readonly startsAtFn: VarsFn<string> | undefined;
     readonly stepsFn: VarsFn<ReadonlyArray<Step>> | undefined;
     readonly uiOnly: boolean | undefined;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@ripplo/testing",
   "description": "TypeScript DSL for defining and running Ripplo e2e workflow tests",
-  "version": "0.4.3",
+  "version": "0.4.5",
   "type": "module",
   "files": [
     "dist"