fix monorepos oxlint not found

Author: arnriu

README.md

@@ -123,6 +123,24 @@ export default {
 }
 ```
 
+### Change oxlint path
+
+If you're using this plugin in a monorepo and encountering "command not found: oxlint" errors, you can manually specify the path to the oxlint binary.
+
+If no local oxlint binary is found, the plugin will gracefully fall back to using npx (or the equivalent command for your package manager).
+
+```javascript
+import oxlintPlugin from 'vite-plugin-oxlint'
+
+export default {
+  plugins: [
+    oxlintPlugin({
+      oxlintPath: '/path/to/your/monorepo/node_modules/.bin/oxlint'
+    })
+  ]
+}
+```
+
 ## Integration with ESLint
 
 If your project still needs ESLint, you can use [vite-plugin-eslint](https://github.com/gxmari007/vite-plugin-eslint) and configure ESLint with [eslint-plugin-oxlint](https://github.com/oxc-project/eslint-plugin-oxlint) to turn off rules already supported by oxlint.

dist/index.mjs

@@ -7,7 +7,7 @@ const oxlintPlugin = (options = {}) => {
     let timeoutId = null;
     const debounceTime = 300;
     const executeCommand = async () => {
-        const { path = '', ignorePattern = '', configFile = 'oxlintrc.json', deny = [], allow = [], warn = [], params = '' } = options;
+        const { path = '', ignorePattern = '', configFile = 'oxlintrc.json', deny = [], allow = [], warn = [], params = '', oxlintPath = '' } = options;
         const args = [];
         if (ignorePattern) {
             args.push(`--ignore-pattern=${ignorePattern}`);
@@ -27,35 +27,52 @@ const oxlintPlugin = (options = {}) => {
         if (!pm)
             throw new Error('Could not detect package manager');
         return new Promise((resolve, reject) => {
-            const { command: cmd, args: cmdArgs } = resolveCommand(pm.agent, 'execute-local', ['oxlint', ...args]);
-            const child = spawn(cmd, cmdArgs, {
-                cwd,
-                stdio: 'pipe',
-                shell: false,
-                env: {
-                    ...process.env,
-                    FORCE_COLOR: '1'
-                }
-            });
-            child.stdout?.pipe(process.stdout);
-            let stderrOutput = '';
-            child.stderr?.on('data', data => {
-                stderrOutput += data.toString();
-                process.stderr.write(data); // Forward stderr with formatting intact
-            });
-            child.on('error', error => {
-                console.error(`oxlint Error: ${error.message}`);
-                reject(error);
-            });
-            child.on('exit', code => {
-                if (code === 0) {
-                    console.log('\nOxlint successfully finished.');
-                }
-                else {
-                    console.warn(`\n\x1b[33mOxlint finished with exit code: ${code}\x1b[0m`);
-                }
-                resolve();
-            });
+            let isExecuteLocal = true;
+            const executeWithFallback = (useExecuteLocal) => {
+                const { command: cmd, args: cmdArgs } = resolveCommand(pm.agent, useExecuteLocal ? 'execute-local' : 'execute', [useExecuteLocal ? oxlintPath || 'oxlint' : 'oxlint', ...args]);
+                const child = spawn(cmd, cmdArgs, {
+                    cwd,
+                    stdio: 'pipe',
+                    shell: false,
+                    env: {
+                        ...process.env,
+                        FORCE_COLOR: '1'
+                    }
+                });
+                // child.stdout?.pipe(process.stdout)
+                let stderrOutput = '';
+                child.stdout?.on('data', data => {
+                    const dataString = data.toString();
+                    if (!dataString.includes('undefined') &&
+                        !(dataString.includes('not found') && useExecuteLocal)) {
+                        stderrOutput += dataString;
+                        process.stdout.write(data);
+                    }
+                });
+                child.stderr?.on('data', data => {
+                    stderrOutput += data.toString();
+                    process.stderr.write(data);
+                });
+                child.on('error', error => {
+                    console.error(`oxlint Error: ${error.message}`);
+                    reject(error);
+                });
+                child.on('exit', code => {
+                    if (code === 0) {
+                        console.log('\nOxlint successfully finished.');
+                        resolve();
+                    }
+                    else if (useExecuteLocal && code !== 1) {
+                        isExecuteLocal = false;
+                        executeWithFallback(isExecuteLocal);
+                    }
+                    else {
+                        console.warn(`\n\x1b[33mOxlint finished with exit code: ${code}\x1b[0m`);
+                        resolve();
+                    }
+                });
+            };
+            executeWithFallback(isExecuteLocal);
         });
     };
     const handleCommandExecution = async () => {

dist/types.d.ts

@@ -6,4 +6,5 @@ export interface Options {
     allow?: string[];
     warn?: string[];
     params?: string;
+    oxlintPath?: string;
 }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "vite-plugin-oxlint",
-  "version": "1.3.4",
+  "version": "1.4.0",
   "description": "Oxlint plugin for vite.",
   "author": "Arnaud Riu",
   "license": "MIT",

src/index.ts

@@ -18,7 +18,8 @@ const oxlintPlugin = (options: Options = {}): Plugin => {
       deny = [],
       allow = [],
       warn = [],
-      params = ''
+      params = '',
+      oxlintPath = ''
     } = options
 
     const args: string[] = []
@@ -44,45 +45,67 @@ const oxlintPlugin = (options: Options = {}): Plugin => {
     if (!pm) throw new Error('Could not detect package manager')
 
     return new Promise<void>((resolve, reject) => {
-      const { command: cmd, args: cmdArgs } = resolveCommand(
-        pm.agent,
-        'execute-local',
-        ['oxlint', ...args]
-      ) as { command: string; args: string[] }
-
-      const child = spawn(cmd, cmdArgs, {
-        cwd,
-        stdio: 'pipe',
-        shell: false,
-        env: {
-          ...process.env,
-          FORCE_COLOR: '1'
-        }
-      })
-
-      child.stdout?.pipe(process.stdout)
-
-      let stderrOutput = ''
-      child.stderr?.on('data', data => {
-        stderrOutput += data.toString()
-        process.stderr.write(data) // Forward stderr with formatting intact
-      })
-
-      child.on('error', error => {
-        console.error(`oxlint Error: ${error.message}`)
-        reject(error)
-      })
-
-      child.on('exit', code => {
-        if (code === 0) {
-          console.log('\nOxlint successfully finished.')
-        } else {
-          console.warn(
-            `\n\x1b[33mOxlint finished with exit code: ${code}\x1b[0m`
-          )
-        }
-        resolve()
-      })
+      let isExecuteLocal = true
+
+      const executeWithFallback = (useExecuteLocal: boolean) => {
+        const { command: cmd, args: cmdArgs } = resolveCommand(
+          pm.agent,
+          useExecuteLocal ? 'execute-local' : 'execute',
+          [useExecuteLocal ? oxlintPath || 'oxlint' : 'oxlint', ...args]
+        ) as { command: string; args: string[] }
+
+        const child = spawn(cmd, cmdArgs, {
+          cwd,
+          stdio: 'pipe',
+          shell: false,
+          env: {
+            ...process.env,
+            FORCE_COLOR: '1'
+          }
+        })
+
+        // child.stdout?.pipe(process.stdout)
+
+        let stderrOutput = ''
+        child.stdout?.on('data', data => {
+          const dataString = data.toString()
+
+          if (
+            !dataString.includes('undefined') &&
+            !(dataString.includes('not found') && useExecuteLocal)
+          ) {
+            stderrOutput += dataString
+            process.stdout.write(data)
+          }
+        })
+
+        child.stderr?.on('data', data => {
+          stderrOutput += data.toString()
+          process.stderr.write(data)
+        })
+
+        child.on('error', error => {
+          console.error(`oxlint Error: ${error.message}`)
+          reject(error)
+        })
+
+        child.on('exit', code => {
+          if (code === 0) {
+            console.log('\nOxlint successfully finished.')
+            resolve()
+          } else if (useExecuteLocal && code !== 1) {
+            isExecuteLocal = false
+            executeWithFallback(isExecuteLocal)
+          } else {
+            console.warn(
+              `\n\x1b[33mOxlint finished with exit code: ${code}\x1b[0m`
+            )
+            resolve()
+          }
+        })
+      }
+
+      executeWithFallback(isExecuteLocal)
     })
   }
 

src/types.ts

@@ -6,4 +6,5 @@ export interface Options {
   allow?: string[]
   warn?: string[]
   params?: string
+  oxlintPath?: string
 }