fix: update cli

hirosystems · Mar 12, 2024 · a1ce8e0 · a1ce8e0
1 parent 7e8097d
commit a1ce8e0
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 20 deletions.
diff --git a/packages/cli/src/cli.ts b/packages/cli/src/cli.ts
@@ -19,7 +19,6 @@ import {
   ClarityValue,
   ContractCallPayload,
   SignedContractDeployOptions,
-  createStacksPrivateKey,
   cvToString,
   estimateTransfer,
   getAbi,
@@ -28,8 +27,6 @@ import {
   makeContractDeploy,
   makeSTXTokenTransfer,
   PostConditionMode,
-  pubKeyfromPrivKey,
-  publicKeyToString,
   ReadOnlyFunctionOptions,
   SignedContractCallOptions,
   SignedTokenTransferOptions,
@@ -44,6 +41,7 @@ import {
   estimateTransaction,
   serializePayload,
   estimateTransactionByteLength,
+  privateKeyToPublic,
 } from '@stacks/transactions';
 import express from 'express';
 import { prompt } from 'inquirer';
@@ -447,7 +445,7 @@ async function migrateSubdomains(network: CLINetworkAdapter, args: string[]): Pr
        * ********************************************************************************
        */
       const hash = crypto.createHash('sha256').update(textToSign).digest('hex');
-      const sig = signWithKey(createStacksPrivateKey(account.dataPrivateKey), hash);
+      const sig = signWithKey(account.dataPrivateKey, hash);
 
       // https://docs.stacks.co/build-apps/references/bns#subdomain-lifecycle
       subDomainOp.signature = sig.data;
@@ -1870,7 +1868,7 @@ async function register(network: CLINetworkAdapter, args: string[]): Promise<str
   const privateKey = args[1];
   const salt = args[2];
   const zonefile = args[3];
-  const publicKey = publicKeyToString(pubKeyfromPrivKey(privateKey));
+  const publicKey = privateKeyToPublic(privateKey);
 
   const api = new StacksNodeApi({ network: network.isMainnet() ? STACKS_MAINNET : STACKS_TESTNET });
 
@@ -1883,7 +1881,7 @@ async function register(network: CLINetworkAdapter, args: string[]): Promise<str
   });
 
   const signer = new TransactionSigner(unsignedTransaction);
-  signer.signOrigin(createStacksPrivateKey(privateKey));
+  signer.signOrigin(privateKey);
 
   return broadcastTransaction({ transaction: signer.transaction, api })
     .then((response: TxBroadcastResult) => {
@@ -1905,7 +1903,7 @@ async function preorder(network: CLINetworkAdapter, args: string[]): Promise<str
   const privateKey = args[1];
   const salt = args[2];
   const stxToBurn = args[3];
-  const publicKey = publicKeyToString(pubKeyfromPrivKey(privateKey));
+  const publicKey = privateKeyToPublic(privateKey);
 
   const api = new StacksNodeApi({ network: network.isMainnet() ? STACKS_MAINNET : STACKS_TESTNET });
 
@@ -1918,7 +1916,7 @@ async function preorder(network: CLINetworkAdapter, args: string[]): Promise<str
   });
 
   const signer = new TransactionSigner(unsignedTransaction);
-  signer.signOrigin(createStacksPrivateKey(privateKey));
+  signer.signOrigin(privateKey);
 
   return broadcastTransaction({ transaction: signer.transaction, api })
     .then((response: TxBroadcastResult) => {

diff --git a/packages/cli/src/common.ts b/packages/cli/src/common.ts
@@ -1,9 +1,9 @@
 import { publicKeyToBtcAddress } from '@stacks/encryption';
-import { pubKeyfromPrivKey } from '@stacks/transactions';
 import * as bitcoinjs from 'bitcoinjs-lib';
 import { TransactionSigner } from 'blockstack';
 import { DEFAULT_MAX_ID_SEARCH_INDEX } from './argparse';
 import { CLINetworkAdapter } from './network';
+import { privateKeyToPublic } from '@stacks/transactions/src';
 
 let maxIDSearchIndex = DEFAULT_MAX_ID_SEARCH_INDEX;
 
@@ -49,8 +49,8 @@ export function getPrivateKeyAddress(
     return privateKey.address;
   }
 
-  const pubKey = pubKeyfromPrivKey(privateKey);
-  const btcAddress = publicKeyToBtcAddress(pubKey.data);
+  const pubKey = privateKeyToPublic(privateKey);
+  const btcAddress = publicKeyToBtcAddress(pubKey);
   return network.coerceAddress(btcAddress);
 }
 

diff --git a/packages/cli/tests/cli.test.ts b/packages/cli/tests/cli.test.ts
@@ -5,7 +5,6 @@ import { CLINetworkAdapter, CLI_NETWORK_OPTS, getNetwork } from '../src/network'
 import {
   Cl,
   ClarityAbi,
-  createStacksPrivateKey,
   publicKeyFromSignatureVrs,
   randomBytes,
   signWithKey,
@@ -417,7 +416,7 @@ describe('Subdomain Migration', () => {
      * ********************************************************************************
      */
     const hash = crypto.createHash('sha256').update(textToSign).digest('hex');
-    const sig = signWithKey(createStacksPrivateKey(privateKey), hash);
+    const sig = signWithKey(privateKey, hash);
 
     subDomainOp.signature = sig.data; // Assign signature to subDomainOp
 

diff --git a/packages/transactions/src/transaction.ts b/packages/transactions/src/transaction.ts
@@ -1,10 +1,4 @@
-import {
-  concatArray,
-  hexToBytes,
-  IntegerType,
-  intToBigInt,
-  writeUInt32BE,
-} from '@stacks/common';
+import { concatArray, hexToBytes, IntegerType, intToBigInt, writeUInt32BE } from '@stacks/common';
 import {
   AddressHashMode,
   AnchorMode,