@aglyn/plugins-ai 1.0.0-beta.258 → 1.0.0-beta.260

package/src/lib/server/assist-edit.js

@@ -18,6 +18,7 @@ import { _ as _extends } from "@swc/helpers/_/_extends";
 import { linealRelationshipPermits } from "@aglyn/aglyn/app-utils/lineal-order";
 import { SCREEN_SEO_TEXT_FIELDS, SCREEN_SEO_TEXT_GUIDANCE } from "@aglyn/aglyn/app-utils/screen-seo-fields";
 import { CANVAS_ROOT_ELEMENT_ID } from "@aglyn/aglyn/foundation/constants/canvas";
+import { ANIMATION_EASINGS, ANIMATION_MAX_DELAY_MS, ANIMATION_MAX_DURATION_MS, ANIMATION_MAX_STAGGER_STEP_MS, ANIMATION_NONE, ANIMATION_PRESETS, ANIMATION_TRIGGERS, NODE_ANIMATION_DELAY_PROP, NODE_ANIMATION_DURATION_PROP, NODE_ANIMATION_EASE_PROP, NODE_ANIMATION_PROP, NODE_ANIMATION_REPEAT_PROP, NODE_ANIMATION_STAGGER_PROP, NODE_ANIMATION_STAGGER_STEP_PROP, NODE_ANIMATION_TRIGGER_PROP } from "@aglyn/aglyn/app-utils/element-animation";
 import { NODE_MAX_INTERACTIONS, nodeInteractionSelector } from "@aglyn/aglyn/app-utils/node-interactions";
 import { ACTION_MAX_STEPS, CLIENT_INTERACTION_STEP_LABELS, validateInteraction } from "@aglyn/aglyn/app-utils/site-interactions";
 import { ASSIST_EDIT_ACTION_ID, ASSIST_EDIT_CONTEXT_MAX_NODES, ASSIST_EDIT_CONTEXT_MAX_PROPS, ASSIST_EDIT_INTERACTION_EVENTS, ASSIST_EDIT_INTERACTION_STEPS, ASSIST_EDIT_MAX_DROPPED_REASONS, ASSIST_EDIT_OP_KINDS, ASSIST_EDIT_OUTLINE_TEXT_CHARS, ASSIST_EDIT_SELECTED_TEXT_CHARS, ASSIST_EDIT_TOOL_NAME, summarizeAssistEditOps } from "../model/assist-edit.js";
@@ -220,6 +221,66 @@ const strictObject = (properties)=>({
         strict: true
     };
 }
