refactor: use math random better for uuid

reallygoodsoftware · Jun 20, 2024 · b6b2e24 · b6b2e24
1 parent c2ab6cf
commit b6b2e24
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/lib/entity.js b/lib/entity.js
@@ -106,13 +106,7 @@ export class Entity {
   }
 
   generateEntityUUID() {
-    return 'Entity' + Date.now() + this.cryptoRandomString()
-  }
-
-  cryptoRandomString() {
-    return ([1e7] + -1e3 + -4e3 + -8e3 + -1e11).replace(/[018]/g, c =>
-      (c ^ crypto.getRandomValues(new Uint8Array(1))[0] & 15 >> c / 4).toString(16)
-    );
+    return 'Entity' + Math.floor(performance.now() * 1000).toString(36) + Math.random().toString(36).substr(2, 9);
   }
 
   async init() {

diff --git a/lib/generators/observer.js b/lib/generators/observer.js
@@ -146,7 +146,7 @@ export class Observer {
   }
 
   generateUUID() {
-    return 'ScriptID' + Date.now() + this.cryptoRandomString()
+    return 'ScriptID' + Math.floor(performance.now() * 1000).toString(36) + Math.random().toString(36).substr(2, 9);
   }
 
   cryptoRandomString() {