@n8n/eslint-plugin-community-nodes 0.11.0 → 0.13.0

package/dist/rules/require-community-node-keyword.js

@@ -0,0 +1,65 @@
+import { AST_NODE_TYPES } from '@typescript-eslint/utils';
+import { createRule, findJsonProperty } from '../utils/index.js';
+const REQUIRED_KEYWORD = 'n8n-community-node-package';
+export const RequireCommunityNodeKeywordRule = createRule({
+    name: 'require-community-node-keyword',
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Require the "n8n-community-node-package" keyword in community node package.json',
+        },
+        fixable: 'code',
+        messages: {
+            missingKeyword: `The "keywords" array must include "${REQUIRED_KEYWORD}". This keyword is required for n8n to discover community node packages.`,
+            missingKeywordsArray: `The package.json must have a "keywords" array containing "${REQUIRED_KEYWORD}".`,
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        if (!context.filename.endsWith('package.json')) {
+            return {};
+        }
+        return {
+            ObjectExpression(node) {
+                if (node.parent?.type !== AST_NODE_TYPES.ExpressionStatement) {
+                    return;
+                }
+                const keywordsProp = findJsonProperty(node, 'keywords');
+                if (!keywordsProp) {
+                    context.report({
+                        node,
+                        messageId: 'missingKeywordsArray',
+                        fix(fixer) {
+                            const lastProp = node.properties[node.properties.length - 1];
+                            if (!lastProp) {
+                                return fixer.replaceText(node, `{ "keywords": ["${REQUIRED_KEYWORD}"] }`);
+                            }
+                            return fixer.insertTextAfter(lastProp, `, "keywords": ["${REQUIRED_KEYWORD}"]`);
+                        },
+                    });
+                    return;
+                }
+                if (keywordsProp.value.type !== AST_NODE_TYPES.ArrayExpression) {
+                    return;
+                }
+                const keywordsArray = keywordsProp.value;
+                const hasRequiredKeyword = keywordsArray.elements.some((element) => element?.type === AST_NODE_TYPES.Literal && element.value === REQUIRED_KEYWORD);
+                if (!hasRequiredKeyword) {
+                    context.report({
+                        node: keywordsProp,
+                        messageId: 'missingKeyword',
+                        fix(fixer) {
+                            const lastElement = keywordsArray.elements[keywordsArray.elements.length - 1];
+                            if (!lastElement) {
+                                return fixer.replaceText(keywordsArray, `["${REQUIRED_KEYWORD}"]`);
+                            }
+                            return fixer.insertTextAfter(lastElement, `, "${REQUIRED_KEYWORD}"`);
+                        },
+                    });
+                }
+            },
+        };
+    },
+});
+//# sourceMappingURL=require-community-node-keyword.js.map