+/* ------------------------------------------------------------------ *
+ * An element's entrance motion (AGL-3838)
+ * ------------------------------------------------------------------ */ /**
+ * The settings an element's Animation section writes, and the values each
+ * takes: the vocabulary's own lists, and a whole number of milliseconds up to
+ * its cap. `none` takes an animation off, as the section's None does.
+ */ const MOTION_SETTINGS = {
+    [NODE_ANIMATION_PROP]: ANIMATION_PRESETS,
+    [NODE_ANIMATION_TRIGGER_PROP]: ANIMATION_TRIGGERS,
+    [NODE_ANIMATION_EASE_PROP]: ANIMATION_EASINGS,
+    [NODE_ANIMATION_DURATION_PROP]: ANIMATION_MAX_DURATION_MS,
+    [NODE_ANIMATION_DELAY_PROP]: ANIMATION_MAX_DELAY_MS,
+    [NODE_ANIMATION_STAGGER_STEP_PROP]: ANIMATION_MAX_STAGGER_STEP_MS,
+    [NODE_ANIMATION_STAGGER_PROP]: 'switch',
+    [NODE_ANIMATION_REPEAT_PROP]: 'switch'
+};
+/**
+ * The line of the edit protocol that offers motion: a pure function of the
+ * vocabulary, so it sits in the cached prefix with the rest of the block.
+ */ export const ASSIST_EDIT_MOTION_LINE = `- Animation: updateProps also sets ${NODE_ANIMATION_PROP} (${ANIMATION_PRESETS.filter((preset)=>preset !== ANIMATION_NONE).join(', ')}; ${ANIMATION_NONE} removes it), ` + `${NODE_ANIMATION_TRIGGER_PROP} (${ANIMATION_TRIGGERS.join(', ')}), ${NODE_ANIMATION_EASE_PROP} (${ANIMATION_EASINGS.join(', ')}), ` + `${NODE_ANIMATION_DURATION_PROP} and ${NODE_ANIMATION_DELAY_PROP} (ms ≤${ANIMATION_MAX_DURATION_MS}), ` + `${NODE_ANIMATION_STAGGER_PROP} ("true": its children in turn), ${NODE_ANIMATION_STAGGER_STEP_PROP} (ms ≤${ANIMATION_MAX_STAGGER_STEP_MS}).`;
+/**
+ * An `updateProps` patch split into its motion settings, held to the
+ * vocabulary, and everything else, which the palette validator judges.
+ *
+ * Every element takes the motion settings — the renderer reads them off any
+ * node — so they are in no palette entry and the palette validator would drop
+ * them all. They are held here instead, and a value outside the vocabulary or
+ * past a cap is REFUSED with its reason, never coerced: a proposal that says
+ * "slide in over ten seconds" and lands as three would be an edit the card
+ * misdescribes.
+ */ export function assistEditMotionPatch(props) {
+    const motion = {};
+    const rest = {};
+    const refused = [];
+    for (const [name, raw] of Object.entries(props)){
+        const allowed = MOTION_SETTINGS[name];
+        if (allowed === undefined) {
+            rest[name] = raw;
+            continue;
+        }
+        const text = String(raw != null ? raw : '').trim();
+        if (allowed === 'switch') {
+            if (text === 'true' || text === 'false') motion[name] = text === 'true';
+            else refused.push(`${name} is "true" or "false"`);
+        } else if (typeof allowed === 'number') {
+            const ms = /^\d{1,5}$/.test(text) ? Number(text) : NaN;
+            if (ms <= allowed) motion[name] = ms;
+            else refused.push(`${name} is a whole number of milliseconds up to ${allowed}`);
+        } else if (allowed.includes(text)) {
+            motion[name] = text;
+        } else {
+            refused.push(`${name} is one of ${allowed.join(', ')}`);
+        }
+    }
+    return {
+        motion,
+        rest,
+        refused
+    };
+}
 /* ------------------------------------------------------------------ *
  * The cached block
  * ------------------------------------------------------------------ */ const DOCUMENT_NOUNS = {
@@ -244,6 +305,7 @@ const strictObject = (properties)=>({
         ASSIST_MUSIC_EDIT_LINE,
         'Operations — every field is required; fill the ones an operation does not use with "", [] or -1:',
         '- updateProps: nodeId and props — only the settings to change, as name/value strings ("true" or "false" for a switch). Every other setting on the element is kept.',
+        ASSIST_EDIT_MOTION_LINE,
         `- updateSx: nodeId and sx as key/value/breakpoint. A color is a theme token — ${AI_SX_TOKENS.palette.join(', ')} — never a hex, rgb or color name. A unitless number is theme spacing units; a size carries its unit ("320px", "50%"). breakpoint is "" or one of ${AI_SX_TOKENS.breakpoints.join(', ')}.`,
         '- insertSubtree: parentId, index (-1 appends) and nodes — the new elements, nodes[0] their root, each listing its children by the ids you gave them in this operation. Build only from the elements below, and keep it small.',
         '- move: nodeId, parentId and index — the position the element takes among the parent’s children once moved (-1 appends).',
@@ -1096,38 +1158,54 @@ const DELAYED_STEPS = new Set([
                     return;
                 }
             case 'updateProps':
+                {
+                    // Motion settings are held to the animation vocabulary, the rest to the palette (AGL-3838).
+                    const { motion, rest, refused } = assistEditMotionPatch(pending.props);
+                    if (refused.length) {
+                        violations.push(`${label}: ${refused.join('; ')}`);
+                        return;
+                    }
+                    if (!Object.keys(rest).length) {
+                        ops.push(_extends({}, pending, {
+                            props: motion
+                        }));
+                        return;
+                    }
+                    const result = validateAiNodePatch(pending.componentId, {
+                        props: rest
+                    });
+                    if (result.ok === false) {
+                        violations.push(`${label}: ${result.error}`);
+                        return;
+                    }
+                    if (!Object.keys(result.props).length) {
+                        violations.push(`${label}: ${result.repairs.length ? result.repairs.slice(0, 2).join('; ') : 'none of the settings named are ones this element takes'}`);
+                        return;
+                    }
+                    ops.push(_extends({}, pending, {
+                        props: _extends({}, result.props, motion)
+                    }));
+                    return;
+                }
             case 'updateSx':
                 {
-                    const result = validateAiNodePatch(pending.componentId, pending.op === 'updateProps' ? {
-                        props: pending.props
-                    } : {
+                    const result = validateAiNodePatch(pending.componentId, {
                         sx: pending.sx
                     });
                     if (result.ok === false) {
                         violations.push(`${label}: ${result.error}`);
                         return;
                     }
-                    // The first repairs say why a patch came out empty — a prop the
+                    // The first repairs say why a patch came out empty — a style the
                     // element does not take reads differently from one that carried
                     // markup, and the reader of `dropped` needs to tell them apart.
-                    const why = (fallback)=>result.repairs.length ? result.repairs.slice(0, 2).join('; ') : fallback;
-                    if (pending.op === 'updateProps') {
-                        if (!Object.keys(result.props).length) {
-                            violations.push(`${label}: ${why('none of the settings named are ones this element takes')}`);
-                            return;
-                        }
-                        ops.push(_extends({}, pending, {
-                            props: result.props
-                        }));
-                    } else {
-                        if (!result.sx) {
-                            violations.push(`${label}: ${why('none of the styles named are allowed')}`);
-                            return;
-                        }
-                        ops.push(_extends({}, pending, {
-                            sx: result.sx
-                        }));
+                    if (!result.sx) {
+                        violations.push(`${label}: ${result.repairs.length ? result.repairs.slice(0, 2).join('; ') : 'none of the styles named are allowed'}`);
+                        return;
                     }
+                    ops.push(_extends({}, pending, {
+                        sx: result.sx
+                    }));
                     return;
                 }
             case 'move':