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

package/dist/rules/require-node-api-error.js

@@ -0,0 +1,77 @@
+import { DefinitionType } from '@typescript-eslint/scope-manager';
+import { AST_NODE_TYPES } from '@typescript-eslint/utils';
+import { isFileType } from '../utils/index.js';
+import { createRule } from '../utils/rule-creator.js';
+const ALLOWED_ERROR_CLASSES = new Set(['NodeApiError', 'NodeOperationError']);
+function getThrowCalleeName(argument) {
+    if (argument.type === AST_NODE_TYPES.NewExpression) {
+        if (argument.callee.type === AST_NODE_TYPES.Identifier) {
+            return argument.callee.name;
+        }
+    }
+    return null;
+}
+function isInsideCatchClause(node) {
+    let current = node.parent;
+    while (current) {
+        if (current.type === AST_NODE_TYPES.CatchClause) {
+            return true;
+        }
+        current = current.parent;
+    }
+    return false;
+}
+export const RequireNodeApiErrorRule = createRule({
+    name: 'require-node-api-error',
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Require NodeApiError or NodeOperationError for error wrapping in catch blocks. ' +
+                'Raw errors lose HTTP context in the n8n UI.',
+        },
+        messages: {
+            useNodeApiError: 'Use `NodeApiError` or `NodeOperationError` instead of re-throwing raw errors. ' +
+                'Example: `throw new NodeApiError(this.getNode(), error as JsonObject)`',
+            useNodeApiErrorInsteadOfGeneric: 'Use `NodeApiError` or `NodeOperationError` instead of `{{ errorClass }}`. ' +
+                'Example: `throw new NodeApiError(this.getNode(), error as JsonObject)`',
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        const isNodeFile = isFileType(context.filename, '.node.ts');
+        const isHelperFile = context.filename.endsWith('.ts') &&
+            !isNodeFile &&
+            !isFileType(context.filename, '.credentials.ts');
+        if (!isNodeFile && !isHelperFile) {
+            return {};
+        }
+        return {
+            ThrowStatement(node) {
+                if (!isInsideCatchClause(node))
+                    return;
+                if (!node.argument)
+                    return;
+                const { argument } = node;
+                if (argument.type === AST_NODE_TYPES.Identifier) {
+                    const scope = context.sourceCode.getScope(node);
+                    const ref = scope.references.find((r) => r.identifier === argument);
+                    const isCatchParam = ref?.resolved?.defs.some((def) => def.type === DefinitionType.CatchClause) ?? false;
+                    if (isCatchParam) {
+                        context.report({ node, messageId: 'useNodeApiError' });
+                    }
+                    return;
+                }
+                const calleeName = getThrowCalleeName(argument);
+                if (calleeName !== null && !ALLOWED_ERROR_CLASSES.has(calleeName)) {
+                    context.report({
+                        node,
+                        messageId: 'useNodeApiErrorInsteadOfGeneric',
+                        data: { errorClass: calleeName },
+                    });
+                }
+            },
+        };
+    },
+});
+//# sourceMappingURL=require-node-api-error.js.map

