Linter: "ignorePatterns" not causing files to be ignored when . is passed in to CLI

[nolanlawson]

Thanks for making Oxc! It's fast and very helpful. 🙂

I've noticed that, in certain cases, "ignorePatterns" in .oxlintrc.json does not ignore certain files when running oxlint --fix. This only seems to occur with deep directories, and only when adding . to the end of the CLI command.

Here is a minimal repro of the issue:

git clone git@github.com:nolanlawson/repro-oxlint-ignore-issue.git repro
cd repro
npm i
npm run lint

Here is what's happening. "ignorePatterns":

  "ignorePatterns": [
    "foo.js",
    "a/bar.js"
  ],

This works, and ignores both files:

oxlint -c .oxlintrc.json --fix

However, adding . to the end of the command causes bar.js to no longer be ignored:

oxlint -c .oxlintrc.json --fix .

If you run this second command, you'll see that foo.js is correctly ignored, but a/bar.js is not:

--- a/a/bar.js
+++ b/a/bar.js
@@ -1,2 +1,2 @@
-debugger
+
 console.log("Don't lint me!!")

[shulaoda]

I am unable to reproduce your issue:

PS D:\shulaoda\repro-oxlint-ignore-issue> pnpm oxlint -c .oxlintrc.json --fix
Found 0 warnings and 0 errors.
Finished in 2ms on 0 files with 61 rules using 20 threads.
PS D:\shulaoda\repro-oxlint-ignore-issue> pnpm oxlint -c .oxlintrc.json --fix .
Found 0 warnings and 0 errors.
Finished in 2ms on 0 files with 61 rules using 20 threads.

Could you tell me the operating system you are using?

[nolanlawson]

I'm using macOS.

[BenShelton]

Can recreate on mac, happy to take this on 😄