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

クローズ 初心者向け
  • 何か問題がありますか？
#66,257 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
82/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
go
領域
tooling

調査の方向性

列挙された4つの呼び出し箇所、pkg/cli/run_workflow_tracking.go、pkg/cli/trial_repository.go、pkg/cli/update_extension_check.go、pkg/workflow/docker_validation.goを読み、各関数がどのようにコンテキストを受け取り、エラーを処理するかを確認してください。中断できない待機をコンテキスト対応の待機に置き換え、その後 make test-unit を実行してください。これらの箇所に注釈のない time.Sleep が残っておらず、ユニットテストが成功すれば完了です。

索引モデルが issue の本文から書いたものです。

説明

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
主要言語
Go
スター
5.6k
フォーク
615
平均マージ
5時間 19分
マージ済み PR（30日）
798

環境構築

Codespaces で開く

このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

github/gh-aw のほかの issue

github/gh-aw の issue をすべて見る

似ている issue

Go の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。