@tbrandenburg/node-red-agents 0.1.3 → 0.1.5

package/nodes/gh/lib/parse-args.js

@@ -1,4 +1,4 @@
-'use strict';
+"use strict";
 
 // Splits a string of gh CLI arguments into an argv array without any shell
 // evaluation. Single- and double-quoted spans are kept as one argument
@@ -11,57 +11,57 @@
 //   'list --label "needs review"'        -> ['list', '--label', 'needs review']
 //   'pr list && rm -rf /'                -> ['pr', 'list', '&&', 'rm', '-rf', '/']
 function parseArgs(str) {
-    if (str === undefined || str === null) return [];
-    if (typeof str !== 'string') {
-        throw new Error('parseArgs: expected a string, got ' + typeof str);
-    }
-
-    const args = [];
-    let current = '';
-    let inArg = false;
-    let quote = null; // '"' or "'" while inside a quoted span
-
-    for (let i = 0; i < str.length; i += 1) {
-        const ch = str[i];
+  if (str === undefined || str === null) return [];
+  if (typeof str !== "string") {
+    throw new Error("parseArgs: expected a string, got " + typeof str);
+  }
 
-        if (quote) {
-            if (ch === quote) {
-                quote = null;
-            } else {
-                current += ch;
-            }
-            continue;
-        }
+  const args = [];
+  let current = "";
+  let inArg = false;
+  let quote = null; // '"' or "'" while inside a quoted span
 
-        if (ch === '"' || ch === "'") {
-            quote = ch;
-            inArg = true;
-            continue;
-        }
-
-        if (ch === ' ' || ch === '\t' || ch === '\n' || ch === '\r') {
-            if (inArg) {
-                args.push(current);
-                current = '';
-                inArg = false;
-            }
-            continue;
-        }
+  for (let i = 0; i < str.length; i += 1) {
+    const ch = str[i];
 
+    if (quote) {
+      if (ch === quote) {
+        quote = null;
+      } else {
         current += ch;
-        inArg = true;
+      }
+      continue;
     }
 
-    if (quote) {
-        // Unterminated quote: fail safe by treating the rest of the
-        // string as literal content rather than throwing, since this is
-        // config text a user can fix, not something to crash a flow over.
-        args.push(current);
-    } else if (inArg) {
+    if (ch === '"' || ch === "'") {
+      quote = ch;
+      inArg = true;
+      continue;
+    }
+
+    if (ch === " " || ch === "\t" || ch === "\n" || ch === "\r") {
+      if (inArg) {
         args.push(current);
+        current = "";
+        inArg = false;
+      }
+      continue;
     }
 
-    return args;
+    current += ch;
+    inArg = true;
+  }
+
+  if (quote) {
+    // Unterminated quote: fail safe by treating the rest of the
+    // string as literal content rather than throwing, since this is
+    // config text a user can fix, not something to crash a flow over.
+    args.push(current);
+  } else if (inArg) {
+    args.push(current);
+  }
+
+  return args;
 }
 
 module.exports = { parseArgs };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tbrandenburg/node-red-agents",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Node-RED nodes for running coding agents (opencode, pi) and GitHub CLI operations from flows.",
   "keywords": [
     "node-red",

package/shared/srt-settings.js

@@ -1,8 +1,8 @@
-'use strict';
+"use strict";
 
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
+const fs = require("fs");
+const os = require("os");
+const path = require("path");
 
 // Translates the simple "Inline" SRT settings UI (two string lists + one
 // checkbox), used by both the `agent` and `agent-server` nodes, into the
@@ -25,18 +25,18 @@ const path = require('path');
 // "<field>: Required" for each one missing. This function always includes
 // all four.
 function buildSettingsJson({ allowedDomains, allowedWriteDirs, strictAllowlist } = {}) {
-    return {
-        network: {
-            allowedDomains: Array.isArray(allowedDomains) ? allowedDomains : [],
-            deniedDomains: [],
-            strictAllowlist: strictAllowlist !== false
-        },
-        filesystem: {
-            allowWrite: Array.isArray(allowedWriteDirs) ? allowedWriteDirs : [],
-            denyRead: [],
-            denyWrite: []
-        }
-    };
+  return {
+    network: {
+      allowedDomains: Array.isArray(allowedDomains) ? allowedDomains : [],
+      deniedDomains: [],
+      strictAllowlist: strictAllowlist !== false,
+    },
+    filesystem: {
+      allowWrite: Array.isArray(allowedWriteDirs) ? allowedWriteDirs : [],
+      denyRead: [],
+      denyWrite: [],
+    },
+  };
 }
 
 // config: { allowedDomains, allowedWriteDirs, strictAllowlist, advancedJson }
@@ -45,14 +45,17 @@ function buildSettingsJson({ allowedDomains, allowedWriteDirs, strictAllowlist }
 // writeInlineSettingsFile so tests can check the generated content without
 // needing a real filesystem.
 function resolveSettingsJsonString(config = {}) {
-    const raw = config.advancedJson && config.advancedJson.trim() ? config.advancedJson : JSON.stringify(buildSettingsJson(config));
-    // Throws a clear SyntaxError if the (usually hand-edited advanced) JSON
-    // is invalid -- deliberately no deeper validation than "is this valid
-    // JSON"; srt's own runtime error for a structurally-wrong-but-valid-JSON
-    // settings file is already clear and surfaces through the normal
-    // execution/spawn-failure path.
-    JSON.parse(raw);
-    return raw;
+  const raw =
+    config.advancedJson && config.advancedJson.trim()
+      ? config.advancedJson
+      : JSON.stringify(buildSettingsJson(config));
+  // Throws a clear SyntaxError if the (usually hand-edited advanced) JSON
+  // is invalid -- deliberately no deeper validation than "is this valid
+  // JSON"; srt's own runtime error for a structurally-wrong-but-valid-JSON
+  // settings file is already clear and surfaces through the normal
+  // execution/spawn-failure path.
+  JSON.parse(raw);
+  return raw;
 }
 
 // nodeId is used only to make the temp filename recognizable/stable per
@@ -61,11 +64,11 @@ function resolveSettingsJsonString(config = {}) {
 // filePrefix distinguishes callers (e.g. 'agent' vs 'agent-server') so two
 // different node types configuring the same nodeId-shaped id can never
 // collide on the same temp file.
-function writeInlineSettingsFile(nodeId, config, filePrefix = 'srt-settings') {
-    const json = resolveSettingsJsonString(config);
-    const filePath = path.join(os.tmpdir(), `${filePrefix}-${nodeId}-${process.pid}.json`);
-    fs.writeFileSync(filePath, json);
-    return filePath;
+function writeInlineSettingsFile(nodeId, config, filePrefix = "srt-settings") {
+  const json = resolveSettingsJsonString(config);
+  const filePath = path.join(os.tmpdir(), `${filePrefix}-${nodeId}-${process.pid}.json`);
+  fs.writeFileSync(filePath, json);
+  return filePath;
 }
 
 module.exports = { buildSettingsJson, resolveSettingsJsonString, writeInlineSettingsFile };