repowarden 4.4.0-next.4 → 4.4.0-next.7

package/ci/tickets/issue.sh

@@ -24,7 +24,7 @@ accepter() {
         body="$(gh pr view "$pr" --json body -q .body)"
         pr_tickets_r "$body" "$head"
         [[ " $REPLY " == *" $n "* ]] || continue
-        if verifier "$pr" "$sha" "$REPLY"; then
+        if verifier "$pr" "$sha" "$REPLY" "$head"; then
             merge_auto "$pr" "$base" "$brouillon"
         fi
     done < <(gh pr list --state open --json number,headRefName,headRefOid,baseRefName,isDraft \

package/ci/tickets/pr.sh

@@ -27,7 +27,7 @@ cmd_pr() {
     # Bots de dépendances, PR de release, livraisons et retours : pas de ticket
     # exigé. Les PR ouvertes par le suivi des tickets lui-même (jeton des
     # Actions, github-actions[bot]) citent leur ticket : suivies normalement.
-    if [[ "${AUTHOR:-}" =~ ^(dependabot|renovate)(\[bot\])?$ || "$HEAD" == release-please--* ||
+    if [[ "${AUTHOR:-}" =~ ^(dependabot|renovate)(\[bot\])?$ || "$HEAD" == release-please--* || "$HEAD" == livraison/* ||
         "$HEAD" == "${INTEGRATION:-develop}" || "$HEAD" == "${MAIN:-main}" ]]; then
         gh api "repos/$GH_REPO/statuses/$HEAD_SHA" -f state=success -f context=ticket \
             -f description="Sans ticket (bot, release ou livraison)" >/dev/null
@@ -64,7 +64,7 @@ cmd_pr() {
     done
     # Le blocage est porté par le statut « ticket » (exigé par la protection) :
     # le job reste vert, un rouge signale une vraie panne
-    if verifier "$PR" "$HEAD_SHA" "$tickets"; then
+    if verifier "$PR" "$HEAD_SHA" "$tickets" "$HEAD"; then
         merge_auto "$PR" "${BASE:-}" "${DRAFT:-false}"
     fi
 }

package/ci/tickets/push.sh

@@ -32,7 +32,7 @@ cmd_push() {
     [ -z "${PUSHER:-}" ] || gh issue edit "$n" --add-assignee "$PUSHER" >/dev/null || true
     est_valide "$n" && statut "$n" "$S_ENCOURS"
     notice_t tk.draft_opened "${url##*/}" "$n"
-    verifier "${url##*/}" "$SHA" "$n" || true
+    verifier "${url##*/}" "$SHA" "$n" "$PUSH_BRANCH" || true
 }
 
 # Revue : corrections demandées → le ticket repasse en cours

package/ci/tickets/verification.sh

@@ -4,14 +4,47 @@
 # Chargé par ci/tickets.sh (après hooks/lib/common.sh et hooks/lib/tickets.sh).
 # shellcheck disable=SC2154 # statuts, réglages et compteurs définis dans les autres modules
 
-# Vérification « ticket » (statut de commit) de la PR $1, commit $2
+# Vrai si le ticket $1 porte une conception écrite par un mainteneur (commentaire
+# « Conception retenue ») : le code suit une conception relue, pas l'inverse
+conception_ecrite() {
+    local corps
+    corps="$(gh issue view "$1" --json comments -q '.comments[]
+        | select(.authorAssociation == "OWNER" or .authorAssociation == "MEMBER" or .authorAssociation == "COLLABORATOR")
+        | .body' 2>/dev/null || true)"
+    grep -qi "conception retenue" <<<"$corps"
+}
+
+# REPLY = premier autre ticket que $2 cité par un commit de la PR $1 (« Ticket : #M »)
+autre_ticket_r() {
+    local cites n
+    cites="$(gh pr view "$1" --json commits -q '.commits[].messageBody' 2>/dev/null || true)"
+    REPLY=""
+    for n in $(grep -oiE '^[[:space:]]*tickets?[[:space:]]*:[[:space:]]*#[0-9]+' <<<"$cites" | grep -oE '[0-9]+' || true); do
+        if [ "$n" != "$2" ]; then REPLY="$n"; return 0; fi
+    done
+    return 1
+}
+
+# Vérification « ticket » (statut de commit) de la PR $1, commit $2, tickets
+# $3, branche $4. Contrôles (pas des consignes) : un seul ticket, validé,
+# porté par la branche (type/N-sujet), avec sa conception écrite ; aucun
+# commit d'un autre ticket.
 verifier() {
-    local pr="$1" sha="$2" tickets="$3" n attente="" liste
+    local pr="$1" sha="$2" tickets="$3" head="${4:-}" n attente="" liste
     for n in $tickets; do est_valide "$n" || attente="$attente #$n"; done
+    [ -z "$head" ] || branch_ticket_r "$head"
     if [ -z "$tickets" ]; then
         _tr tk.none; etat=failure
+    elif [ "$(wc -w <<<"$tickets")" -gt 1 ]; then
+        _tr tk.multi "#${tickets// /, #}"; etat=failure
     elif [ -n "$attente" ]; then
         _tr tk.waiting "${attente# }" "$VALIDE"; etat=failure
+    elif [ -n "$head" ] && [ -z "$REPLY" ]; then
+        _tr tk.bad_branch "$head" "$tickets"; etat=failure
+    elif [ "${REQUIRE_DESIGN:-true}" = true ] && ! conception_ecrite "$tickets"; then
+        _tr tk.no_design "$tickets"; etat=failure
+    elif autre_ticket_r "$pr" "$tickets"; then
+        _tr tk.other_ticket "$REPLY" "$tickets"; etat=failure
     else
         liste=""
         for n in $tickets; do liste="$liste #$n"; done

package/hooks/lib/common.sh

@@ -120,6 +120,8 @@ types_help() {
 branch_name_valid() {
     local branch="$1" allowed pat
     [ -z "$branch" ] && return 0 # HEAD détachée
+    # Instantané de develop créé par la demande de livraison : toujours admis
+    [[ "$branch" != livraison/* ]] || return 0
     cfg_r allowedBranches "$DEFAULT_ALLOWED_BRANCHES"
     allowed="$REPLY"
     set -f # pas d'expansion de "release/*" sur le disque
@@ -143,10 +145,11 @@ pr_targets_r() {
     integration="$REPLY"
     [ -n "$integration" ] || return 0
     cfg_r mainBranch main
-    # Seule la branche d'intégration entre dans la branche principale : tout
-    # le reste (correctifs urgents compris) passe par elle et ses préversions
+    # Seule la branche d'intégration entre dans la branche principale (ou son
+    # instantané livraison/vX.Y.Z, figé à la demande de livraison) : tout le
+    # reste (correctifs urgents compris) passe par elle et ses préversions
     case "$1" in
-        "$integration" | release-please--*) ;;
+        "$integration" | livraison/* | release-please--*) ;;
         *) REPLY="$integration" ;;
     esac
 }

package/hooks/lib/i18n/en.sh

@@ -374,6 +374,10 @@ _msg_en() {
         tk.merged) _M="Merged by PR #%s into %s: in pre-production, available in the next pre-release." ;;
         tk.branch_created) _M="Ticket validated: its branch \`%s\` is created from \`%s\`.\n\nTo work on it:\n\n\`\`\`bash\nrepowarden ticket %s\n\`\`\`\n\n(or \`git fetch && git switch %s\`). The first push opens the draft PR." ;;
         tk.branch_notice) _M="Branch %s created for ticket #%s" ;;
+        tk.multi) _M="PR linked to several tickets (%s): one PR = one ticket, split it." ;;
+        tk.bad_branch) _M="Branch \"%s\": it must carry the ticket number (type/%s-topic); repowarden ticket <n> creates it." ;;
+        tk.no_design) _M="Design missing on #%s: write it as a ticket comment (\"Conception retenue\") before the code." ;;
+        tk.other_ticket) _M="A commit references ticket #%s while the PR handles #%s: one PR = one ticket." ;;
         tk.automerge_armed) _M="PR #%s: auto-merge armed (squash), as soon as the checks are green." ;;
         tk.automerge_no_app) _M="PR #%s: auto-merge not armed, no GitHub App (repowarden app init): with the Actions token, the merge would trigger no workflow." ;;
         tk.automerge_failed) _M="PR #%s: auto-merge not armed (auto-merge not allowed on the repository: repowarden proteger)." ;;

package/hooks/lib/i18n/fr.sh

@@ -374,6 +374,10 @@ _msg_fr() {
         tk.merged) _M="Mergé par la PR #%s dans %s : en préprod, disponible dans la prochaine préversion." ;;
         tk.branch_created) _M="Ticket validé : sa branche \`%s\` est créée depuis \`%s\`.\n\nPour y travailler :\n\n\`\`\`bash\nrepowarden ticket %s\n\`\`\`\n\n(ou \`git fetch && git switch %s\`). Le premier push ouvre la PR en brouillon." ;;
         tk.branch_notice) _M="Branche %s créée pour le ticket #%s" ;;
+        tk.multi) _M="PR liée à plusieurs tickets (%s) : une PR = un ticket, la découper." ;;
+        tk.bad_branch) _M="Branche « %s » : elle doit porter le numéro du ticket (type/%s-sujet) ; repowarden ticket <n> la crée." ;;
+        tk.no_design) _M="Conception absente sur #%s : l'écrire en commentaire du ticket (« Conception retenue ») avant le code." ;;
+        tk.other_ticket) _M="Un commit cite le ticket #%s alors que la PR traite #%s : une PR = un ticket." ;;
         tk.automerge_armed) _M="PR #%s : merge automatique armé (squash), dès que les vérifications sont vertes." ;;
         tk.automerge_no_app) _M="PR #%s : merge automatique non armé, pas de GitHub App (repowarden app init) : avec le jeton des Actions, le merge ne déclencherait aucun workflow." ;;
         tk.automerge_failed) _M="PR #%s : merge automatique non armé (dépôt sans merge automatique autorisé : repowarden proteger)." ;;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "repowarden",
-  "version": "4.4.0-next.4",
+  "version": "4.4.0-next.7",
   "description": "Hooks git et CI multi-langages : messages de commit, nommage des branches, secrets, formatage, tests ciblés, code mort, releases.",
   "keywords": [
     "git-hooks",