feat(linter): implement no-useless-return rule (#16388)

Implement `eslint/no-useless-return` rule.

Disallows redundant return statements that have no effect on function
behavior.

## Test

- 27 valid / 25 invalid test cases
- `just ready` passes

Note: `globalReturn` cases excluded (unsupported by oxc)

Part of #479

---------

Co-authored-by: Cameron Clark <cameron.clark@hey.com>

Author: FMP1783

crates/oxc_linter/src/generated/rule_runner_impls.rs

@@ -870,6 +870,12 @@ impl RuleRunner for crate::rules::eslint::no_useless_rename::NoUselessRename {
     const RUN_FUNCTIONS: RuleRunFunctionsImplemented = RuleRunFunctionsImplemented::Run;
 }
 
+impl RuleRunner for crate::rules::eslint::no_useless_return::NoUselessReturn {
+    const NODE_TYPES: Option<&AstTypesBitset> =
+        Some(&AstTypesBitset::from_types(&[AstType::ReturnStatement]));
+    const RUN_FUNCTIONS: RuleRunFunctionsImplemented = RuleRunFunctionsImplemented::Run;
+}
+
 impl RuleRunner for crate::rules::eslint::no_var::NoVar {
     const NODE_TYPES: Option<&AstTypesBitset> =
         Some(&AstTypesBitset::from_types(&[AstType::VariableDeclaration]));

crates/oxc_linter/src/rules.rs

@@ -169,6 +169,7 @@ pub(crate) mod eslint {
     pub mod no_useless_constructor;
     pub mod no_useless_escape;
     pub mod no_useless_rename;
+    pub mod no_useless_return;
     pub mod no_var;
     pub mod no_void;
     pub mod no_warning_comments;
@@ -800,6 +801,7 @@ oxc_macros::declare_all_lint_rules! {
     eslint::no_useless_constructor,
     eslint::no_useless_escape,
     eslint::no_useless_rename,
+    eslint::no_useless_return,
     eslint::no_var,
     eslint::no_void,
     eslint::no_warning_comments,