claude-artifact-framework 0.17.0 → 0.18.0

package/llms.txt

@@ -32,7 +32,8 @@ https://cdn.jsdelivr.net/npm/claude-artifact-framework/dist/index.global.js
   (+ key when used as a block)
 - documents: label, title, blocks OR types, empty; machine: key, initial,
   states, events (event: from, do, then); chat: system, greeting, placeholder,
-  tools, model, maxTokens; steps step: title, text, fields, result
+  tools, model, maxTokens; steps step: title, text, fields, result,
+  blocks (result steps only — end a wizard on charts/text under the stats)
 - Formats: money, percent (0-100), number, date. Output/chart/compute items:
   { label, value, format, big }
 
@@ -50,8 +51,9 @@ https://cdn.jsdelivr.net/npm/claude-artifact-framework/dist/index.global.js
 5. Phases (games, turns, processes) belong to `machine` — ctx.send(event,
    payload) is the only door; events are no-ops outside their `from` states;
    `after: {seconds, then, do}` timers are framework-owned.
-6. Row actions receive (record, data, viewerId); block actions (data,
-   viewerId). Stop early if you like, but never skip a middle argument.
+6. Row callbacks (when AND run) receive (record, data, viewerId); block
+   actions (data, viewerId). Stop early if you like; a 2nd param literally
+   named viewerId is matched by name and gets the viewer id.
 7. One big:true per screen; the constructive action takes kind:"primary"; a
    counter changed by pressing is an action, not an editable number field.
 8. Parts of a record are `items` — never JSON in a textarea.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-artifact-framework",
-  "version": "0.17.0",
+  "version": "0.18.0",
   "description": "Utilities for building apps inside Claude Artifacts: platform storage, chat tool-calling loops, and other primitives verified against the real runtime.",
   "type": "module",
   "main": "./dist/index.esm.js",

package/src/app.js

