@cosmicdrift/kumiko-testing 0.315.0 → 0.316.0

package/README.md

@@ -18,7 +18,8 @@ await tenant.api.writeOk("my-feature:write:thing:create", { title: "x" });
   `bunfig.integration.toml` and `bunfig.real.toml` (plus `bunfig.dom.toml`). `--dom` needs your own
   `./test-setup/dom.preload.ts`.
 - `kumiko-testing integration [--parallel N] [--timings <file>]`: runs `*.integration.test.ts`
-  with the 15s budget. No `--no-isolate`; `--parallel` only when you ask for it.
+  with the 15s budget. `--parallel` only when you ask for it, and then with `--no-isolate`:
+  bun 1.4.0's implicit `--isolate` leaks native memory per file until CI workers are OOM-killed.
 
 ## E2E (`./e2e`, `./e2e/seed-route`)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-testing",
-  "version": "0.315.0",
+  "version": "0.316.0",
   "description": "Test template for Kumiko apps: per-flow seedTenant, app test stack, canonical test preloads, bunfig generator and integration runner.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -63,9 +63,9 @@
     "kumiko-testing": "./bin/kumiko-testing.ts"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-bundled-features": "0.315.0",
-    "@cosmicdrift/kumiko-dev-server": "0.315.0",
-    "@cosmicdrift/kumiko-framework": "0.315.0",
+    "@cosmicdrift/kumiko-bundled-features": "0.316.0",
+    "@cosmicdrift/kumiko-dev-server": "0.316.0",
+    "@cosmicdrift/kumiko-framework": "0.316.0",
     "zod": "^4.4.3"
   },
   "peerDependencies": {

package/src/changes.json

@@ -1,4 +1,22 @@
 [
+  {
+    "version": "0.316.0",
+    "type": "fix",
+    "title": "Parallel integration runs no longer OOM-kill their workers",
+    "detail": "bun 1.4.0's --parallel implies --isolate, which leaks native memory per\ntest file while the JS heap stays flat: about 4 MB per file for a bare\nsetupTestStack, about 40 MB with bundled-features. In publicstatus\n(--parallel 4) each worker grew from ~290 to ~1000 MiB and the run died\nwith exit 137 on the 3 GiB CI runner. With --no-isolate the same run\npeaks at 380 MiB and finishes in 28.9 s instead of 49.9 s, with no new\nfailures. Without --parallel bun never isolated files, so the tests\nalready isolate through data (seedTenant per flow, queue prefix per\nstack), not through processes."
+  },
+  {
+    "version": "0.316.0",
+    "type": "fix",
+    "title": "A dedicated screenshots Playwright config without SCREENSHOT_DIR now fails with a clear message",
+    "detail": "A dedicated screenshots config (testDir: \"./e2e/screenshots\", used by\nphronexsis and publicstatus) had its entire testDir excluded by\nscreenshotSpecsIgnore()'s \"**/screenshots/**\" pattern whenever\nSCREENSHOT_DIR was unset, so Playwright reported the generic \"No tests\nfound\" with no mention of the missing env var — even though\nrequireScreenshotDir() already existed with a clear message, just never\ncalled on this path. defineAppE2eConfig now calls it up front for any\ntestDir ending in \"screenshots\" outside a screenshot run."
+  },
+  {
+    "version": "0.316.0",
+    "type": "improvement",
+    "title": "tenant.addUser accepts an optional displayName/email identity",
+    "detail": "seedTenant()'s admin already accepted `admin: { displayName, email }`, but\n`tenant.addUser(roles)` (the seed-user route and the in-process seedTenant\nalike) had no such parameter and always fell back to `Seed <hash>` /\n`user-<uuid>@example.test`, which then leaked into screenshots and docs\n(solon, money-horse). `addUser(roles, { displayName, email })` now threads\nan optional identity through to the created user row, the same\n`SeedAdminIdentity` shape the admin already uses. Also: the seedTenant\nfixture logging the shared Playwright `context` in as the tenant admin was\npreviously documented only in a code comment in auth-kit.ts — it is now on\nthe `SeedTenantFixture` type's own JSDoc."
+  },
   {
     "version": "0.314.0",
     "type": "improvement",

package/src/e2e/define-app-e2e-config.ts

@@ -20,7 +20,7 @@ import {
   SEED_TOKEN_ENV,
   STYLESHEET_WATCH_ENV,
 } from "./constants";
-import { isScreenshotRun, screenshotSpecsIgnore } from "./screenshot-dir";
+import { isScreenshotRun, requireScreenshotDir, screenshotSpecsIgnore } from "./screenshot-dir";
 import { E2E_TIMEOUT_MS } from "./timeouts";
 
 // Apps pick Meilisearch over their in-memory search adapter just because
@@ -146,6 +146,13 @@ export function defineAppE2eConfig(input: AppE2eConfigInput): PlaywrightTestConf
   } = input;
   assertNoReservedEnv(env);
   for (const project of projects) assertTemplateOwnedKeysUnset(project);
+  // A dedicated screenshots config (testDir: "./e2e/screenshots") has its
+  // entire testDir excluded by screenshotSpecsIgnore() until SCREENSHOT_DIR
+  // is set, which Playwright reports as a generic "No tests found" with no
+  // mention of the missing env var.
+  if (!isScreenshotRun() && /(?:^|\/)screenshots\/?$/.test(testDir)) {
+    requireScreenshotDir();
+  }
   const baseURL = `http://localhost:${port}`;
   const realRun = isRealProviderRun();
 

package/src/e2e/seed-contract.ts

@@ -60,6 +60,9 @@ export function createSeedUserRequestSchema(extraRoles: readonly string[] = [])
       )
       .min(1)
       .max(seedable.length),
+    displayName: z.string().min(1).max(MAX_DISPLAY_NAME_LENGTH).optional(),
+    // May contain the literal "{tenantId}" placeholder, same as seedAdminIdentitySchema.
+    email: z.string().min(1).max(MAX_EMAIL_LENGTH).optional(),
   });
 }
 

