repowarden 4.1.1 → 4.1.2

package/bin/proteger

@@ -14,7 +14,7 @@
 #                [--sans-tags] [--dry-run]
 # --sans-tags : dépôt qui déplace lui-même un tag (tag majeur flottant v2…).
 # Réglages équivalents dans .repowarden.conf (requiredReviews, codeOwnerReview,
-# environment, environmentReviewers). Demande un gh connecté, admin du dépôt.
+# unattributedApproval, environment, environmentReviewers). Demande un gh connecté, admin du dépôt.
 set -euo pipefail
 
 ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd -P)"
@@ -25,6 +25,12 @@ cfg_r requiredReviews 0
 REVIEWS="$REPLY"
 cfg_r codeOwnerReview false
 CODEOWNERS="$REPLY"
+# Approbation supplémentaire des changements non attribués : GitHub l'active
+# d'office si elle n'est pas posée, et une PR ouverte par une application
+# (jeton des Actions : PR de livraison, PR brouillon des tickets) attend alors
+# un humain, même sans approbation exigée. Posée explicitement, false par défaut.
+cfg_r unattributedApproval false
+UNATTRIBUTED="$REPLY"
 cfg_r environment ""
 ENVIRONMENT="$REPLY"
 cfg_r environmentReviewers ""
@@ -51,6 +57,7 @@ done
 [ -n "$REPO" ] || REPO="$(gh repo view --json nameWithOwner -q .nameWithOwner)"
 [[ "$REVIEWS" =~ ^[0-9]$ ]] || { err_t ci.proteger.reviews_invalid; exit 1; }
 [[ "$CODEOWNERS" =~ ^(true|false)$ ]] || { err_t ci.proteger.codeowners_invalid; exit 1; }
+[[ "$UNATTRIBUTED" =~ ^(true|false)$ ]] || { err_t ci.proteger.unattributed_invalid; exit 1; }
 # Avec relecture : le dernier à pousser ne peut pas approuver son propre push
 last_push=false
 [ "$REVIEWS" = 0 ] || last_push=true
@@ -104,6 +111,7 @@ branch_ruleset() {
       "allowed_merge_methods": [$3],
       "dismiss_stale_reviews_on_push": true,
       "require_code_owner_review": $CODEOWNERS,
+      "require_extra_approval_for_unattributed_changes": $UNATTRIBUTED,
       "require_last_push_approval": ${6:-$last_push},
       "required_approving_review_count": ${5:-$REVIEWS},
       "required_review_thread_resolution": true}},
@@ -206,7 +214,7 @@ echo_t ci.proteger.4 "$methods" "$delete_on_merge"
 [ -z "$integration" ] || echo_t ci.proteger.delivery "$livraison"
 echo_t ci.proteger.5 "$tags_msg"
 [ -z "$integration" ] || echo_t ci.proteger.6 "$integration"
-echo_t ci.proteger.7 "$REVIEWS" "$CODEOWNERS"
+echo_t ci.proteger.7 "$REVIEWS" "$CODEOWNERS" "$UNATTRIBUTED"
 if [ "$REVIEWS" != 0 ] || [ "$CODEOWNERS" = true ]; then
     echo_t ci.proteger.8
 fi

package/hooks/lib/i18n/en.sh

@@ -254,7 +254,7 @@ _msg_en() {
         ci.proteger.4) _M="  merges: %s; automatic branch deletion: %s" ;;
         ci.proteger.5) _M="  v* tags: %s" ;;
         ci.proteger.6) _M="  default branch: %s" ;;
-        ci.proteger.7) _M="  required approvals: %s; CODEOWNERS review: %s" ;;
+        ci.proteger.7) _M="  required approvals: %s; CODEOWNERS review: %s; approval of unattributed changes: %s" ;;
         ci.proteger.8) _M="  (admins keep an explicit bypass, recorded in the PR history)" ;;
         ci.proteger.9) _M="  \"%s\" environment: approval by %s; protected branches and v* tags" ;;
         ci.proteger.10) _M="Ruleset \"%s\" updated." ;;
@@ -263,6 +263,7 @@ _msg_en() {
         ci.proteger.13) _M="Settings applied." ;;
         ci.proteger.reviews_invalid) _M="--relecteurs: a number from 0 to 9 is expected." ;;
         ci.proteger.codeowners_invalid) _M="codeOwnerReview: true or false." ;;
+        ci.proteger.unattributed_invalid) _M="unattributedApproval: true or false." ;;
         ci.proteger.no_branch) _M="None of the branches \"%s\" exists on %s." ;;
         ci.proteger.no_cleanup) _M="Automatic branch deletion disabled (it would delete develop): add a workflow calling nettoyage-branches.yml, otherwise merged branches remain (see In an organisation)." ;;
         ci.proteger.delivery) _M="  delivery into main: %s approval(s) required (moving to a new version); auto-merge disabled" ;;

package/hooks/lib/i18n/fr.sh

@@ -254,7 +254,7 @@ _msg_fr() {
         ci.proteger.4) _M="  merges : %s ; suppression auto des branches : %s" ;;
         ci.proteger.5) _M="  tags v* : %s" ;;
         ci.proteger.6) _M="  branche par défaut : %s" ;;
-        ci.proteger.7) _M="  approbations exigées : %s ; revue des CODEOWNERS : %s" ;;
+        ci.proteger.7) _M="  approbations exigées : %s ; revue des CODEOWNERS : %s ; approbation des changements non attribués : %s" ;;
         ci.proteger.8) _M="  (les admins gardent un contournement explicite, tracé dans l'historique de la PR)" ;;
         ci.proteger.9) _M="  environnement « %s » : approbation de %s ; branches protégées et tags v*" ;;
         ci.proteger.10) _M="Ruleset « %s » mis à jour." ;;
@@ -263,6 +263,7 @@ _msg_fr() {
         ci.proteger.13) _M="Réglages appliqués." ;;
         ci.proteger.reviews_invalid) _M="--relecteurs : nombre de 0 à 9 attendu." ;;
         ci.proteger.codeowners_invalid) _M="codeOwnerReview : true ou false." ;;
+        ci.proteger.unattributed_invalid) _M="unattributedApproval : true ou false." ;;
         ci.proteger.no_branch) _M="Aucune des branches « %s » n'existe sur %s." ;;
         ci.proteger.no_cleanup) _M="Suppression automatique des branches désactivée (elle supprimerait develop) : ajouter un workflow qui appelle nettoyage-branches.yml, sinon les branches mergées restent (voir En organisation)." ;;
         ci.proteger.delivery) _M="  livraison vers main : %s approbation(s) exigée(s) (passage à une nouvelle version) ; merge automatique désactivé" ;;

package/package.json

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