@cosmicdrift/kumiko-samples 0.364.0 → 0.365.0

package/packages/bundled-features/src/template-resolver/template-preview.ts

@@ -27,6 +27,7 @@ import { createTemplateResolverApi, TemplateNotFoundError, type TemplateResource
 import { collectDocumentResourceKeys } from "./compile-document-body.js";
 import {
   DEFAULT_MAIL_LAYOUT_SLUG,
+  DEFAULT_PDF_LAYOUT_SLUG,
   MAIL_PREVIEW_MAX_BYTES,
   SYSTEM_TENANT_ID,
   type TemplateScope,
@@ -54,7 +55,6 @@ import {
 const SAMPLE_LINK = "https://example.com";
 const SAMPLE_DATE = "2026-01-15";
 const SAMPLE_LIST_ROWS = 3;
-const DEFAULT_PDF_LAYOUT_SLUG = "default";
 
 function sampleScalar(name: string, def: TemplateScalarVariableDef): unknown {
   if (def.example !== undefined) return def.example;

package/packages/bundled-features/src/template-resolver/template-resources.ts

@@ -6,8 +6,7 @@ import type {
   WriteResult,
 } from "@cosmicdrift/kumiko-framework/engine";
 import { NotFoundError, writeFailure } from "@cosmicdrift/kumiko-framework/errors";
-import { createFileRefFromBytes, fileRefsTable } from "@cosmicdrift/kumiko-framework/files";
-import { fileContextForTenant } from "../file-foundation/index.js";
+import { fileRefsTable } from "@cosmicdrift/kumiko-framework/files";
 import { requireForTenant } from "../shared/index.js";
 import { executor } from "./handlers/shared.js";
 import type { LinkedResourceEntry } from "./linked-resources.js";
@@ -42,48 +41,8 @@ export async function readTemplateResource(
   return { fileName: row.fileName, contentType: row.mimeType, bytes: await handle.read() };
 }
 
-// Gives the target tenant its own copy of every file behind the entries, so a template that was
-// copied from the system default keeps working once the tenant exports it or measures its
-// attachments. Rendered-PDF links carry no file and pass through. An entry whose source file is
-// gone is left out: it could not be resolved at render time either.
-export async function duplicateLinkedResourceFiles(
-  ctx: HandlerContext,
-  args: {
-    readonly user: SessionUser;
-    readonly tenantId: TenantId;
-    readonly resources: Readonly<Record<string, LinkedResourceEntry>>;
-  },
-): Promise<WriteResult<Record<string, LinkedResourceEntry>>> {
-  const { user, tenantId } = args;
-  const { db } = requireForTenant(ctx, tenantId);
-  const copies: Record<string, LinkedResourceEntry> = {};
-  for (const [key, entry] of Object.entries(args.resources)) {
-    if (typeof entry !== "string" && entry.kind === "rendered-pdf") {
-      copies[key] = entry;
-      continue;
-    }
-    const sourceFileRefId = typeof entry === "string" ? entry : entry.fileRefId;
-    const file = await readTemplateResource(ctx, sourceFileRefId);
-    // skip: the source file is gone, there is nothing to copy
-    if (file === null) continue;
-    const created = await createFileRefFromBytes(
-      {
-        db,
-        files: fileContextForTenant(ctx, ctx.user.tenantId, tenantId),
-        registry: ctx.registry,
-        user: { ...user, tenantId },
-      },
-      { fileName: file.fileName, mimeType: file.contentType, data: file.bytes },
-    );
-    if (!created.isSuccess) return created;
-    copies[key] =
-      typeof entry === "string" ? created.data.id : { ...entry, fileRefId: created.data.id };
-  }
-  return { isSuccess: true, data: copies };
-}
-
 // Replaces the whole key map of one row with files that are already stored (bundle import,
-// copy-locale). Single keys go through storeTemplateResource and resource-remove.
+// copy-locale). Single keys go through storePreparedTemplateResource and resource-remove.
 export async function attachTemplateResources(
   ctx: Pick<HandlerContext, "forTenant">,
   args: {

package/packages/bundled-features/src/template-resolver/template-validation.ts

@@ -128,12 +128,10 @@ function isKnownName(
 ): Resolution {
   const { variables } = scope;
   const root = rootName(name);
-  if (Object.hasOwn(itemFields, name) || Object.hasOwn(itemFields, root)) {
-    return { known: true, used: "" };
-  }
+  // Declared variables and item fields are scalars: a dotted path under one never renders a value.
+  if (Object.hasOwn(itemFields, name)) return { known: true, used: "" };
   if (scope.systemPlaceholders.has(root)) return { known: true, used: "" };
   if (Object.hasOwn(variables, name)) return { known: true, used: name };
-  if (Object.hasOwn(variables, root)) return { known: true, used: root };
   return { known: false, used: "" };
 }
 
@@ -151,6 +149,11 @@ function enclosingEachName(element: ScanElement): string | null {
   return element.closest("tr[data-each]")?.getAttribute("data-each") ?? null;
 }
 
+const NAME_ATTRIBUTE_BY_TAG: Readonly<Record<string, string>> = {
+  span: "data-variable",
+  div: "data-steps",
+};
+
 function scanBodyPlaceholders(body: string, scope: PlaceholderScope): void {
   withParsedDocument(body, (root) => {
     for (const element of Array.from(root.querySelectorAll("tr[data-each]"))) {
@@ -160,12 +163,13 @@ function scanBodyPlaceholders(body: string, scope: PlaceholderScope): void {
       else scope.unknown.add(listName);
     }
     for (const element of Array.from(
-      root.querySelectorAll("span[data-variable], a[href], img[data-resource]"),
+      root.querySelectorAll("span[data-variable], div[data-steps], a[href], img[data-resource]"),
     )) {
       const itemFields = listItemFields(scope.variables, enclosingEachName(element));
       const tag = element.tagName.toLowerCase();
-      if (tag === "span") {
-        recordName(element.getAttribute("data-variable") ?? "", itemFields, scope);
+      const nameAttribute = NAME_ATTRIBUTE_BY_TAG[tag];
+      if (nameAttribute !== undefined) {
+        recordName(element.getAttribute(nameAttribute) ?? "", itemFields, scope);
       } else if (tag === "a") {
         const href = element.getAttribute("href") ?? "";
         if (isDocumentPlaceholderHref(href)) recordName(href.slice(2, -2), itemFields, scope);
@@ -179,7 +183,7 @@ function scanBodyPlaceholders(body: string, scope: PlaceholderScope): void {
 
 const HANDLEBARS_EXPRESSION = /\{\{([^{}]*)\}\}/g;
 
-// Meta text supports only plain `{{name}}` / `{{name.field}}`; every other
+// Meta text supports only plain `{{name}}` references; every other
 // Handlebars construct is reported as unknown so it can never reach the compiler.
 function scanMetaPlaceholders(text: string, scope: PlaceholderScope): void {
   const leftover = text.replace(HANDLEBARS_EXPRESSION, (_match, inner: string) => {
@@ -193,7 +197,9 @@ function scanMetaPlaceholders(text: string, scope: PlaceholderScope): void {
 
 function metaTexts(meta: TemplateMeta | null): readonly string[] {
   if (meta === null || !("subject" in meta)) return [];
-  return [meta.subject, ...(meta.previewText === undefined ? [] : [meta.previewText])];
+  return [meta.subject, meta.previewText, meta.headerChip, meta.tone, meta.footerNote].filter(
+    (text): text is string => text !== undefined,
+  );
 }
 
 export function validateTemplatePlaceholders(input: {

package/packages/bundled-features/src/template-resolver/web/document-editor-extensions.ts

@@ -7,9 +7,11 @@ import {
   DocumentButton,
   DocumentPageBreak,
   DocumentResource,
+  DocumentSteps,
   DocumentVariable,
   documentExtensions,
 } from "../document-schema.js";
+import { readDocumentStepsAttribute } from "../document-steps.js";
 import type { DocumentResourcePreview } from "./document-toolbar-blocks.js";
 
 const DOCUMENT_RESOURCE_SIZE_TOKEN = "{size}";
@@ -20,6 +22,7 @@ export type DocumentEditorLabels = {
   readonly image: string;
   readonly pageBreak: string;
   readonly button?: string;
+  readonly steps?: string;
   /** Canvas block label of an image, e.g. "Image · inline". */
   readonly imageBlock?: string;
   readonly variableLabels?: Readonly<Record<string, string>>;
@@ -175,6 +178,35 @@ function createPageBreakView(pageBreakLabel: string): NodeViewRenderer {
   };
 }
 
+// A static sketch of the row: which stage is reached is only known at render time.
+function createStepsView(labels: DocumentEditorLabels): NodeViewRenderer {
+  return ({ node }) => {
+    const dom = createElementWithClass(
+      "div",
+      "relative my-3 rounded border border-dashed border-input px-3 pb-2 pt-3",
+    );
+    dom.setAttribute(DOCUMENT_BLOCK_ATTRIBUTE, "");
+    dom.contentEditable = "false";
+    const variable = String(node.attrs["variable"]);
+    const variableLabels = labels.variableLabels ?? {};
+    const variableLabel = Object.hasOwn(variableLabels, variable)
+      ? variableLabels[variable]
+      : undefined;
+    dom.append(createBlockLabel(`${labels.steps ?? "Steps"} · ${variableLabel ?? variable}`));
+    const row = createElementWithClass("div", "flex gap-1.5");
+    for (const step of readDocumentStepsAttribute(node.attrs["steps"])) {
+      const cell = createElementWithClass("div", "min-w-0 flex-1");
+      cell.append(
+        createElementWithClass("div", "h-1 rounded-full bg-primary/40"),
+        createElementWithClass("div", "truncate pt-1.5 text-xs text-muted-foreground", step.label),
+      );
+      row.append(cell);
+    }
+    dom.append(row);
+    return { dom };
+  };
+}
+
 // Replaced by name, not appended: the schema must keep exactly one node per
 // name, and only the editor view differs from the shared document schema.
 export function createDocumentEditorExtensions(labels: DocumentEditorLabels): Extensions {
@@ -195,6 +227,8 @@ export function createDocumentEditorExtensions(labels: DocumentEditorLabels): Ex
         return labels.blockLabels
           ? DocumentButton.extend({ addNodeView: () => createButtonView(labels.button ?? "") })
           : extension;
+      case DocumentSteps.name:
+        return DocumentSteps.extend({ addNodeView: () => createStepsView(labels) });
       case DocumentPageBreak.name:
         return DocumentPageBreak.extend({
           addNodeView: () => createPageBreakView(labels.pageBreak),

package/packages/bundled-features/src/template-resolver/web/document-editor.tsx

@@ -21,13 +21,14 @@ export function DocumentContentEditor(props: DocumentEditorProps): ReactNode {
   const editorProps = props.blocks === undefined ? { ...props, blocks: contextBlocks } : props;
   return (
     <Suspense
+      // Read-only without chips: a textarea would insert {{name}} text and plaintext into a document body.
       fallback={
         <TextareaContentEditor
           id={props.id}
           value={props.value}
           onChange={props.onChange}
-          variables={props.variables}
-          readOnly={props.readOnly}
+          variables={[]}
+          readOnly
         />
       }
     >

package/packages/bundled-features/src/template-resolver/web/document-tiptap-editor.tsx

@@ -29,6 +29,7 @@ import {
   Italic as ItalicIcon,
   Link as LinkIcon,
   List,
+  ListChecks,
   ListOrdered,
   Minus,
   MousePointerClick,
@@ -47,6 +48,7 @@ import {
   isDocumentReferenceName,
   isDocumentResourceKey,
 } from "../document-schema.js";
+import { parseDocumentStepsInput } from "../document-steps.js";
 import {
   createDocumentEditorExtensions,
   DOCUMENT_BLOCK_ATTRIBUTE,
@@ -70,7 +72,7 @@ function previewsKey(previews: Readonly<Record<string, DocumentResourcePreview>>
     .join("\n");
 }
 
-type PanelKind = "link" | "placeholder" | "image" | "button" | "repeat" | "align";
+type PanelKind = "link" | "placeholder" | "image" | "button" | "repeat" | "align" | "steps";
 
 // Blocks that stay in the canvas toolbar; the rest moves into the overflow menu.
 const CANVAS_INLINE_BLOCKS: readonly DocumentToolbarBlock[] = ["image", "table"];
@@ -80,6 +82,7 @@ const BLOCK_PANEL: Partial<Record<DocumentToolbarBlock, PanelKind>> = {
   button: "button",
   repeat: "repeat",
   align: "align",
+  steps: "steps",
 };
 
 const ALIGNMENT_ICONS: Record<DocumentTextAlignment, ReactNode> = {
@@ -315,6 +318,9 @@ function InsertPanel({ editor, editorId, kind, variables, onClose }: PanelProps)
   const [buttonText, setButtonText] = useState(() =>
     t("template-resolver.documentEditor.buttonDefaultText"),
   );
+  const [stepsText, setStepsText] = useState(() =>
+    t("template-resolver.documentEditor.stepsDefault"),
+  );
   const [chosenName, setChosenName] = useState("");
   const [error, setError] = useState<string | null>(null);
 
@@ -364,7 +370,17 @@ function InsertPanel({ editor, editorId, kind, variables, onClose }: PanelProps)
     insertAndClose({ type: "resource", attrs: { resource, alt: alt || null } });
   };
 
-  const needsVariables = kind === "button" || kind === "repeat" || kind === "placeholder";
+  const submitSteps = (): void => {
+    const steps = parseDocumentStepsInput(stepsText);
+    if (steps === null) {
+      setError(t("template-resolver.documentEditor.invalidSteps"));
+      return;
+    }
+    insertAndClose({ type: "steps", attrs: { variable: selectedName, steps } });
+  };
+
+  const needsVariables =
+    kind === "button" || kind === "repeat" || kind === "placeholder" || kind === "steps";
   const noVariables = needsVariables && selectable.length === 0;
   const idPrefix = `${editorId}-panel`;
 
@@ -467,6 +483,29 @@ function InsertPanel({ editor, editorId, kind, variables, onClose }: PanelProps)
           onChange={setChosenName}
         />
       )}
+      {kind === "steps" && !noVariables && (
+        <>
+          <NameSelect
+            id={`${idPrefix}-steps-variable`}
+            label={t("template-resolver.documentEditor.stepsVariable")}
+            names={selectable}
+            value={selectedName}
+            onChange={setChosenName}
+          />
+          <Field
+            id={`${idPrefix}-steps-list`}
+            label={t("template-resolver.documentEditor.stepsList")}
+          >
+            <Input
+              kind="text"
+              id={`${idPrefix}-steps-list`}
+              name="steps"
+              value={stepsText}
+              onChange={setStepsText}
+            />
+          </Field>
+        </>
+      )}
       {error !== null && (
         <p role="alert" className="text-sm text-destructive">
           {error}
@@ -495,6 +534,7 @@ function InsertPanel({ editor, editorId, kind, variables, onClose }: PanelProps)
             })}
           {...(kind === "repeat" &&
             !noVariables && { onInsert: () => insertAndClose(repeatTableContent(selectedName)) })}
+          {...(kind === "steps" && !noVariables && { onInsert: submitSteps })}
         />
       </div>
     </div>
@@ -531,6 +571,7 @@ export default function DocumentTiptapEditor({
   const imageBlockLabel = t("template-resolver.documentEditor.imageBlock");
   const pageBreakLabel = t("template-resolver.documentEditor.pageBreak");
   const buttonLabel = t("template-resolver.documentEditor.button");
+  const stepsLabel = t("template-resolver.documentEditor.steps");
   const stableLabels = useStableByKey(variableLabels, JSON.stringify(variableLabels));
   const stableSystem = useStableByKey(systemVariables, systemVariables.join("\n"));
   const stablePreviews = useStableByKey(resourcePreviews, previewsKey(resourcePreviews));
@@ -541,6 +582,7 @@ export default function DocumentTiptapEditor({
         image: imageLabel,
         pageBreak: pageBreakLabel,
         button: buttonLabel,
+        steps: stepsLabel,
         imageBlock: imageBlockLabel,
         variableLabels: stableLabels,
         systemVariables: stableSystem,
@@ -553,6 +595,7 @@ export default function DocumentTiptapEditor({
       imageBlockLabel,
       pageBreakLabel,
       buttonLabel,
+      stepsLabel,
       stableLabels,
       stableSystem,
       stablePreviews,
@@ -573,6 +616,7 @@ export default function DocumentTiptapEditor({
       immediatelyRender: false,
       editorProps: { attributes: { id } },
       onCreate: ({ editor: created }) => onEditorReady?.(createEditorHandle(created)),
+      onDestroy: () => onEditorReady?.(null),
       onUpdate: ({ editor: e }) => onChange(e.getHTML()),
     },
     [extensions],
@@ -614,27 +658,27 @@ export default function DocumentTiptapEditor({
 
   // The entry loads asynchronously, so `value` arrives after useEditor read
   // its initial content; the guard keeps in-flight typing from being clobbered.
+  // When `extensions` change, useEditor's own effect destroys this render's
+  // editor earlier in the same commit (a late validate result in the layout
+  // editor does that); reading it then throws and unmounts the whole app. The
+  // replacement editor re-renders and syncs `value` itself.
   // kumiko-lint-ignore no-raw-hooks DOM integration: syncs the controlled value into the imperative tiptap editor
   useEffect(() => {
-    if (editor && value !== editor.getHTML()) {
+    if (editor && !editor.isDestroyed && value !== editor.getHTML()) {
       editor.commands.setContent(value, { emitUpdate: false });
     }
   }, [value, editor]);
 
   // kumiko-lint-ignore no-raw-hooks DOM integration: tiptap exposes editability only imperatively via setEditable
   useEffect(() => {
-    editor?.setEditable(!readOnly);
+    // setEditable emits "update" by default, which would report the mount as an edit.
+    if (editor && !editor.isDestroyed) editor.setEditable(!readOnly, false);
   }, [readOnly, editor]);
 
+  // Read-only without chips: a textarea would insert {{name}} text and plaintext into a document body.
   if (!editor)
     return (
-      <TextareaContentEditor
-        id={id}
-        value={value}
-        onChange={onChange}
-        variables={variables}
-        readOnly={readOnly}
-      />
+      <TextareaContentEditor id={id} value={value} onChange={onChange} variables={[]} readOnly />
     );
 
   const togglePanel = (kind: PanelKind): void =>
@@ -670,6 +714,7 @@ export default function DocumentTiptapEditor({
     repeat: { label: t("template-resolver.documentEditor.repeat"), icon: <Repeat size={16} /> },
     pageBreak: { label: pageBreakLabel, icon: <Scissors size={16} /> },
     align: { label: t("template-resolver.documentEditor.align"), icon: <AlignLeft size={16} /> },
+    steps: { label: stepsLabel, icon: <ListChecks size={16} /> },
   };
 
   const repeatTable = activeState?.repeatTable ?? null;

package/packages/bundled-features/src/template-resolver/web/document-toolbar-blocks.ts

@@ -9,7 +9,8 @@ export type DocumentToolbarBlock =
   | "divider"
   | "repeat"
   | "pageBreak"
-  | "align";
+  | "align"
+  | "steps";
 
 /** Imperative access for an outside panel (placeholder list, resource list). */
 export type DocumentEditorHandle = {
@@ -45,8 +46,8 @@ export type DocumentEditorProps = ContentEditorProps & {
   /** Canvas only: renders the toolbar (and its insert panel) into this element instead of above the editor;
    *  null renders none. Lets several region editors share one toolbar that follows the focused one. */
   readonly toolbarHost?: HTMLElement | null;
-  /** Called with the handle once the editor exists and with null when it is destroyed. */
-  readonly onEditorReady?: (handle: DocumentEditorHandle) => void;
+  /** Called with the handle once the editor exists and with null when it is destroyed (unmount or rebuild), so a held handle never outlives its editor. */
+  readonly onEditorReady?: (handle: DocumentEditorHandle | null) => void;
 };
 
 export const MAIL_DOCUMENT_BLOCKS = [
@@ -55,6 +56,7 @@ export const MAIL_DOCUMENT_BLOCKS = [
   "button",
   "divider",
   "repeat",
+  "steps",
 ] as const satisfies readonly DocumentToolbarBlock[];
 
 export const PDF_DOCUMENT_BLOCKS = [
@@ -63,6 +65,7 @@ export const PDF_DOCUMENT_BLOCKS = [
   "table",
   "divider",
   "repeat",
+  "steps",
   "pageBreak",
 ] as const satisfies readonly DocumentToolbarBlock[];
 

package/packages/bundled-features/src/template-resolver/web/i18n.ts

@@ -43,6 +43,13 @@ export const defaultTranslations: TranslationsByLocale = {
     "template-resolver.documentEditor.removeLink": "Remove link",
     "template-resolver.documentEditor.repeat": "Repeat",
     "template-resolver.documentEditor.repeatList": "List to repeat",
+    "template-resolver.documentEditor.steps": "Steps",
+    "template-resolver.documentEditor.stepsVariable": "Current step from",
+    "template-resolver.documentEditor.stepsList": "Steps, separated by commas (value=label)",
+    "template-resolver.documentEditor.stepsDefault":
+      "reported=Reported, investigating=Investigating, resolved=Resolved",
+    "template-resolver.documentEditor.invalidSteps":
+      "Enter 1 to 8 different steps; values use letters, digits, - and _.",
     "template-resolver.documentEditor.resourceKey": "Resource key",
     "template-resolver.documentEditor.table": "Table",
     "template-resolver.editor.titleLabel": "Title",

package/packages/bundled-features/src/template-resolver/web/index.ts

@@ -5,6 +5,7 @@
 // `@cosmicdrift/kumiko-bundled-features/template-resolver` and has no React or
 // DOM deps.
 
+export { MAIL_TONES, type MailTone } from "../mail-constants.js";
 export {
   PDF_DEFAULT_MARGIN_MM,
   PDF_FOOTER_MARGIN_MM,

package/packages/bundled-features/src/template-resolver/web/placeholder-line-editor.tsx

@@ -6,7 +6,7 @@
 
 import { cn } from "@cosmicdrift/kumiko-renderer-web";
 import { Extension, Node } from "@tiptap/core";
-import { UndoRedo } from "@tiptap/extensions";
+import { Placeholder, UndoRedo } from "@tiptap/extensions";
 import { EditorContent, useEditor } from "@tiptap/react";
 import StarterKit from "@tiptap/starter-kit";
 import { type ReactNode, useMemo, useState } from "react";
@@ -28,7 +28,9 @@ export type PlaceholderLineEditorProps = {
   readonly readOnly?: boolean;
   /** Drops the input frame so the line sits flat inside a label/value row. */
   readonly flat?: boolean;
-  readonly onEditorReady?: (handle: DocumentEditorHandle) => void;
+  /** Hint shown while the line is empty. */
+  readonly placeholder?: string;
+  readonly onEditorReady?: (handle: DocumentEditorHandle | null) => void;
   readonly "aria-label": string;
 };
 
@@ -53,6 +55,7 @@ function LineEditorBody({
   systemVariables,
   readOnly,
   flat,
+  placeholder,
   onEditorReady,
   ariaLabel,
 }: {
@@ -63,7 +66,8 @@ function LineEditorBody({
   readonly systemVariables: readonly string[];
   readonly readOnly: boolean;
   readonly flat: boolean;
-  readonly onEditorReady: ((handle: DocumentEditorHandle) => void) | undefined;
+  readonly placeholder: string | undefined;
+  readonly onEditorReady: ((handle: DocumentEditorHandle | null) => void) | undefined;
   readonly ariaLabel: string;
 }): ReactNode {
   const extensions = useMemo(
@@ -96,8 +100,9 @@ function LineEditorBody({
       }),
       UndoRedo,
       BlockLineBreaks,
+      ...(placeholder === undefined ? [] : [Placeholder.configure({ placeholder })]),
     ],
-    [variableLabels, systemVariables, flat],
+    [variableLabels, systemVariables, flat, placeholder],
   );
 
   const editor = useEditor({
@@ -115,6 +120,7 @@ function LineEditorBody({
       transformPastedHTML: (html) => singleLine(html.replace(/<[^>]*>/g, " ")),
     },
     onCreate: ({ editor: created }) => onEditorReady?.(createEditorHandle(created)),
+    onDestroy: () => onEditorReady?.(null),
     onUpdate: ({ editor: e }) => onChange(contentToLine(e.getJSON().content?.[0]?.content)),
   });
 
@@ -126,6 +132,7 @@ function LineEditorBody({
         flat ? "min-h-10" : "min-h-9 rounded-md border border-input px-3",
         "[&_.ProseMirror]:w-full [&_.ProseMirror]:whitespace-nowrap [&_.ProseMirror]:outline-none",
         "[&_p]:m-0",
+        "[&_.is-editor-empty]:before:pointer-events-none [&_.is-editor-empty]:before:float-left [&_.is-editor-empty]:before:h-0 [&_.is-editor-empty]:before:text-muted-foreground [&_.is-editor-empty]:before:content-[attr(data-placeholder)]",
         readOnly && "opacity-60",
       )}
     />
@@ -140,6 +147,7 @@ export default function PlaceholderLineEditor({
   systemVariables = NO_NAMES,
   readOnly = false,
   flat = false,
+  placeholder,
   onEditorReady,
   "aria-label": ariaLabel,
 }: PlaceholderLineEditorProps): ReactNode {
@@ -158,7 +166,7 @@ export default function PlaceholderLineEditor({
 
   return (
     <LineEditorBody
-      key={`${epoch}-${readOnly}-${JSON.stringify(stableLabels)}`}
+      key={`${epoch}-${readOnly}-${JSON.stringify(stableLabels)}-${stableSystem.join("\n")}`}
       id={id}
       initialValue={value}
       onChange={(next) => {
@@ -169,6 +177,7 @@ export default function PlaceholderLineEditor({
       systemVariables={stableSystem}
       readOnly={readOnly}
       flat={flat}
+      placeholder={placeholder}
       onEditorReady={onEditorReady}
       ariaLabel={ariaLabel}
     />

package/packages/bundled-features/src/tenant/changes.json

@@ -1,4 +1,10 @@
 [
+  {
+    "version": "0.365.0",
+    "type": "fix",
+    "title": "Post-save seed hooks use the after-commit notify factory",
+    "detail": "afterCommit seed hooks now get a notify function bound to the signup dispatcher's job runner when the dispatcher provides one, falling back to the regular factory. The afterCommit hook context is built only when the hook actually runs."
+  },
   {
     "version": "0.357.0",
     "type": "improvement",

package/packages/bundled-features/src/tenant/handlers/update-member-roles.write.ts

@@ -9,7 +9,6 @@ import {
   mayOverrideTenant,
   mergeAssignedRoles,
   parseTenantId,
-  type SessionUser,
   SYSTEM_ROLE,
   type TenantId,
   withResponseData,
@@ -50,10 +49,6 @@ function resolveTargetTenantId(
   return parseTenantId(payloadTenantId) ?? undefined;
 }
 
-function foreignStreamTenant(user: SessionUser, targetTenantId: TenantId): TenantId | undefined {
-  return targetTenantId === user.tenantId ? undefined : targetTenantId;
-}
-
 // Editing your OWN roles keeps the current session: roles are re-derived per
 // request, so the new roles apply at once without a re-login. Other sessions
 // of that user and any foreign edit still revoke all.
@@ -195,8 +190,8 @@ export const updateMemberRolesWrite = defineWriteHandler({
     }
 
     // The membership stream lives on the membership's tenant (where seedTenantMembership/
-    // invite-accept wrote it); a cross-tenant operator addresses it without becoming a member.
-    // Same-tenant writes run on a tenant-mode db, which refuses a stream override.
+    // invite-accept wrote it); streamTenantId addresses it with the operator as actor, also
+    // without membership there.
     const result = await executor.update(
       {
         id: row["id"] as string, // @cast-boundary db-row
@@ -205,7 +200,7 @@ export const updateMemberRolesWrite = defineWriteHandler({
       },
       event.user,
       db,
-      { streamTenantId: foreignStreamTenant(event.user, targetTenantId) },
+      { streamTenantId: targetTenantId },
     );
 
     return withResponseData(result, {