Skip to content

Correcciones de la revisión de bloque: cola, cancelar, preferencias y catálogo - #3

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

Dathinel merged 6 commits into
mainfrom
claude/arreglos-revision

Conversation

@Dathinel

@Dathinel Dathinel commented Nov 1, 2026

Copy link
Copy Markdown
Owner

Qué cambia

  • Cola sin pisar salidas: las entradas repetidas (misma ruta completa) se quitan y, si dos trabajos resuelven la misma salida (dos clip.mp4 de carpetas distintas con «Carpeta de salida» fijada), el segundo pasa a clip-2, el tercero a clip-3 (JobBatch.Prepare, JobOptions.OutputName).
  • Cancelar conserva lo ya hecho: JobBatch.RunAsync ya no lanza OperationCanceledException; devuelve los resultados con Cancelled en el archivo en curso y en los que faltaban, y la barra dice «3 de 5 listos, cancelado en d.mp4». El tic verde no sale al cancelar.
  • Preferencias: se guardan al cambiar cualquier control y al elegir archivos, no solo en Closed; una carpeta que no exista en ese arranque (disco externo o de red) ya no se olvida (el diálogo ya cae a la de siempre si no está).
  • build/catalogo-entrada.ps1: Resolve-Path del zip antes de OpenRead, que resolvía rutas relativas contra el directorio del proceso.
  • Un solo SyncProgress<T> en Core en vez de dos Relay privados iguales; FileDialogs.OpenSingle (sin llamadores) borrado; listos → ready.
  • CLAUDE.md al estado del 31-oct: PR Arregla la CI de main y añade calidad de vida a la ventana (1.0.1) #1 unido, versión sigue en 1.0.0 hasta publicar la 1.0.1.

Qué queda para Daniel

  • Decidir si FileDialogs se saca a la biblioteca compartida con Rastrear (no se tocó).
  • Subir Version a 1.0.1 y cerrar «[Sin publicar]» del CHANGELOG cuando se publique.
  • Probar a mano: cancelar una cola de varios archivos y ver el resumen; cerrar la app con el Administrador de tareas tras cambiar un ajuste y comprobar que se recuerda.
  • Bajos no tocados: duplicado Preferences.Load/Save ↔ WindowPlacement, ResultFolder en Core, carpetas soltadas en OnDrop sin aviso.

Cómo se probó

  • build\dotnet.cmd build src\Subtitular.slnx -warnaserror: 0 advertencias, 0 errores.
  • build\dotnet.cmd test tests\Subtitular.Tests: 42 pruebas en verde (sin las largas de hardware). Nuevas: Cancel_keeps_the_files_already_saved, Prepare_drops_repeated_inputs_and_renames_outputs_that_collide; ajustadas: cancelación de la cola y preferencias con carpeta inexistente.

🤖 Generated with Claude Code

Dathinel and others added 6 commits October 31, 2026 20:18
… devuelve lo ya hecho

- JobBatch.Prepare quita las entradas repetidas (misma ruta completa) y, si dos trabajos resuelven la misma salida, pone «-2», «-3» (JobOptions.OutputName).
- Cancelar ya no tira la lista: el archivo en curso y los que faltaban vuelven con Cancelled y la barra dice «N de M listos, cancelado en X».
- Un solo relevo de progreso (SyncProgress<T> en Core) en vez de dos Relay privados iguales.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Un apagón o un cierre forzado ya no pierde lo elegido, y una carpeta de un disco externo o de red que no esté en ese arranque se conserva (el diálogo ya cae a la de siempre si no existe).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
OpenRead resolvía una ruta relativa contra el directorio del proceso y no contra $PWD, así que Test-Path pasaba y la huella fallaba.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…asta publicar

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@Dathinel
Dathinel merged commit a32edd5 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