feat(linter): mark nextjs rules as recommended (#27691)

Copying recommended rules 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/nextjs/google_font_display.rs

@@ -74,6 +74,7 @@ declare_oxc_lint!(
     GoogleFontDisplay,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Enforce font-display behavior with Google Fonts.",
 );

crates/oxc_linter/src/rules/nextjs/google_font_preconnect.rs

@@ -45,6 +45,7 @@ declare_oxc_lint!(
     GoogleFontPreconnect,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Enforces the presence of `rel=\"preconnect\"` when using Google Fonts via `<link>` tags.",
 );

crates/oxc_linter/src/rules/nextjs/inline_script_id.rs

@@ -92,6 +92,7 @@ declare_oxc_lint!(
     InlineScriptId,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Enforces that all `next/script` components with inline content or `dangerouslySetInnerHTML` must have an `id` prop.",
 );

crates/oxc_linter/src/rules/nextjs/next_script_for_ga.rs

@@ -81,6 +81,7 @@ declare_oxc_lint!(
     NextScriptForGa,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Enforces the use of the `next/script` component when implementing Google Analytics in Next.js applications, instead of using regular `<script>` tags.",
 );

crates/oxc_linter/src/rules/nextjs/no_assign_module_variable.rs

@@ -58,6 +58,7 @@ declare_oxc_lint!(
     NoAssignModuleVariable,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Prevents the assignment or declaration of variables named `module` in Next.js applications.",
 );

crates/oxc_linter/src/rules/nextjs/no_async_client_component.rs

@@ -86,6 +86,7 @@ declare_oxc_lint!(
     NoAsyncClientComponent,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Prevent client components from being async functions.",
 );

crates/oxc_linter/src/rules/nextjs/no_before_interactive_script_outside_document.rs

@@ -86,6 +86,7 @@ declare_oxc_lint!(
     NoBeforeInteractiveScriptOutsideDocument,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.7",
     short_description = "Prevent the usage of `next/script`'s `beforeInteractive` strategy outside of `pages/_document.js`.",
 );

crates/oxc_linter/src/rules/nextjs/no_css_tags.rs

@@ -66,6 +66,7 @@ declare_oxc_lint!(
     NoCssTags,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Prevent manual inclusion of stylesheets using `<link>` tags in Next.js applications.",
 );

crates/oxc_linter/src/rules/nextjs/no_document_import_in_page.rs

@@ -55,6 +55,7 @@ declare_oxc_lint!(
     NoDocumentImportInPage,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.1",
     short_description = "Prevent importing `next/document` outside of `pages/_document.js`.",
 );

crates/oxc_linter/src/rules/nextjs/no_duplicate_head.rs

@@ -70,6 +70,7 @@ declare_oxc_lint!(
     NoDuplicateHead,
     nextjs,
     correctness,
+    recommended,
     version = "0.3.3",
     short_description = "Prevent duplicate usage of `<Head>` in `pages/_document.js`.",
 );

crates/oxc_linter/src/rules/nextjs/no_head_element.rs

@@ -67,6 +67,7 @@ declare_oxc_lint!(
     NoHeadElement,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.1",
     short_description = "Prevents the usage of the native `<head>` element inside a Next.js application.",
 );

crates/oxc_linter/src/rules/nextjs/no_head_import_in_document.rs

@@ -75,6 +75,7 @@ declare_oxc_lint!(
     NoHeadImportInDocument,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Prevents the usage of `next/head` inside a Next.js document.",
 );

crates/oxc_linter/src/rules/nextjs/no_html_link_for_pages.rs

@@ -76,6 +76,7 @@ declare_oxc_lint!(
     NoHtmlLinkForPages,
     nextjs,
     correctness,
+    recommended,
     version = "1.7.0",
     short_description = "Prevents the usage of `<a>` elements to navigate between Next.js pages.",
 );

crates/oxc_linter/src/rules/nextjs/no_img_element.rs

@@ -63,6 +63,7 @@ declare_oxc_lint!(
     NoImgElement,
     nextjs,
     correctness,
+    recommended,
     pending, // TODO: add `import Image from "next/image"` (if missing), then change `<img />` to `<Image />`
     version = "0.2.0",
     short_description = "Prevent the usage of the `<img>` element due to slower LCP and higher bandwidth.",

crates/oxc_linter/src/rules/nextjs/no_page_custom_font.rs

@@ -75,6 +75,7 @@ declare_oxc_lint!(
     NoPageCustomFont,
     nextjs,
     correctness,
+    recommended,
     version = "0.3.3",
     short_description = "Prevent page-only custom fonts.",
 );

crates/oxc_linter/src/rules/nextjs/no_script_component_in_head.rs

@@ -63,6 +63,7 @@ declare_oxc_lint!(
     NoScriptComponentInHead,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Prevent usage of `next/script` in `next/head` component.",
 );

crates/oxc_linter/src/rules/nextjs/no_styled_jsx_in_document.rs

@@ -75,6 +75,7 @@ declare_oxc_lint!(
     NoStyledJsxInDocument,
     nextjs,
     correctness,
+    recommended,
     version = "0.3.3",
     short_description = "Prevent usage of styled-jsx in `pages/_document.js`.",
 );

crates/oxc_linter/src/rules/nextjs/no_sync_scripts.rs

@@ -56,6 +56,7 @@ declare_oxc_lint!(
     NoSyncScripts,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Prevent the use of synchronous `<script>` tags in Next.js applications.",
 );

crates/oxc_linter/src/rules/nextjs/no_title_in_document_head.rs

@@ -61,6 +61,7 @@ declare_oxc_lint!(
     NoTitleInDocumentHead,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.0",
     short_description = "Prevent usage of `<title>` with `Head` component from `next/document`.",
 );

crates/oxc_linter/src/rules/nextjs/no_typos.rs

@@ -50,6 +50,7 @@ declare_oxc_lint!(
     NoTypos,
     nextjs,
     correctness,
+    recommended,
     suggestion,
     version = "0.2.1",
     short_description = "Detects common typos in Next.js data fetching function names.",

crates/oxc_linter/src/rules/nextjs/no_unwanted_polyfillio.rs

@@ -64,6 +64,7 @@ declare_oxc_lint!(
     NoUnwantedPolyfillio,
     nextjs,
     correctness,
+    recommended,
     version = "0.2.7",
     short_description = "Prevent use of unsafe polyfill.io domains and duplicate polyfills.",
 );