@@ -312,7 +312,7 @@ function validate(spec) {
     if (!Array.isArray(steps) || steps.length === 0) {
       throw new Error('claude-artifact-framework: layout "steps" needs `steps: [...]` with at least one step.');
     }
-    const STEP_KEYS = ["title", "text", "fields", "result"];
+    const STEP_KEYS = ["title", "text", "fields", "result", "blocks"];
     const seen = new Set();
     steps.forEach((st, i) => {
       if (!st || typeof st !== "object") {
@@ -327,6 +327,15 @@ function validate(spec) {
       if (!st.fields && !st.text && !st.result) {
         throw new Error(`claude-artifact-framework: steps[${i}] needs \`fields\`, \`text\`, or \`result\`.`);
       }
+      // A result step may close on more than stats: `blocks` renders any
+      // block set (charts, text, lists) under the result grid. Measured
+      // miss: a wizard that wanted to END on a chart had nowhere to put it.
+      if (st.blocks !== undefined) {
+        if (typeof st.result !== "function") {
+          throw new Error(`claude-artifact-framework: steps[${i}].blocks only renders on a result step — add \`result\` or move the blocks to the step's fields.`);
+        }
+        checkBlocks(st.blocks);
+      }
       checkFields(st.fields, `steps[${i}]`);
       for (const f of st.fields || []) {
         if (seen.has(f.key)) {
@@ -920,7 +929,10 @@ function DocumentsLayout({ spec, ctx }) {
     { className: "caf-documents" },
     h(DocTabbar, { docs, active, dspec, types, close, onAdd, label, ctx }),
     typePicker,
-    h(Panel, { spec: { blocks: aspec.blocks }, ctx: scoped })
+    // Keyed by document id: switching tabs REMOUNTS the pane, so DOM-held
+    // state can't leak across documents. Measured: the native file input
+    // kept showing the previous document's filename.
+    h(Panel, { key: active.id, spec: { blocks: aspec.blocks }, ctx: scoped })
   );
 }
 
@@ -1133,6 +1145,7 @@ export function createApp(spec = {}) {
       colors: (n) => seriesColors(spec.theme && spec.theme.seed, n),
       viewport: narrow ? "mobile" : "desktop",
       _navTabs: narrow && spec.mobile && spec.mobile.nav === "bottom" ? findNavTabs(spec) : null,
+      _Block: Block, // for layouts in other modules (steps) that render blocks
       machine: spec.machine || null,
       send: spec.machine
         ? (eventName, payload) => runMachineEvent(spec.machine, state.update, eventName, payload)
@@ -1192,7 +1205,11 @@ export function createApp(spec = {}) {
           )
         : !state.isHydrated() || libsState === "loading"
         ? h("div", { className: "caf-block caf-loading" }, libsState === "loading" ? "Loading libraries…" : "Loading…")
-        : h(Layout, { spec, ctx }),
+        : // The same containment blocks get, one level up: a crash inside a
+          // layout (a throwing sort, a bad row callback) shows a named error
+          // under the header instead of killing the whole app. Measured: an
+          // unhandled row-action throw once blanked an entire blind app.
+          h(boundary(ctx.React), null, h(Layout, { spec, ctx })),
       spec.brand && spec.brand.footer ? h("footer", { className: "caf-footer" }, spec.brand.footer) : null
     );
   };
@@ -1316,6 +1333,7 @@ const BASE_CSS = `
 .caf-timer-controls { display: flex; gap: 0.6rem; }
 .caf-btn:disabled { opacity: 0.45; cursor: not-allowed; }
 .caf-steps-progress { display: flex; flex-direction: column; gap: 0.35rem; margin-bottom: 0.2rem; }
+.caf-steps-result { display: flex; flex-direction: column; gap: 1rem; min-width: 0; }
 .caf-steps-fill { transition: width 0.25s ease; }
 .caf-step-text { margin: 0; color: var(--caf-muted); font-size: 0.92rem; line-height: 1.55; }
 .caf-step-nav { display: flex; justify-content: space-between; gap: 0.6rem; margin-top: 0.4rem; }

package/src/records.js

@@ -54,6 +54,27 @@ function makeId() {
   return "r" + Date.now().toString(36) + Math.random().toString(36).slice(2, 8);
 }
 
+// Call-by-name for row callbacks. The measured 50/50 trap: both
+// `when: (r, viewerId)` (the old documented form) and `run: (r, viewerId)`
+// (the mistake that killed an entire blind app — `data` skipped, so `d`
+// held a viewer-id string) exist in the wild. Positions lie; names don't:
+// if the second declared parameter is named like the viewer id, hand it
+// the viewer id. Every callback gets what its own names ask for, and the
+// uniform (record, data, viewerId) order in the docs stays the one truth.
+const rowCallLegacy = new WeakMap();
+function rowCall(fn, rec, data, viewerId) {
+  let legacy = rowCallLegacy.get(fn);
+  if (legacy === undefined) {
+    const src = String(fn);
+    const arrow = src.match(/^\s*(?:async\s+)?([\w$]+)\s*=>/); // one bare param
+    const parens = src.match(/^[^(]*\(([^)]*)\)/);
+    const params = arrow ? [arrow[1]] : parens ? parens[1].split(",").map((s) => s.trim().split(/[=\s:]/)[0]) : [];
+    legacy = /viewer|voter|usuario|user/i.test(params[1] || "");
+    rowCallLegacy.set(fn, legacy);
+  }
+  return legacy ? fn(rec, viewerId) : fn(rec, data, viewerId);
+}
+
 // avatar: true — an initials circle per row, colored deterministically from
 // the row title through the theme's series palette. Identity without assets.
 function Avatar({ title, ctx }) {
@@ -184,7 +205,7 @@ function ListScreen({ spec, ctx }) {
                     "div",
                     { className: "caf-row-actions" },
                     spec.actions
-                      .filter((a) => !a.when || a.when(rec, ctx.viewerId))
+                      .filter((a) => !a.when || rowCall(a.when, rec, ctx.data, ctx.viewerId))
                       .map((a) =>
                         h(
                           "button",
@@ -195,7 +216,7 @@ function ListScreen({ spec, ctx }) {
                             onClick: () =>
                               ctx.update((d) => {
                                 const live = d.records.find((x) => x.id === rec.id);
-                                if (live) a.run(live, d, ctx.viewerId);
+                                if (live) rowCall(a.run, live, d, ctx.viewerId);
                               }),
                           },
                           a.label

package/src/steps.js

@@ -22,23 +22,31 @@ function stepErrors(step, data) {
 
 function ResultScreen({ step, ctx }) {
   const items = step.result(ctx.data) || [];
+  const Block = ctx._Block;
   return h(
     "div",
-    { className: "caf-block caf-output" },
-    step.title ? h("h2", { className: "caf-block-title" }, step.title) : null,
+    { className: "caf-steps-result" },
     h(
       "div",
-      { className: "caf-output-grid" },
-      items.map((item, i) =>
-        h(
-          "div",
-          { key: item.label ?? i, className: item.big ? "caf-stat caf-stat-big" : "caf-stat" },
-          h("span", { className: "caf-stat-label" }, item.label),
-          h("span", { className: "caf-stat-value" }, formatValue(item.value, item.format)),
-          item.hint ? h("small", { className: "caf-hint" }, item.hint) : null
+      { className: "caf-block caf-output" },
+      step.title ? h("h2", { className: "caf-block-title" }, step.title) : null,
+      h(
+        "div",
+        { className: "caf-output-grid" },
+        items.map((item, i) =>
+          h(
+            "div",
+            { key: item.label ?? i, className: item.big ? "caf-stat caf-stat-big" : "caf-stat" },
+            h("span", { className: "caf-stat-label" }, item.label),
+            h("span", { className: "caf-stat-value" }, formatValue(item.value, item.format)),
+            item.hint ? h("small", { className: "caf-hint" }, item.hint) : null
+          )
         )
       )
     ),
+    // A result can end on more than stats: any declared blocks (charts,
+    // text, lists) render under the grid with full block services.
+    (step.blocks || []).map((b, i) => h(Block, { key: "rb" + i, block: b, ctx })),
     h(
       "div",
       { className: "caf-step-nav" },