package/dist/rules/require-node-api-error.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"require-node-api-error.js","sourceRoot":"","sources":["../../src/rules/require-node-api-error.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,kCAAkC,CAAC;AAClE,OAAO,EAAE,cAAc,EAAiB,MAAM,0BAA0B,CAAC;AAEzE,OAAO,EAAE,UAAU,EAAE,MAAM,mBAAmB,CAAC;AAC/C,OAAO,EAAE,UAAU,EAAE,MAAM,0BAA0B,CAAC;AAEtD,MAAM,qBAAqB,GAAG,IAAI,GAAG,CAAC,CAAC,cAAc,EAAE,oBAAoB,CAAC,CAAC,CAAC;AAE9E,SAAS,kBAAkB,CAAC,QAA6B;IACxD,IAAI,QAAQ,CAAC,IAAI,KAAK,cAAc,CAAC,aAAa,EAAE,CAAC;QACpD,IAAI,QAAQ,CAAC,MAAM,CAAC,IAAI,KAAK,cAAc,CAAC,UAAU,EAAE,CAAC;YACxD,OAAO,QAAQ,CAAC,MAAM,CAAC,IAAI,CAAC;QAC7B,CAAC;IACF,CAAC;IACD,OAAO,IAAI,CAAC;AACb,CAAC;AAED,SAAS,mBAAmB,CAAC,IAAmB;IAC/C,IAAI,OAAO,GAA8B,IAAI,CAAC,MAAM,CAAC;IACrD,OAAO,OAAO,EAAE,CAAC;QAChB,IAAI,OAAO,CAAC,IAAI,KAAK,cAAc,CAAC,WAAW,EAAE,CAAC;YACjD,OAAO,IAAI,CAAC;QACb,CAAC;QACD,OAAO,GAAG,OAAO,CAAC,MAAM,CAAC;IAC1B,CAAC;IACD,OAAO,KAAK,CAAC;AACd,CAAC;AAED,MAAM,CAAC,MAAM,uBAAuB,GAAG,UAAU,CAAC;IACjD,IAAI,EAAE,wBAAwB;IAC9B,IAAI,EAAE;QACL,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACL,WAAW,EACV,iFAAiF;gBACjF,6CAA6C;SAC9C;QACD,QAAQ,EAAE;YACT,eAAe,EACd,gFAAgF;gBAChF,wEAAwE;YACzE,+BAA+B,EAC9B,4EAA4E;gBAC5E,wEAAwE;SACzE;QACD,MAAM,EAAE,EAAE;KACV;IACD,cAAc,EAAE,EAAE;IAClB,MAAM,CAAC,OAAO;QACb,MAAM,UAAU,GAAG,UAAU,CAAC,OAAO,CAAC,QAAQ,EAAE,UAAU,CAAC,CAAC;QAC5D,MAAM,YAAY,GACjB,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAC,KAAK,CAAC;YAChC,CAAC,UAAU;YACX,CAAC,UAAU,CAAC,OAAO,CAAC,QAAQ,EAAE,iBAAiB,CAAC,CAAC;QAElD,IAAI,CAAC,UAAU,IAAI,CAAC,YAAY,EAAE,CAAC;YAClC,OAAO,EAAE,CAAC;QACX,CAAC;QAED,OAAO;YACN,cAAc,CAAC,IAAI;gBAClB,IAAI,CAAC,mBAAmB,CAAC,IAAI,CAAC;oBAAE,OAAO;gBACvC,IAAI,CAAC,IAAI,CAAC,QAAQ;oBAAE,OAAO;gBAE3B,MAAM,EAAE,QAAQ,EAAE,GAAG,IAAI,CAAC;gBAE1B,IAAI,QAAQ,CAAC,IAAI,KAAK,cAAc,CAAC,UAAU,EAAE,CAAC;oBACjD,MAAM,KAAK,GAAG,OAAO,CAAC,UAAU,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;oBAChD,MAAM,GAAG,GAAG,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,KAAK,QAAQ,CAAC,CAAC;oBACpE,MAAM,YAAY,GACjB,GAAG,EAAE,QAAQ,EAAE,IAAI,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,KAAK,cAAc,CAAC,WAAW,CAAC,IAAI,KAAK,CAAC;oBAErF,IAAI,YAAY,EAAE,CAAC;wBAClB,OAAO,CAAC,MAAM,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,iBAAiB,EAAE,CAAC,CAAC;oBACxD,CAAC;oBACD,OAAO;gBACR,CAAC;gBAED,MAAM,UAAU,GAAG,kBAAkB,CAAC,QAAQ,CAAC,CAAC;gBAChD,IAAI,UAAU,KAAK,IAAI,IAAI,CAAC,qBAAqB,CAAC,GAAG,CAAC,UAAU,CAAC,EAAE,CAAC;oBACnE,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI;wBACJ,SAAS,EAAE,iCAAiC;wBAC5C,IAAI,EAAE,EAAE,UAAU,EAAE,UAAU,EAAE;qBAChC,CAAC,CAAC;gBACJ,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/dist/rules/valid-peer-dependencies.d.ts

@@ -0,0 +1,2 @@
+export declare const ValidPeerDependenciesRule: import("@typescript-eslint/utils/ts-eslint").RuleModule<"missingPeerDependencies" | "invalidPeerDependenciesType" | "missingN8nWorkflow" | "pinnedN8nWorkflow" | "forbiddenPeerDependency", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
+//# sourceMappingURL=valid-peer-dependencies.d.ts.map

package/dist/rules/valid-peer-dependencies.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"valid-peer-dependencies.d.ts","sourceRoot":"","sources":["../../src/rules/valid-peer-dependencies.ts"],"names":[],"mappings":"AASA,eAAO,MAAM,yBAAyB,qQA0GpC,CAAC"}

