@cosmicdrift/kumiko-server-runtime 0.210.0 → 0.212.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-server-runtime",
-  "version": "0.210.0",
+  "version": "0.212.0",
   "description": "Production server-boot runtime for Kumiko apps: connections, schema-drift-gate, seeds, lifecycle, graceful shutdown. Symmetric to kumiko-dev-server's runDevApp, without dev/scaffold/codegen tooling.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -80,8 +80,8 @@
     }
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-bundled-features": "0.210.0",
-    "@cosmicdrift/kumiko-framework": "0.210.0",
+    "@cosmicdrift/kumiko-bundled-features": "0.212.0",
+    "@cosmicdrift/kumiko-framework": "0.212.0",
     "temporal-polyfill": "^0.3.2"
   },
   "publishConfig": {

package/src/__tests__/build-prod-bundle.test.ts

@@ -12,6 +12,7 @@ import { mkdir, mkdtemp, rm, writeFile } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import {
+  buildMissingTemplateError,
   type ClientEntry,
   computeBuildId,
   discoverClientEntries,
@@ -333,4 +334,49 @@ describe("build-prod-bundle/discovery edges", () => {
     await writeFile(join(workDir, "src/client_admin.tsx"), "// bad");
     expect(discoverClientEntries(workDir)).toEqual([]);
   });
+
+  // #2305: a module only imported by another client-*.tsx still matches
+  // the entry pattern and becomes a second bundle entry.
+  test("discoverClientEntries treats a plain imported module named client-<x>.tsx as its own entry", async () => {
+    await mkdir(join(workDir, "src"), { recursive: true });
+    await writeFile(join(workDir, "src/client-app.tsx"), "// real entry");
+    await writeFile(join(workDir, "src/client-features.tsx"), "export const clientFeatures = [];");
+
+    const entries = discoverClientEntries(workDir);
+
+    expect(entries.map((e) => e.name)).toEqual(["app", "features"]);
+  });
+});
+
+describe("build-prod-bundle/buildMissingTemplateError", () => {
+  function multiEntry(): ClientEntry {
+    return {
+      name: "features",
+      sourceFile: "/Users/dev/offlot-app/src/client-features.tsx",
+      manifestKey: "client-features.js",
+      htmlPath: "features.html",
+    };
+  }
+
+  test("names the discovered source file and the client-<suffix> convention", () => {
+    const message = buildMissingTemplateError(
+      { "client-features.js": "/assets/client-features-abcd.js" },
+      multiEntry(),
+    );
+
+    expect(message).toContain("src/client-features.tsx");
+    expect(message).toContain("client-<suffix>.tsx");
+    expect(message).toContain("umbenennen");
+  });
+
+  test("single-mode entry gets no rename hint (nothing was auto-discovered by suffix)", () => {
+    const message = buildMissingTemplateError(
+      { "client.js": "/assets/client-abcd.js" },
+      clientEntry(),
+    );
+
+    expect(message).toContain("src/client.tsx");
+    expect(message).not.toContain("umbenennen");
+    expect(message).not.toContain("client-<suffix>.tsx");
+  });
 });

package/src/__tests__/compose-features.test.ts

@@ -8,7 +8,7 @@
 // dispatched ins Leere → 500.
 
 import { describe, expect, spyOn, test } from "bun:test";
-import { defineFeature } from "@cosmicdrift/kumiko-framework/engine";
+import { defineFeature, validateBoot } from "@cosmicdrift/kumiko-framework/engine";
 import { composeFeatures } from "../compose-features";
 
 const noopFeature = defineFeature("noop-app", () => {});
@@ -130,6 +130,38 @@ describe("composeFeatures", () => {
     expect(handlerNames).toContain("logout");
   });
 
