@valbuild/eslint-plugin 0.46.0 → 0.47.1

package/dist/valbuild-eslint-plugin.cjs.dev.js

@@ -46,8 +46,7 @@ var noIllegalModuleIds = {
   meta: {
     type: "problem",
     docs: {
-      description: "Check that the first argument of export default declaration matches the string from val.config.ts file.",
-      category: "Best Practices",
+      description: "Check that the first argument of export default declaration matches the string from val.config.{j,t}s file.",
       recommended: true
     },
     fixable: "code",
@@ -106,7 +105,7 @@ var noIllegalModuleIds = {
               if (rawArg) {
                 context.report({
                   node: firstArg,
-                  message: "Val: val.content path should match the filename. Expected: '".concat(expectedValue, "'. Found: '").concat(firstArg.value, "'"),
+                  message: "Val: val.content id should match the filename. Expected: '".concat(expectedValue, "'. Found: '").concat(firstArg.value, "'"),
                   fix: function fix(fixer) {
                     return fixer.replaceText(firstArg, "".concat(rawArg).concat(expectedValue).concat(rawArg));
                   }
@@ -123,49 +122,90 @@ var noIllegalModuleIds = {
 // @ts-check
 
 /**
- * @type {Plugin["rules"]}
+ * @type {import('eslint').Rule.RuleModule}
  */
-var rules = {
-  "no-illegal-module-ids": noIllegalModuleIds
+var noIllegalImports = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Check that val files only has valid imports.",
+      recommended: true
+    },
+    schema: []
+  },
+  create: function create(context) {
+    return {
+      ImportDeclaration: function ImportDeclaration(node) {
+        var importSource = node.source.value;
+        var filename = context.filename || context.getFilename();
+        var isValFile = filename.endsWith(".val.ts") || filename.endsWith(".val.js");
+        // only allow: .val files, @valbuild packages, and val config
+        if (isValFile && typeof importSource === "string" && !importSource.match(/\.val(\.ts|\.js|)$/) && !importSource.match(/^@valbuild/) && !importSource.match(/val\.config(\.ts|\.js|)$/)) {
+          if ("importKind" in node && node["importKind"] !== "type" && !node.specifiers.every(function (s) {
+            return "importKind" in s && s["importKind"] === "type";
+          })) {
+            var message = "Val: can only 'import type' or import from source that is either: a .val.{j,t}s file, a @valbuild package, or val.config.{j,t}s.";
+            context.report({
+              node: node.source,
+              message: message
+            });
+          }
+        }
+      }
+    };
+  }
 };
 
+// @ts-check
+
 /**
- * @type {Plugin["processors"]}
+ * @type {import('eslint').Rule.RuleModule}
  */
-var processors = {
-  val: {
-    /**
-     * @param {string} text
-     * @param {string} filename
-     *  @returns {{ filename: string, text: string }[]}
-     */
-    preprocess: function preprocess(text, filename) {
-      console.log("preprocess", {
-        text: text,
-        filename: filename
-      });
-      return [{
-        text: text,
-        filename: filename
-      }];
+var exportContentMustBeValid = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Export val.content should only happen in .val files.",
+      recommended: true
+    },
+    messages: {
+      "val/export-content-must-be-valid": "Val: val.content should only be exported from .val files"
     },
-    /**
-     * Transforms generated messages for output.
-     * @param {LintMessage[][]} messages An array containing one array of messages
-     *     for each code block returned from `preprocess`.
-     * @param {string} filename The filename of the file
-     * @returns {LintMessage[]} A flattened array of messages with mapped locations.
-     */
-    postprocess: function postprocess(messages, filename) {
-      console.log({
-        messages: messages,
-        filename: filename
-      });
-      return messages.flat();
-    }
+    schema: []
+  },
+  create: function create(context) {
+    return {
+      ExportDefaultDeclaration: function ExportDefaultDeclaration(node) {
+        if (node.declaration && node.declaration.type === "CallExpression" && node.declaration.callee.type === "MemberExpression" && node.declaration.callee.object.type === "Identifier" && node.declaration.callee.object.name === "val" && node.declaration.callee.property.type === "Identifier" && node.declaration.callee.property.name === "content") {
+          var filename = context.filename || context.getFilename();
+          if (!(filename !== null && filename !== void 0 && filename.endsWith(".val.ts") || filename !== null && filename !== void 0 && filename.endsWith(".val.js"))) {
+            context.report({
+              node: node.declaration.callee,
+              messageId: "val/export-content-must-be-valid"
+            });
+          }
+        }
+      }
+    };
   }
 };
 
+// @ts-check
+
+/**
+ * @type {Plugin["rules"]}
+ */
+var rules = {
+  "no-illegal-module-ids": noIllegalModuleIds,
+  "no-illegal-imports": noIllegalImports,
+  "export-content-must-be-valid": exportContentMustBeValid
+};
+
+/**
+ * @type {Plugin["processors"]}
+ */
+var processors = {};
+
 /**
  * @type {Plugin}
  */

package/dist/valbuild-eslint-plugin.cjs.prod.js

@@ -46,8 +46,7 @@ var noIllegalModuleIds = {
   meta: {
     type: "problem",
     docs: {
-      description: "Check that the first argument of export default declaration matches the string from val.config.ts file.",
-      category: "Best Practices",
+      description: "Check that the first argument of export default declaration matches the string from val.config.{j,t}s file.",
       recommended: true
     },
     fixable: "code",
@@ -106,7 +105,7 @@ var noIllegalModuleIds = {
               if (rawArg) {
                 context.report({
                   node: firstArg,
-                  message: "Val: val.content path should match the filename. Expected: '".concat(expectedValue, "'. Found: '").concat(firstArg.value, "'"),
+                  message: "Val: val.content id should match the filename. Expected: '".concat(expectedValue, "'. Found: '").concat(firstArg.value, "'"),
                   fix: function fix(fixer) {
                     return fixer.replaceText(firstArg, "".concat(rawArg).concat(expectedValue).concat(rawArg));
                   }
@@ -123,49 +122,90 @@ var noIllegalModuleIds = {
 // @ts-check
 
 /**
- * @type {Plugin["rules"]}
+ * @type {import('eslint').Rule.RuleModule}
  */
-var rules = {
-  "no-illegal-module-ids": noIllegalModuleIds
+var noIllegalImports = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Check that val files only has valid imports.",
+      recommended: true
+    },
+    schema: []
+  },
+  create: function create(context) {
+    return {
+      ImportDeclaration: function ImportDeclaration(node) {
+        var importSource = node.source.value;
+        var filename = context.filename || context.getFilename();
+        var isValFile = filename.endsWith(".val.ts") || filename.endsWith(".val.js");
+        // only allow: .val files, @valbuild packages, and val config
+        if (isValFile && typeof importSource === "string" && !importSource.match(/\.val(\.ts|\.js|)$/) && !importSource.match(/^@valbuild/) && !importSource.match(/val\.config(\.ts|\.js|)$/)) {
+          if ("importKind" in node && node["importKind"] !== "type" && !node.specifiers.every(function (s) {
+            return "importKind" in s && s["importKind"] === "type";
+          })) {
+            var message = "Val: can only 'import type' or import from source that is either: a .val.{j,t}s file, a @valbuild package, or val.config.{j,t}s.";
+            context.report({
+              node: node.source,
+              message: message
+            });
+          }
+        }
+      }
+    };
+  }
 };
 
+// @ts-check
+
 /**
- * @type {Plugin["processors"]}
+ * @type {import('eslint').Rule.RuleModule}
  */
-var processors = {
-  val: {
-    /**
-     * @param {string} text
-     * @param {string} filename
-     *  @returns {{ filename: string, text: string }[]}
-     */
-    preprocess: function preprocess(text, filename) {
-      console.log("preprocess", {
-        text: text,
-        filename: filename
-      });
-      return [{
-        text: text,
-        filename: filename
-      }];
+var exportContentMustBeValid = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Export val.content should only happen in .val files.",
+      recommended: true
+    },
+    messages: {
+      "val/export-content-must-be-valid": "Val: val.content should only be exported from .val files"
     },
-    /**
-     * Transforms generated messages for output.
-     * @param {LintMessage[][]} messages An array containing one array of messages
-     *     for each code block returned from `preprocess`.
-     * @param {string} filename The filename of the file
-     * @returns {LintMessage[]} A flattened array of messages with mapped locations.
-     */
-    postprocess: function postprocess(messages, filename) {
-      console.log({
-        messages: messages,
-        filename: filename
-      });
-      return messages.flat();
-    }
+    schema: []
+  },
+  create: function create(context) {
+    return {
+      ExportDefaultDeclaration: function ExportDefaultDeclaration(node) {
+        if (node.declaration && node.declaration.type === "CallExpression" && node.declaration.callee.type === "MemberExpression" && node.declaration.callee.object.type === "Identifier" && node.declaration.callee.object.name === "val" && node.declaration.callee.property.type === "Identifier" && node.declaration.callee.property.name === "content") {
+          var filename = context.filename || context.getFilename();
+          if (!(filename !== null && filename !== void 0 && filename.endsWith(".val.ts") || filename !== null && filename !== void 0 && filename.endsWith(".val.js"))) {
+            context.report({
+              node: node.declaration.callee,
+              messageId: "val/export-content-must-be-valid"
+            });
+          }
+        }
+      }
+    };
   }
 };
 
+// @ts-check
+
+/**
+ * @type {Plugin["rules"]}
+ */
+var rules = {
+  "no-illegal-module-ids": noIllegalModuleIds,
+  "no-illegal-imports": noIllegalImports,
+  "export-content-must-be-valid": exportContentMustBeValid
+};
+
+/**
+ * @type {Plugin["processors"]}
+ */
+var processors = {};
+
 /**
  * @type {Plugin}
  */

package/dist/valbuild-eslint-plugin.esm.js

@@ -38,8 +38,7 @@ var noIllegalModuleIds = {
   meta: {
     type: "problem",
     docs: {
-      description: "Check that the first argument of export default declaration matches the string from val.config.ts file.",
-      category: "Best Practices",
+      description: "Check that the first argument of export default declaration matches the string from val.config.{j,t}s file.",
       recommended: true
     },
     fixable: "code",
@@ -98,7 +97,7 @@ var noIllegalModuleIds = {
               if (rawArg) {
                 context.report({
                   node: firstArg,
-                  message: "Val: val.content path should match the filename. Expected: '".concat(expectedValue, "'. Found: '").concat(firstArg.value, "'"),
+                  message: "Val: val.content id should match the filename. Expected: '".concat(expectedValue, "'. Found: '").concat(firstArg.value, "'"),
                   fix: function fix(fixer) {
                     return fixer.replaceText(firstArg, "".concat(rawArg).concat(expectedValue).concat(rawArg));
                   }
@@ -115,49 +114,90 @@ var noIllegalModuleIds = {
 // @ts-check
 
 /**
- * @type {Plugin["rules"]}
+ * @type {import('eslint').Rule.RuleModule}
  */
-var rules = {
-  "no-illegal-module-ids": noIllegalModuleIds
+var noIllegalImports = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Check that val files only has valid imports.",
+      recommended: true
+    },
+    schema: []
+  },
+  create: function create(context) {
+    return {
+      ImportDeclaration: function ImportDeclaration(node) {
+        var importSource = node.source.value;
+        var filename = context.filename || context.getFilename();
+        var isValFile = filename.endsWith(".val.ts") || filename.endsWith(".val.js");
+        // only allow: .val files, @valbuild packages, and val config
+        if (isValFile && typeof importSource === "string" && !importSource.match(/\.val(\.ts|\.js|)$/) && !importSource.match(/^@valbuild/) && !importSource.match(/val\.config(\.ts|\.js|)$/)) {
+          if ("importKind" in node && node["importKind"] !== "type" && !node.specifiers.every(function (s) {
+            return "importKind" in s && s["importKind"] === "type";
+          })) {
+            var message = "Val: can only 'import type' or import from source that is either: a .val.{j,t}s file, a @valbuild package, or val.config.{j,t}s.";
+            context.report({
+              node: node.source,
+              message: message
+            });
+          }
+        }
+      }
+    };
+  }
 };
 
+// @ts-check
+
 /**
- * @type {Plugin["processors"]}
+ * @type {import('eslint').Rule.RuleModule}
  */
-var processors = {
-  val: {
-    /**
-     * @param {string} text
-     * @param {string} filename
-     *  @returns {{ filename: string, text: string }[]}
-     */
-    preprocess: function preprocess(text, filename) {
-      console.log("preprocess", {
-        text: text,
-        filename: filename
-      });
-      return [{
-        text: text,
-        filename: filename
-      }];
+var exportContentMustBeValid = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Export val.content should only happen in .val files.",
+      recommended: true
+    },
+    messages: {
+      "val/export-content-must-be-valid": "Val: val.content should only be exported from .val files"
     },
-    /**
-     * Transforms generated messages for output.
-     * @param {LintMessage[][]} messages An array containing one array of messages
-     *     for each code block returned from `preprocess`.
-     * @param {string} filename The filename of the file
-     * @returns {LintMessage[]} A flattened array of messages with mapped locations.
-     */
-    postprocess: function postprocess(messages, filename) {
-      console.log({
-        messages: messages,
-        filename: filename
-      });
-      return messages.flat();
-    }
+    schema: []
+  },
+  create: function create(context) {
+    return {
+      ExportDefaultDeclaration: function ExportDefaultDeclaration(node) {
+        if (node.declaration && node.declaration.type === "CallExpression" && node.declaration.callee.type === "MemberExpression" && node.declaration.callee.object.type === "Identifier" && node.declaration.callee.object.name === "val" && node.declaration.callee.property.type === "Identifier" && node.declaration.callee.property.name === "content") {
+          var filename = context.filename || context.getFilename();
+          if (!(filename !== null && filename !== void 0 && filename.endsWith(".val.ts") || filename !== null && filename !== void 0 && filename.endsWith(".val.js"))) {
+            context.report({
+              node: node.declaration.callee,
+              messageId: "val/export-content-must-be-valid"
+            });
+          }
+        }
+      }
+    };
   }
 };
 
+// @ts-check
+
+/**
+ * @type {Plugin["rules"]}
+ */
+var rules = {
+  "no-illegal-module-ids": noIllegalModuleIds,
+  "no-illegal-imports": noIllegalImports,
+  "export-content-must-be-valid": exportContentMustBeValid
+};
+
+/**
+ * @type {Plugin["processors"]}
+ */
+var processors = {};
+
 /**
  * @type {Plugin}
  */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@valbuild/eslint-plugin",
-  "version": "0.46.0",
+  "version": "0.47.1",
   "description": "ESLint rules for val",
   "keywords": [
     "eslint",
@@ -27,7 +27,8 @@
   },
   "devDependencies": {
     "@types/jest": "^29.5.11",
-    "eslint": "^7.10.0",
+    "@typescript-eslint/rule-tester": "^6.15.0",
+    "eslint": "^8.56.0",
     "jest": "^29.6"
   },
   "scripts": {

package/src/index.js

@@ -7,41 +7,22 @@
  */
 
 import noIllegalModuleIds from "./rules/noIllegalModuleIds";
+import noIllegalImports from "./rules/noIllegalImports";
+import exportContentMustBeValid from "./rules/exportContentMustBeValid";
 
 /**
  * @type {Plugin["rules"]}
  */
 export let rules = {
   "no-illegal-module-ids": noIllegalModuleIds,
+  "no-illegal-imports": noIllegalImports,
+  "export-content-must-be-valid": exportContentMustBeValid,
 };
 
 /**
  * @type {Plugin["processors"]}
  */
-export const processors = {
-  val: {
-    /**
-     * @param {string} text
-     * @param {string} filename
-     *  @returns {{ filename: string, text: string }[]}
-     */
-    preprocess: (text, filename) => {
-      console.log("preprocess", { text, filename });
-      return [{ text, filename }];
-    },
-    /**
-     * Transforms generated messages for output.
-     * @param {LintMessage[][]} messages An array containing one array of messages
-     *     for each code block returned from `preprocess`.
-     * @param {string} filename The filename of the file
-     * @returns {LintMessage[]} A flattened array of messages with mapped locations.
-     */
-    postprocess: (messages, filename) => {
-      console.log({ messages, filename });
-      return messages.flat();
-    },
-  },
-};
+export const processors = {};
 
 /**
  * @type {Plugin}

package/src/rules/exportContentMustBeValid.js

@@ -0,0 +1,44 @@
+// @ts-check
+
+/**
+ * @type {import('eslint').Rule.RuleModule}
+ */
+export default {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Export val.content should only happen in .val files.",
+      recommended: true,
+    },
+    messages: {
+      "val/export-content-must-be-valid":
+        "Val: val.content should only be exported from .val files",
+    },
+    schema: [],
+  },
+  create: function (context) {
+    return {
+      ExportDefaultDeclaration(node) {
+        if (
+          node.declaration &&
+          node.declaration.type === "CallExpression" &&
+          node.declaration.callee.type === "MemberExpression" &&
+          node.declaration.callee.object.type === "Identifier" &&
+          node.declaration.callee.object.name === "val" &&
+          node.declaration.callee.property.type === "Identifier" &&
+          node.declaration.callee.property.name === "content"
+        ) {
+          const filename = context.filename || context.getFilename();
+          if (
+            !(filename?.endsWith(".val.ts") || filename?.endsWith(".val.js"))
+          ) {
+            context.report({
+              node: node.declaration.callee,
+              messageId: "val/export-content-must-be-valid",
+            });
+          }
+        }
+      },
+    };
+  },
+};

package/src/rules/noIllegalImports.js

@@ -0,0 +1,48 @@
+// @ts-check
+
+/**
+ * @type {import('eslint').Rule.RuleModule}
+ */
+export default {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Check that val files only has valid imports.",
+      recommended: true,
+    },
+    schema: [],
+  },
+  create: function (context) {
+    return {
+      ImportDeclaration(node) {
+        const importSource = node.source.value;
+        const filename = context.filename || context.getFilename();
+
+        const isValFile =
+          filename.endsWith(".val.ts") || filename.endsWith(".val.js");
+        // only allow: .val files, @valbuild packages, and val config
+        if (
+          isValFile &&
+          typeof importSource === "string" &&
+          !importSource.match(/\.val(\.ts|\.js|)$/) &&
+          !importSource.match(/^@valbuild/) &&
+          !importSource.match(/val\.config(\.ts|\.js|)$/)
+        ) {
+          if (
+            "importKind" in node &&
+            node["importKind"] !== "type" &&
+            !node.specifiers.every(
+              (s) => "importKind" in s && s["importKind"] === "type"
+            )
+          ) {
+            const message = `Val: can only 'import type' or import from source that is either: a .val.{j,t}s file, a @valbuild package, or val.config.{j,t}s.`;
+            context.report({
+              node: node.source,
+              message,
+            });
+          }
+        }
+      },
+    };
+  },
+};

package/src/rules/noIllegalModuleIds.js

@@ -9,8 +9,7 @@ export default {
     type: "problem",
     docs: {
       description:
-        "Check that the first argument of export default declaration matches the string from val.config.ts file.",
-      category: "Best Practices",
+        "Check that the first argument of export default declaration matches the string from val.config.{j,t}s file.",
       recommended: true,
     },
     fixable: "code",
@@ -92,7 +91,7 @@ export default {
               if (rawArg) {
                 context.report({
                   node: firstArg,
-                  message: `Val: val.content path should match the filename. Expected: '${expectedValue}'. Found: '${firstArg.value}'`,
+                  message: `Val: val.content id should match the filename. Expected: '${expectedValue}'. Found: '${firstArg.value}'`,
                   fix: (fixer) =>
                     fixer.replaceText(
                       firstArg,

package/test/rules/exportContentMustBeValid.test.js

@@ -0,0 +1,56 @@
+import { RuleTester } from "@typescript-eslint/rule-tester";
+import { rules as valRules } from "@valbuild/eslint-plugin";
+import path from "path";
+
+const rule = valRules["export-content-must-be-valid"];
+
+RuleTester.setDefaultConfig({
+  parserOptions: {
+    ecmaVersion: 2018,
+    sourceType: "module",
+    ecmaFeatures: {},
+  },
+});
+
+const ruleTester = new RuleTester();
+
+ruleTester.run("export-content-must-be-valid", rule, {
+  valid: [
+    {
+      filename: path.join(process.cwd(), "./foo/test.val.ts"),
+      code: `
+import { val, s } from '../val.config';
+
+export const schema = s.string();
+export default val.content('/foo/test', schema, '')`,
+    },
+  ],
+  invalid: [
+    {
+      filename: path.join(process.cwd(), "./foo/test.ts"),
+      code: `
+import { val, s } from '../val.config';
+
+export const schema = s.string();
+export default val.content('/foo/test', schema, '')`,
+      errors: [
+        {
+          message: "Val: val.content should only be exported from .val files",
+        },
+      ],
+    },
+    {
+      filename: path.join(process.cwd(), "./foo/test.js"),
+      code: `
+import { val, s } from '../val.config';
+
+export const schema = s.string();
+export default val.content('/foo/test', schema, '')`,
+      errors: [
+        {
+          message: "Val: val.content should only be exported from .val files",
+        },
+      ],
+    },
+  ],
+});

package/test/rules/noIllegalImports.test.js

@@ -0,0 +1,100 @@
+import { RuleTester } from "@typescript-eslint/rule-tester";
+import { rules as valRules } from "@valbuild/eslint-plugin";
+import path from "path";
+
+const rule = valRules["no-illegal-imports"];
+
+RuleTester.setDefaultConfig({
+  parserOptions: {
+    ecmaVersion: 2018,
+    sourceType: "module",
+
+    ecmaFeatures: {},
+  },
+});
+
+const ruleTester = new RuleTester();
+
+ruleTester.run("no-illegal-imports", rule, {
+  valid: [
+    {
+      filename: path.join(process.cwd(), "./foo/test.val.ts"),
+      code: `
+import { val, s } from '../val.config';
+import { eventSchema } from './event.val';
+
+export const schema = s.array(eventSchema);
+export default val.content('/foo/test', schema, [])`,
+    },
+    {
+      filename: path.join(process.cwd(), "./foo/test.val.ts"),
+      code: `
+import { val, s } from '../val.config.ts';
+import { eventSchema } from './event.val.ts';
+
+export const schema = s.array(eventSchema);
+export default val.content('/foo/test', schema, [])`,
+    },
+    {
+      filename: path.join(process.cwd(), "./foo/test.val.ts"),
+      code: `
+import { val, s } from '../val.config.ts';
+import type { Event } from './eventSchema';
+
+export const schema = s.array(s.string());
+type Test = Event;
+export default val.content('/foo/test', schema, [])`,
+    },
+    {
+      filename: path.join(process.cwd(), "./foo/test.val.ts"),
+      code: `
+import { val, s } from '../val.config.ts';
+import { type Event } from './eventSchema';
+
+export const schema = s.array(s.string());
+type Test = Event;
+export default val.content('/foo/test', schema, [])`,
+    },
+    {
+      filename: path.join(process.cwd(), "./foo/test.val.ts"),
+      code: `
+import { val, s } from '../val.config.ts';
+
+export const schema = s.string();
+export default val.content('/foo/test', schema, 'String')`,
+    },
+  ],
+  invalid: [
+    {
+      filename: path.join(process.cwd(), "./foo/test.val.ts"),
+      code: `
+import { val, s } from '../val.config';
+import { eventSchema } from './event';
+
+export const schema = s.array(eventSchema);
+export default val.content('/foo/test', schema, [])`,
+      errors: [
+        {
+          message:
+            "Val: can only 'import type' or import from source that is either: a .val.{j,t}s file, a @valbuild package, or val.config.{j,t}s.",
+        },
+      ],
+    },
+    {
+      filename: path.join(process.cwd(), "./foo/test.val.ts"),
+      code: `
+import { val, s } from '../val.config';
+import { eventSchema, type Unused } from './event';
+
+export const schema = s.array(eventSchema);
+type Event = Unused;
+export default val.content('/foo/test', schema, [])`,
+      errors: [
+        {
+          message:
+            "Val: can only 'import type' or import from source that is either: a .val.{j,t}s file, a @valbuild package, or val.config.{j,t}s.",
+        },
+      ],
+    },
+  ],
+});

package/test/rules/noIllegalModuleIds.test.js

@@ -34,7 +34,7 @@ ruleTester.run("no-illegal-module-ids", rule, {
       errors: [
         {
           message:
-            "Val: val.content path should match the filename. Expected: '/foo/test'. Found: 'foo'",
+            "Val: val.content id should match the filename. Expected: '/foo/test'. Found: 'foo'",
         },
       ],
       output: `import { val, s } from '../val.config.ts';
@@ -49,7 +49,7 @@ ruleTester.run("no-illegal-module-ids", rule, {
       errors: [
         {
           message:
-            "Val: val.content path should match the filename. Expected: '/foo/test'. Found: 'foo'",
+            "Val: val.content id should match the filename. Expected: '/foo/test'. Found: 'foo'",
         },
       ],
       output: `import { val, s } from "../val.config.ts";