Rust: subagent lifecycle hooks are logged as unknown
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 88/100
Línea de trabajo
Comienza en rust/src/hooks.rs, en dispatch_hook, e inspecciona cómo se comparan los valores de HookType. Añade un manejo de paso directo para subagentStart y subagentStop, y después añade una prueba unitaria enfocada que cubra ambos nombres. Se considera terminado cuando estos hooks de ciclo de vida válidos devuelvan la salida vacía estándar sin advertencias, mientras que los nombres realmente desconocidos sigan usando la ruta de advertencia.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Bug
When Rust SDK hooks are enabled and the Copilot CLI starts or stops a task sub-agent, the CLI sends hooks.invoke requests with hookType values subagentStart and subagentStop. The Rust SDK logs both as unknown:
unknown hook type hook_type="subagentStart"
unknown hook type hook_type="subagentStop"
The generated HookType enum already declares both values, but rust/src/hooks.rs::dispatch_hook does not match them. The public SessionHooks API also has no dedicated callbacks for these lifecycle hooks.
Reproduction
- Create a Rust SDK session with hooks enabled.
- Invoke the built-in
tasktool so a sub-agent starts and completes. - Observe the warnings from
github_copilot_sdk::hooks.
Expected behavior
The SDK recognizes these valid callback-only lifecycle hook names and returns the standard empty output when no public callback is registered. Truly unknown hook names must continue to use the warning path.
Proposed fix
Handle subagentStart and subagentStop as pass-through hooks in dispatch_hook, with a focused unit test covering both names.
- Lenguaje dominante
- Java
- Estrellas
- 10.5k
- Forks
- 1.5k
- Merge medio
- 1 d 8 h
- PR fusionados (30 d)
- 111
Preparar el entorno
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/copilot-sdk
-
agentic-workflows
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
github/copilot-sdk#2782 ·
Los mantenedores suelen responder en 1 día
-
agentic-workflows
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
github/copilot-sdk#2779 ·
Los mantenedores suelen responder en 1 día
-
agentic-workflows
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
github/copilot-sdk#2760 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
github/copilot-sdk#2759 ·
Los mantenedores suelen responder en 1 día
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
github/copilot-sdk#2758 ·
Los mantenedores suelen responder en 1 día
Todos los issues de github/copilot-sdk
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
checkstyle/checkstyle#21755 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
Make docs website more visibleAbiertodocumentation Good for newcomer quick win
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
CodeForPhilly/benefit-decision-toolkit#519 ·
Los mantenedores suelen responder en 1 día
-
area-deployment triage:bot-seen triage:needs-human
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
microsoft/aspire#20533 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día