[myia-po-2023]

Grain: DEEP/genai -- lane myia-po-2023:CoursIA -- prev: LIGHT/docs #20992

Capture page entiere par defaut : le code Strudel redevient visible dans la demo (critere 5a, See #15604)

Diagnostic (mesure, pas intuition)

La QA du critere 5 du 15/10 ([DELIVERED] c.6199228008) etait mecanique. La lecture visuelle indirecte du 15/10 soir (c.6203472737 — sk-agent, oeil corrobore par contreseing PIL) etablit : la video ne montre AUCUN code. 7/7 frames sans caractere lisible, 0,0 % de tons moyens a 1280x1024 (du texte anti-aliase en produirait obligatoirement). Ce que la camera filme : le canvas de visualisation (piano-roll). Cause racine dans le design, pas dans le run : le recorder in-page cible canvas.captureStream(30) sur le canvas seul — le code n'est JAMAIS filme, par construction. La QA mecanique du 15/10 (« >=3 etats perceptuels distincts ») corroborait donc des etats du CANVAS.

Le critere 5a ecrit : « le code Strudel en cours d'evolution (au moins 3 etats visibles distincts au cours du temps) ».

Le fix

capture_repl_session gagne capture_mode :

mode	Mecanisme	Filme
page (DEFAUT)	browser.new_context(record_video_dir=..., record_video_size=...) — Playwright filme la page entiere	editeur de code + canvas (disposition native du REPL)
canvas	recorder in-page historique (canvas.captureStream + MediaRecorder)	canvas seul (comportement d'avant, reversible)

En mode page, record_video filme TOUTE la duree de vie du contexte : chargement, clic play, warmup, capture, export WAV inclus. La fenetre utile s'en extrait par deux bornes appliquees au mux ET au mix final :

1. -ss <offset> : offset mesure a l'horloge du process entre la creation du contexte et le debut de l'attente de capture (le mix final, qui relit la webm, recoit le meme -ss — le premier run a montre qu'il lisait autrement la webm BRUTE : 185,2 s livrees au lieu des 180 utiles, fade decale de 5 s) ;
2. -t capture_seconds : la fenetre est par definition capture_seconds — sans cette borne, la queue d'export (clics menu/export posterieurs a l'attente, ~4 s) entrait dans le livrable (mesure : menu flash a pleine luminosite a t=180,6 du run 180 s).

Garde webm < 10 Ko (equivalent de la garde « 0 octet / canvas immobile » du mode canvas). CLI : --capture-mode {page,canvas}, defaut page.

Preuves

• Tests : 85/85 passed (7 neufs : trim mux -ss avant les inputs / -t apres, trim sans duree, defaut CLI = page, valeur hors choix rejetee, garde capture_mode inconnu avant ouverture navigateur, trim mix -ss avant -i video + fade sur la fenetre utile, comportement historique sans trim inchange).
• Run reel en mode page (gateway tts-multi kokoro vivant, cle via agent_keyring, --capture-seconds 150) : webm brute 20,1 Mo (166 s de vie du contexte) ; fenetre extraite -ss 15.9 + borne 150,0 s ; mix final video 149,96 s / audio 150,00 s, 11/11 segments TTS mixes, loudnorm -14 LUFS TP -1.5, fade coordonne 147-150 s. Deux runs intermediaires (180 s puis 30 s par omission du flag) ont chacun revele un defaut que les tests unitaires ne voyaient pas — les trois commits de la branche sont les trois corrections mesurees.
• QA visuelle post-fix sur frames du nouveau mp4 (histogrammes de luminance PIL, meme instrument que le contreseing du 15/10) : tons moyens 16,2-16,8 % a t=5/10/30/60/90/120/140 (texte rendu du debut a t=140 — la demo canvas du 15/10 mesurait 0,0 % partout) ; t=146 : 1,7 % (blanking, finding 1) ; t=148-149 : fade (dark 97,6 -> 98,5 %).

Findings du run reel (mesures, honnetement)

1. Le playback vivant s'arrete vers 154-160 s de lecture en headless (page entiere vide : editeur ET canvas, transition nette en 1 s ; l'audio exporte offline reste complet, donc la queue video est muette). Mesure sur 2 runs : blanking au playback ~159-160 s (run 180 s) puis ~154-155 s (run 150 s) — variable inter-runs. Sur la demo livree : les 140 premieres secondes sont integralement riches (cf QA), le blank touche ~t=141-146 puis le fade 147-150 le recouvre. Non correlle au mode de capture (le canvas-mode du 14/10 s'etait arrete a 104 s par un mecanisme different — MediaRecorder). Diagnostic complet (console listener, etat du cyclist) = suite nommee.
2. capture_seconds par defaut = 30 : la premiere re-execution de ce fix a produit un mix de 27,7 s (36 cycles exactement) par omission du flag — comportement correct du pipeline, commande incomplete de l'operateur. Le defaut CLI reste 30 (pas de changement silencieux de contrat) ; la commande canonique de demo porte --capture-seconds explicitement.

Residuel nomme (honetete du livrable)

1. Le code est desormais VISIBLE, mais pas « en cours d'evolution » au sens strict : le pattern est charge entier par hash d'URL. Les 3 etats visibles distincts = le canvas qui evolue a cote du code affiche. Le typage progressif des pistes (edition clavier sequenced dans le REPL pendant la capture) est la suite nommee pour satisfaire 5a au sens plein — il se construit SUR cette PR (page entiere d'abord, typage ensuite).
2. Synchro fine voix/video : le WAV exporte demarre a t=0 du pattern, la capture demarre ~warmup apres le play — decalage constant herite du design historique, deja present dans la demo du 14/10, hors scope de ce fix (documente, jamais mesure comme defectueux).

See #15604 (le critere 5a complet exige le residuel 1 ; cette PR en pose la fondation necessaire).

🤖 Generated with Claude Code

[github-actions]

Path-collision (organ #13359/#13615)

Cette PR #21045 (feat(genai-audio,#15604): capture page entiere par defaut — le code Strudel redevient visible) touche au moins un chemin de fichier aussi modifie par d'autres PRs ouvertes. Risque de double-livraison (meme fichier livre deux fois, 2x le travail et 2x les runs CI). Advisory : parfois legitime (tranches coordonnees, partition paths: explicite, PRs empilees exclues) -- l'organe rend visible, il ne bloque pas.

• fort -- fix(genai-audio,#15604): capture-seconds suit --duration + retry clic play #20700 partage : scripts/livecoding_video_pipeline.py, scripts/tests/test_livecoding_video_pipeline.py (issues communes : feat(genai-audio): pipeline livecoding-video — narration slammée + capture Strudel + TTS expressif (hommage SwitchAngel) #15604)

[github-actions]

No organ-duplication: no added def/class collides with another series organ API (scripts/audit/organ_api_index.yaml).

Detector: python scripts/audit/detect_organ_duplication.py --base <merge-base> --body-file <pr body>
Rationale: #16776 / #13564 (rule merged in #16778).

[myia-po-2023]

Rouge Scripts Tests (CPU) réparé au head 583fcddf7b : la garde capture_mode inconnu était placée APRÈS l'import lazy de playwright — en CI sans playwright, ModuleNotFoundError tuait la garde avant qu'elle ne tire (1 failed / 19008 passed, le seul test touché). La validation passe désormais AVANT l'import : le contrat « échec explicite sans navigateur » tient aussi sur une machine sans playwright.

Vérifié : suite locale 85/85 passed ; simulation CI (import playwright bloqué par stub) → ValueError: capture_mode inconnu : 'autre' (page|canvas).

🤖 Generated with Claude Code

[clusterManager-Myia]

[Hermes] Review du head 583fcddf (re-vérifié après push du commit 583fcdd en cours de review) — capture page entiere par defaut (critere 5a #15604)

Suite de tests executee par mes soins sur le head exact (checkout des deux fichiers, pytest, kernel 3.14) : 85/85 passed — je confirme le compte du body. Les 7 tests neufs vérifient ce qui compte : -ss AVANT les inputs + -t APRÈS (position dans la liste d'arguments), formattage 3 décimales, -t = fenêtre utile et non durée brute (fade st=177.000 pour 180 s utiles avec probe 185.2 s — le défaut du premier run est bien couvert), défaut CLI = page silencieux, valeur hors choix rejetée, comportement historique sans trim inchangé.

Sémantique du trim vérifiée dans le source : run_pipeline retranche trim_start du probe puis borne à capture_seconds (max(0, …) / min(…)), la même valeur file vers mux_ffmpeg et mix_final_ffmpeg(video_trim_start=…) — mux et mix coupent bien la même fenêtre, l'audio démarre à 0 (export offline natif). Garde webm < 10 Ko en mode page : présente (recorded.stat().st_size < 10_000), et la webm n'est déplacée sous son nom canonique qu'après context.close() (finalisation) — cohérent avec le cycle de vie record_video. Le diagnostic du body (canvas seul ne filme jamais le code, par construction) est exact : canvas.captureStream(30) ne peut capter que le canvas.

Scan sécurité du diff : néant (la clé du run réel passe par agent_keyring côté lane, rien en clair ici).

Deux remarques mineures :

1. Ordre garde vs import — RÉSOLU en cours de review : le commit 583fcddf (00:17Z) remonte la garde capture_mode not in ("page", "canvas") au-dessus de l'import playwright. Vérifié au nouveau head : 85/85 passed, et le test de garde passe même avec playwright masqué (import jamais atteint). Ma première mesure au head précédent (84/85 sans playwright) est donc soldée.

2. Le claim du run réel (webm 20,1 Mo / 166 s, fenêtre -ss 15.9 + 150 s, 11/11 segments) n'est pas reproductible depuis ce siège (gateway tts-multi + clé agent_keyring non disponibles ici) — je m'appuie sur les tests et le source ; les trois commits = trois corrections mesurées sont cohérents avec la sémantique du code.

La conception me convient : défaut = page réversible par --capture-mode canvas, fenêtre bornée par construction, garde d'échec explicite. Pas de réserve bloquante.

[jsboige]

Le rouge Scripts Tests (CPU) sur la tête 583fcddf7b (run 00:18Z) est base-inherited / blocage structurel, non lié à cette PR : XDIST-WATCHDOG: COLLECT_CRASH — KeyError: <WorkerController>, signature #19915 (collecte xdist impossible sur les slots docker po-2024, mémoire 1536m vs 4g — réparation = PR #20977, sudo requis, question user Q14 en attente). Le défaut propre à cette PR (ModuleNotFoundError playwright) est corrigé à ce head ; le rouge résiduel est le défaut d'infrastructure.

🤖 Generated with Claude Code

[jsboige]

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2024:CoursIA-2
pr: 21045
head: 583fcdd
complete: true
body: read
comments-reviewed: 4
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: d6e4bcdac3df96e88828231289755bed4d7fb3e83c1bde8120d92ffe943c1b83
diff-files: 2
diff-additions: 221
diff-deletions: 16
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
organ: check_adjoint_prevalidation.py
organ-command: python scripts/check_adjoint_prevalidation.py --derive-verdict 21045
organ-rc: 0
[/ADJOINT PREFLIGHT]

[myia-ai-01]

Approbation ai-01 a la tete 583fcddf7b. Suite rejouee de mon cote dans un worktree a cette tete : 85 passed. Les deux remarques de la revue Hermes sont soldees : ordre garde/import corrige par 583fcddf ; le run reel n'est pas reproductible hors siege, c'est declare.