package/dist/rules/require-community-node-keyword.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-community-node-keyword.js","sourceRoot":"","sources":["../../src/rules/require-community-node-keyword.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAE1D,OAAO,EAAE,UAAU,EAAE,gBAAgB,EAAE,MAAM,mBAAmB,CAAC;AAEjE,MAAM,gBAAgB,GAAG,4BAA4B,CAAC;AAEtD,MAAM,CAAC,MAAM,+BAA+B,GAAG,UAAU,CAAC;IACzD,IAAI,EAAE,gCAAgC;IACtC,IAAI,EAAE;QACL,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACL,WAAW,EACV,iFAAiF;SAClF;QACD,OAAO,EAAE,MAAM;QACf,QAAQ,EAAE;YACT,cAAc,EAAE,sCAAsC,gBAAgB,0EAA0E;YAChJ,oBAAoB,EAAE,6DAA6D,gBAAgB,IAAI;SACvG;QACD,MAAM,EAAE,EAAE;KACV;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACb,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAC,cAAc,CAAC,EAAE,CAAC;YAChD,OAAO,EAAE,CAAC;QACX,CAAC;QAED,OAAO;YACN,gBAAgB,CAAC,IAA+B;gBAC/C,IAAI,IAAI,CAAC,MAAM,EAAE,IAAI,KAAK,cAAc,CAAC,mBAAmB,EAAE,CAAC;oBAC9D,OAAO;gBACR,CAAC;gBAED,MAAM,YAAY,GAAG,gBAAgB,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;gBAExD,IAAI,CAAC,YAAY,EAAE,CAAC;oBACnB,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI;wBACJ,SAAS,EAAE,sBAAsB;wBACjC,GAAG,CAAC,KAAK;4BACR,MAAM,QAAQ,GAAG,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,UAAU,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;4BAC7D,IAAI,CAAC,QAAQ,EAAE,CAAC;gCACf,OAAO,KAAK,CAAC,WAAW,CAAC,IAAI,EAAE,mBAAmB,gBAAgB,MAAM,CAAC,CAAC;4BAC3E,CAAC;4BACD,OAAO,KAAK,CAAC,eAAe,CAAC,QAAQ,EAAE,mBAAmB,gBAAgB,IAAI,CAAC,CAAC;wBACjF,CAAC;qBACD,CAAC,CAAC;oBACH,OAAO;gBACR,CAAC;gBAED,IAAI,YAAY,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,eAAe,EAAE,CAAC;oBAChE,OAAO;gBACR,CAAC;gBAED,MAAM,aAAa,GAAG,YAAY,CAAC,KAAK,CAAC;gBACzC,MAAM,kBAAkB,GAAG,aAAa,CAAC,QAAQ,CAAC,IAAI,CACrD,CAAC,OAAO,EAAE,EAAE,CACX,OAAO,EAAE,IAAI,KAAK,cAAc,CAAC,OAAO,IAAI,OAAO,CAAC,KAAK,KAAK,gBAAgB,CAC/E,CAAC;gBAEF,IAAI,CAAC,kBAAkB,EAAE,CAAC;oBACzB,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,YAAY;wBAClB,SAAS,EAAE,gBAAgB;wBAC3B,GAAG,CAAC,KAAK;4BACR,MAAM,WAAW,GAAG,aAAa,CAAC,QAAQ,CAAC,aAAa,CAAC,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;4BAC9E,IAAI,CAAC,WAAW,EAAE,CAAC;gCAClB,OAAO,KAAK,CAAC,WAAW,CAAC,aAAa,EAAE,KAAK,gBAAgB,IAAI,CAAC,CAAC;4BACpE,CAAC;4BACD,OAAO,KAAK,CAAC,eAAe,CAAC,WAAW,EAAE,MAAM,gBAAgB,GAAG,CAAC,CAAC;wBACtE,CAAC;qBACD,CAAC,CAAC;gBACJ,CAAC;YACF,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/dist/rules/require-node-description-fields.d.ts

@@ -0,0 +1,2 @@
+export declare const RequireNodeDescriptionFieldsRule: import("@typescript-eslint/utils/ts-eslint").RuleModule<"missingField" | "missingFields", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
+//# sourceMappingURL=require-node-description-fields.d.ts.map

package/dist/rules/require-node-description-fields.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-node-description-fields.d.ts","sourceRoot":"","sources":["../../src/rules/require-node-description-fields.ts"],"names":[],"mappings":"AAiBA,eAAO,MAAM,gCAAgC,mKA2D3C,CAAC"}

package/dist/rules/require-node-description-fields.js

