From 6c3bfb79dd98a91b1b4b33e7475db3798ce9533e Mon Sep 17 00:00:00 2001
From: Ahmed Darrazi <ahmed.darrazi@live.de>
Date: Mon, 23 Feb 2026 16:29:54 +0100
Subject: [PATCH] =?UTF-8?q?fix:=20nav=20architecture=20=E2=80=94=20Review?=
 =?UTF-8?q?=20Packs=20to=20Reporting=20group,=20fix=20duplicate=20Inventor?=
 =?UTF-8?q?y,=20normalize=20header=20CTA=20colors?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Move ReviewPackResource from 'Monitoring' to new 'Reporting' nav group
- Fix duplicate 'Inventory' sidebar entries: set $navigationGroup + $navigationLabel on InventoryCluster so it renders inside 'Inventory' group as 'Items'
- Change header CTA button color from warning to primary on ListInventoryItems and ListEntraGroups
---
 app/Filament/Clusters/Inventory/InventoryCluster.php         | 5 +++++
 .../Resources/EntraGroupResource/Pages/ListEntraGroups.php   | 2 +-
 .../InventoryItemResource/Pages/ListInventoryItems.php       | 2 +-
 app/Filament/Resources/ReviewPackResource.php                | 2 +-
 4 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/app/Filament/Clusters/Inventory/InventoryCluster.php b/app/Filament/Clusters/Inventory/InventoryCluster.php
index 9312759..dc19f56 100644
--- a/app/Filament/Clusters/Inventory/InventoryCluster.php
+++ b/app/Filament/Clusters/Inventory/InventoryCluster.php
@@ -7,10 +7,15 @@
 use BackedEnum;
 use Filament\Clusters\Cluster;
 use Filament\Pages\Enums\SubNavigationPosition;
+use UnitEnum;
 
 class InventoryCluster extends Cluster
 {
     protected static ?SubNavigationPosition $subNavigationPosition = SubNavigationPosition::Start;
 
     protected static string|BackedEnum|null $navigationIcon = 'heroicon-o-squares-2x2';
+
+    protected static string|UnitEnum|null $navigationGroup = 'Inventory';
+
+    protected static ?string $navigationLabel = 'Items';
 }
diff --git a/app/Filament/Resources/EntraGroupResource/Pages/ListEntraGroups.php b/app/Filament/Resources/EntraGroupResource/Pages/ListEntraGroups.php
index 20f1a09..3105be5 100644
--- a/app/Filament/Resources/EntraGroupResource/Pages/ListEntraGroups.php
+++ b/app/Filament/Resources/EntraGroupResource/Pages/ListEntraGroups.php
@@ -31,7 +31,7 @@ protected function getHeaderActions(): array
                 Action::make('sync_groups')
                     ->label('Sync Groups')
                     ->icon('heroicon-o-arrow-path')
-                    ->color('warning')
+                    ->color('primary')
                     ->action(function (): void {
                         $user = auth()->user();
                         $tenant = Tenant::current();
diff --git a/app/Filament/Resources/InventoryItemResource/Pages/ListInventoryItems.php b/app/Filament/Resources/InventoryItemResource/Pages/ListInventoryItems.php
index 2816305..22b1ee4 100644
--- a/app/Filament/Resources/InventoryItemResource/Pages/ListInventoryItems.php
+++ b/app/Filament/Resources/InventoryItemResource/Pages/ListInventoryItems.php
@@ -44,7 +44,7 @@ protected function getHeaderActions(): array
                 Action::make('run_inventory_sync')
                     ->label('Run Inventory Sync')
                     ->icon('heroicon-o-arrow-path')
-                    ->color('warning')
+                    ->color('primary')
                     ->form([
                         Select::make('policy_types')
                             ->label('Policy types')
diff --git a/app/Filament/Resources/ReviewPackResource.php b/app/Filament/Resources/ReviewPackResource.php
index c6df3fe..69b9d77 100644
--- a/app/Filament/Resources/ReviewPackResource.php
+++ b/app/Filament/Resources/ReviewPackResource.php
@@ -42,7 +42,7 @@ class ReviewPackResource extends Resource
 
     protected static string|BackedEnum|null $navigationIcon = 'heroicon-o-document-arrow-down';
 
-    protected static string|UnitEnum|null $navigationGroup = 'Monitoring';
+    protected static string|UnitEnum|null $navigationGroup = 'Reporting';
 
     protected static ?string $navigationLabel = 'Review Packs';