From c501295642a9885a74c90576f50b6f8e587c0922 Mon Sep 17 00:00:00 2001 From: Nicolas Cantu Date: Fri, 27 Mar 2026 18:46:41 +0100 Subject: [PATCH] feat(deploy): host_stays_on_test in conf.json (replaces lecoffreio id branch) - deploy-by-script-to.sh reads deploy.host_stays_on_test via jq - projects/lecoffreio/conf.json: host_stays_on_test true (permanent) - projects/README.md + agents: document flag --- .cursor/agents/deploy-by-script.md | 18 +++++++++--------- .cursor/agents/deploy-pprod-or-prod.md | 6 +++--- deploy/deploy-by-script-to.sh | 26 ++++++++++++++------------ projects/README.md | 1 + projects/lecoffreio/conf.json | 3 ++- 5 files changed, 29 insertions(+), 25 deletions(-) diff --git a/.cursor/agents/deploy-by-script.md b/.cursor/agents/deploy-by-script.md index d8b013e..6ee7ab0 100644 --- a/.cursor/agents/deploy-by-script.md +++ b/.cursor/agents/deploy-by-script.md @@ -1,6 +1,6 @@ --- name: deploy-by-script -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. +description: Lance le déploiement scripts_v2 vers l'environnement passé au script (test|pprod|prod) ; si deploy.host_stays_on_test dans conf : clone sur test pour pprod|prod ; correctifs sur test d'abord ; /deploy-pprod-or-prod si besoin ; métier via deploy.conf ; logs/deploy_*.log. model: inherit is_background: false --- @@ -31,21 +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//docs`** ; la doc ia_dev est dans **`projects/ia_dev/docs`**. -**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 `. 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:`** 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`**. +**`deploy.host_stays_on_test: true` dans `projects//conf.json` (ex. lecoffreio / LeCoffre) :** le clone applicatif (`repository_root`) reste sur la branche Git locale **`test`** pour les déploiements **pprod**/**prod** via **`deploy-by-script-to.sh`**. L’environnement cible est l’**argument** du script. Le `deploy.sh` du dépôt aligne les remotes / **worktree** (ex. **`git push lecoffre_ng test:`** pour LeCoffre) — pas de **`git checkout pprod`**/**`prod`** sur le poste. Vérifier **`test`** aligné avec le remote attendu. Doc applicative : **`deploy/README.md`** ; agents 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 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). +Cet agent lance le déploiement vers l’**environnement passé au script** (ex. `./deploy/deploy-by-script-to.sh test|pprod|prod`) via **scripts_v2**. **Si `deploy.host_stays_on_test` est absent ou false :** en pratique la branche locale du dépôt applicatif correspond souvent à l’env cible pour **pprod**/**prod**. **Rôle de l’agent :** vérifier le contexte (si **`host_stays_on_test`** : branche locale = **test** ; sinon : 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//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é :** 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é. +- **Qualité :** Avant de lancer le script : si **`deploy.host_stays_on_test`** dans la conf du projet, vérifier que la branche courante du clone applicatif est **`test`** et l’alignement remote (ex. **`lecoffre_ng/test`** pour LeCoffre ; voir **`deploy.sh`** / **`deploy/README.md`**). Sinon, 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 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. +- **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** (pour les projets **sans** **`host_stays_on_test`**), 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 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. +- **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** pour un projet **sans** **`host_stays_on_test`**), 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). @@ -54,7 +54,7 @@ Cet agent lance le déploiement vers l’**environnement passé au script** (ex. 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//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). +**Contexte (standalone) :** Le déploiement est exécuté depuis ia_dev. Les chemins du projet cible (scripts, secrets) viennent de `projects//conf.json` (absolus). **Sauf si `deploy.host_stays_on_test`** (voir 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 @@ -68,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é : 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. +- Vérifier que le script a bien été invoqué : si **`deploy.host_stays_on_test`**, branche courante du clone applicatif = **`test`** et argument du script = env cible ; sinon, 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** sans **`host_stays_on_test`**), 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 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. +- 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** sans **`host_stays_on_test`**), 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) diff --git a/.cursor/agents/deploy-pprod-or-prod.md b/.cursor/agents/deploy-pprod-or-prod.md index 68d0fe3..4c61cf7 100644 --- a/.cursor/agents/deploy-pprod-or-prod.md +++ b/.cursor/agents/deploy-pprod-or-prod.md @@ -81,12 +81,12 @@ Cocher explicitement : **« Ordre anti-duplication : respecté (pas de `/push-by 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] `. - - **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/` existe, `reset --hard` sur **`origin/`**, déploiement (orchestrateur ou `deploy.sh`), puis **`checkout test`**. + - **`deploy.host_stays_on_test: true`** dans `projects//conf.json` (LeCoffre : **true** dans `projects/lecoffreio/conf.json`) : le clone applicatif reste sur **`test`** ; `deploy-by-script-to.sh` ne fait **pas** de `checkout` **pprod**/**prod** ni `reset --hard` sur ces branches ; le `deploy.sh` du dépôt aligne les remotes / **worktree** (voir **`deploy/README.md`** du dépôt applicatif). + - **Sinon :** 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/` existe, `reset --hard` sur **`origin/`**, 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. **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`**. +4. **Branche test après l’étape 3 :** si **`deploy.host_stays_on_test`**, le clone est déjà sur **`test`**. Sinon, 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. diff --git a/deploy/deploy-by-script-to.sh b/deploy/deploy-by-script-to.sh index a12cb0a..8603140 100755 --- a/deploy/deploy-by-script-to.sh +++ b/deploy/deploy-by-script-to.sh @@ -1,7 +1,7 @@ #!/usr/bin/env bash # deploy-by-script-to [project_id] : verify .secrets/, deploy target, restore branch test when needed. # Launched from ia_dev root. Project from projects//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). +# If deploy.host_stays_on_test is true in conf: stay on branch test; no checkout/reset of pprod|prod (deploy.sh aligns remotes / worktree). 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}")" @@ -57,13 +57,15 @@ if [[ "$current" != "test" ]]; then exit 1 fi -# 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 +HOST_STAYS_ON_TEST=false +if [[ -n "${PROJECT_CONFIG_PATH:-}" && -f "${PROJECT_CONFIG_PATH}" ]] && command -v jq >/dev/null 2>&1; then + _ht="$(jq -r '.deploy.host_stays_on_test // false' "$PROJECT_CONFIG_PATH" 2>/dev/null || echo false)" + if [[ "$_ht" == "true" ]]; then + HOST_STAYS_ON_TEST=true + fi fi -if [[ "$LECOFFREIO_DEPLOY_STAYS_ON_TEST" != "true" ]]; then +if [[ "$HOST_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" @@ -76,19 +78,19 @@ 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 -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)" +if [[ "$HOST_STAYS_ON_TEST" == "true" ]]; then + echo "[deploy-by-script-to] Step 2: secrets/${TARGET_BRANCH} OK (${SECRETS_DIR}) ; host stays on test (deploy.host_stays_on_test)" else echo "[deploy-by-script-to] Step 2/5: secrets/${TARGET_BRANCH} OK (${SECRETS_DIR})" fi -if [[ "$LECOFFREIO_DEPLOY_STAYS_ON_TEST" != "true" ]]; then +if [[ "$HOST_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 -if [[ "$LECOFFREIO_DEPLOY_STAYS_ON_TEST" == "true" ]]; then +if [[ "$HOST_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)..." @@ -104,8 +106,8 @@ else "$deploy_script" "$TARGET_BRANCH" fi -if [[ "$LECOFFREIO_DEPLOY_STAYS_ON_TEST" == "true" ]]; then - echo "[deploy-by-script-to] OK: deployed to ${TARGET_BRANCH}, still on branch test (lecoffreio)" +if [[ "$HOST_STAYS_ON_TEST" == "true" ]]; then + echo "[deploy-by-script-to] OK: deployed to ${TARGET_BRANCH}, still on branch test" else echo "[deploy-by-script-to] Step 5/5: checkout test..." git checkout test diff --git a/projects/README.md b/projects/README.md index e34d94e..d929ffc 100644 --- a/projects/README.md +++ b/projects/README.md @@ -49,6 +49,7 @@ One JSON file per project: `projects//conf.json` (e.g. `projects/lecoffreio/ | `deploy.project_orchestrator_path` | recommended | **Relative to `deploy.repository_root`** : single project orchestrator script (sequences project-specific deploy steps). Preferred over `deploy.hooks.phases`. | | `deploy.hooks.phases` | legacy | Optional list of scripts relative to `repository_root`; used only if `project_orchestrator_path` is unset. | | `deploy.secrets_path` | no | **Absolute** path to the project’s `.secrets` directory. | +| `deploy.host_stays_on_test` | no | If **true**, `deploy-by-script-to.sh` keeps the app clone on branch **test** for **pprod**/**prod** deploys (no `checkout`/`reset --hard` on those branches); the project’s `deploy.sh` aligns remotes / worktree. **false** or omitted: legacy behaviour (checkout target branch, sync, deploy, checkout **test**). LeCoffre sets **true** permanently in `projects/lecoffreio/conf.json`. | | `version.package_json_paths` | no | **Absolute** paths to package.json files to bump. | | `mail.imap_bridge_env` | no | **Relative to ia_dev root**: path to IMAP bridge env file (e.g. `.secrets/gitea-issues/imap-bridge.env`). | | `git.token_file` | no | **Relative to ia_dev root**: path to Gitea token file (e.g. `.secrets/gitea-issues/token`). | diff --git a/projects/lecoffreio/conf.json b/projects/lecoffreio/conf.json index dea1534..954b91b 100644 --- a/projects/lecoffreio/conf.json +++ b/projects/lecoffreio/conf.json @@ -12,7 +12,8 @@ "scripts_path": "/home/desk/code/lecoffre_ng_test/deploy/scripts_v2", "deploy_script_path": "/home/desk/code/lecoffre_ng_test/deploy/scripts_v2/deploy.sh", "project_orchestrator_path": "deploy/scripts_v2/deploy.sh", - "secrets_path": "/home/desk/code/ia_dev/projects/lecoffreio/.secrets" + "secrets_path": "/home/desk/code/ia_dev/projects/lecoffreio/.secrets", + "host_stays_on_test": true }, "version": { "package_json_paths": [