@jacebenson/jsn 4.3.10 → 4.3.11-dev.69

package/src/commands/setup.js

@@ -1,4 +1,3 @@
-import { createPrompt, useKeypress, usePagination, useState, isDownKey, isEnterKey, isUpKey } from '@inquirer/core';
 import { loginWizard, loginProfile, showAuthStatus, switchProfile, modifyProfile, logoutProfile, removeConnection } from './auth.js';
 import { errUsage } from '../errors.js';
 import { declareCapabilities } from '../capabilities.js';
@@ -18,32 +17,35 @@ const HUB_CHOICES = [
 ];
 
 /** The setup hub prompt; q and Escape resolve with no selected action. */
-export const authHubPrompt = createPrompt((config, done) => {
-  const [active, setActive] = useState(0);
-  const choices = config.choices;
+export const authHubPrompt = async (...args) => {
+  const { createPrompt, useKeypress, usePagination, useState, isDownKey, isEnterKey, isUpKey } = await import('@inquirer/core');
+  return createPrompt((config, done) => {
+    const [active, setActive] = useState(0);
+    const choices = config.choices;
 
-  useKeypress((key) => {
-    if (key.name === 'q' || key.name === 'escape') {
-      done(undefined);
-      return;
-    }
-    if (isEnterKey(key)) {
-      done(choices[active].value);
-      return;
-    }
-    if (isUpKey(key) && active > 0) setActive(active - 1);
-    if (isDownKey(key) && active < choices.length - 1) setActive(active + 1);
-  });
+    useKeypress((key) => {
+      if (key.name === 'q' || key.name === 'escape') {
+        done(undefined);
+        return;
+      }
+      if (isEnterKey(key)) {
+        done(choices[active].value);
+        return;
+      }
+      if (isUpKey(key) && active > 0) setActive(active - 1);
+      if (isDownKey(key) && active < choices.length - 1) setActive(active + 1);
+    });
 
-  const page = usePagination({
-    items: choices,
-    active,
-    pageSize: choices.length,
-    loop: false,
-    renderItem: ({ item, isActive }) => `${isActive ? '❯' : ' '} ${item.name}`,
-  });
-  return `${config.message}\n${page}\n\n↑↓ navigate • ⏎ select • q/Esc exit`;
-});
+    const page = usePagination({
+      items: choices,
+      active,
+      pageSize: choices.length,
+      loop: false,
+      renderItem: ({ item, isActive }) => `${isActive ? '❯' : ' '} ${item.name}`,
+    });
+    return `${config.message}\n${page}\n\n↑↓ navigate • ⏎ select • q/Esc exit`;
+  })(...args);
+};
 
 /** Interactive hub: what do you want to do? Returns an action or undefined on exit. */
 export async function authHubMenu({ promptFn = authHubPrompt, input, output } = {}) {

package/src/commands/skill.js

@@ -333,7 +333,7 @@ export function skillCmd(wrap) {
                 ? `Where should the jsn skill be installed? (${outdated.length} outdated)`
                 : 'Where should the jsn skill be installed?';
 
-              const { checkbox } = await import('@inquirer/prompts');
+              const { checkbox } = await import('../prompts.js');
               const choices = Object.entries(TARGET_NAMES).map(([key, label]) => {
                 const dir = AGENT_SKILL_DIRS[key];
                 const targetPath = key === 'cursor'

package/src/commands/skipped-updates.js

@@ -1,4 +1,4 @@
-import { checkbox, select, confirm } from '@inquirer/prompts';
+import { checkbox, select, confirm } from '../prompts.js';
 import { canUsePicker, interactiveList, getStringField } from '../helpers.js';
 import { declareCapabilities } from '../capabilities.js';
 import { errUsage } from '../errors.js';

package/src/commands/updatesets.js

@@ -1,5 +1,5 @@
 import fs from 'node:fs';
-import { editor, confirm } from '@inquirer/prompts';
+import { editor, confirm } from '../prompts.js';
 import { formatRecordForDisplay, getStringField, interactiveList, assertSafeExactMatch, canPrompt, buildRerunCommand, verifyWriteBack } from '../helpers.js';
 import { getActiveProfile } from '../config.js';
 import { FormatAuto } from '../output.js';

package/src/commands/version.js

@@ -19,14 +19,28 @@ export function getVersion() {
   }
 }
 
-export async function checkLatest() {
+const backgroundChecks = new Set();
+
+export function cancelBackgroundChecks() {
+  for (const controller of backgroundChecks) controller.abort();
+  backgroundChecks.clear();
+}
+
+export async function checkLatest({ timeout = 1500, background = false } = {}) {
+  const controller = new AbortController();
+  const timer = setTimeout(() => controller.abort(), timeout);
+  timer.unref();
+  if (background) backgroundChecks.add(controller);
   try {
-    const res = await fetch(NPM_REGISTRY);
+    const res = await fetch(NPM_REGISTRY, { signal: controller.signal });
     if (!res.ok) return null;
     const data = await res.json();
     return data.version;
   } catch {
     return null;
+  } finally {
+    clearTimeout(timer);
+    backgroundChecks.delete(controller);
   }
 }
 

package/src/flow-designer-interactive.js

@@ -1,7 +1,7 @@
 // Terminal orchestration only. Persistence and lifecycle remain in the SDK.
 import fs from 'node:fs';
 import { randomUUID, randomBytes } from 'node:crypto';
-import { input, select, confirm, editor } from '@inquirer/prompts';
+import { input, select, confirm, editor } from './prompts.js';
 import { canPrompt, interactiveList, getStringField } from './helpers.js';
 import { FormatAuto } from './output.js';
 import { errUsage, errConfirmationRequired } from './errors.js';
@@ -35,7 +35,7 @@ export function interactiveDesigner(argv, app) {
 export async function cancelled(operation) {
   try { return await operation(); }
   catch (error) {
-    if (error === CANCEL || ['ExitPromptError', 'AbortPromptError'].includes(error.name)) return undefined;
+    if (error === CANCEL || ['ExitPromptError', 'AbortPromptError', 'PromptCancelled'].includes(error.name)) return undefined;
     // Keep direct handler errors throwable; opt into the shared JSON renderer at the CLI boundary.
     if (error.code === 'usage_error') error.jsonEnvelope = true;
     throw error;

package/src/frontmatter.js

@@ -1,4 +1,5 @@
-import yaml from 'js-yaml';
+// js-yaml 5 ships ESM-only named exports; there is no default export.
+import { load as loadYaml, dump as dumpYaml } from 'js-yaml';
 
 const DEFAULT_DELIMITERS = ['---', '---'];
 
@@ -27,7 +28,8 @@ export function parseFrontmatter(input, options = {}) {
 
   const closeIndex = frontmatter.indexOf(`\n${close}`);
   const raw = closeIndex === -1 ? frontmatter : frontmatter.slice(0, closeIndex);
-  const parsed = yaml.load(raw);
+  // js-yaml 5 throws on an empty document; an empty block means no metadata.
+  const parsed = raw.trim() === '' ? {} : loadYaml(raw);
   const data = parsed === undefined ? {} : parsed;
 
   if (closeIndex === -1) return { data, content: '' };
@@ -41,7 +43,7 @@ export function parseFrontmatter(input, options = {}) {
 /** Serialize docs metadata and body using gray-matter-compatible YAML framing. */
 export function stringifyFrontmatter(body, data, options = {}) {
   const [open, close] = delimiters(options);
-  const yamlText = yaml.dump(data, options).trim();
+  const yamlText = dumpYaml(data, options).trim();
   const prefix = yamlText === '{}' ? '' : `${open}\n${yamlText}\n${close}\n`;
   const content = String(body);
   return prefix + (content.endsWith('\n') ? content : `${content}\n`);

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
   }

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)'
-    : '(type to search)';
-
-  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,8 +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 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 = 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);
+  };
+}