From a1ac23c7897cb27e9f9e7f04727c870963eee870 Mon Sep 17 00:00:00 2001
From: Eric ZHANG <11440841+ericzhangjx@users.noreply.github.com>
Date: Wed, 26 Aug 2020 00:50:16 +0800
Subject: [PATCH] fix: Support for type operators `readonly` & `unique`

---
 src/lib/converter/types/type-operator.ts | 14 +++++++++++++-
 src/lib/models/types/type-operator.ts    | 13 +++----------
 2 files changed, 16 insertions(+), 11 deletions(-)

diff --git a/src/lib/converter/types/type-operator.ts b/src/lib/converter/types/type-operator.ts
index 872b3551b..77b887888 100644
--- a/src/lib/converter/types/type-operator.ts
+++ b/src/lib/converter/types/type-operator.ts
@@ -11,6 +11,17 @@ export class TypeOperatorConverter extends ConverterTypeComponent implements Typ
      */
     priority = 50;
 
+    /**
+     * add two more operators based on the references below:
+     * https://github.com/microsoft/TypeScript/blob/e83102134e5640abb78b0c62941b3b5003ab6c1a/src/compiler/types.ts#L1602
+     * https://github.com/TypeStrong/typedoc/blob/b7a5b2d5ea1ae088e9510783ede20e842b120d0f/src/lib/converter/types.ts#L375
+     */
+    private readonly supportedOperatorNames = {
+        [ts.SyntaxKind.KeyOfKeyword]: 'keyof',
+        [ts.SyntaxKind.UniqueKeyword]: 'unique',
+        [ts.SyntaxKind.ReadonlyKeyword]: 'readonly'
+    } as const;
+
     /**
      * Test whether this converter can handle the given TypeScript node.
      */
@@ -28,7 +39,8 @@ export class TypeOperatorConverter extends ConverterTypeComponent implements Typ
     convertNode(context: Context, node: ts.TypeOperatorNode): TypeOperatorType | undefined {
         const target = this.owner.convertType(context, node.type);
         if (target) {
-            return new TypeOperatorType(target);
+            const operator = this.supportedOperatorNames[node.operator];
+            return new TypeOperatorType(target, operator);
         }
     }
 }
diff --git a/src/lib/models/types/type-operator.ts b/src/lib/models/types/type-operator.ts
index 75623f4ce..fcd8f81fb 100644
--- a/src/lib/models/types/type-operator.ts
+++ b/src/lib/models/types/type-operator.ts
@@ -14,15 +14,8 @@ export class TypeOperatorType extends Type {
      */
     readonly type = 'typeOperator';
 
-    target: Type;
-
-    // currently, there is only one type operator, this is always "keyof"
-    // but, if more types will be added in the future we are ready.
-    readonly operator = 'keyof';
-
-    constructor(target: Type) {
+    constructor(public target: Type, public operator: 'keyof' | 'unique' | 'readonly') {
         super();
-        this.target = target;
     }
 
     /**
@@ -31,7 +24,7 @@ export class TypeOperatorType extends Type {
      * @return A clone of this type.
      */
     clone(): Type {
-        return new TypeOperatorType(this.target.clone());
+        return new TypeOperatorType(this.target.clone(), this.operator);
     }
 
     /**
@@ -45,7 +38,7 @@ export class TypeOperatorType extends Type {
             return false;
         }
 
-        return type.target.equals(this.target);
+        return type instanceof TypeOperatorType && type.operator === this.operator && type.target.equals(this.target);
     }
 
     /**