refactor(es/ast): Improve type definitions of patterns (#8532)

**Description:**

 - Copy `AssignmentTarget` from `oxc`.
 - Use `BindingIdentifier` in more places.

**Related issue:**
 
 - Closes #8026

Author: kdy1

crates/swc/tests/fixture/issues-6xxx/6217/1/output/input.js

@@ -1,4 +1,4 @@
 let b;
-for(let i = 0; i < 2; ++i)b = 0 === i ? 1 : 2, console.log(0, b);
+for(let i = 0; i < 2; ++i)console.log(0, b = 0 === i ? 1 : 2);
 let c = 1, d;
 while(c--);

crates/swc/tests/fixture/issues-6xxx/6217/2/output/input.js

@@ -1,4 +1,4 @@
 let b, c, d;
-for(let i = 0; i < 2; ++i)b = 0 === i ? 1 : 2, console.log(0, b);
+for(let i = 0; i < 2; ++i)console.log(0, b = 0 === i ? 1 : 2);
 c = 1;
 while(c--);

crates/swc/tests/fixture/issues-8xxx/8412/output/index.js

@@ -1,7 +1,6 @@
 export var fn = function() {
-    var varA;
-    return(// a bad comment
-    varA = condCheck ? "a" : "b", objCreator({
-        varA: varA
-    }));
+    return objCreator({
+        varA: condCheck ? // a bad comment
+        "a" : "b"
+    });
 };

crates/swc/tests/tsc-references/ClassAndModuleThatMergeWithStaticFunctionAndExportedFunctionThatShareAName.1.normal.js

@@ -7,13 +7,13 @@ var Point = /*#__PURE__*/ function() {
         this.x = x;
         this.y = y;
     }
-    Point.Origin // unexpected error here bug 840246
-     = function Origin() {
+    Point.Origin = function Origin() {
         return {
             x: 0,
             y: 0
         };
-    };
+    } // unexpected error here bug 840246
+    ;
     return Point;
 }();
 (function(Point) {
@@ -31,13 +31,13 @@ var A;
             this.x = x;
             this.y = y;
         }
-        Point.Origin // unexpected error here bug 840246
-         = function Origin() {
+        Point.Origin = function Origin() {
             return {
                 x: 0,
                 y: 0
             };
-        };
+        } // unexpected error here bug 840246
+        ;
         return Point;
     }();
     A.Point = Point;

crates/swc/tests/tsc-references/classAbstractSuperCalls.1.normal.js

