From ed0bda2aeca649bd953c4b96b4675a6c35bfc232 Mon Sep 17 00:00:00 2001
From: Spring Chiu <fospring@outlook.com>
Date: Sun, 25 Feb 2024 12:40:31 +0800
Subject: [PATCH] chore: simplify reconstruct schema structure

---
 AUTO_RECONSCTRUCT_BY_JSON_SCHEME.md      | 26 ++++++++----------------
 examples/src/status-deserialize-class.js | 14 +++++--------
 2 files changed, 14 insertions(+), 26 deletions(-)

diff --git a/AUTO_RECONSCTRUCT_BY_JSON_SCHEME.md b/AUTO_RECONSCTRUCT_BY_JSON_SCHEME.md
index df663780..d0d9fe4c 100644
--- a/AUTO_RECONSCTRUCT_BY_JSON_SCHEME.md
+++ b/AUTO_RECONSCTRUCT_BY_JSON_SCHEME.md
@@ -70,29 +70,26 @@ this.car.run() # This now works!
 ## The schema format
 The Schema supports multiple types: 
 
-* Primitive types: `string`, `number`, `boolean`
+* Primitive types: `string`, `number`, `boolean`. We can remove schema format of `Primitive types` since is no need to reconstruct them.
 * Built-in object types: `Date`, `BigInt`.
 * Built-in collections: `array`, `map`
-  * Arrays need to be declared as `{array: {value: valueType}}` 
-  * Maps need to declared as `{map: {key: 'keyType', value: 'valueType'}}`
+  * Arrays need to be declared as `{array: {value: valueType}}`, there are no reconstruct for `Primitive types`, for the value type is `Primitive types`, we can remove this field.
+  * Maps need to declared as `{map: {key: 'keyType', value: 'valueType'}}`, there are no reconstruct for `Primitive types`, for the key and value type are `Primitive types`, we can remove this field.
 * Custom classes are denoted by their name, e.g. `Car`
-* Near SDK Collections (i.e. `Vector`, `LookupMap`, `LookupSet`, `UnorderedMap`, `UnorderedSet`) need to be declared as `{class: ClassType, value: ValueType}`
+* Near SDK Collections (i.e. `Vector`, `LookupMap`, `LookupSet`, `UnorderedMap`, `UnorderedSet`) need to be declared as `{class: ClassType, value: ValueType}` if we need to reconstruct value or we can simplify to mark `ClassType` if we no need to reconstruct value for `Primitive types`.
 
-You can see a complete example in the [status-deserialize-class](./examples/src/status-deserialize-class.js) file, which containts the following Class declaration:
+You can see a complete example in the [status-deserialize-class](./examples/src/status-deserialize-class.js) file, which contains the following Class declaration:
 
 ```js
 export class StatusDeserializeClass {
   static schema = {
-    is_inited: "boolean",
-    records: {map: { key: 'string', value: 'string' }},
     truck: Truck,
-    messages: {array: {value: 'string'}},
-    efficient_recordes: {class: UnorderedMap},
+    efficient_recordes: UnorderedMap,
     nested_efficient_recordes: {class: UnorderedMap, value: UnorderedMap},
-    nested_lookup_recordes:  {class: UnorderedMap, value: {class: LookupMap }},
-    vector_nested_group: {class: Vector, value: { class: LookupMap }},
+    nested_lookup_recordes:  {class: UnorderedMap, value: LookupMap},
+    vector_nested_group: {class: Vector, value: LookupMap},
     lookup_nest_vec: { class: LookupMap, value: Vector },
-    unordered_set: {class: UnorderedSet },
+    unordered_set: UnorderedSet,
     user_car_map: {class: UnorderedMap, value: Car },
     big_num: 'bigint',
     date: 'date'
@@ -103,15 +100,10 @@ export class StatusDeserializeClass {
     this.records = {};
     this.truck = new Truck();
     this.messages = [];
-    // account_id -> message
     this.efficient_recordes = new UnorderedMap("a");
-    // id -> account_id -> message
     this.nested_efficient_recordes = new UnorderedMap("b");
-    // id -> account_id -> message
     this.nested_lookup_recordes = new UnorderedMap("c");
-    // index -> account_id -> message
     this.vector_nested_group = new Vector("d");
-    // account_id -> index -> message
     this.lookup_nest_vec = new LookupMap("e");
     this.unordered_set = new UnorderedSet("f");
     this.user_car_map = new UnorderedMap("g");
diff --git a/examples/src/status-deserialize-class.js b/examples/src/status-deserialize-class.js
index aa76039c..e2bf53f7 100644
--- a/examples/src/status-deserialize-class.js
+++ b/examples/src/status-deserialize-class.js
@@ -28,8 +28,7 @@ class Truck {
     static schema = {
         name: "string",
         speed: "number",
-        // loads: {collection: {reconstructor: UnorderedMap.reconstruct, value: 'string'}}
-        loads: {class: UnorderedMap }
+        loads: UnorderedMap
     };
     constructor() {
         this.name = "";
@@ -48,16 +47,13 @@ class Truck {
 @NearBindgen({})
 export class StatusDeserializeClass {
     static schema = {
-        is_inited: "boolean",
-        records: {map: { key: 'string', value: 'string' }},
         truck: Truck,
-        messages: {array: {value: 'string'}},
-        efficient_recordes: {class: UnorderedMap},
+        efficient_recordes: UnorderedMap,
         nested_efficient_recordes: {class: UnorderedMap, value: UnorderedMap},
-        nested_lookup_recordes:  {class: UnorderedMap, value: {class: LookupMap }},
-        vector_nested_group: {class: Vector, value: { class: LookupMap }},
+        nested_lookup_recordes:  {class: UnorderedMap, value: LookupMap},
+        vector_nested_group: {class: Vector, value: LookupMap},
         lookup_nest_vec: { class: LookupMap, value: Vector },
-        unordered_set: {class: UnorderedSet },
+        unordered_set: UnorderedSet,
         user_car_map: {class: UnorderedMap, value: Car },
         big_num: 'bigint',
         date: 'date'