@n8n/eslint-plugin-community-nodes 0.10.0 → 0.12.0

package/dist/rules/node-connection-type-literal.js

@@ -0,0 +1,77 @@
+import { AST_NODE_TYPES } from '@typescript-eslint/utils';
+import { createRequire } from 'node:module';
+import { isNodeTypeClass, findClassProperty, findObjectProperty, createRule, } from '../utils/index.js';
+// n8n-workflow's ESM dist uses bare module specifiers that Node's native ESM
+// loader cannot resolve. Loading via CJS (createRequire) sidesteps this.
+const { NodeConnectionTypes } = createRequire(import.meta.url)('n8n-workflow');
+// Reverse map: string value (e.g. 'main') → enum key name (e.g. 'Main').
+// Derived directly from NodeConnectionTypes so it stays in sync automatically.
+const VALUE_TO_ENUM_KEY = Object.fromEntries(Object.entries(NodeConnectionTypes).map(([key, value]) => [value, key]));
+export const NodeConnectionTypeLiteralRule = createRule({
+    name: 'node-connection-type-literal',
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Disallow string literals in node description `inputs`/`outputs` — use `NodeConnectionTypes` enum instead',
+        },
+        messages: {
+            stringLiteralInInputs: 'Use NodeConnectionTypes.{{enumKey}} from "n8n-workflow" instead of the string literal "{{value}}" in "inputs".',
+            stringLiteralInOutputs: 'Use NodeConnectionTypes.{{enumKey}} from "n8n-workflow" instead of the string literal "{{value}}" in "outputs".',
+            unknownStringLiteralInInputs: 'Use the NodeConnectionTypes enum from "n8n-workflow" instead of the string literal "{{value}}" in "inputs".',
+            unknownStringLiteralInOutputs: 'Use the NodeConnectionTypes enum from "n8n-workflow" instead of the string literal "{{value}}" in "outputs".',
+        },
+        fixable: 'code',
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        function checkArrayElements(elements, property) {
+            for (const element of elements) {
+                if (element?.type !== AST_NODE_TYPES.Literal)
+                    continue;
+                if (typeof element.value !== 'string')
+                    continue;
+                const value = element.value;
+                const enumKey = VALUE_TO_ENUM_KEY[value];
+                if (enumKey) {
+                    context.report({
+                        node: element,
+                        messageId: property === 'inputs' ? 'stringLiteralInInputs' : 'stringLiteralInOutputs',
+                        data: { value, enumKey },
+                        fix(fixer) {
+                            return fixer.replaceText(element, `NodeConnectionTypes.${enumKey}`);
+                        },
+                    });
+                }
+                else {
+                    context.report({
+                        node: element,
+                        messageId: property === 'inputs'
+                            ? 'unknownStringLiteralInInputs'
+                            : 'unknownStringLiteralInOutputs',
+                        data: { value },
+                    });
+                }
+            }
+        }
+        return {
+            ClassDeclaration(node) {
+                if (!isNodeTypeClass(node))
+                    return;
+                const descriptionProperty = findClassProperty(node, 'description');
+                if (!descriptionProperty)
+                    return;
+                const descriptionValue = descriptionProperty.value;
+                if (descriptionValue?.type !== AST_NODE_TYPES.ObjectExpression)
+                    return;
+                for (const prop of ['inputs', 'outputs']) {
+                    const property = findObjectProperty(descriptionValue, prop);
+                    if (property?.value.type !== AST_NODE_TYPES.ArrayExpression)
+                        continue;
+                    checkArrayElements(property.value.elements, prop);
+                }
+            },
+        };
+    },
+});
+//# sourceMappingURL=node-connection-type-literal.js.map

