@saasmakers/eslint 1.0.22 → 1.0.24

package/dist/index.cjs

@@ -1,7 +1,6 @@
 'use strict';
 
 const index$1 = require('./shared/eslint.BqRQ4tAN.cjs');
-const shared = require('@saasmakers/shared');
 require('eslint');
 require('eslint/use-at-your-own-risk');
 
@@ -519,7 +518,7 @@ const rule$3 = {
   }
 };
 
-const defaultLocales = shared.localeCodes;
+const defaultLocales = ["en", "fr"];
 const indentSpaces = 2;
 const i18nRegex = /(<i18n\s+lang=["']json["']>)([\s\S]*?)<\/i18n>/i;
 const templateRegex$1 = /<template>([\s\S]*)<\/template>/i;
@@ -809,6 +808,11 @@ const templateRegex = /<template>([\s\S]*)<\/template>/i;
 const templateTagLength = "<template>".length;
 const propsPrefixRegex = /\$?props\.(\w+)/g;
 const trueAttributeRegex = /(?<![\w-]):?(?!aria-)([a-z0-9-]+)="true"/gi;
+const eventHandlerRegex = /(?:@|v-on:)[^\s="'<>/]+\s*=\s*"([^"]*)"/g;
+const bareIdentifierRegex = /^[a-z_$][\w$]*$/i;
+const singleCallRegex = /^([a-z_$][\w$]*)\s*\(.*\)$/is;
+const onPrefixRegex = /^on[A-Z]/;
+const exemptCallees = /* @__PURE__ */ new Set(["$emit", "emit"]);
 const dollarTPatterns = [
   {
     pattern: / \$t\(/g,
@@ -834,10 +838,11 @@ const dollarTPatterns = [
 const rule = {
   defaultOptions: [],
   meta: {
-    docs: { description: 'Format Vue templates: strip unnecessary props/$props prefixes, redundant ="true" attributes (except aria- attributes), and enforce t() over $t()' },
+    docs: { description: 'Format Vue templates: strip unnecessary props/$props prefixes, redundant ="true" attributes (except aria- attributes), enforce t() over $t(), and require "on" prefix on named event listener handlers' },
     fixable: "code",
     messages: {
       noPropsPrefix: "Unnecessary props/$props prefix in template. Props are automatically available in template scope.",
+      prefixEventHandlerWithOn: 'Event listener handler "{{name}}" must be prefixed with "on" (e.g. @click="onClick").',
       removeTrueAttribute: 'Unnecessary ="true" attribute. Use the attribute name directly instead.',
       useT: "Use t() instead of $t() in Vue templates as it does not work with <i18n> tags."
     },
@@ -912,6 +917,33 @@ const rule = {
             dollarTMatch = pattern.exec(templateContent);
           }
         }
+        eventHandlerRegex.lastIndex = 0;
+        let eventHandlerMatch = eventHandlerRegex.exec(templateContent);
+        while (eventHandlerMatch !== null) {
+          const handlerValue = eventHandlerMatch[1].trim();
+          let handlerName;
+          if (bareIdentifierRegex.test(handlerValue)) {
+            handlerName = handlerValue;
+          } else {
+            const singleCallMatch = singleCallRegex.exec(handlerValue);
+            if (singleCallMatch) {
+              handlerName = singleCallMatch[1];
+            }
+          }
+          if (handlerName && !exemptCallees.has(handlerName) && !onPrefixRegex.test(handlerName)) {
+            const valueStart = templateStartIndex + eventHandlerMatch.index + eventHandlerMatch[0].length - 1 - eventHandlerMatch[1].length;
+            const valueEnd = valueStart + eventHandlerMatch[1].length;
+            context.report({
+              data: { name: handlerName },
+              loc: {
+                end: sourceCode.getLocFromIndex(valueEnd),
+                start: sourceCode.getLocFromIndex(valueStart)
+              },
+              messageId: "prefixEventHandlerWithOn"
+            });
+          }
+          eventHandlerMatch = eventHandlerRegex.exec(templateContent);
+        }
       }
     };
   }

package/dist/index.d.cts

@@ -5203,7 +5203,7 @@ declare const _default: {
             locales?: string[];
         } | undefined)?], unknown, RuleListener>;
         'vue-format-script': RuleModule<"multilineComputed" | "untypedEmits", [], unknown, RuleListener>;
-        'vue-format-template': RuleModule<"noPropsPrefix" | "removeTrueAttribute" | "useT", [], unknown, RuleListener>;
+        'vue-format-template': RuleModule<"noPropsPrefix" | "prefixEventHandlerWithOn" | "removeTrueAttribute" | "useT", [], unknown, RuleListener>;
     };
 };
 

package/dist/index.d.mts

@@ -5203,7 +5203,7 @@ declare const _default: {
             locales?: string[];
         } | undefined)?], unknown, RuleListener>;
         'vue-format-script': RuleModule<"multilineComputed" | "untypedEmits", [], unknown, RuleListener>;
-        'vue-format-template': RuleModule<"noPropsPrefix" | "removeTrueAttribute" | "useT", [], unknown, RuleListener>;
+        'vue-format-template': RuleModule<"noPropsPrefix" | "prefixEventHandlerWithOn" | "removeTrueAttribute" | "useT", [], unknown, RuleListener>;
     };
 };
 

package/dist/index.d.ts

