fix(windows): CLAUDE_PID names the window before the ancestor walk (#835, slice 1) - #909

Summary

Sous Windows, le hook d'une fenêtre retrouve son claude.exe même quand un bash.exe intermédiaire est déjà sorti. C'est la première tranche de #835 (ADR-1124, qui amende le point 4 d'ADR-1096).

Cause racine. process_ancestry.claude_ancestor_pid remonte la table des processus parent par parent. Dans un vrai hook, la commande bash -c '… "$PY" …' peut faire un exec, et le bash.exe intermédiaire sort (chaîne relevée par le rapporteur le 2026-10-07 : python.exe <- bash.exe <- <pid sorti>). La marche s'arrête alors sur le parent manquant (process_ancestry.py, if parent is None …: return None, le cas épinglé par test_parent_missing_from_table_returns_none). Ensuite groomer_identity.window_pid lève WindowIdentityUnavailableError, SessionStart ne lance aucun cycle de consolidation (ADR-1097) et le registre de la fenêtre n'est pas écrit.

Correctif. Sous Windows, session_registry.find_claude_ancestor appelle le nouveau process_ancestry.window_claude_pid. Celui-ci lit la table une seule fois et renvoie CLAUDE_PID si quatre conditions sont réunies : la valeur est un entier, la table contient ce pid, son exécutable est claude.exe (casse ignorée), et sa date de création est lisible et antérieure ou égale à celle de l'appelant (un pid recyclé est plus jeune que l'appelant). Sinon, la marche d'ADR-1096 répond, sur la même lecture de table. La marche elle-même ne change pas : elle est extraite telle quelle dans _walk. POSIX ne change pas non plus.

Refs #835 (première tranche ; les tranches restantes sont listées dans le commentaire de l'issue).

Type of change

• Bug fix (non-breaking change which fixes an issue)

Test plan

• tests_py/infrastructure/test_process_ancestry_claude_pid.py (nouveau, 9 tests sur une table de processus factice) : la chaîne cassée de Windows: the ADR-1096 ancestor walk returns None in real hooks when an intermediate bash has exited; CLAUDE_PID identifies the window #835 est résolue par CLAUDE_PID ; les cas où CLAUDE_PID est absent, non entier, désigne un autre programme, manque dans la table, est plus jeune que l'appelant ou a une date de création illisible ; la casse ; une seule lecture de table.
• test_hook_with_an_exited_bash_finds_the_window_through_claude_pid dans test_session_registry_windows.py : de bout en bout par find_claude_ancestor. La fixture retire le CLAUDE_PID de l'hôte, qui ne doit pas influencer les tests.
• tests_py/infrastructure/test_session_registry_posix_root.py (nouveau, 2 tests) : la borne ppid <= 1 de la marche POSIX. Elle n'était pas testée, et le gate de mutation la met dans le périmètre parce que find_claude_ancestor change.
• pytest tests_py/infrastructure -k "ancestry or session_registry" : 73 passés, 19 ignorés.
• pytest tests_py/infrastructure tests_py/hooks tests_py/shared : 2937 passés, 5 échecs. Ces 5 échecs (test_embedding_engine.py ×3, test_entry_auto_dispatch.py ×2) se reproduisent à l'identique sur origin/main sans ce diff : sentence-transformers et torch n'ont pas pu être installés ici (erreur de tunnel du proxy).
• Gate de mutation ADR-1122 lancé localement avec la même planification que la CI (scripts/ci_mutation_diff.py plan puis run_job, mutmut 3.8.1 du lock) : process_ancestry.py 97 mutants, RC 0 ; session_registry.py, RC 0. Une première passe avait laissé un mutant équivalent ("" → "XXXX", deux ValueError), que j'ai supprimé en réécrivant la lecture de CLAUDE_PID, et deux mutants survivants de la borne POSIX, tués par les tests ci-dessus.
• ruff check : passe sur les fichiers modifiés. Sur l'ensemble du dépôt, il signale 8 E501 dans scripts/ et tests_py/scripts/, tous déjà présents sur origin/main. ruff format --check passe sur les fichiers modifiés. python scripts/check_craftsmanship.py : OK. python scripts/check_project_wiki.py : les miroirs correspondent.

Audit notes

• Non vérifié : aucun hôte Windows réel ni vrai hook dans cette session. L'existence et la valeur de CLAUDE_PID reposent sur la mesure du rapporteur (Claude Code 2.1.289 à 2.1.309 ; 879 recherches sur 892 du 2026-10-21 au 2026-10-29). La variable n'est pas documentée par l'hôte. Si elle disparaît ou change de sens, le contrôle la rejette et le comportement redevient celui de la marche d'ADR-1096, comme avant ce changement.
• Limites restantes, écrites dans ADR-1124 : un CLAUDE_PID qui désignerait le Claude.exe partagé du bureau serait accepté (même limite que le point 7 d'ADR-1096). Un pid recyclé par un autre claude.exe entre la sortie du claude.exe de la fenêtre et la création de l'appelant passerait aussi le contrôle.
• Hors de cette tranche : le SessionEnd d'une fenêtre dont le claude.exe est déjà sorti, et les sessions de terminal sans CLAUDE_PID.

Coding-standards compliance

• §2.2 Sens des dépendances entre couches préservé (infrastructure → shared uniquement).
• §4.1 / §4.2 Plafonds locaux 300/40 respectés (gate de craftsmanship OK).
• §4.4 window_claude_pid prend 5 paramètres (start_pid, max_depth et trois dépendances injectables nommées, à mot-clé seulement), sur le modèle de claude_ancestor_pid.
• §8 La nouvelle constante _CLAUDE_PID_ENV porte un # source: (Windows: the ADR-1096 ancestor walk returns None in real hooks when an intermediate bash has exited; CLAUDE_PID identifies the window #835).

Reviewer checklist

• CHANGELOG.md mis à jour (Unreleased → Fixed).
• Documentation : ADR-1124 (page wiki canonique, miroir docs/adr/, wiki/manifest.json).
• Aucun secret, identifiant ni donnée personnelle dans le diff.
• CI : pas encore de résultat sur ce commit.

Generated by Claude Code