@ultimat3/testing 27.16.0 → 28.0.0

package/CLAUDE.md

@@ -2,13 +2,15 @@
 
 Tier 5. May import tiers 0–4. Imported by every package's tests and by generated apps.
 
-Deps: `core` (tier 0), plus `time`, `jobs`, `mail`, `db`, `entity`, `render` and `http` — imported **dynamically
-inside the fixture factories only** (`render` inside `render-view.ts`, `http` inside `auth-request.ts`), so a test that never destructures `mail` never loads the mail
-package and a `packages/core` test never loads the entity registry. `entity` is a dependency for
-exactly one call: `nPlusOne()`, so the strict fixture reports the error `x dev` reports, with the
-`fix:` the schema's own relations spell. A second N+1 code owned here would be a second answer to
-one condition. The one static `entity` import is `registry-isolation.ts`, which is why that module
-is its own entry point and not part of the barrel.
+Deps (15): `core`; `cache` `db` `i18n` `time`; `entity` `http` `policy`; `action` `jobs`
+`query` `realtime`; `mail` `pwa` `render`. Static in `.`: `cache`
+(`registry-leak-guard.ts:7`), `i18n`, `jobs`, `policy` (`registry-snapshot.ts:29-38`), `db`
+(`reusable-database.ts:20`), `realtime/server` (`fixture-subscribe.ts:13`). Dynamic-only:
+`action`, `http`, `mail`, `pwa`, `query`, `render`, `time` — no `mail` destructure, no mail
+package; a `packages/core` import never loads `@ultimat3/entity`. `entity` exists for exactly
+one call, `nPlusOne()`: the strict fixture reports `x dev`'s error, with the `fix:` the schema's
+own relations spell — a second N+1 code here would be a second answer to one condition. Its one
+static import: `registry-isolation.ts`, own entry point, off the barrel.
 
 | Rule | Detail |
 |---|---|
@@ -37,7 +39,7 @@ is its own entry point and not part of the barrel.
 | One body shape, five registrars | `test` (= `fixtureTest`), `unitTest`, `contractTest`, `liveTest`, `jobTest` all take `FixtureBody` + `{ timeoutMs }`: `unitTest('…', async ({ runJobs }) => …)`. Never `describe(testName('unit', …))` + `test` to get a fixture |
 | `runJobs` says who and what | `{ actor }` is the WORKER's identity (the org stays the job's declared tenant), `{ tenantId }` the enqueuer's row; `executions[n].result` is the body's return. A fresh event bus per fixture |
 | A `runJobs` pass is a real worker's `tick()` | `fixture-jobs.ts` builds an unstarted `jobWorker` per pass, so admission runs: `concurrency` waits and `whenBusy: 'fail'` refuses (`X_JOB_KEY_BUSY`) exactly as in a fleet. Never `executeJob` on a bare `claim()`. The lease and slot renew every 1 ms of TEST time (`RENEW_MS`, `frozen-scheduler.ts` on `onClockMoved`), never on a wall-clock interval: cancel, `clock.advance(1)`, and the body hears `X_JOB_LEASE_LOST` — no hand-built worker |
-| Per-test state is `per-test-reset.ts` | the app preload's `beforeEach` resets the jobs event bus — only when `@ultimat3/jobs` is already loaded, so a core test never loads it |
+| Per-test state is `per-test-reset.ts` | the app preload's `beforeEach` resets the jobs event bus — only when `@ultimat3/jobs` is already loaded, so the reset never loads it |
 | An island test is `describeIslandState` | one block per declared state: mounted in `beforeAll` (60 s), disposed in `afterAll`, one build per file. `mountIslandState` + `using` for one test's own mount; an undeclared id is `X_TEST_ISLAND_STATE_UNKNOWN` |
 | An authenticator is tested with `authRequest()` | the hook's own two arguments, real — never `as unknown as Parameters<…>` |
 | `LOG_LEVEL` above `info` filters the TERMINAL | `quiet-logs.ts` pins the logger at `info` while core loads, so a collecting `setLogSink` still sees `warn` under `LOG_LEVEL=error`. It stays the preload's FIRST import |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ultimat3/testing",
