CVN-N014-ED-S04 — Standard & gate de transport inter-tâches · hub Story¶
Hub documentaire de la Story S04 de l'Epic CVN-N014-ED — codifier le standard de transport inter-tâches (un guideline lisible) et poser le gate de review qui empêche la régression. Dernière Story de l'Epic : sa fermeture clôt CVN-N014-ED. État live = OpenProject (wp#245, GH #1108).
En une phrase¶
Après S01 (flip object-storage XCom), S02 (s43 en pass-by-référence S3) et S03 (inventaire vide), le standard existe mais rien ne l'écrit ni ne le défend. S04 publie le guideline DAG-author (le quoi faire) et pose le gate de review (le quoi rejeter) — le garde-fou anti-régression auquel S03 a transféré l'enforcement.
Le problème¶
Sans une règle écrite + un filet à la review, la classe de bug revient : un futur auteur de DAG re-bricolera un stockage intermédiaire par-tâche (chemin S3 à la main, /tmp local, np.savez maison) → re-incident s43 (prédictions perdues cross-pod). Cette classe de bug est invisible au test unitaire (tmp_path ne voit pas la frontière de pod) et à la review sans règle explicite.
La correction (post plan_review)¶
Le plan_review (committee 59ba834a, PASSED · EXECUTION_RISK) a relevé un blocker load-bearing : un gate limité à dags/** aurait raté l'incident fondateur, car le code de transport s43 vit dans src/ (src/commun/finetune/diagnostic/hamilton/s43_io.py), pas dans le fichier DAG. Les DAGs orchestrent ; la logique de transport est dans le code appelé par les KubernetesPodOperator. Le scope du gate a donc été étendu à la surface runtime réelle.
Les livrables¶
| Livrable | Quoi | Prio |
|---|---|---|
| Guideline | arbre de décision + exemples allowed/disallowed + règle single-pod, lié à ADR-0100 (SSoT) | P0 |
| Gate de review | .coderabbit.yaml — instruction anti-bespoke-store sur dags/** + src/commun/finetune/** + scripts/**, déclenchement par suspicion, exception single-pod |
P0 |
| Checklist comité | item pr_review (pendant humain du gate CR) — OPERATIONS §15 |
P0 |
| Mirror | prompt-library/review/coderabbit-path-instructions.yaml synchronisé |
P1 |
Les documents¶
| Document | Quoi |
|---|---|
| Plan dossier | le quoi & pourquoi — Partie I (problématisation, constats vérifiés, hypothèses, état de l'art, DoD, consolidation) + Partie II (design, fichiers, risques) + bloc verdict plan_review & corrections P0. plan_review PASSED. |
| Guideline INTER_TASK_DATA_TRANSPORT | la page normative DAG-author (livrable principal, live). |
| architecture | N/A — S04 ne change aucun code/design (docs + config de review). Pattern de référence : architecture S02. |
| runbook | N/A — pas de surface opérationnelle nouvelle ; le transport s43 est couvert par le runbook S02. |
| test_strategy | pas de code → pas de test unitaire ; la vérification = guideline live (URL 200) + gate en place (CR sur 3 globs) + fixture de régression (P1). |
État & gates¶
Specified (OpenProject) — plan révisé post plan_review, livrables P0 en implémentation. Pas de code runtime → la PR = guideline + .coderabbit.yaml + ce hub (docs/config).
In specification✅ ·Specified✅ (plan_review PASSED59ba834a/ Meeting #262).→ Closed(DoD) : guideline live (200) + dans la nav · gate en place sur la surface runtime (dags/**+src/commun/finetune/**+scripts/**) + checklist comité · ADR-0100 référencé depuis le guideline · diff docs/config-only → S04 Closed = Epic CVN-N014-ED complète (régression fortement dissuadée / review-gated).