chore: remove type assertion in objectPatternPropertyCompareFn

Author: trivikr

src/transforms/v2-to-v3/modules/objectPatternPropertyCompareFn.ts

@@ -7,8 +7,6 @@ import type {
   SpreadPropertyPattern,
 } from "jscodeshift";
 
-import { OBJECT_PROPERTY_TYPE_LIST } from "../config";
-
 export type ObjectPatternProperty =
   | Property
   | PropertyPattern
@@ -21,14 +19,14 @@ export const objectPatternPropertyCompareFn = (
   property1: ObjectPatternProperty,
   property2: ObjectPatternProperty
 ) => {
-  if (
-    OBJECT_PROPERTY_TYPE_LIST.includes(property1.type) &&
-    OBJECT_PROPERTY_TYPE_LIST.includes(property2.type)
-  ) {
-    const property1Key = (property1 as Property | ObjectProperty).key;
-    const property2Key = (property2 as Property | ObjectProperty).key;
-    if (property1Key.type === "Identifier" && property2Key.type === "Identifier")
-      return property1Key.name.localeCompare(property2Key.name);
+  if (property1.type !== "Property" && property1.type !== "ObjectProperty") {
+    return 0;
+  }
+  if (property2.type !== "Property" && property2.type !== "ObjectProperty") {
+    return 0;
+  }
+  if (property1.key.type === "Identifier" && property2.key.type === "Identifier") {
+    return property1.key.name.localeCompare(property2.key.name);
   }
   return 0;
 };