package/src/e2e/seed-route.ts

@@ -206,7 +206,12 @@ export function createE2eSeedRoutes(
       const write: SeedWriter = async (handlerQn, payload, tenantId) =>
         unwrapSavedRow(handlerQn, await deps.dispatchSystemWrite({ handlerQn, payload, tenantId }));
       try {
-        return c.json(await persistUserRows(write, verified.tenantId, verified.roles));
+        return c.json(
+          await persistUserRows(write, verified.tenantId, verified.roles, {
+            displayName: verified.displayName,
+            email: verified.email,
+          }),
+        );
       } catch (error) {
         return c.json({ error: error instanceof Error ? error.message : "seed failed" }, 500);
       }

package/src/e2e/seeded-tenant-fixture.ts

@@ -36,6 +36,11 @@ export type E2eSeededTenant = SeededTenant & {
   readonly seed: (seeder: string, body?: unknown) => Promise<unknown>;
 };
 
+/**
+ * Seeds a tenant over HTTP and logs the shared `context` in as its admin, so
+ * a `page` opened from that context is already authenticated — call
+ * `loginAs` to switch it to another seeded user instead.
+ */
 export type SeedTenantFixture = (opts?: E2eSeedTenantOptions) => Promise<E2eSeededTenant>;
 
 type E2eFixtures = {
@@ -125,12 +130,17 @@ export async function provideSeedTenant(
       name: seeded.name,
       admin,
       members: seeded.members.map((member) => toUser(member, [ROLES.Member])),
-      addUser: async (roles = [ROLES.Member]) =>
+      addUser: async (roles = [ROLES.Member], identity = {}) =>
         toUser(
           await postSeedRoute(
             request,
             SEED_ROUTES.seedUser,
-            { tenantId, roles: [...roles] } satisfies SeedUserRequest,
+            {
+              tenantId,
+              roles: [...roles],
+              displayName: identity.displayName,
+              email: identity.email,
+            } satisfies SeedUserRequest,
             seedUserResponseSchema,
           ),
           roles,

package/src/integration-runner.ts

@@ -29,7 +29,10 @@ export function buildIntegrationTestArgs(opts: IntegrationRunOptions): string[]
     "test",
     `--config=${BUNFIG_FILES.integration}`,
     `--timeout=${TEST_TIMEOUT_MS.integration}`,
-    ...(opts.parallel !== undefined ? [`--parallel=${opts.parallel}`] : []),
+    // bun 1.4.0's --parallel implies --isolate, which leaks native memory per test file
+    // (~40-60 MB with bundled-features) until the workers blow the CI runner's limit.
+    // Tests isolate through data (seedTenant per flow, queue prefix per stack), not processes.
+    ...(opts.parallel !== undefined ? [`--parallel=${opts.parallel}`, "--no-isolate"] : []),
     ...(opts.timings !== undefined ? [`--timings=${opts.timings}`] : []),
     ...(opts.updateTimings === true ? ["--update-timings"] : []),
     ...opts.files,

package/src/seed-tenant.ts

@@ -184,9 +184,13 @@ export async function seedTenant(
     name: rows.name,
     admin,
     members: rows.members.map((member) => withSession(member, id, [ROLES.Member])),
-    addUser: async (roles = [ROLES.Member]) =>
+    addUser: async (roles = [ROLES.Member], identity = {}) =>
       withSession(
-        persist ? await persistUserRows(write, id, roles) : lightCredentials(),
+        persist
+          ? await persistUserRows(write, id, roles, identity)
+          : lightCredentials(
+              identity.email !== undefined ? substituteTenantId(identity.email, id) : undefined,
+            ),
         id,
         roles,
       ),

package/src/seed-types.ts

@@ -45,7 +45,10 @@ export type SeededTenant = {
   readonly name: string;
   readonly admin: SeededUser;
   readonly members: readonly SeededUser[];
-  readonly addUser: (roles?: readonly string[]) => Promise<SeededUser>;
+  readonly addUser: (
+    roles?: readonly string[],
+    identity?: SeedAdminIdentity,
+  ) => Promise<SeededUser>;
   readonly api: BoundApi;
   readonly apiAs: (user: SeededUser) => BoundApi;
 };