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

package/dist/rules/valid-credential-references.js

@@ -0,0 +1,77 @@
+import { TSESTree } from '@typescript-eslint/types';
+import { isNodeTypeClass, findClassProperty, findArrayLiteralProperty, extractCredentialNameFromArray, findPackageJson, readPackageJsonCredentials, isFileType, findSimilarStrings, createRule, } from '../utils/index.js';
+export const ValidCredentialReferencesRule = createRule({
+    name: 'valid-credential-references',
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Ensure credentials referenced in node descriptions exist as credential classes in the package',
+        },
+        messages: {
+            credentialNotFound: 'Credential "{{ credentialName }}" does not exist in this package. Check for typos or ensure the credential class is declared and listed in package.json.',
+            didYouMean: "Did you mean '{{ suggestedName }}'?",
+        },
+        schema: [],
+        hasSuggestions: true,
+    },
+    defaultOptions: [],
+    create(context) {
+        if (!isFileType(context.filename, '.node.ts')) {
+            return {};
+        }
+        let packageCredentials = null;
+        const loadPackageCredentials = () => {
+            if (packageCredentials !== null) {
+                return packageCredentials;
+            }
+            const packageJsonPath = findPackageJson(context.filename);
+            if (!packageJsonPath) {
+                packageCredentials = new Set();
+                return packageCredentials;
+            }
+            packageCredentials = readPackageJsonCredentials(packageJsonPath);
+            return packageCredentials;
+        };
+        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 credentialsArray = findArrayLiteralProperty(descriptionProperty.value, 'credentials');
+                if (!credentialsArray) {
+                    return;
+                }
+                const knownCredentials = loadPackageCredentials();
+                if (knownCredentials.size === 0) {
+                    return;
+                }
+                credentialsArray.elements.forEach((element) => {
+                    const credentialInfo = extractCredentialNameFromArray(element);
+                    if (!credentialInfo || knownCredentials.has(credentialInfo.name)) {
+                        return;
+                    }
+                    const similar = findSimilarStrings(credentialInfo.name, knownCredentials);
+                    const suggestions = similar.map((suggestedName) => ({
+                        messageId: 'didYouMean',
+                        data: { suggestedName },
+                        fix(fixer) {
+                            return fixer.replaceText(credentialInfo.node, `"${suggestedName}"`);
+                        },
+                    }));
+                    context.report({
+                        node: credentialInfo.node,
+                        messageId: 'credentialNotFound',
+                        data: { credentialName: credentialInfo.name },
+                        suggest: suggestions,
+                    });
+                });
+            },
+        };
+    },
+});
+//# sourceMappingURL=valid-credential-references.js.map

