@ultimat3/testing 27.16.1 → 29.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ultimat3/testing",
-  "version": "27.16.1",
+  "version": "29.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.4"
   },
   "scripts": {
     "typecheck": "tsc --noEmit -p tsconfig.json",
     "test": "bun test"
   },
   "dependencies": {
-    "@ultimat3/action": "27.16.1",
-    "@ultimat3/cache": "27.16.1",
-    "@ultimat3/core": "27.16.1",
-    "@ultimat3/db": "27.16.1",
-    "@ultimat3/entity": "27.16.1",
-    "@ultimat3/http": "27.16.1",
-    "@ultimat3/i18n": "27.16.1",
-    "@ultimat3/jobs": "27.16.1",
-    "@ultimat3/mail": "27.16.1",
-    "@ultimat3/policy": "27.16.1",
-    "@ultimat3/pwa": "27.16.1",
-    "@ultimat3/query": "27.16.1",
-    "@ultimat3/realtime": "27.16.1",
-    "@ultimat3/render": "27.16.1",
-    "@ultimat3/time": "27.16.1"
+    "@ultimat3/action": "29.0.0",
+    "@ultimat3/cache": "29.0.0",
+    "@ultimat3/core": "29.0.0",
+    "@ultimat3/db": "29.0.0",
+    "@ultimat3/entity": "29.0.0",
+    "@ultimat3/http": "29.0.0",
+    "@ultimat3/i18n": "29.0.0",
+    "@ultimat3/jobs": "29.0.0",
+    "@ultimat3/mail": "29.0.0",
+    "@ultimat3/policy": "29.0.0",
+    "@ultimat3/pwa": "29.0.0",
+    "@ultimat3/query": "29.0.0",
+    "@ultimat3/realtime": "29.0.0",
+    "@ultimat3/render": "29.0.0",
+    "@ultimat3/time": "29.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';
@@ -55,11 +54,18 @@ installPerTestReset();
 // actor's bucket left it spent for the next file on the worker, so a "the 31st call is a 429" test
 // was red whenever another file calling that action ran first. Each file starts with full buckets,
 // the store back to one process' memory, as a file run alone would have it.
+//
+// And the LIFECYCLE, which is one process' too: a drain is terminal, so a file that drained the
+// process — or that emitted a real SIGINT at it, as a signal test does — left every later file on
+// the worker answering 503 X_DRAINING for a reason nothing in that file explains. Each file starts
+// from a live lifecycle, as a file run alone would have it.
 if (Bun.env[ISOLATED_ENV] !== '1') {
   const { forgetHandedOutActionPaths } = await import('@ultimat3/action');
   onFileBoundary(forgetHandedOutActionPaths);
   const { resetRateLimitStore } = await import('@ultimat3/http');
   onFileBoundary(resetRateLimitStore);
+  const { resetLifecycle } = await import('@ultimat3/core');
+  onFileBoundary(resetLifecycle);
 }
 
 // Isolated runs only (`x test` says so): Bun keeps every finished file alive while a plugin is
@@ -72,8 +78,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/render-view.ts

@@ -4,7 +4,7 @@
 
 import type { Actor } from '@ultimat3/core';
 import { assert, ctxOf, runWithContext } from '@ultimat3/core';
-import type { IslandDirective, RouteConfig, RouteParams } from '@ultimat3/render';
+import type { AssetPath, IslandDirective, RouteConfig, RouteParams } from '@ultimat3/render';
 
 /** What a component rendered. */
 export interface RenderedView {
@@ -24,6 +24,16 @@ export interface RenderedRoute<TData> extends RenderedView {
   readonly meta: Awaited<ReturnType<RouteConfig<TData>['meta']>>;
   /** The islands this render emitted, in order. Empty on a page that ships no JavaScript. */
   readonly islands: readonly IslandDirective[];
+  /**
+   * The app-wide favicon's URL, when one was declared and the app's site asset table answered:
+   * the content-hashed URL `/assets/*` serves — the same one the document's
+   * `<link rel="icon" type="image/svg+xml">` carries.
+   *
+   * `undefined` when the app declares no favicon, and `undefined` when the URL cannot be
+   * resolved: a unit test that never loaded the app has no site asset table installed, and a
+   * route still renders.
+   */
+  readonly favicon?: string | undefined;
 }
 
 /**
@@ -42,6 +52,16 @@ export interface RouteRequest {
   readonly params?: RouteParams;
   /** Who is asking. `load` and the page run inside a request context holding this actor. */
   readonly actor?: Actor;
+  /**
+   * The app's favicon — the app-wide declaration in `app.config.ts` (`site.favicon`), the one
+   * SVG every document of the app links. It is the APP's fact, not the route's: the route
+   * repeats nothing, and the render resolves it through the app's own site asset table so a
+   * test can see the URL `/assets/*` will serve.
+   *
+   * Optional: an app that declares no favicon, and a test that never loaded the app, both ask
+   * for a route with no favicon to resolve.
+   */
+  readonly favicon?: AssetPath | undefined;
 }
 
 const DEFAULT_URL = 'http://localhost/';
@@ -64,6 +84,29 @@ const visibleText = (html: string): string =>
 
 const viewOf = (html: string): RenderedView => ({ html, text: visibleText(html) });
 
+/**
+ * The favicon's URL through the app's own site asset table — `asset()`'s answer, the same URL
+ * the document's `<link rel="icon">` carries and `/assets/*` serves.
+ *
+ * `undefined` when the app declares no favicon, and `undefined` when the URL cannot be
+ * resolved: a unit test that never loaded the app has no site asset table installed, and
+ * `asset()` refuses by name there. A route render is not a document, so a favicon it cannot
+ * resolve is no reason to fail the render — the app's BOOT is where a declared favicon that is
+ * not on disk is refused (`X_ASSET_MISSING`), and a test asks about a route, not about the
+ * app's icon.
+ */
+const faviconUrl = async (
+  render: typeof import('@ultimat3/render'),
+  favicon: AssetPath | undefined,
+): Promise<string | undefined> => {
+  if (favicon === undefined) return undefined;
+  try {
+    return render.asset(favicon);
+  } catch {
+    return undefined;
+  }
+};
+
 /**
  * Render one component with the props a page would hand it. The props are the component's own
  * type, so a renamed prop is a compile error in the test rather than an `undefined` in the markup.
@@ -102,6 +145,9 @@ export async function renderRoute<TData>(
   const url = request.url ?? DEFAULT_URL;
   const ctx = { params: request.params ?? {}, url };
   const islands = render.islandCollector({ file: url, hydrate: config.hydrate });
+  // The app's favicon, resolved through the app's own asset table — and skipped when there is
+  // no table to ask, so a route renders with no favicon setup at all.
+  const favicon = await faviconUrl(render, request.favicon);
 
   const run = async (): Promise<RenderedRoute<TData>> => {
     const data = await render.routeDataFor(config, ctx);
@@ -110,7 +156,7 @@ export async function renderRoute<TData>(
     const html = await renderToHtml(component({ data, params: ctx.params, url, query }), {
       islands,
     });
-    return { ...viewOf(html), data, meta, islands: islands.directives };
+    return { ...viewOf(html), data, meta, islands: islands.directives, favicon };
   };
 
   if (request.actor === undefined) return run();

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;
-}