+  // fw#2223 regression: without this wiring, includeBundled apps that DO
+  // configure authOptions.invite still got /members with no invite button —
+  // the screen is bound to auth-email-password's invite-create handler,
+  // which only exists when opts.invite is set (feature.ts:238-239).
+  test("authOptions.invite → tenant bekommt invite-Screen + Toolbar-Action; ohne invite bootet sauber ohne beides", () => {
+    const withInvite = composeFeatures([noopFeature], {
+      includeBundled: true,
+      authOptions: { invite: { appUrl: "https://app/invite" } },
+    });
+    const tenantWith = withInvite.find((f) => f.name === "tenant");
+    expect(tenantWith).toBeDefined();
+    if (!tenantWith) return;
+    const membersScreenWith = tenantWith.screens["members"];
+    if (membersScreenWith?.type !== "projectionList") {
+      throw new Error("expected members screen to be projectionList");
+    }
+    expect(membersScreenWith.toolbarActions?.some((a) => a.id === "invite")).toBe(true);
+    expect(tenantWith.screens["invite-create"]).toBeDefined();
+
+    const withoutInvite = composeFeatures([noopFeature], { includeBundled: true });
+    const tenantWithout = withoutInvite.find((f) => f.name === "tenant");
+    expect(tenantWithout).toBeDefined();
+    if (!tenantWithout) return;
+    const membersScreenWithout = tenantWithout.screens["members"];
+    if (membersScreenWithout?.type !== "projectionList") {
+      throw new Error("expected members screen to be projectionList");
+    }
+    expect(membersScreenWithout.toolbarActions ?? []).toHaveLength(0);
+    expect(tenantWithout.screens["invite-create"]).toBeUndefined();
+    expect(() => validateBoot(withoutInvite)).not.toThrow();
+  });
+
   test("app feature duplicating a bundled name is dropped (no createRegistry crash)", () => {
     // create-kumiko-app's picker hands back createAuthEmailPasswordFeature()
     // because the user ticked it in the recommended set; runDevApp then adds

package/src/build-prod-bundle.ts

@@ -536,7 +536,9 @@ async function renderHtml(
   return injectAssetTags(template, manifest, entry, buildInfo);
 }
 
-function buildMissingTemplateError(manifest: BuildManifest, entry: ClientEntry): string {
+// @internal — exported for unit tests only. See #2305: the message must
+// name the source file and the filename convention.
+export function buildMissingTemplateError(manifest: BuildManifest, entry: ClientEntry): string {
   const cssLine = manifest["styles.css"]
     ? `    <link rel="stylesheet" href="/styles.css" />\n`
     : "";
@@ -544,8 +546,22 @@ function buildMissingTemplateError(manifest: BuildManifest, entry: ClientEntry):
     ? // html-ok: Error-Hilfetext (Tag-Snippet als Text), wird nie gerendert.
       `    <script type="module" src="/${entry.manifestKey}"></script>\n`
     : "";
+  const sourceBasename = basenameOf(entry.sourceFile);
+  // Single-mode entries always carry manifestKey "client.js"; any other
+  // value came from discoverMultiClientEntries' filename match.
+  const isMultiEntry = entry.manifestKey !== "client.js";
+  const discoveryHint = isMultiEntry
+    ? `"src/${sourceBasename}" wurde automatisch als eigener Bundle-Entry erkannt — ` +
+      `jede Datei nach dem Muster src/client-<suffix>.tsx zählt als Entry (hier: Suffix "${entry.name}"). ` +
+      `War das nicht beabsichtigt, z. B. weil die Datei nur ein von einem anderen client-*.tsx ` +
+      `importiertes Modul ist: Datei umbenennen (ohne "client-"-Präfix), dann verschwindet der Entry.\n` +
+      `\n` +
+      `War es beabsichtigt:\n`
+    : "";
   return (
-    `[kumiko build] kein ${entry.htmlPath} gefunden, aber es gibt JS/CSS-Output.\n` +
+    `[kumiko build] kein ${entry.htmlPath} gefunden für Entry "src/${sourceBasename}", ` +
+    `aber es gibt JS/CSS-Output.\n` +
+    discoveryHint +
     `Leg ein public/${basenameOf(entry.htmlPath)} oder ${basenameOf(entry.htmlPath)} im App-Root an, z. B.:\n` +
     `\n` +
     `<!doctype html>\n` +

package/src/compose-features.ts

@@ -78,7 +78,11 @@ export function composeFeatures(
   const bundled = [
     createConfigFeature(),
     createUserFeature(),
-    createTenantFeature(),
+    // inviteScreen mirrors authOptions.invite: that's the exact option that
+    // makes auth-email-password register the invite-create write-handler
+    // the /members invite drawer is bound to — without this, includeBundled
+    // apps that DO configure invite still get /members with no invite button.
+    createTenantFeature({ inviteScreen: Boolean(authOptions?.invite) }),
     createAuthEmailPasswordFeature(authOptions ?? {}),
     // signup-request/signup-confirm are registered whenever authOptions.signup
     // is set (see above), but the handler itself no-ops unless the companion