[release][high] Rendre le bootstrap contrôlable pendant les helpers de credentials #72

New issue

Open

opened 2026-10-03 16:49:23 +00:00 by hephaistos · 0 comments

hephaistos commented

2026-10-03 16:49:23 +00:00

Owner

Revue de préparation publique

Branche de travail : release.
Code examiné : 6052defc07a8d85a26a62014a72d23c4855a4786.
Sévérité : high. Domaine : session.

Constat et impact

La construction STT/cleanup précède ipc.Listen. Un helper lent empêche
status/stop/cancel de contrôler la session en préparation. Le timeout
parent tue le leader, sans preuve du nettoyage des groupes descendants.

Preuves

internal/cli/start_unix.go:126-153,258-286;
internal/session/factory.go:66-72; internal/core/core.go:70-74,149-152.
Le contrat annoncé dans docs/cli.md:30-35 doit être reconcilié.

Travail recommandé

Exposer le contrôle avant les opérations lentes conformément au contrat,
et garantir un’arrêt borné du bootstrap et de ses helpers. Ne pas
présenter la survie des descendants comme reproduite: elle reste à tester.

Validation et caractère bloquant

Avec des helpers locaux synchronisés, tester status/cancel pendant leur
attente et le timeout parent. Aucun helper ni verrou orphelin. Bloque
le parcours avec résolution lente des credentials.

Suivi

En file d’attente issue de la revue, pas un correctif en cours.
Héphaïstos porte le diagnostic et les propositions techniques sur release.
Mathieu arbitre les choix de produit et autorise les fusions.

## Revue de préparation publique Branche de travail : `release`. Code examiné : `6052defc07a8d85a26a62014a72d23c4855a4786`. Sévérité : **high**. Domaine : **session**. ## Constat et impact La construction STT/cleanup précède ipc.Listen. Un helper lent empêche status/stop/cancel de contrôler la session en préparation. Le timeout parent tue le leader, sans preuve du nettoyage des groupes descendants. ## Preuves internal/cli/start_unix.go:126-153,258-286; internal/session/factory.go:66-72; internal/core/core.go:70-74,149-152. Le contrat annoncé dans docs/cli.md:30-35 doit être reconcilié. ## Travail recommandé Exposer le contrôle avant les opérations lentes conformément au contrat, et garantir un’arrêt borné du bootstrap et de ses helpers. Ne pas présenter la survie des descendants comme reproduite: elle reste à tester. ## Validation et caractère bloquant Avec des helpers locaux synchronisés, tester status/cancel pendant leur attente et le timeout parent. Aucun helper ni verrou orphelin. Bloque le parcours avec résolution lente des credentials. ## Suivi En file d’attente issue de la revue, pas un correctif en cours. Héphaïstos porte le diagnostic et les propositions techniques sur `release`. Mathieu arbitre les choix de produit et autorise les fusions.