@tenonhq/dovetail-servicenow 0.0.43 → 0.0.44

package/dist/flowDesigner/defineActionType.d.ts

@@ -301,5 +301,6 @@ export interface PlanActionDefinitionParams {
  * before/after views; throws on any invalid or dangling reference.
  */
 export declare function planActionDefinition(params: PlanActionDefinitionParams): DefineActionPlan;
+export declare function compareForVerify(expected: ActionView, actual: ActionView): DefineActionVerify;
 export declare function defineActionType(params: DefineActionTypeParams): Promise<DefineActionTypeResult>;
 export {};

package/dist/flowDesigner/defineActionType.js

@@ -50,6 +50,7 @@ exports.canonValue = canonValue;
 exports.viewAction = viewAction;
 exports.diffViews = diffViews;
 exports.planActionDefinition = planActionDefinition;
+exports.compareForVerify = compareForVerify;
 exports.defineActionType = defineActionType;
 const crypto_1 = require("crypto");
 const actionTypeApi_1 = require("./actionTypeApi");
@@ -1547,9 +1548,28 @@ async function resolveAliasDisplays(client, ids) {
     }
     return out;
 }
+// Variable attributes the server assigns a default to on save when the spec left
+// them unset (live-observed 2026-09-30: a new boolean input came back with
+// maxLength 40). A blank expected value for one of these is "unspecified", not a
+// mismatch; an explicitly specified value is still verified.
+var SERVER_DEFAULTED_FIELDS = ["maxLength"];
+function dropServerDefaults(changed) {
+    return changed
+        .map(function (c) {
+        var kept = c.changes.filter(function (f) {
+            return !(SERVER_DEFAULTED_FIELDS.indexOf(f.field) !== -1 && f.before === "");
+        });
+        return Object.assign({}, c, { changes: kept });
+    })
+        .filter(function (c) {
+        return c.changes.length > 0;
+    });
+}
 function compareForVerify(expected, actual) {
     var notes = [];
     var d = diffViews(expected, actual, {});
+    d.inputs.changed = dropServerDefaults(d.inputs.changed);
+    d.outputs.changed = dropServerDefaults(d.outputs.changed);
     // Expected -> actual: any difference means the save did not land as planned.
     d.action.forEach(function (c) {
         notes.push("action." + c.field + " is '" + c.after + "', expected '" + c.before + "'");
@@ -1585,10 +1605,13 @@ function compareForVerify(expected, actual) {
     d.steps.changed.forEach(function (s) {
         notes.push("step '" + s.label + "': " + s.changes.join("; "));
     });
-    if (notes.length === 0) {
+    // ok is derived from the filtered notes, not d.empty: server-defaulted fields
+    // were dropped above and must not fail the verify.
+    var ok = notes.length === 0;
+    if (ok) {
         notes.push("read-back matches the planned definition");
     }
-    return { ok: d.empty, notes: notes };
+    return { ok: ok, notes: notes };
 }
 async function defineActionType(params) {
     var client = params.client;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tenonhq/dovetail-servicenow",
-  "version": "0.0.43",
+  "version": "0.0.44",
   "engines": {
     "node": ">=22"
   },