@sparkerp/plugin-sdk 0.1.0 → 1.1.0

package/bundle/validators/block-engine.mjs

@@ -3384,6 +3384,25 @@ function endsWith(args) {
   const [s, suffix] = args;
   return rvBool(str(s).endsWith(str(suffix)));
 }
+function displayString(v) {
+  switch (v.type) {
+    case "String":
+      return v.value;
+    case "Integer":
+      return v.value.toString();
+    case "Decimal":
+      return v.value.toString();
+    case "Boolean":
+      return String(v.value);
+    case "Null":
+      return "";
+    default:
+      throw new EvaluationError(`concat() does not support ${v.type} operands`);
+  }
+}
+function concatFn(args) {
+  return rvString(args.map(displayString).join(""));
+}
 function registerStringFunctions(registry, version) {
   registry.register(version, "upper", upper);
   registry.register(version, "lower", lower);
@@ -3392,6 +3411,7 @@ function registerStringFunctions(registry, version) {
   registry.register(version, "substring", substring);
   registry.register(version, "startsWith", startsWith);
   registry.register(version, "endsWith", endsWith);
+  registry.register(version, "concat", concatFn);
 }
 function contains(args) {
   const [collection, needle] = args;
@@ -3438,12 +3458,19 @@ function size(args) {
 function isEmpty(args) {
   return rvBool(sizeOf(args[0]) === 0);
 }
+function pluck(args) {
+  const [arr, key] = args;
+  if (arr.type !== "Array") throw new EvaluationError(`pluck() expects an Array as its first argument, got ${arr.type}`);
+  if (key.type !== "String") throw new EvaluationError("pluck() expects a String field name as its second argument");
+  return rvArray(arr.value.map((row) => row.type === "Object" ? row.value[key.value] ?? RV_NULL : RV_NULL));
+}
 function registerCollectionFunctions(registry, version) {
   registry.register(version, "contains", contains);
   registry.register(version, "first", first);
   registry.register(version, "last", last);
   registry.register(version, "size", size);
   registry.register(version, "isEmpty", isEmpty);
+  registry.register(version, "pluck", pluck);
 }
 function ifImpl(args) {
   const [cond, thenValue, elseValue] = args;
@@ -3516,12 +3543,14 @@ var FUNCTION_CATALOG = [
   c({ name: "substring", minArity: 2, maxArity: 3, argKinds: ["String", "Numeric", "Numeric"], returnType: "String", category: "String", description: "Part of a string from a 0-based start position, optionally limited to a length." }),
   c({ name: "startsWith", minArity: 2, maxArity: 2, argKinds: ["String", "String"], returnType: "Boolean", category: "String", description: "True when the string begins with the given prefix." }),
   c({ name: "endsWith", minArity: 2, maxArity: 2, argKinds: ["String", "String"], returnType: "Boolean", category: "String", description: "True when the string ends with the given suffix." }),
+  c({ name: "concat", minArity: 2, maxArity: Infinity, argKinds: ["Any"], returnType: "String", category: "String", description: `Joins 2+ values' display text into one string, e.g. concat("Total: ", page.count).` }),
   // Collection
   c({ name: "contains", minArity: 2, maxArity: 2, argKinds: ["Collection", "Any"], returnType: "Boolean", category: "Collection", description: "True when a list, string, or object contains the given element/substring/key." }),
   c({ name: "first", minArity: 1, maxArity: 1, argKinds: ["Array"], returnType: "Any", category: "Collection", description: "The first element of a list." }),
   c({ name: "last", minArity: 1, maxArity: 1, argKinds: ["Array"], returnType: "Any", category: "Collection", description: "The last element of a list." }),
   c({ name: "size", minArity: 1, maxArity: 1, argKinds: ["Collection"], returnType: "Integer", category: "Collection", description: "Number of elements in a list, characters in a string, or keys in an object." }),
   c({ name: "isEmpty", minArity: 1, maxArity: 1, argKinds: ["Collection"], returnType: "Boolean", category: "Collection", description: "True when a list, string, or object has no elements." }),
+  c({ name: "pluck", minArity: 2, maxArity: 2, argKinds: ["Array", "String"], returnType: "Array", category: "Collection", description: `pluck(rows, "fieldName") \u2014 a new list of that field's value from each object in the list, e.g. turning [{date, value}, ...] into [value, value, ...].` }),
   // Utility
   c({ name: "if", minArity: 3, maxArity: 3, argKinds: ["Boolean", "Any", "Any"], returnType: "Any", category: "Utility", description: "if(condition, thenValue, elseValue) \u2014 returns one of two values based on a condition." }),
   c({ name: "coalesce", minArity: 1, maxArity: Infinity, argKinds: ["Any"], returnType: "Any", category: "Utility", description: "The first argument that is not null." }),
@@ -4744,9 +4773,9 @@ function resolvePath(scopes, rawPath) {
   return value;
 }
 function resolveString(scopes, text) {
-  const pluck = PLUCK_TEMPLATE.exec(text);
-  if (pluck) {
-    const [, path, field] = pluck;
+  const pluck2 = PLUCK_TEMPLATE.exec(text);
+  if (pluck2) {
+    const [, path, field] = pluck2;
     const resolved = runtimeValueToJs(resolvePath(scopes, path));
     if (!Array.isArray(resolved)) {
       throw new ActionError("unresolved-variable", `'${path}|pluck:${field}' requires an array at '${path}'`);
@@ -11057,6 +11086,26 @@ ${failedSteps}` : "")
     });
     if (faulted && this.phase === "ready") this.present();
   }
+  /**
+   * A real, previously-undiscovered gap (2026-09-22, found live authoring the HCM Permissions
+   * "Create Permission" dialog): a `core.text-input` (or any other inputClass block) whose page
+   * JSON only ever sets `properties.value.binding` — the "obviously two-way" way EVERY such field
+   * in this codebase's own already-shipped pages is authored (`hcm-users.json`'s Create User
+   * dialog included) — never actually wrote a keystroke back to that bound page-scope key. `commit`
+   * → `writeOutput` only ever consulted `config.outputs[name]`, a SEPARATE, explicit redirect
+   * nothing in this codebase's own page-authoring convention ever sets; absent that, every write
+   * landed in a `{scope:"block", scopeKey:this.instanceId}` bucket no page action chain can ever
+   * read — the DOM still showed the typed characters (via `RevealableTextField`'s own local echo
+   * state), so this was invisible in every purely-visual check, only surfacing when a create
+   * dialog's Submit button's `${page.createX}` params were inspected against the REAL network
+   * payload, which showed every field empty. Fixed by falling back to the SAME resolved binding
+   * target `resolveProperties()` already computed for the identically-named PROPERTY's own read
+   * side (`this.boundProps`) when no explicit `config.outputs[name]` redirect exists — making a
+   * `source:"binding"` property's own binding genuinely two-way by default, matching what every
+   * page author already (reasonably) assumed it meant. An explicit `config.outputs[name]` still
+   * wins unchanged (e.g. `core.file-upload`'s own redirect to a different scope/key than its
+   * `value` property reads from); this only fills in the previously-silent gap.
+   */
   async writeOutput(name, value) {
     const port = (this.type.definition.outputs ?? []).find((o) => o.name === name);
     if (!port) {
@@ -11064,7 +11113,7 @@ ${failedSteps}` : "")
       return;
     }
     const redirect = this.config.outputs?.[name];
-    const target = redirect ? resolveBoundKey(redirect, this.bindingContext()) : { scope: "block", scopeKey: this.instanceId, key: name };
+    const target = redirect ? resolveBoundKey(redirect, this.bindingContext()) : this.boundProps.get(name) ?? { scope: "block", scopeKey: this.instanceId, key: name };
     await this.ctx.state.set(target, value);
   }
   /* ---------------- error isolation & standard states (W2-04) ---------------- */
@@ -12884,6 +12933,47 @@ var imageDefinition = definition({
       type: "string",
       sources: ["static"],
       designer: { group: "content", editor: "text", labelKey: "core.image.property.height" }
+    },
+    {
+      // 2026-09-17 (page-builder "Customize" panel parity) — a clip-path-class
+      // shape, the same fixed vocabulary a real page-builder's own "Shape"
+      // picker offers (never an arbitrary path — that stays a `style`
+      // escape-hatch concern).
+      name: "shape",
+      type: "string",
+      sources: ["static"],
+      default: "none",
+      constraints: { options: [{ value: "none", labelKey: "core.image.shape.none" }, { value: "circle", labelKey: "core.image.shape.circle" }, { value: "rounded", labelKey: "core.image.shape.rounded" }] },
+      designer: { group: "content", editor: "select", labelKey: "core.image.property.shape" }
+    },
+    {
+      /** A native browser tooltip (HTML `title` attribute) — distinct from `alt` (accessibility text, always present) and from a rich hover-card (out of scope here, a real different feature). */
+      name: "tooltip",
+      type: "string",
+      sources: ["static"],
+      designer: { group: "content", editor: "text", labelKey: "core.image.property.tooltip" }
+    },
+    {
+      /** A raw CSS `transform` value, e.g. "rotate(4deg)" — same "text travels as data" posture as `PageTheme.customCss`, never `dangerouslySetInnerHTML`, just an inline style value. */
+      name: "transform",
+      type: "string",
+      sources: ["static"],
+      designer: { group: "content", editor: "text", labelKey: "core.image.property.transform" }
+    },
+    {
+      name: "filter",
+      type: "string",
+      sources: ["static"],
+      default: "none",
+      constraints: {
+        options: [
+          { value: "none", labelKey: "core.image.filter.none" },
+          { value: "grayscale", labelKey: "core.image.filter.grayscale" },
+          { value: "sepia", labelKey: "core.image.filter.sepia" },
+          { value: "blur", labelKey: "core.image.filter.blur" }
+        ]
+      },
+      designer: { group: "content", editor: "select", labelKey: "core.image.property.filter" }
     }
   ],
   events: [],
@@ -12905,7 +12995,11 @@ var imageLogic = {
         src: String(io.props["src"] ?? ""),
         alt: io.props["alt"] !== void 0 ? String(io.props["alt"]) : "",
         width: io.props["width"] !== void 0 ? String(io.props["width"]) : void 0,
-        height: io.props["height"] !== void 0 ? String(io.props["height"]) : void 0
+        height: io.props["height"] !== void 0 ? String(io.props["height"]) : void 0,
+        shape: io.props["shape"] !== void 0 ? String(io.props["shape"]) : void 0,
+        tooltip: io.props["tooltip"] !== void 0 ? String(io.props["tooltip"]) : void 0,
+        transform: io.props["transform"] !== void 0 ? String(io.props["transform"]) : void 0,
+        filter: io.props["filter"] !== void 0 ? String(io.props["filter"]) : void 0
       }
     });
   }
@@ -16275,6 +16369,31 @@ var gridDefinition = definition({
       designer: { group: "data", editor: "binding", labelKey: "core.grid.property.refreshTrigger" }
     }
   ],
+  // 2026-09-23 — these are the block's own internal state (written via
+  // `io.setOwnState` in this file's `query()`/handlers, read back into
+  // `node.props` for the adapter), not authored properties: a page author
+  // never sets them, but a plugin/theme developer inspecting `core.grid`
+  // via `erp blocks list --type core.grid` had no way to discover they
+  // exist at all — the empty-grid production bug this session traced back
+  // to exactly this state (rows/total both came from the same query
+  // response, one reached the render, the other didn't) took a live
+  // Playwright session plus a source read to pin down, when this list
+  // alone would have shown the full internal-state surface immediately.
+  outputs: [
+    { name: "rows", type: "json" },
+    { name: "total", type: "number" },
+    { name: "pageIndex", type: "number" },
+    { name: "sort", type: "json" },
+    { name: "filter", type: "json" },
+    { name: "groups", type: "json" },
+    { name: "aggregates", type: "json" },
+    { name: "expandedGroups", type: "json" },
+    { name: "error", type: "string" },
+    { name: "circuitOpen", type: "boolean" },
+    { name: "selectedIds", type: "json" },
+    { name: "density", type: "string" },
+    { name: "groupBy", type: "string" }
+  ],
   events: [
     { name: "selectionChanged", catalogType: "GridSelectionChanged" },
     { name: "rowClicked", catalogType: "GridRowClicked" },
@@ -17182,6 +17301,30 @@ var fileUploadDefinition = definition({
       type: "string",
       sources: ["static", "expression"],
       designer: { group: "data", editor: "text", labelKey: "core.file-upload.property.cabinetId" }
+    },
+    {
+      // 2026-09-21 (HCM Users spec gap-closing pass) — every pre-existing
+      // page never sets this (undefined === "dms"), so it's a zero-risk
+      // addition: `"dms"` keeps today's exact behavior (a real DMS cabinet
+      // upload via the injected `FileUploadSource`, `value` becomes real
+      // `UploadedFileRef`s). `"raw"` skips the DMS upload entirely — `value`
+      // becomes the picked file's own `{filename, contentType, dataUrl}`
+      // straight from the browser's file picker, with no `id`/`url` (nothing
+      // was uploaded anywhere) — meant to feed a `callApi` action's
+      // `multipart: true` body (see that handler's own doc comment), the
+      // real fix for the disclosed "core.file-upload cannot emit raw bytes
+      // into a callApi body" gap (`ai/patterns/import.md`).
+      name: "mode",
+      type: "string",
+      sources: ["static"],
+      default: "dms",
+      constraints: {
+        options: [
+          { value: "dms", labelKey: "core.file-upload.mode.dms" },
+          { value: "raw", labelKey: "core.file-upload.mode.raw" }
+        ]
+      },
+      designer: { group: "behavior", editor: "select", labelKey: "core.file-upload.property.mode" }
     }
   ],
   outputs: [{ name: "value", type: "json" }],
@@ -17194,15 +17337,23 @@ var fileUploadDefinition = definition({
     category: "input",
     propertyGroups: [
       { id: "data", titleKey: "core.designer.group.data" },
-      { id: "content", titleKey: "core.designer.group.content" }
+      { id: "content", titleKey: "core.designer.group.content" },
+      { id: "behavior", titleKey: "core.designer.group.behavior" }
     ],
     allowedTargets: ["page", "form", "container", "dashboard"],
     preview: { kind: "field" }
   }
 });
+var rawFileIdCounter = 0;
+function nextRawFileId() {
+  if (typeof crypto !== "undefined" && typeof crypto.randomUUID === "function") return crypto.randomUUID();
+  rawFileIdCounter += 1;
+  return `raw-file-${Date.now()}-${rawFileIdCounter}`;
+}
 function createFileUploadLogic(uploadSource) {
   return {
     render(io) {
+      const mode = io.props["mode"] === "raw" ? "raw" : "dms";
       const files = Array.isArray(io.props["value"]) ? io.props["value"] : [];
       const accept = typeof io.props["accept"] === "string" ? io.props["accept"] : void 0;
       const multiple = io.props["multiple"] !== false;
@@ -17211,8 +17362,17 @@ function createFileUploadLogic(uploadSource) {
         props: { value: files, accept, multiple },
         handlers: {
           upload: async (payload) => {
-            if (!uploadSource) return;
             const incoming = payload ?? [];
+            if (mode === "raw") {
+              await io.runLoading(async () => {
+                const picked = incoming.map((f) => ({ id: nextRawFileId(), ...f }));
+                const next = [...files, ...picked];
+                await io.writeOutput("value", next);
+                await io.emit("committed", { field: "value", old: files, new: next });
+              });
+              return;
+            }
+            if (!uploadSource) return;
             await io.runLoading(async () => {
               const uploaded = [];
               for (const f of incoming) uploaded.push(await uploadSource.upload(f, cabinetId));