Gaknaad/Site_Projeticone
1
0
Fork 0
Code Issues Pull requests 2 Projects Releases Packages Wiki Activity Actions

fix(ci+chatbot): DB de test ephemere + retrait continue-on-error + affichage reponses chatbot #62

Open
Gaknaad wants to merge 2 commits from fix/ci-no-ignored-errors into main
pull from: fix/ci-no-ignored-errors
merge into: Gaknaad:main
Conversation 0 Commits 2 Files changed 2 +65 -56
Gaknaad commented 2026-06-03 10:44:13 +00:00
Owner
Copy link
  1. CI: tests integration sur PostgreSQL ephemere (fin du couplage prod + mdp hardcode perime), retrait des continue-on-error (audit + integration) -> aucune erreur ignoree. Verifie run #346: tous les steps verts.
  2. Chatbot: les reponses ne saffichaient pas (handler SSE sur reponse JSON). Parsing JSON -> reponses visibles.

Generated with Claude Code

1) CI: tests integration sur PostgreSQL ephemere (fin du couplage prod + mdp hardcode perime), retrait des continue-on-error (audit + integration) -> aucune erreur ignoree. Verifie run #346: tous les steps verts. 2) Chatbot: les reponses ne saffichaient pas (handler SSE sur reponse JSON). Parsing JSON -> reponses visibles. Generated with Claude Code
- Tests d'intégration : remplace la base partagée prod (postgres_shared, mot de
  passe hardcodé périmé → P1000 masqué par continue-on-error) par une base
  PostgreSQL jetable et isolée (port 5433, credentials connus), détruite après
  le run via le socket docker hôte. Plus aucun couplage prod ni secret.
- Retire `continue-on-error: true` sur l'audit npm ET les tests d'intégration :
  toute défaillance bloque désormais le déploiement (aucune erreur ignorée).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Le frontend n'envoie pas `stream: true` → la route /api/chatbot/send renvoie du
JSON `{messages, ctas, suggestions}`. Mais sendMessage parsait la réponse comme
du SSE (readSSEStream, events chunk/done) → aucun event sur du JSON → la bulle
assistant restait vide → réponses invisibles (seul le welcome, qui parse déjà le
JSON, s'affichait). Régression introduite par ad85cfa (désactivation du
streaming côté requête sans basculer le handler).

Fix : le handler parse maintenant le JSON et insère les messages (comme le flux
welcome). Suppression du type SSEEvent et de readSSEStream devenus morts.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This pull request has changes conflicting with the target branch.
  • .forgejo/workflows/site-developpement.yml
  • Site_Developpement/components/chatbot/ChatbotProvider.tsx
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/ci-no-ignored-errors:fix/ci-no-ignored-errors
git switch fix/ci-no-ignored-errors

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff fix/ci-no-ignored-errors
git switch fix/ci-no-ignored-errors
git rebase main
git switch main
git merge --ff-only fix/ci-no-ignored-errors
git switch fix/ci-no-ignored-errors
git rebase main
git switch main
git merge --no-ff fix/ci-no-ignored-errors
git switch main
git merge --squash fix/ci-no-ignored-errors
git switch main
git merge --ff-only fix/ci-no-ignored-errors
git switch main
git merge fix/ci-no-ignored-errors
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Gaknaad/Site_Projeticone!62
No description provided.