maximus/vps-health-api
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Constats non bloquants de la revue des épingles CI (PR #34) #35

New issue
Closed
opened 2026-09-29 23:38:56 +00:00 by maximus · 0 comments
maximus commented 2026-09-29 23:38:56 +00:00
Owner
Copy link

Constats non bloquants de la revue de la PR #34 (commentaire 17003), à ne pas perdre avec la PR fermée.

  1. --noproxy ne protège que le pré-contrôle de scripts/boot-from-copy.sh. La boucle d'attente et les assertions lancent curl sans cette option : avec http_proxy=http://127.0.0.1:9, le pré-contrôle voit le port libre, puis le script échoue en did not answer on :3097 within 10 s. Échec franc, pas un faux vert. Correctif proposé : export NO_PROXY='*' no_proxy='*' en tête de script (couvre tous les appels), et retirer le --noproxy devenu redondant.
  2. continue-on-error en style flow : - { name: x, run: false, continue-on-error: true } échappe à l'épingle de __tests__/ci.test.js, ancrée en début de ligne. Retirer l'ancrage (/continue-on-error\s*:/ sur ciCode, déjà débarrassé des lignes de commentaire) ferme le cas. Prouver le rouge par mutation en style flow.
  3. Trous connus de l'heuristique « require en colonne 0 » (à garder en tête, pas forcément à corriger) : une fonction écrite sur une seule ligne en colonne 0 (function lazy() { return require("x") }) et un require( renvoyé à la ligne en colonne 0 dans une fonction passent au travers. Aucune des deux ne correspond au style du dépôt.
Constats non bloquants de la revue de la PR #34 (commentaire 17003), à ne pas perdre avec la PR fermée. 1. **`--noproxy` ne protège que le pré-contrôle** de `scripts/boot-from-copy.sh`. La boucle d'attente et les assertions lancent `curl` sans cette option : avec `http_proxy=http://127.0.0.1:9`, le pré-contrôle voit le port libre, puis le script échoue en `did not answer on :3097 within 10 s`. Échec franc, pas un faux vert. Correctif proposé : `export NO_PROXY='*' no_proxy='*'` en tête de script (couvre tous les appels), et retirer le `--noproxy` devenu redondant. 2. **`continue-on-error` en style flow** : `- { name: x, run: false, continue-on-error: true }` échappe à l'épingle de `__tests__/ci.test.js`, ancrée en début de ligne. Retirer l'ancrage (`/continue-on-error\s*:/` sur `ciCode`, déjà débarrassé des lignes de commentaire) ferme le cas. Prouver le rouge par mutation en style flow. 3. **Trous connus de l'heuristique « `require` en colonne 0 »** (à garder en tête, pas forcément à corriger) : une fonction écrite sur une seule ligne en colonne 0 (`function lazy() { return require("x") }`) et un `require(` renvoyé à la ligne en colonne 0 dans une fonction passent au travers. Aucune des deux ne correspond au style du dépôt.
maximus added the
status:ready
type:infra
source:human
 labels 2026-09-29 23:38:56 +00:00
maximus added
status:in-progress
 and removed
status:ready
 labels 2026-10-01 00:55:10 +00:00
maximus added
status:review
 and removed
status:in-progress
 labels 2026-10-01 00:57:54 +00:00
maximus added
status:approved
 and removed
status:review
 labels 2026-10-01 01:08:16 +00:00
maximus added
status:review
 and removed
status:approved
 labels 2026-10-01 19:57:38 +00:00
maximus added
status:needs-fix
 and removed
status:review
 labels 2026-10-01 20:11:24 +00:00
maximus added
status:review
 and removed
status:needs-fix
 labels 2026-10-01 20:14:58 +00:00
maximus added
status:approved
 and removed
status:review
 labels 2026-10-01 20:27:37 +00:00
maximus added
status:review
 and removed
status:approved
 labels 2026-10-01 21:42:50 +00:00
maximus added
status:approved
 and removed
status:review
 labels 2026-10-01 21:53:01 +00:00
maximus removed the
status:approved
 label 2026-10-01 22:01:08 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
issue-4-runtime-only-coolify-secrets
No results found.
Labels
Clear labels   
autopilot:pending-human

PR produite par /autopilot, attend review humaine (jamais mergee par le run)

  
source:analyste

Créée par le Super-Analyste

  
source:defenseur

Créée par l Escouade Défenseur

  
source:human

Créée par un humain

  
source:medic

Créée par le Medic (auto-détection)

  
status:approved

PR approuvée, prête à merger

  
status:blocked

Bloquée (question, dépendance, clarification)

  
status:in-progress

Un agent (Medic/Dev) travaille dessus

  
status:needs-clarification

Issue bloquee par une ambiguite LOW pendant un run autonome, Max doit trancher

  
status:needs-fix

Reviewer demande des corrections

  
status:ready

Analysée et documentée, prête à prendre

  
status:review

PR créée, en attente du Reviewer

  
status:triage

Nouvelle issue, pas encore analysée

  
type:bug

Correction de bug → Medic

  
type:feature

Nouvelle fonctionnalité → Dev

  
type:infra

Config, CI/CD, déploiement → Dev

  
type:refactor

Refactoring / amélioration technique → Dev

  
type:schema

Modèle de données / migrations → Dev

  
type:security

Issue de sécurité → Medic ou Dev

No labels
autopilot:pending-human
source:analyste
source:defenseur
source:human
source:medic
status:approved
status:blocked
status:in-progress
status:needs-clarification
status:needs-fix
status:ready
status:review
status:triage
type:bug
type:feature
type:infra
type:refactor
type:schema
type:security
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: maximus/vps-health-api#35
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?