feat(util): add more robust library

we now use the most popular uuid library out there, which is also environment aware. This means the lib provides a browser field in package.json which will be used by bundlers to use the correct implementation to generate secure Random numbers. (using the native crypto implementation if available falling back to Math.random)

Author: michaelwittwer

package-lock.json

@@ -71,12 +71,14 @@
     "tslint": "^5.11.0",
     "tslint-config-prettier": "^1.16.0",
     "typedoc": "^0.13.0",
-    "typescript": "^3.1.6"
+    "typescript": "^3.1.6",
+    "uuid": "^3.3.2"
   },
   "peerDependencies": {
     "aws-sdk": "^2.286.1",
     "lodash": "^4.17.10",
     "reflect-metadata": "^0.1.12",
-    "rxjs": "^6.0.0"
+    "rxjs": "^6.0.0",
+    "uuid": "^3.3.2"
   }
 }

package.json

@@ -1,4 +1,5 @@
 import { isNumber, isString } from 'lodash'
+import {v4} from 'uuid'
 import { AttributeCollectionType, AttributeType } from './type/attribute-type.type'
 import { AttributeValueType } from './type/attribute-value-type.type'
 import { Attribute } from './type/attribute.type'
@@ -252,12 +253,6 @@ export function typeName(type: any): 'Null' | 'Undefined' | string {
   throw new Error(`was not able to resolve type name for type ${type}`)
 }
 
-// FIXME UUID replace with a more bullet proof implementation node uuid module requires crypto, need to figure out how to use it with browser
 export function uuidv4() {
-  return 'xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx'.replace(/[xy]/g, c => {
-    // tslint:disable
-    let r = (Math.random() * 16) | 0,
-      v = c == 'x' ? r : (r & 0x3) | 0x8
-    return v.toString(16)
-  })
+  return v4()
 }