From 9867adbcc25e9f7abbf46b63cd22e7007f8881e2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Manus=20D=C3=A9ploiement=20Santinova?=
 <deploy@santinova-soft.org>
Date: Fri, 4 Sep 2026 15:51:49 +0000
Subject: [PATCH] =?UTF-8?q?feat:=20placer=20les=20services=20infra=20en=20?=
 =?UTF-8?q?t=C3=AAte=20des=20listes?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/backend/src/routes.js                |  3 ++-
 src/frontend/src/pages/AppsPage.jsx      |  6 +++++-
 src/frontend/src/pages/DashboardPage.jsx |  4 +++-
 src/frontend/src/utils/appSorting.js     | 15 +++++++++++++++
 4 files changed, 25 insertions(+), 3 deletions(-)
 create mode 100644 src/frontend/src/utils/appSorting.js

diff --git a/src/backend/src/routes.js b/src/backend/src/routes.js
index 81ad828..a593b64 100644
--- a/src/backend/src/routes.js
+++ b/src/backend/src/routes.js
@@ -447,8 +447,9 @@ router.get('/public/apps', (req, res) => {
     // getAllStatuses() retourne un tableau vide si aucun check n'a encore été effectué
     // On ne déclenche PAS checkAllApps() ici pour éviter de bloquer la réponse
     const statuses = getAllStatuses();
+    // Le portail répartit lui-même les applications par onglet, y compris INFRA.
+    // L’API expose donc toutes les applications du registre local, sans en masquer.
     const publicApps = config.apps
-      .filter((a) => a.category !== 'INFRA') // Exclure les apps infra (portail, dashboard)
       .map((a) => {
         const status = statuses.find((s) => s.id === a.id);
         return {
diff --git a/src/frontend/src/pages/AppsPage.jsx b/src/frontend/src/pages/AppsPage.jsx
index b5d727e..cc880d1 100644
--- a/src/frontend/src/pages/AppsPage.jsx
+++ b/src/frontend/src/pages/AppsPage.jsx
@@ -3,6 +3,7 @@ import { Box, RefreshCw, Plus } from 'lucide-react';
 import AppCard from '../components/AppCard';
 import AddAppModal from '../components/AddAppModal';
 import { getApps, getGiteaCommits } from '../utils/api';
+import { sortAppsWithInfraFirst } from '../utils/appSorting';
 
 export default function AppsPage({ apps, onRefresh }) {
   const [commitsMap, setCommitsMap] = useState({});
@@ -26,6 +27,9 @@ export default function AppsPage({ apps, onRefresh }) {
     });
   }, [apps]);
 
+  // Les services d’infrastructure restent visibles en premier, sans modifier les données API.
+  const sortedApps = sortAppsWithInfraFirst(apps);
+
   const handleAddSuccess = () => {
     setShowAddModal(false);
     // Rafraîchir la liste après un délai pour laisser le temps au health check
@@ -86,7 +90,7 @@ export default function AppsPage({ apps, onRefresh }) {
         </div>
       ) : (
         <div className="space-y-4">
-          {apps.map((app) => (
+          {sortedApps.map((app) => (
             <AppCard
               key={app.id}
               app={app}
diff --git a/src/frontend/src/pages/DashboardPage.jsx b/src/frontend/src/pages/DashboardPage.jsx
index 7f9752d..05a1757 100644
--- a/src/frontend/src/pages/DashboardPage.jsx
+++ b/src/frontend/src/pages/DashboardPage.jsx
@@ -12,6 +12,7 @@ import {
 } from 'lucide-react';
 import { getApps, getSystemInfo } from '../utils/api';
 import StatusBadge from '../components/StatusBadge';
+import { sortAppsWithInfraFirst } from '../utils/appSorting';
 
 function StatCard({ icon: Icon, label, value, color, subtext }) {
   return (
@@ -44,6 +45,7 @@ export default function DashboardPage({ apps }) {
   const onlineApps = apps.filter((a) => a.status === 'online').length;
   const offlineApps = apps.filter((a) => a.status === 'offline').length;
   const errorApps = apps.filter((a) => a.status === 'error').length;
+  const sortedApps = sortAppsWithInfraFirst(apps);
 
   return (
     <div className="space-y-6">
@@ -147,7 +149,7 @@ export default function DashboardPage({ apps }) {
             <p className="text-gray-500 text-sm">Aucune application détectée</p>
           ) : (
             <div className="space-y-3">
-              {apps.map((app) => {
+              {sortedApps.map((app) => {
                 const isInfra = app.category === "INFRA" || ["portail-santinova", "manus-dashboard"].includes(app.id);
                 return (
                 <div
diff --git a/src/frontend/src/utils/appSorting.js b/src/frontend/src/utils/appSorting.js
new file mode 100644
index 0000000..5715bce
--- /dev/null
+++ b/src/frontend/src/utils/appSorting.js
@@ -0,0 +1,15 @@
+/**
+ * Retourne une copie triée sans modifier la liste fournie par l'API.
+ * Les services INFRA sont affichés en premier ; l'ordre d'origine est conservé
+ * au sein de chaque groupe afin d'éviter des déplacements visuels inutiles.
+ */
+export function sortAppsWithInfraFirst(apps) {
+  return (Array.isArray(apps) ? apps : [])
+    .map((app, index) => ({ app, index }))
+    .sort((left, right) => {
+      const leftPriority = left.app?.category === 'INFRA' ? 0 : 1;
+      const rightPriority = right.app?.category === 'INFRA' ? 0 : 1;
+      return leftPriority - rightPriority || left.index - right.index;
+    })
+    .map(({ app }) => app);
+}