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

Chiusa Adatta ai principianti
  • Qualcosa non va?
#66,257 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
go
Ambito
tooling

Direzione di ricerca

Leggi i quattro punti di chiamata elencati in pkg/cli/run_workflow_tracking.go, pkg/cli/trial_repository.go, pkg/cli/update_extension_check.go e pkg/workflow/docker_validation.go, verificando come ogni funzione riceve il proprio contesto e gestisce gli errori. Sostituisci le attese non interrompibili con attese consapevoli del contesto, quindi esegui make test-unit; il lavoro è terminato quando in quei punti non rimane alcun time.Sleep non annotato e i test unitari passano.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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
Lingua principale
Go
Stelle
5.6k
Fork
615
Merge medio
5h 19m
PR unite (30g)
798

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di github/gh-aw

Tutte le issue di github/gh-aw

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.