Rust: subagent lifecycle hooks are logged as unknown
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 88/100
Piste de recherche
Commencez dans rust/src/hooks.rs, au niveau de dispatch_hook, et examinez comment les valeurs de HookType sont comparées. Ajoutez une gestion pass-through pour subagentStart et subagentStop, puis ajoutez un test unitaire ciblé couvrant les deux noms. Le travail est terminé lorsque ces hooks de cycle de vie valides renvoient la sortie vide standard sans avertissements, tandis que les noms réellement inconnus continuent d’emprunter le chemin d’avertissement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Java
- Étoiles
- 10.5k
- Forks
- 1.5k
- Merge moyen
- 1 j 5 h
- PR mergées (30 j)
- 108
Préparer son environnement
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de github/copilot-sdk
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
github/copilot-sdk#2793 ·
Les mainteneurs répondent en général sous 1 jour
-
agentic-workflows
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
github/copilot-sdk#2782 ·
Les mainteneurs répondent en général sous 1 jour
-
agentic-workflows
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
github/copilot-sdk#2779 ·
Les mainteneurs répondent en général sous 1 jour
-
agentic-workflows
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
github/copilot-sdk#2760 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
github/copilot-sdk#2759 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de github/copilot-sdk
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
Unify jpa4 into orm8Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
enhancement good first issue
Difficulté 1/5 1-3 heures Accessibilité débutants 88/100
-
category/integration priority/P3 scope/sdk scope/testing status/ready-for-agent type/enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
QwenLM/qwen-code#13053 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour