PR #13921 post-merge follow-up: Hermes [Hermes] COMMENT_WITH_CONCERNS reports conscient concerns 1+2 (docstring align, code mort) — substance OK pour merge #14030

Issue de suivi pour PR #13921 — Voie 3 B.0 (Tell NEW c.867-L1 ★ ★★)

PR parente : #13921 « feat(ict,#13903): Greffe 5 tranche 2/3 — adaptateurs cross-engine Quasi-Experimental / PyMC-05 »

Réserve à reporter : review jsboige ([Hermes] bot), state: COMMENTED mais préfixe [Hermes] COMMENT_WITH_CONCERNS au sens Tell c.745-L2 ★★★ B.0.

Verbatim body review (3 concerns non-bloquants au sens formel, mais formellement [Hermes] COMMENT_WITH_CONCERNS) :

1. Docstring ≠ implémentation : quasi_experimental.py docstring annonce « exécute le notebook en subprocess (papermill) puis extrait les estimateurs par introspection » mais le code recopie les estimateurs à la main (make_panel_did, iv_replay, _panel_did_two_by_two, _iv_2sls_scalaire). Le body PR assume l'option contraire (duplication déclarée byte-identique). pymc_enumerate.py fait la déclaration correcte — aligner la docstring sur celle-ci.

2. Code mort : _execute_notebook_capture / execute_quasi_experimental ne sont appelées par aucun test ni adaptateur (0 référence dans test_bridges.py). Les retirer, ou ajouter un test d'exécution réelle (skip-if-no-papermill).

3. Ordre de merge : ce PR (base main) importe ict.causal_attribution introuvable sur main (404 vérifié) → feat(ict,#13903): Greffe 5 tranche 1/3 -- module ict/causal_attribution.py (close-form NumPy) #13916 doit merger en premier, sinon la collection pytest de test_bridges.py casse sur main.

   • État au 2026-09-01T10:30Z : PR feat(ict,#13903): Greffe 5 tranche 1/3 -- module ict/causal_attribution.py (close-form NumPy) #13916 MERGED ✓ — ce concern est levé en substance par le merge upstream, mais l'import était déjà satisfait par co-localisation (ICT-Series/ict/).

Pourquoi Voie 3 B.0 plutôt que Voie 1/2

Tell NEW c.867-L1 ★ ★★ MAJEUR : sur ce dépôt, self-review cap structurel #12319/#13316 ferme les voies 1 (phrase de levée par tiers) et 2 (thread inline résolu) à l'auteur PR. L'auteur de #13921 est jsboige (self-bot partagé multi-lanes). Le seul lever formel possible = voie 3 = issue de suivi ouverte AVANT merge + body amendé HORS worktree + comment PR nommant l'issue.

Le cutoff organe (check_unaddressed_nits.py) = now() en mode gate → created < cutoff OK : l'issue doit être créée avant le merge, ce qui est le cas ici.

Verdict reporté — assumé par documentation, pas de fix substance en ce cycle

Tell c.745-L2 ★★★ strict 0 fix substance hâtif sans diagnostic complet + 3 tactiques. Pour concerns 1 (docstring) et 2 (code mort), le fix est de portée locale et peut être mené en PR de suivi. La présente issue consigne le report conscient, ne le fixe pas.

Acceptance reportée :

• Concern 1 (docstring ≠ impl) — reporté à PR de suivi (alignement docstring sur pymc_enumerate.py)
• Concern 2 (code mort) — reporté à PR de suivi (retrait ou test skip-if-no-papermill)
• Concern 3 (ordre merge) — résolu upstream par merge feat(ict,#13903): Greffe 5 tranche 1/3 -- module ict/causal_attribution.py (close-form NumPy) #13916 (a0d8a7d06aa6 2026-09-01T04:01:35Z)

Liens

• Tell NEW c.867-L1 ★ ★★ (self-review cap → voie 3 unique)
• Tell c.745-L2 ★★★ (3 BANNED, comment = canal de levée, issue de suivi = report conscient)
• Tell NEW c.836-L1 ★★ (issue-suivi-AVANT-merge = canal de levée)
• Tell NEW c.866-L7 ★ (gh pr edit --body-file HORS worktree)
• Tell c.795-2 ★ ★★ (--body-file strict)

— myia-po-2027:CoursIA-2 c.303