package/dist/rules/node-connection-type-literal.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"node-connection-type-literal.js","sourceRoot":"","sources":["../../src/rules/node-connection-type-literal.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAE1D,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAE5C,OAAO,EACN,eAAe,EACf,iBAAiB,EACjB,kBAAkB,EAClB,UAAU,GACV,MAAM,mBAAmB,CAAC;AAE3B,6EAA6E;AAC7E,yEAAyE;AACzE,MAAM,EAAE,mBAAmB,EAAE,GAAG,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,cAAc,CAE5E,CAAC;AAEF,yEAAyE;AACzE,+EAA+E;AAC/E,MAAM,iBAAiB,GAA2B,MAAM,CAAC,WAAW,CACnE,MAAM,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,EAAE,KAAK,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC,CACvE,CAAC;AAEF,MAAM,CAAC,MAAM,6BAA6B,GAAG,UAAU,CAAC;IACvD,IAAI,EAAE,8BAA8B;IACpC,IAAI,EAAE;QACL,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACL,WAAW,EACV,0GAA0G;SAC3G;QACD,QAAQ,EAAE;YACT,qBAAqB,EACpB,gHAAgH;YACjH,sBAAsB,EACrB,iHAAiH;YAClH,4BAA4B,EAC3B,6GAA6G;YAC9G,6BAA6B,EAC5B,8GAA8G;SAC/G;QACD,OAAO,EAAE,MAAM;QACf,MAAM,EAAE,EAAE;KACV;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACb,SAAS,kBAAkB,CAC1B,QAA8C,EAC9C,QAA8B;YAE9B,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;gBAChC,IAAI,OAAO,EAAE,IAAI,KAAK,cAAc,CAAC,OAAO;oBAAE,SAAS;gBACvD,IAAI,OAAO,OAAO,CAAC,KAAK,KAAK,QAAQ;oBAAE,SAAS;gBAEhD,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC;gBAC5B,MAAM,OAAO,GAAG,iBAAiB,CAAC,KAAK,CAAC,CAAC;gBAEzC,IAAI,OAAO,EAAE,CAAC;oBACb,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,OAAO;wBACb,SAAS,EAAE,QAAQ,KAAK,QAAQ,CAAC,CAAC,CAAC,uBAAuB,CAAC,CAAC,CAAC,wBAAwB;wBACrF,IAAI,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE;wBACxB,GAAG,CAAC,KAAK;4BACR,OAAO,KAAK,CAAC,WAAW,CAAC,OAAO,EAAE,uBAAuB,OAAO,EAAE,CAAC,CAAC;wBACrE,CAAC;qBACD,CAAC,CAAC;gBACJ,CAAC;qBAAM,CAAC;oBACP,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,OAAO;wBACb,SAAS,EACR,QAAQ,KAAK,QAAQ;4BACpB,CAAC,CAAC,8BAA8B;4BAChC,CAAC,CAAC,+BAA+B;wBACnC,IAAI,EAAE,EAAE,KAAK,EAAE;qBACf,CAAC,CAAC;gBACJ,CAAC;YACF,CAAC;QACF,CAAC;QAED,OAAO;YACN,gBAAgB,CAAC,IAAI;gBACpB,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC;oBAAE,OAAO;gBAEnC,MAAM,mBAAmB,GAAG,iBAAiB,CAAC,IAAI,EAAE,aAAa,CAAC,CAAC;gBACnE,IAAI,CAAC,mBAAmB;oBAAE,OAAO;gBAEjC,MAAM,gBAAgB,GAAG,mBAAmB,CAAC,KAAK,CAAC;gBACnD,IAAI,gBAAgB,EAAE,IAAI,KAAK,cAAc,CAAC,gBAAgB;oBAAE,OAAO;gBAEvE,KAAK,MAAM,IAAI,IAAI,CAAC,QAAQ,EAAE,SAAS,CAAU,EAAE,CAAC;oBACnD,MAAM,QAAQ,GAAG,kBAAkB,CAAC,gBAAgB,EAAE,IAAI,CAAC,CAAC;oBAC5D,IAAI,QAAQ,EAAE,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,eAAe;wBAAE,SAAS;oBACtE,kBAAkB,CAAC,QAAQ,CAAC,KAAK,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC;gBACnD,CAAC;YACF,CAAC;SACD,CAAC;IACH,CAAC;CACD,CAAC,CAAC"}

package/dist/rules/options-sorted-alphabetically.d.ts

@@ -0,0 +1,2 @@
+export declare const OptionsSortedAlphabeticallyRule: import("@typescript-eslint/utils/ts-eslint").RuleModule<"optionsNotSorted", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
+//# sourceMappingURL=options-sorted-alphabetically.d.ts.map

package/dist/rules/options-sorted-alphabetically.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"options-sorted-alphabetically.d.ts","sourceRoot":"","sources":["../../src/rules/options-sorted-alphabetically.ts"],"names":[],"mappings":"AAYA,eAAO,MAAM,+BAA+B,qJAoH1C,CAAC"}

package/dist/rules/options-sorted-alphabetically.js