package/dist/rules/valid-peer-dependencies.js

@@ -0,0 +1,107 @@
+import { AST_NODE_TYPES } from '@typescript-eslint/utils';
+import { createRule, findJsonProperty } from '../utils/index.js';
+const REQUIRED_DEP = 'n8n-workflow';
+const REQUIRED_VERSION = '*';
+const ALLOWED_DEPS = new Set([REQUIRED_DEP, 'ai-node-sdk']);
+export const ValidPeerDependenciesRule = createRule({
+    name: 'valid-peer-dependencies',
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Require community node package.json peerDependencies to contain only "n8n-workflow": "*" (and optionally "ai-node-sdk")',
+        },
+        fixable: 'code',
+        messages: {
+            missingPeerDependencies: `The package.json must have a "peerDependencies" section containing "${REQUIRED_DEP}": "${REQUIRED_VERSION}".`,
+            invalidPeerDependenciesType: `"peerDependencies" must be an object mapping package names to version ranges (containing "${REQUIRED_DEP}": "${REQUIRED_VERSION}").`,
+            missingN8nWorkflow: `"peerDependencies" must include "${REQUIRED_DEP}": "${REQUIRED_VERSION}".`,
+            pinnedN8nWorkflow: `"peerDependencies.${REQUIRED_DEP}" must be "${REQUIRED_VERSION}", got {{ value }}.`,
+            forbiddenPeerDependency: '"{{ name }}" is not allowed in "peerDependencies". Only "n8n-workflow" and "ai-node-sdk" are permitted.',
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        if (!context.filename.endsWith('package.json')) {
+            return {};
+        }
+        return {
+            ObjectExpression(node) {
+                if (node.parent?.type !== AST_NODE_TYPES.ExpressionStatement) {
+                    return;
+                }
+                const peerDepsProp = findJsonProperty(node, 'peerDependencies');
+                if (!peerDepsProp) {
+                    context.report({
+                        node,
+                        messageId: 'missingPeerDependencies',
+                        fix(fixer) {
+                            const insertion = `"peerDependencies": { "${REQUIRED_DEP}": "${REQUIRED_VERSION}" }`;
+                            const lastProp = node.properties[node.properties.length - 1];
+                            if (!lastProp) {
+                                return fixer.replaceText(node, `{ ${insertion} }`);
+                            }
+                            return fixer.insertTextAfter(lastProp, `, ${insertion}`);
+                        },
+                    });
+                    return;
+                }
+                if (peerDepsProp.value.type !== AST_NODE_TYPES.ObjectExpression) {
+                    context.report({
+                        node: peerDepsProp,
+                        messageId: 'invalidPeerDependenciesType',
+                    });
+                    return;
+                }
+                const peerDepsObject = peerDepsProp.value;
+                const workflowEntry = findJsonProperty(peerDepsObject, REQUIRED_DEP);
+                if (!workflowEntry) {
+                    context.report({
+                        node: peerDepsProp,
+                        messageId: 'missingN8nWorkflow',
+                        fix(fixer) {
+                            const insertion = `"${REQUIRED_DEP}": "${REQUIRED_VERSION}"`;
+                            const lastProp = peerDepsObject.properties[peerDepsObject.properties.length - 1];
+                            if (!lastProp) {
+                                return fixer.replaceText(peerDepsObject, `{ ${insertion} }`);
+                            }
+                            return fixer.insertTextAfter(lastProp, `, ${insertion}`);
+                        },
+                    });
+                }
+                else if (workflowEntry.value.type !== AST_NODE_TYPES.Literal ||
+                    workflowEntry.value.value !== REQUIRED_VERSION) {
+                    const valueNode = workflowEntry.value;
+                    const rawValue = valueNode.type === AST_NODE_TYPES.Literal ? String(valueNode.raw) : 'non-literal';
+                    context.report({
+                        node: workflowEntry,
+                        messageId: 'pinnedN8nWorkflow',
+                        data: { value: rawValue },
+                        fix(fixer) {
+                            if (valueNode.type !== AST_NODE_TYPES.Literal)
+                                return null;
+                            return fixer.replaceText(valueNode, `"${REQUIRED_VERSION}"`);
+                        },
+                    });
+                }
+                for (const prop of peerDepsObject.properties) {
+                    if (prop.type !== AST_NODE_TYPES.Property)
+                        continue;
+                    if (prop.key.type !== AST_NODE_TYPES.Literal)
+                        continue;
+                    const name = prop.key.value;
+                    if (typeof name !== 'string')
+                        continue;
+                    if (ALLOWED_DEPS.has(name))
+                        continue;
+                    context.report({
+                        node: prop,
+                        messageId: 'forbiddenPeerDependency',
+                        data: { name },
+                    });
+                }
+            },
+        };
+    },
+});
+//# sourceMappingURL=valid-peer-dependencies.js.map

