fix(deploy): lecoffreio deploy-by-script-to stays on test; align agents with LeCoffre deploy.sh
- deploy-by-script-to.sh: skip checkout/reset/pprod|prod for project lecoffreio - deploy-by-script.md: env from script arg; host test for lecoffreio - deploy-pprod-or-prod.md: document lecoffreio vs other projects step 3
This commit is contained in:
parent
f6c1ce0399
commit
5624f24193
@ -1,6 +1,6 @@
|
||||
---
|
||||
name: deploy-by-script
|
||||
description: Lance le déploiement scripts_v2 sur l'environnement courant (branche locale), après vérification du suivi des branches ; correctifs applicatifs sur test d'abord ; si pprod/prod, rejouer /deploy-pprod-or-prod ; métier via deploy.conf ; logs/deploy_*.log.
|
||||
description: Lance le déploiement scripts_v2 vers l'environnement passé au script (test|pprod|prod) ; lecoffreio : clone applicatif sur branche test pour tous les env ; correctifs sur test d'abord ; si besoin pprod/prod, /deploy-pprod-or-prod ; métier via deploy.conf ; logs/deploy_*.log.
|
||||
model: inherit
|
||||
is_background: false
|
||||
---
|
||||
@ -31,19 +31,21 @@ En tant qu'agent, avant de solliciter l'ia, regarde ce que tu peux scripter (imp
|
||||
|
||||
**Documentation** : La doc des projets gérés est dans **`projects/<id>/docs`** ; la doc ia_dev est dans **`projects/ia_dev/docs`**.
|
||||
|
||||
Cet agent lance le déploiement pour **l'environnement courant** (nom de la branche locale) via le script scripts_v2. **Rôle de l'agent :** vérifier le contexte (branche = env cible), lancer le script, contrôler la sortie et le code de retour, synthèse et clôture. **Rôle du script :** exécution et orchestration sûre (suivi branches, sync, log, déploiement).
|
||||
**Projet lecoffreio (LeCoffre, dépôt `lecoffre_ng`) :** le clone applicatif (`repository_root` dans `projects/lecoffreio/conf.json`) reste sur la branche Git locale **`test`** pour **tous** les déploiements (**test**, **pprod**, **prod**). L’environnement cible est l’**argument** de `./deploy/deploy-by-script-to.sh lecoffreio <env>`. Le script `deploy/scripts_v2/deploy.sh` aligne **`lecoffre_ng/pprod`** ou **`lecoffre_ng/prod`** sur le tip de **`test`** via **`git push lecoffre_ng test:<branche>`** puis **worktree** — **ne pas** exiger **`git checkout pprod`**/**`prod`** sur le poste. Vérifier **`test`** aligné avec **`lecoffre_ng/test`**. Référence : **`deploy/README.md`** dans le dépôt applicatif ; agents délégants LeCoffre : **`LECOFFRE_REPO/.cursor/agents/deploy-by-script.md`**.
|
||||
|
||||
Cet agent lance le déploiement vers l’**environnement passé au script** (ex. `./deploy/deploy-by-script-to.sh <id> test|pprod|prod`) via **scripts_v2**. **Hors lecoffreio :** en pratique la branche locale du dépôt applicatif correspond souvent à l’env cible. **Rôle de l’agent :** vérifier le contexte (pour **lecoffreio** : branche locale = **test** ; pour les autres : cohérence branche / env selon le projet), lancer le script, contrôler la sortie et le code de retour, synthèse et clôture. **Rôle du script :** exécution et orchestration sûre (suivi branches, sync, log, déploiement).
|
||||
|
||||
**Corrections applicatives : branche test d'abord (obligatoire) :** Si une correction de **code**, **configuration** ou **documentation** du dépôt applicatif (`repository_root` dans `projects/<id>/conf.json`) est nécessaire après un échec ou une analyse pendant un déploiement :
|
||||
- **Branche locale du dépôt applicatif = test** : appliquer la correction sur **test**, committer/pousser selon les règles du projet, puis relancer le déploiement adapté (cet agent si l’env reste **test**).
|
||||
- **Branche locale = pprod ou prod** : **interdit** de traiter le correctif comme un commit **uniquement** sur **pprod**/**prod**. **`git checkout test`** sur le dépôt applicatif, appliquer la correction, puis **rejouer le flux officiel** vers l’environnement visé : exécuter intégralement **`/deploy-pprod-or-prod`** (cible **pprod** ou **prod**) conformément à `.cursor/agents/deploy-pprod-or-prod.md`, en particulier la section **Corrections découvertes sur pprod ou prod** (**`/change-to-all-branches`** puis **`deploy-by-script-to`**). Ne pas se limiter à relancer **seulement** le script de déploiement local sur **pprod**/**prod** après un fix hors **test**.
|
||||
|
||||
**Focus qualité et résolution de problèmes :**
|
||||
- **Qualité :** Vérifier que la branche courante correspond à l'environnement cible avant de lancer le script. Ne pas déployer depuis un état non poussé ou non aligné sans l'avoir vérifié.
|
||||
- **Qualité :** Avant de lancer le script : pour **lecoffreio**, vérifier que la branche courante du clone applicatif est **`test`** et que l’alignement **`lecoffre_ng/test`** est satisfait (voir **`deploy.sh`** / **`deploy/README.md`**). Pour les **autres** projets, vérifier que la branche courante correspond à l’environnement cible. Ne pas déployer depuis un état non poussé ou non aligné sans l’avoir vérifié.
|
||||
- **Résolution de problèmes :** Si le script sort en erreur, analyser la sortie (et le fichier log dans `logs/` si présent) pour identifier la cause (git, SSH, déploiement, migrations, build, lint, TypeScript) ; appliquer la correction puis relancer le script (boucle corriger → pousser → relancer). Ne s'arrêter que si la correction n'est pas possible sans instruction utilisateur.
|
||||
|
||||
- **Boucle corriger-et-retenter (obligatoire) :** En cas de code de sortie non nul, 1) identifier la cause dans la sortie et/ou logs/deploy_*.log ; 2) appliquer la correction dans le code du projet cible en respectant **Corrections applicatives : branche test d'abord** (ci-dessus) ; 3) committer et pousser via push-by-script si des fichiers ont été modifiés et que le flux le permet ; si la branche applicative était **pprod**/**prod**, enchaîner **`/deploy-pprod-or-prod`** au lieu d’un simple relancement isolé sur cette branche ; 4) relancer le déploiement approprié. Répéter jusqu'à succès ou blocage nécessitant instruction utilisateur. Ne pas se contenter de rapporter l'échec sans corriger et retenter.
|
||||
- **Boucle corriger-et-retenter (obligatoire) :** En cas de code de sortie non nul, 1) identifier la cause dans la sortie et/ou logs/deploy_*.log ; 2) appliquer la correction dans le code du projet cible en respectant **Corrections applicatives : branche test d'abord** (ci-dessus) ; 3) committer et pousser via push-by-script si des fichiers ont été modifiés et que le flux le permet ; si le déploiement visait **pprod**/**prod** (argument du script) **ou** si la branche applicative locale était **pprod**/**prod** (hors lecoffreio), enchaîner **`/deploy-pprod-or-prod`** au lieu d’un simple relancement isolé ; 4) relancer le déploiement approprié. Répéter jusqu'à succès ou blocage nécessitant instruction utilisateur. Ne pas se contenter de rapporter l'échec sans corriger et retenter.
|
||||
|
||||
- **Logs et corrections :** Toujours consulter la sortie du script et le fichier logs/deploy_*.log après exécution. En cas d'échec, utiliser ces logs pour identifier la cause, appliquer les corrections (code, config, doc, scripts) en respectant **Corrections applicatives : branche test d'abord**, committer et pousser via push-by-script si des fichiers ont été modifiés (ou enchaîner **`/deploy-pprod-or-prod`** si **pprod**/**prod**), puis relancer le déploiement ou le workflow complet jusqu'à succès ou blocage nécessitant instruction utilisateur.
|
||||
- **Logs et corrections :** Toujours consulter la sortie du script et le fichier logs/deploy_*.log après exécution. En cas d'échec, utiliser ces logs pour identifier la cause, appliquer les corrections (code, config, doc, scripts) en respectant **Corrections applicatives : branche test d'abord**, committer et pousser via push-by-script si des fichiers ont été modifiés (ou enchaîner **`/deploy-pprod-or-prod`** si la cible du script était **pprod**/**prod** ou si la branche applicative locale était **pprod**/**prod** hors lecoffreio), puis relancer le déploiement ou le workflow complet jusqu'à succès ou blocage nécessitant instruction utilisateur.
|
||||
|
||||
**Horodatage et contexte** : appliquer intégralement le bloc défini dans `.cursor/rules/cloture-evolution.mdc` (début et fin d'exécution, lancement et retour des sub-agents).
|
||||
|
||||
@ -52,7 +54,7 @@ Cet agent lance le déploiement pour **l'environnement courant** (nom de la bran
|
||||
2. Présenter à l'utilisateur un résumé de ce que le script va faire : étapes principales, options utilisées, effets attendus.
|
||||
3. Lancer le script uniquement après cette présentation.
|
||||
|
||||
**Contexte (standalone) :** Le déploiement est exécuté depuis ia_dev. Les chemins du projet cible (scripts, secrets) viennent de `projects/<id>/conf.json` (absolus). Pour déployer la **prod**, le projet cible doit être sur la branche **prod** à jour avec `origin/prod` (ex. après un branch-align depuis test).
|
||||
**Contexte (standalone) :** Le déploiement est exécuté depuis ia_dev. Les chemins du projet cible (scripts, secrets) viennent de `projects/<id>/conf.json` (absolus). **Sauf lecoffreio** (voir bloc **Projet lecoffreio** ci-dessus) : pour déployer la **prod**, le projet cible est en général sur la branche **prod** à jour avec `origin/prod` (ex. après un branch-align depuis test).
|
||||
|
||||
## 1. Commande à exécuter
|
||||
|
||||
@ -66,12 +68,12 @@ Le script fait alors automatiquement : suivi des branches origin, sync de la bra
|
||||
|
||||
## 2. Contrôle et résolution de problèmes
|
||||
|
||||
- Vérifier que le script a bien été invoqué (branche courante = environnement cible). En cas de code de sortie non nul, consulter la sortie et le log (logs/deploy_*.log), identifier la cause, appliquer les corrections nécessaires selon **Corrections applicatives : branche test d'abord**, committer et pousser via push-by-script si des fichiers ont été modifiés (ou enchaîner **`/deploy-pprod-or-prod`** si l’env applicatif était **pprod**/**prod**), puis **relancer** le script ou le workflow complet. Répéter jusqu'à succès. Ne pas relancer sans avoir corrigé uniquement si la correction n'est pas possible sans instruction utilisateur.
|
||||
- Vérifier que le script a bien été invoqué : pour **lecoffreio**, branche courante du clone applicatif = **`test`** et argument du script = env cible ; pour les **autres** projets, en pratique branche courante = environnement cible. En cas de code de sortie non nul, consulter la sortie et le log (logs/deploy_*.log), identifier la cause, appliquer les corrections nécessaires selon **Corrections applicatives : branche test d'abord**, committer et pousser via push-by-script si des fichiers ont été modifiés (ou enchaîner **`/deploy-pprod-or-prod`** si la cible invoquée était **pprod**/**prod** ou si la branche applicative locale était **pprod**/**prod** hors lecoffreio), puis **relancer** le script ou le workflow complet. Répéter jusqu'à succès. Ne pas relancer sans avoir corrigé uniquement si la correction n'est pas possible sans instruction utilisateur.
|
||||
|
||||
## 3. Après l'exécution
|
||||
|
||||
- Si le script sort avec 0 : rapporter le succès.
|
||||
- Si le script sort avec un code non nul : consulter les logs (sortie + logs/deploy_*.log), identifier la cause, appliquer les corrections selon **Corrections applicatives : branche test d'abord**, mettre à jour git (push-by-script) ou enchaîner **`/deploy-pprod-or-prod`** si l’env applicatif était **pprod**/**prod**, puis relancer le flux adapté. Rapporter la cause identifiée et la résolution ; ne pas relancer sans correction ou instruction utilisateur si la correction n'a pas pu être faite.
|
||||
- Si le script sort avec un code non nul : consulter les logs (sortie + logs/deploy_*.log), identifier la cause, appliquer les corrections selon **Corrections applicatives : branche test d'abord**, mettre à jour git (push-by-script) ou enchaîner **`/deploy-pprod-or-prod`** si la cible invoquée était **pprod**/**prod** (ou branche applicative locale **pprod**/**prod** hors lecoffreio), puis relancer le flux adapté. Rapporter la cause identifiée et la résolution ; ne pas relancer sans correction ou instruction utilisateur si la correction n'a pas pu être faite.
|
||||
|
||||
## 4. Lint min. 5 avant clôture (obligatoire)
|
||||
|
||||
|
||||
@ -16,7 +16,7 @@ En tant qu'agent, avant de solliciter l'ia, regarde ce que tu peux scripter (imp
|
||||
- Sous-agents : uniquement si nécessaire ; descriptions courtes ; éviter « explore » si grep/read/chemin connu suffit.
|
||||
- Réponses concises, sans répéter règles ou docs déjà référencées.
|
||||
|
||||
- **Lint (obligatoire avant clôture)** : Sur le dépôt applicatif du projet (`repository_root` et `build_dirs` dans `projects/<id>/conf.json`), exécuter `npm run lint` (ou équivalent) pour **chaque** `build_dir` de la conf — **tout** le périmètre à chaque fois, pas seulement le sous-projet modifié dans la session (ex. tâche front : lancer aussi le lint sur les autres `build_dirs`). Compter **erreurs + warnings**. Si **N ≥ 1** : appliquer des corrections dans **ce** run jusqu'à traiter **au moins min(5, N)** diagnostics (donc **au moins 5** lorsque N ≥ 5 ; si N < 5, tout corriger jusqu'à 0). **Interdit** de s'exonérer par un lint déjà passé dans `pousse`/build **sans** changements ESLint dans le workspace, ou en reportant sur un **`/fix-lint` ultérieur** : les corrections (min. 5 quand N ≥ 5) font partie **du même run** que la clôture. Clôture : commandes, périmètres, **décompte avant/après**. Voir `.cursor/rules/cloture-lint.mdc`, dont la section **Diagnostics préexistants / hors périmètre de la session** (correction obligatoire pour tout diagnostic du périmètre, y compris hors fichiers modifiés dans ce run ; **interdit** en clôture : « warning existant », « hors scope session », « préexistait »).
|
||||
- **Lint (obligatoire avant clôture)** : Sur le dépôt applicatif du projet (`repository_root` et `build_dirs` dans `projects/<id>/conf.json`), exécuter `npm run lint` (ou équivalent) pour **chaque** zone concernée par la conf. Compter **erreurs + warnings**. Si **N ≥ 1** : appliquer des corrections dans **ce** run jusqu'à traiter **au moins min(5, N)** diagnostics (donc **au moins 5** lorsque N ≥ 5 ; si N < 5, tout corriger jusqu'à 0). **Interdit** de s'exonérer par un lint déjà passé dans `pousse`/build **sans** changements ESLint dans le workspace, ou en reportant sur un **`/fix-lint` ultérieur** : les corrections (min. 5 quand N ≥ 5) font partie **du même run** que la clôture. Clôture : commandes, périmètres, **décompte avant/après**. Voir `.cursor/rules/cloture-lint.mdc`, dont la section **Diagnostics préexistants / hors périmètre de la session** (correction obligatoire pour tout diagnostic du périmètre, y compris hors fichiers modifiés dans ce run ; **interdit** en clôture : « warning existant », « hors scope session », « préexistait »).
|
||||
|
||||
# Agent deploy-pprod-or-prod
|
||||
|
||||
@ -79,13 +79,14 @@ Cocher explicitement : **« Ordre anti-duplication : respecté (pas de `/push-by
|
||||
- **Si KO :** Analyser la sortie et les logs (logs/deploy_*.log), identifier la cause. Toute correction sur le dépôt applicatif : **test** d’abord (voir section **Corrections découvertes sur pprod ou prod**), puis relancer **`/change-to-all-branches`** jusqu'à succès.
|
||||
- **Si OK :** Passer à l'étape 3.
|
||||
|
||||
3. **Lancer le script deploy-by-script-to** avec la branche en paramètre (`pprod` ou `prod`) :
|
||||
3. **Lancer le script deploy-by-script-to** avec l’environnement en paramètre (`pprod` ou `prod`) :
|
||||
- Le script est lancé depuis la racine de ia_dev. Avec project_id (optionnel), MAIL_TO ou AI_AGENT_TOKEN le dépôt cible est celui de la conf (deploy.secrets_path). Lancer : `./deploy/deploy-by-script-to.sh [project_id] <pprod|prod>`.
|
||||
- Le script fait : passage dans le dépôt du projet (conf), checkout sur la branche en paramètre, vérification que `.secrets/<env>` existe, mise à jour forcée de la branche locale sur la branche distante, déploiement (orchestrateur ou `deploy.sh` ; flags métier via `deploy.conf` uniquement), checkout test.
|
||||
- **Projet lecoffreio :** le clone applicatif reste sur **`test`** ; `deploy-by-script-to.sh` ne fait **pas** de `checkout` **pprod**/**prod** ni `reset --hard` sur ces branches ; `deploy/scripts_v2/deploy.sh` aligne **`lecoffre_ng`** et utilise le **worktree** (voir **`deploy/README.md`** du dépôt LeCoffre).
|
||||
- **Autres projets :** le script fait en général : passage dans le dépôt du projet, `checkout` sur la branche paramètre, vérification que `.secrets/<env>` existe, `reset --hard` sur **`origin/<branche>`**, déploiement (orchestrateur ou `deploy.sh`), puis **`checkout test`**.
|
||||
- **Si KO :** Analyser la sortie et les logs, identifier la cause. **Ne pas** se limiter à corriger sur la branche **pprod**/**prod** : retour **test**, corrections, puis **rejouer depuis l’étape 2** (**`/change-to-all-branches`** puis **`deploy-by-script-to`**) jusqu'à succès (section **Corrections découvertes sur pprod ou prod**).
|
||||
- **Si OK :** Passer à l'étape 4.
|
||||
|
||||
4. **Checkout test** : Le script remet déjà sur test. Vérifier que la branche courante est test après le script.
|
||||
4. **Branche test après l’étape 3 :** pour **lecoffreio**, le clone est déjà sur **`test`**. Pour les **autres** projets, le script a remis **`test`** : vérifier que la branche courante est **`test`**.
|
||||
|
||||
5. **Lancer /push-by-script** : Exécuter intégralement l'agent push-by-script (commande /push-by-script). Message de commit fourni par l'agent selon les règles du projet.
|
||||
|
||||
|
||||
@ -1,6 +1,7 @@
|
||||
#!/usr/bin/env bash
|
||||
# deploy-by-script-to [project_id] <target_branch>: checkout target, verify .secrets/<env>, force sync with origin, deploy target, checkout test.
|
||||
# deploy-by-script-to [project_id] <target_branch>: verify .secrets/<env>, deploy target, restore branch test when needed.
|
||||
# Launched from ia_dev root. Project from projects/<id>/conf.json; id from param, or MAIL_TO or AI_AGENT_TOKEN. Target: pprod | prod only.
|
||||
# lecoffreio: stays on branch test; deploy.sh aligns lecoffre_ng and uses worktree (no checkout pprod/prod on host).
|
||||
set -euo pipefail
|
||||
|
||||
SCRIPT_REAL="$(readlink -f "${BASH_SOURCE[0]:-$0}" 2>/dev/null || realpath "${BASH_SOURCE[0]:-$0}" 2>/dev/null || echo "${BASH_SOURCE[0]:-$0}")"
|
||||
@ -56,9 +57,17 @@ if [[ "$current" != "test" ]]; then
|
||||
exit 1
|
||||
fi
|
||||
|
||||
echo "[deploy-by-script-to] Step 1/5: checkout ${TARGET_BRANCH}..."
|
||||
if [[ "$(git rev-parse --abbrev-ref HEAD)" != "$TARGET_BRANCH" ]]; then
|
||||
git checkout "$TARGET_BRANCH"
|
||||
# LeCoffre (lecoffre_ng): deploy/scripts_v2/deploy.sh requires host branch test; aligns remote via git push + worktree.
|
||||
LECOFFREIO_DEPLOY_STAYS_ON_TEST=false
|
||||
if [[ "${PROJECT_ID:-}" == "lecoffreio" || "${IA_PROJECT_ID:-}" == "lecoffreio" ]]; then
|
||||
LECOFFREIO_DEPLOY_STAYS_ON_TEST=true
|
||||
fi
|
||||
|
||||
if [[ "$LECOFFREIO_DEPLOY_STAYS_ON_TEST" != "true" ]]; then
|
||||
echo "[deploy-by-script-to] Step 1/5: checkout ${TARGET_BRANCH}..."
|
||||
if [[ "$(git rev-parse --abbrev-ref HEAD)" != "$TARGET_BRANCH" ]]; then
|
||||
git checkout "$TARGET_BRANCH"
|
||||
fi
|
||||
fi
|
||||
|
||||
SECRETS_PARENT="${SECRETS_BASE:-${LECOFFRE_SECRETS_BASE:-$PROJECT_ROOT/.secrets}}"
|
||||
@ -67,13 +76,23 @@ if [[ ! -d "$SECRETS_DIR" ]]; then
|
||||
echo "[deploy-by-script-to][ERROR] secrets env dir missing: ${SECRETS_DIR} (set SECRETS_BASE or deploy.secrets_path in conf)" >&2
|
||||
exit 1
|
||||
fi
|
||||
echo "[deploy-by-script-to] Step 2/5: secrets/${TARGET_BRANCH} OK (${SECRETS_DIR})"
|
||||
if [[ "$LECOFFREIO_DEPLOY_STAYS_ON_TEST" == "true" ]]; then
|
||||
echo "[deploy-by-script-to] Step 2: secrets/${TARGET_BRANCH} OK (${SECRETS_DIR}) ; host stays on test (lecoffreio)"
|
||||
else
|
||||
echo "[deploy-by-script-to] Step 2/5: secrets/${TARGET_BRANCH} OK (${SECRETS_DIR})"
|
||||
fi
|
||||
|
||||
echo "[deploy-by-script-to] Step 3/5: force sync local branch with origin/${TARGET_BRANCH}..."
|
||||
git fetch origin
|
||||
git reset --hard "origin/${TARGET_BRANCH}"
|
||||
if [[ "$LECOFFREIO_DEPLOY_STAYS_ON_TEST" != "true" ]]; then
|
||||
echo "[deploy-by-script-to] Step 3/5: force sync local branch with origin/${TARGET_BRANCH}..."
|
||||
git fetch origin
|
||||
git reset --hard "origin/${TARGET_BRANCH}"
|
||||
fi
|
||||
|
||||
echo "[deploy-by-script-to] Step 4/5: deploy ${TARGET_BRANCH} (business flags from deploy.conf only)..."
|
||||
if [[ "$LECOFFREIO_DEPLOY_STAYS_ON_TEST" == "true" ]]; then
|
||||
echo "[deploy-by-script-to] Step 3: deploy ${TARGET_BRANCH} from branch test (business flags from deploy.conf only)..."
|
||||
else
|
||||
echo "[deploy-by-script-to] Step 4/5: deploy ${TARGET_BRANCH} (business flags from deploy.conf only)..."
|
||||
fi
|
||||
if [[ -n "${IA_PROJECT_ID:-}" && -x "${DEPLOY_IA}/orchestrator.sh" ]]; then
|
||||
"${DEPLOY_IA}/orchestrator.sh" "$TARGET_BRANCH"
|
||||
else
|
||||
@ -85,7 +104,10 @@ else
|
||||
"$deploy_script" "$TARGET_BRANCH"
|
||||
fi
|
||||
|
||||
echo "[deploy-by-script-to] Step 5/5: checkout test..."
|
||||
git checkout test
|
||||
|
||||
echo "[deploy-by-script-to] OK: aligned, synced, deployed to ${TARGET_BRANCH}, back on test"
|
||||
if [[ "$LECOFFREIO_DEPLOY_STAYS_ON_TEST" == "true" ]]; then
|
||||
echo "[deploy-by-script-to] OK: deployed to ${TARGET_BRANCH}, still on branch test (lecoffreio)"
|
||||
else
|
||||
echo "[deploy-by-script-to] Step 5/5: checkout test..."
|
||||
git checkout test
|
||||
echo "[deploy-by-script-to] OK: aligned, synced, deployed to ${TARGET_BRANCH}, back on test"
|
||||
fi
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user