package/dist/rules/valid-credential-references.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"valid-credential-references.js","sourceRoot":"","sources":["../../src/rules/valid-credential-references.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,0BAA0B,CAAC;AAGpD,OAAO,EACN,eAAe,EACf,iBAAiB,EACjB,wBAAwB,EACxB,8BAA8B,EAC9B,eAAe,EACf,0BAA0B,EAC1B,UAAU,EACV,kBAAkB,EAClB,UAAU,GACV,MAAM,mBAAmB,CAAC;AAE3B,MAAM,CAAC,MAAM,6BAA6B,GAAG,UAAU,CAAC;IACvD,IAAI,EAAE,6BAA6B;IACnC,IAAI,EAAE;QACL,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACL,WAAW,EACV,+FAA+F;SAChG;QACD,QAAQ,EAAE;YACT,kBAAkB,EACjB,0JAA0J;YAC3J,UAAU,EAAE,qCAAqC;SACjD;QACD,MAAM,EAAE,EAAE;QACV,cAAc,EAAE,IAAI;KACpB;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,IAAI,kBAAkB,GAAuB,IAAI,CAAC;QAElD,MAAM,sBAAsB,GAAG,GAAgB,EAAE;YAChD,IAAI,kBAAkB,KAAK,IAAI,EAAE,CAAC;gBACjC,OAAO,kBAAkB,CAAC;YAC3B,CAAC;YAED,MAAM,eAAe,GAAG,eAAe,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;YAC1D,IAAI,CAAC,eAAe,EAAE,CAAC;gBACtB,kBAAkB,GAAG,IAAI,GAAG,EAAE,CAAC;gBAC/B,OAAO,kBAAkB,CAAC;YAC3B,CAAC;YAED,kBAAkB,GAAG,0BAA0B,CAAC,eAAe,CAAC,CAAC;YACjE,OAAO,kBAAkB,CAAC;QAC3B,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,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,wBAAwB,CAAC,mBAAmB,CAAC,KAAK,EAAE,aAAa,CAAC,CAAC;gBAC5F,IAAI,CAAC,gBAAgB,EAAE,CAAC;oBACvB,OAAO;gBACR,CAAC;gBAED,MAAM,gBAAgB,GAAG,sBAAsB,EAAE,CAAC;gBAClD,IAAI,gBAAgB,CAAC,IAAI,KAAK,CAAC,EAAE,CAAC;oBACjC,OAAO;gBACR,CAAC;gBAED,gBAAgB,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,EAAE;oBAC7C,MAAM,cAAc,GAAG,8BAA8B,CAAC,OAAO,CAAC,CAAC;oBAC/D,IAAI,CAAC,cAAc,IAAI,gBAAgB,CAAC,GAAG,CAAC,cAAc,CAAC,IAAI,CAAC,EAAE,CAAC;wBAClE,OAAO;oBACR,CAAC;oBAED,MAAM,OAAO,GAAG,kBAAkB,CAAC,cAAc,CAAC,IAAI,EAAE,gBAAgB,CAAC,CAAC;oBAC1E,MAAM,WAAW,GAChB,OAAO,CAAC,GAAG,CAAC,CAAC,aAAa,EAAE,EAAE,CAAC,CAAC;wBAC/B,SAAS,EAAE,YAAqB;wBAChC,IAAI,EAAE,EAAE,aAAa,EAAE;wBACvB,GAAG,CAAC,KAAK;4BACR,OAAO,KAAK,CAAC,WAAW,CAAC,cAAc,CAAC,IAAI,EAAE,IAAI,aAAa,GAAG,CAAC,CAAC;wBACrE,CAAC;qBACD,CAAC,CAAC,CAAC;oBAEL,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,cAAc,CAAC,IAAI;wBACzB,SAAS,EAAE,oBAAoB;wBAC/B,IAAI,EAAE,EAAE,cAAc,EAAE,cAAc,CAAC,IAAI,EAAE;wBAC7C,OAAO,EAAE,WAAW;qBACpB,CAAC,CAAC;gBACJ,CAAC,CAAC,CAAC;YACJ,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" | "emptyWebhookMethods" | "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,6MAkFvC,CAAC"}

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