@@ -44,10 +44,10 @@ var C = /*#__PURE__*/ function(B) {
     _proto.foo = function foo() {
         return 2;
     };
-    _proto.qux // 2 errors, foo is abstract
-     = function qux() {
+    _proto.qux = function qux() {
         return _get(_get_prototype_of(C.prototype), "foo", this).call(this) || _get(_get_prototype_of(C.prototype), "foo", this);
-    };
+    } // 2 errors, foo is abstract
+    ;
     _proto.norf = function norf() {
         return _get(_get_prototype_of(C.prototype), "bar", this).call(this);
     };

crates/swc/tests/tsc-references/classConstructorAccessibility2.1.normal.js

@@ -79,14 +79,14 @@ var DerivedB = /*#__PURE__*/ function(BaseB1) {
     _proto.createInstance = function createInstance() {
         new DerivedB(7);
     };
-    _proto.createBaseInstance // ok
-     = function createBaseInstance() {
+    _proto.createBaseInstance = function createBaseInstance() {
         new BaseB(8);
-    };
-    DerivedB.staticBaseInstance // ok
-     = function staticBaseInstance() {
+    } // ok
+    ;
+    DerivedB.staticBaseInstance = function staticBaseInstance() {
         new BaseB(9);
-    };
+    } // ok
+    ;
     return DerivedB;
 }(BaseB);
 var DerivedC = /*#__PURE__*/ function(BaseC1) {
@@ -104,14 +104,14 @@ var DerivedC = /*#__PURE__*/ function(BaseC1) {
     _proto.createInstance = function createInstance() {
         new DerivedC(9);
     };
-    _proto.createBaseInstance // error
-     = function createBaseInstance() {
+    _proto.createBaseInstance = function createBaseInstance() {
         new BaseC(10);
-    };
-    DerivedC.staticBaseInstance // error
-     = function staticBaseInstance() {
+    } // error
+    ;
+    DerivedC.staticBaseInstance = function staticBaseInstance() {
         new BaseC(11);
-    };
+    } // error
+    ;
     return DerivedC;
 }(BaseC);
 var ba = new BaseA(1);

crates/swc/tests/tsc-references/classConstructorAccessibility5.1.normal.js

@@ -14,20 +14,20 @@ var Derived = /*#__PURE__*/ function(Base1) {
         _class_call_check(this, Derived);
         return _super.apply(this, arguments);
     }
-    Derived.make // ok
-     = function make() {
+    Derived.make = function make() {
         new Base();
-    };
+    } // ok
+    ;
     return Derived;
 }(Base);
 var Unrelated = /*#__PURE__*/ function() {
     "use strict";
     function Unrelated() {
         _class_call_check(this, Unrelated);
     }
-    Unrelated.fake // error
-     = function fake() {
+    Unrelated.fake = function fake() {
         new Base();
-    };
+    } // error
+    ;
     return Unrelated;
 }();

crates/swc/tests/tsc-references/computedPropertyNames11_ES5.2.minified.js

@@ -1,7 +1,7 @@
 //// [computedPropertyNames11_ES5.ts]
 import { _ as _define_enumerable_properties } from "@swc/helpers/_/_define_enumerable_properties";
-var s, n, a, _mutatorMap = {};
-_mutatorMap[s] = _mutatorMap[s] || {}, _mutatorMap[s].get = function() {
+var s, n, a, _obj, _mutatorMap = {};
+_obj = {}, _mutatorMap[s] = _mutatorMap[s] || {}, _mutatorMap[s].get = function() {
     return 0;
 }, _mutatorMap[n] = _mutatorMap[n] || {}, _mutatorMap[n].set = function(v) {}, _mutatorMap[s + s] = _mutatorMap[s + s] || {}, _mutatorMap[s + s].get = function() {
     return 0;
@@ -13,4 +13,4 @@ _mutatorMap[s] = _mutatorMap[s] || {}, _mutatorMap[s].get = function() {
     return 0;
 }, _mutatorMap["hello bye"] = _mutatorMap["hello bye"] || {}, _mutatorMap["hello bye"].set = function(v) {}, _mutatorMap["hello ".concat(a, " bye")] = _mutatorMap["hello ".concat(a, " bye")] || {}, _mutatorMap["hello ".concat(a, " bye")].get = function() {
     return 0;
-}, _define_enumerable_properties({}, _mutatorMap);
+}, _define_enumerable_properties(_obj, _mutatorMap);

crates/swc/tests/tsc-references/computedPropertyNames1_ES5.2.minified.js

@@ -1,6 +1,6 @@
 //// [computedPropertyNames1_ES5.ts]
 import { _ as _define_enumerable_properties } from "@swc/helpers/_/_define_enumerable_properties";
-var _mutatorMap = {};
-_mutatorMap[1] = _mutatorMap[1] || {}, _mutatorMap[1].get = function() {
+var _obj, _mutatorMap = {};
+_obj = {}, _mutatorMap[1] = _mutatorMap[1] || {}, _mutatorMap[1].get = function() {
     return 0;
-}, _mutatorMap[1] = _mutatorMap[1] || {}, _mutatorMap[1].set = function(v) {}, _define_enumerable_properties({}, _mutatorMap);
+}, _mutatorMap[1] = _mutatorMap[1] || {}, _mutatorMap[1].set = function(v) {}, _define_enumerable_properties(_obj, _mutatorMap);

crates/swc/tests/tsc-references/derivedClassTransitivity.1.normal.js

@@ -21,8 +21,8 @@ var D = /*#__PURE__*/ function(C) {
         return _super.apply(this, arguments);
     }
     var _proto = D.prototype;
-    _proto.foo // ok to drop parameters
-     = function foo() {};
+    _proto.foo = function foo() {} // ok to drop parameters
+    ;
     return D;
 }(C);
 var E = /*#__PURE__*/ function(D) {
@@ -34,8 +34,8 @@ var E = /*#__PURE__*/ function(D) {
         return _super.apply(this, arguments);
     }
     var _proto = E.prototype;
-    _proto.foo // ok to add optional parameters
-     = function foo(x) {};
+    _proto.foo = function foo(x) {} // ok to add optional parameters
+    ;
     return E;
 }(D);
 var c;