@pixerate/editor 0.263.0 → 0.264.1

package/dist/brushes/index.js

@@ -8,7 +8,7 @@ import {
   createBrushLibraryToolProvider,
   createBrushLibraryTools,
   mergeBrushPatch
-} from "../chunk-CNLO6E2K.js";
+} from "../chunk-4OJC4MCD.js";
 import "../chunk-6JL2D5SG.js";
 import {
   BRUSH_FACET_SCHEMAS,
@@ -44,9 +44,11 @@ import {
   brushColourKind,
   brushToJSON,
   checkBrush,
+  checkBrushEdit,
   createBrushLibrary,
   degrade,
   getPath,
+  isBrushLibrary,
   listPresets,
   migrateBrushV1,
   normalizeBrushSpec,
@@ -57,8 +59,9 @@ import {
   resolveEngine,
   resolveWetMedium,
   setPath,
-  snapshotBrushLibrary
-} from "../chunk-P5343XZO.js";
+  snapshotBrushLibrary,
+  withKeptIssues
+} from "../chunk-EE7SAYBF.js";
 import {
   STAMP_KINDS
 } from "../chunk-ACFSFUQH.js";
@@ -125,7 +128,11 @@ function brushEditorSections(brush, profile = PAINT_PROFILE) {
     const v = getPath(resolved, field.engine);
     return field.map ? field.map.toFacet(v) : v;
   };
-  const warned = new Map(checkBrush(brush, profile).issues.filter((i) => i.severity === "warning").map((i) => [i.facet, i.reason]));
+  const issues = checkBrush(brush, profile).issues;
+  const warned = new Map(issues.filter((i) => i.severity === "warning").map((i) => [i.facet, i.reason]));
+  const refused = new Set(issues.filter((i) => i.severity === "error").map((i) => i.facet));
+  const full = (s) => s === "full";
+  const engineOk = !["kind", "tool", "tip.shape", "marks", "behaviour.surface"].some((f) => refused.has(f));
   const noteFor = (path) => [...warned].find(([facet]) => path === facet || path.startsWith(`${facet}.`))?.[1];
   const withNote = (c) => {
     const note = noteFor(c.path);
@@ -147,22 +154,23 @@ function brushEditorSections(brush, profile = PAINT_PROFILE) {
   const isTool = brush.kind === "tool";
   const shape = brush.tip.shape;
   const marks = brush.marks?.kind ?? "continuous";
+  const exchangeFull = full(brush.behaviour?.exchange?.reservoir ? profile.reservoir : profile.exchange);
   add("medium", "Medium", [
-    profile.kinds.length > 1 && choice("kind", "Kind", options(profile.kinds, { mark: "Lays a material", tool: "Works on paint" })),
+    profile.kinds.length > 1 && profile.kinds.includes(brush.kind) && choice("kind", "Kind", options(profile.kinds, { mark: "Lays a material", tool: "Works on paint" })),
     !isTool && choice("medium", "Medium", [{ value: "", label: "Any" }, ...options(BRUSH_MEDIA, MEDIUM_LABELS)], ""),
     !isTool && withNote({ kind: "text", path: "variety", label: "Variety", ariaLabel: "Brush variety", value: brush.variety ?? "", placeholder: "pencil, oil, highlighter\u2026" }),
-    isTool && profile.tools.length > 0 && choice("tool", "Tool", options(profile.tools, TOOL_LABELS))
+    isTool && !!brush.tool && profile.tools.includes(brush.tool) && choice("tool", "Tool", options(profile.tools, TOOL_LABELS))
   ]);
   const engineControls = brushEngineControls(resolved);
   const tipSliders = BRUSH_SLIDERS.filter((c) => c.path.startsWith("tip.") && c.path !== "tip.smoothing");
   add("shape", "Shape", [
-    choice("tip.shape", "Tip", options(profile.tipShapes.filter((s) => !isTool || s !== "stamp"), SHAPE_LABELS)),
+    !refused.has("tip.shape") && choice("tip.shape", "Tip", options(profile.tipShapes.filter((s) => !isTool || s !== "stamp"), SHAPE_LABELS)),
     ...tipSliders.map((c) => slider(c)),
     engine === "dab" && choice("tip.blend", "Blend", BLEND_MODES.map((m) => ({ value: m.id, label: m.label })), "normal"),
-    ...shape === "bristle" && engineControls ? engineControls.sliders.filter((c) => c.path.startsWith("bristle.")).map((c) => slider(c)) : [],
-    shape === "stamp" && withNote({ kind: "stamps", path: "tip.stamps", label: "Stamps", ariaLabel: "Brush stamps", value: [...value("tip.stamps") ?? []], max: marks === "particles" ? MAX_DOT_TEXTURES : MAX_STICKER_STAMPS, options: STAMP_OPTIONS })
+    ...shape === "bristle" && engineOk && engineControls ? engineControls.sliders.filter((c) => c.path.startsWith("bristle.")).map((c) => slider(c)) : [],
+    shape === "stamp" && engineOk && withNote({ kind: "stamps", path: "tip.stamps", label: "Stamps", ariaLabel: "Brush stamps", value: [...value("tip.stamps") ?? []], max: marks === "particles" ? MAX_DOT_TEXTURES : MAX_STICKER_STAMPS, options: STAMP_OPTIONS })
   ]);
-  if (!isTool && profile.marks.length > 1) {
+  if (!isTool && profile.marks.length > 1 && engineOk) {
     const particles = marks === "particles" && engineControls ? engineControls.sliders.filter((c) => c.path.startsWith("particle.") && c.path !== "particle.waveSpeed") : [];
     const objects = marks === "objects" && engineControls ? engineControls.sliders.filter((c) => c.path.startsWith("sticker.")) : [];
     const render = value("marks.render");
@@ -177,7 +185,7 @@ function brushEditorSections(brush, profile = PAINT_PROFILE) {
       ...objects.map((c) => slider(c))
     ]);
   }
-  const feel = BRUSH_SLIDERS.filter((c) => ["tip.smoothing", "pressureSmoothing", "dynamics.minSize", "dynamics.tiltSize", "dynamics.velocitySize"].includes(c.path)).filter((c) => !(c.path === "dynamics.tiltSize" && profile.tilt === "none") && !(c.path === "dynamics.velocitySize" && profile.velocity === "none"));
+  const feel = BRUSH_SLIDERS.filter((c) => ["tip.smoothing", "pressureSmoothing", "dynamics.minSize", "dynamics.tiltSize", "dynamics.velocitySize"].includes(c.path)).filter((c) => !(c.path === "dynamics.tiltSize" && !full(profile.tilt)) && !(c.path === "dynamics.velocitySize" && !full(profile.velocity)));
   add("feel", "Feel", [
     ...feel.map((c) => slider(c)),
     toggle("dynamics.pressureSize", "Pressure \u2192 size"),
@@ -185,19 +193,19 @@ function brushEditorSections(brush, profile = PAINT_PROFILE) {
     choice("dynamics.rotation", "Rotation", options(BRUSH_ROTATIONS, ROTATION_LABELS))
   ]);
   const surfaces = profile.surfaces;
-  const engineSliders = (engineControls?.sliders ?? []).filter((c) => !/^(bristle|particle|sticker)\./.test(c.path) && c.path !== "light.amount" && c.path !== "impasto.rim" && c.path !== "impasto.clarity");
-  const exchangeOk = (path) => !path?.startsWith("behaviour.exchange") || (path.startsWith("behaviour.exchange.reservoir") ? profile.reservoir !== "none" : profile.exchange !== "none");
+  const engineSliders = (engineOk ? engineControls?.sliders ?? [] : []).filter((c) => !/^(bristle|particle|sticker)\./.test(c.path) && c.path !== "light.amount" && c.path !== "impasto.rim" && c.path !== "impasto.clarity");
+  const exchangeOk = (path) => !path?.startsWith("behaviour.exchange") || (path.startsWith("behaviour.exchange.reservoir") ? full(profile.reservoir) : exchangeFull);
   add("behaviour", engineControls?.title ?? "Behaviour", [
-    !isTool && surfaces.length > 1 && choice("behaviour.surface", "Surface", options(surfaces, SURFACE_LABELS), "dry"),
-    profile.grain !== "none" && slider({ path: "grain", label: "Grain", min: 0, max: 1, step: 0.01 }, "behaviour.grain"),
-    engine === "wet" && !isTool && choice("behaviour.wet.medium", "Medium", options(WET_MEDIA, WET_MEDIUM_LABELS), "watercolor"),
-    engine === "impasto" && choice("behaviour.thick.applicator", "Tool", options(IMPASTO_TOOLS, APPLICATOR_LABELS), "brush"),
+    !isTool && surfaces.length > 1 && !refused.has("behaviour.surface") && choice("behaviour.surface", "Surface", options(surfaces, SURFACE_LABELS), "dry"),
+    full(profile.grain) && slider({ path: "grain", label: "Grain", min: 0, max: 1, step: 0.01 }, "behaviour.grain"),
+    engine === "wet" && engineOk && !isTool && choice("behaviour.wet.medium", "Medium", options(WET_MEDIA, WET_MEDIUM_LABELS), "watercolor"),
+    engine === "impasto" && engineOk && choice("behaviour.thick.applicator", "Tool", options(IMPASTO_TOOLS, APPLICATOR_LABELS), "brush"),
     ...engineSliders.map((c) => exchangeOk(facetOf(c.path)) ? slider(c) : null),
-    engineControls?.mixingPath && fieldOf("behaviour.mixing") && choice("behaviour.mixing", "Mixing", options(["pigment", "rgb"], { pigment: "Pigment", rgb: "RGB" })),
-    engineControls?.dirty && profile.exchange !== "none" && fieldOf("behaviour.exchange.dirty") && toggle("behaviour.exchange.dirty", "Dirty brush"),
-    engine === "mixer" && toggle("behaviour.exchange.reservoir.refill", "Load each stroke")
+    engineOk && engineControls?.mixingPath && (engine === "wet" || exchangeFull) && fieldOf("behaviour.mixing") && choice("behaviour.mixing", "Mixing", options(["pigment", "rgb"], { pigment: "Pigment", rgb: "RGB" })),
+    engineOk && engineControls?.dirty && exchangeFull && fieldOf("behaviour.exchange.dirty") && toggle("behaviour.exchange.dirty", "Dirty brush"),
+    engine === "mixer" && engineOk && full(profile.reservoir) && toggle("behaviour.exchange.reservoir.refill", "Load each stroke")
   ]);
-  if (!isTool && profile.light !== "none" && engineControls?.light) {
+  if (!isTool && full(profile.light) && engineOk && engineControls?.light) {
     add("light", "Light", [
       slider({ path: "light.amount", label: "Light", min: 0, max: 1, step: 0.01 }),
       toggle("light.pressure", "Light follows pressure"),
@@ -206,13 +214,13 @@ function brushEditorSections(brush, profile = PAINT_PROFILE) {
       toggle("light.glow", "Suggest a glow")
     ]);
   }
-  if (profile.motion !== "none" && marks === "particles") add("motion", "Motion", [slider({ path: "particle.waveSpeed", label: "Wave speed", min: 0, max: 1, step: 0.01 })]);
+  if (full(profile.motion) && engineOk && marks === "particles") add("motion", "Motion", [slider({ path: "particle.waveSpeed", label: "Wave speed", min: 0, max: 1, step: 0.01 })]);
   if (!isTool) {
     const own = brush.colour?.default;
     const hex = (c) => typeof c === "string" ? brushColourHex(c, "light") ?? null : null;
     const pair = isDualColor(own) ? own : null;
     add("colour", "Colour", [
-      withNote({ kind: "colour", path: "colour.default", label: "Default colour", ariaLabel: "Brush default colour", value: pair ? brushColourHex(pair, "light") ?? null : hex(own), dark: pair ? brushColourHex(pair, "dark") ?? null : null, gradient: own !== void 0 && typeof own === "object" && !pair }),
+      !refused.has("colour.default") && withNote({ kind: "colour", path: "colour.default", label: "Default colour", ariaLabel: "Brush default colour", value: pair ? brushColourHex(pair, "light") ?? null : hex(own), dark: pair ? brushColourHex(pair, "dark") ?? null : null, gradient: own !== void 0 && typeof own === "object" && !pair }),
       withNote({
         kind: "palette",
         path: "colour.palette",
@@ -226,6 +234,78 @@ function brushEditorSections(brush, profile = PAINT_PROFILE) {
   }
   return sections;
 }
+var MARKS_WORDS = { particles: "particle marks", objects: "sticker marks", continuous: "continuous marks" };
+function hiddenLabel(issue, b) {
+  switch (issue.facet) {
+    case "kind":
+    case "tool":
+      return b.kind === "tool" ? `the ${(TOOL_LABELS[b.tool] ?? b.tool ?? "tool").toLowerCase()} tool` : "its marks";
+    case "tip.shape":
+      return `a ${b.tip.shape} tip`;
+    case "marks":
+      return MARKS_WORDS[b.marks?.kind ?? "continuous"] ?? "its marks";
+    case "behaviour.surface":
+      return `${b.behaviour?.surface ?? "dry"} behaviour`;
+    case "behaviour.exchange.reservoir":
+      return "a Mixer Brush reservoir";
+    case "behaviour.exchange":
+      return "paint pick-up";
+    case "behaviour.grain":
+      return "paper grain";
+    case "light":
+      return "light";
+    case "motion":
+      return "motion";
+    case "dynamics.velocity":
+      return "velocity dynamics";
+    case "dynamics.tilt":
+      return "tilt dynamics";
+    case "colour.default":
+      return `a ${brushColourKind(b.colour.default)} default colour`;
+    default:
+      return null;
+  }
+}
+var sentence = (words) => words.length < 2 ? words[0] ?? "" : `${words.slice(0, -1).join(", ")} and ${words.at(-1)}`;
+function brushEditorScope(brush, profile = PAINT_PROFILE) {
+  const hidden = [];
+  for (const issue of checkBrush(brush, profile).issues) {
+    const label = hiddenLabel(issue, brush);
+    if (label && !hidden.some((h) => h.label === label)) hidden.push({ facet: issue.facet, label });
+  }
+  return { hidden, note: hidden.length ? `Also has ${sentence(hidden.map((h) => h.label))}, which ${profile.name} can't use.` : null };
+}
+function brushIdFromName(name) {
+  return name.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "").slice(0, 60) || "brush";
+}
+function newBrushFor(profile, from, name = "New brush") {
+  const id = brushIdFromName(name);
+  const base = { id, name, kind: profile.kinds.includes("mark") ? "mark" : profile.kinds[0], ...profile.kinds.includes("mark") ? {} : { tool: profile.tools[0] }, tip: { shape: profile.tipShapes.includes("round") ? "round" : profile.tipShapes[0] } };
+  if (from) {
+    const stripped = degrade(from, profile, { ignored: true });
+    if (!stripped.issues.length) return { ...stripped.brush, id, name };
+  }
+  return base;
+}
+async function saveBrushEdit(library, options2) {
+  const { original, profile } = options2;
+  const check = checkBrushEdit(original, options2.brush, profile);
+  if (!check.ok) {
+    const errors = check.issues.filter((i) => i.severity === "error").map((i) => `${i.facet}: ${i.reason} ${i.fix}`);
+    throw new BrushLibraryError("invalid", `${profile.name} can't use this brush: ${errors.join(" ")} Nothing was saved.`);
+  }
+  const inPlace = !!original && options2.name === void 0 && !library.isBuiltin(original.id) && original.id === options2.brush.id && !!await library.get(original.id);
+  let brush = options2.brush;
+  if (!inPlace) {
+    const name = ((options2.name ?? (original ? `${original.name} copy` : brush.name)).trim() || "Brush").slice(0, 80);
+    const stem = brushIdFromName(name).slice(0, 56);
+    let id = stem;
+    for (let n = 2; library.isBuiltin(id) || await library.get(id); n++) id = `${stem}-${n}`;
+    brush = { ...brush, id, name };
+  }
+  const saved = await library.save(brush, { overwrite: inPlace });
+  return { ...saved, kept: check.kept };
+}
 function setBrushEditorValue(brush, path, value) {
   const next = JSON.parse(JSON.stringify(brush));
   if (path === "kind") {
@@ -418,23 +498,30 @@ export {
   brushColourHex,
   brushColourKind,
   brushContracts,
+  brushEditorScope,
   brushEditorSections,
+  brushIdFromName,
   brushLibraryGroups,
   changedBrushPaths,
   checkBrush,
+  checkBrushEdit,
   createBrushLibrary,
   createBrushLibraryToolProvider,
   createBrushLibraryTools,
   degrade,
+  isBrushLibrary,
   listPresets,
   mergeBrushPatch,
   migrateBrushV1,
+  newBrushFor,
   normalizeBrushSpec,
   parseBrushLibrary,
   resolveBrush,
   resolveCategory,
   resolveEngine,
   resolveWetMedium,
+  saveBrushEdit,
   setBrushEditorValue,
-  snapshotBrushLibrary
+  snapshotBrushLibrary,
+  withKeptIssues
 };

package/dist/brushes/library.d.cts

@@ -91,6 +91,8 @@ export interface BrushLibrary {
     /** Called after every change. Returns an unsubscribe function. */
     subscribe(listener: () => void): () => void;
 }
+/** Whether `library` is a whole `BrushLibrary` (it can save), not only a list to read: hosts offer brush editing on one. */
+export declare function isBrushLibrary(library: unknown): library is BrushLibrary;
 /**
  * Brushes read from a library file: `{ format: 'pixerate-brushes', version: 1 | 2, brushes }` or
  * a bare array. Version 1 brushes are migrated (`migrateBrushV1`); every brush is normalised.

package/dist/brushes/library.d.ts

@@ -91,6 +91,8 @@ export interface BrushLibrary {
     /** Called after every change. Returns an unsubscribe function. */
     subscribe(listener: () => void): () => void;
 }
+/** Whether `library` is a whole `BrushLibrary` (it can save), not only a list to read: hosts offer brush editing on one. */
+export declare function isBrushLibrary(library: unknown): library is BrushLibrary;
 /**
  * Brushes read from a library file: `{ format: 'pixerate-brushes', version: 1 | 2, brushes }` or
  * a bare array. Version 1 brushes are migrated (`migrateBrushV1`); every brush is normalised.

package/dist/brushes/profiles.d.cts

@@ -66,6 +66,21 @@ export declare const BRUSH_HOST_PROFILES: Readonly<Record<string, BrushHostProfi
 export declare function brushColourKind(paint: BrushPaint): BrushColourKind;
 /** What a brush needs from a host, and what the host does about it. */
 export declare function checkBrush(brush: Brush, profile: BrushHostProfile): BrushCheck;
+export interface BrushEditCheck extends BrushCheck {
+    /**
+     * Errors the brush had before the edit: facets this host can't use, kept unchanged for the
+     * hosts that can (a light paint brush edited in Canvas keeps its light). Not in `issues`.
+     */
+    kept: BrushIssue[];
+}
+/**
+ * A check of an edit: errors the original brush already had (`before`, its check for the same
+ * host) are kept rather than refused, so a host edits what it renders and leaves the rest alone.
+ * Without `before` (a new brush) every error refuses it.
+ */
+export declare function withKeptIssues(check: BrushCheck, before: BrushCheck | null): BrushEditCheck;
+/** `checkBrush` for an edit of `original` (`null`: a new brush), keeping facets it already had that the host can't use. */
+export declare function checkBrushEdit(original: Brush | null, brush: Brush, profile: BrushHostProfile): BrushEditCheck;
 /** The presets (default: the built-in ones) a host can render, in library order. */
 export declare function listPresets(profile: BrushHostProfile, presets?: readonly Brush[]): Brush[];
 export interface DegradedBrush {
@@ -80,6 +95,10 @@ export interface DegradedBrush {
  * The brush the host can render: facets it refuses are removed (light, motion, a reservoir, a
  * wet or thick surface, particle or object marks, bristle and stamp tips become round, an
  * unsupported default colour), each reported in `dropped`. A tool the host lacks can't be
- * stripped and stays in `issues`.
+ * stripped and stays in `issues`. `ignored: true` also removes what the host paints without
+ * (its warnings: velocity on canvas, grain in the image editor, palette colours it skips), for
+ * a brush made only of what the host uses.
  */
-export declare function degrade(brush: Brush, profile: BrushHostProfile): DegradedBrush;
+export declare function degrade(brush: Brush, profile: BrushHostProfile, options?: {
+    ignored?: boolean;
+}): DegradedBrush;

package/dist/brushes/profiles.d.ts

@@ -66,6 +66,21 @@ export declare const BRUSH_HOST_PROFILES: Readonly<Record<string, BrushHostProfi
 export declare function brushColourKind(paint: BrushPaint): BrushColourKind;
 /** What a brush needs from a host, and what the host does about it. */
 export declare function checkBrush(brush: Brush, profile: BrushHostProfile): BrushCheck;
+export interface BrushEditCheck extends BrushCheck {
+    /**
+     * Errors the brush had before the edit: facets this host can't use, kept unchanged for the
+     * hosts that can (a light paint brush edited in Canvas keeps its light). Not in `issues`.
+     */
+    kept: BrushIssue[];
+}
+/**
+ * A check of an edit: errors the original brush already had (`before`, its check for the same
+ * host) are kept rather than refused, so a host edits what it renders and leaves the rest alone.
+ * Without `before` (a new brush) every error refuses it.
+ */
+export declare function withKeptIssues(check: BrushCheck, before: BrushCheck | null): BrushEditCheck;
+/** `checkBrush` for an edit of `original` (`null`: a new brush), keeping facets it already had that the host can't use. */
+export declare function checkBrushEdit(original: Brush | null, brush: Brush, profile: BrushHostProfile): BrushEditCheck;
 /** The presets (default: the built-in ones) a host can render, in library order. */
 export declare function listPresets(profile: BrushHostProfile, presets?: readonly Brush[]): Brush[];
 export interface DegradedBrush {
@@ -80,6 +95,10 @@ export interface DegradedBrush {
  * The brush the host can render: facets it refuses are removed (light, motion, a reservoir, a
  * wet or thick surface, particle or object marks, bristle and stamp tips become round, an
  * unsupported default colour), each reported in `dropped`. A tool the host lacks can't be
- * stripped and stays in `issues`.
+ * stripped and stays in `issues`. `ignored: true` also removes what the host paints without
+ * (its warnings: velocity on canvas, grain in the image editor, palette colours it skips), for
+ * a brush made only of what the host uses.
  */
-export declare function degrade(brush: Brush, profile: BrushHostProfile): DegradedBrush;
+export declare function degrade(brush: Brush, profile: BrushHostProfile, options?: {
+    ignored?: boolean;
+}): DegradedBrush;

package/dist/brushes/ui.d.cts

@@ -8,8 +8,8 @@
  * Sliders reuse the paint editor's tables (`BRUSH_SLIDERS`, `brushEngineControls`), mapped from
  * engine paths to facet paths through `BRUSH_FIELDS`, so the panels and the editor agree.
  */
-import type { BrushLibraryEntry } from './library.cjs';
-import { type BrushHostProfile } from './profiles.cjs';
+import { type BrushLibrary, type BrushLibraryEntry, type SaveBrushResult } from './library.cjs';
+import { type BrushHostProfile, type BrushIssue } from './profiles.cjs';
 import { type Brush } from './types.cjs';
 export type BrushEditorSectionId = 'medium' | 'shape' | 'marks' | 'feel' | 'behaviour' | 'light' | 'motion' | 'colour';
 interface ControlBase {
@@ -79,10 +79,63 @@ export interface BrushEditorSection {
 }
 /**
  * Sections of controls for a brush, in order, as the host `profile` (default the paint editor)
- * can render them: sections and controls it refuses are left out, ones it paints without carry
- * a note. Values are the brush's own, else the defaults of the engine it resolves to.
+ * can render them: only the sections, controls and options the host uses are listed. Facets of
+ * the brush the host can't use (a paint brush's light opened in Canvas) get no control, so an
+ * edit keeps them as they are; `brushEditorScope` names them. Values are the brush's own, else
+ * the defaults of the engine it resolves to.
  */
 export declare function brushEditorSections(brush: Brush, profile?: BrushHostProfile): BrushEditorSection[];
+/** What a brush has that the host can't use: one facet, in words. */
+export interface BrushEditorHiddenFacet {
+    facet: string;
+    /** In words, for a sentence: `light`, `wet behaviour`, `a bristle tip`. */
+    label: string;
+}
+export interface BrushEditorScope {
+    /** Facets the brush has that the host can't use (refused or painted without): they get no control and an edit keeps them. */
+    hidden: BrushEditorHiddenFacet[];
+    /** A short note naming them ("Also has light and wet behaviour, which Canvas can't use."), or `null`. */
+    note: string | null;
+}
+/**
+ * What the editor leaves out of a brush for this host: the facets the host refuses or paints
+ * without, which `brushEditorSections` gives no control, so saving an edit keeps them unchanged
+ * for the hosts that use them. `note` names them for the editor to show.
+ */
+export declare function brushEditorScope(brush: Brush, profile?: BrushHostProfile): BrushEditorScope;
+/** A brush id from a name: lowercase letters, digits and dashes (`'brush'` when nothing is left). */
+export declare function brushIdFromName(name: string): string;
+/**
+ * A new brush for a host to start editing: `from` (the brush in use) with every facet the host
+ * can't use removed (`degrade` with `ignored`), else a plain round brush. It passes `checkBrush`
+ * for that host, and the editor then offers only what the host uses, so it stays that way.
+ */
+export declare function newBrushFor(profile: BrushHostProfile, from?: Brush | null, name?: string): Brush;
+export interface SaveBrushEditOptions {
+    /** The brush as it was before editing (`null`: a new brush). Facets of it the host can't use are kept. */
+    original: Brush | null;
+    /** The edited brush. */
+    brush: Brush;
+    /** The host the edit was made in. */
+    profile: BrushHostProfile;
+    /**
+     * Save as a new brush with this name (its id made from the name, `-2`… when taken). Without
+     * it the original is changed in place, or a copy is saved when it is built in or new.
+     */
+    name?: string;
+}
+/** What `saveBrushEdit` stored, plus the facets kept for other hosts. */
+export interface SaveBrushEditResult extends SaveBrushResult {
+    /** Facets this host can't use that the brush keeps (see `checkBrushEdit`). */
+    kept: BrushIssue[];
+}
+/**
+ * Saves a brush edited in a host's editor to the shared library, by the rule the library tools
+ * follow too: the edit may not add what the host can't use (a new brush must pass `checkBrush`),
+ * and what the original had for other hosts is kept. Built-in presets are never changed: the
+ * edit is saved as a copy (`<name> copy`). Refusals are `BrushLibraryError`s with the facet and fix.
+ */
+export declare function saveBrushEdit(library: Pick<BrushLibrary, 'get' | 'save' | 'isBuiltin'>, options: SaveBrushEditOptions): Promise<SaveBrushEditResult>;
 /**
  * The brush with one control's value set (`undefined` removes it), normalised. Changing the kind,
  * the marks or the tip shape starts that facet afresh. Returns the brush and what was clamped or

package/dist/brushes/ui.d.ts

@@ -8,8 +8,8 @@
  * Sliders reuse the paint editor's tables (`BRUSH_SLIDERS`, `brushEngineControls`), mapped from
  * engine paths to facet paths through `BRUSH_FIELDS`, so the panels and the editor agree.
  */
-import type { BrushLibraryEntry } from './library.js';
-import { type BrushHostProfile } from './profiles.js';
+import { type BrushLibrary, type BrushLibraryEntry, type SaveBrushResult } from './library.js';
+import { type BrushHostProfile, type BrushIssue } from './profiles.js';
 import { type Brush } from './types.js';
 export type BrushEditorSectionId = 'medium' | 'shape' | 'marks' | 'feel' | 'behaviour' | 'light' | 'motion' | 'colour';
 interface ControlBase {
@@ -79,10 +79,63 @@ export interface BrushEditorSection {
 }
 /**
  * Sections of controls for a brush, in order, as the host `profile` (default the paint editor)
- * can render them: sections and controls it refuses are left out, ones it paints without carry
- * a note. Values are the brush's own, else the defaults of the engine it resolves to.
+ * can render them: only the sections, controls and options the host uses are listed. Facets of
+ * the brush the host can't use (a paint brush's light opened in Canvas) get no control, so an
+ * edit keeps them as they are; `brushEditorScope` names them. Values are the brush's own, else
+ * the defaults of the engine it resolves to.
  */
 export declare function brushEditorSections(brush: Brush, profile?: BrushHostProfile): BrushEditorSection[];
+/** What a brush has that the host can't use: one facet, in words. */
+export interface BrushEditorHiddenFacet {
+    facet: string;
+    /** In words, for a sentence: `light`, `wet behaviour`, `a bristle tip`. */
+    label: string;
+}
+export interface BrushEditorScope {
+    /** Facets the brush has that the host can't use (refused or painted without): they get no control and an edit keeps them. */
+    hidden: BrushEditorHiddenFacet[];
+    /** A short note naming them ("Also has light and wet behaviour, which Canvas can't use."), or `null`. */
+    note: string | null;
+}
+/**
+ * What the editor leaves out of a brush for this host: the facets the host refuses or paints
+ * without, which `brushEditorSections` gives no control, so saving an edit keeps them unchanged
+ * for the hosts that use them. `note` names them for the editor to show.
+ */
+export declare function brushEditorScope(brush: Brush, profile?: BrushHostProfile): BrushEditorScope;
+/** A brush id from a name: lowercase letters, digits and dashes (`'brush'` when nothing is left). */
+export declare function brushIdFromName(name: string): string;
+/**
+ * A new brush for a host to start editing: `from` (the brush in use) with every facet the host
+ * can't use removed (`degrade` with `ignored`), else a plain round brush. It passes `checkBrush`
+ * for that host, and the editor then offers only what the host uses, so it stays that way.
+ */
+export declare function newBrushFor(profile: BrushHostProfile, from?: Brush | null, name?: string): Brush;
+export interface SaveBrushEditOptions {
+    /** The brush as it was before editing (`null`: a new brush). Facets of it the host can't use are kept. */
+    original: Brush | null;
+    /** The edited brush. */
+    brush: Brush;
+    /** The host the edit was made in. */
+    profile: BrushHostProfile;
+    /**
+     * Save as a new brush with this name (its id made from the name, `-2`… when taken). Without
+     * it the original is changed in place, or a copy is saved when it is built in or new.
+     */
+    name?: string;
+}
+/** What `saveBrushEdit` stored, plus the facets kept for other hosts. */
+export interface SaveBrushEditResult extends SaveBrushResult {
+    /** Facets this host can't use that the brush keeps (see `checkBrushEdit`). */
+    kept: BrushIssue[];
+}
+/**
+ * Saves a brush edited in a host's editor to the shared library, by the rule the library tools
+ * follow too: the edit may not add what the host can't use (a new brush must pass `checkBrush`),
+ * and what the original had for other hosts is kept. Built-in presets are never changed: the
+ * edit is saved as a copy (`<name> copy`). Refusals are `BrushLibraryError`s with the facet and fix.
+ */
+export declare function saveBrushEdit(library: Pick<BrushLibrary, 'get' | 'save' | 'isBuiltin'>, options: SaveBrushEditOptions): Promise<SaveBrushEditResult>;
 /**
  * The brush with one control's value set (`undefined` removes it), normalised. Changing the kind,
  * the marks or the tip shape starts that facet afresh. Returns the brush and what was clamped or

package/dist/{brushstrokeRender-S5DABANG.js → brushstrokeRender-3MOI7PH4.js}

@@ -1,6 +1,6 @@
 import {
   pathDataToPolyline
-} from "./chunk-LG6MEEV7.js";
+} from "./chunk-JEML4FSA.js";
 import {
   BRUSH_PRESETS,
   BUILTIN_BRUSHES,
@@ -11,7 +11,7 @@ import {
   normalizeBrushSpec,
   resolveBrush,
   resolveCategory
-} from "./chunk-P5343XZO.js";
+} from "./chunk-EE7SAYBF.js";
 import "./chunk-ACFSFUQH.js";
 import {
   formatNumber,

package/dist/{brushstrokeRender-2GBME4XD.cjs → brushstrokeRender-SMMPIJF2.cjs}

@@ -1,6 +1,6 @@
 "use strict";Object.defineProperty(exports, "__esModule", {value: true}); function _nullishCoalesce(lhs, rhsFn) { if (lhs != null) { return lhs; } else { return rhsFn(); } } function _optionalChain(ops) { let lastAccessLHS = undefined; let value = ops[0]; let i = 1; while (i < ops.length) { const op = ops[i]; const fn = ops[i + 1]; i += 2; if ((op === 'optionalAccess' || op === 'optionalCall') && value == null) { return undefined; } if (op === 'access' || op === 'optionalAccess') { lastAccessLHS = value; value = fn(value); } else if (op === 'call' || op === 'optionalCall') { value = fn((...args) => value.call(lastAccessLHS, ...args)); lastAccessLHS = undefined; } } return value; }
 
-var _chunk4E6BGHWNcjs = require('./chunk-4E6BGHWN.cjs');
+var _chunkG5ADO47Gcjs = require('./chunk-G5ADO47G.cjs');
 
 
 
@@ -11,7 +11,7 @@ var _chunk4E6BGHWNcjs = require('./chunk-4E6BGHWN.cjs');
 
 
 
-var _chunkZBACQHW3cjs = require('./chunk-ZBACQHW3.cjs');
+var _chunkBY5EBP26cjs = require('./chunk-BY5EBP26.cjs');
 require('./chunk-OTN6VB5S.cjs');
 
 
@@ -45,17 +45,17 @@ require('./chunk-5UEFBM2M.cjs');
 require('./chunk-IPJ3TEYR.cjs');
 
 // src/canvas/brushstrokeRender.ts
-var PRESETS = _chunkZBACQHW3cjs.listPresets.call(void 0, _chunkZBACQHW3cjs.CANVAS_PROFILE).map((spec) => _chunkZBACQHW3cjs.BUILTIN_BRUSHES.find((b) => b.id === spec.id));
+var PRESETS = _chunkBY5EBP26cjs.listPresets.call(void 0, _chunkBY5EBP26cjs.CANVAS_PROFILE).map((spec) => _chunkBY5EBP26cjs.BUILTIN_BRUSHES.find((b) => b.id === spec.id));
 var presetIds = () => PRESETS.map((b) => b.id).join(", ");
-var refusal = (brush) => _chunkZBACQHW3cjs.checkBrush.call(void 0, brush, _chunkZBACQHW3cjs.CANVAS_PROFILE).issues.filter((i) => i.severity === "error").map((i) => `${i.reason} ${i.fix}`).join(" ");
+var refusal = (brush) => _chunkBY5EBP26cjs.checkBrush.call(void 0, brush, _chunkBY5EBP26cjs.CANVAS_PROFILE).issues.filter((i) => i.severity === "error").map((i) => `${i.reason} ${i.fix}`).join(" ");
 function readCanvasBrushSpec(spec) {
   let brush;
   try {
-    brush = _chunkZBACQHW3cjs.normalizeBrushSpec.call(void 0, spec);
+    brush = _chunkBY5EBP26cjs.normalizeBrushSpec.call(void 0, spec);
   } catch (e) {
     throw new Error(`Not a brush: ${e.message}`);
   }
-  if (!_chunkZBACQHW3cjs.checkBrush.call(void 0, brush, _chunkZBACQHW3cjs.CANVAS_PROFILE).ok) throw new Error(`Brush '${brush.id}' (${brush.name}) can't brush a canvas stroke: ${refusal(brush)}`);
+  if (!_chunkBY5EBP26cjs.checkBrush.call(void 0, brush, _chunkBY5EBP26cjs.CANVAS_PROFILE).ok) throw new Error(`Brush '${brush.id}' (${brush.name}) can't brush a canvas stroke: ${refusal(brush)}`);
   return brush;
 }
 function listCanvasBrushes(user = []) {
@@ -63,12 +63,12 @@ function listCanvasBrushes(user = []) {
     const d = _optionalChain([spec, 'access', _2 => _2.colour, 'optionalAccess', _3 => _3.default]);
     return typeof d === "string" || _chunkMLLTWAXHcjs.isDualColor.call(void 0, d) ? { defaultColor: d } : {};
   };
-  const out = PRESETS.map((b) => ({ id: b.id, name: b.name, category: b.category, size: b.tip.size, builtin: true, ...colourOf(_chunkZBACQHW3cjs.BRUSH_PRESETS.find((p) => p.id === b.id)) }));
+  const out = PRESETS.map((b) => ({ id: b.id, name: b.name, category: b.category, size: b.tip.size, builtin: true, ...colourOf(_chunkBY5EBP26cjs.BRUSH_PRESETS.find((p) => p.id === b.id)) }));
   for (const raw of user) {
     try {
       const brush = readCanvasBrushSpec(raw);
       if (out.some((b) => b.id === brush.id)) continue;
-      out.push({ id: brush.id, name: brush.name, category: _chunkZBACQHW3cjs.resolveCategory.call(void 0, brush), size: _chunkZBACQHW3cjs.resolveBrush.call(void 0, brush).tip.size, builtin: false, spec: brush, ...colourOf(brush) });
+      out.push({ id: brush.id, name: brush.name, category: _chunkBY5EBP26cjs.resolveCategory.call(void 0, brush), size: _chunkBY5EBP26cjs.resolveBrush.call(void 0, brush).tip.size, builtin: false, spec: brush, ...colourOf(brush) });
     } catch (e2) {
     }
   }
@@ -79,15 +79,15 @@ function resolveCanvasBrush(brush, width, spec) {
   const sized = (b) => given === void 0 ? b : { ...b, tip: { ...b.tip, size: given } };
   if (typeof brush === "string") {
     if (spec !== void 0 && spec !== null) {
-      const saved = _chunkZBACQHW3cjs.resolveBrush.call(void 0, readCanvasBrushSpec(spec));
+      const saved = _chunkBY5EBP26cjs.resolveBrush.call(void 0, readCanvasBrushSpec(spec));
       if (!_chunkPXDIAYB6cjs.isBrushstrokeBrush.call(void 0, saved)) throw new Error(`Brush '${saved.id}' (${saved.name}) can't brush a canvas stroke: it is not a dry dab brush.`);
       return sized(saved);
     }
     const preset = PRESETS.find((b) => b.id === brush);
     if (preset) return sized(preset);
-    const other = _chunkZBACQHW3cjs.BRUSH_PRESETS.find((b) => b.id === brush);
+    const other = _chunkBY5EBP26cjs.BRUSH_PRESETS.find((b) => b.id === brush);
     if (other) throw new Error(`Brush '${brush}' (${other.name}) can't brush a canvas stroke: ${refusal(other)} Use one of: ${presetIds()}, or a saved brush the canvas can paint.`);
-    throw new Error(`Unknown brush '${brush}'. ${_chunkZBACQHW3cjs.closest.call(void 0, brush, PRESETS.map((b) => b.id))}Use one of the dry presets (${presetIds()}), a saved brush from the brush library, or tip settings ({ hardness, spacing, flow, \u2026 }).`);
+    throw new Error(`Unknown brush '${brush}'. ${_chunkBY5EBP26cjs.closest.call(void 0, brush, PRESETS.map((b) => b.id))}Use one of the dry presets (${presetIds()}), a saved brush from the brush library, or tip settings ({ hardness, spacing, flow, \u2026 }).`);
   }
   if (!brush || typeof brush !== "object" || Array.isArray(brush)) {
     throw new Error("A brush is a preset or saved brush id (see listCanvasBrushes / canvas.listBrushes) or an object of tip settings.");
@@ -123,7 +123,7 @@ function renderShapeBrushstrokeNow(shape, scale, place = {}) {
   const textured = typeof seed === "number" && Number.isFinite(seed);
   const size = _nullishCoalesce(shape.brush.size, () => ( (textured ? void 0 : _nullishCoalesce(shape.strokeWidth, () => ( 3)))));
   const brush = resolveCanvasBrush(shape.brush.brush, size, shape.brush.spec);
-  const points = _chunk4E6BGHWNcjs.pathDataToPolyline.call(void 0, shape.d);
+  const points = _chunkG5ADO47Gcjs.pathDataToPolyline.call(void 0, shape.d);
   if (!points.length) return null;
   const { pressure, tiltX, tiltY } = shape.brush;
   const input = {