eslint-plugin-sfmc 4.4.0 → 4.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "eslint-plugin-sfmc",
-    "version": "4.4.0",
+    "version": "4.5.0",
     "description": "ESLint plugin for Salesforce Marketing Cloud Engagement+Next - AMPscript, Server-Side JavaScript (SSJS) and Handlebars",
     "type": "module",
     "main": "src/index.js",
@@ -42,7 +42,7 @@
         "eslint-plugin-mso-email": "^2.0.0",
         "handlebars-data": "^0.3.0",
         "mso-conditional-parser": "^2.0.1",
-        "ssjs-data": "^0.19.0"
+        "ssjs-data": "^0.21.0"
     },
     "peerDependencies": {
         "eslint": ">=9.0.0"

package/src/rules/ssjs/no-nonexistent-global.js

@@ -2,13 +2,17 @@
  * Rule: ssjs-no-nonexistent-global
  *
  * Flags bare-name SSJS globals that are officially documented but proven NOT to
- * exist at runtime — calling them throws a ReferenceError. Currently covers:
+ * exist at runtime as callable identifiers — calling them throws a ReferenceError.
  *
- *   - Redirect(url, movedPermanently) — undefined under every Core version;
- *     use Platform.Response.Redirect(url, movedPermanently) instead.
+ * The offending names come from ssjs-data's `notDefinedAtRuntime` flag, so phantom
+ * globals are picked up automatically without editing this rule. Only bare-name
+ * callable phantoms (Identifier callees) are flagged here; object-namespace
+ * phantoms accessed via member calls (e.g. `Recipient.GetAttributeValue(...)`) are
+ * out of scope for this rule.
  *
- * The offending names come from ssjs-data's `notDefinedAtRuntime` flag, so new
- * phantom globals are picked up automatically without editing this rule.
+ * Note: many bare-name Core globals (Write, Stringify, Base64Encode, Format,
+ * Redirect, …) are NOT phantom — they exist after `Platform.Load("core")` when
+ * called in the same scope as the load, so they are intentionally not flagged.
  */
 
 import { notDefinedAtRuntimeGlobalLookup } from 'ssjs-data';

package/src/rules/ssjs/platform-function-arity.js

@@ -7,6 +7,20 @@
 
 import { platformFunctionLookup } from 'ssjs-data';
 
+/**
+ * Render a set of permitted argument counts as human-readable text,
+ * e.g. `[1, 6]` → "1 or 6" and `[1, 3, 6]` → "1, 3 or 6".
+ *
+ * @param {number[]} arities - The permitted argument counts.
+ * @returns {string} The rendered list.
+ */
+function formatArities(arities) {
+    if (arities.length === 1) {
+        return String(arities[0]);
+    }
+    return `${arities.slice(0, -1).join(', ')} or ${arities.at(-1)}`;
+}
+
 export default {
     meta: {
         type: 'problem',
@@ -18,6 +32,8 @@ export default {
                 "'Platform.Function.{{name}}' requires at least {{min}} argument(s) but was called with {{actual}}.",
             tooManyArgs:
                 "'Platform.Function.{{name}}' accepts at most {{max}} argument(s) but was called with {{actual}}.",
+            invalidArity:
+                "'Platform.Function.{{name}}' must be called with exactly {{arities}} arguments (got {{actual}}); intermediate argument counts throw at runtime.",
         },
         schema: [],
     },
@@ -66,6 +82,22 @@ export default {
                                 actual: String(actual),
                             },
                         });
+                    } else if (
+                        Array.isArray(entry.validArities) &&
+                        !entry.validArities.includes(actual)
+                    ) {
+                        // Discontinuous overload: actual is within [minArgs, maxArgs]
+                        // but not one of the exact permitted arities (e.g. HTTPGet
+                        // accepts only 1 or 6 arguments; 2-5 throw at runtime).
+                        context.report({
+                            node: callee.property,
+                            messageId: 'invalidArity',
+                            data: {
+                                name: entry.name,
+                                arities: formatArities(entry.validArities),
+                                actual: String(actual),
+                            },
+                        });
                     }
                 }
             },