Preview : les MP3 TTS manquants sont générés dans la PR - #107
Merged
Merged
Conversation
Jusqu'ici, generate-tts.yml ne tournait que sur main, après la fusion : une matière nouvelle se testait muette en preview, puis il fallait fusionner, lancer le workflow et attendre son déploiement. Le nouveau job `tts` de preview.yml génère les MP3 manquants (idempotent : rien sur une PR sans nouvelle clé), les passe au build de la preview, les commite sur la branche de la PR et relance la CI sur ce commit. Hors du groupe de concurrence gh-pages, pour ne pas retenir les déploiements pendant la génération. Les PR de forks, sans le secret, s'en passent. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01RZKuXTJUA75jCTdpwDngjD
Contributor
|
Preview supprimée (PR fermée). Les URLs ne sont plus accessibles. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Jusqu'ici,
generate-tts.ymlne tournait que surmain, après la fusion. Une matière nouvelle se testait donc muette en preview, et il fallait ensuite fusionner, lancer le workflow et attendre son déploiement. Avec cette PR, la preview a déjà le son, et il n'y a plus qu'à fusionner.Changements
preview.yml, nouveau jobtts, qui tourne avant le build :scripts/generate-tts.mjsavec le secretMISTRAL_API_KEY. Le script est idempotent : sur une PR sans nouvelle clé, il ne génère rien et n'appelle pas l'API ;assets(tts): MP3 générés par la preview, depuis un worktree partant de la tête de branche (le checkout de PR est un commit de fusion). Si la branche a bougé entre-temps, le push échoue sans dommage et le push suivant régénère les fichiers ;gh workflow run ci.yml --ref <branche>), puisqu'un push fait avec leGITHUB_TOKENne déclenche aucun workflow. Ça évite aussi toute boucle avec la preview ;gh-pages(queue: max) passe au niveau du jobbuild-and-deploy. La génération, qui peut prendre quelques minutes, ne retient donc plus les déploiements des autres PR ni demain.ci.yml: ajout deworkflow_dispatch, pour que la preview puisse relancer la CI.CLAUDE.md: le nouveau mécanisme est documenté, y compris legit pullà faire avant de pousser à nouveau après un commit du bot.generate-tts.ymlreste disponible à la main pour un rattrapage surmain.Vérification
main, aucun MP3 ne manque (995 fr et 775 en, vérifié avant d'ouvrir la PR). La preview de cette PR doit donc tourner sans rien générer.preview-slug.test.mjs(45 tests) ; les YAML se chargent sans erreur.queue: maxau niveau d'un job. La preview de cette PR le dira.maindans isc/tablito#101 (verbes irréguliers), sa preview devra générer les 128 MP3irr-*.🤖 Generated with Claude Code
https://claude.ai/code/session_01RZKuXTJUA75jCTdpwDngjD
Generated by Claude Code