package/dist/rules/valid-peer-dependencies.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"valid-peer-dependencies.js","sourceRoot":"","sources":["../../src/rules/valid-peer-dependencies.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAE1D,OAAO,EAAE,UAAU,EAAE,gBAAgB,EAAE,MAAM,mBAAmB,CAAC;AAEjE,MAAM,YAAY,GAAG,cAAc,CAAC;AACpC,MAAM,gBAAgB,GAAG,GAAG,CAAC;AAC7B,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,CAAC,YAAY,EAAE,aAAa,CAAC,CAAC,CAAC;AAE5D,MAAM,CAAC,MAAM,yBAAyB,GAAG,UAAU,CAAC;IACnD,IAAI,EAAE,yBAAyB;IAC/B,IAAI,EAAE;QACL,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACL,WAAW,EACV,yHAAyH;SAC1H;QACD,OAAO,EAAE,MAAM;QACf,QAAQ,EAAE;YACT,uBAAuB,EAAE,uEAAuE,YAAY,OAAO,gBAAgB,IAAI;YACvI,2BAA2B,EAAE,6FAA6F,YAAY,OAAO,gBAAgB,KAAK;YAClK,kBAAkB,EAAE,oCAAoC,YAAY,OAAO,gBAAgB,IAAI;YAC/F,iBAAiB,EAAE,qBAAqB,YAAY,cAAc,gBAAgB,qBAAqB;YACvG,uBAAuB,EACtB,yGAAyG;SAC1G;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,kBAAkB,CAAC,CAAC;gBAEhE,IAAI,CAAC,YAAY,EAAE,CAAC;oBACnB,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI;wBACJ,SAAS,EAAE,yBAAyB;wBACpC,GAAG,CAAC,KAAK;4BACR,MAAM,SAAS,GAAG,0BAA0B,YAAY,OAAO,gBAAgB,KAAK,CAAC;4BACrF,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,KAAK,SAAS,IAAI,CAAC,CAAC;4BACpD,CAAC;4BACD,OAAO,KAAK,CAAC,eAAe,CAAC,QAAQ,EAAE,KAAK,SAAS,EAAE,CAAC,CAAC;wBAC1D,CAAC;qBACD,CAAC,CAAC;oBACH,OAAO;gBACR,CAAC;gBAED,IAAI,YAAY,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,gBAAgB,EAAE,CAAC;oBACjE,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,YAAY;wBAClB,SAAS,EAAE,6BAA6B;qBACxC,CAAC,CAAC;oBACH,OAAO;gBACR,CAAC;gBAED,MAAM,cAAc,GAAG,YAAY,CAAC,KAAK,CAAC;gBAC1C,MAAM,aAAa,GAAG,gBAAgB,CAAC,cAAc,EAAE,YAAY,CAAC,CAAC;gBAErE,IAAI,CAAC,aAAa,EAAE,CAAC;oBACpB,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,YAAY;wBAClB,SAAS,EAAE,oBAAoB;wBAC/B,GAAG,CAAC,KAAK;4BACR,MAAM,SAAS,GAAG,IAAI,YAAY,OAAO,gBAAgB,GAAG,CAAC;4BAC7D,MAAM,QAAQ,GAAG,cAAc,CAAC,UAAU,CAAC,cAAc,CAAC,UAAU,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;4BACjF,IAAI,CAAC,QAAQ,EAAE,CAAC;gCACf,OAAO,KAAK,CAAC,WAAW,CAAC,cAAc,EAAE,KAAK,SAAS,IAAI,CAAC,CAAC;4BAC9D,CAAC;4BACD,OAAO,KAAK,CAAC,eAAe,CAAC,QAAQ,EAAE,KAAK,SAAS,EAAE,CAAC,CAAC;wBAC1D,CAAC;qBACD,CAAC,CAAC;gBACJ,CAAC;qBAAM,IACN,aAAa,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,OAAO;oBACnD,aAAa,CAAC,KAAK,CAAC,KAAK,KAAK,gBAAgB,EAC7C,CAAC;oBACF,MAAM,SAAS,GAAG,aAAa,CAAC,KAAK,CAAC;oBACtC,MAAM,QAAQ,GACb,SAAS,CAAC,IAAI,KAAK,cAAc,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC;oBACnF,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,aAAa;wBACnB,SAAS,EAAE,mBAAmB;wBAC9B,IAAI,EAAE,EAAE,KAAK,EAAE,QAAQ,EAAE;wBACzB,GAAG,CAAC,KAAK;4BACR,IAAI,SAAS,CAAC,IAAI,KAAK,cAAc,CAAC,OAAO;gCAAE,OAAO,IAAI,CAAC;4BAC3D,OAAO,KAAK,CAAC,WAAW,CAAC,SAAS,EAAE,IAAI,gBAAgB,GAAG,CAAC,CAAC;wBAC9D,CAAC;qBACD,CAAC,CAAC;gBACJ,CAAC;gBAED,KAAK,MAAM,IAAI,IAAI,cAAc,CAAC,UAAU,EAAE,CAAC;oBAC9C,IAAI,IAAI,CAAC,IAAI,KAAK,cAAc,CAAC,QAAQ;wBAAE,SAAS;oBACpD,IAAI,IAAI,CAAC,GAAG,CAAC,IAAI,KAAK,cAAc,CAAC,OAAO;wBAAE,SAAS;oBACvD,MAAM,IAAI,GAAG,IAAI,CAAC,GAAG,CAAC,KAAK,CAAC;oBAC5B,IAAI,OAAO,IAAI,KAAK,QAAQ;wBAAE,SAAS;oBACvC,IAAI,YAAY,CAAC,GAAG,CAAC,IAAI,CAAC;wBAAE,SAAS;oBACrC,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,IAAI;wBACV,SAAS,EAAE,yBAAyB;wBACpC,IAAI,EAAE,EAAE,IAAI,EAAE;qBACd,CAAC,CAAC;gBACJ,CAAC;YACF,CAAC;SACD,CAAC;IACH,CAAC;CACD,CAAC,CAAC"}

