@tenonhq/dovetail-servicenow 0.0.29 → 0.0.30

package/dist/cli.js

@@ -60,6 +60,7 @@ const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const loadEnv_1 = require("./loadEnv");
 const client_1 = require("./client");
+const fieldsFromJson_1 = require("./fieldsFromJson");
 const choices_1 = require("./choices");
 const formatter_1 = require("./formatter");
 const views_1 = require("./layout/views");
@@ -897,11 +898,12 @@ function printHelp() {
         "                     DRY-RUN BY DEFAULT — nothing is sent without --confirm\n" +
         "                     (--method <GET|POST|PUT|DELETE> --path /api/<scope>/<service>/<resource>\n" +
         "                      [--body '<json>' | --body-json <path>] [--confirm] [--dry-run] [--json])\n" +
-        "  set-field          Set scalar field value(s) on an EXISTING record, into an update set, then verify\n" +
-        '                     (--table <t> --sys-id <id>|--query <q> --fields "k=v,k2=v2"\n' +
-        "                      --update-set <sys_id> [--dry-run] [--json])\n" +
+        "  set-field          Set field value(s) on an EXISTING record, into an update set, then verify\n" +
+        "                     (--table <t> --sys-id <id>|--query <q> --update-set <sys_id>\n" +
+        '                      (--fields "k=v,k2=v2" | --from-json <path>) [--dry-run] [--json])\n' +
         "  create-record      Create ONE NEW record in a data table, into an update set, then verify\n" +
-        '                     (--table <t> --fields "k=v,k2=v2" --scope <s> --update-set <sys_id>\n' +
+        "                     (--table <t> --scope <s> --update-set <sys_id>\n" +
+        '                      (--fields "k=v,k2=v2" | --from-json <path>)\n' +
         "                      [--if-absent <encoded-query>] [--dry-run] [--json])\n" +
         "  host-assets        Deploy a built dist/ to ServiceNow (carrier sys_ui_script + attachment + m2m)\n" +
         "                     (--dir <dist> --app <sys_id> --scope <namespace>\n" +
@@ -1228,24 +1230,48 @@ function parseFieldsInline(input) {
     }
     return out;
 }