@@ -0,0 +1,95 @@
+import { AST_NODE_TYPES } from '@typescript-eslint/utils';
+import { isNodeTypeClass, findClassProperty, findObjectProperty, getStringLiteralValue, isFileType, createRule, } from '../utils/index.js';
+export const OptionsSortedAlphabeticallyRule = createRule({
+    name: 'options-sorted-alphabetically',
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Enforce alphabetical ordering of options arrays in n8n node properties',
+        },
+        messages: {
+            optionsNotSorted: 'Options in "{{ displayName }}" are not sorted alphabetically. Expected order: {{ expectedOrder }}.',
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        if (!isFileType(context.filename, '.node.ts')) {
+            return {};
+        }
+        const analyzeNodeDescription = (descriptionValue) => {
+            if (!descriptionValue || descriptionValue.type !== AST_NODE_TYPES.ObjectExpression) {
+                return;
+            }
+            const propertiesProperty = findObjectProperty(descriptionValue, 'properties');
+            if (!propertiesProperty?.value ||
+                propertiesProperty.value.type !== AST_NODE_TYPES.ArrayExpression) {
+                return;
+            }
+            for (const property of propertiesProperty.value.elements) {
+                if (!property || property.type !== AST_NODE_TYPES.ObjectExpression) {
+                    continue;
+                }
+                const typeProperty = findObjectProperty(property, 'type');
+                const type = typeProperty ? getStringLiteralValue(typeProperty.value) : null;
+                if (type !== 'options') {
+                    continue;
+                }
+                const optionsProperty = findObjectProperty(property, 'options');
+                if (!optionsProperty?.value ||
+                    optionsProperty.value.type !== AST_NODE_TYPES.ArrayExpression) {
+                    continue;
+                }
+                const optionsArray = optionsProperty.value;
+                // Extract all option names; skip if any name is non-literal (dynamic options)
+                const names = [];
+                let hasDynamicName = false;
+                for (const option of optionsArray.elements) {
+                    if (!option || option.type !== AST_NODE_TYPES.ObjectExpression) {
+                        hasDynamicName = true;
+                        break;
+                    }
+                    const nameProperty = findObjectProperty(option, 'name');
+                    const name = nameProperty ? getStringLiteralValue(nameProperty.value) : null;
+                    if (name === null) {
+                        hasDynamicName = true;
+                        break;
+                    }
+                    names.push(name);
+                }
+                if (hasDynamicName || names.length <= 1) {
+                    continue;
+                }
+                const sorted = [...names].sort((a, b) => a.localeCompare(b, undefined, { sensitivity: 'base' }));
+                const isSorted = names.every((name, i) => name === sorted[i]);
+                if (isSorted) {
+                    continue;
+                }
+                const displayNameProperty = findObjectProperty(property, 'displayName');
+                const displayName = (displayNameProperty ? getStringLiteralValue(displayNameProperty.value) : null) ??
+                    'unknown';
+                context.report({
+                    node: optionsArray,
+                    messageId: 'optionsNotSorted',
+                    data: {
+                        displayName,
+                        expectedOrder: sorted.join(', '),
+                    },
+                });
+            }
+        };
+        return {
+            ClassDeclaration(node) {
+                if (!isNodeTypeClass(node)) {
+                    return;
+                }
+                const descriptionProperty = findClassProperty(node, 'description');
+                if (!descriptionProperty) {
+                    return;
+                }
+                analyzeNodeDescription(descriptionProperty.value);
+            },
+        };
+    },
+});
+//# sourceMappingURL=options-sorted-alphabetically.js.map