@@ -0,0 +1,58 @@
+import { TSESTree } from '@typescript-eslint/utils';
+import { isNodeTypeClass, findClassProperty, findObjectProperty, isFileType, createRule, } from '../utils/index.js';
+// Fields that are optional in the TypeScript interface but required by
+// community node review standards. TypeScript-required fields (displayName,
+// name, group, version, description, etc.) are already caught by tsc.
+const REQUIRED_FIELDS = ['icon', 'subtitle'];
+export const RequireNodeDescriptionFieldsRule = createRule({
+    name: 'require-node-description-fields',
+    meta: {
+        type: 'problem',
+        docs: {
+            description: `Node class description must define all required fields: ${REQUIRED_FIELDS.join(', ')}`,
+        },
+        messages: {
+            missingField: 'Node class description is missing required `{{field}}` property',
+            missingFields: 'Node class description is missing required properties: {{fields}}',
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        if (!isFileType(context.filename, '.node.ts')) {
+            return {};
+        }
+        return {
+            ClassDeclaration(node) {
+                if (!isNodeTypeClass(node)) {
+                    return;
+                }
+                const descriptionProperty = findClassProperty(node, 'description');
+                if (!descriptionProperty?.value ||
+                    descriptionProperty.value.type !== TSESTree.AST_NODE_TYPES.ObjectExpression) {
+                    return;
+                }
+                const descriptionValue = descriptionProperty.value;
+                const missingFields = REQUIRED_FIELDS.filter((field) => !findObjectProperty(descriptionValue, field));
+                if (missingFields.length === 0) {
+                    return;
+                }
+                if (missingFields.length === 1) {
+                    context.report({
+                        node: descriptionProperty,
+                        messageId: 'missingField',
+                        data: { field: missingFields[0] },
+                    });
+                }
+                else {
+                    context.report({
+                        node: descriptionProperty,
+                        messageId: 'missingFields',
+                        data: { fields: missingFields.map((f) => `\`${f}\``).join(', ') },
+                    });
+                }
+            },
+        };
+    },
+});
+//# sourceMappingURL=require-node-description-fields.js.map

package/dist/rules/require-node-description-fields.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-node-description-fields.js","sourceRoot":"","sources":["../../src/rules/require-node-description-fields.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,0BAA0B,CAAC;AAEpD,OAAO,EACN,eAAe,EACf,iBAAiB,EACjB,kBAAkB,EAClB,UAAU,EACV,UAAU,GACV,MAAM,mBAAmB,CAAC;AAE3B,uEAAuE;AACvE,4EAA4E;AAC5E,sEAAsE;AACtE,MAAM,eAAe,GAAG,CAAC,MAAM,EAAE,UAAU,CAAU,CAAC;AAItD,MAAM,CAAC,MAAM,gCAAgC,GAAG,UAAU,CAAC;IAC1D,IAAI,EAAE,iCAAiC;IACvC,IAAI,EAAE;QACL,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACL,WAAW,EAAE,2DAA2D,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE;SACpG;QACD,QAAQ,EAAE;YACT,YAAY,EAAE,iEAAiE;YAC/E,aAAa,EAAE,mEAAmE;SAClF;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,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,IACC,CAAC,mBAAmB,EAAE,KAAK;oBAC3B,mBAAmB,CAAC,KAAK,CAAC,IAAI,KAAK,QAAQ,CAAC,cAAc,CAAC,gBAAgB,EAC1E,CAAC;oBACF,OAAO;gBACR,CAAC;gBAED,MAAM,gBAAgB,GAAG,mBAAmB,CAAC,KAAK,CAAC;gBAEnD,MAAM,aAAa,GAAoB,eAAe,CAAC,MAAM,CAC5D,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,kBAAkB,CAAC,gBAAgB,EAAE,KAAK,CAAC,CACvD,CAAC;gBAEF,IAAI,aAAa,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;oBAChC,OAAO;gBACR,CAAC;gBAED,IAAI,aAAa,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;oBAChC,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,mBAAmB;wBACzB,SAAS,EAAE,cAAc;wBACzB,IAAI,EAAE,EAAE,KAAK,EAAE,aAAa,CAAC,CAAC,CAAC,EAAE;qBACjC,CAAC,CAAC;gBACJ,CAAC;qBAAM,CAAC;oBACP,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,mBAAmB;wBACzB,SAAS,EAAE,eAAe;wBAC1B,IAAI,EAAE,EAAE,MAAM,EAAE,aAAa,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE;qBACjE,CAAC,CAAC;gBACJ,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-class-description-icon-missing.md

@@ -1,6 +1,6 @@
 # Node class description must have an `icon` property defined (`@n8n/community-nodes/node-class-description-icon-missing`)
 
-💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+❌ This rule is **deprecated**. Use [`require-node-description-fields`](require-node-description-fields.md) instead.
 
 💡 This rule is manually fixable by [editor suggestions](https://eslint.org/docs/latest/use/core-concepts#rule-suggestions).
 

package/docs/rules/require-community-node-keyword.md

@@ -0,0 +1,45 @@
+# Require the "n8n-community-node-package" keyword in package.json (`@n8n/community-nodes/require-community-node-keyword`)
+
+⚠️ This rule is set to `warn` in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+🔧 This rule is automatically fixable by the [`--fix` CLI option](https://eslint.org/docs/latest/use/command-line-interface#--fix).
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+Validates that the `package.json` of a community node package includes `"n8n-community-node-package"` in its `keywords` array. This keyword is required for n8n to discover and list community node packages.
+
+## Examples
+
+### ❌ Incorrect
+
+```json
+{
+  "name": "n8n-nodes-my-service",
+  "version": "1.0.0"
+}
+```
+
+```json
+{
+  "name": "n8n-nodes-my-service",
+  "keywords": ["n8n", "automation"]
+}
+```
+
+### ✅ Correct
+
+```json
+{
+  "name": "n8n-nodes-my-service",
+  "keywords": ["n8n-community-node-package"]
+}
+```
+
+```json
+{
+  "name": "n8n-nodes-my-service",
+  "keywords": ["n8n", "automation", "n8n-community-node-package"]
+}
+```

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/docs/rules/require-node-description-fields.md

@@ -0,0 +1,46 @@
+# Node class description must define all required fields (`@n8n/community-nodes/require-node-description-fields`)
+
+💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+Enforces that community node class descriptions include fields that are optional in the TypeScript interface but required by community node review standards.
+
+This rule supersedes `node-class-description-icon-missing`, which only checked for the `icon` field.
+
+## Examples
+
+### ❌ Incorrect
+
+```typescript
+export class MyNode implements INodeType {
+  description: INodeTypeDescription = {
+    displayName: 'My Node',
+    name: 'myNode',
+    icon: 'file:myNode.svg',
+    group: ['transform'],
+    version: 1,
+    description: 'My node description',
+    // Missing subtitle
+  };
+}
+```
+
+### ✅ Correct
+
+```typescript
+export class MyNode implements INodeType {
+  description: INodeTypeDescription = {
+    displayName: 'My Node',
+    name: 'myNode',
+    icon: 'file:myNode.svg',
+    group: ['transform'],
+    version: 1,
+    description: 'My node description',
+    subtitle: '={{$parameter["operation"]}}',
+    // ...
+  };
+}
+```

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@n8n/eslint-plugin-community-nodes",
   "type": "module",
-  "version": "0.11.0",
+  "version": "0.13.0",
   "main": "./dist/plugin.js",
   "types": "./dist/plugin.d.ts",
   "exports": {
@@ -23,8 +23,8 @@
     "typescript": "6.0.2",
     "vitest": "^4.1.1",
     "@n8n/typescript-config": "1.4.0",
-    "@n8n/vitest-config": "1.9.0",
-    "n8n-workflow": "2.16.0"
+    "n8n-workflow": "2.18.0",
+    "@n8n/vitest-config": "1.9.0"
   },
   "peerDependencies": {
     "eslint": ">= 9",

package/src/plugin.ts

@@ -29,14 +29,18 @@ 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-community-node-keyword': 'warn',
+			'@n8n/community-nodes/require-continue-on-fail': 'error',
+			'@n8n/community-nodes/require-node-description-fields': 'error',
 		},
 	},
 	recommendedWithoutN8nCloudSupport: {
@@ -52,14 +56,18 @@ 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-community-node-keyword': 'warn',
+			'@n8n/community-nodes/require-continue-on-fail': 'error',
+			'@n8n/community-nodes/require-node-description-fields': 'error',
 		},
 	},
 } satisfies Record<string, Linter.Config>;

