@jacebenson/jsn 3.0.3 → 3.0.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jacebenson/jsn",
-  "version": "3.0.3",
+  "version": "3.0.5",
   "description": "A command-line interface for ServiceNow that follows the Unix philosophy: simple, composable, and scriptable.",
   "type": "module",
   "bin": {
@@ -43,4 +43,4 @@
     "README.md",
     "LICENSE"
   ]
-}
+}

package/skills/servicenow/SKILL.md

@@ -10,12 +10,12 @@ description: |
 license: MIT
 compatibility: |
   Requires jsn CLI (github.com/jacebenson/jsn). Install via:
-  curl -fsSL https://jsn.jace.pro/install | bash
+  npm install -g @jacebenson/jsn
   Works with Claude Code, OpenCode, Cursor, and agentskills-compatible tools.
   
-  Windows users: Download from GitHub releases. If jsn is not in your PATH, the setup
-  command will automatically detect its location and show the correct command path
-  (e.g., `.\jsn.exe tables list` instead of `jsn tables list`).
+  Windows users: Install Node.js first, then the npm command above. If jsn is not
+  in your PATH, the setup command will automatically detect its location and show
+  the correct command path.
 metadata:
   author: jacebenson
   version: "1.0.0"

package/src/commands/dev/eval.js

@@ -7,10 +7,19 @@ export function evalCmd(wrap) {
     builder: (yargs) => {
       return yargs
         .option('script', { alias: 's', type: 'string', describe: "JavaScript code to execute (single-quote the script, double quotes inside: --script 'gs.log(\"Hello\")')" })
-        .option('file', { alias: 'f', type: 'string', describe: 'Read script from file (avoids shell quoting issues)' });
+        .option('file', { alias: 'f', type: 'string', describe: 'Read script from file (avoids shell quoting issues)' })
+        .option('scope', { alias: null, type: 'string', describe: 'Scope name or sys_id to run the script under (default: global)' });
     },
     handler: wrap(async (argv, app) => {
       let script;
+      let scopeSysId = null;
+
+      if (argv.scope) {
+        scopeSysId = await app.sdk.resolveScope(argv.scope);
+        if (!scopeSysId) {
+          throw new Error(`Scope not found: ${argv.scope}`);
+        }
+      }
 
       if (argv.file) {
         script = fs.readFileSync(argv.file, 'utf-8');
@@ -20,13 +29,20 @@ export function evalCmd(wrap) {
         throw new Error('--script or --file is required');
       }
 
-      const output = await app.sdk.executeScript(script);
+      if (scopeSysId) {
+        script = `var $scopeSysId = '${scopeSysId}';\n${script}`;
+      }
+
+      const warning = scopeSysId
+        ? ` ⚠️ Records without explicit sys_scope will land in global. Use $scopeSysId variable.`
+        : '';
+      const output = await app.sdk.executeScript(script, argv.scope);
       app.ok({
         script,
         output,
         instance: app.getEffectiveInstance(),
       }, {
-        summary: 'Script executed',
+        summary: 'Script executed' + warning,
         breadcrumbs: [{
           action: 'eval',
           cmd: 'jsn dev eval --script \'...\'',

package/src/commands/skill.js

@@ -8,6 +8,8 @@ const SKILL_REPO_PATH = path.join(
   '..', '..', 'skills', 'servicenow', 'SKILL.md'
 );
 
+const HERMES_SKILL_PATH = path.join(os.homedir(), '.hermes', 'skills', 'servicenow', 'SKILL.md');
+
 const SKILL_RAW_URL = 'https://raw.githubusercontent.com/jacebenson/jsn/nodejs/skills/servicenow/SKILL.md';
 
 function readBundledSkill() {
@@ -18,9 +20,23 @@ function readBundledSkill() {
   }
 }
 
+function readInstalledSkill() {
+  try {
+    return fs.readFileSync(HERMES_SKILL_PATH, 'utf-8');
+  } catch {
+    return null;
+  }
+}
+
 export async function checkSkill() {
-  const bundled = readBundledSkill();
-  if (!bundled) return { current: false, error: 'Skill file not found in package' };
+  // Compare the Hermes-installed copy's version against GitHub (not the npm-bundled copy).
+  // Uses the YAML frontmatter "version" field so publishing jsn doesn't flag it.
+  const installed = readInstalledSkill() || readBundledSkill();
+  if (!installed) return { current: false, error: 'Skill file not found' };
+
+  const installedVersion = extractVersion(installed);
+  if (!installedVersion) return { current: false, error: 'No version field in installed skill' };
+
   try {
     const controller = new AbortController();
     const timer = setTimeout(() => controller.abort(), 10000);
@@ -28,18 +44,27 @@ export async function checkSkill() {
     clearTimeout(timer);
     if (!res.ok) return { current: false, error: `GitHub returned ${res.status}` };
     const upstream = await res.text();
-    const current = bundled === upstream;
+    const upstreamVersion = extractVersion(upstream);
+    if (!upstreamVersion) return { current: false, error: 'No version field in upstream skill' };
+
+    const current = installedVersion === upstreamVersion;
     return {
       current,
-      bundled_lines: bundled.split('\n').length,
-      upstream_lines: upstream.split('\n').length,
-      error: current ? null : 'Bundled skill is outdated — run "jsn skill install" to update',
+      installed_version: installedVersion,
+      upstream_version: upstreamVersion,
+      error: current ? null : `Skill version ${installedVersion} vs GitHub ${upstreamVersion} — run "jsn skill install" to update`,
     };
   } catch {
     return { current: false, error: 'Could not check — GitHub unreachable' };
   }
 }
 
+function extractVersion(content) {
+  // Matches version in YAML frontmatter (top-level or nested under metadata:)
+  const m = content.match(/^\s*version:\s*["']?(.+?)["']?\s*$/m);
+  return m ? m[1] : null;
+}
+
 export function skillCmd(wrap) {
   return {
     command: 'skill',
@@ -70,9 +95,9 @@ export function skillCmd(wrap) {
             if (result.error && !result.current) {
               app.ok(result, { summary: result.error });
             } else if (result.current) {
-              app.ok(result, { summary: `✓ Skill is current (${result.bundled_lines} lines)` });
+              app.ok(result, { summary: `✓ Skill is current (v${result.installed_version})` });
             } else {
-              app.ok(result, { summary: `⚠ Skill is outdated (bundled ${result.bundled_lines} lines vs upstream ${result.upstream_lines} lines) — run "jsn skill install" to update` });
+              app.ok(result, { summary: `⚠ Skill outdated: installed v${result.installed_version} vs GitHub v${result.upstream_version} — run "jsn skill install" to update` });
             }
           }),
         })

package/src/sdk.js

@@ -188,6 +188,25 @@ export class SDKClient {
     };
   }
 
+  /**
+   * Resolve a scope name or sys_id to a sys_id.
+   * @param {string} scope - Scope name (e.g. "sn_notif_dashboard") or sys_id
+   * @returns {Promise<string|null>} The scope sys_id, or null if not found
+   */
+  async resolveScope(scope) {
+    if (!scope) return null;
+    // If it looks like a sys_id (32 hex chars), try it directly
+    if (/^[0-9a-fA-F]{32}$/.test(scope)) return scope;
+
+    const params = new URLSearchParams();
+    params.set('sysparm_query', `scope=${scope}`);
+    params.set('sysparm_limit', '1');
+    params.set('sysparm_fields', 'sys_id,name');
+    const records = await this.list('sys_scope', params);
+    if (records.length === 0) return null;
+    return records[0].sys_id?.value || records[0].sys_id;
+  }
+
   async inspectFlow(identifier) {
     const isSysID = identifier.length === 32 && /^[0-9a-fA-F]+$/.test(identifier);
 
@@ -372,7 +391,7 @@ export class SDKClient {
    * @param {string} script - JavaScript code to execute
    * @returns {Promise<string>} The script's output text
    */
-  async executeScript(script) {
+  async executeScript(script, scope) {
     // Step 1: Warm up the session by hitting any REST API — this makes
     // ServiceNow issue session cookies for subsequent UI page requests.
     // We capture cookies to forward them (Node.js fetch() has no built-in cookie jar).
@@ -387,7 +406,7 @@ export class SDKClient {
     formBody.set('script', script);
     formBody.set('sysparm_ck', csrfToken);
     formBody.set('runscript', 'Run script');
-    formBody.set('sys_scope', 'global');
+    formBody.set('sys_scope', scope || 'global');
     formBody.set('record_for_rollback', 'on');
     formBody.set('quota_managed_transaction', 'on');