package/dist/rules/options-sorted-alphabetically.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"options-sorted-alphabetically.js","sourceRoot":"","sources":["../../src/rules/options-sorted-alphabetically.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAE1D,OAAO,EACN,eAAe,EACf,iBAAiB,EACjB,kBAAkB,EAClB,qBAAqB,EACrB,UAAU,EACV,UAAU,GACV,MAAM,mBAAmB,CAAC;AAE3B,MAAM,CAAC,MAAM,+BAA+B,GAAG,UAAU,CAAC;IACzD,IAAI,EAAE,+BAA+B;IACrC,IAAI,EAAE;QACL,IAAI,EAAE,YAAY;QAClB,IAAI,EAAE;YACL,WAAW,EAAE,wEAAwE;SACrF;QACD,QAAQ,EAAE;YACT,gBAAgB,EACf,oGAAoG;SACrG;QACD,MAAM,EAAE,EAAE;KACV;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACb,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,QAAQ,EAAE,UAAU,CAAC,EAAE,CAAC;YAC/C,OAAO,EAAE,CAAC;QACX,CAAC;QAED,MAAM,sBAAsB,GAAG,CAAC,gBAA4C,EAAQ,EAAE;YACrF,IAAI,CAAC,gBAAgB,IAAI,gBAAgB,CAAC,IAAI,KAAK,cAAc,CAAC,gBAAgB,EAAE,CAAC;gBACpF,OAAO;YACR,CAAC;YAED,MAAM,kBAAkB,GAAG,kBAAkB,CAAC,gBAAgB,EAAE,YAAY,CAAC,CAAC;YAC9E,IACC,CAAC,kBAAkB,EAAE,KAAK;gBAC1B,kBAAkB,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,eAAe,EAC/D,CAAC;gBACF,OAAO;YACR,CAAC;YAED,KAAK,MAAM,QAAQ,IAAI,kBAAkB,CAAC,KAAK,CAAC,QAAQ,EAAE,CAAC;gBAC1D,IAAI,CAAC,QAAQ,IAAI,QAAQ,CAAC,IAAI,KAAK,cAAc,CAAC,gBAAgB,EAAE,CAAC;oBACpE,SAAS;gBACV,CAAC;gBAED,MAAM,YAAY,GAAG,kBAAkB,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;gBAC1D,MAAM,IAAI,GAAG,YAAY,CAAC,CAAC,CAAC,qBAAqB,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;gBAE7E,IAAI,IAAI,KAAK,SAAS,EAAE,CAAC;oBACxB,SAAS;gBACV,CAAC;gBAED,MAAM,eAAe,GAAG,kBAAkB,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAC;gBAChE,IACC,CAAC,eAAe,EAAE,KAAK;oBACvB,eAAe,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,eAAe,EAC5D,CAAC;oBACF,SAAS;gBACV,CAAC;gBAED,MAAM,YAAY,GAAG,eAAe,CAAC,KAAK,CAAC;gBAE3C,8EAA8E;gBAC9E,MAAM,KAAK,GAAa,EAAE,CAAC;gBAC3B,IAAI,cAAc,GAAG,KAAK,CAAC;gBAE3B,KAAK,MAAM,MAAM,IAAI,YAAY,CAAC,QAAQ,EAAE,CAAC;oBAC5C,IAAI,CAAC,MAAM,IAAI,MAAM,CAAC,IAAI,KAAK,cAAc,CAAC,gBAAgB,EAAE,CAAC;wBAChE,cAAc,GAAG,IAAI,CAAC;wBACtB,MAAM;oBACP,CAAC;oBACD,MAAM,YAAY,GAAG,kBAAkB,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;oBACxD,MAAM,IAAI,GAAG,YAAY,CAAC,CAAC,CAAC,qBAAqB,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;oBAC7E,IAAI,IAAI,KAAK,IAAI,EAAE,CAAC;wBACnB,cAAc,GAAG,IAAI,CAAC;wBACtB,MAAM;oBACP,CAAC;oBACD,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBAClB,CAAC;gBAED,IAAI,cAAc,IAAI,KAAK,CAAC,MAAM,IAAI,CAAC,EAAE,CAAC;oBACzC,SAAS;gBACV,CAAC;gBAED,MAAM,MAAM,GAAG,CAAC,GAAG,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CACvC,CAAC,CAAC,aAAa,CAAC,CAAC,EAAE,SAAS,EAAE,EAAE,WAAW,EAAE,MAAM,EAAE,CAAC,CACtD,CAAC;gBAEF,MAAM,QAAQ,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC,IAAI,EAAE,CAAC,EAAE,EAAE,CAAC,IAAI,KAAK,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC;gBAC9D,IAAI,QAAQ,EAAE,CAAC;oBACd,SAAS;gBACV,CAAC;gBAED,MAAM,mBAAmB,GAAG,kBAAkB,CAAC,QAAQ,EAAE,aAAa,CAAC,CAAC;gBACxE,MAAM,WAAW,GAChB,CAAC,mBAAmB,CAAC,CAAC,CAAC,qBAAqB,CAAC,mBAAmB,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;oBAC/E,SAAS,CAAC;gBAEX,OAAO,CAAC,MAAM,CAAC;oBACd,IAAI,EAAE,YAAY;oBAClB,SAAS,EAAE,kBAAkB;oBAC7B,IAAI,EAAE;wBACL,WAAW;wBACX,aAAa,EAAE,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC;qBAChC;iBACD,CAAC,CAAC;YACJ,CAAC;QACF,CAAC,CAAC;QAEF,OAAO;YACN,gBAAgB,CAAC,IAAI;gBACpB,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC5B,OAAO;gBACR,CAAC;gBAED,MAAM,mBAAmB,GAAG,iBAAiB,CAAC,IAAI,EAAE,aAAa,CAAC,CAAC;gBACnE,IAAI,CAAC,mBAAmB,EAAE,CAAC;oBAC1B,OAAO;gBACR,CAAC;gBAED,sBAAsB,CAAC,mBAAmB,CAAC,KAAK,CAAC,CAAC;YACnD,CAAC;SACD,CAAC;IACH,CAAC;CACD,CAAC,CAAC"}

package/dist/rules/require-continue-on-fail.d.ts

@@ -0,0 +1,2 @@
+export declare const RequireContinueOnFailRule: import("@typescript-eslint/utils/ts-eslint").RuleModule<"missingContinueOnFail", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
+//# sourceMappingURL=require-continue-on-fail.d.ts.map

package/dist/rules/require-continue-on-fail.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-continue-on-fail.d.ts","sourceRoot":"","sources":["../../src/rules/require-continue-on-fail.ts"],"names":[],"mappings":"AA8CA,eAAO,MAAM,yBAAyB,0JAyCpC,CAAC"}

