eslint-plugin-slop

🔗 Linked issue closes #8 🧭 Context For users: easy access rule's docs For devs: easy inspect the rule 📚 Description Just added back manually for each rules, can try for automating it like if needed

Describe the bug can u add the url to the rules so the editor shows a clickable link? see repro below, the url is not part of the is included in your other plugins, e.g. but not in Reproduction see podman command System Info Used Package Manager npm Validations [x] Follow our Code of Conduct [x] Read the Contributing Guide. [x] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate. [x] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead. [x] The provided reproduction is a minimal reproducible of the bug. Contributions [ ] I am willing to submit a PR to fix this issue [ ] I am willing to submit a PR with failing tests (actually just go ahead and do it, thanks!)

Describe the bug Description inspection is significantly slower in a large repository(about 200k LOC and 1500 files), likely because it synchronously executes Git commands for each inspected file. I know this is an ESLint plugin, but oxlint is compactible with it, so I used it anyway. All measurements below were made through Oxlint's JS plugin support. I have not benchmarked ESLint itself. Environment eslint-plugin-slop: 0.1.2 Oxlint: 1.80.0 The same Git inspection implementation is currently present on : Results Average of three warm runs on the same codebase: Suspected cause performs the following work for each source file: synchronously runs this for every file: then starts another synchronous Git process: avoids loading the same file twice, but its key contains , so a full lint still starts approximately one process per file. The baseline revision itself is not cached. Tested optimizations I tested two local modifications to the installed plugin: Diagnostics remained identical, and an untracked test file was still reported correctly. Suggested fix Cache the baseline revision and changed-file set, then avoid loading baseline content for unchanged files. Cache invalidation should account for HEAD and working-tree changes in long-running ESLint or editor processes. Reproduction https://github.com/rrbe/eslint-plugin-slop-oxlint-perf-repro System Info Used Package Manager pnpm Validations [x] Follow our Code of Conduct [x] Read the Contributing Guide. [x] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate. [x] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead. [x] The provided reproduction is a minimal reproducible of the bug. Contributions [x] I am willing to submit a PR to fix this issue [x] I am willing to submit a PR with failing tests (actually just go ahead and do it, thanks!)