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

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#66,257 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
82/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
go
Lĩnh vực
tooling

Hướng nghiên cứu

Đọc bốn vị trí gọi được liệt kê trong pkg/cli/run_workflow_tracking.go, pkg/cli/trial_repository.go, pkg/cli/update_extension_check.go và pkg/workflow/docker_validation.go, kiểm tra cách mỗi hàm nhận context và xử lý lỗi. Thay các lần chờ không thể ngắt bằng các lần chờ có xét đến context, sau đó chạy make test-unit; công việc hoàn tất khi không còn time.Sleep nào chưa được chú thích tại các vị trí đó và các bài kiểm thử đơn vị đều chạy thành công.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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
Ngôn ngữ chính
Go
Star
5.6k
Fork
615
Merge trung bình
5 giờ 44 phút
Pull request đã merge (30 ngày)
808

Chuẩn bị môi trường

Mở trong Codespaces

Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của github/gh-aw

Tất cả issue của github/gh-aw

Issue tương tự

Thêm issue về Go

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.