package/dist/rules/require-continue-on-fail.js

@@ -0,0 +1,76 @@
+import { AST_NODE_TYPES } from '@typescript-eslint/utils';
+import { isNodeTypeClass } from '../utils/index.js';
+import { createRule } from '../utils/rule-creator.js';
+/** Keys that are not child AST nodes (back-references, metadata). */
+const NON_CHILD_KEYS = new Set(['parent', 'loc', 'range', 'start', 'end', 'tokens', 'comments']);
+/**
+ * Recursively checks whether any descendant of the given AST node is a
+ * `this.continueOnFail()` call expression.
+ */
+function containsContinueOnFailCall(node) {
+    if (node.type === AST_NODE_TYPES.CallExpression &&
+        node.callee.type === AST_NODE_TYPES.MemberExpression &&
+        node.callee.object.type === AST_NODE_TYPES.ThisExpression &&
+        node.callee.property.type === AST_NODE_TYPES.Identifier &&
+        node.callee.property.name === 'continueOnFail') {
+        return true;
+    }
+    for (const key of Object.keys(node)) {
+        if (NON_CHILD_KEYS.has(key))
+            continue;
+        const value = node[key];
+        if (Array.isArray(value)) {
+            for (const child of value) {
+                if (child && typeof child === 'object' && 'type' in child) {
+                    if (containsContinueOnFailCall(child)) {
+                        return true;
+                    }
+                }
+            }
+        }
+        else if (value && typeof value === 'object' && 'type' in value) {
+            if (containsContinueOnFailCall(value)) {
+                return true;
+            }
+        }
+    }
+    return false;
+}
+export const RequireContinueOnFailRule = createRule({
+    name: 'require-continue-on-fail',
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Require continueOnFail() handling in execute() methods of node classes',
+        },
+        messages: {
+            missingContinueOnFail: 'execute() method must handle this.continueOnFail() for proper error handling. ' +
+                'Wrap item processing in a try/catch and check this.continueOnFail() in the catch block.',
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        return {
+            ClassDeclaration(node) {
+                if (!isNodeTypeClass(node)) {
+                    return;
+                }
+                for (const member of node.body.body) {
+                    if (member.type !== AST_NODE_TYPES.MethodDefinition ||
+                        member.key.type !== AST_NODE_TYPES.Identifier ||
+                        member.key.name !== 'execute') {
+                        continue;
+                    }
+                    if (member.value.body && !containsContinueOnFailCall(member.value.body)) {
+                        context.report({
+                            node: member.key,
+                            messageId: 'missingContinueOnFail',
+                        });
+                    }
+                }
+            },
+        };
+    },
+});
+//# sourceMappingURL=require-continue-on-fail.js.map

