@ledgerhq/coin-polkadot 0.1.7 → 0.2.0-next.1

package/.eslintrc.js

@@ -1,57 +1,19 @@
 module.exports = {
-  parser: "@typescript-eslint/parser",
   env: {
     browser: true,
     es6: true,
-    node: true,
-    jest: true,
   },
-  extends: [
-    "eslint:recommended",
-    "plugin:@typescript-eslint/eslint-recommended",
-    "plugin:@typescript-eslint/recommended",
-    "prettier",
+  overrides: [
+    {
+      files: ["src/**/*.test.{ts,tsx}"],
+      env: {
+        "jest/globals": true,
+      },
+      plugins: ["jest"],
+    },
   ],
-  globals: {
-    Atomics: "readonly",
-    SharedArrayBuffer: "readonly",
-  },
-  plugins: ["@typescript-eslint", "prettier"],
   rules: {
     "no-console": ["error", { allow: ["warn", "error"] }],
-    "linebreak-style": ["error", "unix"],
-    semi: ["error", "always"],
-    "no-unused-vars": "off",
-    "import/prefer-default-export": 0,
-    "no-plusplus": 0,
-    "no-underscore-dangle": 0,
-    "prefer-template": 0,
-    "no-await-in-loop": 0,
-    "no-restricted-syntax": 0,
-    "consistent-return": 0,
-    "no-lonely-if": 0,
-    "no-use-before-define": 0,
-    "no-nested-ternary": 0,
-    "import/no-cycle": 0,
-    "no-multi-assign": 0,
-    "guard-for-in": 0,
-    "no-continue": 0,
-    "lines-between-class-members": 0,
-    "prefer-destructuring": 0,
-    "prettier/prettier": "error",
-    "@typescript-eslint/no-use-before-define": "off",
-    "@typescript-eslint/no-unused-vars": ["error", { argsIgnorePattern: "^_" }],
     "@typescript-eslint/no-empty-function": "off",
-    "@typescript-eslint/no-namespace": ["error", { allowDeclarations: true }],
-    "@typescript-eslint/no-explicit-any": 0,
-    "@typescript-eslint/ban-types": [
-      "error",
-      {
-        extendDefaults: true,
-        types: {
-          "{}": false,
-        },
-      },
-    ],
   },
 };

package/.turbo/turbo-build.log

@@ -1,4 +1,4 @@
 
-> @ledgerhq/coin-polkadot@0.1.6 build /home/runner/work/ledger-live/ledger-live/libs/coin-polkadot
+> @ledgerhq/coin-polkadot@0.1.7-next.0 build /home/runner/work/ledger-live/ledger-live/libs/coin-polkadot
 > tsc && tsc -m ES6 --outDir lib-es
 

package/CHANGELOG.md

@@ -1,15 +1,24 @@
 # @ledgerhq/coin-polkadot
 
-## 0.1.7
+## 0.2.0-next.1
+
+### Minor Changes
+
+- [#3660](https://github.com/LedgerHQ/ledger-live/pull/3660) [`895205e898`](https://github.com/LedgerHQ/ledger-live/commit/895205e8981612616d3147de9261d19051dbc0b2) Thanks [@live-github-bot](https://github.com/apps/live-github-bot)! - Update lockedbalance retrieval polkadot
+
+## 0.1.7-next.0
 
 ### Patch Changes
 
-- Updated dependencies [[`4a1454beb3`](https://github.com/LedgerHQ/ledger-live/commit/4a1454beb3f86405ba2686e07879c12a7d35ad8e), [`809065c571`](https://github.com/LedgerHQ/ledger-live/commit/809065c57198646a49adea112b9d799e35a57d25), [`ebe5b07afe`](https://github.com/LedgerHQ/ledger-live/commit/ebe5b07afec441ea3e2d9103da9e1175972add47)]:
-  - @ledgerhq/cryptoassets@9.8.0
-  - @ledgerhq/types-cryptoassets@7.2.1
-  - @ledgerhq/types-live@6.35.1
-  - @ledgerhq/coin-framework@0.3.6
-  - @ledgerhq/live-portfolio@0.0.8
+- Updated dependencies [[`9adc1862dd`](https://github.com/LedgerHQ/ledger-live/commit/9adc1862dda605a722d19f3b6895bd324834c914), [`4a1454beb3`](https://github.com/LedgerHQ/ledger-live/commit/4a1454beb3f86405ba2686e07879c12a7d35ad8e), [`809065c571`](https://github.com/LedgerHQ/ledger-live/commit/809065c57198646a49adea112b9d799e35a57d25), [`d1aa522db7`](https://github.com/LedgerHQ/ledger-live/commit/d1aa522db75f7ea850efe412abaa4dc7d37af6b7), [`ebe5b07afe`](https://github.com/LedgerHQ/ledger-live/commit/ebe5b07afec441ea3e2d9103da9e1175972add47)]:
+  - @ledgerhq/errors@6.12.7-next.0
+  - @ledgerhq/cryptoassets@9.8.0-next.0
+  - @ledgerhq/types-cryptoassets@7.2.1-next.0
+  - @ledgerhq/types-live@6.35.1-next.0
+  - @ledgerhq/coin-framework@0.3.6-next.0
+  - @ledgerhq/devices@8.0.4-next.0
+  - @ledgerhq/hw-app-polkadot@6.27.16-next.0
+  - @ledgerhq/live-portfolio@0.0.8-next.0
 
 ## 0.1.6
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ledgerhq/coin-polkadot",
-  "version": "0.1.7",
+  "version": "0.2.0-next.1",
   "description": "Ledger Polkadot Coin integration",
   "keywords": [
     "Ledger",
@@ -54,32 +54,23 @@
     "invariant": "^2.2.2",
     "lodash": "^4.17.21",
     "rxjs": "^6.6.7",
-    "@ledgerhq/coin-framework": "^0.3.6",
-    "@ledgerhq/cryptoassets": "^9.8.0",
-    "@ledgerhq/devices": "^8.0.3",
-    "@ledgerhq/errors": "^6.12.6",
-    "@ledgerhq/hw-app-polkadot": "^6.27.15",
+    "@ledgerhq/coin-framework": "^0.3.6-next.0",
+    "@ledgerhq/cryptoassets": "^9.8.0-next.0",
+    "@ledgerhq/devices": "^8.0.4-next.0",
+    "@ledgerhq/errors": "^6.12.7-next.0",
+    "@ledgerhq/hw-app-polkadot": "^6.27.16-next.0",
     "@ledgerhq/live-env": "^0.3.0",
-    "@ledgerhq/live-portfolio": "^0.0.8",
+    "@ledgerhq/live-portfolio": "^0.0.8-next.0",
     "@ledgerhq/live-promise": "^0.0.1",
     "@ledgerhq/logs": "^6.10.1",
-    "@ledgerhq/types-cryptoassets": "^7.2.1",
-    "@ledgerhq/types-live": "^6.35.1"
+    "@ledgerhq/types-cryptoassets": "^7.2.1-next.0",
+    "@ledgerhq/types-live": "^6.35.1-next.0"
   },
   "devDependencies": {
     "@types/invariant": "^2.2.2",
     "@types/jest": "^29.2.4",
     "@types/lodash": "^4.14.191",
-    "@typescript-eslint/eslint-plugin": "^5.46.1",
-    "@typescript-eslint/parser": "^5.46.1",
-    "eslint": "^7.32.0",
-    "eslint-config-prettier": "^8.3.0",
-    "eslint-config-typescript": "^3.0.0",
-    "eslint-formatter-pretty": "^3.0.1",
-    "eslint-plugin-prettier": "^3.4.0",
-    "eslint-plugin-typescript": "^0.14.0",
     "jest": "^28.1.1",
-    "prettier": "^2.8.1",
     "ts-jest": "^28.0.5"
   },
   "scripts": {
@@ -88,7 +79,7 @@
     "prewatch": "pnpm build",
     "watch": "tsc --watch",
     "doc": "documentation readme src/** --section=API --pe ts --re ts --re d.ts",
-    "lint": "eslint ./src --no-error-on-unmatched-pattern --ext .ts,.tsx",
+    "lint": "eslint ./src --no-error-on-unmatched-pattern --ext .ts,.tsx --cache",
     "lint:fix": "pnpm lint --fix",
     "test": "jest"
   }

package/src/account.ts

@@ -6,10 +6,7 @@ import { formatCurrencyUnit } from "@ledgerhq/coin-framework/currencies/index";
 import { PolkadotAccount, PolkadotResources } from "./types";
 import type { Unit } from "@ledgerhq/types-cryptoassets";
 
-function formatOperationSpecifics(
-  op: Operation,
-  unit: Unit | null | undefined
-): string {
+function formatOperationSpecifics(op: Operation, unit: Unit | null | undefined): string {
   const {
     validators,
     bondedAmount,
@@ -27,15 +24,11 @@ function formatOperationSpecifics(
   str +=
     bondedAmount && !bondedAmount.isNaN()
       ? `\n    bondedAmount: ${
-          unit
-            ? formatCurrencyUnit(unit, bondedAmount, formatConfig)
-            : bondedAmount
+          unit ? formatCurrencyUnit(unit, bondedAmount, formatConfig) : bondedAmount
         }`
       : unbondedAmount && !unbondedAmount.isNaN()
       ? `\n    unbondedAmount: ${
-          unit
-            ? formatCurrencyUnit(unit, unbondedAmount, formatConfig)
-            : unbondedAmount
+          unit ? formatCurrencyUnit(unit, unbondedAmount, formatConfig) : unbondedAmount
         }`
       : withdrawUnbondedAmount && !withdrawUnbondedAmount.isNaN()
       ? `\n    withdrawUnbondedAmount: ${
@@ -46,9 +39,7 @@ function formatOperationSpecifics(
       : "";
   str += validatorStash ? `\n    validatorStash: ${validatorStash}` : "";
   str += amount
-    ? `\n    amount: ${
-        unit ? formatCurrencyUnit(unit, amount, formatConfig) : amount
-      }`
+    ? `\n    amount: ${unit ? formatCurrencyUnit(unit, amount, formatConfig) : amount}`
     : "";
   return str;
 }
@@ -63,23 +54,15 @@ function formatAccountSpecifics(account: PolkadotAccount): string {
     showCode: true,
   };
   let str = " ";
-  str +=
-    formatCurrencyUnit(unit, account.spendableBalance, formatConfig) +
-    " spendable. ";
+  str += formatCurrencyUnit(unit, account.spendableBalance, formatConfig) + " spendable. ";
 
   if (polkadotResources.lockedBalance.gt(0)) {
-    str +=
-      formatCurrencyUnit(unit, polkadotResources.lockedBalance, formatConfig) +
-      " locked. ";
+    str += formatCurrencyUnit(unit, polkadotResources.lockedBalance, formatConfig) + " locked. ";
   }
 
   if (polkadotResources.unlockedBalance.gt(0)) {
     str +=
-      formatCurrencyUnit(
-        unit,
-        polkadotResources.unlockedBalance,
-        formatConfig
-      ) + " unlocked. ";
+      formatCurrencyUnit(unit, polkadotResources.unlockedBalance, formatConfig) + " unlocked. ";
   }
 
   if (polkadotResources.stash) {
@@ -92,17 +75,13 @@ function formatAccountSpecifics(account: PolkadotAccount): string {
 
   if (polkadotResources.nominations?.length) {
     str += "\nNominations\n";
-    str += (polkadotResources.nominations as any[])
-      .map((v) => `  to ${v.address}`)
-      .join("\n");
+    str += (polkadotResources.nominations as any[]).map(v => `  to ${v.address}`).join("\n");
   }
 
   return str;
 }
 
-export function fromOperationExtraRaw(
-  extra: Record<string, any> | null | undefined
-) {
+export function fromOperationExtraRaw(extra: Record<string, any> | null | undefined) {
   if (extra && extra.transferAmount) {
     extra = { ...extra, transferAmount: extra.transferAmount.toString() };
   }
@@ -129,9 +108,7 @@ export function fromOperationExtraRaw(
 
   return extra;
 }
-export function toOperationExtraRaw(
-  extra: Record<string, any> | null | undefined
-) {
+export function toOperationExtraRaw(extra: Record<string, any> | null | undefined) {
   if (extra && extra.transferAmount) {
     extra = { ...extra, transferAmount: extra.transferAmount.toString() };
   }

package/src/address.ts

@@ -11,16 +11,12 @@ const POLKADOT_SS58_PREFIX = 0;
 // TODO Cache this to improve perf
 export const isValidAddress = (
   address: string,
-  ss58Format: number = POLKADOT_SS58_PREFIX
+  ss58Format: number = POLKADOT_SS58_PREFIX,
 ): boolean => {
   if (!address) return false;
 
   try {
-    encodeAddress(
-      isHex(address)
-        ? hexToU8a(address)
-        : decodeAddress(address, false, ss58Format)
-    );
+    encodeAddress(isHex(address) ? hexToU8a(address) : decodeAddress(address, false, ss58Format));
     return true;
   } catch (err) {
     return false;

package/src/api/bisontrails.ts

@@ -30,7 +30,7 @@ const getAccountOperationUrl = (
   addr: string,
   offset: number,
   startAt: number,
-  limit: number = LIMIT
+  limit: number = LIMIT,
 ): string =>
   `${getBaseApiUrl()}/accounts/${addr}/operations?${querystring.stringify({
     limit,
@@ -41,9 +41,7 @@ const getAccountOperationUrl = (
 const getWithdrawUnbondedAmount = (extrinsic: any) => {
   return (
     extrinsic?.staking?.eventStaking.reduce((acc: any, curr: any) => {
-      return curr.method === "Withdrawn"
-        ? new BigNumber(acc).plus(curr.value)
-        : new BigNumber(0);
+      return curr.method === "Withdrawn" ? new BigNumber(acc).plus(curr.value) : new BigNumber(0);
     }, new BigNumber(0)) || new BigNumber(0)
   );
 };
@@ -146,9 +144,7 @@ const getExtra = (type: OperationType, extrinsic: any): Record<string, any> => {
  */
 const getValue = (extrinsic: any, type: OperationType): BigNumber => {
   if (!extrinsic.isSuccess) {
-    return type === "IN"
-      ? new BigNumber(0)
-      : new BigNumber(extrinsic.partialFee || 0);
+    return type === "IN" ? new BigNumber(0) : new BigNumber(extrinsic.partialFee || 0);
   }
 
   switch (type) {
@@ -178,15 +174,11 @@ const getValue = (extrinsic: any, type: OperationType): BigNumber => {
 const extrinsicToOperation = (
   addr: string,
   accountId: string,
-  extrinsic: any
+  extrinsic: any,
 ): Partial<Operation> | null => {
   let type = getOperationType(extrinsic.section, extrinsic.method);
 
-  if (
-    type === "OUT" &&
-    extrinsic.affectedAddress1 === addr &&
-    extrinsic.signer !== addr
-  ) {
+  if (type === "OUT" && extrinsic.affectedAddress1 === addr && extrinsic.signer !== addr) {
     type = "IN";
   }
 
@@ -208,8 +200,7 @@ const extrinsicToOperation = (
     recipients: [extrinsic.affectedAddress1, extrinsic.affectedAddress2]
       .filter(Boolean)
       .filter(isValidAddress),
-    transactionSequenceNumber:
-      extrinsic.signer === addr ? extrinsic.nonce : undefined,
+    transactionSequenceNumber: extrinsic.signer === addr ? extrinsic.nonce : undefined,
     hasFailed: !extrinsic.isSuccess,
   };
 };
@@ -223,11 +214,7 @@ const extrinsicToOperation = (
  *
  * @returns {Operation | null}
  */
-const rewardToOperation = (
-  addr: string,
-  accountId: string,
-  reward: any
-): Partial<Operation> => {
+const rewardToOperation = (addr: string, accountId: string, reward: any): Partial<Operation> => {
   const hash = reward.extrinsicHash;
   const type = "REWARD_PAYOUT";
   return {
@@ -254,11 +241,7 @@ const rewardToOperation = (
  *
  * @returns {Operation | null}
  */
-const slashToOperation = (
-  addr: string,
-  accountId: string,
-  slash: any
-): Partial<Operation> => {
+const slashToOperation = (addr: string, accountId: string, slash: any): Partial<Operation> => {
   const hash = `${slash.blockNumber}`;
   const type = "SLASH";
   return {
@@ -293,38 +276,24 @@ const fetchOperationList =
     addr: string,
     startAt: number,
     offset = 0,
-    prevOperations: Operation[] = []
+    prevOperations: Operation[] = [],
   ): Promise<Operation[]> => {
     const { data } = await network({
       method: "GET",
       url: getAccountOperationUrl(addr, offset, startAt),
     });
     const operations = data.extrinsics.map((extrinsic: any) =>
-      extrinsicToOperation(addr, accountId, extrinsic)
-    );
-    const rewards = data.rewards.map((reward: any) =>
-      rewardToOperation(addr, accountId, reward)
-    );
-    const slashes = data.slashes.map((slash: any) =>
-      slashToOperation(addr, accountId, slash)
+      extrinsicToOperation(addr, accountId, extrinsic),
     );
+    const rewards = data.rewards.map((reward: any) => rewardToOperation(addr, accountId, reward));
+    const slashes = data.slashes.map((slash: any) => slashToOperation(addr, accountId, slash));
     const mergedOp = [...prevOperations, ...operations, ...rewards, ...slashes];
 
-    if (
-      operations.length < LIMIT &&
-      rewards.length < LIMIT &&
-      slashes.length < LIMIT
-    ) {
+    if (operations.length < LIMIT && rewards.length < LIMIT && slashes.length < LIMIT) {
       return mergedOp.filter(Boolean).sort((a, b) => b.date - a.date);
     }
 
-    return await fetchOperationList(network)(
-      accountId,
-      addr,
-      startAt,
-      offset + LIMIT,
-      mergedOp
-    );
+    return await fetchOperationList(network)(accountId, addr, startAt, offset + LIMIT, mergedOp);
   };
 
 /**

package/src/api/common.ts

@@ -10,10 +10,7 @@ import type { OperationType } from "@ledgerhq/types-live";
  *
  * @returns {string} - OperationType
  */
-export const getOperationType = (
-  pallet: string,
-  palletMethod: string
-): OperationType => {
+export const getOperationType = (pallet: string, palletMethod: string): OperationType => {
   switch (palletMethod) {
     case "transfer":
     case "transferKeepAlive":
@@ -43,10 +40,7 @@ export const getOperationType = (
       return "FEES";
 
     default:
-      log(
-        "polkadot/api",
-        `Unknown operation type ${pallet}.${palletMethod} - fallback to FEES`
-      );
+      log("polkadot/api", `Unknown operation type ${pallet}.${palletMethod} - fallback to FEES`);
       return "FEES";
   }
 };

package/src/api/index.ts

@@ -91,9 +91,7 @@ export class PolkadotAPI {
 
   private getMinimumBondBalanceFn: CacheRes<Array<void>, BigNumber> | undefined;
   private getRegistryFn: CacheRes<Array<void>, Registry> | undefined;
-  private getTransactionParamsFn:
-    | CacheRes<Array<void>, Record<string, any>>
-    | undefined;
+  private getTransactionParamsFn: CacheRes<Array<void>, Record<string, any>> | undefined;
   private isControllerAddressFn: CacheRes<Array<string>, boolean> | undefined;
   private isElectionClosedFn: CacheRes<Array<void>, boolean> | undefined;
   private isNewAccountFn: CacheRes<Array<string>, boolean> | undefined;
@@ -110,26 +108,20 @@ export class PolkadotAPI {
     return sidecardGetAccount(this.network, this.cache)(address);
   }
 
-  async getOperations(
-    accountId: string,
-    addr: string,
-    startA = 0
-  ): Promise<Operation[]> {
+  async getOperations(accountId: string, addr: string, startA = 0): Promise<Operation[]> {
     return bisonGetOperations(this.network)(accountId, addr, startA);
   }
 
   async getMinimumBondBalance(): Promise<BigNumber> {
-    if (this.getMinimumBondBalanceFn !== undefined)
-      return this.getMinimumBondBalanceFn();
+    if (this.getMinimumBondBalanceFn !== undefined) return this.getMinimumBondBalanceFn();
 
     this.getMinimumBondBalanceFn = this.cache(
-      async (): Promise<BigNumber> =>
-        sidecarGetMinimumBondBalance(this.network)(),
+      async (): Promise<BigNumber> => sidecarGetMinimumBondBalance(this.network)(),
       () => "polkadot",
       {
         max: 1, // Store only one object since we only have polkadot.
         ttl: 60 * 60 * 1000, // 1 hour
-      }
+      },
     );
 
     return this.getMinimumBondBalanceFn();
@@ -145,7 +137,7 @@ export class PolkadotAPI {
       () => "polkadot",
       {
         ttl: 60 * 60 * 1000, // 1 hour - could be Infinity
-      }
+      },
     );
 
     return this.getRegistryFn();
@@ -156,37 +148,30 @@ export class PolkadotAPI {
   }
 
   async getValidators(
-    stashes:
-      | SidecarValidatorsParamStatus
-      | SidecarValidatorsParamAddresses = "elected"
+    stashes: SidecarValidatorsParamStatus | SidecarValidatorsParamAddresses = "elected",
   ): Promise<PolkadotValidator[]> {
     return sidecarGetValidators(this.network)(stashes);
   }
 
   async getTransactionParams(
-    { force }: CacheOpts = { force: false }
+    { force }: CacheOpts = { force: false },
   ): Promise<Record<string, any>> {
     if (this.getTransactionParamsFn !== undefined) {
-      return force
-        ? this.getTransactionParamsFn.force()
-        : this.getTransactionParamsFn();
+      return force ? this.getTransactionParamsFn.force() : this.getTransactionParamsFn();
     }
 
     this.getTransactionParamsFn = this.cache(
-      async (): Promise<Record<string, any>> =>
-        sidecarGetTransactionParams(this.network)(),
+      async (): Promise<Record<string, any>> => sidecarGetTransactionParams(this.network)(),
       () => "polkadot",
       {
         ttl: 5 * 60 * 1000, // 5 minutes
-      }
+      },
     );
 
     return this.getTransactionParamsFn();
   }
 
-  async getPaymentInfo(
-    data: PaymentInfoParams
-  ): Promise<Pick<SidecarPaymentInfo, "partialFee">> {
+  async getPaymentInfo(data: PaymentInfoParams): Promise<Pick<SidecarPaymentInfo, "partialFee">> {
     if (this.paymentInfoFn !== undefined) return this.paymentInfoFn(data);
 
     this.paymentInfoFn = this.cache(
@@ -200,23 +185,21 @@ export class PolkadotAPI {
       ({ a, t, signedTx }) => hashTransactionParams(a, t, signedTx),
       {
         ttl: 5 * 60 * 1000, // 5 minutes
-      }
+      },
     );
 
     return this.paymentInfoFn(data);
   }
 
   async isControllerAddress(address: string): Promise<boolean> {
-    if (this.isControllerAddressFn !== undefined)
-      return this.isControllerAddressFn(address);
+    if (this.isControllerAddressFn !== undefined) return this.isControllerAddressFn(address);
 
     this.isControllerAddressFn = this.cache(
-      async (address): Promise<boolean> =>
-        sidecarIsControllerAddress(this.network)(address),
-      (address) => address,
+      async (address): Promise<boolean> => sidecarIsControllerAddress(this.network)(address),
+      address => address,
       {
         ttl: 5 * 60 * 1000, // 5 minutes
-      }
+      },
     );
 
     return this.isControllerAddressFn(address);
@@ -230,7 +213,7 @@ export class PolkadotAPI {
       () => "",
       {
         ttl: 60 * 1000, // 1 minute
-      }
+      },
     );
 
     return this.isElectionClosedFn();
@@ -240,12 +223,11 @@ export class PolkadotAPI {
     if (this.isNewAccountFn !== undefined) return this.isNewAccountFn(address);
 
     this.isNewAccountFn = this.cache(
-      async (address): Promise<boolean> =>
-        sidecarIsNewAccount(this.network)(address),
-      (address) => address,
+      async (address): Promise<boolean> => sidecarIsNewAccount(this.network)(address),
+      address => address,
       {
         ttl: 60 * 1000, // 1 minute
-      }
+      },
     );
 
     return this.isNewAccountFn(address);
@@ -268,11 +250,7 @@ export class PolkadotAPI {
  *
  * @returns {string} hash
  */
-const hashTransactionParams = (
-  a: PolkadotAccount,
-  t: Transaction,
-  signedTx: string
-) => {
+const hashTransactionParams = (a: PolkadotAccount, t: Transaction, signedTx: string) => {
   // Nonce is added to discard previous estimation when account is synced.
   const prefix = `${a.id}_${a.polkadotResources?.nonce || 0}_${t.mode}`;
   // Fees depends on extrinsic bytesize