Merge pull request #383 from JLG-WOCFR-DEV/codex/develop-features-per-roadmap

feat: enrich Slack notifications with customizable identity

Author: JLG-WOCFR-DEV

README.md

@@ -23,7 +23,7 @@ Liens Morts Detector est une extension WordPress qui détecte les liens et image
 - Actions en ligne et groupées : modifier une URL, proposer/appliquer une redirection détectée, re-vérifier, ignorer/restaurer ou dissocier un lien en conservant l’ancre.【F:liens-morts-detector-jlg/includes/class-blc-links-list-table.php†L530-L881】
 
 ### Notifications et suivi
-- Notifications par e-mail ou webhook personnalisable, avec choix du canal (générique JSON, Slack, Microsoft Teams ou Mattermost), du gabarit de message et des catégories de statuts HTTP qui déclenchent un envoi.【F:liens-morts-detector-jlg/includes/blc-settings-fields.php†L295-L366】【F:liens-morts-detector-jlg/includes/blc-notification-payloads.php†L1-L318】
+- Notifications par e-mail ou webhook personnalisable, avec choix du canal (générique JSON, Slack, Microsoft Teams ou Mattermost), du gabarit de message, des catégories de statuts HTTP qui déclenchent un envoi et d’une identité Slack dédiée (canal cible, nom, icône, sections affichées).【F:liens-morts-detector-jlg/includes/blc-settings-fields.php†L1957-L2094】【F:liens-morts-detector-jlg/includes/blc-notification-payloads.php†L61-L222】
 - Résumés post-scan envoyés automatiquement depuis le cœur du scanner dès que des destinataires ou un webhook sont configurés.【F:liens-morts-detector-jlg/includes/blc-scanner.php†L1231-L1250】
 - Chaque lot publie des métriques (durée, progression, réussite/échec) stockées côté WordPress et exposées via un hook pour alimenter des tableaux de bord externes.【F:liens-morts-detector-jlg/includes/blc-scanner.php†L3214-L3250】
 

docs/comparaison-professionnelle.md

@@ -27,6 +27,7 @@
 - **Télémétrie exploitable** : chaque run publie des métriques (durée, progression, réussite) persistées côté WordPress et exposées via un hook pour alimentation de dashboards externes.【F:liens-morts-detector-jlg/includes/blc-scanner.php†L3214-L3250】
 - **Délégation granulaire** : de nouvelles capacités `blc_view_reports`, `blc_fix_links` et `blc_manage_settings` permettent d’accorder l’accès aux tableaux, actions et réglages sans recourir à `manage_options`, et sont attribuées automatiquement aux rôles clés lors de l’activation.【F:liens-morts-detector-jlg/includes/blc-capabilities.php†L9-L113】【F:liens-morts-detector-jlg/includes/blc-activation.php†L507-L546】
 - **Préférences d’accessibilité** : un panneau dédié aux contrastes, animations et tailles de police applique des classes CSS/JS pour adapter immédiatement l’interface aux besoins spécifiques sans surcharger le thème WordPress.【F:liens-morts-detector-jlg/includes/blc-settings-fields.php†L728-L789】【F:liens-morts-detector-jlg/liens-morts-detector-jlg.php†L142-L175】【F:liens-morts-detector-jlg/assets/css/blc-admin-styles.css†L29-L120】【F:liens-morts-detector-jlg/assets/js/blc-admin-scripts.js†L4-L220】
+- **Alertes Slack enrichies** : le connecteur permet maintenant de personnaliser le canal, l’identité d’envoi et les blocs visibles (filtres, top issues) pour refléter les conventions d’escalade professionnelles sans passer par un middleware.【F:liens-morts-detector-jlg/includes/blc-settings-fields.php†L1957-L2094】【F:liens-morts-detector-jlg/includes/blc-notification-payloads.php†L61-L222】【F:tests/NotificationPayloadsTest.php†L72-L194】
 
 ## Lacunes face aux outils professionnels
 - **Pas de supervision native** : l’historique et les métriques sont stockés côté base, mais aucune interface n’expose encore ces données (tableaux ou graphiques) pour des équipes support non techniques.【F:liens-morts-detector-jlg/includes/blc-scanner.php†L25-L148】【F:liens-morts-detector-jlg/includes/blc-admin-pages.php†L147-L236】

docs/priorites-manquantes.md

@@ -1,7 +1,7 @@
 # État des priorités restantes
 
 ## Manques face à la feuille de route interne