package/dist/rules/require-continue-on-fail.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-continue-on-fail.js","sourceRoot":"","sources":["../../src/rules/require-continue-on-fail.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAiB,MAAM,0BAA0B,CAAC;AAEzE,OAAO,EAAE,eAAe,EAAE,MAAM,mBAAmB,CAAC;AACpD,OAAO,EAAE,UAAU,EAAE,MAAM,0BAA0B,CAAC;AAEtD,qEAAqE;AACrE,MAAM,cAAc,GAAG,IAAI,GAAG,CAAC,CAAC,QAAQ,EAAE,KAAK,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,QAAQ,EAAE,UAAU,CAAC,CAAC,CAAC;AAEjG;;;GAGG;AACH,SAAS,0BAA0B,CAAC,IAAmB;IACtD,IACC,IAAI,CAAC,IAAI,KAAK,cAAc,CAAC,cAAc;QAC3C,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,cAAc,CAAC,gBAAgB;QACpD,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,cAAc,CAAC,cAAc;QACzD,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,cAAc,CAAC,UAAU;QACvD,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,KAAK,gBAAgB,EAC7C,CAAC;QACF,OAAO,IAAI,CAAC;IACb,CAAC;IAED,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACrC,IAAI,cAAc,CAAC,GAAG,CAAC,GAAG,CAAC;YAAE,SAAS;QAEtC,MAAM,KAAK,GAAI,IAA2C,CAAC,GAAG,CAAC,CAAC;QAEhE,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;YAC1B,KAAK,MAAM,KAAK,IAAI,KAAK,EAAE,CAAC;gBAC3B,IAAI,KAAK,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,MAAM,IAAI,KAAK,EAAE,CAAC;oBAC3D,IAAI,0BAA0B,CAAC,KAAsB,CAAC,EAAE,CAAC;wBACxD,OAAO,IAAI,CAAC;oBACb,CAAC;gBACF,CAAC;YACF,CAAC;QACF,CAAC;aAAM,IAAI,KAAK,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,MAAM,IAAI,KAAK,EAAE,CAAC;YAClE,IAAI,0BAA0B,CAAC,KAAsB,CAAC,EAAE,CAAC;gBACxD,OAAO,IAAI,CAAC;YACb,CAAC;QACF,CAAC;IACF,CAAC;IAED,OAAO,KAAK,CAAC;AACd,CAAC;AAED,MAAM,CAAC,MAAM,yBAAyB,GAAG,UAAU,CAAC;IACnD,IAAI,EAAE,0BAA0B;IAChC,IAAI,EAAE;QACL,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACL,WAAW,EAAE,wEAAwE;SACrF;QACD,QAAQ,EAAE;YACT,qBAAqB,EACpB,gFAAgF;gBAChF,yFAAyF;SAC1F;QACD,MAAM,EAAE,EAAE;KACV;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACb,OAAO;YACN,gBAAgB,CAAC,IAAI;gBACpB,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC5B,OAAO;gBACR,CAAC;gBAED,KAAK,MAAM,MAAM,IAAI,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC;oBACrC,IACC,MAAM,CAAC,IAAI,KAAK,cAAc,CAAC,gBAAgB;wBAC/C,MAAM,CAAC,GAAG,CAAC,IAAI,KAAK,cAAc,CAAC,UAAU;wBAC7C,MAAM,CAAC,GAAG,CAAC,IAAI,KAAK,SAAS,EAC5B,CAAC;wBACF,SAAS;oBACV,CAAC;oBAED,IAAI,MAAM,CAAC,KAAK,CAAC,IAAI,IAAI,CAAC,0BAA0B,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;wBACzE,OAAO,CAAC,MAAM,CAAC;4BACd,IAAI,EAAE,MAAM,CAAC,GAAG;4BAChB,SAAS,EAAE,uBAAuB;yBAClC,CAAC,CAAC;oBACJ,CAAC;gBACF,CAAC;YACF,CAAC;SACD,CAAC;IACH,CAAC;CACD,CAAC,CAAC"}

package/docs/rules/missing-paired-item.md

