@cosmicdrift/kumiko-renderer 0.367.0 → 0.369.0

package/dist/app/kumiko-screen.js

@@ -93,8 +93,7 @@ export function KumikoScreen({ schema, qn, translate, entityId, onRowClick, onCo
     return _jsx(ReturnHostProvider, { value: ownHost, children: body });
 }
 function ScreenVisibilityGate({ featureName, visibleWhen, fallback, translate, children, }) {
-    const { Banner, PageHeader } = usePrimitives();
-    const pageHeaderSlotAvailable = usePageHeaderSlotAvailable();
+    const { Banner } = usePrimitives();
     const result = useQuery(visibleWhen.query, {}, { live: true });
     const wasShown = useRef(false);
     const verdict = evalVisibleWhenAfterShown(visibleWhen, result, wasShown.current);
@@ -107,9 +106,19 @@ function ScreenVisibilityGate({ featureName, visibleWhen, fallback, translate, c
     if (verdict === "loading") {
         return (_jsx(Banner, { padded: true, variant: "loading", testId: "kumiko-screen-loading", children: "Loading\u2026" }));
     }
-    // The route still names the locked screen; its breadcrumb must not label what stands in for it.
-    const hideLockedBreadcrumb = PageHeader !== undefined && pageHeaderSlotAvailable ? _jsx(PageHeader, { hideBreadcrumb: true }) : null;
-    return (_jsxs(_Fragment, { children: [hideLockedBreadcrumb, fallback !== undefined ? (_jsx(FallbackScreen, { featureName: featureName, fallback: fallback, translate: translate })) : (_jsx(ScreenUnavailableBanner, {}))] }));
+    if (fallback !== undefined) {
+        return _jsx(FallbackScreen, { featureName: featureName, fallback: fallback, translate: translate });
+    }
+    return (_jsxs(_Fragment, { children: [_jsx(LockedScreenHeader, {}), _jsx(ScreenUnavailableBanner, {})] }));
+}
+// The route still names the locked screen; its breadcrumb must not label what stands in for it.
+// With a stand-in the header is labelled with that screen instead, none if it has no title.
+function LockedScreenHeader({ standInScreenId }) {
+    const { PageHeader } = usePrimitives();
+    const pageHeaderSlotAvailable = usePageHeaderSlotAvailable();
+    if (PageHeader === undefined || !pageHeaderSlotAvailable)
+        return null;
+    return standInScreenId !== undefined ? (_jsx(PageHeader, { hideBreadcrumb: true, breadcrumbScreenId: standInScreenId })) : (_jsx(PageHeader, { hideBreadcrumb: true }));
 }
 function ScreenMissingHomeButton({ homeScreenId, testId, }) {
     const { Button } = usePrimitives();
@@ -144,9 +153,10 @@ function FallbackScreen({ featureName, fallback, translate, }) {
     const target = useEmbeddedScreen(featureName, fallback);
     // Also undefined when the user lacks access to the fallback screen, which is
     // not a framework error.
-    if (target === undefined)
-        return _jsx(ScreenUnavailableBanner, {});
-    return _jsx(KumikoScreen, { schema: target.schema, qn: target.qn, translate: translate });
+    if (target === undefined) {
+        return (_jsxs(_Fragment, { children: [_jsx(LockedScreenHeader, {}), _jsx(ScreenUnavailableBanner, {})] }));
+    }
+    return (_jsxs(_Fragment, { children: [_jsx(LockedScreenHeader, { standInScreenId: lastSegment(target.qn) }), _jsx(KumikoScreen, { schema: target.schema, qn: target.qn, translate: translate })] }));
 }
 function renderScreenBody({ schema, screen, translate, entityId, onRowClick, onCopyLink, }) {
     switch (screen.type) {
@@ -2645,6 +2655,13 @@ function buildConfigEditDefaults(fields) {
     }
     return defaults;
 }
+// A field hidden by its `visible` condition (a provider's settings while another
+// provider is picked) is not saved: the form saves what it shows.
+function hiddenLayoutFieldNames(screen, values) {
+    return new Set(layoutEditFields(screen)
+        .filter((spec) => spec.visible !== undefined && !evalFieldCondition(spec.visible, values))
+        .map((spec) => spec.field));
+}
 function isEmptyNumericFormValue(value) {
     if (value === undefined || value === null || value === "")
         return true;
@@ -2660,6 +2677,8 @@ function ConfigEditBody({ schema, screen, translate, }) {
     // unsere Form-Field-Werte.
     const valuesQuery = useQuery("config:query:values", {});
     const cascadeQuery = useQuery("config:query:cascade", {});
+    const secretKeys = screen.secretKeys;
+    const secretsQuery = useQuery("secrets:query:list", {}, { enabled: secretKeys !== undefined });
     const synthEntity = useMemo(() => synthesizeConfigEditEntity(screen.fields), [screen.fields]);
     const synthScreen = useMemo(() => synthesizeConfigEditScreen(screen), [screen]);
     const boundsSchema = useMemo(() => buildConfigEditSchema(screen.fields), [screen.fields]);
@@ -2671,9 +2690,17 @@ function ConfigEditBody({ schema, screen, translate, }) {
     const initial = useMemo(() => {
         if (valuesQuery.data === null)
             return null;
+        if (secretKeys !== undefined && secretsQuery.data === null)
+            return null;
+        const storedSecretKeys = new Set((secretsQuery.data ?? []).map((row) => row.key));
         const out = {};
         const defaults = buildConfigEditDefaults(screen.fields);
         for (const [shortName, fieldDef] of Object.entries(screen.fields)) {
+            const secretKey = secretKeys?.[shortName];
+            if (secretKey !== undefined) {
+                out[shortName] = storedSecretKeys.has(secretKey) ? true : null;
+                continue;
+            }
             const qualified = screen.configKeys[shortName];
             if (qualified === undefined) {
                 // Author hat ein Field deklariert ohne Mapping — nimm Default.
@@ -2709,7 +2736,14 @@ function ConfigEditBody({ schema, screen, translate, }) {
             }
         }
         return out;
-    }, [valuesQuery.data, screen.fields, screen.configKeys, screen.scope]);
+    }, [
+        valuesQuery.data,
+        secretsQuery.data,
+        secretKeys,
+        screen.fields,
+        screen.configKeys,
+        screen.scope,
+    ]);
     // Cascade-Lookup: qualifiedKey → ConfigCascade für die
     // Cascade-Anzeige unter jedem Feld. Defensive `levels`-Shape-Check
     // damit fremde Query-Mocks (z.B. configEdit-Unit-Tests die nur
@@ -2739,7 +2773,23 @@ function ConfigEditBody({ schema, screen, translate, }) {
     const customSubmit = useCallback(async (snapshot) => {
         const commands = [];
         const commandFieldNames = [];
+        const hiddenFieldNames = hiddenLayoutFieldNames(synthScreen, snapshot.values);
         for (const [shortName, value] of Object.entries(snapshot.changes)) {
+            if (hiddenFieldNames.has(shortName))
+                continue;
+            const secretKey = secretKeys?.[shortName];
+            if (secretKey !== undefined) {
+                // Same wire shape as a writeOnly config field: null removes, a string replaces.
+                if (value === null) {
+                    commandFieldNames.push(shortName);
+                    commands.push({ type: "secrets:write:delete", payload: { key: secretKey } });
+                }
+                else if (typeof value === "string" && value.trim() !== "") {
+                    commandFieldNames.push(shortName);
+                    commands.push({ type: "secrets:write:set", payload: { key: secretKey, value } });
+                }
+                continue;
+            }
             const qualified = screen.configKeys[shortName];
             if (qualified === undefined)
                 continue;
@@ -2811,20 +2861,28 @@ function ConfigEditBody({ schema, screen, translate, }) {
         // bleibt die Anzeige stale bis Reload — gleiche refetch-Calls wie onReset.
         // allSettled: der Write ist schon committed, ein gescheitertes Refetch
         // (stale Anzeige) darf das Erfolgsergebnis nicht in einen Fehler kippen.
-        await Promise.allSettled([valuesQuery.refetch?.(), cascadeQuery.refetch?.()]);
+        await Promise.allSettled([
+            valuesQuery.refetch?.(),
+            cascadeQuery.refetch?.(),
+            ...(secretKeys !== undefined ? [secretsQuery.refetch?.()] : []),
+        ]);
         return { validationBlocked: false, isSuccess: true, data: undefined };
     }, [
         dispatcher,
+        synthScreen,
+        secretKeys,
         screen.configKeys,
         screen.fields,
         screen.scope,
         valuesQuery.refetch,
         cascadeQuery.refetch,
+        secretsQuery.refetch,
     ]);
     if (valuesQuery.loading && valuesQuery.data === null)
         return _jsx(QueryLoadingBanner, {});
-    if (valuesQuery.error) {
-        return (_jsx(QueryErrorBanner, { error: valuesQuery.error, translate: effectiveTranslate, onRetry: valuesQuery.refetch }));
+    const failedQuery = valuesQuery.error ? valuesQuery : secretsQuery.error ? secretsQuery : null;
+    if (failedQuery?.error) {
+        return (_jsx(QueryErrorBanner, { error: failedQuery.error, translate: effectiveTranslate, onRetry: failedQuery.refetch }));
     }
     if (initial === null)
         return _jsx(QueryLoadingBanner, {});

package/dist/primitives.d.ts

@@ -1353,6 +1353,9 @@ export type PageHeaderProps = {
     /** Hide the shell header's breadcrumb while mounted, for a screen that stands in
      *  for the one the route names (a locked screen's fallback). */
     readonly hideBreadcrumb?: boolean;
+    /** Label the breadcrumb with this screen (id as in the route) instead of the one
+     *  the route names, for a locked screen's fallback. Wins over `hideBreadcrumb`. */
+    readonly breadcrumbScreenId?: string;
 };
 /** One item in an `ActionOverflowMenu` (A7: header/row actions beyond the
  *  primary one collapse into this menu instead of a wall of buttons). */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-renderer",
-  "version": "0.367.0",
+  "version": "0.369.0",
   "description": "Platform-agnostic React renderer for Kumiko screens. Contains the shared logic — primitives-contract, hooks, KumikoScreen, navigation & SSE abstractions — that any platform-specific renderer (web, native) composes. No DOM, no EventSource, no react-dom.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -15,9 +15,9 @@
     }
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-framework": "0.367.0",
-    "@cosmicdrift/kumiko-headless": "0.367.0",
-    "@cosmicdrift/kumiko-types": "0.367.0",
+    "@cosmicdrift/kumiko-framework": "0.369.0",
+    "@cosmicdrift/kumiko-headless": "0.369.0",
+    "@cosmicdrift/kumiko-types": "0.369.0",
     "react": "^19.2.6",
     "temporal-polyfill": "^0.3.2",
     "zod": "^4.4.3"
@@ -28,7 +28,7 @@
     "@types/react-dom": "^19.2.3",
     "jsdom": "^29.1.1",
     "react-dom": "^19.2.6",
-    "@cosmicdrift/kumiko-locale-de": "0.367.0"
+    "@cosmicdrift/kumiko-locale-de": "0.369.0"
   },
   "repository": {
     "type": "git",

package/src/changes.json

@@ -1,4 +1,10 @@
 [
+  {
+    "version": "0.368.0",
+    "type": "fix",
+    "title": "A screen that falls back via visibleWhen shows the fallback's title",
+    "detail": "When visibleWhen locks a screen and its fallback renders instead, the header no longer stays empty. It shows the title of the fallback screen (resolved like any other screen title, from its nav label), never the locked screen's. A fallback without a title still leaves the header empty. PageHeader takes a new breadcrumbScreenId prop for this."
+  },
   {
     "version": "0.367.0",
     "type": "fix",