@nx/oxlint 23.2.0-pr.36866.449f6f3 → 23.2.0-rc.1

package/dist/src/plugins/plugin.js

@@ -27,6 +27,10 @@ const LINTABLE_EXTENSIONS = [
     'astro',
 ];
 const LINTABLE_FILES_GLOB = `**/*.{${LINTABLE_EXTENSIONS.join(',')}}`;
+// Oxlint honours both, so both decide which files it lints.
+const IGNORE_FILENAMES = ['.eslintignore', '.gitignore'];
+// The one rule that looks across projects; every other rule sees one file.
+const BOUNDARIES_PLUGIN_SPECIFIER = '@nx/oxlint/boundaries-plugin';
 const PROJECT_CONFIG_FILENAMES = ['project.json', 'package.json'];
 const OXLINT_CONFIG_GLOB = combineGlobPatterns([
     ...OXLINT_CONFIG_FILENAMES.map((f) => `**/${f}`),
@@ -116,7 +120,7 @@ export const createNodes = [
                     ...allConfigs.flatMap((config) => localJsPluginFiles(config, jsPluginSpecifiersByConfig.get(config) ?? [])),
                     // Change which files Oxlint considers lintable, and therefore
                     // whether a target is inferred at all.
-                    ...ancestorEslintignorePaths(root),
+                    ...ancestorIgnorePaths(root),
                     lockFilePattern,
                     ...(tsconfigChainsByProjectRoot.get(root) ?? []),
                 ]),
@@ -516,16 +520,18 @@ async function collectLintableFilesByProjectRoot(projectRoots, context) {
     return lintableFilesPerProjectRoot;
 }
 /**
- * `.eslintignore` candidates in every ancestor directory of the project root,
+ * Ignore-file candidates in every ancestor directory of the project root,
  * workspace root included. The project's own directory is excluded — files
- * there are covered by the `default` input — except at the workspace root,
- * which is its own ancestor and so keeps its `.eslintignore`.
+ * there are covered by the in-project ignore-file input — except at the
+ * workspace root, which is its own ancestor and so keeps its ignore files.
  */
-function ancestorEslintignorePaths(projectRoot) {
+function ancestorIgnorePaths(projectRoot) {
     const result = [];
     let dir = projectRoot === '.' ? '.' : dirname(projectRoot);
     while (true) {
-        result.push(dir === '.' ? '.eslintignore' : `${dir}/.eslintignore`);
+        for (const filename of IGNORE_FILENAMES) {
+            result.push(dir === '.' ? filename : `${dir}/${filename}`);
+        }
         if (dir === '.') {
             break;
         }
@@ -572,19 +578,35 @@ function getProjectUsingOxlintConfig(configFilePath, projectRoot, options, conte
     const isRootProject = projectRoot === '.';
     const lintPath = isRootProject && standaloneSrcPath ? `./${standaloneSrcPath}` : '.';
     const jsPluginFiles = new Set(configInputs.flatMap((config) => localJsPluginFiles(config, jsPluginSpecifiersByConfig.get(config) ?? [])));
+    const lintableFiles = `{projectRoot}/${LINTABLE_FILES_GLOB}`;
+    const usesBoundariesBridge = configInputs.some((config) => (jsPluginSpecifiersByConfig.get(config) ?? []).includes(BOUNDARIES_PLUGIN_SPECIFIER));
     const targetConfig = {
         command: `oxlint ${lintPath}`,
         options: { cwd: projectRoot },
         cache: true,
         inputs: [
-            'default',
-            '^default',
+            // Only what Oxlint can lint, so a README or JSON edit is not a re-lint.
+            { fileset: lintableFiles },
+            // The bridge checks the project graph, which a dependency's imports and
+            // package.json shape.
+            ...(usesBoundariesBridge
+                ? [
+                    { fileset: lintableFiles, dependencies: true },
+                    {
+                        fileset: '{projectRoot}/package.json',
+                        dependencies: true,
+                    },
+                ]
+                : []),
+            // Configs, ignore files and tsconfigs inside the project.
+            {
+                fileset: `{projectRoot}/**/{${[...OXLINT_CONFIG_FILENAMES, ...IGNORE_FILENAMES, 'tsconfig*.json'].join(',')}}`,
+            },
             ...configInputs.map((config) => `{workspaceRoot}/${config}`),
             ...[...jsPluginFiles].map((file) => `{workspaceRoot}/${file}`),
-            // Oxlint layers .eslintignore files from every ancestor of a linted
-            // file; the project's own is covered by `default`. Declared even when
-            // absent, like the extends chain.
-            ...ancestorEslintignorePaths(projectRoot).map((file) => `{workspaceRoot}/${file}`),
+            // Oxlint layers ignore files from every ancestor of a linted file.
+            // Declared even when absent, like the extends chain.
+            ...ancestorIgnorePaths(projectRoot).map((file) => `{workspaceRoot}/${file}`),
             ...tsconfigChainOutsideProjectRoot.map((file) => `{workspaceRoot}/${file}`),
             { externalDependencies: ['oxlint'] },
         ],

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@nx/oxlint",
   "description": "Experimental. The Oxlint plugin for Nx contains generators and utilities used for linting JavaScript/TypeScript projects within an Nx workspace.",
-  "version": "23.2.0-pr.36866.449f6f3",
+  "version": "23.2.0-rc.1",
   "type": "module",
   "files": [
     "dist",
@@ -69,9 +69,9 @@
   "generators": "./generators.json",
   "dependencies": {
     "tslib": "^2.3.0",
-    "@nx/devkit": "23.2.0-pr.36866.449f6f3",
-    "@nx/eslint-plugin": "23.2.0-pr.36866.449f6f3",
-    "@nx/js": "23.2.0-pr.36866.449f6f3"
+    "@nx/devkit": "23.2.0-rc.1",
+    "@nx/eslint-plugin": "23.2.0-rc.1",
+    "@nx/js": "23.2.0-rc.1"
   },
   "peerDependencies": {
     "oxlint": "^1.43.0"
@@ -85,6 +85,6 @@
     "node": "^20.19.0 || >=22.12.0"
   },
   "devDependencies": {
-    "nx": "23.2.0-pr.36866.449f6f3"
+    "nx": "23.2.0-rc.1"
   }
 }