@@ -0,0 +1,70 @@
+# Require pairedItem on INodeExecutionData objects in execute() methods to preserve item linking (`@n8n/community-nodes/missing-paired-item`)
+
+💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+Every `INodeExecutionData` object returned from `execute()` should include a `pairedItem` property. Without it, downstream nodes cannot trace data lineage and expressions like `$('NodeName').item` will silently fail.
+
+The rule detects object literals with a `json` property but no `pairedItem` inside `execute()` methods of `INodeType` classes.
+
+## Examples
+
+### Incorrect
+
+```typescript
+class MyNode implements INodeType {
+  async execute() {
+    const items = this.getInputData();
+    // Missing pairedItem
+    return [items.map((item) => ({ json: item.json }))];
+  }
+}
+```
+
+```typescript
+class MyNode implements INodeType {
+  async execute() {
+    const returnData: INodeExecutionData[] = [];
+    // Missing pairedItem
+    returnData.push({ json: { result: true } });
+    return [returnData];
+  }
+}
+```
+
+### Correct
+
+```typescript
+class MyNode implements INodeType {
+  async execute() {
+    const items = this.getInputData();
+    return [items.map((item, index) => ({ json: item.json, pairedItem: { item: index } }))];
+  }
+}
+```
+
+```typescript
+class MyNode implements INodeType {
+  async execute() {
+    const returnData: INodeExecutionData[] = [];
+    returnData.push({ json: { result: true }, pairedItem: { item: 0 } });
+    return [returnData];
+  }
+}
+```
+
+## When to Disable
+
+If your node intentionally does not support item linking (e.g. it aggregates all input items into a single output), you can suppress this rule:
+
+```typescript
+// eslint-disable-next-line @n8n/community-nodes/missing-paired-item
+returnData.push({ json: aggregatedResult });
+```
+
+## Further Reading
+
+- [n8n Paired Items Documentation](https://docs.n8n.io/integrations/creating-nodes/build/reference/paired-items/)

package/docs/rules/no-forbidden-lifecycle-scripts.md

@@ -0,0 +1,46 @@
+# Ban lifecycle scripts (prepare, preinstall, postinstall, etc.) in community node packages (`@n8n/community-nodes/no-forbidden-lifecycle-scripts`)
+
+💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+npm lifecycle scripts (`prepare`, `preinstall`, `install`, `postinstall`, `prepublish`, `preprepare`, `postprepare`) run automatically — without user confirmation — during `npm install`. In the context of n8n community nodes, this means arbitrary code executes on the n8n instance the moment a community node is installed.
+
+n8n community nodes are distributed as pre-built npm packages. Unlike regular npm libraries, there is no legitimate reason for a community node to hook into install-time lifecycle events — the package should already contain compiled code ready to use. A `prepare` or `postinstall` script in a community node is either a misconfiguration (the author forgot to remove a build step meant for development) or a supply-chain attack vector.
+
+## Examples
+
+### Incorrect
+
+```json
+{
+  "name": "n8n-nodes-example",
+  "scripts": {
+    "prepare": "npm run build"
+  }
+}
+```
+
+```json
+{
+  "name": "n8n-nodes-example",
+  "scripts": {
+    "build": "tsc",
+    "postinstall": "node setup.js"
+  }
+}
+```
+
+### Correct
+
+```json
+{
+  "name": "n8n-nodes-example",
+  "scripts": {
+    "build": "tsc",
+    "test": "jest"
+  }
+}
+```

package/docs/rules/node-connection-type-literal.md

@@ -0,0 +1,46 @@
+# Disallow string literals in node description `inputs`/`outputs` — use `NodeConnectionTypes` enum instead (`@n8n/community-nodes/node-connection-type-literal`)
+
+💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+🔧 This rule is automatically fixable by the [`--fix` CLI option](https://eslint.org/docs/latest/user-guide/command-line-interface#--fix).
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+Using raw string literals like `'main'` in `inputs` and `outputs` is fragile: the values are not type-checked, and typos or renamed connection types will go undetected. The `NodeConnectionTypes` object from `n8n-workflow` is the single source of truth and should always be used instead.
+
+## Examples
+
+### ❌ Incorrect
+
+```typescript
+import type { INodeType, INodeTypeDescription } from 'n8n-workflow';
+
+export class MyNode implements INodeType {
+  description: INodeTypeDescription = {
+    displayName: 'My Node',
+    name: 'myNode',
+    inputs: ['main'],
+    outputs: ['main'],
+    properties: [],
+  };
+}
+```
+
+### ✅ Correct
+
+```typescript
+import type { INodeType, INodeTypeDescription } from 'n8n-workflow';
+import { NodeConnectionTypes } from 'n8n-workflow';
+
+export class MyNode implements INodeType {
+  description: INodeTypeDescription = {
+    displayName: 'My Node',
+    name: 'myNode',
+    inputs: [NodeConnectionTypes.Main],
+    outputs: [NodeConnectionTypes.Main],
+    properties: [],
+  };
+}
+```

package/docs/rules/options-sorted-alphabetically.md

@@ -0,0 +1,63 @@
+# Enforce alphabetical ordering of options arrays in n8n node properties (`@n8n/community-nodes/options-sorted-alphabetically`)
+
+⚠️ This rule _warns_ in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+Warns when an `options`-type parameter has its options array not sorted alphabetically by name. Applies to all `type: 'options'` parameters — including `resource`, `operation`, and any other dropdowns.
+
+Alphabetical ordering is an [official n8n UI design requirement](https://docs.n8n.io/integrations/creating-nodes/plan/node-ui-design/#lists) and the most frequently flagged issue in community node reviews.
+
+Comparison is case-insensitive and locale-aware (handles non-ASCII names such as Spanish or Portuguese labels).
+
+## Examples
+
+### ❌ Incorrect
+
+```typescript
+export class MyNode implements INodeType {
+  description: INodeTypeDescription = {
+    displayName: 'My Service',
+    name: 'myService',
+    properties: [
+      {
+        displayName: 'Resource',
+        name: 'resource',
+        type: 'options',
+        options: [
+          { name: 'User', value: 'user' },
+          { name: 'Contact', value: 'contact' }, // out of order
+          { name: 'Project', value: 'project' },
+        ],
+        default: 'user',
+      },
+    ],
+  };
+}
+```
+
+### ✅ Correct
+
+```typescript
+export class MyNode implements INodeType {
+  description: INodeTypeDescription = {
+    displayName: 'My Service',
+    name: 'myService',
+    properties: [
+      {
+        displayName: 'Resource',
+        name: 'resource',
+        type: 'options',
+        options: [
+          { name: 'Contact', value: 'contact' },
+          { name: 'Project', value: 'project' },
+          { name: 'User', value: 'user' },
+        ],
+        default: 'contact',
+      },
+    ],
+  };
+}
+```

package/docs/rules/require-continue-on-fail.md

@@ -0,0 +1,56 @@
+# Require continueOnFail() handling in execute() methods of node classes (`@n8n/community-nodes/require-continue-on-fail`)
+
+💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+Ensures that `execute()` methods in node classes include a `this.continueOnFail()` check. Without this, a single item error will abort the entire workflow instead of allowing execution to continue past the failing item.
+
+## Examples
+
+### ❌ Incorrect
+
+```typescript
+export class MyNode implements INodeType {
+  description: INodeTypeDescription = { /* ... */ };
+
+  async execute(this: IExecuteFunctions): Promise<INodeExecutionData[][]> {
+    const items = this.getInputData();
+    const returnData: INodeExecutionData[] = [];
+    for (let i = 0; i < items.length; i++) {
+      // No error handling — one bad item kills the whole workflow
+      const result = await someApiCall(items[i]);
+      returnData.push({ json: result });
+    }
+    return [returnData];
+  }
+}
+```
+
+### ✅ Correct
+
+```typescript
+export class MyNode implements INodeType {
+  description: INodeTypeDescription = { /* ... */ };
+
+  async execute(this: IExecuteFunctions): Promise<INodeExecutionData[][]> {
+    const items = this.getInputData();
+    const returnData: INodeExecutionData[] = [];
+    for (let i = 0; i < items.length; i++) {
+      try {
+        const result = await someApiCall(items[i]);
+        returnData.push({ json: result });
+      } catch (error) {
+        if (this.continueOnFail()) {
+          returnData.push({ json: { error: error.message }, pairedItem: { item: i } });
+          continue;
+        }
+        throw error;
+      }
+    }
+    return [returnData];
+  }
+}
+```

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@n8n/eslint-plugin-community-nodes",
   "type": "module",
-  "version": "0.10.0",
+  "version": "0.12.0",
   "main": "./dist/plugin.js",
   "types": "./dist/plugin.d.ts",
   "exports": {
@@ -15,17 +15,20 @@
     "fastest-levenshtein": "1.0.16"
   },
   "devDependencies": {
+    "@types/node": "24.10.1",
     "@typescript-eslint/rule-tester": "^8.35.0",
     "eslint-doc-generator": "^2.2.2",
     "eslint-plugin-eslint-plugin": "^7.0.0",
     "rimraf": "6.0.1",
-    "typescript": "5.9.2",
-    "vitest": "^3.1.3",
-    "@n8n/typescript-config": "1.3.0",
-    "@n8n/vitest-config": "1.7.0"
+    "typescript": "6.0.2",
+    "vitest": "^4.1.1",
+    "@n8n/typescript-config": "1.4.0",
+    "@n8n/vitest-config": "1.9.0",
+    "n8n-workflow": "2.17.0"
   },
   "peerDependencies": {
-    "eslint": ">= 9"
+    "eslint": ">= 9",
+    "n8n-workflow": ">=2"
   },
   "eslint-doc-generator": {
     "configEmoji": [

package/src/plugin.ts

@@ -29,12 +29,17 @@ const configs = {
 			'@n8n/community-nodes/package-name-convention': 'error',
 			'@n8n/community-nodes/credential-test-required': 'error',
 			'@n8n/community-nodes/no-credential-reuse': 'error',
+			'@n8n/community-nodes/no-forbidden-lifecycle-scripts': 'error',
 			'@n8n/community-nodes/no-http-request-with-manual-auth': 'error',
 			'@n8n/community-nodes/icon-validation': 'error',
+			'@n8n/community-nodes/options-sorted-alphabetically': 'warn',
 			'@n8n/community-nodes/resource-operation-pattern': 'warn',
 			'@n8n/community-nodes/credential-documentation-url': 'error',
 			'@n8n/community-nodes/node-class-description-icon-missing': 'error',
 			'@n8n/community-nodes/cred-class-field-icon-missing': 'error',
+			'@n8n/community-nodes/node-connection-type-literal': 'error',
+			'@n8n/community-nodes/missing-paired-item': 'error',
+			'@n8n/community-nodes/require-continue-on-fail': 'error',
 		},
 	},
 	recommendedWithoutN8nCloudSupport: {
@@ -50,12 +55,17 @@ const configs = {
 			'@n8n/community-nodes/package-name-convention': 'error',
 			'@n8n/community-nodes/credential-test-required': 'error',
 			'@n8n/community-nodes/no-credential-reuse': 'error',
+			'@n8n/community-nodes/no-forbidden-lifecycle-scripts': 'error',
 			'@n8n/community-nodes/no-http-request-with-manual-auth': 'error',
 			'@n8n/community-nodes/icon-validation': 'error',
+			'@n8n/community-nodes/options-sorted-alphabetically': 'warn',
 			'@n8n/community-nodes/credential-documentation-url': 'error',
 			'@n8n/community-nodes/resource-operation-pattern': 'warn',
 			'@n8n/community-nodes/node-class-description-icon-missing': 'error',
 			'@n8n/community-nodes/cred-class-field-icon-missing': 'error',
+			'@n8n/community-nodes/node-connection-type-literal': 'error',
+			'@n8n/community-nodes/missing-paired-item': 'error',
+			'@n8n/community-nodes/require-continue-on-fail': 'error',
 		},
 	},
 } satisfies Record<string, Linter.Config>;