Skip to content

Arreglos de la revisión de bloque: salida a medias, cierre, cola, nombres y Lanzador - #3

Merged
Dathinel merged 5 commits into
mainfrom
claude/arreglos-revision
Nov 1, 2026
Merged

Dathinel merged 5 commits into
mainfrom
claude/arreglos-revision

Conversation

@Dathinel

@Dathinel Dathinel commented Nov 1, 2026

Copy link
Copy Markdown
Owner

Qué cambia

  • Salida a medias: al cancelar o fallar se borra el archivo parcial junto al video (RunToolAsync, ignorando IOException).
  • Cerrar la ventana con una ejecución en marcha cancela y mata ffmpeg y el módulo antes de guardar preferencias.
  • Carrera al cancelar: TaskExecutionController.CancelTask guarda la Task de CancelAsync y OnEjecutarClick la espera antes de marcar cancelado, así «Seguía vivo: …» es fiable.
  • Reanudar: el video cancelado vuelve a pendiente y se repite (BatchQueue.RequeueCancelled).
  • Cola con todos fallidos ya no dice «Hecho»: estado con error y el primer mensaje.
  • Nombre de salida único: sufijo «reencuadrado» (el que ya publica la Cli 1.0.0) y la Cli usa OutputNaming.OutputPathFor, así tampoco pisa un resultado anterior (añade « (2)»).
  • Una sola lista de extensiones: el filtro de «Elegir video…» se construye desde BatchQueue.VideoExtensions.
  • Bajos: MatteFrameAsync libera la SharedMemoryRegion; UserPreferences.Save escribe temporal + File.Move.
  • Lanzador: comprobado en Lanzador/build/instalador/Lanzador.iss (DefaultDirName={userpf}\Lanzador = %LOCALAPPDATA%\Programs\Lanzador); la ruta ya era candidata, pasa a ser la primera y la documentada.

Qué queda para Daniel

  • La Cli ya no sobreescribe en silencio: si existe clip-fondo.webm crea clip-fondo (2).webm (cambio de comportamiento desde 1.0.0, consistente con la ventana). Si prefieres que la Cli pise, se quita en una línea.
  • Probar a mano: cancelar «Quitar fondo» a medias y comprobar que no queda clip-fondo.webm; cerrar la ventana con «Reencuadrar» en marcha y ver que no queda Rodaje.Module.Vision.exe en el Administrador de tareas.
  • No se tocó nada del aviso de versión nueva (otra sesión).

Cómo se probó

  • build\dotnet.cmd build src\Rastrear.slnx -warnaserror: 0 advertencias, 0 errores.
  • build\dotnet.cmd test tests\Rastrear.Tests: 35 de 35 (sin las largas de hardware). Pruebas nuevas: RequeueCancelled en ColaTests, la ruta Programs\Lanzador en AtajosYLanzadorTests, y que no queda .tmp tras Save en PreferenciasTests.

🤖 Generated with Claude Code

Dathinel and others added 5 commits October 31, 2026 20:17
…kill y repetir el cancelado

- Al cancelar o fallar se borra el archivo a medias junto al video (ignorando IOException).
- Closed cancela y mata ffmpeg y el módulo antes de guardar preferencias.
- OnEjecutarClick espera la Task de CancelAsync (TaskExecutionController.CancelTask) antes de marcar cancelado, así «Seguía vivo» es fiable.
- Al reanudar, el cancelado vuelve a pendiente (BatchQueue.RequeueCancelled) y se repite.
- Con todos los videos fallidos no sale el tic de «Hecho»: estado Faulted con el primer error.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
- Sufijo único «reencuadrado» (el que ya publica la Cli 1.0.0); la Cli usa OutputNaming.OutputPathFor y ya no pisa un resultado anterior.
- El filtro del diálogo «Elegir video…» se construye desde BatchQueue.VideoExtensions.
- Prueba de RequeueCancelled en ColaTests.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…s de forma atómica

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…s\Lanzador)

Comprobado en Lanzador/build/instalador/Lanzador.iss (DefaultDirName={userpf}\Lanzador); la ruta ya era candidata, pasa a ser la primera y la documentada.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@Dathinel
Dathinel merged commit 6d5f97f into main Nov 1, 2026
1 check passed
@Dathinel
Dathinel deleted the claude/arreglos-revision branch November 1, 2026 01:31
Dathinel added a commit that referenced this pull request Nov 1, 2026
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant