From bb7d7e79532d207aba911a1e281fddc38eb3bcdc Mon Sep 17 00:00:00 2001
From: Matthieu <contact@malio.fr>
Date: Sat, 20 Jun 2026 19:52:13 +0200
Subject: [PATCH] feat(mail) : extract Mail front into Nuxt module layer

LST-67 (2.5) front. Completes the Mail module.

- New frontend/modules/mail/ layer (auto-detected): /mail page (3 columns),
  7 components, mail service + DTO, mail store (folders/messages/unread polling).
- sanitizeMailHtml util and useSystemFolderLabel composable stay global;
  AdminMailTab stays in /admin (service import repointed).
- Consumers repointed: AdminMailTab and PM TaskModal -> ~/modules/mail/...;
  the store is auto-imported (Pinia storesDirs) so the layout badge/polling is
  unchanged.
- /mail gated by the mail module: sidebar.php item with module=mail (so
  SidebarFilter disables /mail when the module is off); the layout filters /mail
  from the API sections to avoid a visual duplicate. ROLE_CLIENT exclusion kept.
- i18n key sidebar.general.mail added.

nuxt build passes; /mail and all other routes preserved.
---
 config/sidebar.php                                |  7 ++++++-
 frontend/app/layouts/default.vue                  | 15 ++++++++++-----
 frontend/components/admin/AdminMailTab.vue        |  2 +-
 frontend/i18n/locales/fr.json                     |  3 ++-
 .../mail/components}/MailAttachmentPreview.vue    |  0
 .../mail/components}/MailCreateTaskModal.vue      |  4 ++--
 .../mail/components}/MailFolderTree.vue           |  2 +-
 .../mail/components}/MailLinkTaskModal.vue        |  2 +-
 .../mail/components}/MailMessageList.vue          |  2 +-
 .../mail/components}/MailMessageViewer.vue        |  4 ++--
 .../mail/components}/MailRefreshButton.vue        |  2 +-
 frontend/modules/mail/nuxt.config.ts              |  1 +
 frontend/{ => modules/mail}/pages/mail.vue        |  2 +-
 frontend/{ => modules/mail}/services/dto/mail.ts  |  0
 frontend/{ => modules/mail}/services/mail.ts      |  2 +-
 frontend/{ => modules/mail}/stores/mail.ts        |  4 ++--
 .../project-management/components/TaskModal.vue   |  4 ++--
 17 files changed, 34 insertions(+), 22 deletions(-)
 rename frontend/{components/mail => modules/mail/components}/MailAttachmentPreview.vue (100%)
 rename frontend/{components/mail => modules/mail/components}/MailCreateTaskModal.vue (97%)
 rename frontend/{components/mail => modules/mail/components}/MailFolderTree.vue (98%)
 rename frontend/{components/mail => modules/mail/components}/MailLinkTaskModal.vue (99%)
 rename frontend/{components/mail => modules/mail/components}/MailMessageList.vue (98%)
 rename frontend/{components/mail => modules/mail/components}/MailMessageViewer.vue (98%)
 rename frontend/{components/mail => modules/mail/components}/MailRefreshButton.vue (89%)
 create mode 100644 frontend/modules/mail/nuxt.config.ts
 rename frontend/{ => modules/mail}/pages/mail.vue (99%)
 rename frontend/{ => modules/mail}/services/dto/mail.ts (100%)
 rename frontend/{ => modules/mail}/services/mail.ts (99%)
 rename frontend/{ => modules/mail}/stores/mail.ts (99%)

diff --git a/config/sidebar.php b/config/sidebar.php
index 12d9a50..835e132 100644
--- a/config/sidebar.php
+++ b/config/sidebar.php
@@ -10,8 +10,11 @@ declare(strict_types=1);
  *   - `permission` : section ou item masqué si la permission effective absente (RBAC fin —
  *                    `User::getEffectivePermissions()` ; ROLE_ADMIN bypasse via le voter, mais la
  *                    sidebar évalue les permissions effectives réelles — combiner avec `roles` au besoin).
- * Les items contextuels (Kanban/Groupes/Archives), feature-flag (Documents, Mail) et user-flag
+ * Les items contextuels (Kanban/Groupes/Archives), feature-flag (Documents) et user-flag
  * (Mes absences) restent rendus côté layout, hors de cet endpoint.
