@sdods/core 0.3.0 → 0.3.2

package/dist/config/runner.js

@@ -3,7 +3,7 @@ import { join, resolve as resolvePath } from 'node:path';
 import { devices } from '@playwright/test';
 import { cucumberReporter, defineBddConfig } from 'playwright-bdd';
 import { runnerProjectName, runFiles } from '@sdods/contracts';
-import { coreStepsGlob } from '../steps/glob.js';
+import { coreStepsPatterns } from '../steps/glob.js';
 import { combineTagExpr, normalizeTagExpr } from './tags.js';
 const DEVICE_FOR_BROWSER = {
     chromium: 'Desktop Chrome',
@@ -121,7 +121,7 @@ export function buildRunnerConfig(registry, sel = {}) {
             const testDir = defineBddConfig({
                 features: `${toPosix(p.root)}/features/**/*.feature`,
                 steps: [
-                    coreStepsGlob(),
+                    ...coreStepsPatterns(cfg.project.steps?.core?.exclude ?? []),
                     `${toPosix(p.root)}/steps/**/*.ts`,
                     `${toPosix(p.root)}/pages/**/*.ts`,
                 ],

package/dist/fixtures/test.js

@@ -90,9 +90,15 @@ export const test = base.extend({
     ],
     // ── test scope ──────────────────────────────────────────────────────────
     /**
-     * Declared FIRST in test scope, and automatic, so it decides before anything
-     * expensive happens — before a pool account is leased, before a browser
-     * context is built, before a session is minted.
+     * Automatic, and an explicit DEPENDENCY of `user` — which is what actually makes
+     * it decide before anything expensive happens.
+     *
+     * Declaration order alone does not: Playwright instantiates fixtures in
+     * dependency order, and `user` is pulled in by `storageState`, which the browser
+     * context needs, so the pool lease happened first regardless of where the gate
+     * sat in the object. An `@env:local @user:noconsent` scenario on staging failed
+     * with `No users with role "noconsent"` instead of skipping — the gate never got
+     * to run. Depending on it is the only ordering guarantee Playwright offers.
      *
      * `@env:`, `@skip:<browser>` and `@flag:` were validated at LINT time and had
      * no runtime path whatsoever. That is the worst arrangement available: the
@@ -171,7 +177,11 @@ export const test = base.extend({
         }
         void apiContext;
     },
-    user: async ({ userPool, $tags }, use, testInfo) => {
+    // `$sdodsTagGate` first, and it is not unused: it is here to force the ordering.
+    // Leasing an account for a scenario that the tag gate is about to skip burns a
+    // pool slot, and on a role with no rows it throws before the skip can happen.
+    user: async ({ $sdodsTagGate, userPool, $tags }, use, testInfo) => {
+        void $sdodsTagGate;
         const role = parseTagValue($tags, 'user');
         await use(role ? await userPool.lease(role, testInfo.parallelIndex) : undefined);
     },

package/dist/index.d.ts

@@ -12,6 +12,6 @@ export { PageRegistry } from './fixtures/pages.js';
 export { AuthStateCache } from './fixtures/auth.js';
 export { ScenarioMeta } from './fixtures/scenario.js';
 export type { TestFixtures, WorkerFixtures, SdodsOption } from './fixtures/types.js';
-export { coreStepsGlob, coreStepsDir } from './steps/glob.js';
+export { coreStepsGlob, coreStepsPatterns, coreStepNames, coreStepsDir } from './steps/glob.js';
 export { VERSION } from './version.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/index.js

@@ -11,6 +11,6 @@ export { BasePage } from './pages/base-page.js';
 export { PageRegistry } from './fixtures/pages.js';
 export { AuthStateCache } from './fixtures/auth.js';
 export { ScenarioMeta } from './fixtures/scenario.js';
-export { coreStepsGlob, coreStepsDir } from './steps/glob.js';
+export { coreStepsGlob, coreStepsPatterns, coreStepNames, coreStepsDir } from './steps/glob.js';
 export { VERSION } from './version.js';
 //# sourceMappingURL=index.js.map

package/dist/steps/glob.d.ts

@@ -3,6 +3,8 @@
  * TS transform sees a workspace path (not node_modules) and playwright-bdd loads it.
  */
 export declare function coreStepsDir(): string;
+/** The library names present in this build — `a11y` for `a11y.steps.ts`. */
+export declare function coreStepNames(): string[];
 /**
  * Both extensions, because this resolves against whichever copy of the package is running: the
  * workspace checkout ships `*.steps.ts`, the published package ships the compiled `*.steps.js`.
@@ -10,6 +12,19 @@ export declare function coreStepsDir(): string;
  * outside the pattern. playwright-bdd passes explicit patterns to tinyglobby untouched — it only
  * appends extensions to bare directory patterns — and neither it nor tinyglobby excludes
  * node_modules, so the published layout globs fine.
- */
+ * */
 export declare function coreStepsGlob(): string;
+/**
+ * The patterns bddgen should load, honouring `steps.core.exclude`.
+ *
+ * Separate from `coreStepsGlob()` because that one is exported API and returns a single string;
+ * widening its return type would break every caller for the sake of an option most projects
+ * never set. With nothing excluded this returns exactly that same glob, so the common path is
+ * byte-identical to what shipped before.
+ *
+ * An unknown name THROWS rather than being ignored. A silent no-op on a typo would leave the
+ * project believing it had excluded a library while the collision it was avoiding still fails
+ * generation — and the error it would then read points at the step, not at the typo.
+ */
+export declare function coreStepsPatterns(exclude?: readonly string[]): string[];
 //# sourceMappingURL=glob.d.ts.map

package/dist/steps/glob.js

@@ -1,5 +1,7 @@
+import { readdirSync } from 'node:fs';
 import { dirname } from 'node:path';
 import { fileURLToPath } from 'node:url';
+import { SdodsError } from '../errors.js';
 /**
  * Absolute glob of the core step library. Built from this file's real path so Playwright's
  * TS transform sees a workspace path (not node_modules) and playwright-bdd loads it.
@@ -7,6 +9,13 @@ import { fileURLToPath } from 'node:url';
 export function coreStepsDir() {
     return dirname(fileURLToPath(import.meta.url)).replace(/\\/g, '/');
 }
+/** The library names present in this build — `a11y` for `a11y.steps.ts`. */
+export function coreStepNames() {
+    return readdirSync(coreStepsDir())
+        .map((f) => /^(.+)\.steps\.(?:js|ts)$/.exec(f)?.[1])
+        .filter((n) => Boolean(n))
+        .sort();
+}
 /**
  * Both extensions, because this resolves against whichever copy of the package is running: the
  * workspace checkout ships `*.steps.ts`, the published package ships the compiled `*.steps.js`.
@@ -14,8 +23,38 @@ export function coreStepsDir() {
  * outside the pattern. playwright-bdd passes explicit patterns to tinyglobby untouched — it only
  * appends extensions to bare directory patterns — and neither it nor tinyglobby excludes
  * node_modules, so the published layout globs fine.
- */
+ * */
 export function coreStepsGlob() {
     return `${coreStepsDir()}/*.steps.{js,ts}`;
 }
+/**
+ * The patterns bddgen should load, honouring `steps.core.exclude`.
+ *
+ * Separate from `coreStepsGlob()` because that one is exported API and returns a single string;
+ * widening its return type would break every caller for the sake of an option most projects
+ * never set. With nothing excluded this returns exactly that same glob, so the common path is
+ * byte-identical to what shipped before.
+ *
+ * An unknown name THROWS rather than being ignored. A silent no-op on a typo would leave the
+ * project believing it had excluded a library while the collision it was avoiding still fails
+ * generation — and the error it would then read points at the step, not at the typo.
+ */
+export function coreStepsPatterns(exclude = []) {
+    if (exclude.length === 0)
+        return [coreStepsGlob()];
+    const available = coreStepNames();
+    const unknown = exclude.filter((n) => !available.includes(n));
+    if (unknown.length) {
+        throw new SdodsError('CONFIG_INVALID', `steps.core.exclude names ${unknown.length === 1 ? 'a library' : 'libraries'} that ` +
+            `${unknown.length === 1 ? 'does' : 'do'} not exist: ${unknown.join(', ')}.`, {
+            hint: `Available: ${available.join(', ')}. Names are the file basenames, so "a11y" for a11y.steps.ts.`,
+        });
+    }
+    // Explicit per-library patterns rather than a negated glob: playwright-bdd hands these to
+    // tinyglobby verbatim, and a `!`-prefixed entry in that list is treated as a path, not an
+    // exclusion.
+    return available
+        .filter((n) => !exclude.includes(n))
+        .map((n) => `${coreStepsDir()}/${n}.steps.{js,ts}`);
+}
 //# sourceMappingURL=glob.js.map

package/dist/steps/index.d.ts

@@ -18,5 +18,5 @@ import './dom.steps.js';
 import './net.steps.js';
 import './perf.steps.js';
 import '../shots/hooks.js';
-export { coreStepsGlob, coreStepsDir } from './glob.js';
+export { coreStepsGlob, coreStepsPatterns, coreStepNames, coreStepsDir } from './glob.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/steps/index.js

@@ -18,5 +18,5 @@ import './dom.steps.js';
 import './net.steps.js';
 import './perf.steps.js';
 import '../shots/hooks.js';
-export { coreStepsGlob, coreStepsDir } from './glob.js';
+export { coreStepsGlob, coreStepsPatterns, coreStepNames, coreStepsDir } from './glob.js';
 //# sourceMappingURL=index.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sdods/core",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "description": "SDODS runtime: configuration, project registry, fixtures, step libraries, data providers, screenshot narratives and self-healing locators.",
   "license": "Apache-2.0",
   "author": "SDODS <admin@sdods.com>",
@@ -42,7 +42,7 @@
     "playwright-bdd": ">=9"
   },
   "dependencies": {
-    "@sdods/contracts": "0.3.0",
+    "@sdods/contracts": "0.3.2",
     "@cucumber/gherkin": "^42.0.1",
     "@cucumber/messages": "^34.2.1",
     "@faker-js/faker": "^10.6.0",
@@ -58,7 +58,7 @@
     "ts-morph": "^28.0.0",
     "yaml": "^2.9.0",
     "zod": "^4.5.4",
-    "@sdods/db": "0.3.0"
+    "@sdods/db": "0.3.2"
   },
   "homepage": "https://sdods.com",
   "bugs": {