package/src/rules/index.ts

@@ -6,8 +6,10 @@ import { CredentialDocumentationUrlRule } from './credential-documentation-url.j
 import { CredentialPasswordFieldRule } from './credential-password-field.js';
 import { CredentialTestRequiredRule } from './credential-test-required.js';
 import { IconValidationRule } from './icon-validation.js';
+import { MissingPairedItemRule } from './missing-paired-item.js';
 import { NoCredentialReuseRule } from './no-credential-reuse.js';
 import { NoDeprecatedWorkflowFunctionsRule } from './no-deprecated-workflow-functions.js';
+import { NoForbiddenLifecycleScriptsRule } from './no-forbidden-lifecycle-scripts.js';
 import { NoHttpRequestWithManualAuthRule } from './no-http-request-with-manual-auth.js';
 import { NoRestrictedGlobalsRule } from './no-restricted-globals.js';
 import { NoRestrictedImportsRule } from './no-restricted-imports.js';
@@ -16,6 +18,9 @@ import { NodeConnectionTypeLiteralRule } from './node-connection-type-literal.js
 import { NodeUsableAsToolRule } from './node-usable-as-tool.js';
 import { OptionsSortedAlphabeticallyRule } from './options-sorted-alphabetically.js';
 import { PackageNameConventionRule } from './package-name-convention.js';
+import { RequireCommunityNodeKeywordRule } from './require-community-node-keyword.js';
+import { RequireContinueOnFailRule } from './require-continue-on-fail.js';
+import { RequireNodeDescriptionFieldsRule } from './require-node-description-fields.js';
 import { ResourceOperationPatternRule } from './resource-operation-pattern.js';
 
 export const rules = {
@@ -29,6 +34,7 @@ export const rules = {
 	'package-name-convention': PackageNameConventionRule,
 	'credential-test-required': CredentialTestRequiredRule,
 	'no-credential-reuse': NoCredentialReuseRule,
+	'no-forbidden-lifecycle-scripts': NoForbiddenLifecycleScriptsRule,
 	'no-http-request-with-manual-auth': NoHttpRequestWithManualAuthRule,
 	'icon-validation': IconValidationRule,
 	'resource-operation-pattern': ResourceOperationPatternRule,
@@ -36,4 +42,8 @@ export const rules = {
 	'node-class-description-icon-missing': NodeClassDescriptionIconMissingRule,
 	'cred-class-field-icon-missing': CredClassFieldIconMissingRule,
 	'node-connection-type-literal': NodeConnectionTypeLiteralRule,
+	'missing-paired-item': MissingPairedItemRule,
+	'require-community-node-keyword': RequireCommunityNodeKeywordRule,
+	'require-continue-on-fail': RequireContinueOnFailRule,
+	'require-node-description-fields': RequireNodeDescriptionFieldsRule,
 } satisfies Record<string, AnyRuleModule>;