@cosmicdrift/kumiko-renderer-web 0.286.0 → 0.287.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-renderer-web",
-  "version": "0.286.0",
+  "version": "0.287.0",
   "description": "Web-platform bindings for @cosmicdrift/kumiko-renderer. HTML default-primitives, browser history-based navigation, EventSource-backed live events, and a one-call createKumikoApp that mounts the whole stack via react-dom.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -16,9 +16,9 @@
     "./styles.css": "./src/styles.css"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-dispatcher-live": "0.286.0",
-    "@cosmicdrift/kumiko-headless": "0.286.0",
-    "@cosmicdrift/kumiko-renderer": "0.286.0",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.287.0",
+    "@cosmicdrift/kumiko-headless": "0.287.0",
+    "@cosmicdrift/kumiko-renderer": "0.287.0",
     "@radix-ui/react-dialog": "^1.1.15",
     "@radix-ui/react-dropdown-menu": "^2.1.16",
     "@radix-ui/react-label": "^2.1.8",
@@ -66,7 +66,7 @@
     "@types/react-dom": "^19.2.3",
     "jsdom": "^29.1.1",
     "tailwindcss": "^4.3.0",
-    "@cosmicdrift/kumiko-locale-de": "0.286.0"
+    "@cosmicdrift/kumiko-locale-de": "0.287.0"
   },
   "repository": {
     "type": "git",

package/src/index.ts

@@ -112,6 +112,8 @@ export type { DefaultAppShellProps } from "./layout/default-app-shell";
 export { DefaultAppShell } from "./layout/default-app-shell";
 export type { EditorPanelProps, ResolverComponent } from "./layout/editor-panel";
 export { EditorPanel } from "./layout/editor-panel";
+export type { NavReparentOverride } from "./layout/filter-app-schema-navs";
+export { filterAppSchemaNavsByAllowlist } from "./layout/filter-app-schema-navs";
 export type { LanguageSwitcherProps, LocaleOption } from "./layout/language-switcher";
 export { LanguageSwitcher } from "./layout/language-switcher";
 export type { NavTreeProps } from "./layout/nav-tree";

package/src/layout/__tests__/filter-app-schema-navs.test.ts

@@ -0,0 +1,113 @@
+import { describe, expect, test } from "bun:test";
+import type { AppSchema, FeatureSchema } from "@cosmicdrift/kumiko-renderer";
+import { filterAppSchemaNavsByAllowlist } from "../filter-app-schema-navs";
+
+function feature(overrides: Partial<FeatureSchema> & { featureName: string }): FeatureSchema {
+  return {
+    entities: {},
+    screens: [],
+    ...overrides,
+  } as FeatureSchema;
+}
+
+describe("filterAppSchemaNavsByAllowlist", () => {
+  test("keeps only navs whose qualified QN is in the allowlist", () => {
+    const schema: AppSchema = {
+      features: [
+        feature({
+          featureName: "vehicles",
+          navs: [{ id: "vehicle-list", label: "x", screen: "vehicles:screen:vehicle-list" }],
+        }),
+        feature({
+          featureName: "app-shell",
+          navs: [{ id: "vehicle-list", label: "x", screen: "vehicles:screen:vehicle-list" }],
+        }),
+      ],
+    };
+
+    const filtered = filterAppSchemaNavsByAllowlist(
+      schema,
+      new Set(["app-shell:nav:vehicle-list"]),
+    );
+
+    expect(filtered.features.find((f) => f.featureName === "vehicles")?.navs).toEqual([]);
+    expect(filtered.features.find((f) => f.featureName === "app-shell")?.navs).toHaveLength(1);
+  });
+
+  test("passes through an already-qualified id unchanged when allowed", () => {
+    const schema: AppSchema = {
+      features: [
+        feature({
+          featureName: "config",
+          navs: [{ id: "config:nav:audience-tenant", label: "x" }],
+        }),
+      ],
+    };
+
+    const filtered = filterAppSchemaNavsByAllowlist(
+      schema,
+      new Set(["config:nav:audience-tenant"]),
+    );
+
+    expect(filtered.features[0]?.navs).toHaveLength(1);
+  });
+
+  test("a feature without navs passes through unchanged", () => {
+    const schema: AppSchema = {
+      features: [feature({ featureName: "no-nav-feature" })],
+    };
+
+    const filtered = filterAppSchemaNavsByAllowlist(schema, new Set());
+
+    expect(filtered.features[0]?.navs).toBeUndefined();
+  });
+
+  test("an empty allowlist yields empty navs arrays, not a broken schema", () => {
+    const schema: AppSchema = {
+      features: [feature({ featureName: "vehicles", navs: [{ id: "vehicle-list", label: "x" }] })],
+    };
+
+    const filtered = filterAppSchemaNavsByAllowlist(schema, new Set());
+
+    expect(filtered.features[0]?.navs).toEqual([]);
+    expect(filtered.features).toHaveLength(1);
+  });
+
+  test("reparents an adopted foreign nav onto the given section, leaving other fields intact", () => {
+    const schema: AppSchema = {
+      features: [
+        feature({
+          featureName: "config",
+          navs: [
+            { id: "audience-tenant", label: "config.settings.tenant", order: 0 },
+            {
+              id: "dealers-tenant",
+              label: "dealers.settings",
+              icon: "building",
+              parent: "audience-tenant",
+              screen: "config:screen:dealers-tenant",
+              order: 1,
+            },
+          ],
+        }),
+      ],
+    };
+
+    const filtered = filterAppSchemaNavsByAllowlist(
+      schema,
+      new Set(["config:nav:dealers-tenant"]),
+      new Map([["config:nav:dealers-tenant", { parent: "app-shell:nav:tenant", order: 10 }]]),
+    );
+
+    expect(filtered.features[0]?.navs).toEqual([
+      {
+        id: "dealers-tenant",
+        label: "dealers.settings",
+        icon: "building",
+        parent: "app-shell:nav:tenant",
+        screen: "config:screen:dealers-tenant",
+        order: 10,
+      },
+    ]);
+  });
+});

package/src/layout/filter-app-schema-navs.ts

@@ -0,0 +1,40 @@
+import type { AppSchema, FeatureSchema } from "@cosmicdrift/kumiko-renderer";
+import { qualifyNavId } from "./nav-tree";
+
+export type NavReparentOverride = { readonly parent: string; readonly order: number };
+
+// Reduces every feature's `navs` to the app's sidebar allowlist, leaving
+// screens/entities/handlers/translations and every other schema field
+// untouched. Pure and defensive: a feature with no `navs` passes through
+// unchanged, and a feature whose every nav gets stripped ends up with an
+// empty `navs` array, never `undefined` turned into something broken.
+//
+// `reparentOverrides` (keyed by the nav's qualified QN) lets a foreign,
+// otherwise-unmodifiable nav (e.g. a settings-hub-synthesized entry that
+// can't be hand-re-registered under a new parent at boot-validation time)
+// be "adopted" into a different section by rewriting its parent/order as
+// it survives the allowlist filter. Without an override, a surviving nav's
+// parent/order pass through unchanged.
+export function filterAppSchemaNavsByAllowlist(
+  schema: AppSchema,
+  allowedNavQns: ReadonlySet<string>,
+  reparentOverrides: ReadonlyMap<string, NavReparentOverride> = new Map(),
+): AppSchema {
+  return {
+    ...schema,
+    features: schema.features.map((feature): FeatureSchema => {
+      if (feature.navs === undefined) return feature;
+      return {
+        ...feature,
+        navs: feature.navs.flatMap((nav) => {
+          const qn = qualifyNavId(feature.featureName, nav.id);
+          if (!allowedNavQns.has(qn)) return [];
+          const override = reparentOverrides.get(qn);
+          return override === undefined
+            ? [nav]
+            : [{ ...nav, parent: override.parent, order: override.order }];
+        }),
+      };
+    }),
+  };
+}

package/src/layout/nav-tree.tsx

@@ -970,7 +970,10 @@ export function buildNavRegistrySliceForApp(
   };
 }
 
-function qualifyNavId(feature: string, id: string): string {
+// Exported (not part of the package's public index.ts surface) so other
+// modules under layout/ — e.g. filter-app-schema-navs.ts — qualify nav ids
+// the same way instead of keeping a second copy in sync.
+export function qualifyNavId(feature: string, id: string): string {
   return id.includes(":nav:") ? id : `${feature}:nav:${id}`;
 }