@sarj/eslint-plugin 16.14.4 → 16.15.1

package/dist/rule-examples.cjs

@@ -48,6 +48,7 @@ async function verifyRuleExamples(rule, setup = {}) {
     try {
       for (const file of example.files) {
         const path = fixturePath(root, file.path);
+        validateFixtureSyntax(example.id, file, path);
         await (0, import_promises.mkdir)((0, import_node_path.dirname)(path), { recursive: true });
         await (0, import_promises.writeFile)(path, file.source);
       }
@@ -92,6 +93,14 @@ function fixturePath(root, path) {
   if (!resolved.startsWith(root + import_node_path.sep)) throw new Error(`unsafe example path: ${path}`);
   return resolved;
 }
+function validateFixtureSyntax(exampleId, file, path) {
+  if (!/\.[cm]?[jt]sx?$/u.test(file.path)) return;
+  try {
+    parser.parse(file.source, { filePath: path });
+  } catch (error) {
+    throw new Error(`${exampleId}: invalid fixture ${file.path}`, { cause: error });
+  }
+}
 async function verifyExample(linter, config, example, root, ruleId) {
   const focus = example.files.find((file) => file.path === example.focusPath);
   if (focus === void 0) throw new Error(`${example.id}: focus file is missing`);
@@ -100,6 +109,19 @@ async function verifyExample(linter, config, example, root, ruleId) {
   if (messages.some((message) => message.fatal || message.ruleId !== ruleId)) {
     throw new Error(`${example.id}: invalid fixture: ${messages.map((message) => message.message).join("; ")}`);
   }
+  const lines = linter.getSourceCode().lines;
+  if (messages.some((message) => !physicalDiagnosticLocation(message, lines))) {
+    throw new Error(`${example.id}: invalid diagnostic location`);
+  }
+  const locations = messages.map((message) => JSON.stringify([
+    message.ruleId,
+    message.messageId ?? message.message,
+    message.line,
+    message.column,
+    message.endLine,
+    message.endColumn
+  ]));
+  if (new Set(locations).size !== locations.length) throw new Error(`${example.id}: duplicate diagnostic`);
   if (messages.length !== example.expectedCount) {
     throw new Error(`${example.id}: expected ${example.expectedCount} findings, received ${messages.length}`);
   }
@@ -116,6 +138,17 @@ async function verifyExample(linter, config, example, root, ruleId) {
     }
   }
 }
+function physicalDiagnosticLocation(message, lines) {
+  if (!physicalPosition(lines, message.line, message.column)) return false;
+  if (message.endLine === void 0 && message.endColumn === void 0) return true;
+  if (!physicalPosition(lines, message.endLine, message.endColumn)) return false;
+  return message.endLine !== void 0 && message.endColumn !== void 0 && (message.endLine > message.line || message.endLine === message.line && message.endColumn >= message.column);
+}
+function physicalPosition(lines, line, column) {
+  if (line === void 0 || column === void 0 || !Number.isInteger(line) || !Number.isInteger(column)) return false;
+  const text = lines[line - 1];
+  return text !== void 0 && column >= 1 && column <= text.length + 1;
+}
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   verifyRuleExamples

package/dist/rule-examples.js

@@ -14,6 +14,7 @@ async function verifyRuleExamples(rule, setup = {}) {
     try {
       for (const file of example.files) {
         const path = fixturePath(root, file.path);
+        validateFixtureSyntax(example.id, file, path);
         await mkdir(dirname(path), { recursive: true });
         await writeFile(path, file.source);
       }
@@ -58,6 +59,14 @@ function fixturePath(root, path) {
   if (!resolved.startsWith(root + sep)) throw new Error(`unsafe example path: ${path}`);
   return resolved;
 }
+function validateFixtureSyntax(exampleId, file, path) {
+  if (!/\.[cm]?[jt]sx?$/u.test(file.path)) return;
+  try {
+    parser.parse(file.source, { filePath: path });
+  } catch (error) {
+    throw new Error(`${exampleId}: invalid fixture ${file.path}`, { cause: error });
+  }
+}
 async function verifyExample(linter, config, example, root, ruleId) {
   const focus = example.files.find((file) => file.path === example.focusPath);
   if (focus === void 0) throw new Error(`${example.id}: focus file is missing`);
@@ -66,6 +75,19 @@ async function verifyExample(linter, config, example, root, ruleId) {
   if (messages.some((message) => message.fatal || message.ruleId !== ruleId)) {
     throw new Error(`${example.id}: invalid fixture: ${messages.map((message) => message.message).join("; ")}`);
   }
+  const lines = linter.getSourceCode().lines;
+  if (messages.some((message) => !physicalDiagnosticLocation(message, lines))) {
+    throw new Error(`${example.id}: invalid diagnostic location`);
+  }
+  const locations = messages.map((message) => JSON.stringify([
+    message.ruleId,
+    message.messageId ?? message.message,
+    message.line,
+    message.column,
+    message.endLine,
+    message.endColumn
+  ]));
+  if (new Set(locations).size !== locations.length) throw new Error(`${example.id}: duplicate diagnostic`);
   if (messages.length !== example.expectedCount) {
     throw new Error(`${example.id}: expected ${example.expectedCount} findings, received ${messages.length}`);
   }
@@ -82,6 +104,17 @@ async function verifyExample(linter, config, example, root, ruleId) {
     }
   }
 }
+function physicalDiagnosticLocation(message, lines) {
+  if (!physicalPosition(lines, message.line, message.column)) return false;
+  if (message.endLine === void 0 && message.endColumn === void 0) return true;
+  if (!physicalPosition(lines, message.endLine, message.endColumn)) return false;
+  return message.endLine !== void 0 && message.endColumn !== void 0 && (message.endLine > message.line || message.endLine === message.line && message.endColumn >= message.column);
+}
+function physicalPosition(lines, line, column) {
+  if (line === void 0 || column === void 0 || !Number.isInteger(line) || !Number.isInteger(column)) return false;
+  const text = lines[line - 1];
+  return text !== void 0 && column >= 1 && column <= text.length + 1;
+}
 export {
   verifyRuleExamples
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sarj/eslint-plugin",
-  "version": "16.14.4",
+  "version": "16.15.1",
   "packageManager": "npm@12.2.0",
   "description": "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   "type": "module",