Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat(linter): eslint-plugin-react checked-requires-onchange-or-readonly
- Loading branch information
1 parent
5edd196
commit 3f1104e
Showing
3 changed files
with
362 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
259 changes: 259 additions & 0 deletions
259
crates/oxc_linter/src/rules/react/checked_requires_onchange_or_readonly.rs
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,259 @@ | ||
use oxc_ast::{ | ||
ast::{Argument, Expression, JSXAttributeItem, ObjectPropertyKind}, | ||
AstKind, | ||
}; | ||
use oxc_diagnostics::{ | ||
miette::{self, Diagnostic}, | ||
thiserror::Error, | ||
}; | ||
use oxc_macros::declare_oxc_lint; | ||
use oxc_span::Span; | ||
use phf::phf_set; | ||
|
||
use crate::{ | ||
context::LintContext, | ||
rule::Rule, | ||
utils::{get_element_type, get_jsx_attribute_name, is_create_element_call}, | ||
AstNode, | ||
}; | ||
|
||
#[derive(Debug, Error, Diagnostic)] | ||
enum CheckedRequiresOnchangeOrReadonlyDiagnostic { | ||
#[error("eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`.")] | ||
#[diagnostic(severity(warning), help("Add either `onChange` or `readOnly`."))] | ||
MissingProperty(#[label] Span), | ||
|
||
#[error("eslint-plugin-react(checked-requires-onchange-or-readonly): Use either `checked` or `defaultChecked`, but not both.")] | ||
#[diagnostic(severity(warning), help("Remove either `checked` or `defaultChecked`."))] | ||
ExclusiveCheckedAttribute(#[label] Span), | ||
} | ||
|
||
#[derive(Debug, Default, Clone)] | ||
pub struct CheckedRequiresOnchangeOrReadonly { | ||
ignore_missing_properties: bool, | ||
ignore_exclusive_checked_attribute: bool, | ||
} | ||
|
||
declare_oxc_lint!( | ||
/// ### What it does | ||
/// This rule enforces onChange or readonly attribute for checked property of input elements. | ||
/// It also warns when checked and defaultChecked properties are used together. | ||
/// | ||
/// ### Example | ||
/// ```javascript | ||
/// // Bad | ||
/// <input type="checkbox" checked /> | ||
/// <input type="checkbox" checked defaultChecked /> | ||
/// <input type="radio" checked defaultChecked /> | ||
/// | ||
/// React.createElement('input', { checked: false }); | ||
/// React.createElement('input', { type: 'checkbox', checked: true }); | ||
/// React.createElement('input', { type: 'checkbox', checked: true, defaultChecked: true }); | ||
/// | ||
/// // Good | ||
/// <input type="checkbox" checked onChange={() => {}} /> | ||
/// <input type="checkbox" checked readOnly /> | ||
/// <input type="checkbox" checked onChange readOnly /> | ||
/// <input type="checkbox" defaultChecked /> | ||
/// | ||
/// React.createElement('input', { type: 'checkbox', checked: true, onChange() {} }); | ||
/// React.createElement('input', { type: 'checkbox', checked: true, readOnly: true }); | ||
/// React.createElement('input', { type: 'checkbox', checked: true, onChange() {}, readOnly: true }); | ||
/// React.createElement('input', { type: 'checkbox', defaultChecked: true }); | ||
/// ``` | ||
CheckedRequiresOnchangeOrReadonly, | ||
correctness | ||
); | ||
|
||
pub const TARGET_PROPS: phf::Set<&'static str> = phf_set! { | ||
"checked", | ||
"onChange", | ||
"readOnly", | ||
"defaultChecked", | ||
}; | ||
|
||
fn is_target_prop(name: &str) -> bool { | ||
TARGET_PROPS.contains(name) | ||
} | ||
|
||
impl Rule for CheckedRequiresOnchangeOrReadonly { | ||
fn run<'a>(&self, node: &AstNode<'a>, ctx: &LintContext<'a>) { | ||
match node.kind() { | ||
AstKind::JSXOpeningElement(jsx_opening_el) => { | ||
let Some(element_type) = get_element_type(ctx, jsx_opening_el) else { return }; | ||
if element_type != "input" { | ||
return; | ||
} | ||
|
||
let prop_names: Vec<String> = jsx_opening_el | ||
.attributes | ||
.iter() | ||
.filter_map(|attr| { | ||
if let JSXAttributeItem::Attribute(jsx_attr) = attr { | ||
let attr_name = get_jsx_attribute_name(&jsx_attr.name); | ||
if is_target_prop(&attr_name) { | ||
return Some(attr_name); | ||
} | ||
} | ||
None | ||
}) | ||
.collect(); | ||
|
||
self.check_attributes_and_report(ctx, &prop_names, jsx_opening_el.span); | ||
} | ||
AstKind::CallExpression(call_expr) => { | ||
if !is_create_element_call(call_expr) { | ||
return; | ||
} | ||
|
||
let Some(Argument::Expression(Expression::StringLiteral(element_name))) = | ||
call_expr.arguments.first() | ||
else { | ||
return; | ||
}; | ||
|
||
if element_name.value != "input" { | ||
return; | ||
} | ||
|
||
let Some(Argument::Expression(Expression::ObjectExpression(obj_expr))) = | ||
call_expr.arguments.get(1) | ||
else { | ||
return; | ||
}; | ||
|
||
let prop_names: Vec<String> = obj_expr | ||
.properties | ||
.iter() | ||
.filter_map(|attr| { | ||
if let ObjectPropertyKind::ObjectProperty(obj_prop) = attr { | ||
let name = &obj_prop.key.name()?; | ||
if is_target_prop(name) { | ||
return Some(name.to_string()); | ||
} | ||
} | ||
None | ||
}) | ||
.collect(); | ||
|
||
self.check_attributes_and_report(ctx, &prop_names, call_expr.span); | ||
} | ||
_ => {} | ||
} | ||
} | ||
|
||
fn from_configuration(value: serde_json::Value) -> Self { | ||
let value = value.as_array().and_then(|arr| arr.first()).and_then(|val| val.as_object()); | ||
|
||
Self { | ||
ignore_missing_properties: value | ||
.and_then(|val| { | ||
val.get("ignoreMissingProperties").and_then(serde_json::Value::as_bool) | ||
}) | ||
.unwrap_or(false), | ||
ignore_exclusive_checked_attribute: value | ||
.and_then(|val| { | ||
val.get("ignoreExclusiveCheckedAttribute").and_then(serde_json::Value::as_bool) | ||
}) | ||
.unwrap_or(false), | ||
} | ||
} | ||
} | ||
|
||
impl CheckedRequiresOnchangeOrReadonly { | ||
fn check_attributes_and_report(&self, ctx: &LintContext, props: &[String], span: Span) { | ||
if !props.contains(&"checked".to_string()) { | ||
return; | ||
} | ||
|
||
if !self.ignore_exclusive_checked_attribute && props.contains(&"defaultChecked".to_string()) | ||
{ | ||
ctx.diagnostic(CheckedRequiresOnchangeOrReadonlyDiagnostic::ExclusiveCheckedAttribute( | ||
span, | ||
)); | ||
} | ||
|
||
if !(self.ignore_missing_properties | ||
|| props.contains(&"onChange".to_string()) | ||
|| props.contains(&"readOnly".to_string())) | ||
{ | ||
ctx.diagnostic(CheckedRequiresOnchangeOrReadonlyDiagnostic::MissingProperty(span)); | ||
} | ||
} | ||
} | ||
|
||
#[test] | ||
fn test() { | ||
use crate::tester::Tester; | ||
|
||
let pass = vec![ | ||
(r"<input type='checkbox' />", None), | ||
(r"<input type='checkbox' onChange={noop} />", None), | ||
(r"<input type='checkbox' readOnly />", None), | ||
(r"<input type='checkbox' checked onChange={noop} />", None), | ||
(r"<input type='checkbox' checked={true} onChange={noop} />", None), | ||
(r"<input type='checkbox' checked={false} onChange={noop} />", None), | ||
(r"<input type='checkbox' checked readOnly />", None), | ||
(r"<input type='checkbox' checked={true} readOnly />", None), | ||
(r"<input type='checkbox' checked={false} readOnly />", None), | ||
(r"<input type='checkbox' defaultChecked />", None), | ||
(r"React.createElement('input')", None), | ||
(r"React.createElement('input', { checked: true, onChange: noop })", None), | ||
(r"React.createElement('input', { checked: false, onChange: noop })", None), | ||
(r"React.createElement('input', { checked: true, readOnly: true })", None), | ||
(r"React.createElement('input', { checked: true, onChange: noop, readOnly: true })", None), | ||
(r"React.createElement('input', { checked: foo, onChange: noop, readOnly: true })", None), | ||
( | ||
r"<input type='checkbox' checked />", | ||
Some(serde_json::json!([{ "ignoreMissingProperties": true }])), | ||
), | ||
( | ||
r"<input type='checkbox' checked={true} />", | ||
Some(serde_json::json!([{ "ignoreMissingProperties": true }])), | ||
), | ||
( | ||
r"<input type='checkbox' onChange={noop} checked defaultChecked />", | ||
Some(serde_json::json!([{ "ignoreExclusiveCheckedAttribute": true }])), | ||
), | ||
( | ||
r"<input type='checkbox' onChange={noop} checked={true} defaultChecked />", | ||
Some(serde_json::json!([{ "ignoreExclusiveCheckedAttribute": true }])), | ||
), | ||
( | ||
r"<input type='checkbox' onChange={noop} checked defaultChecked />", | ||
Some( | ||
serde_json::json!([{ "ignoreMissingProperties": true, "ignoreExclusiveCheckedAttribute": true }]), | ||
), | ||
), | ||
(r"<span/>", None), | ||
(r"React.createElement('span')", None), | ||
(r"(()=>{})()", None), | ||
]; | ||
|
||
let fail = vec![ | ||
(r"<input type='radio' checked />", None), | ||
(r"<input type='radio' checked={true} />", None), | ||
(r"<input type='checkbox' checked />", None), | ||
(r"<input type='checkbox' checked={true} />", None), | ||
(r"<input type='checkbox' checked={condition ? true : false} />", None), | ||
(r"<input type='checkbox' checked defaultChecked />", None), | ||
(r"React.createElement('input', { checked: false })", None), | ||
(r"React.createElement('input', { checked: true, defaultChecked: true })", None), | ||
( | ||
r"<input type='checkbox' checked defaultChecked />", | ||
Some(serde_json::json!([{ "ignoreMissingProperties": true }])), | ||
), | ||
( | ||
r"<input type='checkbox' checked defaultChecked />", | ||
Some(serde_json::json!([{ "ignoreExclusiveCheckedAttribute": true }])), | ||
), | ||
( | ||
r"<input type='checkbox' checked defaultChecked />", | ||
Some( | ||
serde_json::json!([{ "ignoreMissingProperties": false, "ignoreExclusiveCheckedAttribute": false }]), | ||
), | ||
), | ||
]; | ||
|
||
Tester::new(CheckedRequiresOnchangeOrReadonly::NAME, pass, fail).test_and_snapshot(); | ||
} |
101 changes: 101 additions & 0 deletions
101
crates/oxc_linter/src/snapshots/checked_requires_onchange_or_readonly.snap
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,101 @@ | ||
--- | ||
source: crates/oxc_linter/src/tester.rs | ||
expression: checked_requires_onchange_or_readonly | ||
--- | ||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='radio' checked /> | ||
· ────────────────────────────── | ||
╰──── | ||
help: Add either `onChange` or `readOnly`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='radio' checked={true} /> | ||
· ───────────────────────────────────── | ||
╰──── | ||
help: Add either `onChange` or `readOnly`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='checkbox' checked /> | ||
· ───────────────────────────────── | ||
╰──── | ||
help: Add either `onChange` or `readOnly`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='checkbox' checked={true} /> | ||
· ──────────────────────────────────────── | ||
╰──── | ||
help: Add either `onChange` or `readOnly`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='checkbox' checked={condition ? true : false} /> | ||
· ──────────────────────────────────────────────────────────── | ||
╰──── | ||
help: Add either `onChange` or `readOnly`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): Use either `checked` or `defaultChecked`, but not both. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='checkbox' checked defaultChecked /> | ||
· ──────────────────────────────────────────────── | ||
╰──── | ||
help: Remove either `checked` or `defaultChecked`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='checkbox' checked defaultChecked /> | ||
· ──────────────────────────────────────────────── | ||
╰──── | ||
help: Add either `onChange` or `readOnly`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ React.createElement('input', { checked: false }) | ||
· ──────────────────────────────────────────────── | ||
╰──── | ||
help: Add either `onChange` or `readOnly`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): Use either `checked` or `defaultChecked`, but not both. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ React.createElement('input', { checked: true, defaultChecked: true }) | ||
· ───────────────────────────────────────────────────────────────────── | ||
╰──── | ||
help: Remove either `checked` or `defaultChecked`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ React.createElement('input', { checked: true, defaultChecked: true }) | ||
· ───────────────────────────────────────────────────────────────────── | ||
╰──── | ||
help: Add either `onChange` or `readOnly`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): Use either `checked` or `defaultChecked`, but not both. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='checkbox' checked defaultChecked /> | ||
· ──────────────────────────────────────────────── | ||
╰──── | ||
help: Remove either `checked` or `defaultChecked`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='checkbox' checked defaultChecked /> | ||
· ──────────────────────────────────────────────── | ||
╰──── | ||
help: Add either `onChange` or `readOnly`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): Use either `checked` or `defaultChecked`, but not both. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='checkbox' checked defaultChecked /> | ||
· ──────────────────────────────────────────────── | ||
╰──── | ||
help: Remove either `checked` or `defaultChecked`. | ||
|
||
⚠ eslint-plugin-react(checked-requires-onchange-or-readonly): `checked` should be used with either `onChange` or `readOnly`. | ||
╭─[checked_requires_onchange_or_readonly.tsx:1:1] | ||
1 │ <input type='checkbox' checked defaultChecked /> | ||
· ──────────────────────────────────────────────── | ||
╰──── | ||
help: Add either `onChange` or `readOnly`. |