@pinepaper.studio/mcp-server 1.6.27 → 1.6.28

package/dist/types/code-generator.js

@@ -1333,6 +1333,25 @@ function editDistance(a, b) {
     return row[b.length];
 }
 export function relationOptionWarning(relationType, params, injected = []) {
+    const flip = equationFlipNote(relationType, params);
+    const unread = unreadOptionWarning(relationType, params, injected);
+    return unread && flip ? `${unread} ${flip}` : unread ?? flip;
+}
+/**
+ * An equation route is in math coordinates, y up, unless flipY: false. Authored
+ * as screen coordinates it rode the curve mirrored about cy, every centroid
+ * within a pixel of the mirror and up to 300 px off the curve as written, with
+ * nothing said (ORBDUR26, ASYMORB27). Said when the caller left flipY to the default.
+ */
+function equationFlipNote(relationType, params) {
+    if (relationType !== 'moves_along_path' || !params || typeof params !== 'object')
+        return null;
+    const eq = params.equation;
+    if (!eq || typeof eq !== 'object' || eq.flipY !== undefined)
+        return null;
+    return 'params.equation is read with y UP (flipY defaults to true), so on screen the curve is mirrored about cy. If you wrote it in screen coordinates (y down), add flipY: false to the equation.';
+}
+function unreadOptionWarning(relationType, params, injected) {
     const keys = typeof relationType === 'string' ? RELATION_PARAM_KEYS[relationType] : undefined;
     if (!keys || !params || typeof params !== 'object')
         return null;
@@ -1710,10 +1729,18 @@ const relations = results.map(r => ({
 /**
  * Template for simple animation
  */
-function generateAnimateCode(itemId, animationType, speed, intensity, delay) {
+function generateAnimateCode(itemId, animationType, speed, intensity, delay, waveform, origin) {
     const ID = jsLiteral(itemId); // caller text: a quoted literal, never spliced raw
     const extra = (intensity !== undefined ? `,\n  animationIntensity: ${intensity}` : '') +
-        (delay !== undefined ? `,\n  animationDelay: ${delay}` : '');
+        (delay !== undefined ? `,\n  animationDelay: ${delay}` : '') +
+        (waveform !== undefined ? `,\n  animationWaveform: ${jsLiteral(waveform)}` : '') +
+        (origin !== undefined ? `,\n  animationOrigin: ${jsLiteral(origin)}` : '');
+    // A studio from before field motions (FxTool #255) drops waveform and origin
+    // without a word, and the wave runs as sine from the centre. Read them back.
+    const optionCheck = waveform === undefined && origin === undefined ? '' : `
+  const __unset = [${waveform !== undefined ? `['waveform', 'animationWaveform', ${jsLiteral(waveform)}]` : ''}${waveform !== undefined && origin !== undefined ? ', ' : ''}${origin !== undefined ? `['origin', 'animationOrigin', ${jsLiteral(origin)}]` : ''}]
+    .filter(function (o) { return !item.data || item.data[o[1]] !== o[2]; }).map(function (o) { return o[0]; });
+  if (__unset.length) out.warning = __unset.join(' and ') + ' did not take: this studio predates field-motion options (or refused the value), so the wave runs as sine from the centre. Update the studio.';`;
     return `
 // Apply ${oneLine(animationType)} animation to ${ID}
 const item = app.getItemById(${ID});
@@ -1740,7 +1767,8 @@ app.animate(item, {
       known: r.known,
     };
   }
-  return { success: true, itemId: ${ID}, animationType: ${jsLiteral(animationType)} };
+  const out = { success: true, itemId: ${ID}, animationType: ${jsLiteral(animationType)} };${optionCheck}
+  return out;
 })();
 `.trim();
 }
@@ -1816,6 +1844,12 @@ const TEXT_KEY_ENGINE_NAMES = {
     textAlign: ['textAlign', 'justification'],
     lineHeight: ['lineHeight', 'leading'],
     letterSpacing: ['letterSpacing'],
+    // Text style keys no studio has animated so far. Keyed, they were stored and
+    // drew nothing, with success:true (WORDSPC26). Checked against the same list,
+    // so a studio that learns one is believed and the warning stops.
+    wordSpacing: ['wordSpacing'],
+    textDecoration: ['textDecoration'],
+    textTransform: ['textTransform'],
 };
 /** A JS expression (run in the page) → the keys of `keys` this studio does not animate. '' when none are text keys. */
 function unanimatedTextKeysJs(keys) {
@@ -1830,7 +1864,7 @@ function unanimatedTextKeysJs(keys) {
   })()`;
 }
 /** The sentence for the keys unanimatedTextKeysJs found, built in the page. */
-const UNANIMATED_TEXT_KEYS_SENTENCE = `' is not an animatable property in this studio (app.listAnimatableProperties does not list it), so it had no effect. The current studio animates fontStyle, textAlign, lineHeight and letterSpacing; here, set a fixed value with pinepaper_modify_item, or animate fontSize or scaleX.'`;
+const UNANIMATED_TEXT_KEYS_SENTENCE = `' is not an animatable property in this studio (app.listAnimatableProperties does not list it), so it had no effect: a text key animates only where that list names it. Here, set a fixed value with pinepaper_modify_item, or animate fontSize or scaleX.'`;
 function generateKeyframeAnimateCode(itemId, keyframes, duration, loop = false, timeOffset, clipInPoint, clipOutPoint, timeUnits, append = false, easingAt) {
     const ID = jsLiteral(itemId); // caller text: a quoted literal, never spliced raw
     const keyframesJson = JSON.stringify(normalizeKeyframePositions(keyframes), null, 2);
@@ -3282,7 +3316,7 @@ export class PinePaperCodeGenerator {
      */
     generateAnimate(input) {
         const validated = AnimateItemInputSchema.parse(input);
-        return generateAnimateCode(validated.itemId, validated.animationType, validated.speed, validated.intensity, validated.delay);
+        return generateAnimateCode(validated.itemId, validated.animationType, validated.speed, validated.intensity, validated.delay, validated.waveform, validated.origin);
     }
     /**
      * Generate code for keyframe animation
@@ -4878,7 +4912,24 @@ return __created;
                 // The emitter embeds the id as a literal; a batch ref ($N) is only known
                 // at run time, so it is generated with a placeholder and the runtime
                 // variable put in its place.
-                const single = generateModifyItemCode('__MID__', (op.properties || {})).trim()
+                // A TOP-LEVEL position moves the item, as it places one on create (#42: it reported modified:true and the
+                // item stayed put; only properties were read). properties.x / y / position, when given, win.
+                const modProps = { ...(op.properties || {}) };
+                const pos = op.position;
+                const hasOwn = modProps.x !== undefined || modProps.y !== undefined || modProps.position !== undefined;
+                if (pos !== undefined && !hasOwn) {
+                    const [px, py] = Array.isArray(pos) ? pos : [pos.x, pos.y];
+                    if (typeof px === 'number')
+                        modProps.x = px;
+                    if (typeof py === 'number')
+                        modProps.y = py;
+                }
+                if (!Object.keys(modProps).length) {
+                    return `
+return { success: false, itemId: ${itemRef}, error: 'modify changes nothing: pass properties ({"x": 700}, {"fillColor": "#f00"}) or a top-level position.' };
+`;
+                }
+                const single = generateModifyItemCode('__MID__', modProps).trim()
                     .replace(/"__MID__"/g, '__mid')
                     .replace(/__MID__/g, "' + __mid + '")
                     .replace(/;\s*$/, '');
@@ -4940,7 +4991,7 @@ const targetId = ${noTarget ? 'null' : tgtRef};
 const target = ${noTarget ? 'null' : 'app.getItemById(targetId)'};
 ${noTarget ? '' : "if (!target) throw new Error('Target not found: ' + targetId);\n"}${op.update === true ? "if (typeof app.modifyRelationParams !== 'function') throw new Error('this studio cannot change a bound relation (app.modifyRelationParams): update PinePaper Studio');\n" : ''}const __done = [], __failed = [];
 __members.forEach((m, i) => {
-  const __p = Object.assign({}, ${relOpts}${glob ? ', { index: i, count: __members.length }' : ''});
+  const __p = Object.assign({}, ${relOpts}${glob ? `, { index: i, count: __members.length, pattern: ${jsLiteral(op.sourceId)} }` : ''});
   const ok = ${op.update === true
                         ? `app.modifyRelationParams(m.id, targetId, ${type}, __p) || (targetId === null && app.modifyRelationParams(m.id, undefined, ${type}, __p))`
                         : glob
@@ -4955,7 +5006,7 @@ if (!__done.length) {
   const __why = ${op.update === true ? `'no ' + ${type} + ' relation is bound from ' + __failed.join(', ') + (targetId === null ? '' : ' to ' + targetId) + ' to change: bind it first (a relation op without update)'` : relationRefusalReasonJs(type)};
   return { success: false, relationType: ${type}, sourceIds: __failed, error: ${op.update === true ? '__why' : `'the studio refused the ' + ${type} + ' relation for ' + __failed.join(', ') + ' — nothing was bound' + (__why ? ': ' + __why : '.')`} };
 }
-return { success: true, relationType: ${type}, ${op.update === true ? 'updated' : 'bound'}: __done, targetId${glob ? ', count: __members.length' : ''}${(() => { const w = relationOptionWarning(op.relationType, op.relationOptions, ['index', 'count']); return w ? `, optionWarning: ${jsLiteral(w)}` : ''; })()}, ...(__failed.length ? { failed: __failed, warning: __failed.length + ' of ' + __members.length + ' were not ${op.update === true ? 'changed' : 'bound'}: ' + __failed.join(', ') } : {}) };
+return { success: true, relationType: ${type}, ${op.update === true ? 'updated' : 'bound'}: __done, targetId${glob ? ', count: __members.length' : ''}${(() => { const w = relationOptionWarning(op.relationType, op.relationOptions, ['index', 'count', 'pattern']); return w ? `, optionWarning: ${jsLiteral(w)}` : ''; })()}, ...(__failed.length ? { failed: __failed, warning: __failed.length + ' of ' + __members.length + ' were not ${op.update === true ? 'changed' : 'bound'}: ' + __failed.join(', ') } : {}) };
 `;
                 }
                 return `
@@ -9124,8 +9175,17 @@ ${input.cutSound !== undefined ? `  const __cuts = (__cr && __cr.cuts) || (app.s
             }
             case 'modify': {
                 const ch = { ...(v.params ? { params: v.params } : {}), ...(v.members ? { members: v.members } : {}), ...(v.window !== undefined ? { window: v.window } : {}) };
+                // A pattern made with members as a plain array is stored as { refs }, a
+                // shape the engine's own re-check does not accept, so a params-only
+                // modify of it failed with "members is …" (PATIMG24). The stored refs
+                // are handed back as the array they came from.
+                const keep = v.members ? '' : `
+  const __cur = typeof app.getPattern === 'function' ? app.getPattern(${S(v.id)}) : null;
+  const __m = __cur && __cur.spec && __cur.spec.members;
+  if (__m && Array.isArray(__m.refs)) __ch.members = __m.refs.slice();`;
                 body = `${need('modifyPattern')}
-  const r = app.modifyPattern(${S(v.id)}, ${S(ch)});
+  const __ch = ${S(ch)};${keep}
+  const r = app.modifyPattern(${S(v.id)}, __ch);
   return Object.assign({ success: !!(r && r.ok) }, r || {}, app.lastReceipt ? { receipt: app.lastReceipt } : {});`;
                 break;
             }
@@ -11982,12 +12042,45 @@ ${checks.includes('contrast') ? `  // SAMPLED OVER TIME (round 12, 12.9): text i
     }));
   } catch (e) { /* the relations are an addition; the lint stands without them */ }
   const __custom = (typeof window !== 'undefined' && window.__ppCustomRelations) ? Object.values(window.__ppCustomRelations) : [];
+  // HOW TO CHANGE IT (#43): an editing agent needs the op, not the schemas. Each relation comes with the batch op that
+  // changes it; a group bound by a name pattern ("dot-*", recorded in each member's params) comes once, with the ops
+  // that add and remove a member; a custom motion with the call that redefines it.
+  const __edit = { hint: 'To change this scene, send the ops below (your values in relationOptions) in ONE pinepaper_agent_batch_execute call; a custom motion changes by sending its pinepaper_register_custom_relation call again.', groups: [], relations: [], customRelations: [] };
+  try {
+    const __byPat = new Map();
+    __rels.forEach((r) => {
+      const pat = r.params && r.params.pattern;
+      if (typeof pat === 'string') {
+        const k = pat + '|' + r.type + '|' + r.to;
+        if (!__byPat.has(k)) __byPat.set(k, { pattern: pat, type: r.type, to: r.to, members: [] });
+        __byPat.get(k).members.push(r);
+        return;
+      }
+      __edit.relations.push({ from: r.from, ...(r.fromName ? { fromName: r.fromName } : {}), to: r.to, relationType: r.type,
+        change: { type: 'relation', sourceId: r.fromName || r.from, ...(r.to != null ? { targetId: r.to } : {}), relationType: r.type, update: true, relationOptions: r.params } });
+    });
+    __byPat.forEach((g) => {
+      const base = Object.assign({}, g.members[0].params); delete base.index; delete base.count; delete base.pattern;
+      const names = g.members.map((m) => m.fromName).filter(Boolean).sort((a, b) => a.localeCompare(b, undefined, { numeric: true }));
+      const prefix = g.pattern.split('*')[0];
+      const nums = names.map((n) => Number(n.slice(prefix.length))).filter((n) => Number.isFinite(n));
+      const next = prefix + (nums.length ? Math.max.apply(null, nums) + 1 : g.members.length + 1);
+      const firstEntry = app.itemRegistry.get(g.members[0].from);
+      const bind = { type: 'relation', sourceId: g.pattern, ...(g.to != null ? { targetId: g.to } : {}), relationType: g.type, relationOptions: base };
+      __edit.groups.push({ pattern: g.pattern, relationType: g.type, members: g.members.length, ...(names.length ? { first: names[0], last: names[names.length - 1] } : {}), relationOptions: base,
+        change: Object.assign({}, bind, { update: true }),
+        addMember: [{ type: 'create', itemType: (firstEntry && firstEntry.type) || 'circle', position: [0, 0], properties: { name: next } }, bind],
+        ...(names.length ? { removeMember: [{ type: 'delete', itemId: names[names.length - 1] }, bind] } : {}) });
+    });
+    __custom.forEach((d) => __edit.customRelations.push({ name: d.name, change: { tool: 'pinepaper_register_custom_relation', args: d } }));
+  } catch (e) { /* the read-back stands without the edit ops */ }
   // warnings: what the engine's own rules found (TEXT_CLIPS_FRAME: a centre-anchored text running past the
   // frame edge). Passed through when the studio reports them; an older studio has none to pass.
   return { success: true, density: result.density, suggestions: result.suggestions,
     ...(Array.isArray(result.warnings) && result.warnings.length ? { warnings: result.warnings } : {}),
     relations: __rels, ...(__relCount > __rels.length ? { relationsShown: __rels.length, relationCount: __relCount } : {}),
-    ...(__custom.length ? { customRelations: __custom } : {}) };
+    ...(__custom.length ? { customRelations: __custom } : {}),
+    ...(__edit.groups.length || __edit.relations.length || __edit.customRelations.length ? { edit: __edit } : {}) };
 })();`.trim();
     }
     /**