@exodus/bigint 1.1.0 → 1.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/bigint",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "bigint wrappers for native BigUnt, bn.js and others",
   "main": "src/index.js",
   "author": "Exodus Movement, Inc.",
@@ -16,13 +16,9 @@
     "lodash": "^4.17.11",
     "minimalistic-assert": "^1.0.1"
   },
-  "devDependencies": {
-    "eslint": "^8.43.0",
-    "jest": "^29.3.1"
-  },
   "scripts": {
-    "test": "jest",
-    "lint": "eslint . --ignore-path ../../.gitignore",
+    "test": "run -T jest",
+    "lint": "run -T eslint . --ignore-path ../../.gitignore",
     "lint:fix": "yarn lint --fix"
   },
   "repository": {
@@ -32,5 +28,5 @@
   "bugs": {
     "url": "https://github.com/ExodusMovement/exodus-core/issues?q=is%3Aissue+is%3Aopen+label%3Abigint"
   },
-  "gitHead": "e5bfe616c692bc9f8f28af812777646895306a76"
+  "gitHead": "bcdb90e9b23d714d7f8c648ab11bc92832396ae6"
 }

package/src/bn.js

@@ -8,14 +8,26 @@ const FACTORY_SYMBOL = Symbol('bn-wrapper')
 
 const unwrap = (value) => (value instanceof Wrapper ? value.unwrap() : value)
 
+const BI_ZERO = new BN(0)
+
 export default class Wrapper {
   static name = 'bn.js'
   static isUnderlyingInstance(a) {
     return BN.isBN(a)
   }
 
-  static wrap(value, base) {
-    return new Wrapper(FACTORY_SYMBOL, value, base)
+  static wrap(value, base = 10) {
+    if (!BN.isBN(value)) {
+      if (typeof value !== 'string' && typeof value !== 'number') {
+        throw new TypeError(`Unsupported type: ${typeof value}`)
+      }
+
+      if (base !== 10 && base !== 16) throw new TypeError(`Unsupported base: ${base}`)
+
+      value = new BN(value, base)
+    }
+
+    return new Wrapper(FACTORY_SYMBOL, value)
   }
 
   static get ZERO() {
@@ -28,20 +40,10 @@ export default class Wrapper {
 
   __value__
 
-  constructor(factorySymbol, value, base = 10) {
+  constructor(factorySymbol, value) {
     if (factorySymbol !== FACTORY_SYMBOL) throw new Error('use wrap() instead')
 
-    if (BN.isBN(value)) {
-      this.__value__ = value
-    } else {
-      if (typeof value !== 'string' && typeof value !== 'number') {
-        throw new TypeError(`Unsupported type: ${typeof value}`)
-      }
-
-      if (base !== 10 && base !== 16) throw new TypeError(`Unsupported base: ${base}`)
-
-      this.__value__ = new BN(value, base)
-    }
+    this.__value__ = value
   }
 
   unwrap() {
@@ -138,7 +140,7 @@ export default class Wrapper {
   }
 
   isPositive() {
-    return this.__value__.gt(ZERO.unwrap())
+    return this.__value__.gt(BI_ZERO)
   }
 
   toNumber() {
@@ -159,5 +161,5 @@ export default class Wrapper {
   }
 }
 
-const ZERO = new Wrapper(FACTORY_SYMBOL, 0)
-const TEN = new Wrapper(FACTORY_SYMBOL, 10)
+const ZERO = Wrapper.wrap(0)
+const TEN = Wrapper.wrap(10)

package/src/native-bigint.js

@@ -1,17 +1,13 @@
 // TODO: calculator delegator should wrap/unwrap all BigInt instances
 // so that callers never gett a raw BigInt instance back
 
-import { memoize } from 'lodash'
 import assert from 'minimalistic-assert'
 
 const FACTORY_SYMBOL = Symbol('bigint-wrapper')
 
 const unwrap = (value) => (value instanceof Wrapper ? value.unwrap() : value)
 
-const memoizedWrap = memoize((value) => Wrapper.wrap(value))
-
 const BI_ZERO = BigInt(0)
-const BI_TEN = BigInt(10)
 
 export default class Wrapper {
   static name = 'native-bigint'
@@ -36,11 +32,11 @@ export default class Wrapper {
   }
 
   static get ZERO() {
-    return memoizedWrap(BI_ZERO)
+    return ZERO
   }
 
   static get TEN() {
-    return memoizedWrap(BI_TEN)
+    return TEN
   }
 
   __value__
@@ -174,3 +170,6 @@ export default class Wrapper {
     return Buffer.from(hex.padStart(width * 2, '0').slice(0, width * 2), 'hex')
   }
 }
+
+const ZERO = Wrapper.wrap(0)
+const TEN = Wrapper.wrap(10)