Skip to content

fix(repo): supprime les marqueurs de conflit commités sur dev (débloque #301) - #325

Merged
KChapron merged 1 commit into
devfrom
fix/dev-conflict-markers-llmaas
Aug 10, 2026
Merged

fix(repo): supprime les marqueurs de conflit commités sur dev (débloque #301)#325
KChapron merged 1 commit into
devfrom
fix/dev-conflict-markers-llmaas

Conversation

@KChapron

Copy link
Copy Markdown
Contributor

Problème

Le merge ef9e0bb8 "Merge origin/dev" a commité une résolution de conflit incomplète : des marqueurs de conflit sont restés dans deux fichiers de dev.

Fichier Impact sur dev
memory-bank/swagger.yaml Fichier YAML invalide. Réintroduit l'entrée http://preprod.api.ai.cloud-temple.com (« Alias DNS pour LB02 (Préprod API) ») retirée par le correctif 43e454e8. Le dépôt étant public, cet endpoint interne était exposé.
tests/llmaas/README.md Structure Markdown cassée. Retour de la section « Token par Défaut » supprimée par ce même correctif.

C'est également la cause unique des deux conflits de la PR #301 (maindev).

Correctif

Les deux fichiers sont réalignés sur la version de main, qui porte la résolution correcte de 43e454e8.

Changement purement soustractif : 27 suppressions, 0 ajout. Aucun contenu légitime n'est perdu — le contenu de dev était exactement celui de main auquel s'ajoutaient les blocs périmés dupliqués et les marqueurs.

Vérifications effectuées

  • Aucun marqueur de conflit restant dans les deux fichiers.
  • memory-bank/swagger.yaml reparsé avec succès ; servers ne contient plus que https://api.ai.cloud-temple.com (API de production).
  • Section « Token par Défaut » absente de tests/llmaas/README.md.
  • Les deux fichiers sont désormais identiques à leur version sur main.
  • Fusion de main simulée après ce correctif : 0 conflit. Le reliquat de la PR 3.25.0 #301 se réduit à static/mail/console_organisation_menu.png.
  • .gitignore durci et tests/llmaas/.env.example sont déjà présents sur dev : rien à reprendre de ce côté.

Suite

Une fois cette PR fusionnée, la PR #301 devient fusionnable sans conflit.

⚠️ Ne pas utiliser le bouton « Resolve conflicts » de GitHub sur la PR #301 : sa branche source étant main, GitHub fusionnerait dev dans main et ferait descendre le contenu non publié de dev dans la branche de production.

Point de suivi (hors périmètre)

La valeur test-token-for-docs reste codée en dur comme repli dans 7 fichiers de tests/llmaas/ sur dev et sur main. C'est un placeholder et non un credential réel, mais il pourrait être remplacé par un échec explicite si LLMAAS_API_KEY est absente.

🤖 Generated with Claude Code

Le merge ef9e0bb a laissé des marqueurs de conflit non résolus
(`=======`, `>>>>>>> 43e454e`) dans deux fichiers de `dev` :

- `memory-bank/swagger.yaml` : le fichier n'est plus du YAML valide et
  réintroduit l'entrée `http://preprod.api.ai.cloud-temple.com`
  (« Alias DNS pour LB02 (Préprod API) ») que le correctif 43e454e
  avait justement retirée. Le dépôt étant public, cet endpoint interne
  était exposé.
- `tests/llmaas/README.md` : structure Markdown cassée et retour de la
  section « Token par Défaut » supprimée par ce même correctif.

Les deux fichiers sont réalignés sur la version de `main`, qui porte la
résolution correcte de 43e454e. Changement purement soustractif :
27 suppressions, 0 ajout, aucun contenu légitime perdu.

Débloque la PR #301 (main -> dev) : ses deux conflits provenaient
uniquement de cette corruption. Fusion vérifiée sans conflit après ce
correctif.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@KChapron KChapron mentioned this pull request Aug 10, 2026
@KChapron
KChapron merged commit 89cba44 into dev Aug 10, 2026
3 of 4 checks passed
@KChapron
KChapron deleted the fix/dev-conflict-markers-llmaas branch August 10, 2026 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant