github / github/app

Status messages interrupt active child sessions and trigger unbounded review restarts

Abierto
#2,315 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Bugs
Lenguaje dominante
Sin datos de lenguaje
Estrellas
2.1k
Forks
153
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

## Summary

Cross-session status messages sent to an active child session interrupted its certification work. The child repeatedly restarted review, expanded its changes, and consumed substantial resources without checkpointing, committing, or opening a pull request.

My status questions were reasonable and exposed the defect. The repeated work was caused by orchestration behavior, not user error.

## Reproduction

1. Delegate a long-running review to a child session.

2. Request its status through the parent session while it is processing.

3. The parent sends the status request into the child as a new user turn.

4. Active work is interrupted or superseded.

5. The child restarts review instead of resuming from a checkpoint.

6. Repeated status checks cause repeated review cycles and scope growth.

## Expected behavior

- Status checks should be read-only and not interrupt active child work.

- Any unavoidable interruption should require confirmation.

- Interrupted work should resume from a durable checkpoint.

- Review loops should have duplicate-work detection and resource budgets.

## Actual behavior and impact

- Status-only messages became disruptive child turns.

- A bounded audit expanded into 16 modified files and over 2,500 inserted lines.

- No commit or pull request was produced before the work was manually frozen.

- Local telemetry recorded 1,295 model-usage events, approximately 194.2M input tokens and 1.02M output tokens. These are local counters that may include replayed or cached context—not asserted billable totals.

## Suggested fixes

1. Provide a read-only child-status API.

2. Queue parent messages behind active child work by default.

3. Warn before cancelling or superseding an active child turn.

4. Persist checkpoints and resume after interruption.

5. Detect repeated reviews over substantially identical diffs.

6. Enforce time, token, retry and diff-size budgets.

7. Distinguish user, parent-orchestrator, sibling and system messages in diagnostics.

I can provide the full diagnostic report and session identifiers privately to maintainers for server-side telemetry correlation.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

No files, tests, or entry points are identified in the issue. Start with the reproduction steps and trace how parent status requests become child turns; done should include non-interrupting status checks, safe interruption handling, checkpoint resumption, duplicate-work detection, and resource budgets.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Área
ai-infra-agents
Tipo de issue
Error
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Tranquilo
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.