@valbuild/eslint-plugin 0.43.1 → 0.43.2

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

@@ -75,7 +75,7 @@ var noIllegalModuleIds = {
               // in a monorepo, the root dir will be the root of the monorepo, not the root of the package
               // so we need to account for that
               // Assume the import of the val.config is correct and that it is in the root folder
-              var numberOfDirsToRoot = valConfigImportSource.split("..").reduce(function (acc, curr) {
+              var numberOfDirsToRoot = valConfigImportSource.split(".." + path__default["default"].sep).reduce(function (acc, curr) {
                 return curr === "" ? acc + 1 : acc;
               }, 0);
               var pathSegments = expectedValue.split(path__default["default"].sep);

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

@@ -75,7 +75,7 @@ var noIllegalModuleIds = {
               // in a monorepo, the root dir will be the root of the monorepo, not the root of the package
               // so we need to account for that
               // Assume the import of the val.config is correct and that it is in the root folder
-              var numberOfDirsToRoot = valConfigImportSource.split("..").reduce(function (acc, curr) {
+              var numberOfDirsToRoot = valConfigImportSource.split(".." + path__default["default"].sep).reduce(function (acc, curr) {
                 return curr === "" ? acc + 1 : acc;
               }, 0);
               var pathSegments = expectedValue.split(path__default["default"].sep);

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

@@ -67,7 +67,7 @@ var noIllegalModuleIds = {
               // in a monorepo, the root dir will be the root of the monorepo, not the root of the package
               // so we need to account for that
               // Assume the import of the val.config is correct and that it is in the root folder
-              var numberOfDirsToRoot = valConfigImportSource.split("..").reduce(function (acc, curr) {
+              var numberOfDirsToRoot = valConfigImportSource.split(".." + path.sep).reduce(function (acc, curr) {
                 return curr === "" ? acc + 1 : acc;
               }, 0);
               var pathSegments = expectedValue.split(path.sep);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@valbuild/eslint-plugin",
-  "version": "0.43.1",
+  "version": "0.43.2",
   "description": "ESLint rules for val",
   "keywords": [
     "eslint",

package/src/rules/noIllegalModuleIds.js

@@ -52,7 +52,7 @@ export default {
               // so we need to account for that
               // Assume the import of the val.config is correct and that it is in the root folder
               const numberOfDirsToRoot = valConfigImportSource
-                .split("..")
+                .split(".." + path.sep)
                 .reduce((acc, curr) => (curr === "" ? acc + 1 : acc), 0);
               const pathSegments = expectedValue.split(path.sep);
               expectedValue = `/${path.join(

package/test/plugin.test.js

@@ -46,4 +46,45 @@ export default val.content(
     expect(results[0].messages).toHaveLength(1);
     expect(results[0].messages[0].fix?.text).toEqual('"/app/test"');
   });
+
+  test("no illegal modules for monorepos (projects that are not at root) - nested", async () => {
+    const code = `import { s, val } from "../../../../val.config";
+
+export const schema = s.string();
+
+export default val.content(
+  "/something",
+  schema,
+  "React Server components also works"
+);`;
+    const results = await eslint.lintText(code, {
+      filePath: "./content/stuff/with/all/test.val.ts",
+    });
+
+    expect(results).toHaveLength(1);
+    expect(results[0].messages).toHaveLength(1);
+    expect(results[0].messages[0].fix?.text).toEqual(
+      '"/content/stuff/with/all/test"'
+    );
+  });
+  test("no illegal modules for monorepos (projects that are not at root) - src", async () => {
+    const code = `import { s, val } from "../../../../val.config";
+
+export const schema = s.string();
+
+export default val.content(
+  "/something",
+  schema,
+  "React Server components also works"
+);`;
+    const results = await eslint.lintText(code, {
+      filePath: "./src/content/stuff/with/all/test.val.ts",
+    });
+
+    expect(results).toHaveLength(1);
+    expect(results[0].messages).toHaveLength(1);
+    expect(results[0].messages[0].fix?.text).toEqual(
+      '"/content/stuff/with/all/test"'
+    );
+  });
 });