@cosmicdrift/kumiko-document-designer 0.396.0 → 0.398.0

package/dist/web/constants.d.ts

@@ -7,6 +7,7 @@ export declare const PDF_TEMPLATE_EDITOR_SCREEN_ID: "pdf-template-editor";
 export declare const MAIL_EDITOR_SCREEN_ID: "mail-editor";
 export declare const TEMPLATES_NAV_ID: "templates";
 export declare const PLATFORM_TEMPLATES_NAV_ID: "platform-templates";
+export declare const MANAGED_PAGES_FEATURE_NAME: "managed-pages";
 export declare const BRANDING_SCREEN_QN: "managed-pages:screen:branding-settings";
 export declare const BRANDING_QUERY_QN: "managed-pages:query:branding";
 export declare const USER_ME_QUERY_QN: "user:query:user:me";

package/dist/web/constants.js

@@ -8,6 +8,7 @@ export const PDF_TEMPLATE_EDITOR_SCREEN_ID = "pdf-template-editor";
 export const MAIL_EDITOR_SCREEN_ID = "mail-editor";
 export const TEMPLATES_NAV_ID = "templates";
 export const PLATFORM_TEMPLATES_NAV_ID = "platform-templates";
+export const MANAGED_PAGES_FEATURE_NAME = "managed-pages";
 export const BRANDING_SCREEN_QN = "managed-pages:screen:branding-settings";
 export const BRANDING_QUERY_QN = "managed-pages:query:branding";
 // Pinned like the branding names: the user features' constants are not exported to the web barrel.

package/dist/web/layout-rails.js

@@ -1,6 +1,6 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { useAppFeatures, useNav, usePrimitives, useTranslation, } from "@cosmicdrift/kumiko-renderer";
-import { BRANDING_SCREEN_QN } from "./constants.js";
+import { BRANDING_SCREEN_QN, MANAGED_PAGES_FEATURE_NAME } from "./constants.js";
 import { isHexColor, LAYOUT_ACCENT_LINES, LAYOUT_BUTTON_STYLES, LAYOUT_FONTS, LAYOUT_WIDTHS, } from "./layout-meta.js";
 import { layoutUsers } from "./layout-users.js";
 import { PLACEHOLDER_SYMBOL, SYSTEM_PLACEHOLDER_TYPE, systemPlaceholderLabel, } from "./mail-draft.js";