package/dist/rules/webhook-lifecycle-complete.d.ts

@@ -0,0 +1,2 @@
+export declare const WebhookLifecycleCompleteRule: import("@typescript-eslint/utils/ts-eslint").RuleModule<"missingWebhookMethods" | "missingLifecycleMethod", [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
+//# sourceMappingURL=webhook-lifecycle-complete.d.ts.map

package/dist/rules/webhook-lifecycle-complete.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"webhook-lifecycle-complete.d.ts","sourceRoot":"","sources":["../../src/rules/webhook-lifecycle-complete.ts"],"names":[],"mappings":"AA+CA,eAAO,MAAM,4BAA4B,qLAwEvC,CAAC"}

package/dist/rules/webhook-lifecycle-complete.js

@@ -0,0 +1,98 @@
+import { AST_NODE_TYPES } from '@typescript-eslint/utils';
+import { createRule, findClassProperty, findObjectProperty, isNodeTypeClass, } from '../utils/index.js';
+const REQUIRED_METHODS = ['checkExists', 'create', 'delete'];
+/**
+ * Returns true if the description declares webhook endpoints, indicating the
+ * node is a webhook-based trigger that needs a complete lifecycle.
+ *
+ * Polling triggers (group `['trigger']` without a `webhooks` array) do not
+ * register remote webhooks and are intentionally out of scope.
+ */
+function hasWebhooksDeclared(descriptionValue) {
+    const webhooksProperty = findObjectProperty(descriptionValue, 'webhooks');
+    if (webhooksProperty?.value.type !== AST_NODE_TYPES.ArrayExpression)
+        return false;
+    return webhooksProperty.value.elements.length > 0;
+}
+/** Returns true when the property defines a (possibly async) method named `name`. */
+function isMethodProperty(property, name) {
+    if (property.type !== AST_NODE_TYPES.Property)
+        return false;
+    if (property.computed)
+        return false;
+    const keyMatches = (property.key.type === AST_NODE_TYPES.Identifier && property.key.name === name) ||
+        (property.key.type === AST_NODE_TYPES.Literal && property.key.value === name);
+    if (!keyMatches)
+        return false;
+    return (property.value.type === AST_NODE_TYPES.FunctionExpression ||
+        property.value.type === AST_NODE_TYPES.ArrowFunctionExpression);
+}
+function findMissingMethods(group) {
+    return REQUIRED_METHODS.filter((method) => !group.properties.some((property) => isMethodProperty(property, method)));
+}
+export const WebhookLifecycleCompleteRule = createRule({
+    name: 'webhook-lifecycle-complete',
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Require webhook trigger nodes to implement the complete webhookMethods lifecycle (checkExists, create, delete)',
+        },
+        messages: {
+            missingWebhookMethods: 'Webhook trigger node is missing the `webhookMethods` property. Implement `checkExists`, `create`, and `delete` to register, verify, and clean up the webhook on the third-party service.',
+            missingLifecycleMethod: 'Webhook trigger lifecycle is incomplete. `webhookMethods.{{group}}` is missing: {{missing}}. All of `checkExists`, `create`, and `delete` must be implemented.',
+        },
+        schema: [],
+    },
+    defaultOptions: [],
+    create(context) {
+        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;
+                const webhookMethodsProperty = findClassProperty(node, 'webhookMethods');
+                if (!hasWebhooksDeclared(descriptionValue) && !webhookMethodsProperty) {
+                    return;
+                }
+                if (!webhookMethodsProperty?.value) {
+                    context.report({
+                        node: node.id ?? node,
+                        messageId: 'missingWebhookMethods',
+                    });
+                    return;
+                }
+                if (webhookMethodsProperty.value.type !== AST_NODE_TYPES.ObjectExpression) {
+                    return;
+                }
+                for (const groupProperty of webhookMethodsProperty.value.properties) {
+                    if (groupProperty.type !== AST_NODE_TYPES.Property)
+                        continue;
+                    if (groupProperty.value.type !== AST_NODE_TYPES.ObjectExpression)
+                        continue;
+                    const groupName = groupProperty.key.type === AST_NODE_TYPES.Identifier
+                        ? groupProperty.key.name
+                        : groupProperty.key.type === AST_NODE_TYPES.Literal
+                            ? String(groupProperty.key.value)
+                            : 'default';
+                    const missing = findMissingMethods(groupProperty.value);
+                    if (missing.length === 0)
+                        continue;
+                    context.report({
+                        node: groupProperty.key,
+                        messageId: 'missingLifecycleMethod',
+                        data: {
+                            group: groupName,
+                            missing: missing.map((m) => `\`${m}\``).join(', '),
+                        },
+                    });
+                }
+            },
+        };
+    },
+});
+//# sourceMappingURL=webhook-lifecycle-complete.js.map

