[XLM-17] SignDialog genérico y cierre de la Fase X5

[grantfox-oss]

Contexto obligatorio: PLAN_STELLAR.md, PLAN_XAMAN.md (Fase X5 y decisión D6) y ISSUES_STELLAR.md.
Ola 4 — bloqueada por #17.
🔧 Salda deuda técnica previa: cierra la Fase X5 de PLAN_XAMAN.md, abierta desde junio.

Problema

Dos cosas que necesitan exactamente la misma solución:

1. xaman_sign_dialog.py es casi genérico — el QR, el polling, el timeout y resolve_status() no saben nada de XRPL — pero está atado a XamanClient y a txjson.
2. La Fase X5 de PLAN_XAMAN.md sigue abierta: escrow_view.py:98 deshabilita los botones cuando no hay seed, así que hoy no se puede gestionar un escrow con Xaman. El aviso "estará disponible en la próxima versión" lleva meses en pantalla.

Por qué importa

Las dos necesitan la misma generalización. Hacerlas por separado sería hacer el trabajo dos veces — y la segunda tendría que deshacer parte de la primera.

Alcance

Generalizar el diálogo

• Mover a shared_ui/sign_dialog.py, parametrizando el cliente de firma y el texto.
• resolve_status() se conserva como función pura y sus tests no cambian.
• payment_app/ui_payment/xaman_sign_dialog.py queda como shim que reexporta, para no romper llamadas existentes.

Cerrar la Fase X5

• Habilitar la gestión de escrow XRPL con Xaman en escrow_view.py.
• EscrowFinish y EscrowCancel se construyen como txjson y se firman con el diálogo.
• El offer_sequence se obtiene consultando la transacción ya validada, según la decisión D6 de PLAN_XAMAN.md.
• Eliminar el aviso de escrow_view.py:99.

Fuera de alcance

• LOBSTR — es [XLM-19]. El diálogo debe quedar listo para recibirlo, no cableado a él.

Criterios de aceptación

• Un pago con Xaman se comporta exactamente igual que antes.
• Un escrow XRPL completo, firmado con Xaman de punta a punta, en testnet — crear, aprobar y liberar. Esto cierra X5.
• El aviso de "escrow con Xaman en próxima versión" desaparece.
• Los tests de resolve_status pasan sin modificarse.

Pruebas

Los tests existentes del diálogo siguen verdes; prueba manual del ciclo de escrow con Xaman documentada en un comentario, con hashes.

Dependencias

• Bloqueada por: [XLM-13] escrow_view multicadena #17 (escrow_view multicadena — mismo archivo, va después).
• Bloquea a: [XLM-19].

[mayborn005]

Hello @maintainer, I’ve gone through the requirements and I’m confident I can implement a proper solution that matches the acceptance criteria. Kindly assign this issue to me. And I will deliver at the shortest possible time.

[ericmt-98]

👋 Gracias por aplicar. Esta issue está bloqueada: depende de #17, que a su vez depende de #15 → #6 → #4. Son cuatro niveles de profundidad en el grafo.

Toca escrow_view.py, del que #17 es dueña ahora mismo, así que empezar antes provocaría conflictos.

---

👋 Thanks for applying. This one is blocked: it depends on #17, which depends on #15 → #6 → #4 — four levels deep. It touches escrow_view.py, currently owned by #17.

Libres ahora / Open right now: #1, #2, #3, #4 — sin bloqueadores, se pueden tomar hoy. Ver el grafo completo en ISSUES_STELLAR.md.

Gracias por el interés — la issue sigue abierta y se puede tomar en cuanto se desbloquee.

[ericmt-98]

👋 Gracias a los dos por aplicar. Para asignarla necesito una señal de que leyeron el issue.

Contesten esto en un comentario:

Además de generalizar el diálogo, ¿qué deuda técnica previa cierra esta issue, y qué función pura tiene que seguir pasando sus tests sin que se modifiquen?

Las dos cosas están en el cuerpo. Asigno al primero que conteste bien.

⚠️ Aviso: está bloqueada por #17, que a su vez espera a #15 → #6 → #4. Son cuatro niveles: no se puede empezar en días.

---

👋 Answer this: besides generalising the dialog, what pre-existing technical debt does this issue close, and which pure function must keep passing its tests unmodified? Note this is four levels deep in the dependency graph.

---

Si prefieres algo que puedas empezar de inmediato, #1, #2, #3 y #4 están libres y sin bloqueadores. / If you would rather start immediately, #1–#4 are open with no blockers.