snow-flow 10.0.1-dev.461 → 10.0.1-dev.463

package/package.json

@@ -1,6 +1,6 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
-  "version": "10.0.1-dev.461",
+  "version": "10.0.1-dev.463",
   "name": "snow-flow",
   "description": "Snow-Flow - ServiceNow Multi-Agent Development Framework powered by AI",
   "license": "Elastic-2.0",

package/src/servicenow/servicenow-mcp-unified/tools/flow-designer/snow_manage_flow.ts

@@ -365,6 +365,417 @@ async function calculateInsertOrder(
   return 1;
 }
 
+/**
+ * Flatten an attributes object { key: "val" } into comma-separated "key=val," string (matching UI format).
+ * If already a string, returns as-is.
+ */
+function flattenAttributes(attrs: any): string {
+  if (!attrs || typeof attrs === 'string') return attrs || '';
+  return Object.entries(attrs).map(([k, v]) => k + '=' + v).join(',') + ',';
+}
+
+/**
+ * Build full trigger input and output objects for the INSERT mutation by fetching from the
+ * triggerpicker API (/api/now/hub/triggerpicker/{id}) — the same endpoint Flow Designer UI uses.
+ *
+ * The UI sends ALL inputs with full parameter definitions (choices, defaults, attributes) and
+ * ALL outputs in a single INSERT mutation. This function replicates that format exactly.
+ *
+ * Fallback: if the triggerpicker API fails, queries sys_hub_trigger_input / sys_hub_trigger_output
+ * via the Table API (same approach as buildActionInputsForInsert / buildFlowLogicInputsForInsert).
+ */
+/**
+ * Parse XML string from triggerpicker API to extract input/output elements.
+ * The triggerpicker endpoint may return XML instead of JSON on some instances.
+ */
+function parseTriggerpickerXml(xmlStr: string): { inputs: any[]; outputs: any[] } {
+  var inputs: any[] = [];
+  var outputs: any[] = [];
+
+  // Helper: extract text content of an XML element by tag name
+  var getTag = function (xml: string, tag: string): string {
+    var m = xml.match(new RegExp('<' + tag + '>([\\s\\S]*?)</' + tag + '>'));
+    return m ? m[1].trim() : '';
+  };
+
+  // Helper: extract all occurrences of a repeated element
+  var getAll = function (xml: string, tag: string): string[] {
+    var re = new RegExp('<' + tag + '>([\\s\\S]*?)</' + tag + '>', 'g');
+    var results: string[] = [];
+    var m;
+    while ((m = re.exec(xml)) !== null) results.push(m[1]);
+    return results;
+  };
+
+  // Try to find input elements — XML may wrap them in <inputs><element>...</element></inputs>
+  // or <trigger_inputs><input>...</input></trigger_inputs> etc.
+  var inputsSection = getTag(xmlStr, 'inputs') || getTag(xmlStr, 'trigger_inputs') || xmlStr;
+  var inputElements = getAll(inputsSection, 'element');
+  if (inputElements.length === 0) inputElements = getAll(inputsSection, 'input');
+  if (inputElements.length === 0) inputElements = getAll(inputsSection, 'trigger_input');
+
+  for (var ii = 0; ii < inputElements.length; ii++) {
+    var el = inputElements[ii];
+    var name = getTag(el, 'name') || getTag(el, 'element');
+    if (!name) continue;
+    inputs.push({
+      id: getTag(el, 'sys_id') || getTag(el, 'id'),
+      name: name,
+      label: getTag(el, 'label') || name,
+      type: getTag(el, 'type') || getTag(el, 'internal_type') || 'string',
+      type_label: getTag(el, 'type_label') || '',
+      mandatory: getTag(el, 'mandatory') === 'true',
+      order: parseInt(getTag(el, 'order') || '0', 10),
+      maxsize: parseInt(getTag(el, 'maxsize') || getTag(el, 'max_length') || '4000', 10),
+      hint: getTag(el, 'hint'),
+      defaultValue: getTag(el, 'defaultValue') || getTag(el, 'default_value'),
+      defaultDisplayValue: getTag(el, 'defaultDisplayValue') || getTag(el, 'default_display_value'),
+      choiceOption: getTag(el, 'choiceOption') || getTag(el, 'choice_option'),
+      reference: getTag(el, 'reference'),
+      reference_display: getTag(el, 'reference_display'),
+      use_dependent: getTag(el, 'use_dependent') === 'true',
+      dependent_on: getTag(el, 'dependent_on'),
+      internal_link: getTag(el, 'internal_link'),
+      attributes: getTag(el, 'attributes')
+    });
+  }
+
+  var outputsSection = getTag(xmlStr, 'outputs') || getTag(xmlStr, 'trigger_outputs') || '';
+  var outputElements = getAll(outputsSection, 'element');
+  if (outputElements.length === 0) outputElements = getAll(outputsSection, 'output');
+  if (outputElements.length === 0) outputElements = getAll(outputsSection, 'trigger_output');
+
+  for (var oi = 0; oi < outputElements.length; oi++) {
+    var oel = outputElements[oi];
+    var oname = getTag(oel, 'name') || getTag(oel, 'element');
+    if (!oname) continue;
+    outputs.push({
+      id: getTag(oel, 'sys_id') || getTag(oel, 'id'),
+      name: oname,
+      label: getTag(oel, 'label') || oname,
+      type: getTag(oel, 'type') || getTag(oel, 'internal_type') || 'string',
+      type_label: getTag(oel, 'type_label') || '',
+      mandatory: getTag(oel, 'mandatory') === 'true',
+      order: parseInt(getTag(oel, 'order') || '0', 10),
+      maxsize: parseInt(getTag(oel, 'maxsize') || getTag(oel, 'max_length') || '200', 10),
+      hint: getTag(oel, 'hint'),
+      reference: getTag(oel, 'reference'),
+      reference_display: getTag(oel, 'reference_display'),
+      use_dependent: getTag(oel, 'use_dependent') === 'true',
+      dependent_on: getTag(oel, 'dependent_on'),
+      internal_link: getTag(oel, 'internal_link'),
+      attributes: getTag(oel, 'attributes')
+    });
+  }
+
+  return { inputs, outputs };
+}
+
+/**
+ * Build a single trigger input object in GraphQL mutation format.
+ * Used by buildTriggerInputsForInsert and the hardcoded fallback.
+ */
+function buildTriggerInputObj(inp: any, userTable?: string, userCondition?: string): any {
+  var paramType = inp.type || 'string';
+  var name = inp.name || '';
+  var label = inp.label || name;
+  var attrs = typeof inp.attributes === 'object' ? flattenAttributes(inp.attributes) : (inp.attributes || '');
+
+  // Determine value: user-provided > default
+  var value = '';
+  if (name === 'table' && userTable) value = userTable;
+  else if (name === 'condition') value = userCondition || '^EQ';
+  else if (inp.defaultValue) value = inp.defaultValue;
+
+  var parameter: any = {
+    id: inp.id || '', label: label, name: name, type: paramType,
+    type_label: inp.type_label || TYPE_LABELS[paramType] || paramType,
+    order: inp.order || 0, extended: inp.extended || false,
+    mandatory: inp.mandatory || false, readonly: inp.readonly || false,
+    maxsize: inp.maxsize || 4000, data_structure: '',
+    reference: inp.reference || '', reference_display: inp.reference_display || '',
+    ref_qual: inp.ref_qual || '', choiceOption: inp.choiceOption || '',
+    table: '', columnName: '', defaultValue: inp.defaultValue || '',
+    use_dependent: inp.use_dependent || false, dependent_on: inp.dependent_on || '',
+    internal_link: inp.internal_link || '', show_ref_finder: inp.show_ref_finder || false,
+    local: inp.local || false, attributes: attrs, sys_class_name: '', children: []
+  };
+  if (inp.hint) parameter.hint = inp.hint;
+  if (inp.defaultDisplayValue) parameter.defaultDisplayValue = inp.defaultDisplayValue;
+  if (inp.choices) parameter.choices = inp.choices;
+  if (inp.defaultChoices) parameter.defaultChoices = inp.defaultChoices;
+
+  var inputObj: any = {
+    name: name, label: label, internalType: paramType,
+    mandatory: inp.mandatory || false, order: inp.order || 0,
+    valueSysId: '', field_name: name, type: paramType, children: [],
+    displayValue: { value: '' },
+    value: value ? { schemaless: false, schemalessValue: '', value: value } : { value: '' },
+    parameter: parameter
+  };
+
+  if (inp.choices && Array.isArray(inp.choices)) {
+    inputObj.choiceList = inp.choices.map(function (c: any) {
+      return { label: c.label, value: c.value };
+    });
+  }
+
+  return inputObj;
+}
+
+/**
+ * Build a single trigger output object in GraphQL mutation format.
+ */
+function buildTriggerOutputObj(out: any): any {
+  var paramType = out.type || 'string';
+  var name = out.name || '';
+  var label = out.label || name;
+  var attrs = typeof out.attributes === 'object' ? flattenAttributes(out.attributes) : (out.attributes || '');
+
+  var parameter: any = {
+    id: out.id || '', label: label, name: name, type: paramType,
+    type_label: out.type_label || TYPE_LABELS[paramType] || paramType,
+    hint: out.hint || '', order: out.order || 0, extended: out.extended || false,
+    mandatory: out.mandatory || false, readonly: out.readonly || false,
+    maxsize: out.maxsize || 200, data_structure: '',
+    reference: out.reference || '', reference_display: out.reference_display || '',
+    ref_qual: '', choiceOption: '', table: '', columnName: '', defaultValue: '',
+    use_dependent: out.use_dependent || false, dependent_on: out.dependent_on || '',
+    internal_link: out.internal_link || '', show_ref_finder: false, local: false,
+    attributes: attrs, sys_class_name: ''
+  };
+
+  var children: any[] = [];
+  var paramChildren: any[] = [];
+  if (out.children && Array.isArray(out.children)) {
+    children = out.children.map(function (child: any) {
+      return { id: '', name: child.name || '', scriptActive: false, children: [], value: { value: '' }, script: null };
+    });
+    paramChildren = out.children.map(function (child: any) {
+      return {
+        id: '', label: child.label || child.name || '', name: child.name || '',
+        type: child.type || 'string', type_label: child.type_label || TYPE_LABELS[child.type || 'string'] || 'String',
+        hint: '', order: child.order || 0, extended: false, mandatory: false, readonly: false, maxsize: 0,
+        data_structure: '', reference: '', reference_display: '', ref_qual: '', choiceOption: '',
+        table: '', columnName: '', defaultValue: '', defaultDisplayValue: '',
+        use_dependent: false, dependent_on: false, show_ref_finder: false, local: false,
+        attributes: '', sys_class_name: '',
+        uiDisplayType: child.uiDisplayType || child.type || 'string',
+        uiDisplayTypeLabel: child.type_label || 'String',
+        internal_link: '', value: '', display_value: '', scriptActive: false,
+        parent: out.id || '',
+        fieldFacetMap: 'uiTypeLabel=' + (child.type_label || 'String') + ',',
+        children: [], script: null
+      };
+    });
+  }
+  parameter.children = paramChildren;
+
+  return {
+    name: name, value: '', displayValue: '', type: paramType,
+    order: out.order || 0, label: label, children: children, parameter: parameter
+  };
+}
+
+/**
+ * Hardcoded record trigger inputs — used as ultimate fallback when API and Table lookups fail.
+ * These definitions match the exact format captured from the Flow Designer UI for record-based triggers
+ * (record_create, record_update, record_create_or_update). Field names and types are consistent across instances.
+ */
+function getRecordTriggerFallbackInputs(): any[] {
+  return [
+    { name: 'table', label: 'Table', type: 'table_name', type_label: 'Table Name', mandatory: true, order: 1, maxsize: 80, attributes: 'filter_table_source=RECORD_WATCHER_RESTRICTED,' },
+    { name: 'condition', label: 'Condition', type: 'conditions', type_label: 'Conditions', mandatory: false, order: 100, maxsize: 4000, use_dependent: true, dependent_on: 'table', attributes: 'extended_operators=VALCHANGES;CHANGESFROM;CHANGESTO,wants_to_add_conditions=true,modelDependent=trigger_inputs,' },
+    { name: 'run_on_extended', label: 'run_on_extended', type: 'choice', type_label: 'Choice', mandatory: false, order: 100, maxsize: 40, defaultValue: 'false', defaultDisplayValue: 'Run only on current table', choiceOption: '3', attributes: 'advanced=true,', choices: [{ label: 'Run only on current table', value: 'false', order: 0 }, { label: 'Run on current and extended tables', value: 'true', order: 1 }], defaultChoices: [{ label: 'Run only on current table', value: 'false', order: 1 }, { label: 'Run on current and extended tables', value: 'true', order: 2 }] },
+    { name: 'run_flow_in', label: 'run_flow_in', type: 'choice', type_label: 'Choice', mandatory: false, order: 100, maxsize: 40, defaultValue: 'any', defaultDisplayValue: 'any', choiceOption: '3', attributes: 'advanced=true,', choices: [{ label: 'Run flow in background (default)', value: 'background', order: 0 }, { label: 'Run flow in foreground', value: 'foreground', order: 1 }], defaultChoices: [{ label: 'Run flow in background (default)', value: 'background', order: 1 }, { label: 'Run flow in foreground', value: 'foreground', order: 2 }] },
+    { name: 'run_when_user_list', label: 'run_when_user_list', type: 'glide_list', type_label: 'List', mandatory: false, order: 100, maxsize: 4000, reference: 'sys_user', reference_display: 'User', attributes: 'advanced=true,' },
+    { name: 'run_when_setting', label: 'run_when_setting', type: 'choice', type_label: 'Choice', mandatory: false, order: 100, maxsize: 40, defaultValue: 'both', defaultDisplayValue: 'Run for Both Interactive and Non-Interactive Sessions', choiceOption: '3', attributes: 'advanced=true,', choices: [{ label: 'Only Run for Non-Interactive Session', value: 'non_interactive', order: 0 }, { label: 'Only Run for User Interactive Session', value: 'interactive', order: 1 }, { label: 'Run for Both Interactive and Non-Interactive Sessions', value: 'both', order: 2 }], defaultChoices: [{ label: 'Only Run for Non-Interactive Session', value: 'non_interactive', order: 1 }, { label: 'Only Run for User Interactive Session', value: 'interactive', order: 2 }, { label: 'Run for Both Interactive and Non-Interactive Sessions', value: 'both', order: 3 }] },
+    { name: 'run_when_user_setting', label: 'run_when_user_setting', type: 'choice', type_label: 'Choice', mandatory: false, order: 100, maxsize: 40, defaultValue: 'any', defaultDisplayValue: 'Run for any user', choiceOption: '3', attributes: 'advanced=true,', choices: [{ label: 'Do not run if triggered by the following users', value: 'not_one_of', order: 0 }, { label: 'Only Run if triggered by the following users', value: 'one_of', order: 1 }, { label: 'Run for any user', value: 'any', order: 2 }], defaultChoices: [{ label: 'Do not run if triggered by the following users', value: 'not_one_of', order: 1 }, { label: 'Only Run if triggered by the following users', value: 'one_of', order: 2 }, { label: 'Run for any user', value: 'any', order: 3 }] },
+    { name: 'trigger_strategy', label: 'Run Trigger', type: 'choice', type_label: 'Choice', mandatory: false, order: 200, maxsize: 40, defaultValue: 'once', defaultDisplayValue: 'Once', choiceOption: '3', hint: 'Run Trigger every time the condition matches, or only the first time.', choices: [{ label: 'Once', value: 'once', order: 0 }, { label: 'For each unique change', value: 'unique_changes', order: 1 }, { label: 'Only if not currently running', value: 'always', order: 2 }, { label: 'For every update', value: 'every', order: 3 }], defaultChoices: [{ label: 'Once', value: 'once', order: 1 }, { label: 'For each unique change', value: 'unique_changes', order: 2 }, { label: 'Only if not currently running', value: 'always', order: 3 }, { label: 'For every update', value: 'every', order: 4 }] }
+  ];
+}
+
+function getRecordTriggerFallbackOutputs(): any[] {
+  return [
+    { name: 'current', label: 'Record', type: 'document_id', type_label: 'Document ID', mandatory: true, order: 100, maxsize: 200, use_dependent: true, dependent_on: 'table_name', internal_link: 'table' },
+    { name: 'changed_fields', label: 'Changed Fields', type: 'array.object', type_label: 'Array.Object', mandatory: false, order: 100, maxsize: 4000, attributes: 'uiTypeLabel=Array.Object,co_type_name=FDCollection,child_label=FDChangeDetails,child_type_label=Object,element_mapping_provider=com.glide.flow_design.action.data.FlowDesignVariableMapper,pwd2droppable=true,uiType=array.object,child_type=object,child_name=FDChangeDetails,', children: [{ name: 'field_name', label: 'Field Name', type: 'string', type_label: 'String', order: 1 }, { name: 'previous_value', label: 'Previous Value', type: 'string', type_label: 'String', order: 2 }, { name: 'current_value', label: 'Current Value', type: 'string', type_label: 'String', order: 3 }, { name: 'previous_display_value', label: 'Previous Display Value', type: 'string', type_label: 'String', order: 4 }, { name: 'current_display_value', label: 'Current Display Value', type: 'string', type_label: 'String', order: 5 }] },
+    { name: 'table_name', label: 'Table Name', type: 'table_name', type_label: 'Table Name', mandatory: false, order: 101, maxsize: 200, internal_link: 'table', attributes: 'test_input_hidden=true,' },
+    { name: 'run_start_time', label: 'Run Start Time UTC', type: 'glide_date_time', type_label: 'Date/Time', mandatory: false, order: 110, maxsize: 200, attributes: 'test_input_hidden=true,' },
+    { name: 'run_start_date_time', label: 'Run Start Date/Time', type: 'glide_date_time', type_label: 'Date/Time', mandatory: false, order: 110, maxsize: 200, attributes: 'test_input_hidden=true,' }
+  ];
+}
+
+async function buildTriggerInputsForInsert(
+  client: any,
+  trigDefId: string,
+  trigType: string,
+  userTable?: string,
+  userCondition?: string
+): Promise<{ inputs: any[]; outputs: any[]; source: string; error?: string }> {
+  var apiInputs: any[] = [];
+  var apiOutputs: any[] = [];
+  var fetchError = '';
+  var source = '';
+
+  // Strategy 1: triggerpicker API (primary — same as Flow Designer UI)
+  try {
+    var tpResp = await client.get('/api/now/hub/triggerpicker/' + trigDefId, {
+      params: { sysparm_transaction_scope: 'global' },
+      headers: { Accept: 'application/json' }
+    });
+    var tpRaw = tpResp.data;
+    var tpData = tpRaw?.result || tpRaw;
+
+    // Handle JSON object response
+    if (tpData && typeof tpData === 'object' && !Array.isArray(tpData)) {
+      // Try common field name variations
+      var foundInputs = Array.isArray(tpData.inputs) ? tpData.inputs
+        : Array.isArray(tpData.trigger_inputs) ? tpData.trigger_inputs
+        : Array.isArray(tpData.input) ? tpData.input : null;
+      var foundOutputs = Array.isArray(tpData.outputs) ? tpData.outputs
+        : Array.isArray(tpData.trigger_outputs) ? tpData.trigger_outputs
+        : Array.isArray(tpData.output) ? tpData.output : null;
+      if (foundInputs) { apiInputs = foundInputs; source = 'triggerpicker_json'; }
+      if (foundOutputs) apiOutputs = foundOutputs;
+
+      // If no arrays found, try to explore nested structure
+      if (!foundInputs && !foundOutputs) {
+        for (var key of Object.keys(tpData)) {
+          var val = tpData[key];
+          if (val && typeof val === 'object' && !Array.isArray(val)) {
+            if (Array.isArray(val.inputs)) { apiInputs = val.inputs; source = 'triggerpicker_json.' + key; }
+            if (Array.isArray(val.outputs)) apiOutputs = val.outputs;
+          }
+        }
+      }
+    }
+
+    // Handle XML string response
+    if (apiInputs.length === 0 && typeof tpData === 'string' && tpData.includes('<')) {
+      var xmlResult = parseTriggerpickerXml(tpData);
+      if (xmlResult.inputs.length > 0) {
+        apiInputs = xmlResult.inputs;
+        apiOutputs = xmlResult.outputs;
+        source = 'triggerpicker_xml';
+      }
+    }
+    // Also check if the raw response itself is XML (not wrapped in result)
+    if (apiInputs.length === 0 && typeof tpRaw === 'string' && tpRaw.includes('<')) {
+      var xmlResult2 = parseTriggerpickerXml(tpRaw);
+      if (xmlResult2.inputs.length > 0) {
+        apiInputs = xmlResult2.inputs;
+        apiOutputs = xmlResult2.outputs;
+        source = 'triggerpicker_xml_raw';
+      }
+    }
+  } catch (tpErr: any) {
+    fetchError = 'triggerpicker: ' + (tpErr.message || 'unknown');
+  }
+
+  // Strategy 2: Table API fallback (query sys_hub_trigger_input / sys_hub_trigger_output)
+  if (apiInputs.length === 0) {
+    try {
+      var tiResp = await client.get('/api/now/table/sys_hub_trigger_input', {
+        params: {
+          sysparm_query: 'model=' + trigDefId,
+          sysparm_fields: 'sys_id,element,label,internal_type,mandatory,default_value,order,max_length,hint,read_only,attributes,reference,reference_display,choice,dependent_on_field,use_dependent_field',
+          sysparm_display_value: 'false',
+          sysparm_limit: 50
+        }
+      });
+      var tableInputs = tiResp.data.result || [];
+      if (tableInputs.length > 0) {
+        apiInputs = tableInputs.map(function (rec: any) {
+          return {
+            id: str(rec.sys_id), name: str(rec.element), label: str(rec.label) || str(rec.element),
+            type: str(rec.internal_type) || 'string',
+            type_label: TYPE_LABELS[str(rec.internal_type) || 'string'] || str(rec.internal_type),
+            mandatory: str(rec.mandatory) === 'true',
+            order: parseInt(str(rec.order) || '0', 10),
+            maxsize: parseInt(str(rec.max_length) || '4000', 10),
+            hint: str(rec.hint), defaultValue: str(rec.default_value),
+            reference: str(rec.reference), reference_display: str(rec.reference_display),
+            use_dependent: str(rec.use_dependent_field) === 'true',
+            dependent_on: str(rec.dependent_on_field),
+            attributes: str(rec.attributes)
+          };
+        });
+        source = 'table_api';
+        fetchError = '';
+      }
+    } catch (tiErr: any) {
+      fetchError += '; table_api_inputs: ' + (tiErr.message || 'unknown');
+    }
+  }
+  if (apiOutputs.length === 0) {
+    try {
+      var toResp = await client.get('/api/now/table/sys_hub_trigger_output', {
+        params: {
+          sysparm_query: 'model=' + trigDefId,
+          sysparm_fields: 'sys_id,element,label,internal_type,mandatory,order,max_length,hint,attributes,reference,reference_display,use_dependent_field,dependent_on_field',
+          sysparm_display_value: 'false',
+          sysparm_limit: 50
+        }
+      });
+      var tableOutputs = toResp.data.result || [];
+      if (tableOutputs.length > 0) {
+        apiOutputs = tableOutputs.map(function (rec: any) {
+          return {
+            id: str(rec.sys_id), name: str(rec.element), label: str(rec.label) || str(rec.element),
+            type: str(rec.internal_type) || 'string',
+            type_label: TYPE_LABELS[str(rec.internal_type) || 'string'] || str(rec.internal_type),
+            mandatory: str(rec.mandatory) === 'true',
+            order: parseInt(str(rec.order) || '0', 10),
+            maxsize: parseInt(str(rec.max_length) || '200', 10),
+            hint: str(rec.hint), reference: str(rec.reference), reference_display: str(rec.reference_display),
+            use_dependent: str(rec.use_dependent_field) === 'true',
+            dependent_on: str(rec.dependent_on_field),
+            attributes: str(rec.attributes)
+          };
+        });
+      }
+    } catch (_) {}
+  }
+
+  // Strategy 3: Hardcoded fallback for record-based triggers (ultimate safety net)
+  // Uses exact definitions captured from the Flow Designer UI
+  var isRecordTrigger = /record/.test(trigType.toLowerCase());
+  if (apiInputs.length === 0 && isRecordTrigger) {
+    apiInputs = getRecordTriggerFallbackInputs();
+    source = 'hardcoded_fallback';
+  }
+  if (apiOutputs.length === 0 && isRecordTrigger) {
+    apiOutputs = getRecordTriggerFallbackOutputs();
+  }
+
+  // Transform to GraphQL mutation format
+  var inputs = apiInputs.map(function (inp: any) { return buildTriggerInputObj(inp, userTable, userCondition); });
+  var outputs = apiOutputs.map(function (out: any) { return buildTriggerOutputObj(out); });
+
+  // Final safety net: ensure table and condition inputs are ALWAYS present for record triggers
+  if (isRecordTrigger) {
+    var hasTable = inputs.some(function (i: any) { return i.name === 'table'; });
+    var hasCondition = inputs.some(function (i: any) { return i.name === 'condition'; });
+    if (!hasTable) {
+      inputs.unshift(buildTriggerInputObj(
+        { name: 'table', label: 'Table', type: 'table_name', type_label: 'Table Name', mandatory: true, order: 1, maxsize: 80, attributes: 'filter_table_source=RECORD_WATCHER_RESTRICTED,' },
+        userTable, userCondition
+      ));
+      source += '+table_injected';
+    }
+    if (!hasCondition) {
+      var condIdx = inputs.findIndex(function (i: any) { return i.name === 'table'; });
+      inputs.splice(condIdx + 1, 0, buildTriggerInputObj(
+        { name: 'condition', label: 'Condition', type: 'conditions', type_label: 'Conditions', mandatory: false, order: 100, maxsize: 4000, use_dependent: true, dependent_on: 'table', attributes: 'extended_operators=VALCHANGES;CHANGESFROM;CHANGESTO,wants_to_add_conditions=true,modelDependent=trigger_inputs,' },
+        userTable, userCondition
+      ));
+      source += '+condition_injected';
+    }
+  }
+
+  return { inputs, outputs, source: source || 'none', error: fetchError || undefined };
+}
+
 async function addTriggerViaGraphQL(
   client: any,
   flowId: string,
@@ -432,8 +843,13 @@ async function addTriggerViaGraphQL(
   }
   if (!trigDefId) return { success: false, error: 'Trigger definition not found for: ' + triggerType, steps };
 
+  // Build full trigger inputs and outputs from triggerpicker API (matching UI format)
+  var triggerData = await buildTriggerInputsForInsert(client, trigDefId!, trigType, table, condition);
+  steps.trigger_data = { inputCount: triggerData.inputs.length, outputCount: triggerData.outputs.length, source: triggerData.source, error: triggerData.error };
+
   const triggerResponseFields = 'triggerInstances { inserts { sysId uiUniqueIdentifier __typename } updates deletes __typename }';
   try {
+    // Single INSERT with full inputs and outputs (matching UI behavior — no separate UPDATE needed)
     const insertResult = await executeFlowPatchMutation(client, {
       flowId: flowId,
       triggerInstances: {
@@ -445,8 +861,8 @@ async function addTriggerViaGraphQL(
           type: trigType,
           hasDynamicOutputs: false,
           metadata: '{"predicates":[]}',
-          inputs: [],
-          outputs: []
+          inputs: triggerData.inputs,
+          outputs: triggerData.outputs
         }]
       }
     }, triggerResponseFields);
@@ -455,31 +871,6 @@ async function addTriggerViaGraphQL(
     steps.insert = { success: !!triggerId, triggerId };
     if (!triggerId) return { success: false, steps, error: 'GraphQL trigger INSERT returned no trigger ID' };
 
-    if (table) {
-      const updateInputs: any[] = [
-        {
-          name: 'table',
-          displayField: 'number',
-          displayValue: { schemaless: false, schemalessValue: '', value: table.charAt(0).toUpperCase() + table.slice(1) },
-          value: { schemaless: false, schemalessValue: '', value: table }
-        },
-        {
-          name: 'condition',
-          displayValue: { schemaless: false, schemalessValue: '', value: condition || '^EQ' },
-          value: { schemaless: false, schemalessValue: '', value: condition || '^EQ' }
-        }
-      ];
-      try {
-        await executeFlowPatchMutation(client, {
-          flowId: flowId,
-          triggerInstances: { update: [{ id: triggerId, inputs: updateInputs }] }
-        }, triggerResponseFields);
-        steps.update = { success: true, table, condition: condition || '^EQ' };
-      } catch (e: any) {
-        steps.update = { success: false, error: e.message };
-      }
-    }
-
     return { success: true, triggerId, steps };
   } catch (e: any) {
     steps.insert = { success: false, error: e.message };
@@ -487,6 +878,18 @@ async function addTriggerViaGraphQL(
   }
 }
 
+// Common short-name aliases for action types — maps user-friendly names to ServiceNow internal names
+const ACTION_TYPE_ALIASES: Record<string, string[]> = {
+  script: ['script_step', 'run_script', 'Run Script'],
+  log: ['log_message', 'Log Message', 'Log'],
+  create_record: ['Create Record'],
+  update_record: ['Update Record'],
+  notification: ['send_notification', 'send_email', 'Send Notification', 'Send Email'],
+  field_update: ['set_field_values', 'Set Field Values'],
+  wait: ['wait_for', 'Wait For Duration', 'Wait'],
+  approval: ['ask_for_approval', 'create_approval', 'Ask for Approval'],
+};
+
 async function addActionViaGraphQL(
   client: any,
   flowId: string,
@@ -499,7 +902,7 @@ async function addActionViaGraphQL(
 ): Promise<{ success: boolean; actionId?: string; steps?: any; error?: string }> {
   const steps: any = {};
 
-  // Dynamically look up action definition in sys_hub_action_type_snapshot
+  // Dynamically look up action definition in sys_hub_action_type_snapshot and sys_hub_action_type_definition
   // Prefer global/core actions over spoke-specific ones (e.g. core "Update Record" vs spoke-specific "Update Record")
   const snapshotFields = 'sys_id,internal_name,name,sys_scope,sys_package';
   let actionDefId: string | null = null;
@@ -527,41 +930,66 @@ async function addActionViaGraphQL(
     return candidates[0];
   };
 
-  for (const field of ['internal_name', 'name']) {
-    if (actionDefId) break;
-    try {
-      const resp = await client.get('/api/now/table/sys_hub_action_type_snapshot', {
-        params: { sysparm_query: field + '=' + actionType, sysparm_fields: snapshotFields, sysparm_limit: 10 }
-      });
-      const results = resp.data.result || [];
-      if (results.length > 1) {
-        steps.def_lookup_candidates = results.map((r: any) => ({ sys_id: r.sys_id, internal_name: str(r.internal_name), name: str(r.name), scope: str(r.sys_scope), package: str(r.sys_package) }));
+  // Helper: search a table for action definitions by exact match and LIKE
+  const searchTable = async (tableName: string, searchTerms: string[]): Promise<void> => {
+    for (var si = 0; si < searchTerms.length && !actionDefId; si++) {
+      var term = searchTerms[si];
+      // Exact match on internal_name and name
+      for (const field of ['internal_name', 'name']) {
+        if (actionDefId) break;
+        try {
+          const resp = await client.get('/api/now/table/' + tableName, {
+            params: { sysparm_query: field + '=' + term, sysparm_fields: snapshotFields, sysparm_limit: 10 }
+          });
+          const results = resp.data.result || [];
+          if (results.length > 1) {
+            steps.def_lookup_candidates = results.map((r: any) => ({ sys_id: r.sys_id, internal_name: str(r.internal_name), name: str(r.name), scope: str(r.sys_scope), package: str(r.sys_package) }));
+          }
+          const found = pickBest(results);
+          if (found?.sys_id) {
+            actionDefId = found.sys_id;
+            steps.def_lookup = { id: found.sys_id, internal_name: str(found.internal_name), name: str(found.name), scope: str(found.sys_scope), package: str(found.sys_package), matched: tableName + ':' + field + '=' + term };
+          }
+        } catch (_) {}
       }
-      const found = pickBest(results);
-      if (found?.sys_id) {
-        actionDefId = found.sys_id;
-        steps.def_lookup = { id: found.sys_id, internal_name: str(found.internal_name), name: str(found.name), scope: str(found.sys_scope), package: str(found.sys_package), matched: field + '=' + actionType };
+      // LIKE search
+      if (!actionDefId) {
+        try {
+          const resp = await client.get('/api/now/table/' + tableName, {
+            params: {
+              sysparm_query: 'internal_nameLIKE' + term + '^ORnameLIKE' + term,
+              sysparm_fields: snapshotFields, sysparm_limit: 10
+            }
+          });
+          const results = resp.data.result || [];
+          if (results.length > 0 && !steps.def_lookup_fallback_candidates) {
+            steps.def_lookup_fallback_candidates = results.map((r: any) => ({ sys_id: r.sys_id, internal_name: str(r.internal_name), name: str(r.name), scope: str(r.sys_scope), package: str(r.sys_package) }));
+          }
+          const found = pickBest(results);
+          if (found?.sys_id) {
+            actionDefId = found.sys_id;
+            steps.def_lookup = { id: found.sys_id, internal_name: str(found.internal_name), name: str(found.name), scope: str(found.sys_scope), package: str(found.sys_package), matched: tableName + ':LIKE ' + term };
+          }
+        } catch (_) {}
       }
-    } catch (_) {}
-  }
+    }
+  };
+
+  // Build search terms: original actionType + any alias variations
+  var searchTerms = [actionType];
+  var aliases = ACTION_TYPE_ALIASES[actionType.toLowerCase()];
+  if (aliases) searchTerms = searchTerms.concat(aliases);
+
+  // Search 1: sys_hub_action_type_snapshot (published action snapshots)
+  await searchTable('sys_hub_action_type_snapshot', searchTerms);
+
+  // Search 2: sys_hub_action_type_definition (action definitions — includes built-in/native actions)
   if (!actionDefId) {
-    try {
-      const resp = await client.get('/api/now/table/sys_hub_action_type_snapshot', {
-        params: {
-          sysparm_query: 'internal_nameLIKE' + actionType + '^ORnameLIKE' + actionType,
-          sysparm_fields: snapshotFields, sysparm_limit: 10
-        }
-      });
-      const results = resp.data.result || [];
-      steps.def_lookup_fallback_candidates = results.map((r: any) => ({ sys_id: r.sys_id, internal_name: str(r.internal_name), name: str(r.name), scope: str(r.sys_scope), package: str(r.sys_package) }));
-      const found = pickBest(results);
-      if (found?.sys_id) {
-        actionDefId = found.sys_id;
-        steps.def_lookup = { id: found.sys_id, internal_name: str(found.internal_name), name: str(found.name), scope: str(found.sys_scope), package: str(found.sys_package), matched: 'LIKE ' + actionType };
-      }
-    } catch (_) {}
+    steps.snapshot_not_found = true;
+    await searchTable('sys_hub_action_type_definition', searchTerms);
   }
-  if (!actionDefId) return { success: false, error: 'Action definition not found for: ' + actionType, steps };
+
+  if (!actionDefId) return { success: false, error: 'Action definition not found for: ' + actionType + ' (searched snapshot + definition tables with terms: ' + searchTerms.join(', ') + ')', steps };
 
   // Build full input objects with parameter definitions (matching UI format)
   const inputResult = await buildActionInputsForInsert(client, actionDefId, inputs);
@@ -1151,8 +1579,7 @@ export const toolDefinition: MCPToolDefinition = {
       },
       action_type: {
         type: 'string',
-        enum: ['log', 'create_record', 'update_record', 'notification', 'script', 'field_update', 'wait', 'approval'],
-        description: 'Action type to add (for add_action)',
+        description: 'Action type to add (for add_action). Looked up dynamically by internal_name or name in sys_hub_action_type_snapshot and sys_hub_action_type_definition. Common short names: log, script, create_record, update_record, notification, field_update, wait, approval. You can also use the exact ServiceNow internal name (e.g. "sn_fd.script_step", "global.update_record") or display name (e.g. "Run Script", "Update Record").',
         default: 'log'
       },
       action_name: {