diff --git a/its/ruling/src/test/expected/js/fireact/javascript-S6749.json b/its/ruling/src/test/expected/js/fireact/javascript-S6749.json new file mode 100644 index 00000000000..cde87428833 --- /dev/null +++ b/its/ruling/src/test/expected/js/fireact/javascript-S6749.json @@ -0,0 +1,23 @@ +{ +"fireact:src/components/Loader/index.js": [ +24 +], +"fireact:src/components/Logo/index.js": [ +6 +], +"fireact:src/components/menus/UserMenu/index.js": [ +22 +], +"fireact:src/pages/auth/accounts/AddUser/index.js": [ +60 +], +"fireact:src/pages/auth/accounts/DeleteAccount/index.js": [ +57 +], +"fireact:src/pages/auth/accounts/PaymentList/index.js": [ +72 +], +"fireact:src/pages/auth/accounts/PaymentMethod/index.js": [ +131 +] +} diff --git a/its/ruling/src/test/expected/ts/ant-design/typescript-S6749.json b/its/ruling/src/test/expected/ts/ant-design/typescript-S6749.json new file mode 100644 index 00000000000..0126b7eb332 --- /dev/null +++ b/its/ruling/src/test/expected/ts/ant-design/typescript-S6749.json @@ -0,0 +1,5 @@ +{ +"ant-design:components/button/__tests__/index.test.tsx": [ +345 +] +} diff --git a/its/ruling/src/test/expected/ts/courselit/typescript-S6749.json b/its/ruling/src/test/expected/ts/courselit/typescript-S6749.json new file mode 100644 index 00000000000..1387f7bd4cc --- /dev/null +++ b/its/ruling/src/test/expected/ts/courselit/typescript-S6749.json @@ -0,0 +1,27 @@ +{ +"courselit:apps/web/components/admin/media/index.tsx": [ +173 +], +"courselit:apps/web/components/public/base-layout/scaffold/drawer-content.tsx": [ +57 +], +"courselit:apps/web/components/public/base-layout/template/section.tsx": [ +40 +], +"courselit:apps/web/components/public/checkout/index.tsx": [ +32, +33 +], +"courselit:apps/web/components/public/code-injector.tsx": [ +45 +], +"courselit:apps/web/components/public/items.tsx": [ +90 +], +"courselit:apps/web/components/public/purchase-status.tsx": [ +105 +], +"courselit:packages/common-widgets/src/tagged-content/widget.tsx": [ +77 +] +} diff --git a/its/ruling/src/test/expected/ts/desktop/typescript-S6749.json b/its/ruling/src/test/expected/ts/desktop/typescript-S6749.json new file mode 100644 index 00000000000..54455699d74 --- /dev/null +++ b/its/ruling/src/test/expected/ts/desktop/typescript-S6749.json @@ -0,0 +1,10 @@ +{ +"desktop:app/src/ui/diff/diff-options.tsx": [ +123 +], +"desktop:app/src/ui/drag-elements/commit-drag-element.tsx": [ +93, +101, +111 +] +} diff --git a/its/ruling/src/test/expected/ts/eigen/typescript-S6749.json b/its/ruling/src/test/expected/ts/eigen/typescript-S6749.json new file mode 100644 index 00000000000..213e43621ea --- /dev/null +++ b/its/ruling/src/test/expected/ts/eigen/typescript-S6749.json @@ -0,0 +1,77 @@ +{ +"eigen:src/app/Components/AuctionResultsList.tsx": [ +90 +], +"eigen:src/app/Components/States/ZeroState.tsx": [ +17, +27 +], +"eigen:src/app/Components/StickyTabPage/StickyTabPage.tsx": [ +51 +], +"eigen:src/app/Scenes/Artist/SearchCriteria.tests.tsx": [ +18, +33, +52, +57, +66 +], +"eigen:src/app/Scenes/Artwork/Components/CommercialButtons/InquiryModal.tsx": [ +77 +], +"eigen:src/app/Scenes/Artwork/Components/CommercialEditionSetInformation.tsx": [ +57 +], +"eigen:src/app/Scenes/City/Components/SavedEventSection/index.tsx": [ +46 +], +"eigen:src/app/Scenes/Home/Components/AuctionResultsRail.tsx": [ +51 +], +"eigen:src/app/Scenes/Home/Components/FairsRail.tsx": [ +57 +], +"eigen:src/app/Scenes/Home/Home.tsx": [ +206 +], +"eigen:src/app/Scenes/MyBids/MyBids.tsx": [ +114 +], +"eigen:src/app/Scenes/MyCollection/Screens/Artwork/Components/ArtworkInsights/MyCollectionArtworkArtistAuctionResults.tsx": [ +50 +], +"eigen:src/app/Scenes/MyCollection/Screens/Artwork/Components/MyCollectionArtworkHeader.tsx": [ +112 +], +"eigen:src/app/Scenes/SavedAddresses/SavedAddresses.tsx": [ +165 +], +"eigen:src/app/Scenes/SavedSearchAlertsList/Components/SavedSearchAlertsListPlaceholder.tsx": [ +8 +], +"eigen:src/app/Scenes/Search/components/placeholders/AlgoliaSearchPlaceholder.tsx": [ +15 +], +"eigen:src/app/Scenes/SellWithArtsy/SubmitArtwork/SubmitArtwork.tsx": [ +217 +], +"eigen:src/app/Scenes/Tag/Tag.tsx": [ +65 +], +"eigen:src/palette/elements/CollapsibleMenuItem/CollapsibleMenuItem.stories.tsx": [ +154 +], +"eigen:src/palette/organisms/screenStructure/Screen.tsx": [ +152 +], +"eigen:src/palette/svgs/CreditCardIcon.tsx": [ +23, +35, +46, +56, +71 +], +"eigen:src/shared/utils/flattenChildren.tests.tsx": [ +13 +] +} diff --git a/its/ruling/src/test/expected/ts/moose/typescript-S6749.json b/its/ruling/src/test/expected/ts/moose/typescript-S6749.json new file mode 100644 index 00000000000..2b26ff1961b --- /dev/null +++ b/its/ruling/src/test/expected/ts/moose/typescript-S6749.json @@ -0,0 +1,5 @@ +{ +"moose:renderer/components/Cast/Cast.tsx": [ +42 +] +} diff --git a/its/ruling/src/test/expected/ts/vuetify/typescript-S6749.json b/its/ruling/src/test/expected/ts/vuetify/typescript-S6749.json new file mode 100644 index 00000000000..40333473e34 --- /dev/null +++ b/its/ruling/src/test/expected/ts/vuetify/typescript-S6749.json @@ -0,0 +1,5 @@ +{ +"vuetify:packages/vuetify/src/components/VTabs/__tests__/VTabs.spec.cy.tsx": [ +10 +] +} diff --git a/packages/jsts/src/linter/quickfixes/messages.ts b/packages/jsts/src/linter/quickfixes/messages.ts index e6c4594885c..3c760302ff8 100644 --- a/packages/jsts/src/linter/quickfixes/messages.ts +++ b/packages/jsts/src/linter/quickfixes/messages.ts @@ -27,35 +27,36 @@ const quickFixMessages = new Map([ ['comma-dangle', 'Remove this trailing comma'], ['eol-last', 'Add a new line at the end of file'], + ['jsx-no-useless-fragment', 'Remove redundant fragment'], + ['no-empty-interface', 'Replace with type alias'], ['no-extra-bind', 'Remove .bind() call'], ['no-extra-boolean-cast', 'Remove extra cast'], ['no-extra-semi', 'Remove extra semicolon'], + ['no-inferrable-types', 'Remove type declaration'], + ['no-lonely-if', "Replace with 'else if'"], + ['no-non-null-assertion', "Replace with optional chaining '.?'"], ['no-trailing-spaces', 'Remove trailing space'], + ['no-undef-init', 'Remove initialization'], + ['no-unnecessary-type-arguments', 'Remove type argument'], + ['no-unnecessary-type-assertion', 'Remove type assertion'], + ['no-unneeded-ternary', 'Replace with a simpler expression'], + ['no-useless-rename', 'Remove alias'], ['no-var', "Replace 'var' with 'let'"], ['object-shorthand', 'Use shorthand property'], ['prefer-as-const', "Replace with 'as const'"], ['prefer-const', "Replace with 'const'"], + ['prefer-function-type', 'Replace with a function type'], + ['prefer-immediate-return', 'Return value immediately'], + ['prefer-namespace-keyword', "Replace with 'namespace' keyword"], ['prefer-object-has-own', 'Replace with Object.hasOwn()'], ['prefer-object-spread', 'Replace with object spread syntax'], + ['prefer-readonly', "Add 'readonly'"], ['prefer-return-this-type', "Replace return type with 'this'"], ['prefer-template', 'Replace with template string literal'], + ['prefer-while', "Replace with 'while' loop"], ['quotes', 'Fix quotes'], ['radix', 'Add 10 as radix'], ['semi', 'Add semicolon'], - ['prefer-immediate-return', 'Return value immediately'], - ['prefer-while', "Replace with 'while' loop"], - ['no-empty-interface', 'Replace with type alias'], - ['no-inferrable-types', 'Remove type declaration'], - ['no-lonely-if', "Replace with 'else if'"], - ['no-undef-init', 'Remove initialization'], - ['no-unnecessary-type-arguments', 'Remove type argument'], - ['no-unnecessary-type-assertion', 'Remove type assertion'], - ['prefer-function-type', 'Replace with a function type'], - ['prefer-namespace-keyword', "Replace with 'namespace' keyword"], - ['prefer-readonly', "Add 'readonly'"], - ['no-non-null-assertion', "Replace with optional chaining '.?'"], - ['no-unneeded-ternary', 'Replace with a simpler expression'], - ['no-useless-rename', 'Remove alias'], ]); /** diff --git a/packages/jsts/src/linter/quickfixes/rules.ts b/packages/jsts/src/linter/quickfixes/rules.ts index 77641549439..98fe04848db 100644 --- a/packages/jsts/src/linter/quickfixes/rules.ts +++ b/packages/jsts/src/linter/quickfixes/rules.ts @@ -77,6 +77,7 @@ export const quickFixRules = new Set([ 'prefer-while', // eslint-plugin-react + 'jsx-no-useless-fragment', 'no-unknown-property', // @typescript-eslint plugin diff --git a/packages/jsts/tests/linter/fixtures/wrapper/quickfixes/jsx-no-useless-fragment.jsx b/packages/jsts/tests/linter/fixtures/wrapper/quickfixes/jsx-no-useless-fragment.jsx new file mode 100644 index 00000000000..2a6b428a5e4 --- /dev/null +++ b/packages/jsts/tests/linter/fixtures/wrapper/quickfixes/jsx-no-useless-fragment.jsx @@ -0,0 +1 @@ +<> diff --git a/sonar-plugin/javascript-checks/src/main/java/org/sonar/javascript/checks/CheckList.java b/sonar-plugin/javascript-checks/src/main/java/org/sonar/javascript/checks/CheckList.java index c1586949880..7c026785b56 100644 --- a/sonar-plugin/javascript-checks/src/main/java/org/sonar/javascript/checks/CheckList.java +++ b/sonar-plugin/javascript-checks/src/main/java/org/sonar/javascript/checks/CheckList.java @@ -202,6 +202,7 @@ public static List> getAllChecks() { JsxNoCommentTextnodesCheck.class, JsxNoConstructedContextValuesCheck.class, JsxNoLeakedRenderCheck.class, + JsxNoUselessFragmentCheck.class, JumpStatementInFinallyCheck.class, LabelPlacementCheck.class, LabelledStatementCheck.class, diff --git a/sonar-plugin/javascript-checks/src/main/java/org/sonar/javascript/checks/JsxNoUselessFragmentCheck.java b/sonar-plugin/javascript-checks/src/main/java/org/sonar/javascript/checks/JsxNoUselessFragmentCheck.java new file mode 100644 index 00000000000..33b94e3ab64 --- /dev/null +++ b/sonar-plugin/javascript-checks/src/main/java/org/sonar/javascript/checks/JsxNoUselessFragmentCheck.java @@ -0,0 +1,47 @@ +/** + * SonarQube JavaScript Plugin + * Copyright (C) 2011-2023 SonarSource SA + * mailto:info AT sonarsource DOT com + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 3 of the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public License + * along with this program; if not, write to the Free Software Foundation, + * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. + */ +package org.sonar.javascript.checks; + +import java.util.List; +import org.sonar.check.Rule; +import org.sonar.plugins.javascript.api.EslintBasedCheck; +import org.sonar.plugins.javascript.api.JavaScriptRule; +import org.sonar.plugins.javascript.api.TypeScriptRule; + +@JavaScriptRule +@TypeScriptRule +@Rule(key = "S6749") +public class JsxNoUselessFragmentCheck implements EslintBasedCheck { + + @Override + public String eslintKey() { + return "jsx-no-useless-fragment"; + } + + @Override + public List configurations() { + return List.of(new Config()); + } + + private static class Config { + + boolean allowExpressions = true; + } +} diff --git a/sonar-plugin/javascript-checks/src/main/resources/org/sonar/l10n/javascript/rules/javascript/S6749.html b/sonar-plugin/javascript-checks/src/main/resources/org/sonar/l10n/javascript/rules/javascript/S6749.html new file mode 100644 index 00000000000..605f595c65a --- /dev/null +++ b/sonar-plugin/javascript-checks/src/main/resources/org/sonar/l10n/javascript/rules/javascript/S6749.html @@ -0,0 +1,19 @@ +

