@tbrandenburg/node-red-agents 0.4.0 → 0.4.1

package/nodes/agent/agent.js

@@ -715,7 +715,21 @@ module.exports = function (RED) {
                       "",
                     );
                   });
-                  return substituteInputs(raw, inputsMap);
+                  // issue #29: warn once per distinct unmatched $INPUTS.<name>
+                  // token found in this run (de-duplicated so a typo'd token
+                  // repeated in `arguments` doesn't spam the log) -- purely
+                  // additive observability, the substituted text itself (and
+                  // thus the eventual invocation) is unchanged.
+                  const unmatchedNames = new Set();
+                  const substituted = substituteInputs(raw, inputsMap, (name) =>
+                    unmatchedNames.add(name),
+                  );
+                  unmatchedNames.forEach((name) => {
+                    node.warn(
+                      `$INPUTS.${name} has no matching 'inputs' entry and was left unsubstituted`,
+                    );
+                  });
+                  return substituted;
                 })()
               : undefined,
           cwd: (() => {

package/nodes/agent/lib/execution/inputs.js

@@ -7,15 +7,26 @@
 // tokens are left as literal text rather than throwing, so a typo in a
 // flow's arguments string degrades to visible-but-harmless output instead
 // of a hard failure.
+//
+// issue #29: this module has no access to the Node-RED `node` object (and
+// unit tests call it directly without a fake node), so it never warns
+// itself -- callers that do have a `node` may pass an optional
+// onUnmatched(name) callback, invoked for every token left as literal text
+// (missing from inputsMap, or present but mapped to null/undefined -- both
+// produce the same visible symptom of a literal token reaching the LLM).
 const TOKEN_RE = /\$INPUTS\.([A-Za-z0-9_]+)/g;
 
-function substituteInputs(text, inputsMap) {
+function substituteInputs(text, inputsMap, onUnmatched) {
   if (typeof text !== "string" || !text) return text;
   const map = inputsMap || {};
   return text.replace(TOKEN_RE, (match, name) => {
-    if (!Object.prototype.hasOwnProperty.call(map, name)) return match;
-    const value = map[name];
-    return value === undefined || value === null ? match : String(value);
+    const hasName = Object.prototype.hasOwnProperty.call(map, name);
+    const value = hasName ? map[name] : undefined;
+    if (!hasName || value === undefined || value === null) {
+      if (typeof onUnmatched === "function") onUnmatched(name);
+      return match;
+    }
+    return String(value);
   });
 }
 

package/package.json

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