-- **Notifications multicanales** : le moteur mutualisé (formatage, throttling, historique) est désormais assuré par `NotificationManager`, mais les connecteurs Slack/Teams/Mattermost décrits dans la roadmap restent à livrer. 【F:liens-morts-detector-jlg/includes/Notifications/NotificationManager.php†L36-L300】【F:liens-morts-detector-jlg/includes/Notifications/NotificationManager.php†L408-L555】
+- **Notifications multicanales** : Slack dispose désormais d’un connecteur paramétrable (canal, identité, blocs), mais les déclinaisons Teams/Mattermost et les scénarios d’escalade restent à produire. 【F:liens-morts-detector-jlg/includes/blc-settings-fields.php†L1957-L2094】【F:liens-morts-detector-jlg/includes/blc-notification-payloads.php†L61-L222】
 - **Optimisations d’interface** : les tests d’accessibilité automatisés (axe-core) et la documentation de la charte UX restent à livrer malgré la création du composant `DashboardSummary`.
 - **Renforcement de la qualité logicielle** : les scénarios Playwright et la chaîne CI complète (lint, packaging) restent planifiés mais absents du code.
 - **Surveillance proactive** : la gestion de seuils configurables, l’escalade multicanale et les visualisations sparkline doivent encore être conçues.
@@ -11,4 +11,4 @@
 - **Scalabilité horizontale** : aucune file distribuée (Redis/SQS) ni worker externe ne complète encore WP-Cron pour les catalogues volumineux.
 - **Résilience réseau avancée** : la rotation de proxys/IP et les stratégies multi-sorties restent à ajouter pour égaler les solutions premium.
 - **Workflows collaboratifs** : l’assignation, la journalisation fine des corrections et le partage de vues enregistrées ne sont pas encore disponibles.
-- **Connecteurs prêts à l’emploi** : l’extension ne propose pas d’intégrations Slack/Jira/ServiceNow ni de documentation REST industrialisée comme les concurrents.
+- **Connecteurs prêts à l’emploi** : l’extension ne propose pas encore de modules Jira/ServiceNow ni de documentation REST industrialisée comme les concurrents, malgré l’adaptateur Slack amélioré.

docs/roadmap-ameliorations.md

@@ -35,6 +35,7 @@ Cette feuille de route décline les pistes d'amélioration listées dans le `REA
 3. Ajouter un adaptateur Teams/Mattermost basé sur les cartes Adaptive Cards.
 
 > ✅ Un gestionnaire centralisé `NotificationManager` mutualise désormais le formatage des messages, la mise en attente anti-doublon et l'historique des envois pour les canaux e-mail et webhook. 【F:liens-morts-detector-jlg/includes/Notifications/NotificationManager.php†L36-L300】【F:liens-morts-detector-jlg/includes/Notifications/NotificationManager.php†L408-L555】
+> ✅ L’adaptateur Slack permet de choisir le canal cible, le nom d’expéditeur, l’icône et les blocs (filtres, principaux problèmes) pour aligner les alertes sur les runbooks d’escalade internes. 【F:liens-morts-detector-jlg/includes/blc-settings-fields.php†L1957-L2094】【F:liens-morts-detector-jlg/includes/blc-notification-payloads.php†L61-L222】【F:tests/NotificationPayloadsTest.php†L72-L194】
 
 **Indicateurs de réussite**
 - Notifications livrées en < 5 secondes après la fin d'un scan.

liens-morts-detector-jlg/assets/css/blc-admin-styles.css

@@ -568,6 +568,25 @@
     gap: 6px;
 }
 
+.blc-notification-channel-advanced {
+    margin-top: 24px;
+    padding: 16px 20px;
+    border: 1px solid var(--blc-admin-border-subtle);
+    border-radius: var(--blc-admin-radius-sm);
+    background: var(--blc-admin-surface-subtle);
+    box-shadow: inset 0 1px 0 rgba(255, 255, 255, 0.35);
+    transition: border-color var(--blc-admin-transition), box-shadow var(--blc-admin-transition);
+}
+
+.blc-notification-channel-advanced[hidden] {
+    display: none;
+}
+
+.blc-notification-channel-advanced.is-active {
+    border-color: var(--blc-admin-border-strong);
+    box-shadow: inset 0 1px 0 rgba(255, 255, 255, 0.4), 0 0 0 1px rgba(110, 86, 207, 0.05);
+}
+
 /* Styles pour l'encadré des statistiques */
 .blc-stats-box {
     display: flex;

liens-morts-detector-jlg/assets/js/blc-admin-scripts.js

@@ -773,6 +773,41 @@ jQuery(document).ready(function($) {
         });
     })();
 
