[BUG]: Proxy startup may report success when retries are exhausted and process does not stabilize
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 45/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- typescript
- 領域
- ci-cd
調査の方向性
start-proxy-action.ts から始めて、retry、早期終了、安定化遅延、出力の各パスを追跡します。retry の枯渇と最後の exit code がどのように表現されるかを確認し、終端失敗、プロセスの稼働状態、新しいエフェメラルポートでの retry を対象とするテストを定義します。既存の出力、テレメトリ、wrapper のセマンティクスを変更せずに、不安定な起動が明示的に失敗すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Problem
The proxy startup logic in start-proxy-action.ts can report successful initialization even when the proxy process fails to stabilize after retry attempts.
The previous implementation:
- Retries on spawn or early exit
- Only throws on immediate spawn errors
- Does not explicitly fail when all retry attempts are exhausted
- May proceed with outputs set despite no stable process remaining alive
This can lead to false-positive startup.
Impact
When startup silently fails:
- Downstream steps may fail with opaque dependency or networking errors
- Failures manifest later in the workflow
- Root cause becomes difficult to diagnose
Instead of failing fast during initialization, workflows continue in a degraded state.
Expected Behavior
If the proxy fails to stabilize after retry attempts, startup should fail explicitly with a clear diagnostic error.
Proposed Solution
- Detect retry exhaustion explicitly
- Throw a clear error including the last exit code
- Verify process remains alive after stabilization delay
- Retry using a new ephemeral port before terminal failure
- Extract startup logic into a dedicated launcher module for testability
This preserves existing outputs, telemetry behavior, and wrapper semantics while preventing silent failure.
- 主要言語
- TypeScript
- スター
- 1.6k
- フォーク
- 493
- 平均マージ
- 1日 13時間
- マージ済み PR（30日）
- 44
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github/codeql-action のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
github/codeql-action#4052 · コメント 4 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
github/codeql-action#4078 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
github/codeql-action#4008 · コメント 9 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
github/codeql-action#3978 · コメント 4 件 · リアクション 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
github/codeql-action#3915 · コメント 6 件 · リアクション 3 件 ·
github/codeql-action の issue をすべて見る
似ている issue
-
clawsweeper:linked-pr-open clawsweeper:no-new-fix-pr clawsweeper:source-repro impact:message-loss issue-rating: 🦞 diamond lobster maturity:stable P2
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
Eynzof/Hermes-CN-Desktop#616 ·
-
ZCode 3.14.3 に対応する オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
supermomonga/zcode-acp#24 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
growthbook/growthbook#7100 ·
-
triage
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100