-  "version": "27.16.0",
+  "version": "28.0.0",
   "description": "Test harness: cloned template DBs per worker, frozen clock, sealed network, 6 test types",
   "license": "MIT",
   "type": "module",
@@ -29,27 +29,27 @@
     "LICENSE"
   ],
   "engines": {
-    "bun": ">=1.4.2"
+    "bun": ">=1.4.3"
   },
   "scripts": {
     "typecheck": "tsc --noEmit -p tsconfig.json",
     "test": "bun test"
   },
   "dependencies": {
-    "@ultimat3/action": "27.16.0",
-    "@ultimat3/cache": "27.16.0",
-    "@ultimat3/core": "27.16.0",
-    "@ultimat3/db": "27.16.0",
-    "@ultimat3/entity": "27.16.0",
-    "@ultimat3/http": "27.16.0",
-    "@ultimat3/i18n": "27.16.0",
-    "@ultimat3/jobs": "27.16.0",
-    "@ultimat3/mail": "27.16.0",
-    "@ultimat3/policy": "27.16.0",
-    "@ultimat3/pwa": "27.16.0",
-    "@ultimat3/query": "27.16.0",
-    "@ultimat3/realtime": "27.16.0",
-    "@ultimat3/render": "27.16.0",
-    "@ultimat3/time": "27.16.0"
+    "@ultimat3/action": "28.0.0",
+    "@ultimat3/cache": "28.0.0",
+    "@ultimat3/core": "28.0.0",
+    "@ultimat3/db": "28.0.0",
+    "@ultimat3/entity": "28.0.0",
+    "@ultimat3/http": "28.0.0",
+    "@ultimat3/i18n": "28.0.0",
+    "@ultimat3/jobs": "28.0.0",
+    "@ultimat3/mail": "28.0.0",
+    "@ultimat3/policy": "28.0.0",
+    "@ultimat3/pwa": "28.0.0",
+    "@ultimat3/query": "28.0.0",
+    "@ultimat3/realtime": "28.0.0",
+    "@ultimat3/render": "28.0.0",
+    "@ultimat3/time": "28.0.0"
   }
 }

package/src/app-root.ts

@@ -0,0 +1,13 @@
+// Whether this test run's root is an APP — an `app.config.ts` beside it. The registry leak guard
+// restores an app's lazily declared registries as a union, and the framework's own suites exactly.
+
+// why: Bun ships no synchronous file-exists check; Bun.file().exists() is async and a preload
+// decides this before any test file loads.
+import { existsSync } from 'node:fs';
+// why: Bun ships no path-join primitive.
+import { join } from 'node:path';
+
+export const APP_CONFIG = 'app.config.ts';
+
+export const isAppRoot = (root: string = process.cwd()): boolean =>
+  existsSync(join(root, APP_CONFIG));

package/src/preload.ts

@@ -10,7 +10,6 @@ import './quiet-logs';
 import { installDeterminism } from './determinism';
 import { registerFrameworkFixtures } from './framework-fixtures';
 import './matchers';
-import { installAppJsxLoader } from './app-jsx-loader';
 import { onFileBoundary } from './file-boundary';
 import { ISOLATED_ENV, releasePluginsAfterIsolatedFile } from './isolated-plugins';
 import { installPerTestReset } from './per-test-reset';
