[BUG]: Proxy startup may report success when retries are exhausted and process does not stabilize
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
- Área
- ci-cd
Línea de trabajo
Comienza con start-proxy-action.ts y sigue las rutas de reintento, salida anticipada, retraso de estabilización y salida. Confirma cómo se representan el agotamiento de los reintentos y el último código de salida, y define pruebas para el fallo terminal, la actividad del proceso y los reintentos con un nuevo puerto efímero; se considerará terminado cuando un inicio inestable falle explícitamente sin cambiar las salidas existentes, la telemetría ni la semántica del wrapper.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- TypeScript
- Estrellas
- 1.6k
- Forks
- 493
- Merge medio
- 1 d 13 h
- PR fusionados (30 d)
- 44
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de github/codeql-action
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
github/codeql-action#4052 · 4 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
github/codeql-action#4078 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
github/codeql-action#4008 · 9 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
github/codeql-action#3978 · 4 comentarios · 1 reacción ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
github/codeql-action#3915 · 6 comentarios · 3 reacciones ·
Todos los issues de github/codeql-action
Issues similares
-
[Bug]: Discord Activity titles with emoji are rejected as over 80 characters when they are not Abiertoclawsweeper:linked-pr-open clawsweeper:no-new-fix-pr clawsweeper:source-repro impact:message-loss issue-rating: 🦞 diamond lobster maturity:stable P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Eynzof/Hermes-CN-Desktop#616 ·
-
ZCode 3.14.3 に対応する Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
supermomonga/zcode-acp#24 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
growthbook/growthbook#7100 ·
-
triage
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100