feat(linter): mark promise rules as recommended (#27693)

Copying recommended state from upstream.

---

<sub>Stack created with <a
href="https://github.com/github/gh-stack">GitHub Stacks CLI</a> • <a
href="https://gh.io/stacks-feedback">Give Feedback 💬</a></sub>

crates/oxc_linter/src/rules/promise/always_return.rs

@@ -183,6 +183,7 @@ declare_oxc_lint!(
     AlwaysReturn,
     promise,
     suspicious,
+    recommended,
     config = AlwaysReturnConfig,
     version = "1.13.0",
     short_description = "Require returning inside each `then()` to create readable and reusable Promise chains.",

crates/oxc_linter/src/rules/promise/catch_or_return.rs

@@ -165,6 +165,7 @@ declare_oxc_lint!(
     CatchOrReturn,
     promise,
     restriction,
+    recommended,
     config = CatchOrReturnConfig,
     version = "0.9.2",
     short_description = "Ensure that each `then()` applied to a promise also has a `catch()`.",

crates/oxc_linter/src/rules/promise/no_callback_in_promise.rs

@@ -86,6 +86,7 @@ declare_oxc_lint!(
     NoCallbackInPromise,
     promise,
     correctness,
+    recommended,
     config = NoCallbackInPromiseConfig,
     version = "0.10.0",
     short_description = "Disallows calling a callback function (`cb()`) inside a `Promise.prototype.then()` or `Promise.prototype.catch()`.",

crates/oxc_linter/src/rules/promise/no_nesting.rs

@@ -66,6 +66,7 @@ declare_oxc_lint!(
     NoNesting,
     promise,
     style,
+    recommended,
     pending,
     version = "0.15.13",
     short_description = "Disallow nested `then()` or `catch()` statements.",

crates/oxc_linter/src/rules/promise/no_new_statics.rs

@@ -41,6 +41,7 @@ declare_oxc_lint!(
     NoNewStatics,
     promise,
     correctness,
+    recommended,
     fix,
     version = "0.6.1",
     short_description = "Disallows calling new on static `Promise` methods.",

crates/oxc_linter/src/rules/promise/no_promise_in_callback.rs

@@ -60,6 +60,7 @@ declare_oxc_lint!(
     NoPromiseInCallback,
     promise,
     suspicious,
+    recommended,
     config = NoPromiseInCallback,
     version = "0.13.1",
     short_description = "Disallows the use of Promises within error-first callback functions.",

crates/oxc_linter/src/rules/promise/no_return_in_finally.rs

@@ -43,6 +43,7 @@ declare_oxc_lint!(
     NoReturnInFinally,
     promise,
     nursery,
+    recommended,
     version = "0.7.1",
     short_description = "Disallow return statements in a `finally()` callback of a promise.",
 );

crates/oxc_linter/src/rules/promise/no_return_wrap.rs

@@ -134,6 +134,7 @@ declare_oxc_lint!(
     NoReturnWrap,
     promise,
     style,
+    recommended,
     pending,
     config = NoReturnWrap,
     version = "0.15.14",

crates/oxc_linter/src/rules/promise/param_names.rs

@@ -80,6 +80,7 @@ declare_oxc_lint!(
     ParamNames,
     promise,
     style,
+    recommended,
     config = ParamNamesConfig,
     version = "0.6.1",
     short_description = "Enforce standard parameter names for Promise constructors.",

crates/oxc_linter/src/rules/promise/valid_params.rs

@@ -63,6 +63,7 @@ declare_oxc_lint!(
     ValidParams,
     promise,
     correctness,
+    recommended,
     version = "0.7.1",
     short_description = "Enforce that Promise functions are called with the correct number of arguments.",
 );