@@ -72,8 +71,15 @@ if (Bun.env[ISOLATED_ENV] !== '1') {
 // 8 files, 1 -> 8 live global objects (this file's own leak fixture).
 releasePluginsAfterIsolatedFile();
 
-// An app's `.tsx` compiles with the app's JSX factory on the first file, never cached classic.
-await installAppJsxLoader();
+// Every `.tsx` compiles with the framework's JSX factory from the first file on, in every test
+// process — an app root or any workspace below it. Bun's own compile of a `.tsx` is wrong on every
+// Bun (`@ultimat3/render`'s `module-loader.ts`): through 1.4.2 a cached `React.createElement`
+// (`React is not defined`, 18 notificado.co tests), from 1.4.3 a link-time `SyntaxError: Export
+// named 'jsxDEV' not found` in any file that merely imports a component (issue #883). It was
+// installed only where `app.config.ts` sat beside the run, so a workspace package's own tests
+// needed a hand-written preload (castlefight.online, ai-maxxing).
+const { installRenderLoader } = await import('@ultimat3/render/server');
+installRenderLoader();
 
 // Opt-out exists for one case: a test that deliberately exercises a real integration in a job the
 // team runs on purpose. It is an env var, not an API, so it cannot be set from inside a test file.

package/src/registry-leak-guard.ts

@@ -5,7 +5,7 @@
 
 import { afterAll } from 'bun:test';
 import { knownTags, registeredTiers } from '@ultimat3/cache';
-import { isAppRoot } from './app-jsx-loader';
+import { isAppRoot } from './app-root';
 import { RegistryLeakError } from './errors';
 import { runBoundaryHooks, runFileBoundary } from './file-boundary';
 import type { ProcessRegistrySnapshot } from './registry-snapshot';
@@ -196,16 +196,12 @@ export function installRegistryLeakGuard(): void {
     name: 'ultimate-registry-leak-guard',
     setup(build) {
       // `.test.ts` only, never `.test.tsx`. A load handler that answered `loader: 'tsx'` would
-      // compile JSX with Bun's CLASSIC React fallback — the very factory `@ultimat3/render`'s own
-      // loader exists to replace — and, because the first matching handler wins and this one is
-      // registered from the preload, it would shadow render's transform for that file. Routing
-      // `.tsx` through `transformTsx` is not the alternative either: it needs `@ultimat3/render`,
-      // and the transform itself is on `@ultimat3/render/server`, whose import installs that global
-      // loader into every test process in the repo. (The `.` barrel does NOT — `installRenderLoader()`
-      // moved to `server.ts:15` in the 9.0.0 split, and this comment claimed otherwise until
-      // 2026-08-23. The `server` half is still the one that would have to be imported.) Zero
-      // `.test.tsx` files exist and the convention is `<file>.test.ts`, so the narrower filter
-      // costs nothing today; a `.test.tsx` added later is unguarded rather than mis-compiled.
+      // compile JSX with Bun's own transform — the one `@ultimat3/render`'s loader exists to
+      // replace, a link-time `jsxDEV` error from Bun 1.4.3 (issue #883) — and, because the first
+      // matching handler wins and this one is registered from the preload, it would shadow
+      // render's transform for that file. Zero `.test.tsx` files exist and the convention is
+      // `<file>.test.ts`, so the narrower filter costs nothing today; a `.test.tsx` added later is
+      // unguarded rather than mis-compiled.
       build.onLoad({ filter: /\.test\.ts$/ }, async (args) => {
         // Before the registry restore: an island disposed here still needs its fake DOM.
         runFileBoundary();

package/src/app-jsx-loader.ts

@@ -1,30 +0,0 @@
-// An APP's tests compile every `.tsx` with the app's JSX factory from the first file on.
-//
-// Since 22.7 a worker runs many files in one module registry (no `--isolate`), so a `.tsx` module
-// is compiled ONCE per worker and cached. `@ultimat3/render`'s loader is installed by importing
-// `@ultimat3/render/server`; a file that reached a page component before any file had imported it
-// cached the page compiled with Bun's classic factory, and every later file on that worker failed
-// with `React is not defined` (18 notificado.co tests, measured). Installing the loader in the
-// preload makes the first compile the right one.
-//
-// Only in an app (an `app.config.ts` at the run's root): the framework repository's own tests
-// include suites that pin what a process WITHOUT the loader does, and they keep that process.
-
-// why: Bun ships no synchronous file-exists check; Bun.file().exists() is async and a preload
-// decides this before any test file loads.
-import { existsSync } from 'node:fs';
-// why: Bun ships no path-join primitive.
-import { join } from 'node:path';
-
-export const APP_CONFIG = 'app.config.ts';
-
-export const isAppRoot = (root: string = process.cwd()): boolean =>
-  existsSync(join(root, APP_CONFIG));
-
-/** Install the render loader when this run's root is an app. Returns whether it did. */
-export async function installAppJsxLoader(root: string = process.cwd()): Promise<boolean> {
-  if (!isAppRoot(root)) return false;
-  const { installRenderLoader } = await import('@ultimat3/render/server');
-  installRenderLoader();
-  return true;
-}