₿ Bitcoin Core PR Review Club en Español

Pregunta 6

wallet: Add importdescriptors interface by polespinasa · Pull Requ...

Pol Espinasa 31/03/2026 11:14
Pregunta 6 -> En el commit b88462d758e2d24f278607dc5fbd9dcee6c1a076 algunos reviewers mencionan que gran parte de la funcionalidad que hace importDescriptors() puede moverse a ImportDescriptors(). ¿Por qué crees que mencionan esto? ¿Estás de acuerdo?
08/04/2026 16:43
Quien se atreve con esta?
> 08/04/2026 16:44
Pienso que es para mantener la implementación de la interfaz minima. Yo estoy parcialmente de acuerdo, pero lo que me preocupa un poco es que podría agregar lógica indirectamente a la capa de RPC que antes no existía.
arejula27 08/04/2026 16:46
segun entiendo lo que biscan es migrar de la interfaz a la wallet
08/04/2026 16:46
por lo que no caeria en RPC
> 08/04/2026 16:46
por ej. el wallet rescan que corre en la interfaz
👀 1
08/04/2026 16:47
Hay codigo duplicado en la interfaz y RPC, el wallet rescan y el mismo bucle que llama a ProcessDescriptorImport. Esa logica podria mirar de extraerse a la funciona ImportDescriptors() para asegurar que el comportamiento es el mismo y minimizar le codigo duplicado
🔥 1
08/04/2026 16:47
Sin embargo hay que mirar bien como extraer esa parte del codigo ya que toca mensajes de error y otras cosas especificas de RPC
arejula27 08/04/2026 16:48
por lo que he visto, muchas veces en core no se transmite un error sino que se hace un non fatal print, como lo gestionarias en ese caso si el error es propio de rpc? lo veo dificil
Pol Espinasa 08/04/2026 16:49
El error no es propio de RPC, pero el codigo que hay esta pensado para RPC
👍 1
08/04/2026 16:49
seria tema de traducir el codigo y/o ignorarlo en otras interfaces
08/04/2026 16:49
tengo que terminar de mirar como hacerlo ya que eso fue sugerencia de los reviewers y aun no me he puesto a ello!
arejula27 08/04/2026 16:50
np, lo mirare por que justo eso me interesa bastante
Pol Espinasa 08/04/2026 16:50
Vamos a la siguiente!