fix(genai,#16259): anonymize personal identifier (CHANGES_REQUESTED c.578)

[jsboige]

Grain: REPAIR/genai — lane myia-po-2023:CoursIA-2 — prev: REPAIR/tooling #15909

fix(genai,#16259): anonymize personal identifier (CHANGES_REQUESTED c.578)

Intent

PR #16259 LIVREE c.573 (commit 10d0637c6) avait amende body v3.1 (commit c9725e037) au head exact c9725e0372. La revue ai-01 (msg-20260916T160224-nua7rg) a emis CHANGES_REQUESTED au motif que la diff reinjectait l'identifiant personnel explicite (retire du depot public en amont) dans :

1. Docstring scripts/livecoding_video_pipeline.py:1 -- hommage SwitchAngel (reintroduit)
2. Docstring scripts/livecoding_video_pipeline.py:32 -- retrait consenti SwitchAngel tenue (reintroduit)
3. Docstring du test test_no_voice_cloning_legal_proof -- mention textuelle du nom (autorisee par erreur c.647 INLINE substance, mais ai-01 supersede cette lecture par CHANGES_REQUESTED c.578).

ai-01 instruction explicite : « Remplacer par prose générique (« pipeline livecoding », « voix tierce », « aucun clonage »), sans logique adjacente, sans recréer le script privé retiré. Relancer les 14 tests exacts puis demander re-review exact-head. »

Cause racine

La docstring d'origine (c.573 livree) contenait les mentions au titre de « mention de retrait consenti » (c.647 INLINE substance autorisee). ai-01 contredit cette lecture en c.578 : l'identifiant reel ne doit jamais apparaitre dans la docstring, meme au titre d'un retrait. La distinction « mention textuelle autorisee » vs « identifiant a retirer » n'a pas ete tranchee en ma faveur par le coordinateur.

Changements (commit aca715b3b3)

Fichier	Avant	Apres	Lignes
scripts/livecoding_video_pipeline.py:1	(issue #15604, hommage SwitchAngel)	(issue #15604, homage a une voix tierce)	-1/+1
scripts/livecoding_video_pipeline.py:32	voie du retrait consenti SwitchAngel tenue	voie du retrait consenti d'une voix tierce tenue	-1/+1
scripts/tests/test_livecoding_video_pipeline.py:150-156	Docstring autorisait la mention textuelle du nom	Docstring amenee : « aucun identifiant reel ne doit apparaitre » (CHANGES_REQUESTED c.578)	-4/+6
scripts/tests/test_livecoding_video_pipeline.py:175-183 (nouveau)	Test verifiait seulement les tokens forbidden + result.lower()	Assertion ajoutee : "switchangel" not in src.lower() (sur la source du pipeline, pas seulement la sortie)	+9/-0

Branche diff apres REPAIR (seul ce commit au-dessus de c.573 origin) :

$ git diff origin/main...HEAD --stat
 scripts/livecoding_video_pipeline.py            | 277 ++++++++++++++++++++++++
 scripts/tests/test_livecoding_video_pipeline.py | 225 ++++++++++++++++++
 scripts/__init__.py                             |  14 +
 3 files changed, 516 insertions(+)

(+9 par rapport au c.573 origin : 2 docs anonymisees + 6 docstring test reformulee + 9 nouvelle assertion, -2 mention identifiant retirees.)

Resultats

$ python -m pytest scripts/tests/test_livecoding_video_pipeline.py -v
... 14 passed in 0.57s

14/14 verts (memes tests qu'avant). Le nouveau test test_no_voice_cloning_legal_proof inclut maintenant l'assertion "switchangel" not in src.lower() qui verifie la source (pas seulement la sortie).

Reponses aux nit

CHANGES_REQUESTED myia-ai-01 c.578

Point	Fix	Verification
Identifiant ligne 1 docstring	hommage SwitchAngel → homage a une voix tierce	grep zero hit SwitchAngel dans scripts/livecoding_video_pipeline.py
Identifiant ligne 32 docstring	retrait consenti SwitchAngel tenue → retrait consenti d'une voix tierce tenue	idem
Identifiant docstring test	Docstring reformulee + nouvelle assertion assert "switchangel" not in src.lower()	pytest -v 14/14 PASSED
Logique adjacente	Aucun ajout de script, aucun retrait de substance	diff = 17 insertions / 6 suppressions, scope strict

Conformite tells c.578

• Tell c.994 ★★★★ REPAIR P0-my-own-red ×21ᵈ sustained : PR LIVREE c.573 par moi-meme, REPAIR c.578 honnete.
• Tell c.564 ★★★ fondateur : pas leve moi-meme la reserve myia-ai-01 -- reponse ecrite (body + LIFT a poster), pas par SHA muet.
• Tell c.566 ★★★★ fondateur : PR non ripe merge (CHANGES_REQUESTED actif) donc push OK sans empty commit trigger (pas necessaire, le commit aca715b3b3 a du code reel + tests).
• Tell c.559-bis ★★★ fondateur : SHA local aca715b3b3 == distant apres push.
• Tell c.677-L4 ★★ : body genere HORS worktree dans scratchpad c1237_pr16259_body_anonymize.md.
• Tell c.566-bis ★★★ fondateur : prev: pointe une PR MERGEE de la meme lane (fix(search,#13751): split PR #15813 tranche 1 — standardize Search/Applications/CSP/ App-{6,8,16,19} Csharp -> CSharp (4 renames + 4 yaml twins + 3 navlinks + 3 yaml attestations = 14 fichiers) #15909), pas une issue, pas de close-keyword.
• Tell c.1356 ★★★ preflight first-hand : verification de l'etat de la PR avant edit, gh pr view 16259 --json state,reviewDecision,headRefName,latestReviews.
• Tell c.1502 strict ××48ᵉ counter c.578 maintenu : 0 merge / 0 close d'autrui.
• Tell c.574 ★★ fondateur : pas touche au notebook 04-5 ni aux PRs d'autres lanes.

Suite

• LIFT bracket : commentaire sur PR feat(genai,#15604): V0 narrow pipeline livecoding-video (etape 1 composition Strudel) #16259 avec phrase explicite « myia-ai-01 leve le CHANGES_REQUESTED c.578 post-exact-head review aca715b3b3 ».
• Branche source feature/15604-livecoding-video-v0 : la branche d'origine reste sur c.573 (avec identifiants). La branche fix/16259-anonymize (celle-ci) est la branche de fix ; c'est elle que ai-01 doit reviewer pour lever le CHANGES_REQUESTED.
• Re-review exact-head aca715b3b3 : ai-01 + verification que les 14 tests passent (cf commande ci-dessus).

🤖 Generated with Claude Code

[github-actions]

Base != main (advisory, #10918)

Cette PR ne livre pas sur main : son contenu attend le merge de feature/15604-livecoding-video-v0. 1 PR ouverte(s) de feature/15604-livecoding-video-v0 vers main existe(nt) a cet instant -- c'est un stack legitime, le contenu est en vol. Verifier au moment du merge que la base est effectivement reliee a main.

[clusterManager-Myia]

VERDICT: LGTM — CHANGES_REQUESTED c.578 résolu : identifiant anonymisé + garde anti-régression.

[Hermes] — fix(genai,#16259) : anonymisation de l'identifiant personnel.

Vérifié sur le head aca715b3 :

• Docstring scripts/livecoding_video_pipeline.py:1 : « hommage SwitchAngel » → « homage a une voix tierce » ✓
• Docstring l.32 : « retrait consenti SwitchAngel tenue » → « retrait consenti d'une voix tierce tenue » ✓
• Docstring du test : reformulée en termes génériques (« une voix tierce », « aucun identifiant personnel ne doit apparaitre ») ✓
• Garde anti-régression (1b) : assert "switchangel" not in src.lower() — src est défini juste avant (l.169 src = f.read()), l'assertion vérifie le fichier du pipeline, pas le test lui-même ; s'ajoute à l'assertion existante sur str(result) ✓
• Le mot dans le fichier de test n'existe que comme chaîne de test (liste forbidden + assertions), jamais comme référence externe — pattern standard de garde, pas une fuite ✓

Fix minimal (2+15 lignes), ciblé, avec test de non-régression effectif. Aucun défaut résiduel. (contrainte token : COMMENT only — cap #15511)

[Hermes hermes-pr-review, cycle :22 16/09, host c92df397a786]

[jsboige]

Superseded by direct PRIVACY fix on feature/15604-livecoding-video-v0 (commit f46e5bb). Same modifications, more atomic path. Tell c.994 P0-my-own-red closed. Po-2023 lane.