@exodus/bigint 1.1.0 → 2.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/bigint",
-  "version": "1.1.0",
+  "version": "2.0.0",
   "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": "8e7de47c76a1bfcb1e29a7663a445ee8cf7268c7"
 }

package/src/bn.js

@@ -8,40 +8,34 @@ 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}`)
+      }
 
-  static get ZERO() {
-    return ZERO
-  }
+      if (base !== 10 && base !== 16) throw new TypeError(`Unsupported base: ${base}`)
 
-  static get TEN() {
-    return TEN
+      value = new BN(value, base)
+    }
+
+    return new Wrapper(FACTORY_SYMBOL, value)
   }
 
   __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 +132,7 @@ export default class Wrapper {
   }
 
   isPositive() {
-    return this.__value__.gt(ZERO.unwrap())
+    return this.__value__.gt(BI_ZERO)
   }
 
   toNumber() {
@@ -157,7 +151,8 @@ export default class Wrapper {
   toBaseBufferBE(length) {
     return this.__value__.toBuffer('be', length)
   }
-}
 
-const ZERO = new Wrapper(FACTORY_SYMBOL, 0)
-const TEN = new Wrapper(FACTORY_SYMBOL, 10)
+  isBigIntWrapper() {
+    return true
+  }
+}

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'
@@ -35,14 +31,6 @@ export default class Wrapper {
     return typeof value === 'bigint'
   }
 
-  static get ZERO() {
-    return memoizedWrap(BI_ZERO)
-  }
-
-  static get TEN() {
-    return memoizedWrap(BI_TEN)
-  }
-
   __value__
 
   constructor(factorySymbol, value) {
@@ -173,4 +161,8 @@ export default class Wrapper {
     const hex = this.__value__.toString(16)
     return Buffer.from(hex.padStart(width * 2, '0').slice(0, width * 2), 'hex')
   }
+
+  isBigIntWrapper() {
+    return true
+  }
 }