paritytech/hardhat-polkadot

refactor constructCommandArgs

Open

#279 aperta il 14 ago 2025

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)TypeScript (10 fork)auto 404
P2 - Mediumgood first issue

Metriche repository

Star
 (16 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

📝 Description

The function constructCommandArgs is used every time a user interacts with hardhat. It accepts arguments from cli and hardhat.config. It should only accept one set of arguments and cli + config parameters should be resolved before i.e. cli.port ?? config.port (cli commands take precedence) in function interface. It should look like zksync constructCommandArgs. The logic inside the function is simpler with just if (parameter) then push command. Should be applied to nodeCommands.push and adapterCommands.push

🤔 Motivation

The constructCommandArgs is quite error prone, difficult to debug and it is the most crucial functional in our plugin.

Guida contributor

Direzione di ricerca
Ispeziona l'attuale funzione constructCommandArgs in packages/hardhat polkadot node/src/utils.ts e confrontala con il riferimento zksync in packages/hardhat zksync node/src/utils.ts. Refactoring per accettare argomenti unificati da cli e config, risolvendo la precedenza prima della chiamata alla funzione. Semplifica la logica utilizzando push condizionali per nodeCommands e adapterCommands.
Tech stack
typescript
Dominio
devtools
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60