refactor: remove unused code

Author: Rel1cx

packages/plugins/eslint-plugin-react-x/src/rules/no-component-will-mount.ts

@@ -16,12 +16,6 @@ export const RULE_FEATURES = [
 
 export type MessageID = CamelCase<typeof RULE_NAME>;
 
-function isComponentWillMount(node: TSESTree.ClassElement) {
-  return AST.isMethodOrProperty(node)
-    && node.key.type === T.Identifier
-    && node.key.name === "componentWillMount";
-}
-
 export default createRule<[], MessageID>({
   meta: {
     type: "problem",
@@ -51,7 +45,7 @@ export function create(context: RuleContext<MessageID, []>): RuleListener {
       for (const { node: component } of components.values()) {
         const { body } = component.body;
         for (const member of body) {
-          if (isComponentWillMount(member)) {
+          if (ER.isComponentWillMount(member)) {
             context.report({
               messageId: "noComponentWillMount",
               node: member,

packages/plugins/eslint-plugin-react-x/src/rules/no-component-will-receive-props.ts

@@ -16,12 +16,6 @@ export const RULE_FEATURES = [
 
 export type MessageID = CamelCase<typeof RULE_NAME>;
 
-function isComponentWillUpdate(node: TSESTree.ClassElement) {
-  return AST.isMethodOrProperty(node)
-    && node.key.type === T.Identifier
-    && node.key.name === "componentWillReceiveProps";
-}
-
 export default createRule<[], MessageID>({
   meta: {
     type: "problem",
@@ -50,7 +44,7 @@ export function create(context: RuleContext<MessageID, []>): RuleListener {
       for (const { node: component } of components.values()) {
         const { body } = component.body;
         for (const member of body) {
-          if (isComponentWillUpdate(member)) {
+          if (ER.isComponentWillReceiveProps(member)) {
             context.report({
               messageId: "noComponentWillReceiveProps",
               node: member,

packages/plugins/eslint-plugin-react-x/src/rules/no-component-will-update.ts

@@ -16,12 +16,6 @@ export const RULE_FEATURES = [
 
 export type MessageID = CamelCase<typeof RULE_NAME>;
 
-function isComponentWillUpdate(node: TSESTree.ClassElement) {
-  return AST.isMethodOrProperty(node)
-    && node.key.type === T.Identifier
-    && node.key.name === "componentWillUpdate";
-}
-
 export default createRule<[], MessageID>({
   meta: {
     type: "problem",
@@ -51,7 +45,7 @@ export function create(context: RuleContext<MessageID, []>): RuleListener {
       for (const { node: component } of components.values()) {
         const { body } = component.body;
         for (const member of body) {
-          if (isComponentWillUpdate(member)) {
+          if (ER.isComponentWillUpdate(member)) {
             context.report({
               messageId: "noComponentWillUpdate",
               node: member,

packages/plugins/eslint-plugin-react-x/src/rules/no-set-state-in-component-did-mount.ts

@@ -14,12 +14,6 @@ export const RULE_FEATURES = [] as const satisfies RuleFeature[];
 
 export type MessageID = CamelCase<typeof RULE_NAME>;
 
-function isComponentDidMount(node: TSESTree.Node) {
-  return AST.isMethodOrProperty(node)
-    && node.key.type === T.Identifier
-    && node.key.name === "componentDidMount";
-}
-
 export default createRule<[], MessageID>({
   meta: {
     type: "problem",
@@ -46,7 +40,7 @@ export function create(context: RuleContext<MessageID, []>): RuleListener {
         return;
       }
       const clazz = AST.findParentNode(node, ER.isClassComponent);
-      const method = AST.findParentNode(node, (n) => n === clazz || isComponentDidMount(n));
+      const method = AST.findParentNode(node, (n) => n === clazz || ER.isComponentDidMount(n));
       if (clazz == null || method == null || method === clazz) return;
       const methodScope = context.sourceCode.getScope(method);
       const upperScope = context.sourceCode.getScope(node).upper;

packages/plugins/eslint-plugin-react-x/src/rules/no-set-state-in-component-did-update.ts

@@ -14,12 +14,6 @@ export const RULE_FEATURES = [] as const satisfies RuleFeature[];
 
 export type MessageID = CamelCase<typeof RULE_NAME>;
 
-function isComponentDidUpdate(node: TSESTree.Node) {
-  return AST.isMethodOrProperty(node)
-    && node.key.type === T.Identifier
-    && node.key.name === "componentDidUpdate";
-}
-
 export default createRule<[], MessageID>({
   meta: {
     type: "problem",
@@ -46,7 +40,7 @@ export function create(context: RuleContext<MessageID, []>): RuleListener {
         return;
       }
       const clazz = AST.findParentNode(node, ER.isClassComponent);
-      const method = AST.findParentNode(node, (n) => n === clazz || isComponentDidUpdate(n));
+      const method = AST.findParentNode(node, (n) => n === clazz || ER.isComponentDidUpdate(n));
       if (clazz == null || method == null || method === clazz) return;
       const methodScope = context.sourceCode.getScope(method);
       const upperScope = context.sourceCode.getScope(node).upper;

packages/plugins/eslint-plugin-react-x/src/rules/no-set-state-in-component-will-update.ts

@@ -14,12 +14,6 @@ export const RULE_FEATURES = [] as const satisfies RuleFeature[];
 
 export type MessageID = CamelCase<typeof RULE_NAME>;
 
-function isComponentWillUpdate(node: TSESTree.Node) {
-  return AST.isMethodOrProperty(node)
-    && node.key.type === T.Identifier
-    && node.key.name === "componentWillUpdate";
-}
-
 export default createRule<[], MessageID>({
   meta: {
     type: "problem",
@@ -47,7 +41,7 @@ export function create(context: RuleContext<MessageID, []>): RuleListener {
         return;
       }
       const clazz = AST.findParentNode(node, ER.isClassComponent);
-      const method = AST.findParentNode(node, (n) => n === clazz || isComponentWillUpdate(n));
+      const method = AST.findParentNode(node, (n) => n === clazz || ER.isComponentWillUpdate(n));
       if (clazz == null || method == null || method === clazz) return;
       const methodScope = context.sourceCode.getScope(method);
       const upperScope = context.sourceCode.getScope(node).upper;

packages/utilities/kit/docs/@eslint-react/namespaces/LanguagePreference/README.md

@@ -9,3 +9,7 @@
 ## Variables
 
 - [DEFAULT\_LANGUAGE\_PREFERENCE](variables/DEFAULT_LANGUAGE_PREFERENCE.md)
+
+## Functions
+
+- [make](functions/make.md)

packages/utilities/kit/docs/@eslint-react/namespaces/LanguagePreference/functions/make.md

@@ -0,0 +1,39 @@
+[**@eslint-react/kit**](../../../../README.md)
+
+***
+
+[@eslint-react/kit](../../../../README.md) / [LanguagePreference](../README.md) / make
+
+# Function: make()
+
+> **make**(): `object`
+
+Get a copy of the default LanguagePreference.
+
+## Returns
+
+`object`
+
+### indentStyle
+
+> **indentStyle**: `"tab"` \| `"space"`
+
+### indentWidth
+
+> **indentWidth**: `number`
+
+### jsxQuoteStyle
+
+> **jsxQuoteStyle**: `"single"` \| `"double"`
+
+### quoteStyle
+
+> **quoteStyle**: `"single"` \| `"double"`
+
+### semicolons
+
+> **semicolons**: `"always"` \| `"asNeeded"`
+
+### trailingCommas
+
+> **trailingCommas**: `"all"` \| `"es5"` \| `"none"`

packages/utilities/kit/src/LanguagePreference/LanguagePreference.ts

@@ -19,6 +19,15 @@ export const DEFAULT_LANGUAGE_PREFERENCE = {
   trailingCommas: "all",
 } as const satisfies LanguagePreference;
 
+/**
+ * Get a copy of the default LanguagePreference.
+ */
+export function make(): LanguagePreference {
+  return {
+    ...DEFAULT_LANGUAGE_PREFERENCE,
+  };
+}
+
 declare module "@typescript-eslint/utils/ts-eslint" {
   export interface SharedConfigurationSettings {
     // TODO: Add the language preference to the shared configuration settings when it is ready.