Why is this an issue?

+

React fragments are a feature in React that allows you to group multiple elements together without adding an extra DOM element. They are a way to +return multiple elements from a component’s render method without requiring a wrapping parent element.

+

However, a fragment is redundant if it contains only one child, or if it is the child of an HTML element.

+
+<><Foo /></>;    // Noncompliant: The fragment has only one child
+<p><>foo</></p>; // Noncompliant: The fragment is the child of the HTML element 'p'
+
+

You can safely remove the redundant fragment while preserving the original behaviour.

+
+<Foo />;
+<p>foo</p>;
+
+

Resources

+

Documentation

+ + diff --git a/sonar-plugin/javascript-checks/src/main/resources/org/sonar/l10n/javascript/rules/javascript/S6749.json b/sonar-plugin/javascript-checks/src/main/resources/org/sonar/l10n/javascript/rules/javascript/S6749.json new file mode 100644 index 00000000000..ce0c37b7819 --- /dev/null +++ b/sonar-plugin/javascript-checks/src/main/resources/org/sonar/l10n/javascript/rules/javascript/S6749.json @@ -0,0 +1,29 @@ +{ + "title": "Redundant React fragments should be removed", + "type": "CODE_SMELL", + "status": "ready", + "remediation": { + "func": "Constant\/Issue", + "constantCost": "5min" + }, + "tags": [ + "react" + ], + "defaultSeverity": "Major", + "ruleSpecification": "RSPEC-6749", + "sqKey": "S6749", + "scope": "All", + "quickfix": "covered", + "code": { + "impacts": { + "MAINTAINABILITY": "HIGH", + "RELIABILITY": "MEDIUM", + "SECURITY": "LOW" + }, + "attribute": "CONVENTIONAL" + }, + "compatibleLanguages": [ + "JAVASCRIPT", + "TYPESCRIPT" + ] +} diff --git a/sonar-plugin/javascript-checks/src/main/resources/org/sonar/l10n/javascript/rules/javascript/Sonar_way_profile.json b/sonar-plugin/javascript-checks/src/main/resources/org/sonar/l10n/javascript/rules/javascript/Sonar_way_profile.json index 580e6d1ee6c..0616117995a 100644 --- a/sonar-plugin/javascript-checks/src/main/resources/org/sonar/l10n/javascript/rules/javascript/Sonar_way_profile.json +++ b/sonar-plugin/javascript-checks/src/main/resources/org/sonar/l10n/javascript/rules/javascript/Sonar_way_profile.json @@ -275,6 +275,7 @@ "S6676", "S6679", "S6746", - "S6747" + "S6747", + "S6749" ] } diff --git a/sonar-plugin/javascript-checks/src/test/java/org/sonar/javascript/checks/JsxNoUselessFragmentCheckTest.java b/sonar-plugin/javascript-checks/src/test/java/org/sonar/javascript/checks/JsxNoUselessFragmentCheckTest.java new file mode 100644 index 00000000000..e0ba8386c88 --- /dev/null +++ b/sonar-plugin/javascript-checks/src/test/java/org/sonar/javascript/checks/JsxNoUselessFragmentCheckTest.java @@ -0,0 +1,34 @@ +/* + * SonarQube JavaScript Plugin + * Copyright (C) 2011-2023 SonarSource SA + * mailto:info AT sonarsource DOT com + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 3 of the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public License + * along with this program; if not, write to the Free Software Foundation, + * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. + */ +package org.sonar.javascript.checks; + +import static org.assertj.core.api.Assertions.assertThat; + +import com.google.gson.Gson; +import org.junit.jupiter.api.Test; + +class JsxNoUselessFragmentCheckTest { + + @Test + void test() { + var configAsString = new Gson().toJson(new JsxNoUselessFragmentCheck().configurations()); + assertThat(configAsString).isEqualTo("[{\"allowExpressions\":true}]"); + } +}