+/**
+ * Merge the two field sources for the record-write verbs: inline `--fields "k=v,k2=v2"`
+ * and `--from-json <path>` (a JSON object — the only form that can carry a large or
+ * multiline value, since the inline form splits on commas and trims). On a key present in
+ * both, `--from-json` wins: it is the explicit spec file. Throws on an unreadable file,
+ * malformed JSON, or a non-scalar value (surfaced by the caller as a bad-args error).
+ */
+function mergeFields(flags) {
+    var inline = parseFieldsInline(flags.fields || "");
+    if (!flags["from-json"])
+        return inline;
+    return Object.assign(inline, (0, fieldsFromJson_1.readFieldsFromJsonFile)(flags["from-json"]));
+}
 /**
  * dove-sn set-field:
  *   --table x_cadso_core_metric_point_type
  *   --sys-id <id>  |  --query "name=send_size"   (query must resolve to exactly 1 row)
- *   --fields "order=20"                          (comma-separated key=value pairs)
+ *   [--fields "order=20"]                        (comma-separated key=value pairs)
+ *   [--from-json <path>]                         (JSON { field: value }; carries large or
+ *                                                 multiline values the inline form can't;
+ *                                                 overrides --fields on a shared key)
+ *                                                — at least one of --fields / --from-json is required
  *   --update-set <sys_id>                        (required — the change is captured here)
  *   [--dry-run] [--json]
  * Exit codes: 0 applied/dry-run, 1 bad args, 2 write landed but read-back unverified.
  */
 async function runSetField(flags) {
     var table = flags.table;
-    var fields = parseFieldsInline(flags.fields || "");
+    var fields;
+    try {
+        fields = mergeFields(flags);
+    }
+    catch (err) {
+        process.stderr.write("set-field: " + (err instanceof Error ? err.message : String(err)) + "\n");
+        return 1;
+    }
     var hasTarget = Boolean(flags["sys-id"] || flags.query);
     if (!table ||
         Object.keys(fields).length === 0 ||
         !hasTarget ||
         !flags["update-set"]) {
-        process.stderr.write('set-field: --table, --fields "k=v", one of --sys-id/--query, and --update-set are required\n');
+        process.stderr.write('set-field: --table, one of --sys-id/--query, --update-set, and at least one field (--fields "k=v" or --from-json <path>) are required\n');
         return 1;
     }
     var params = {
@@ -1285,7 +1311,10 @@ async function runSetField(flags) {
 /**
  * dove-sn create-record:
  *   --table x_cadso_core_metric_point_type
- *   --fields "name=avg_message_parts,label=Avg. Message Parts,order=35"
+ *   [--fields "name=avg_message_parts,label=Avg. Message Parts,order=35"]
+ *   [--from-json <path>]                         (JSON { field: value }; carries large or
+ *                                                 multiline values the inline form can't)
+ *                                                — at least one of --fields / --from-json is required
  *   --scope x_cadso_core                         (the app that owns the new record)
  *   --update-set <sys_id>                        (required — the insert is captured here)
  *   [--if-absent "name=avg_message_parts"]       (skip the insert when this query already matches)
@@ -1295,12 +1324,21 @@ async function runSetField(flags) {
  */
 async function runCreateRecord(flags) {
     var table = flags.table;
-    var fields = parseFieldsInline(flags.fields || "");
+    var fields;
+    try {
+        fields = mergeFields(flags);
+    }
+    catch (err) {
+        process.stderr.write("create-record: " +
+            (err instanceof Error ? err.message : String(err)) +
+            "\n");
+        return 1;
+    }
     if (!table ||
         Object.keys(fields).length === 0 ||
         !flags.scope ||
         !flags["update-set"]) {
-        process.stderr.write('create-record: --table, --fields "k=v", --scope and --update-set are required\n');
+        process.stderr.write('create-record: --table, --scope, --update-set, and at least one field (--fields "k=v" or --from-json <path>) are required\n');
         return 1;
     }
     var params = {

package/dist/fieldsFromJson.d.ts

@@ -0,0 +1,22 @@
+/**
+ * `--from-json` field-map loader for the record-write verbs (set-field, create-record).
+ *
+ * The inline `--fields "k=v,k2=v2"` form splits on commas and trims each piece — so it
+ * cannot carry a value that contains a comma, a newline, an `=`, or leading/trailing
+ * whitespace. That rules out every large field: a script body, an HTML/XML/CSS field, a
+ * JSON blob. `--from-json <path>` closes that gap: the file is a flat JSON object of
+ * `field → value`, and JSON quoting carries any character faithfully.
+ *
+ * Values may be string | number | boolean. Numbers/booleans are stringified — matching
+ * the inline parser and the wire format, where ServiceNow scalar fields are strings.
+ * `null`/`undefined` values are skipped. Nested objects and arrays are rejected: a scalar
+ * field cannot take one, and silently JSON-encoding it would write garbage into the record.
+ */
+export declare function coerceFieldsFromJson(parsed: unknown): Record<string, string>;
+/**
+ * Read + parse a `--from-json` file into a validated field map. The path is resolved
+ * against the process cwd, matching the other `--from-json` consumers in the CLI.
+ * Throws a clean, prefixed message on a missing file or malformed JSON so the caller can
+ * surface it as a bad-args error rather than an uncaught stack trace.
+ */
+export declare function readFieldsFromJsonFile(filePath: string): Record<string, string>;

package/dist/fieldsFromJson.js

@@ -0,0 +1,107 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.coerceFieldsFromJson = coerceFieldsFromJson;
+exports.readFieldsFromJsonFile = readFieldsFromJsonFile;
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+/**
+ * `--from-json` field-map loader for the record-write verbs (set-field, create-record).
+ *
+ * The inline `--fields "k=v,k2=v2"` form splits on commas and trims each piece — so it
+ * cannot carry a value that contains a comma, a newline, an `=`, or leading/trailing
+ * whitespace. That rules out every large field: a script body, an HTML/XML/CSS field, a
+ * JSON blob. `--from-json <path>` closes that gap: the file is a flat JSON object of
+ * `field → value`, and JSON quoting carries any character faithfully.
+ *
+ * Values may be string | number | boolean. Numbers/booleans are stringified — matching
+ * the inline parser and the wire format, where ServiceNow scalar fields are strings.
+ * `null`/`undefined` values are skipped. Nested objects and arrays are rejected: a scalar
+ * field cannot take one, and silently JSON-encoding it would write garbage into the record.
+ */
+function coerceFieldsFromJson(parsed) {
+    if (parsed === null || typeof parsed !== "object" || Array.isArray(parsed)) {
+        throw new Error("--from-json must contain a JSON object of field → value pairs");
+    }
+    var obj = parsed;
+    var out = {};
+    var keys = Object.keys(obj);
+    for (var i = 0; i < keys.length; i += 1) {
+        var key = keys[i];
+        var value = obj[key];
+        if (value === null || value === undefined)
+            continue;
+        var valueType = typeof value;
+        if (valueType === "string") {
+            out[key] = value;
+        }
+        else if (valueType === "number" || valueType === "boolean") {
+            out[key] = String(value);
+        }
+        else {
+            throw new Error("--from-json field '" +
+                key +
+                "' must be a string, number, or boolean (got " +
+                valueType +
+                ")");
+        }
+    }
+    return out;
+}
+/**
+ * Read + parse a `--from-json` file into a validated field map. The path is resolved
+ * against the process cwd, matching the other `--from-json` consumers in the CLI.
+ * Throws a clean, prefixed message on a missing file or malformed JSON so the caller can
+ * surface it as a bad-args error rather than an uncaught stack trace.
+ */
+function readFieldsFromJsonFile(filePath) {
+    var raw;
+    try {
+        raw = fs.readFileSync(path.resolve(filePath), "utf8");
+    }
+    catch (err) {
+        var readMsg = err instanceof Error ? err.message : String(err);
+        throw new Error("--from-json: cannot read '" + filePath + "': " + readMsg);
+    }
+    var parsed;
+    try {
+        parsed = JSON.parse(raw);
+    }
+    catch (err) {
+        var parseMsg = err instanceof Error ? err.message : String(err);
+        throw new Error("--from-json: '" + filePath + "' is not valid JSON: " + parseMsg);
+    }
+    return coerceFieldsFromJson(parsed);
+}

package/package.json

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