@@ -0,0 +1,106 @@
+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.',
+            emptyWebhookMethods: 'Webhook trigger node has an empty `webhookMethods` object. Define at least one lifecycle group with `checkExists`, `create`, and `delete` methods.',
+            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;
+                }
+                if (webhookMethodsProperty.value.properties.length === 0) {
+                    context.report({
+                        node: webhookMethodsProperty.key,
+                        messageId: 'emptyWebhookMethods',
+                    });
+                    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,mBAAmB,EAClB,oJAAoJ;YACrJ,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,IAAI,sBAAsB,CAAC,KAAK,CAAC,UAAU,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;oBAC1D,OAAO,CAAC,MAAM,CAAC;wBACd,IAAI,EAAE,sBAAsB,CAAC,GAAG;wBAChC,SAAS,EAAE,qBAAqB;qBAChC,CAAC,CAAC;oBACH,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/no-runtime-dependencies.md

@@ -0,0 +1,58 @@
+# Disallow non-empty "dependencies" in community node package.json (`@n8n/community-nodes/no-runtime-dependencies`)
+
+💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+The `dependencies` field in `package.json` declares packages that are installed alongside the node at runtime. In the context of n8n community nodes this is dangerous:
+
+- Community nodes run inside the shared n8n runtime alongside all other installed nodes. Any package listed in `dependencies` gets installed into that shared environment and can shadow or conflict with versions already used by n8n or other nodes.
+- Unlike application packages, community nodes should not own their runtime environment. Shared libraries must be declared in `peerDependencies` (so the host runtime supplies them) or bundled at build time into the published artifact.
+- A non-empty `dependencies` section is a strong signal that the package was scaffolded from a generic Node.js template without adapting it to the n8n community node model.
+
+## Examples
+
+### Incorrect
+
+```json
+{
+  "name": "n8n-nodes-example",
+  "dependencies": {
+    "axios": "1.0.0"
+  }
+}
+```
+
+```json
+{
+  "name": "n8n-nodes-example",
+  "dependencies": {
+    "axios": "1.7.0",
+    "fast-xml-parser": "4.4.0",
+    "minimatch": "9.0.5"
+  }
+}
+```
+
+### Correct
+
+```json
+{
+  "name": "n8n-nodes-example",
+  "peerDependencies": {
+    "n8n-workflow": "*"
+  }
+}
+```
+
+```json
+{
+  "name": "n8n-nodes-example",
+  "dependencies": {},
+  "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 **deprecated**. Use [`require-node-description-fields`](require-node-description-fields.md) instead.
+❌ 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

@@ -1,8 +1,8 @@
-# Require the "n8n-community-node-package" keyword in package.json (`@n8n/community-nodes/require-community-node-keyword`)
+# Require the "n8n-community-node-package" keyword in community node package.json (`@n8n/community-nodes/require-community-node-keyword`)
 
-⚠️ This rule is set to `warn` in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+⚠️ This rule _warns_ 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).
+🔧 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 -->
 

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

@@ -1,4 +1,4 @@
-# Node class description must define all required fields (`@n8n/community-nodes/require-node-description-fields`)
+# 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`.
 

package/docs/rules/valid-credential-references.md

@@ -0,0 +1,78 @@
+# Ensure credentials referenced in node descriptions exist as credential classes in the package (`@n8n/community-nodes/valid-credential-references`)
+
+💼 This rule is enabled in the following configs: ✅ `recommended`, ☑️ `recommendedWithoutN8nCloudSupport`.
+
+💡 This rule is manually fixable by [editor suggestions](https://eslint.org/docs/latest/use/core-concepts#rule-suggestions).
+
+<!-- end auto-generated rule header -->
+
+## Rule Details
+
+For each entry in `description.credentials[]`, this rule verifies that the referenced `name` matches the `name` class field of a credential class declared in the same package (as listed in `package.json` under `n8n.credentials`).
+
+This catches typos and broken references. When `cred-class-name-suffix` is also enabled, this rule naturally enforces the naming convention in the common case while still allowing legitimately named credentials such as `httpHeaderAuth` or `webhookAuth`.
+
+## Examples
+
+### ❌ Incorrect
+
+```typescript
+// MyApiCredential.credentials.ts
+export class MyApiCredential implements ICredentialType {
+  name = 'myApiCredential';
+  // ...
+}
+
+// package.json: "n8n": { "credentials": ["dist/credentials/MyApiCredential.credentials.js"] }
+
+export class MyNode implements INodeType {
+  description: INodeTypeDescription = {
+    credentials: [
+      {
+        name: 'myApiCredentail', // Typo — no credential with this name exists
+        required: true,
+      },
+    ],
+    // ...
+  };
+}
+```
+
+### ✅ Correct
+
+```typescript
+// MyApiCredential.credentials.ts
+export class MyApiCredential implements ICredentialType {
+  name = 'myApiCredential';
+  // ...
+}
+
+// package.json: "n8n": { "credentials": ["dist/credentials/MyApiCredential.credentials.js"] }
+
+export class MyNode implements INodeType {
+  description: INodeTypeDescription = {
+    credentials: [
+      {
+        name: 'myApiCredential', // Matches the credential class name property
+        required: true,
+      },
+    ],
+    // ...
+  };
+}
+```
+
+## Setup
+
+Declare your credential files in `package.json` so the rule can resolve credential class names:
+
+```json
+{
+  "name": "n8n-nodes-my-service",
+  "n8n": {
+    "credentials": [
+      "dist/credentials/MyApiCredential.credentials.js"
+    ]
+  }
+}
+```