From 2335511b07761e08d8d5917949bfc23bd2716966 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 9 Dec 2025 22:01:31 +0000 Subject: [PATCH 1/3] Initial plan From cf741ddadb70c5c3afefe60b7a6483016b8cba97 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 9 Dec 2025 22:08:33 +0000 Subject: [PATCH 2/3] fix: Improve JSDoc descriptions and fix typo in operationId - Fixed typo in accelerate route operationId (advancedwalletalletacceleratetx -> advancedwalletacceleratetx) - Added missing description to consolidateUnspents route - Improved description clarity and consistency across API spec: - Changed "Recover a multisig transaction" to "Sign a multisig recovery transaction" - Changed "Generate an independent key" to "Generate an independent keychain" - Changed "Sign a MPC transaction" to "Sign an MPC transaction" - Changed "Initialize MPC for EdDSA key generation" to "Initialize MPC v2 for ECDSA key generation" - Changed "Perform a round in the MPC protocol" to "Perform a round in the MPC v2 DKG protocol" - Changed "Finalize the MPC protocol" to "Finalize the MPC v2 DKG protocol" - Changed "Recover a MPC transaction" to "Sign an MPC v2 recovery transaction" Co-authored-by: vmccarty <107434158+vmccarty@users.noreply.github.com> --- .../routers/advancedWalletManagerApiSpec.ts | 14 +++++++------- src/masterBitgoExpress/routers/accelerateRoute.ts | 2 +- .../routers/consolidateUnspentsRoute.ts | 1 + 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/src/advancedWalletManager/routers/advancedWalletManagerApiSpec.ts b/src/advancedWalletManager/routers/advancedWalletManagerApiSpec.ts index adeffaf..c3348a7 100644 --- a/src/advancedWalletManager/routers/advancedWalletManagerApiSpec.ts +++ b/src/advancedWalletManager/routers/advancedWalletManagerApiSpec.ts @@ -327,7 +327,7 @@ export const AdvancedWalletManagerApiSpec = apiSpec({ body: RecoveryMultisigRequest, }), response: RecoveryMultisigResponse, - description: 'Recover a multisig transaction', + description: 'Sign a multisig recovery transaction', }), }, 'v1.mpc.recovery': { @@ -355,7 +355,7 @@ export const AdvancedWalletManagerApiSpec = apiSpec({ body: IndependentKeyRequest, }), response: IndependentKeyResponse, - description: 'Generate an independent key', + description: 'Generate an independent keychain', }), }, 'v1.mpc.key.initialize': { @@ -392,7 +392,7 @@ export const AdvancedWalletManagerApiSpec = apiSpec({ body: SignMpcRequest, }), response: SignMpcResponse, - description: 'Sign a MPC transaction', + description: 'Sign an MPC transaction', }), }, @@ -423,7 +423,7 @@ export const AdvancedWalletManagerApiSpec = apiSpec({ 200: MpcV2InitializeResponseType, ...ErrorResponses, }, - description: 'Initialize MPC for EdDSA key generation', + description: 'Initialize MPC v2 for ECDSA key generation', }), }, 'v1.mpcv2.round': { @@ -438,7 +438,7 @@ export const AdvancedWalletManagerApiSpec = apiSpec({ 200: MpcV2RoundResponseType, ...ErrorResponses, }, - description: 'Perform a round in the MPC protocol', + description: 'Perform a round in the MPC v2 DKG protocol', }), }, 'v1.mpcv2.finalize': { @@ -453,7 +453,7 @@ export const AdvancedWalletManagerApiSpec = apiSpec({ 200: MpcV2FinalizeResponseType, ...ErrorResponses, }, - description: 'Finalize the MPC protocol', + description: 'Finalize the MPC v2 DKG protocol', }), }, 'v1.mpcv2.recovery': { @@ -468,7 +468,7 @@ export const AdvancedWalletManagerApiSpec = apiSpec({ 200: MpcV2RecoveryResponseType, ...ErrorResponses, }, - description: 'Recover a MPC transaction', + description: 'Sign an MPC v2 recovery transaction', }), }, }); diff --git a/src/masterBitgoExpress/routers/accelerateRoute.ts b/src/masterBitgoExpress/routers/accelerateRoute.ts index df4722c..6761d7d 100644 --- a/src/masterBitgoExpress/routers/accelerateRoute.ts +++ b/src/masterBitgoExpress/routers/accelerateRoute.ts @@ -83,7 +83,7 @@ const AccelerateResponse: HttpResponse = { * Use this endpoint only with advanced wallets. For other wallet types, use [Accelerate Transaction](https://developers.bitgo.com/reference/expresswalletacceleratetx). * * @tag Advanced Wallets - * @operationId advancedwalletalletacceleratetx + * @operationId advancedwalletacceleratetx */ export const AccelerateRoute = httpRoute({ method: 'POST', diff --git a/src/masterBitgoExpress/routers/consolidateUnspentsRoute.ts b/src/masterBitgoExpress/routers/consolidateUnspentsRoute.ts index b749bad..a81eb78 100644 --- a/src/masterBitgoExpress/routers/consolidateUnspentsRoute.ts +++ b/src/masterBitgoExpress/routers/consolidateUnspentsRoute.ts @@ -94,4 +94,5 @@ export const ConsolidateUnspentsRoute = httpRoute({ body: ConsolidateUnspentsRequest, }), response: ConsolidateUnspentsResponse, + description: 'Consolidate unspents', }); From 20b528b26a48a36854477952ebd66a90f7dd8e1a Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 9 Dec 2025 22:09:22 +0000 Subject: [PATCH 3/3] docs: Improve consolidateUnspents route description Made description more detailed to match the level of JSDoc comments Co-authored-by: vmccarty <107434158+vmccarty@users.noreply.github.com> --- src/masterBitgoExpress/routers/consolidateUnspentsRoute.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/masterBitgoExpress/routers/consolidateUnspentsRoute.ts b/src/masterBitgoExpress/routers/consolidateUnspentsRoute.ts index a81eb78..fe69490 100644 --- a/src/masterBitgoExpress/routers/consolidateUnspentsRoute.ts +++ b/src/masterBitgoExpress/routers/consolidateUnspentsRoute.ts @@ -94,5 +94,5 @@ export const ConsolidateUnspentsRoute = httpRoute({ body: ConsolidateUnspentsRequest, }), response: ConsolidateUnspentsResponse, - description: 'Consolidate unspents', + description: 'Build, sign, and send a transaction to consolidate unspents for UTXO-based assets', });