@jacebenson/jsn 4.3.11 → 4.3.12-dev.70

package/src/helpers.js

@@ -362,7 +362,7 @@ export function assertSafeExactMatch(value) {
 export async function interactiveList({ app, table, singular, columns, limit = 50, query = '', formatLabel, labelField = 'name', message, formatValue, promptFn }) {
   const prompt = promptFn || paginatedSearch;
   app.requireInstance();
-  const effectiveFormat = app.output.getFormat() === FormatAuto ? (isTTY(process.stdout) ? FormatAuto : FormatAuto) : app.output.getFormat();
+  const effectiveFormat = app.output.getFormat();
   if (!promptFn && (effectiveFormat !== FormatAuto || !canPrompt())) {
     return null; // not interactive — caller should fall back to text/table
   }
@@ -522,6 +522,50 @@ export function parseDataArg(argv) {
   }
 }
 
+/**
+ * Parse repeatable -f/--field name=value options.
+ * Values remain strings, including empty values and values containing '='.
+ */
+export function parseFieldPairs(argv = {}) {
+  const raw = argv.field == null ? [] : (Array.isArray(argv.field) ? argv.field : [argv.field]);
+  const result = {};
+  for (const token of raw) {
+    const text = String(token);
+    const equals = text.indexOf('=');
+    if (equals < 0) throw new Error(`-f expects field=value, got "${text}"`);
+    const name = text.slice(0, equals);
+    let value = text.slice(equals + 1);
+    if (!name || /\\s/.test(name) || !/^[A-Za-z0-9_.]+$/.test(name)) {
+      throw new Error(`-f expects a valid field name, got "${name}"`);
+    }
+    if (value.startsWith('@@')) {
+      value = value.slice(1);
+    } else if (value.startsWith('@')) {
+      value = fs.readFileSync(value.slice(1), 'utf-8');
+      if (value.charCodeAt(0) === 0xFEFF) value = value.slice(1);
+    }
+    result[name] = value;
+  }
+  return result;
+}
+
+/**
+ * Build a record mutation payload from JSON input and/or field pairs.
+ */
+export function buildRecordPayload(argv = {}, { required = true } = {}) {
+  const hasData = Boolean(argv.data || argv['data-file'] || argv['data-stdin']);
+  const hasFields = argv.field != null && (Array.isArray(argv.field) ? argv.field.length > 0 : true);
+  if (!hasData && !hasFields) {
+    if (required) throw new Error('Provide fields with -f name=value, or a JSON payload with --data/--data-file/--data-stdin');
+    return {};
+  }
+  const payload = hasData ? parseDataArg(argv) : {};
+  if (!payload || typeof payload !== 'object' || Array.isArray(payload)) {
+    throw new Error('Record payload must be a JSON object');
+  }
+  return { ...payload, ...parseFieldPairs(argv) };
+}
+
 /**
  * Translate human-readable type names to ServiceNow item_option_new type IDs.
  * Maps common names like "date", "select", "multilinetext" to their integer IDs.

package/src/paginated-search.js

@@ -2,89 +2,110 @@
  * Paginated search prompt built with @inquirer/core hooks.
  */
 
-import { createPrompt, useState, useKeypress, usePagination, useEffect, useRef } from '@inquirer/core';
-import { isEnterKey, isUpKey, isDownKey } from '@inquirer/core';
-
-export const paginatedSearch = createPrompt((config, done) => {
-  const { message, pageSize = 10, totalCount = 0, source } = config;
-
-  const [status, setStatus] = useState('loading');
-  const [searchTerm, setSearchTerm] = useState('');
-  const [choices, setChoices] = useState([]);
-  const [active, setActive] = useState(0);
-  const [loaded, setLoaded] = useState(0);
-  const [searchMode, setSearchMode] = useState(false);
-
-  // Refs for mutable state (no functional updaters — @inquirer/core doesn't support them)
-  const choicesRef = useRef([]);
-  const loadedRef = useRef(0);
-  const activeRef = useRef(0);
-  const totalCountRef = useRef(totalCount);
-  const searchModeRef = useRef(false);
-
-  const fullMessage = totalCount > 0
-    ? `${message} (${loaded} of ${totalCount} loaded)`
-    : message;
-
-  // Load initial page
-  useEffect(() => {
-    const controller = new AbortController();
-    setStatus('loading');
-
-    const load = async () => {
-      try {
-        const items = await source(searchTerm || undefined, 0, { signal: controller.signal });
-        if (!controller.signal.aborted) {
-          choicesRef.current = items;
-          loadedRef.current = items.length;
-          activeRef.current = 0;
-          setChoices(items);
-          setLoaded(items.length);
-          setActive(0);
-          setStatus('done');
-        }
-      } catch {
-        if (!controller.signal.aborted) {
-          setStatus('done');
-          setChoices([]);
-          setLoaded(0);
+
+export const paginatedSearch = async (...args) => {
+  const { createPrompt, useState, useKeypress, usePagination, useEffect, useRef, isEnterKey, isUpKey, isDownKey } = await import('@inquirer/core');
+  return createPrompt((config, done) => {
+    const { message, pageSize = 10, totalCount = 0, source } = config;
+
+    const [status, setStatus] = useState('loading');
+    const [searchTerm, setSearchTerm] = useState('');
+    const [choices, setChoices] = useState([]);
+    const [active, setActive] = useState(0);
+    const [loaded, setLoaded] = useState(0);
+    const [searchMode, setSearchMode] = useState(false);
+
+    // Refs for mutable state (no functional updaters — @inquirer/core doesn't support them)
+    const choicesRef = useRef([]);
+    const loadedRef = useRef(0);
+    const activeRef = useRef(0);
+    const totalCountRef = useRef(totalCount);
+    const searchModeRef = useRef(false);
+
+    const fullMessage = totalCount > 0
+      ? `${message} (${loaded} of ${totalCount} loaded)`
+      : message;
+
+    // Load initial page
+    useEffect(() => {
+      const controller = new AbortController();
+      setStatus('loading');
+
+      const load = async () => {
+        try {
+          const items = await source(searchTerm || undefined, 0, { signal: controller.signal });
+          if (!controller.signal.aborted) {
+            choicesRef.current = items;
+            loadedRef.current = items.length;
+            activeRef.current = 0;
+            setChoices(items);
+            setLoaded(items.length);
+            setActive(0);
+            setStatus('done');
+          }
+        } catch {
+          if (!controller.signal.aborted) {
+            setStatus('done');
+            setChoices([]);
+            setLoaded(0);
+          }
         }
+      };
+      load();
+      return () => controller.abort();
+    }, []);
+
+    // Load more when scrolling past end
+    async function loadMore() {
+      if (loadedRef.current >= totalCountRef.current || searchModeRef.current) return;
+      setStatus('loading');
+      try {
+        const newItems = await source('', loadedRef.current, { signal: new AbortController().signal });
+        const updated = choicesRef.current.concat(newItems);
+        choicesRef.current = updated;
+        loadedRef.current = updated.length;
+        setChoices([...updated]);
+        setLoaded(updated.length);
+      } catch (err) {
+        console.error('Scroll load error:', err.message || err);
+        loadedRef.current = totalCountRef.current;
+        setLoaded(totalCountRef.current);
       }
-    };
-    load();
-    return () => controller.abort();
-  }, []);
-
-  // Load more when scrolling past end
-  async function loadMore() {
-    if (loadedRef.current >= totalCountRef.current || searchModeRef.current) return;
-    setStatus('loading');
-    try {
-      const newItems = await source('', loadedRef.current, { signal: new AbortController().signal });
-      const updated = choicesRef.current.concat(newItems);
-      choicesRef.current = updated;
-      loadedRef.current = updated.length;
-      setChoices([...updated]);
-      setLoaded(updated.length);
-    } catch (err) {
-      console.error('Scroll load error:', err.message || err);
-      loadedRef.current = totalCountRef.current;
-      setLoaded(totalCountRef.current);
+      setStatus('done');
     }
-    setStatus('done');
-  }
-
-  // Search when term changes
-  useEffect(() => {
-    if (!searchTerm) {
-      setSearchMode(false);
-      searchModeRef.current = false;
+
+    // Search when term changes
+    useEffect(() => {
+      if (!searchTerm) {
+        setSearchMode(false);
+        searchModeRef.current = false;
+        const controller = new AbortController();
+        setStatus('loading');
+        source('', 0, { signal: controller.signal }).then(items => {
+          if (!controller.signal.aborted) {
+            choicesRef.current = items;
+            loadedRef.current = Math.min(items.length, totalCount || Infinity);
+            activeRef.current = 0;
+            setChoices(items);
+            setLoaded(items.length);
+            setActive(0);
+            setStatus('done');
+          }
+        }).catch(() => {
+          if (!controller.signal.aborted) setStatus('done');
+        });
+        return () => controller.abort();
+      }
+
       const controller = new AbortController();
       setStatus('loading');
-      source('', 0, { signal: controller.signal }).then(items => {
+      setSearchMode(true);
+      searchModeRef.current = true;
+
+      source(searchTerm, undefined, { signal: controller.signal }).then(items => {
         if (!controller.signal.aborted) {
           choicesRef.current = items;
-          loadedRef.current = Math.min(items.length, totalCount || Infinity);
+          loadedRef.current = items.length;
           activeRef.current = 0;
           setChoices(items);
           setLoaded(items.length);
@@ -92,103 +113,83 @@ export const paginatedSearch = createPrompt((config, done) => {
           setStatus('done');
         }
       }).catch(() => {
-        if (!controller.signal.aborted) setStatus('done');
+        if (!controller.signal.aborted) {
+          setStatus('done');
+          setChoices([]);
+        }
       });
-      return () => controller.abort();
-    }
-
-    const controller = new AbortController();
-    setStatus('loading');
-    setSearchMode(true);
-    searchModeRef.current = true;
-
-    source(searchTerm, undefined, { signal: controller.signal }).then(items => {
-      if (!controller.signal.aborted) {
-        choicesRef.current = items;
-        loadedRef.current = items.length;
-        activeRef.current = 0;
-        setChoices(items);
-        setLoaded(items.length);
-        setActive(0);
-        setStatus('done');
-      }
-    }).catch(() => {
-      if (!controller.signal.aborted) {
-        setStatus('done');
-        setChoices([]);
-      }
-    });
 
-    return () => controller.abort();
-  }, [searchTerm]);
+      return () => controller.abort();
+    }, [searchTerm]);
 
-  // Key handling
-  useKeypress((key, rl) => {
-    if (status === 'loading') return;
+    // Key handling
+    useKeypress((key, rl) => {
+      if (status === 'loading') return;
 
-    if (isEnterKey(key)) {
-      if (choicesRef.current.length > 0 && activeRef.current >= 0 && activeRef.current < choicesRef.current.length) {
-        done(choicesRef.current[activeRef.current]);
+      if (isEnterKey(key)) {
+        if (choicesRef.current.length > 0 && activeRef.current >= 0 && activeRef.current < choicesRef.current.length) {
+          done(choicesRef.current[activeRef.current]);
+        }
+        return;
       }
-      return;
-    }
 
-    if (key.name === 'escape') {
-      done(undefined);
-      return;
-    }
+      if (key.name === 'escape') {
+        done(undefined);
+        return;
+      }
 
-    if (isUpKey(key)) {
-      if (activeRef.current > 0) {
-        activeRef.current--;
-        setActive(activeRef.current);
+      if (isUpKey(key)) {
+        if (activeRef.current > 0) {
+          activeRef.current--;
+          setActive(activeRef.current);
+        }
+        return;
       }
-      return;
-    }
 
-    if (isDownKey(key)) {
-      if (activeRef.current < choicesRef.current.length - 1) {
-        activeRef.current++;
-        setActive(activeRef.current);
-      } else if (!searchModeRef.current && loadedRef.current < totalCountRef.current) {
-        loadMore().then(() => {
+      if (isDownKey(key)) {
+        if (activeRef.current < choicesRef.current.length - 1) {
           activeRef.current++;
           setActive(activeRef.current);
-        });
+        } else if (!searchModeRef.current && loadedRef.current < totalCountRef.current) {
+          loadMore().then(() => {
+            activeRef.current++;
+            setActive(activeRef.current);
+          });
+        }
+        return;
       }
-      return;
-    }
 
-    // Use readline's built-in line buffer for text input
-    const newTerm = rl.line;
-    if (newTerm !== searchTerm) {
-      setSearchTerm(newTerm);
-    }
-  });
-
-  // Paginated rendering
-  const page = usePagination({
-    items: Array.isArray(choices) ? choices : [],
-    active,
-    pageSize,
-    renderItem: ({ item, isActive }) => {
-      const prefix = isActive ? '\x1b[7m' : ' ';
-      const suffix = isActive ? '\x1b[0m' : '';
-      const name = item.name || String(item.value || item);
-      const display = name.length > 70 ? name.slice(0, 67) + '...' : name;
-      return ` ${prefix}${display}${suffix}`;
-    },
-    loop: false,
-  });
-
-  const helpText = loaded < totalCount && !searchMode
-    ? '(scroll to load more, type to search, Esc to cancel)'
-    : '(type to search, Esc to cancel)';
-
-  return [
-    `${fullMessage} Search: ${searchTerm}${status === 'loading' ? ' ...' : ''}`,
-    `${page}\n\n${helpText}`,
-  ];
-});
+      // Use readline's built-in line buffer for text input
+      const newTerm = rl.line;
+      if (newTerm !== searchTerm) {
+        setSearchTerm(newTerm);
+      }
+    });
+
+    // Paginated rendering
+    const page = usePagination({
+      items: Array.isArray(choices) ? choices : [],
+      active,
+      pageSize,
+      renderItem: ({ item, isActive }) => {
+        const prefix = isActive ? '\x1b[7m' : ' ';
+        const suffix = isActive ? '\x1b[0m' : '';
+        const name = item.name || String(item.value || item);
+        const display = name.length > 70 ? name.slice(0, 67) + '...' : name;
+        return ` ${prefix}${display}${suffix}`;
+      },
+      loop: false,
+    });
+
+    const helpText = loaded < totalCount && !searchMode
+      ? '(scroll to load more, type to search, Esc to cancel)'
+      : '(type to search, Esc to cancel)';
+
+    return [
+      `${fullMessage} Search: ${searchTerm}${status === 'loading' ? ' ...' : ''}`,
+      `${page}\n\n${helpText}`,
+    ];
+  })(...args);
+};
 
 export default paginatedSearch;

package/src/prompts.js

@@ -1,5 +1,3 @@
-import * as prompts from '@inquirer/prompts';
-
 export class PromptCancelled extends Error {
   constructor() {
     super('Cancelled.');
@@ -30,14 +28,22 @@ export function cancellablePrompt(prompt) {
   };
 }
 
-export const input = cancellablePrompt(prompts.input);
-export const password = cancellablePrompt(prompts.password);
-export const select = cancellablePrompt(prompts.select);
-export const search = cancellablePrompt(prompts.search);
-export const confirm = cancellablePrompt(prompts.confirm);
-export const checkbox = cancellablePrompt(prompts.checkbox);
+export const input = lazyPrompt('input');
+export const password = lazyPrompt('password');
+export const select = lazyPrompt('select');
+export const search = lazyPrompt('search');
+export const confirm = lazyPrompt('confirm');
+export const checkbox = lazyPrompt('checkbox');
 
 // The editor prompt hands the keyboard to $EDITOR, so Escape belongs to that
 // editor. It lives here so every command imports prompts from one module;
 // cancelling an editor still uses Inquirer's Ctrl+C abort.
-export const editor = prompts.editor;
+export const editor = async (...args) => (await import('@inquirer/prompts')).editor(...args);
+
+// Load Inquirer only when a prompt is actually shown.
+function lazyPrompt(name) {
+  return async (...args) => {
+    const prompts = await import('@inquirer/prompts');
+    return cancellablePrompt(prompts[name])(...args);
+  };
+}