package/dist/rules/webhook-lifecycle-complete.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"webhook-lifecycle-complete.js","sourceRoot":"","sources":["../../src/rules/webhook-lifecycle-complete.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAiB,MAAM,0BAA0B,CAAC;AAEzE,OAAO,EACN,UAAU,EACV,iBAAiB,EACjB,kBAAkB,EAClB,eAAe,GACf,MAAM,mBAAmB,CAAC;AAE3B,MAAM,gBAAgB,GAAG,CAAC,aAAa,EAAE,QAAQ,EAAE,QAAQ,CAAU,CAAC;AAGtE;;;;;;GAMG;AACH,SAAS,mBAAmB,CAAC,gBAA2C;IACvE,MAAM,gBAAgB,GAAG,kBAAkB,CAAC,gBAAgB,EAAE,UAAU,CAAC,CAAC;IAC1E,IAAI,gBAAgB,EAAE,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,eAAe;QAAE,OAAO,KAAK,CAAC;IAClF,OAAO,gBAAgB,CAAC,KAAK,CAAC,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC;AACnD,CAAC;AAED,qFAAqF;AACrF,SAAS,gBAAgB,CAAC,QAAuC,EAAE,IAAY;IAC9E,IAAI,QAAQ,CAAC,IAAI,KAAK,cAAc,CAAC,QAAQ;QAAE,OAAO,KAAK,CAAC;IAC5D,IAAI,QAAQ,CAAC,QAAQ;QAAE,OAAO,KAAK,CAAC;IAEpC,MAAM,UAAU,GACf,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,KAAK,cAAc,CAAC,UAAU,IAAI,QAAQ,CAAC,GAAG,CAAC,IAAI,KAAK,IAAI,CAAC;QAC/E,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,KAAK,cAAc,CAAC,OAAO,IAAI,QAAQ,CAAC,GAAG,CAAC,KAAK,KAAK,IAAI,CAAC,CAAC;IAC/E,IAAI,CAAC,UAAU;QAAE,OAAO,KAAK,CAAC;IAE9B,OAAO,CACN,QAAQ,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,kBAAkB;QACzD,QAAQ,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,uBAAuB,CAC9D,CAAC;AACH,CAAC;AAED,SAAS,kBAAkB,CAAC,KAAgC;IAC3D,OAAO,gBAAgB,CAAC,MAAM,CAC7B,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE,EAAE,CAAC,gBAAgB,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC,CACpF,CAAC;AACH,CAAC;AAED,MAAM,CAAC,MAAM,4BAA4B,GAAG,UAAU,CAAC;IACtD,IAAI,EAAE,4BAA4B;IAClC,IAAI,EAAE;QACL,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACL,WAAW,EACV,gHAAgH;SACjH;QACD,QAAQ,EAAE;YACT,qBAAqB,EACpB,0LAA0L;YAC3L,sBAAsB,EACrB,gKAAgK;SACjK;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;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,MAAM,sBAAsB,GAAG,iBAAiB,CAAC,IAAI,EAAE,gBAAgB,CAAC,CAAC;gBAEzE,IAAI,CAAC,mBAAmB,CAAC,gBAAgB,CAAC,IAAI,CAAC,sBAAsB,EAAE,CAAC;oBACvE,OAAO;gBACR,CAAC;gBAED,IAAI,CAAC,sBAAsB,EAAE,KAAK,EAAE,CAAC;oBACpC,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,IAAI,CAAC,EAAE,IAAI,IAAI;wBACrB,SAAS,EAAE,uBAAuB;qBAClC,CAAC,CAAC;oBACH,OAAO;gBACR,CAAC;gBAED,IAAI,sBAAsB,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,gBAAgB,EAAE,CAAC;oBAC3E,OAAO;gBACR,CAAC;gBAED,KAAK,MAAM,aAAa,IAAI,sBAAsB,CAAC,KAAK,CAAC,UAAU,EAAE,CAAC;oBACrE,IAAI,aAAa,CAAC,IAAI,KAAK,cAAc,CAAC,QAAQ;wBAAE,SAAS;oBAC7D,IAAI,aAAa,CAAC,KAAK,CAAC,IAAI,KAAK,cAAc,CAAC,gBAAgB;wBAAE,SAAS;oBAE3E,MAAM,SAAS,GACd,aAAa,CAAC,GAAG,CAAC,IAAI,KAAK,cAAc,CAAC,UAAU;wBACnD,CAAC,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI;wBACxB,CAAC,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,KAAK,cAAc,CAAC,OAAO;4BAClD,CAAC,CAAC,MAAM,CAAC,aAAa,CAAC,GAAG,CAAC,KAAK,CAAC;4BACjC,CAAC,CAAC,SAAS,CAAC;oBAEf,MAAM,OAAO,GAAG,kBAAkB,CAAC,aAAa,CAAC,KAAK,CAAC,CAAC;oBACxD,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC;wBAAE,SAAS;oBAEnC,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,aAAa,CAAC,GAAG;wBACvB,SAAS,EAAE,wBAAwB;wBACnC,IAAI,EAAE;4BACL,KAAK,EAAE,SAAS;4BAChB,OAAO,EAAE,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;yBAClD;qBACD,CAAC,CAAC;gBACJ,CAAC;YACF,CAAC;SACD,CAAC;IACH,CAAC;CACD,CAAC,CAAC"}