@@ -24,7 +24,7 @@ export function BrandingRail({ data, unavailable, }) {
     const nav = useNav();
     const features = useAppFeatures();
     const { Link } = usePrimitives();
-    const hasBrandingScreen = features.some((feature) => feature.featureName === "managed-pages" &&
+    const hasBrandingScreen = features.some((feature) => feature.featureName === MANAGED_PAGES_FEATURE_NAME &&
         feature.screens.some((screen) => screen.id === BRANDING_SCREEN_SHORT_ID));
     const notSet = t("documentDesigner.branding.notSet");
     const value = (text) => text === undefined || text === "" ? notSet : text;

package/dist/web/use-branding.d.ts

@@ -0,0 +1,3 @@
+import { type UseQueryResult } from "@cosmicdrift/kumiko-renderer";
+import type { BrandingView } from "./layout-types.ts";
+export declare function useBranding(): UseQueryResult<BrandingView>;

package/dist/web/use-branding.js

@@ -0,0 +1,8 @@
+import { useAppFeatures, useQuery } from "@cosmicdrift/kumiko-renderer";
+import { BRANDING_QUERY_QN, MANAGED_PAGES_FEATURE_NAME } from "./constants.js";
+// Branding belongs to managed-pages. In an app without it the query has no handler, so every
+// editor open would log a 404; the editors show branding as unavailable instead.
+export function useBranding() {
+    const mounted = useAppFeatures().some((feature) => feature.featureName === MANAGED_PAGES_FEATURE_NAME);
+    return useQuery(BRANDING_QUERY_QN, {}, { enabled: mounted });
+}

package/dist/web/use-layout-editor-state.d.ts

@@ -1,5 +1,5 @@
 import { type EditorTab, type Region, type Translate } from "./layout-editor-logic.ts";
-import type { BrandingView, LayoutKind, LayoutSourceView, LayoutValidation, TenantOverrideCounts } from "./layout-types.ts";
+import type { LayoutKind, LayoutSourceView, LayoutValidation, TenantOverrideCounts } from "./layout-types.ts";
 import type { OverviewRowView } from "./overview-row.ts";
 export type LayoutEditorBodyProps = {
     readonly kind: LayoutKind;
@@ -35,7 +35,7 @@ export declare function useLayoutEditorState(props: LayoutEditorBodyProps, t: Tr
     setPendingKind: import("react").Dispatch<import("react").SetStateAction<LayoutKind | null>>;
     saveError: string | null;
     setSaveError: import("react").Dispatch<import("react").SetStateAction<string | null>>;
-    branding: import("@cosmicdrift/kumiko-renderer").UseQueryResult<BrandingView>;
+    branding: import("@cosmicdrift/kumiko-renderer").UseQueryResult<import("./layout-types.ts").BrandingView>;
     mjmlMode: boolean;
     appearanceFromMjml: boolean;
     dirty: boolean;

package/dist/web/use-layout-editor-state.js

@@ -1,10 +1,10 @@
 import { TemplateResolverHandlers, TemplateResolverQueries, } from "@cosmicdrift/kumiko-bundled-features/template-resolver/web";
 import { useMutation, useQuery } from "@cosmicdrift/kumiko-renderer";
 import { useState } from "react";
-import { BRANDING_QUERY_QN } from "./constants.js";
 import { accentLineShown, appearanceFlags, footerMessageOf, initialTab, layoutBlocks, layoutContent, layoutDirty, layoutSaveFlags, resourcePreviewsOf, saveLayout, shownRowOf, } from "./layout-editor-logic.js";
 import { appearanceOf } from "./layout-meta.js";
 import { systemLabelMap } from "./mail-draft.js";
+import { useBranding } from "./use-branding.js";
 import { useDebouncedValue } from "./use-debounced-value.js";
 // Server-side validation compiles the whole layout (mjml), so wait for a pause in typing.
 const VALIDATE_DEBOUNCE_MS = 400;
@@ -26,7 +26,7 @@ export function useLayoutEditorState(props, t) {
     const upsertTenant = useMutation(TemplateResolverHandlers.upsertTenant);
     const upsertSystem = useMutation(TemplateResolverHandlers.upsertSystem);
     const publishSystem = useMutation(TemplateResolverHandlers.publishSystem);
-    const branding = useQuery(BRANDING_QUERY_QN, {});
+    const branding = useBranding();
     const mjmlMode = tab === "mjml";
     const { appearanceFromMjml, appearanceEditable, appearanceValid } = appearanceFlags({
         isMail,

package/dist/web/use-pdf-editor-state.js

@@ -2,13 +2,13 @@ import { TemplateResolverQueries } from "@cosmicdrift/kumiko-bundled-features/te
 import { useQuery, useUserRoles } from "@cosmicdrift/kumiko-renderer";
 import { RendererPuppeteerClientQueries } from "@cosmicdrift/kumiko-renderer-puppeteer-client/qualified-names";
 import { useEffect, useMemo, useRef, useState } from "react";
-import { BRANDING_QUERY_QN } from "./constants.js";
 import { systemLabelMap, variableLabelMap } from "./mail-draft.js";
 import { draftFromDetail, imageLabelSource, NO_VARIABLES, snapshotOf, textVariablesOf, } from "./pdf-editor-draft.js";
 import { visibleSystemPlaceholders } from "./pdf-left-rail.js";
 import { serializePdfMeta } from "./pdf-meta.js";
 import { localesOf, parseJsonObject, pickRow, resolveLocale } from "./template-rows.js";
 import { DOCUMENT_PDF_KIND, PDF_LAYOUT_KIND, } from "./types.js";
+import { useBranding } from "./use-branding.js";
 import { useFilesCapability } from "./use-files-capability.js";
 import { useInsertTargets } from "./use-insert-targets.js";
 import { useOverviewRows } from "./use-overview-rows.js";
@@ -104,7 +104,7 @@ function usePdfServerQueries(input) {
     const layout = previewQuery.error === null ? (previewQuery.data?.layout ?? null) : null;
     const layouts = useOverviewRows(PDF_LAYOUT_KIND);
     // The built-in "logo" resource of a layout is the tenant's branding logo.
-    const branding = useQuery(BRANDING_QUERY_QN, {});
+    const branding = useBranding();
     const brandingLogoFile = branding.data?.logoFile ?? "";
     const statusQuery = useQuery(RendererPuppeteerClientQueries.status, {});
     const showOrigin = scope === "system";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-document-designer",
-  "version": "0.396.0",
+  "version": "0.398.0",
   "description": "Document designer for Kumiko: template overview, layout editor (mail layout, PDF letterhead) and the PDF template editor.",
   "license": "MIT",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -20,15 +20,15 @@
     "./web/styles.css": "./src/web/styles.css"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-bundled-features": "0.396.0",
-    "@cosmicdrift/kumiko-framework": "0.396.0",
-    "@cosmicdrift/kumiko-renderer": "0.396.0",
-    "@cosmicdrift/kumiko-renderer-puppeteer-client": "0.396.0",
-    "@cosmicdrift/kumiko-types": "0.396.0",
+    "@cosmicdrift/kumiko-bundled-features": "0.398.0",
+    "@cosmicdrift/kumiko-framework": "0.398.0",
+    "@cosmicdrift/kumiko-renderer": "0.398.0",
+    "@cosmicdrift/kumiko-renderer-puppeteer-client": "0.398.0",
+    "@cosmicdrift/kumiko-types": "0.398.0",
     "react": "^19.2.6"
   },
   "devDependencies": {
-    "@testing-library/react": "^16.3.2",
+    "@testing-library/react": "^16.3.3",
     "@types/react": "^19.2.14"
   },
   "repository": {

package/src/changes.json

@@ -1,4 +1,10 @@
 [
+  {
+    "version": "0.397.0",
+    "type": "fix",
+    "title": "Editors skip the branding query without managed-pages",
+    "detail": "The layout editor and the PDF template editor queried managed-pages:query:branding unconditionally. In an app without managed-pages that query has no handler and answered 404 each time an editor opened. They now check the app schema for managed-pages first and show branding as unavailable without asking."
+  },
   {
     "version": "0.390.0",
     "type": "improvement",