dan/oxc
1
0
Fork 0
mirror of https://github.com/danbulant/oxc synced 2026-05-19 04:08:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(linter): can't disable no-nested-ternary rule anymore (#8600)

Browse source 
closes #8485 

Since we currently support two rules with the same `rule_name` but
different `plugin_names`, some of the original logic is no longer
applicable. As a result, I have made some adjustments.
This commit is contained in:
dalaoshu 2025-01-31 21:03:53 +08:00 committed by GitHub
parent e929f26826
commit 8ce21d1d48
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
5 changed files with 72 additions and 32 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
apps/oxlint/fixtures/two_rules_with_same_name/.oxlintrc.json Normal file
View file

@ -0,0 +1,10 @@
{
"plugins": [
"oxc",
"unicorn"
],
"rules": {
"eslint/no-nested-ternary": "off",
"unicorn/no-nested-ternary": "off"
}
}

1
apps/oxlint/fixtures/two_rules_with_same_name/test.js Normal file
View file

@ -0,0 +1 @@
console.log(bar ? baz : qux === quxx ? bing : bam);

7
apps/oxlint/src/lint.rs
View file

@ -824,6 +824,13 @@ mod test {
.test_and_snapshot_multiple(&[args_1, args_2]);
}
#[test]
fn test_two_rules_with_same_name_from_different_plugin_names() {
// Issue: <https://github.com/oxc-project/oxc/issues/8485>
let args = &["-c", ".oxlintrc.json", "test.js"];
Tester::new().with_cwd("fixtures/two_rules_with_same_name".into()).test_and_snapshot(args);
}
#[test]
fn test_adjust_ignore_patterns() {
let base = PathBuf::from("/project/root");

12
apps/oxlint/src/snapshots/fixtures__two_rules_with_same_name_-c .oxlintrc.json test.js@oxlint.snap Normal file
View file

@ -0,0 +1,12 @@
---
source: apps/oxlint/src/tester.rs
---
##########
arguments: -c .oxlintrc.json test.js
working directory: fixtures/two_rules_with_same_name
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 1 file with 74 rules using 1 threads.
----------
CLI result: LintSucceeded
----------

74
crates/oxc_linter/src/config/rules.rs
View file

@ -110,28 +110,43 @@ impl OxlintRules {
}
}
_ => {
// For overlapping rule names, use the "error" one
// "no-loss-of-precision": "off",
// "@typescript-eslint/no-loss-of-precision": "error"
if let Some(rule_config) =
rule_configs.iter().find(|r| r.severity.is_warn_deny())
{
let config = rule_config.config.clone().unwrap_or_default();
let rules = rules_for_override
.iter()
.filter_map(|r| {
if r.name() == *name {
Some((r.plugin_name(), r))
} else {
None
}
})
.collect::<FxHashMap<_, _>>();
if let Some(rule) = rules_for_override.iter().find(|r| r.name() == *name) {
rules_to_replace
.push(RuleWithSeverity::new(rule.read_json(config), rule.severity));
}
// If the given rule is not found in the rule list (for example, if all rules are disabled),
// then look it up in the entire rules list and add it.
else if let Some(rule) = all_rules.iter().find(|r| r.name() == *name) {
rules_to_replace.push(RuleWithSeverity::new(
rule.read_json(config),
rule_config.severity,
));
}
} else if rule_configs.iter().all(|r| r.severity.is_allow()) {
if let Some(rule) = rules_for_override.iter().find(|r| r.name() == *name) {
for rule_config in rule_configs {
let (rule_name, plugin_name) = transform_rule_and_plugin_name(
&rule_config.rule_name,
&rule_config.plugin_name,
);
if rule_config.severity.is_warn_deny() {
let config = rule_config.config.clone().unwrap_or_default();
if let Some(rule) = rules.get(&plugin_name) {
rules_to_replace.push(RuleWithSeverity::new(
rule.read_json(config),
rule.severity,
));
}
// If the given rule is not found in the rule list (for example, if all rules are disabled),
// then look it up in the entire rules list and add it.
else if let Some(rule) = all_rules
.iter()
.find(|r| r.name() == rule_name && r.plugin_name() == plugin_name)
{
rules_to_replace.push(RuleWithSeverity::new(
rule.read_json(config),
rule_config.severity,
));
}
} else if let Some(&rule) = rules.get(&plugin_name) {
rules_to_remove.push(rule.clone());
}
}
@ -152,17 +167,12 @@ fn transform_rule_and_plugin_name<'a>(
rule_name: &'a str,
plugin_name: &'a str,
) -> (&'a str, &'a str) {
if plugin_name == "vitest" && is_jest_rule_adapted_to_vitest(rule_name) {
return (rule_name, "jest");
}
if plugin_name == "typescript" && is_eslint_rule_adapted_to_typescript(rule_name) {
return (rule_name, "eslint");
}
if plugin_name == "unicorn" && rule_name == "no-negated-condition" {
return ("no-negated-condition", "eslint");
}
let plugin_name = match plugin_name {
"vitest" if is_jest_rule_adapted_to_vitest(rule_name) => "jest",
"unicorn" if rule_name == "no-negated-condition" => "eslint",
"typescript" if is_eslint_rule_adapted_to_typescript(rule_name) => "eslint",
_ => plugin_name,
};
(rule_name, plugin_name)
}