feat(require-event-prefix): implemented the rule

Author: marekdedic

.changeset/rich-dogs-design.md

@@ -0,0 +1,5 @@
+---
+'eslint-plugin-svelte': minor
+---
+
+feat: added the `require-event-prefix` rule

README.md

@@ -395,6 +395,7 @@ These rules relate to style guidelines, and are therefore quite subjective:
 | [svelte/no-spaces-around-equal-signs-in-attribute](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-spaces-around-equal-signs-in-attribute/) | disallow spaces around equal signs in attribute | :wrench: |
 | [svelte/prefer-class-directive](https://sveltejs.github.io/eslint-plugin-svelte/rules/prefer-class-directive/) | require class directives instead of ternary expressions | :wrench: |
 | [svelte/prefer-style-directive](https://sveltejs.github.io/eslint-plugin-svelte/rules/prefer-style-directive/) | require style directives instead of style attribute | :wrench: |
+| [svelte/require-event-prefix](https://sveltejs.github.io/eslint-plugin-svelte/rules/require-event-prefix/) | require component event names to start with "on" |  |
 | [svelte/shorthand-attribute](https://sveltejs.github.io/eslint-plugin-svelte/rules/shorthand-attribute/) | enforce use of shorthand syntax in attribute | :wrench: |
 | [svelte/shorthand-directive](https://sveltejs.github.io/eslint-plugin-svelte/rules/shorthand-directive/) | enforce use of shorthand syntax in directives | :wrench: |
 | [svelte/sort-attributes](https://sveltejs.github.io/eslint-plugin-svelte/rules/sort-attributes/) | enforce order of attributes | :wrench: |

docs/rules.md

@@ -92,6 +92,7 @@ These rules relate to style guidelines, and are therefore quite subjective:
 | [svelte/no-spaces-around-equal-signs-in-attribute](./rules/no-spaces-around-equal-signs-in-attribute.md) | disallow spaces around equal signs in attribute                                    | :wrench: |
 | [svelte/prefer-class-directive](./rules/prefer-class-directive.md)                                       | require class directives instead of ternary expressions                            | :wrench: |
 | [svelte/prefer-style-directive](./rules/prefer-style-directive.md)                                       | require style directives instead of style attribute                                | :wrench: |
+| [svelte/require-event-prefix](./rules/require-event-prefix.md)                                           | require component event names to start with "on"                                   |          |
 | [svelte/shorthand-attribute](./rules/shorthand-attribute.md)                                             | enforce use of shorthand syntax in attribute                                       | :wrench: |
 | [svelte/shorthand-directive](./rules/shorthand-directive.md)                                             | enforce use of shorthand syntax in directives                                      | :wrench: |
 | [svelte/sort-attributes](./rules/sort-attributes.md)                                                     | enforce order of attributes                                                        | :wrench: |

packages/eslint-plugin-svelte/src/rule-types.ts

@@ -306,6 +306,11 @@ export interface RuleOptions {
    * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/require-event-dispatcher-types/
    */
   'svelte/require-event-dispatcher-types'?: Linter.RuleEntry<[]>
+  /**
+   * require component event names to start with "on"
+   * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/require-event-prefix/
+   */
+  'svelte/require-event-prefix'?: Linter.RuleEntry<SvelteRequireEventPrefix>
   /**
    * require style attributes that can be optimized
    * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/require-optimized-style-attribute/
@@ -531,6 +536,10 @@ type SveltePreferConst = []|[{
   destructuring?: ("any" | "all")
   ignoreReadBeforeAssign?: boolean
 }]
+// ----- svelte/require-event-prefix -----
+type SvelteRequireEventPrefix = []|[{
+  checkAsyncFunctions?: boolean
+}]
 // ----- svelte/shorthand-attribute -----
 type SvelteShorthandAttribute = []|[{
   prefer?: ("always" | "never")

packages/eslint-plugin-svelte/src/rules/require-event-prefix.ts

@@ -0,0 +1,108 @@
+import { createRule } from '../utils/index.js';
+import { type TSTools, getTypeScriptTools } from 'src/utils/ts-utils/index.js';
+import {
+	type MethodSignature,
+	type Symbol,
+	SymbolFlags,
+	SyntaxKind,
+	type Type,
+	type TypeReferenceNode
+} from 'typescript';
+import type { CallExpression } from 'estree';
+
+export default createRule('require-event-prefix', {
+	meta: {
+		docs: {
+			description: 'require component event names to start with "on"',
+			category: 'Stylistic Issues',
+			conflictWithPrettier: false,
+			recommended: false
+		},
+		schema: [
+			{
+				type: 'object',
+				properties: {
+					checkAsyncFunctions: {
+						type: 'boolean'
+					}
+				},
+				additionalProperties: false
+			}
+		],
+		messages: {
+			nonPrefixedFunction: 'Component event name must start with "on".'
+		},
+		type: 'suggestion',
+		conditions: [
+			{
+				svelteVersions: ['5'],
+				svelteFileTypes: ['.svelte']
+			}
+		]
+	},
+	create(context) {
+		const tsTools = getTypeScriptTools(context);
+		if (!tsTools) {
+			return {};
+		}
+
+		const checkAsyncFunctions = context.options[0]?.checkAsyncFunctions ?? false;
+
+		return {
+			CallExpression(node) {
+				const propsType = getPropsType(node, tsTools);
+				if (propsType === undefined) {
+					return;
+				}
+				for (const property of propsType.getProperties()) {
+					if (property.getFlags() & SymbolFlags.Method && !property.getName().startsWith('on')) {
+						if (!checkAsyncFunctions && isFunctionAsync(property)) {
+							continue;
+						}
+						context.report({
+							node,
+							messageId: 'nonPrefixedFunction'
+						});
+					}
+				}
+			}
+		};
+	}
+});
+
+function getPropsType(node: CallExpression, tsTools: TSTools): Type | undefined {
+	if (
+		node.callee.type !== 'Identifier' ||
+		node.callee.name !== '$props' ||
+		node.parent.type !== 'VariableDeclarator'
+	) {
+		return undefined;
+	}
+
+	const tsNode = tsTools.service.esTreeNodeToTSNodeMap.get(node.parent.id);
+	if (tsNode === undefined) {
+		return undefined;
+	}
+
+	const checker = tsTools.service.program.getTypeChecker();
+	return checker.getTypeAtLocation(tsNode);
+}
+
+function isFunctionAsync(functionSymbol: Symbol): boolean {
+	return (
+		functionSymbol.getDeclarations()?.some((declaration) => {
+			if (declaration.kind !== SyntaxKind.MethodSignature) {
+				return false;
+			}
+			const declarationType = (declaration as MethodSignature).type;
+			if (declarationType?.kind !== SyntaxKind.TypeReference) {
+				return false;
+			}
+			const declarationTypeName = (declarationType as TypeReferenceNode).typeName;
+			return (
+				declarationTypeName.kind === SyntaxKind.Identifier &&
+				declarationTypeName.escapedText === 'Promise'
+			);
+		}) ?? false
+	);
+}

packages/eslint-plugin-svelte/src/utils/rules.ts

@@ -60,6 +60,7 @@ import preferDestructuredStoreProps from '../rules/prefer-destructured-store-pro
 import preferStyleDirective from '../rules/prefer-style-directive.js';
 import requireEachKey from '../rules/require-each-key.js';
 import requireEventDispatcherTypes from '../rules/require-event-dispatcher-types.js';
+import requireEventPrefix from '../rules/require-event-prefix.js';
 import requireOptimizedStyleAttribute from '../rules/require-optimized-style-attribute.js';
 import requireStoreCallbacksUseSetParam from '../rules/require-store-callbacks-use-set-param.js';
 import requireStoreReactiveAccess from '../rules/require-store-reactive-access.js';
@@ -133,6 +134,7 @@ export const rules = [
 	preferStyleDirective,
 	requireEachKey,
 	requireEventDispatcherTypes,
+	requireEventPrefix,
 	requireOptimizedStyleAttribute,
 	requireStoreCallbacksUseSetParam,
 	requireStoreReactiveAccess,