[deep-report] Replace 4 remaining production time.Sleep calls with ctx-aware waits

Cerrado Apto para principiantes
  • ¿Algo no está bien?
#66,257 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
82/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
go
Área
tooling

Línea de trabajo

Lee los cuatro puntos de llamada indicados en pkg/cli/run_workflow_tracking.go, pkg/cli/trial_repository.go, pkg/cli/update_extension_check.go y pkg/workflow/docker_validation.go, y comprueba cómo recibe el contexto cada función y cómo gestiona los errores. Sustituye las esperas que no se pueden interrumpir por esperas que tengan en cuenta el contexto y, después, ejecuta make test-unit; el trabajo estará terminado cuando no quede ningún time.Sleep sin anotar en esos puntos y pasen las pruebas unitarias.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

automation code-quality cookie improvement quick-win task-mining
Description

Four production time.Sleep calls remain uninterruptible by cancellation or caller timeout, confirmed by direct read:

  • pkg/cli/run_workflow_tracking.go:73
  • pkg/cli/trial_repository.go:199
  • pkg/cli/update_extension_check.go:412
  • pkg/workflow/docker_validation.go:220 (note: source report cited this as docker_validation.go without the pkg/workflow/ prefix — confirmed actual location via find)
Expected Impact

Closes the last few time.Sleep violations the repo's own timesleepnocontext linter targets; these waits become cancellable rather than blocking for their full duration regardless of context cancellation.

Suggested Fix

Replace each with a ctx-aware wait:

select {
case <-ctx.Done():
    return ctx.Err()
case <-time.After(d):
}
Acceptance Criteria
  • No unannotated time.Sleep remains in the four listed locations
  • make test-unit passes
Suggested Agent

Go-focused agent; small, mechanical change per site.

Estimated Effort

Quick (< 1 hour).

Data Source

Repository Quality: Context Propagation & Cancellation Hygiene — 2026-10-06 (#66149), Task 4 (path corrected for docker_validation.go after verification).

Generated by 🔬 Deep Report · claude · agent · 421.8 AIC · ⌖ 9.14 AIC · ⊞ 7.1K · ◷

  • expires on Oct 8, 2026, 10:47 AM UTC-08:00
Lenguaje dominante
Go
Estrellas
5.6k
Forks
615
Merge medio
5 h 19 min
PR fusionados (30 d)
798

Preparar el entorno

Abrir en Codespaces

Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de github/gh-aw

Todos los issues de github/gh-aw

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.