@@ -5203,7 +5203,7 @@ declare const _default: {
             locales?: string[];
         } | undefined)?], unknown, RuleListener>;
         'vue-format-script': RuleModule<"multilineComputed" | "untypedEmits", [], unknown, RuleListener>;
-        'vue-format-template': RuleModule<"noPropsPrefix" | "removeTrueAttribute" | "useT", [], unknown, RuleListener>;
+        'vue-format-template': RuleModule<"noPropsPrefix" | "prefixEventHandlerWithOn" | "removeTrueAttribute" | "useT", [], unknown, RuleListener>;
     };
 };
 

package/dist/index.mjs

@@ -1,5 +1,4 @@
 import { d as distExports } from './shared/eslint.DOaqyhfZ.mjs';
-import { localeCodes } from '@saasmakers/shared';
 import 'eslint';
 import 'eslint/use-at-your-own-risk';
 
@@ -517,7 +516,7 @@ const rule$3 = {
   }
 };
 
-const defaultLocales = localeCodes;
+const defaultLocales = ["en", "fr"];
 const indentSpaces = 2;
 const i18nRegex = /(<i18n\s+lang=["']json["']>)([\s\S]*?)<\/i18n>/i;
 const templateRegex$1 = /<template>([\s\S]*)<\/template>/i;
@@ -807,6 +806,11 @@ const templateRegex = /<template>([\s\S]*)<\/template>/i;
 const templateTagLength = "<template>".length;
 const propsPrefixRegex = /\$?props\.(\w+)/g;
 const trueAttributeRegex = /(?<![\w-]):?(?!aria-)([a-z0-9-]+)="true"/gi;
+const eventHandlerRegex = /(?:@|v-on:)[^\s="'<>/]+\s*=\s*"([^"]*)"/g;
+const bareIdentifierRegex = /^[a-z_$][\w$]*$/i;
+const singleCallRegex = /^([a-z_$][\w$]*)\s*\(.*\)$/is;
+const onPrefixRegex = /^on[A-Z]/;
+const exemptCallees = /* @__PURE__ */ new Set(["$emit", "emit"]);
 const dollarTPatterns = [
   {
     pattern: / \$t\(/g,
@@ -832,10 +836,11 @@ const dollarTPatterns = [
 const rule = {
   defaultOptions: [],
   meta: {
-    docs: { description: 'Format Vue templates: strip unnecessary props/$props prefixes, redundant ="true" attributes (except aria- attributes), and enforce t() over $t()' },
+    docs: { description: 'Format Vue templates: strip unnecessary props/$props prefixes, redundant ="true" attributes (except aria- attributes), enforce t() over $t(), and require "on" prefix on named event listener handlers' },
     fixable: "code",
     messages: {
       noPropsPrefix: "Unnecessary props/$props prefix in template. Props are automatically available in template scope.",
+      prefixEventHandlerWithOn: 'Event listener handler "{{name}}" must be prefixed with "on" (e.g. @click="onClick").',
       removeTrueAttribute: 'Unnecessary ="true" attribute. Use the attribute name directly instead.',
       useT: "Use t() instead of $t() in Vue templates as it does not work with <i18n> tags."
     },
@@ -910,6 +915,33 @@ const rule = {
             dollarTMatch = pattern.exec(templateContent);
           }
         }
+        eventHandlerRegex.lastIndex = 0;
+        let eventHandlerMatch = eventHandlerRegex.exec(templateContent);
+        while (eventHandlerMatch !== null) {
+          const handlerValue = eventHandlerMatch[1].trim();
+          let handlerName;
+          if (bareIdentifierRegex.test(handlerValue)) {
+            handlerName = handlerValue;
+          } else {
+            const singleCallMatch = singleCallRegex.exec(handlerValue);
+            if (singleCallMatch) {
+              handlerName = singleCallMatch[1];
+            }
+          }
+          if (handlerName && !exemptCallees.has(handlerName) && !onPrefixRegex.test(handlerName)) {
+            const valueStart = templateStartIndex + eventHandlerMatch.index + eventHandlerMatch[0].length - 1 - eventHandlerMatch[1].length;
+            const valueEnd = valueStart + eventHandlerMatch[1].length;
+            context.report({
+              data: { name: handlerName },
+              loc: {
+                end: sourceCode.getLocFromIndex(valueEnd),
+                start: sourceCode.getLocFromIndex(valueStart)
+              },
+              messageId: "prefixEventHandlerWithOn"
+            });
+          }
+          eventHandlerMatch = eventHandlerRegex.exec(templateContent);
+        }
       }
     };
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saasmakers/eslint",
-  "version": "1.0.22",
+  "version": "1.0.24",
   "private": false,
   "description": "Shared ESLint config and rules for SaaS Makers projects",
   "license": "MIT",
@@ -27,13 +27,13 @@
   ],
   "dependencies": {
     "@antfu/eslint-config": "7.4.3",
-    "@saasmakers/shared": "0.2.8",
     "@unocss/eslint-plugin": "66.7.0",
     "@vitest/eslint-plugin": "1.6.20",
     "eslint-plugin-package-json": "0.31.0",
     "eslint-plugin-turbo": "2.9.18",
     "eslint-plugin-zod": "3.4.0",
-    "typescript-eslint": "8.61.0"
+    "typescript-eslint": "8.61.0",
+    "@saasmakers/shared": "0.2.8"
   },
   "devDependencies": {
     "@eslint/config-inspector": "1.4.2",