nodality 1.3.1 → 1.3.3

package/lib/webmcp-adapter.js

@@ -1,5 +1,5 @@
 /*!
- * nodality v1.3.1
+ * nodality v1.3.3
  * (c) 2026 Filip Vabrousek
  * License: MIT
  */
@@ -279,18 +279,83 @@ export function installAgentSurface({ mount, elements, nodes, morphs }) {
                 return refuse("MISSING_REQUIRED", missing, Object.keys(props));
             }
 
-            const filled = [];
+            // Setting a control is not the same as the control TAKING the
+            // value, and the difference used to be invisible. Assigning a
+            // `<select>` a value it has no option for leaves it untouched and
+            // throws nothing; the old loop pushed the key onto `filled`
+            // regardless, so the tool answered `ok: true, filled: [... "topic"]`
+            // for a field that never reached the payload. An agent was told
+            // its submission carried something it did not.
+            //
+            // So every field is READ BACK after it is set, and what a control
+            // refused is reported rather than counted as done.
+            const filled = [], rejected = [], absent = [];
+
+            /** The values a control will actually accept, where it is a closed set. */
+            const accepts = (field) => field.tagName === "SELECT"
+                ? [...field.options].map((o) => o.value) : [];
+
             for (const key of Object.keys(props)) {
                 if (args[key] === undefined) continue;
+                const want = args[key];
                 const field = form.querySelector(
                     `[name="${key}"], #${CSS.escape(key)}`);
-                if (!field) continue;
-                if (field.type === "checkbox") field.checked = !!args[key];
-                else field.value = String(args[key]);
+                // Named by the schema and carried by nothing. Also silent
+                // before, and the same lie by omission.
+                if (!field) { absent.push(key); continue; }
+
+                if (field.type === "checkbox") {
+                    field.checked = !!want;
+                } else if (field.type === "radio") {
+                    // A group, not a field: `querySelector` returns the FIRST
+                    // radio, and assigning `.value` to it rewrites that
+                    // button's value instead of choosing the one asked for —
+                    // which then reads back as a match and reports success
+                    // while nothing is selected.
+                    const group = [...form.querySelectorAll(`[name="${key}"]`)];
+                    const hit = group.find((r) => r.value === String(want));
+                    if (hit) hit.checked = true;
+                    else {
+                        rejected.push({ field: key, got: null,
+                            valid: group.map((r) => r.value) });
+                        continue;
+                    }
+                } else {
+                    field.value = String(want);
+                }
                 // Both, because component state may listen for either.
                 field.dispatchEvent(new Event("input", { bubbles: true }));
                 field.dispatchEvent(new Event("change", { bubbles: true }));
-                filled.push(key);
+
+                const got = field.type === "checkbox" ? field.checked
+                    : field.type === "radio" ? String(want) : field.value;
+                const took = field.type === "checkbox"
+                    ? (!!want === got) : (String(want) === got);
+                if (took) filled.push(key);
+                else rejected.push({ field: key, got, valid: accepts(field) });
+            }
+
+            // Nothing is submitted when a field did not take. A partial
+            // payload sent under `ok: true` is the failure this whole
+            // architecture exists to remove, and it is worse on the agent
+            // path than anywhere else: the caller has no eyes on the page and
+            // no reason to doubt the report. The refusal names the field and,
+            // where the control has a closed set, what it would have accepted
+            // — repairable in one turn.
+            if (rejected.length || absent.length) {
+                return {
+                    ...refuse("FIELD_NOT_SET",
+                        [...rejected.map((r) => r.field), ...absent],
+                        [...rejected.map((r) => r.valid.length
+                            ? `"${r.field}" accepts ${r.valid.map((v) => JSON.stringify(v)).join(", ")}`
+                            : `"${r.field}" did not take the value given`),
+                         ...absent.map((k) =>
+                            `"${k}" is named by the schema but no control in the form carries it`)]),
+                    // What DID land, because the form is now partly filled and
+                    // a caller retrying needs to know that.
+                    filled,
+                    submitted: null,
+                };
             }
 
             // Submitted through the form's OWN path, so its listeners,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodality",
-  "version": "1.3.1",
+  "version": "1.3.3",
   "description": "A lightweight library for declarative UI elements.",
   "main": "dist/index.cjs.js",
   "module": "dist/index.esm.js",

package/schema.json

@@ -1941,7 +1941,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "keySet",
@@ -2132,7 +2133,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"
@@ -2737,7 +2739,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "keySet",
@@ -2937,7 +2940,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"
@@ -3267,7 +3271,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "keySet",
@@ -3467,7 +3472,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"
@@ -3775,7 +3781,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "justifo"
@@ -3972,7 +3979,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "toColumn"
@@ -4318,7 +4326,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "keySet",
@@ -4518,7 +4527,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"
@@ -4589,7 +4599,8 @@
           "description": "DOM id on the input."
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "length"
@@ -4674,7 +4685,8 @@
           "description": "Explicit light/dark overrides — {light: {...}, dark: {...}}."
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "type",
@@ -4730,7 +4742,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "type",
@@ -4780,7 +4793,8 @@
           "description": "Corner radius in pixels. A bare number, not a CSS length."
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "type",
@@ -4798,11 +4812,13 @@
       "params": [
         {
           "name": "arrayMargin",
-          "description": "Margin on named sides. {sides: [\"left\"], value: \"1rem\"}."
+          "description": "Margin on named sides. {sides: [\"left\"], value: \"1rem\"}.",
+          "deprecated": "superseded by `mar`. `mar: [{a: 10}]` is the same margin. Still works, but warns."
         },
         {
           "name": "arrayPadding",
-          "description": "Padding on named sides. {sides: [\"left\",\"top\"], value: \"1rem\"}."
+          "description": "Padding on named sides. {sides: [\"left\",\"top\"], value: \"1rem\"}.",
+          "deprecated": "superseded by `pad`. `pad: [{a: \"0.5rem\"}]` is the same padding. Still works, but warns."
         },
         {
           "name": "background"
@@ -4812,10 +4828,12 @@
           "description": "DOM id on the input."
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
-          "name": "length"
+          "name": "label",
+          "description": "Accessible name for the field."
         },
         {
           "name": "mar",
@@ -4833,6 +4851,10 @@
           "name": "radius",
           "description": "Corner radius in pixels. A bare number, not a CSS length."
         },
+        {
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
+        },
         {
           "name": "type",
           "description": "Input type attribute. Defaults to \"text\"; \"search\" adds the native clear button."
@@ -5149,7 +5171,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "keySet",
@@ -5334,7 +5357,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"
@@ -5734,7 +5758,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "keySet",
@@ -5919,7 +5944,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"
@@ -6166,7 +6192,8 @@
           "description": "Heading level to render, independent of `size`/`fluid`."
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"
@@ -6443,7 +6470,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "keySet",
@@ -6628,7 +6656,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"
@@ -6929,7 +6958,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "keySet",
@@ -7114,7 +7144,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"
@@ -7727,7 +7758,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "keySet",
@@ -7912,7 +7944,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"
@@ -8385,7 +8418,8 @@
           "description": "DOM id on the input."
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "layout"
@@ -8672,7 +8706,8 @@
           "name": "italic"
         },
         {
-          "name": "items"
+          "name": "items",
+          "description": "The choices, each a `[value, text]` pair — or a plain string used as both."
         },
         {
           "name": "keySet",
@@ -8857,7 +8892,8 @@
           "name": "text"
         },
         {
-          "name": "title"
+          "name": "title",
+          "description": "Accepted as a synonym for `label`, for parity with `labelInput`."
         },
         {
           "name": "top"