+ * Mail est déclaré ici UNIQUEMENT pour le gating module (disabledRoutes si module inactif) ;
+ * son rendu visuel + badge non-lus reste géré côté layout, qui filtre `/mail` de translatedSections
+ * pour éviter le doublon.
  * Les labels sont des clés i18n (sidebar.<domaine>.<item>).
  */
 return [
@@ -23,6 +26,8 @@ return [
             ['label' => 'sidebar.general.myTasks', 'to' => '/my-tasks', 'icon' => 'mdi:clipboard-check-outline', 'module' => 'project-management'],
             ['label' => 'sidebar.general.projects', 'to' => '/projects', 'icon' => 'mdi:folder-outline', 'module' => 'project-management'],
             ['label' => 'sidebar.general.timeTracking', 'to' => '/time-tracking', 'icon' => 'mdi:calendar-edit-outline', 'module' => 'time-tracking'],
+            // Gating module uniquement (cf. en-tête) : rendu visuel + badge gérés côté layout.
+            ['label' => 'sidebar.general.mail', 'to' => '/mail', 'icon' => 'mdi:email-outline', 'module' => 'mail'],
         ],
     ],
     [
diff --git a/frontend/app/layouts/default.vue b/frontend/app/layouts/default.vue
index 7ae9da6..a7c1729 100644
--- a/frontend/app/layouts/default.vue
+++ b/frontend/app/layouts/default.vue
@@ -139,15 +139,20 @@ const route = useRoute()
 const { t } = useI18n()
 const { sections } = useSidebar()
 
+// `/mail` est déclaré dans config/sidebar.php pour le gating module (disabledRoutes),
+// mais son rendu visuel + badge non-lus est géré manuellement ci-dessous (feature-flag Mail).
+// On le filtre des sections dynamiques pour éviter un doublon dans la nav.
 const translatedSections = computed(() =>
     sections.value.map((section) => ({
         label: t(section.label),
         icon: section.icon,
-        items: section.items.map((item) => ({
-            label: t(item.label),
-            to: item.to,
-            icon: item.icon,
-        })),
+        items: section.items
+            .filter((item) => item.to !== '/mail')
+            .map((item) => ({
+                label: t(item.label),
+                to: item.to,
+                icon: item.icon,
+            })),
     })),
 )
 
diff --git a/frontend/components/admin/AdminMailTab.vue b/frontend/components/admin/AdminMailTab.vue
index 0d54a64..a6cecd5 100644
--- a/frontend/components/admin/AdminMailTab.vue
+++ b/frontend/components/admin/AdminMailTab.vue
@@ -140,7 +140,7 @@
 </template>
 
 <script setup lang="ts">
-import { useMailService } from '~/services/mail'
+import { useMailService } from '~/modules/mail/services/mail'
 
 const { getConfiguration, updateConfiguration, testConfiguration } = useMailService()
 
diff --git a/frontend/i18n/locales/fr.json b/frontend/i18n/locales/fr.json
index 9f167de..7d46a2a 100644
--- a/frontend/i18n/locales/fr.json
+++ b/frontend/i18n/locales/fr.json
@@ -353,7 +353,8 @@
       "dashboard": "Tableau de bord",
       "myTasks": "Mes tâches",
       "projects": "Projets",
-      "timeTracking": "Suivi de temps"
+      "timeTracking": "Suivi de temps",
+      "mail": "Messagerie"
     },
     "admin": {
       "section": "Administration",
diff --git a/frontend/components/mail/MailAttachmentPreview.vue b/frontend/modules/mail/components/MailAttachmentPreview.vue
similarity index 100%
rename from frontend/components/mail/MailAttachmentPreview.vue
rename to frontend/modules/mail/components/MailAttachmentPreview.vue
diff --git a/frontend/components/mail/MailCreateTaskModal.vue b/frontend/modules/mail/components/MailCreateTaskModal.vue
similarity index 97%
rename from frontend/components/mail/MailCreateTaskModal.vue
rename to frontend/modules/mail/components/MailCreateTaskModal.vue
index 93f666b..ad839f8 100644
--- a/frontend/components/mail/MailCreateTaskModal.vue
+++ b/frontend/modules/mail/components/MailCreateTaskModal.vue
@@ -1,10 +1,10 @@
 <script setup lang="ts">
-import type { MailMessageDetailDto } from '~/services/dto/mail'
+import type { MailMessageDetailDto } from '~/modules/mail/services/dto/mail'
 import type { Task } from '~/modules/project-management/services/dto/task'
 import type { Project } from '~/modules/project-management/services/dto/project'
 import type { TaskGroup } from '~/modules/project-management/services/dto/task-group'
 import type { UserData } from '~/services/dto/user-data'
-import { useMailService } from '~/services/mail'
+import { useMailService } from '~/modules/mail/services/mail'
 import { useProjectService } from '~/modules/project-management/services/projects'
 import { useTaskGroupService } from '~/modules/project-management/services/task-groups'
 import { useUserService } from '~/services/users'
diff --git a/frontend/components/mail/MailFolderTree.vue b/frontend/modules/mail/components/MailFolderTree.vue
similarity index 98%
rename from frontend/components/mail/MailFolderTree.vue
rename to frontend/modules/mail/components/MailFolderTree.vue
index 604aaba..ac267ff 100644
--- a/frontend/components/mail/MailFolderTree.vue
+++ b/frontend/modules/mail/components/MailFolderTree.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import type { MailFolderDto } from '~/services/dto/mail'
+import type { MailFolderDto } from '~/modules/mail/services/dto/mail'
 
 const props = defineProps<{
     /** Arbre de dossiers (getter folderTree du store) */
diff --git a/frontend/components/mail/MailLinkTaskModal.vue b/frontend/modules/mail/components/MailLinkTaskModal.vue
similarity index 99%
rename from frontend/components/mail/MailLinkTaskModal.vue
rename to frontend/modules/mail/components/MailLinkTaskModal.vue
index 00c641c..2cbbfca 100644
--- a/frontend/components/mail/MailLinkTaskModal.vue
+++ b/frontend/modules/mail/components/MailLinkTaskModal.vue
@@ -1,7 +1,7 @@
 <script setup lang="ts">
 import type { Task } from '~/modules/project-management/services/dto/task'
 import type { Project } from '~/modules/project-management/services/dto/project'
-import { useMailService } from '~/services/mail'
+import { useMailService } from '~/modules/mail/services/mail'
 import { useTaskService } from '~/modules/project-management/services/tasks'
 import { useProjectService } from '~/modules/project-management/services/projects'
 
diff --git a/frontend/components/mail/MailMessageList.vue b/frontend/modules/mail/components/MailMessageList.vue
similarity index 98%
rename from frontend/components/mail/MailMessageList.vue
rename to frontend/modules/mail/components/MailMessageList.vue
index 1367ff1..f4099d2 100644
--- a/frontend/components/mail/MailMessageList.vue
+++ b/frontend/modules/mail/components/MailMessageList.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import type { MailMessageHeaderDto } from '~/services/dto/mail'
+import type { MailMessageHeaderDto } from '~/modules/mail/services/dto/mail'
 
 const props = defineProps<{
     messages: readonly MailMessageHeaderDto[]
diff --git a/frontend/components/mail/MailMessageViewer.vue b/frontend/modules/mail/components/MailMessageViewer.vue
similarity index 98%
rename from frontend/components/mail/MailMessageViewer.vue
rename to frontend/modules/mail/components/MailMessageViewer.vue
index 34467e1..51e0ef3 100644
--- a/frontend/components/mail/MailMessageViewer.vue
+++ b/frontend/modules/mail/components/MailMessageViewer.vue
@@ -1,7 +1,7 @@
 <script setup lang="ts">
-import type { MailMessageDetailDto, MailAddressDto, MailAttachmentDto } from '~/services/dto/mail'
+import type { MailMessageDetailDto, MailAddressDto, MailAttachmentDto } from '~/modules/mail/services/dto/mail'
 import { sanitizeMailHtml } from '~/utils/sanitizeMailHtml'
-import { useMailService } from '~/services/mail'
+import { useMailService } from '~/modules/mail/services/mail'
 
 const props = defineProps<{
     /** Détail complet du message. null = aucun message sélectionné. */
diff --git a/frontend/components/mail/MailRefreshButton.vue b/frontend/modules/mail/components/MailRefreshButton.vue
similarity index 89%
rename from frontend/components/mail/MailRefreshButton.vue
rename to frontend/modules/mail/components/MailRefreshButton.vue
index 2d29e26..c3aaf01 100644
--- a/frontend/components/mail/MailRefreshButton.vue
+++ b/frontend/modules/mail/components/MailRefreshButton.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import { useMailStore } from '~/stores/mail'
+import { useMailStore } from '~/modules/mail/stores/mail'
 
 const store = useMailStore()
 const { syncing } = storeToRefs(store)
diff --git a/frontend/modules/mail/nuxt.config.ts b/frontend/modules/mail/nuxt.config.ts
new file mode 100644
index 0000000..268da7f
--- /dev/null
+++ b/frontend/modules/mail/nuxt.config.ts
@@ -0,0 +1 @@
+export default defineNuxtConfig({})
diff --git a/frontend/pages/mail.vue b/frontend/modules/mail/pages/mail.vue
similarity index 99%
rename from frontend/pages/mail.vue
rename to frontend/modules/mail/pages/mail.vue
index 9673b03..2690254 100644
--- a/frontend/pages/mail.vue
+++ b/frontend/modules/mail/pages/mail.vue
@@ -1,6 +1,6 @@
 <script setup lang="ts">
 import type { Task } from '~/modules/project-management/services/dto/task'
-import { useMailStore } from '~/stores/mail'
+import { useMailStore } from '~/modules/mail/stores/mail'
 
 const { t } = useI18n()
 const router = useRouter()
diff --git a/frontend/services/dto/mail.ts b/frontend/modules/mail/services/dto/mail.ts
similarity index 100%
rename from frontend/services/dto/mail.ts
rename to frontend/modules/mail/services/dto/mail.ts
diff --git a/frontend/services/mail.ts b/frontend/modules/mail/services/mail.ts
similarity index 99%
rename from frontend/services/mail.ts
rename to frontend/modules/mail/services/mail.ts
index c3215c7..944d6d9 100644
--- a/frontend/services/mail.ts
+++ b/frontend/modules/mail/services/mail.ts
@@ -11,7 +11,7 @@ import type {
     MailCreateTaskInput,
     MailLinkTaskInput,
     MailSyncResultDto,
-} from './dto/mail'
+} from '~/modules/mail/services/dto/mail'
 import type { Task } from '~/modules/project-management/services/dto/task'
 
 type BackendMailMessage = {
diff --git a/frontend/stores/mail.ts b/frontend/modules/mail/stores/mail.ts
similarity index 99%
rename from frontend/stores/mail.ts
rename to frontend/modules/mail/stores/mail.ts
index 05d89e0..c779c09 100644
--- a/frontend/stores/mail.ts
+++ b/frontend/modules/mail/stores/mail.ts
@@ -3,8 +3,8 @@ import type {
     MailFolderDto,
     MailMessageHeaderDto,
     MailMessageDetailDto,
-} from '~/services/dto/mail'
-import { useMailService } from '~/services/mail'
+} from '~/modules/mail/services/dto/mail'
+import { useMailService } from '~/modules/mail/services/mail'
 
 const POLL_INTERVAL_MS = 30 * 1000 // 30 secondes
 
diff --git a/frontend/modules/project-management/components/TaskModal.vue b/frontend/modules/project-management/components/TaskModal.vue
index 62e6309..7a42fef 100644
--- a/frontend/modules/project-management/components/TaskModal.vue
+++ b/frontend/modules/project-management/components/TaskModal.vue
@@ -551,8 +551,8 @@ import { useTaskService } from '~/modules/project-management/services/tasks'
 import { useTaskRecurrenceService } from '~/modules/project-management/services/task-recurrences'
 
 import type { Project } from '~/modules/project-management/services/dto/project'
-import { useMailService } from '~/services/mail'
-import type { MailMessageHeaderDto } from '~/services/dto/mail'
+import { useMailService } from '~/modules/mail/services/mail'
+import type { MailMessageHeaderDto } from '~/modules/mail/services/dto/mail'
 
 const props = defineProps<{
     modelValue: boolean