package/docs/rules/no-overrides-field.md

@@ -0,0 +1,50 @@
+# Ban the "overrides" field in community node package.json (`@n8n/community-nodes/no-overrides-field`)
+
+💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+The `overrides` field in `package.json` lets a package force specific versions of its (transitive) dependencies. In the context of n8n community nodes this is dangerous:
+
+- Community nodes are installed into a shared n8n runtime alongside other nodes. Overriding a shared library (e.g. `axios`, `@langchain/core`, `minimatch`) can silently substitute an incompatible version for every other node that depends on it, causing hard-to-diagnose runtime failures.
+- Community nodes are distributed as pre-built packages with their dependencies already bundled or declared as `peerDependencies`. Any version pinning that the node actually needs should happen during development, not at install time on the user's n8n instance.
+- `overrides` is frequently copy-pasted from an unrelated internal project and is almost never intentional in a community node.
+
+If you have a genuine compatibility need, bundle the dependency into the published artifact or declare it via `peerDependencies` instead.
+
+## Examples
+
+### Incorrect
+
+```json
+{
+  "name": "n8n-nodes-example",
+  "overrides": {
+    "axios": "1.7.0"
+  }
+}
+```
+
+```json
+{
+  "name": "n8n-nodes-example",
+  "overrides": {
+    "axios": "1.7.0",
+    "@langchain/core": "0.3.0",
+    "minimatch": "9.0.5"
+  }
+}
+```
+
+### Correct
+
+```json
+{
+  "name": "n8n-nodes-example",
+  "peerDependencies": {
+    "n8n-workflow": "*"
+  }
+}
+```

