@saasmakers/eslint 1.0.21 → 1.0.23

package/dist/eslint.config.cjs

@@ -2,6 +2,7 @@
 
 const antfu = require('@antfu/eslint-config');
 const saasmakers = require('@saasmakers/eslint');
+const shared = require('@saasmakers/shared');
 const vitest = require('@vitest/eslint-plugin');
 const packageJson = require('eslint-plugin-package-json');
 const index$1 = require('./shared/eslint.BqRQ4tAN.cjs');
@@ -10869,7 +10870,7 @@ const eslint_config = antfu__default(
     rules: {
       "saasmakers/ts-format-layout": "error",
       "saasmakers/ts-format-tests": "error",
-      "saasmakers/vue-format-i18n": "error",
+      "saasmakers/vue-format-i18n": ["error", { locales: shared.localeCodes }],
       "saasmakers/vue-format-script": "error",
       "saasmakers/vue-format-template": "error"
     }

package/dist/eslint.config.d.cts

@@ -1,5 +1,6 @@
 import antfu from '@antfu/eslint-config';
 import saasmakers from '@saasmakers/eslint';
+import { localeCodes } from '@saasmakers/shared';
 import vitest from '@vitest/eslint-plugin';
 import packageJson from 'eslint-plugin-package-json';
 import { Linter, ESLint } from 'eslint';
@@ -149,7 +150,7 @@ var eslint_config = antfu(
     rules: {
       "saasmakers/ts-format-layout": "error",
       "saasmakers/ts-format-tests": "error",
-      "saasmakers/vue-format-i18n": "error",
+      "saasmakers/vue-format-i18n": ["error", { locales: localeCodes }],
       "saasmakers/vue-format-script": "error",
       "saasmakers/vue-format-template": "error"
     }

package/dist/eslint.config.d.mts

@@ -1,5 +1,6 @@
 import antfu from '@antfu/eslint-config';
 import saasmakers from '@saasmakers/eslint';
+import { localeCodes } from '@saasmakers/shared';
 import vitest from '@vitest/eslint-plugin';
 import packageJson from 'eslint-plugin-package-json';
 import { Linter, ESLint } from 'eslint';
@@ -149,7 +150,7 @@ var eslint_config = antfu(
     rules: {
       "saasmakers/ts-format-layout": "error",
       "saasmakers/ts-format-tests": "error",
-      "saasmakers/vue-format-i18n": "error",
+      "saasmakers/vue-format-i18n": ["error", { locales: localeCodes }],
       "saasmakers/vue-format-script": "error",
       "saasmakers/vue-format-template": "error"
     }

package/dist/eslint.config.d.ts

@@ -1,5 +1,6 @@
 import antfu from '@antfu/eslint-config';
 import saasmakers from '@saasmakers/eslint';
+import { localeCodes } from '@saasmakers/shared';
 import vitest from '@vitest/eslint-plugin';
 import packageJson from 'eslint-plugin-package-json';
 import { Linter, ESLint } from 'eslint';
@@ -149,7 +150,7 @@ var eslint_config = antfu(
     rules: {
       "saasmakers/ts-format-layout": "error",
       "saasmakers/ts-format-tests": "error",
-      "saasmakers/vue-format-i18n": "error",
+      "saasmakers/vue-format-i18n": ["error", { locales: localeCodes }],
       "saasmakers/vue-format-script": "error",
       "saasmakers/vue-format-template": "error"
     }

package/dist/eslint.config.mjs

@@ -1,5 +1,6 @@
 import antfu from '@antfu/eslint-config';
 import saasmakers from '@saasmakers/eslint';
+import { localeCodes } from '@saasmakers/shared';
 import vitest from '@vitest/eslint-plugin';
 import packageJson from 'eslint-plugin-package-json';
 import { d as distExports } from './shared/eslint.DOaqyhfZ.mjs';
@@ -10843,7 +10844,7 @@ const eslint_config = antfu(
     rules: {
       "saasmakers/ts-format-layout": "error",
       "saasmakers/ts-format-tests": "error",
-      "saasmakers/vue-format-i18n": "error",
+      "saasmakers/vue-format-i18n": ["error", { locales: localeCodes }],
       "saasmakers/vue-format-script": "error",
       "saasmakers/vue-format-template": "error"
     }

package/dist/index.cjs

@@ -281,6 +281,9 @@ const rule$4 = {
     type: "layout"
   },
   create(context) {
+    if (context.filename.endsWith(".d.ts")) {
+      return {};
+    }
     const sourceCode = context.sourceCode;
     const maxCharacters = context.options[0]?.maxCharacters ?? defaultMaxCharacters;
     const maxLineLength = context.options[0]?.maxLineLength ?? defaultMaxLineLength;
@@ -805,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-Za-z_$][\w$]*$/;
+const singleCallRegex = /^([A-Za-z_$][\w$]*)\s*\(.*\)$/s;
+const onPrefixRegex = /^on[A-Z]/;
+const exemptCallees = /* @__PURE__ */ new Set(["$emit", "emit"]);
 const dollarTPatterns = [
   {
     pattern: / \$t\(/g,
@@ -830,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."
     },
@@ -908,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

@@ -279,6 +279,9 @@ const rule$4 = {
     type: "layout"
   },
   create(context) {
+    if (context.filename.endsWith(".d.ts")) {
+      return {};
+    }
     const sourceCode = context.sourceCode;
     const maxCharacters = context.options[0]?.maxCharacters ?? defaultMaxCharacters;
     const maxLineLength = context.options[0]?.maxLineLength ?? defaultMaxLineLength;
@@ -803,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-Za-z_$][\w$]*$/;
+const singleCallRegex = /^([A-Za-z_$][\w$]*)\s*\(.*\)$/s;
+const onPrefixRegex = /^on[A-Z]/;
+const exemptCallees = /* @__PURE__ */ new Set(["$emit", "emit"]);
 const dollarTPatterns = [
   {
     pattern: / \$t\(/g,
@@ -828,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."
     },
@@ -906,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.21",
+  "version": "1.0.23",
   "private": false,
   "description": "Shared ESLint config and rules for SaaS Makers projects",
   "license": "MIT",
@@ -32,7 +32,8 @@
     "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",