refactor(nodes)!: object properties

Signed-off-by: Lexus Drumgold <unicornware@flexdevelopment.llc>

src/__tests__/esast.spec-d.ts

@@ -34,7 +34,7 @@ describe('unit-d:esast', () => {
         | mdast.Nodes
         | TestSubject.Comment
         | TestSubject.ExpressionStatement
-        | TestSubject.ShorthandProperty
+        | TestSubject.MethodProperty
         | TestSubject.TemplateLiteral
       >
 
@@ -52,9 +52,9 @@ describe('unit-d:esast', () => {
         .toBeNever()
     })
 
-    it('should not include ShorthandProperty', () => {
+    it('should not include MethodProperty', () => {
       expectTypeOf<Subject>()
-        .extract<TestSubject.ShorthandProperty>()
+        .extract<TestSubject.MethodProperty>()
         .toBeNever()
     })
 

src/nodes/__tests__/property-shorthand.spec-d.ts

@@ -17,10 +17,6 @@ describe('unit-d:nodes/ShorthandProperty', () => {
     expectTypeOf<Subject>().toHaveProperty('assignment').toEqualTypeOf<false>()
   })
 
-  it('should match [computed: false]', () => {
-    expectTypeOf<Subject>().toHaveProperty('computed').toEqualTypeOf<false>()
-  })
-
   it('should match [kind: Extract<PropertyKind, "init">]', () => {
     expectTypeOf<Subject>()
       .toHaveProperty('kind')

src/nodes/definition-method.ts

@@ -11,7 +11,8 @@ import type {
   FunctionExpression,
   MethodKind,
   ModifierList,
-  Parent
+  Parent,
+  PropertyName
 } from '@flex-development/esast'
 
 /**
@@ -38,6 +39,7 @@ interface MethodDefinition extends Parent {
    * @see {@linkcode Expression}
    * @see {@linkcode FunctionExpression}
    * @see {@linkcode ModifierList}
+   * @see {@linkcode PropertyName}
    */
   children:
     | [
@@ -53,7 +55,7 @@ interface MethodDefinition extends Parent {
       ...comments: InternalComments,
       value: FunctionExpression
     ]
-    | [name: Expression, ...comments: Comments, value: FunctionExpression]
+    | [name: PropertyName, ...comments: Comments, value: FunctionExpression]
 
   /**
    * Boolean indicating if method name is computed.
@@ -85,4 +87,4 @@ interface MethodDefinition extends Parent {
   type: 'methodDefinition'
 }
 
-export type { MethodDefinitionData, MethodDefinition as default }
+export type { MethodDefinition as default, MethodDefinitionData }

src/nodes/expression-object.ts

@@ -6,10 +6,8 @@
 import type {
   Comment,
   Data,
-  MethodProperty,
   Parent,
-  ShorthandProperty,
-  SimpleProperty,
+  Property,
   SpreadElement
 } from '@flex-development/esast'
 
@@ -34,18 +32,10 @@ interface ObjectExpression extends Parent {
    * List of children.
    *
    * @see {@linkcode Comment}
-   * @see {@linkcode MethodProperty}
-   * @see {@linkcode ShorthandProperty}
-   * @see {@linkcode SimpleProperty}
+   * @see {@linkcode Property}
    * @see {@linkcode SpreadElement}
    */
-  children: (
-    | Comment
-    | MethodProperty
-    | ShorthandProperty
-    | SimpleProperty
-    | SpreadElement
-  )[]
+  children: (Comment | Property | SpreadElement)[]
 
   /**
    * Info from the ecosystem.
@@ -60,4 +50,4 @@ interface ObjectExpression extends Parent {
   type: 'objectExpression'
 }
 
-export type { ObjectExpressionData, ObjectExpression as default }
+export type { ObjectExpression as default, ObjectExpressionData }

src/nodes/index.ts

@@ -285,7 +285,6 @@ export type {
 export type { default as Property, PropertyData } from './property'
 export type { default as AssignmentProperty } from './property-assignment'
 export type { default as MethodProperty } from './property-method'
-export type { default as RenamedProperty } from './property-renamed'
 export type { default as ShorthandProperty } from './property-shorthand'
 export type { default as SimpleProperty } from './property-simple'
 export type { default as Root, RootData } from './root'

src/nodes/pattern-array.ts

@@ -6,6 +6,7 @@
 import type {
   Comment,
   Data,
+  Expression,
   Nothing,
   Parent,
   Pattern
@@ -32,10 +33,11 @@ interface ArrayPattern extends Parent {
    * List of children.
    *
    * @see {@linkcode Comment}
+   * @see {@linkcode Expression}
    * @see {@linkcode Nothing}
    * @see {@linkcode Pattern}
    */
-  children: (Comment | Nothing | Pattern)[]
+  children: (Comment | Expression | Nothing | Pattern)[]
 
   /**
    * Info from the ecosystem.

src/nodes/pattern-object.ts

@@ -4,13 +4,11 @@
  */
 
 import type {
-  AssignmentProperty,
   Comment,
   Data,
   Parent,
-  RenamedProperty,
-  RestElement,
-  ShorthandProperty
+  Property,
+  RestElement
 } from '@flex-development/esast'
 
 /**
@@ -33,19 +31,11 @@ interface ObjectPattern extends Parent {
   /**
    * List of children.
    *
-   * @see {@linkcode AssignmentProperty}
    * @see {@linkcode Comment}
-   * @see {@linkcode RenamedProperty}
+   * @see {@linkcode Property}
    * @see {@linkcode RestElement}
-   * @see {@linkcode ShorthandProperty}
    */
-  children: (
-    | AssignmentProperty
-    | Comment
-    | RenamedProperty
-    | RestElement
-    | ShorthandProperty
-  )[]
+  children: (Comment | Property | RestElement)[]
 
   /**
    * Info from the ecosystem.
@@ -60,4 +50,4 @@ interface ObjectPattern extends Parent {
   type: 'objectPattern'
 }
 
-export type { ObjectPatternData, ObjectPattern as default }
+export type { ObjectPattern as default, ObjectPatternData }

src/nodes/property-assignment.ts

@@ -6,9 +6,11 @@
 import type { InternalComments } from '#internal'
 import type {
   ArrayPattern,
+  AssignmentExpression,
   AssignmentPattern,
   Comments,
   Expression,
+  Identifier,
   ObjectPattern,
   Property,
   PropertyKind,
@@ -32,9 +34,11 @@ interface AssignmentProperty extends Property {
    * List of children.
    *
    * @see {@linkcode ArrayPattern}
+   * @see {@linkcode AssignmentExpression}
    * @see {@linkcode AssignmentPattern}
    * @see {@linkcode Comments}
    * @see {@linkcode Expression}
+   * @see {@linkcode Identifier}
    * @see {@linkcode ObjectPattern}
    * @see {@linkcode PropertyName}
    */
@@ -43,14 +47,14 @@ interface AssignmentProperty extends Property {
       ...comments: Comments,
       key: Expression,
       ...comments: InternalComments,
-      value: ArrayPattern | AssignmentPattern | ObjectPattern
+      value: ArrayPattern | AssignmentPattern | Identifier | ObjectPattern
     ]
     | [
       key: PropertyName,
       ...comments: Comments,
-      value: ArrayPattern | AssignmentPattern | ObjectPattern
+      value: ArrayPattern | AssignmentPattern | Identifier | ObjectPattern
     ]
-    | [assignment: AssignmentPattern]
+    | [assignment: AssignmentExpression | AssignmentPattern]
 
   /**
    * Property kind.

src/nodes/property-method.ts

@@ -3,15 +3,7 @@
  * @module esast/nodes/MethodProperty
  */
 
-import type { InternalComments } from '#internal'
-import type {
-  Comments,
-  Expression,
-  FunctionExpression,
-  ModifierList,
-  Property,
-  PropertyName
-} from '@flex-development/esast'
+import type { MethodDefinition, Property } from '@flex-development/esast'
 
 /**
  * A method property.
@@ -29,27 +21,9 @@ interface MethodProperty extends Property {
   /**
    * List of children.
    *
-   * @see {@linkcode Comments}
-   * @see {@linkcode Expression}
-   * @see {@linkcode FunctionExpression}
-   * @see {@linkcode PropertyName}
-   * @see {@linkcode ModifierList}
+   * @see {@linkcode MethodDefinition}
    */
-  children:
-    | [
-      ...comments: Comments,
-      key: Expression,
-      ...comments: InternalComments,
-      value: FunctionExpression
-    ]
-    | [
-      modifiers: ModifierList,
-      ...comments: Comments,
-      key: Expression,
-      ...comments: InternalComments,
-      value: FunctionExpression
-    ]
-    | [key: PropertyName, ...comments: Comments, value: FunctionExpression]
+  children: [MethodDefinition]
 
   /**
    * Boolean indicating if property value is a method.

src/nodes/property-shorthand.ts

@@ -3,10 +3,13 @@
  * @module esast/nodes/ShorthandProperty
  */
 
+import type { InternalComments } from '#internal'
 import type {
-  Identifier,
+  Comments,
+  Expression,
   Property,
-  PropertyKind
+  PropertyKind,
+  PropertyName
 } from '@flex-development/esast'
 
 /**
@@ -25,14 +28,17 @@ interface ShorthandProperty extends Property {
   /**
    * List of children.
    *
-   * @see {@linkcode Identifier}
+   * @see {@linkcode Comments}
+   * @see {@linkcode Expression}
+   * @see {@linkcode PropertyName}
    */
-  children: [key: Identifier]
-
-  /**
-   * Boolean indicating if property is computed.
-   */
-  computed: false
+  children:
+    | [
+      ...comments: Comments,
+      key: Expression,
+      ...comments: InternalComments
+    ]
+    | [key: PropertyName]
 
   /**
    * Property kind.

src/nodes/property-simple.ts

@@ -7,7 +7,6 @@ import type { InternalComments } from '#internal'
 import type {
   Comments,
   Expression,
-  FunctionExpression,
   Property,
   PropertyKind,
   PropertyName
@@ -38,13 +37,9 @@ interface SimpleProperty extends Property {
       ...comments: Comments,
       key: Expression,
       ...comments: InternalComments,
-      value: Exclude<Expression, FunctionExpression>
-    ]
-    | [
-      key: PropertyName,
-      ...comments: Comments,
-      value: Exclude<Expression, FunctionExpression>
+      value: Expression
     ]
+    | [key: PropertyName, ...comments: Comments, value: Expression]
 
   /**
    * Property kind.