package/docs/rules/node-class-description-icon-missing.md

@@ -1,11 +1,13 @@
-# Node class description must have an `icon` property defined (`@n8n/community-nodes/node-class-description-icon-missing`)
+# Node class description must have an `icon` property defined. Deprecated: use `require-node-description-fields` instead (`@n8n/community-nodes/node-class-description-icon-missing`)
 
-💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+❌ This rule is deprecated.
 
 💡 This rule is manually fixable by [editor suggestions](https://eslint.org/docs/latest/use/core-concepts#rule-suggestions).
 
 <!-- end auto-generated rule header -->
 
+> **Deprecated:** Use [`require-node-description-fields`](require-node-description-fields.md) instead.
+
 ## Rule Details
 
 Validates that node classes define an `icon` property in their `description` object. Icons are required for nodes to display correctly in the n8n editor.

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

@@ -0,0 +1,45 @@
+# Require the "n8n-community-node-package" keyword in community node package.json (`@n8n/community-nodes/require-community-node-keyword`)
+
+⚠️ This rule _warns_ 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
+
+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-node-api-error.md

@@ -0,0 +1,62 @@
+# Require NodeApiError or NodeOperationError for error wrapping in catch blocks. Raw errors lose HTTP context in the n8n UI (`@n8n/community-nodes/require-node-api-error`)
+
+💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+When errors are caught and re-thrown in n8n nodes, they must be wrapped in
+`NodeApiError` or `NodeOperationError`. Raw re-throws and generic `Error`
+constructors lose HTTP context (status code, response body, etc.) that the n8n
+UI relies on to display meaningful error information to users.
+
+## Examples
+
+### Incorrect
+
+```js
+try {
+  await apiRequest();
+} catch (error) {
+  throw error;
+}
+```
+
+```js
+try {
+  await apiRequest();
+} catch (error) {
+  throw new Error('Request failed');
+}
+```
+
+### Correct
+
+```js
+try {
+  await apiRequest();
+} catch (error) {
+  throw new NodeApiError(this.getNode(), error as JsonObject);
+}
+```
+
+```js
+try {
+  await apiRequest();
+} catch (error) {
+  throw new NodeOperationError(this.getNode(), 'Operation failed', { itemIndex: i });
+}
+```
+
+```js
+try {
+  await apiRequest();
+} catch (error) {
+  if (this.continueOnFail()) {
+    returnData.push({ json: { error: error.message } });
+    continue;
+  }
+  throw new NodeApiError(this.getNode(), error as JsonObject);
+}
+```

package/docs/rules/require-node-description-fields.md

@@ -0,0 +1,46 @@
+# Node class description must define all required fields: icon, subtitle (`@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"]}}',
+    // ...
+  };
+}
+```