+    (function setupWebhookChannelAdvancedOptions() {
+        var $channelField = $('#blc_notification_webhook_channel');
+        if (!$channelField.length) {
+            return;
+        }
+
+        var $containers = $('.blc-notification-channel-advanced');
+        if (!$containers.length) {
+            return;
+        }
+
+        function refreshVisibility() {
+            var currentChannel = String($channelField.val() || '');
+
+            $containers.each(function() {
+                var $container = $(this);
+                var targetChannel = String($container.data('blcWebhookChannel') || '');
+                var shouldShow = !targetChannel || targetChannel === currentChannel;
+
+                $container.toggleClass('is-active', shouldShow);
+
+                if (shouldShow) {
+                    $container.removeAttr('hidden');
+                    $container.attr('aria-hidden', 'false');
+                } else {
+                    $container.attr('hidden', 'hidden');
+                    $container.attr('aria-hidden', 'true');
+                }
+            });
+        }
+
+        $channelField.on('change', refreshVisibility);
+        refreshVisibility();
+    })();
+
     (function announceBulkNotice() {
         var $notice = $('.blc-bulk-notice');
 

liens-morts-detector-jlg/includes/blc-notification-payloads.php

@@ -72,10 +72,48 @@ function blc_build_slack_notification_payload($message, array $summary, array $s
     $filter_labels = blc_translate_notification_status_filters($filters);
     $trend_label   = blc_format_notification_trend($difference, $previous);
 
+    $channel_override = isset($settings['slack_channel_override']) ? (string) $settings['slack_channel_override'] : '';
+    if (function_exists('blc_normalize_notification_slack_channel_override')) {
+        $channel_override = blc_normalize_notification_slack_channel_override($channel_override);
+    }
+
+    $slack_username = isset($settings['slack_username']) ? (string) $settings['slack_username'] : '';
+    if (function_exists('blc_normalize_notification_slack_username')) {
+        $slack_username = blc_normalize_notification_slack_username($slack_username);
+    }
+
+    $slack_icon = isset($settings['slack_icon']) ? (string) $settings['slack_icon'] : '';
+    if (function_exists('blc_normalize_notification_slack_icon')) {
+        $slack_icon = blc_normalize_notification_slack_icon($slack_icon);
+    }
+
+    $show_filters = true;
+    if (array_key_exists('slack_show_filters', $settings)) {
+        $show_filters = (bool) $settings['slack_show_filters'];
+    }
+
+    $show_top_issues = true;
+    if (array_key_exists('slack_show_top_issues', $settings)) {
+        $show_top_issues = (bool) $settings['slack_show_top_issues'];
+    }
+
+    $title_template = isset($settings['slack_title_template']) ? (string) $settings['slack_title_template'] : '{{subject}}';
+    if (function_exists('blc_normalize_notification_slack_title_template')) {
+        $title_template = blc_normalize_notification_slack_title_template($title_template);
+    }
+
     if ($dataset_label === '' && $dataset_type !== '') {
         $dataset_label = $dataset_type;
     }
 
+    if (!$show_filters) {
+        $filter_labels = array();
+    }
+
+    if (!$show_top_issues) {
+        $top_issues = array();
+    }
+
     $overview_fields = array();
 
     if ($site_name !== '') {
@@ -120,12 +158,25 @@ function blc_build_slack_notification_payload($message, array $summary, array $s
 
     $blocks = array();
 
-    if ($subject !== '') {
+    $header_text = $subject;
+    if ($title_template !== '') {
+        $rendered_title = function_exists('blc_render_notification_message_template')
+            ? blc_render_notification_message_template($title_template, $summary)
+            : $title_template;
+
+        $rendered_title = blc_trim_slack_plain_text($rendered_title, 150);
+
+        if ($rendered_title !== '') {
+            $header_text = $rendered_title;
+        }
+    }
+
+    if ($header_text !== '') {
         $blocks[] = array(
             'type' => 'header',
             'text' => array(
                 'type' => 'plain_text',
-                'text' => $subject,
+                'text' => $header_text,
                 'emoji' => true,
             ),
         );
@@ -215,10 +266,28 @@ function blc_build_slack_notification_payload($message, array $summary, array $s
         );
     }
 
-    return array(
+    $payload = array(
         'text'   => $message,
         'blocks' => $blocks,
     );
+
+    if ($channel_override !== '') {
+        $payload['channel'] = $channel_override;
+    }
+
+    if ($slack_username !== '') {
+        $payload['username'] = $slack_username;
+    }
+
+    if ($slack_icon !== '') {
+        if ($slack_icon[0] === ':') {
+            $payload['icon_emoji'] = $slack_icon;
+        } else {
+            $payload['icon_url'] = $slack_icon;
+        }
+    }
+
+    return $payload;
 }
 
 /**