From 16f052ee50bf747137d11277edddd32e29114b46 Mon Sep 17 00:00:00 2001
From: Taylor Downs <taylor@openfn.org>
Date: Fri, 23 Oct 2026 06:42:56 +0200
Subject: [PATCH 1/3] Add default-open attribute for expanded groups

---
 resources/views/components/system-status.blade.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/resources/views/components/system-status.blade.php b/resources/views/components/system-status.blade.php
index 9efc3768..5a708039 100644
--- a/resources/views/components/system-status.blade.php
+++ b/resources/views/components/system-status.blade.php
@@ -12,7 +12,7 @@
     <div class="divide-y divide-zinc-900/10 dark:divide-white/15">
         @foreach ($groups as $group)
             {{ \Cachet\Facades\CachetView::renderHook(\Cachet\View\RenderHook::STATUS_PAGE_SYSTEM_STATUS_GROUP_BEFORE) }}
-            <div data-component="system-status-group" data-component-group-id="{{ $group['model']->getKey() }}" x-data x-disclosure>
+            <div data-component="system-status-group" data-component-group-id="{{ $group['model']->getKey() }}" x-data x-disclosure @if ($group['model']->isExpanded(auth()->user())) default-open @endif>
                 <button data-slot="trigger" x-disclosure:button class="w-full px-5 py-4 text-left transition hover:bg-zinc-50/60 dark:hover:bg-white/[0.02] sm:px-6">
                     <div class="flex items-center justify-between gap-4">
                         <span class="flex min-w-0 items-center gap-2">

From c8fcae103eb52ca7f40377c91f3cce6cf7b9307e Mon Sep 17 00:00:00 2001
From: Taylor Downs <taylor@openfn.org>
Date: Fri, 23 Oct 2026 07:17:46 +0200
Subject: [PATCH 2/3] slightly more robust fix

---
 .../views/components/system-status.blade.php  |  2 +-
 src/View/Components/SystemStatus.php          | 40 ++++++++-
 tests/Feature/StatusPage/StatusPageTest.php   | 87 +++++++++++++++++++
 3 files changed, 126 insertions(+), 3 deletions(-)

diff --git a/resources/views/components/system-status.blade.php b/resources/views/components/system-status.blade.php
index 5a708039..f0ab995d 100644
--- a/resources/views/components/system-status.blade.php
+++ b/resources/views/components/system-status.blade.php
@@ -12,7 +12,7 @@
     <div class="divide-y divide-zinc-900/10 dark:divide-white/15">
         @foreach ($groups as $group)
             {{ \Cachet\Facades\CachetView::renderHook(\Cachet\View\RenderHook::STATUS_PAGE_SYSTEM_STATUS_GROUP_BEFORE) }}
-            <div data-component="system-status-group" data-component-group-id="{{ $group['model']->getKey() }}" x-data x-disclosure @if ($group['model']->isExpanded(auth()->user())) default-open @endif>
+            <div data-component="system-status-group" data-component-group-id="{{ $group['model']->getKey() }}" x-data x-disclosure @if ($group['expanded']) default-open @endif>
                 <button data-slot="trigger" x-disclosure:button class="w-full px-5 py-4 text-left transition hover:bg-zinc-50/60 dark:hover:bg-white/[0.02] sm:px-6">
                     <div class="flex items-center justify-between gap-4">
                         <span class="flex min-w-0 items-center gap-2">
diff --git a/src/View/Components/SystemStatus.php b/src/View/Components/SystemStatus.php
index 34065568..475abfb6 100644
--- a/src/View/Components/SystemStatus.php
+++ b/src/View/Components/SystemStatus.php
@@ -2,6 +2,8 @@
 
 namespace Cachet\View\Components;
 
+use Cachet\Cachet;
+use Cachet\Enums\ComponentGroupVisibilityEnum;
 use Cachet\Enums\ComponentStatusEnum;
 use Cachet\Enums\IncidentStatusEnum;
 use Cachet\Models\Component;
@@ -24,7 +26,7 @@
  * @phpstan-type DayEvent array{label: string, status: ComponentStatusEnum, icon: string}
  * @phpstan-type DayEvents Collection<string, list<DayEvent>>
  * @phpstan-type ComponentRow array{model: Component, status: ComponentStatusEnum, days: ComponentDays, events: DayEvents, uptime: float|null, available_seconds: float, operational_seconds: float}
- * @phpstan-type GroupRow array{model: ComponentGroup, components: Collection<int, covariant ComponentRow>, days: StatusDays, events: DayEvents, uptime: float|null, available_seconds: float, operational_seconds: float}
+ * @phpstan-type GroupRow array{model: ComponentGroup, expanded: bool, components: Collection<int, covariant ComponentRow>, days: StatusDays, events: DayEvents, uptime: float|null, available_seconds: float, operational_seconds: float}
  */
 class SystemStatus extends ViewComponent
 {
@@ -62,13 +64,15 @@ public function render(): View|Closure|string
      */
     private function groups(): Collection
     {
+        $componentsWithActiveIncidents = $this->componentsWithActiveIncidents();
+
         $groups = ComponentGroup::query()
             ->visible(auth()->check())
             ->orderBy('order')
             ->when(auth()->check(), fn (Builder $query) => $query->users(), fn ($query) => $query->guests())
             ->get()
             ->toBase()
-            ->map(function (ComponentGroup $group): array {
+            ->map(function (ComponentGroup $group) use ($componentsWithActiveIncidents): array {
                 $components = $this->components(
                     $group->components()->enabled()->orderBy('order')
                 );
@@ -76,6 +80,7 @@ private function groups(): Collection
                 /** @var GroupRow $row */
                 $row = [
                     'model' => $group,
+                    'expanded' => $this->isExpanded($group, $components, $componentsWithActiveIncidents),
                     'components' => $components,
                     'days' => $this->aggregateDays($components),
                     'events' => $this->aggregateEvents($components),
@@ -91,6 +96,37 @@ private function groups(): Collection
         return $groups;
     }
 
+    /**
+     * Fetch the IDs of components affected by an unresolved incident the current viewer can see.
+     *
+     * @return Collection<int, int>
+     */
+    private function componentsWithActiveIncidents(): Collection
+    {
+        $authenticated = Cachet::user() !== null;
+
+        return Component::query()
+            ->whereHas('incidents', fn ($query) => $query->unresolved()->viewableBy($authenticated))
+            ->pluck('id')
+            ->toBase();
+    }
+
+    /**
+     * Determine whether a group renders open, judged only by the components it displays.
+     *
+     * @param  Collection<int, covariant ComponentRow>  $components
+     * @param  Collection<int, int>  $componentsWithActiveIncidents
+     */
+    private function isExpanded(ComponentGroup $group, Collection $components, Collection $componentsWithActiveIncidents): bool
+    {
+        return match ($group->collapsed) {
+            ComponentGroupVisibilityEnum::expanded => true,
+            ComponentGroupVisibilityEnum::collapsed_unless_incident => $components
+                ->contains(fn (array $component): bool => $componentsWithActiveIncidents->contains($component['model']->getKey())),
+            default => false,
+        };
+    }
+
     /**
      * Load the history needed to calculate each component's uptime.
      *
diff --git a/tests/Feature/StatusPage/StatusPageTest.php b/tests/Feature/StatusPage/StatusPageTest.php
index 5ae2113f..12df8be7 100644
--- a/tests/Feature/StatusPage/StatusPageTest.php
+++ b/tests/Feature/StatusPage/StatusPageTest.php
@@ -1,5 +1,6 @@
 <?php
 
+use Cachet\Enums\ComponentGroupVisibilityEnum;
 use Cachet\Enums\ComponentStatusEnum;
 use Cachet\Enums\ComponentStatusSourceEnum;
 use Cachet\Enums\IncidentStatusEnum;
@@ -16,6 +17,7 @@
 use Illuminate\Support\Carbon;
 use Illuminate\Support\Facades\Cache;
 use Illuminate\Support\Str;
+use Workbench\App\User;
 
 it('renders the status page', function () {
     $this->get(route('cachet.status-page'))
@@ -321,6 +323,91 @@
         ], escape: false);
 });
 
+it('opens system status groups by default according to their collapsed setting', function () {
+    $settings = app(AppSettings::class);
+    $settings->display_system_status = true;
+    $settings->save();
+
+    $expanded = ComponentGroup::factory()->create(['collapsed' => ComponentGroupVisibilityEnum::expanded]);
+    $collapsed = ComponentGroup::factory()->create(['collapsed' => ComponentGroupVisibilityEnum::collapsed]);
+    $collapsedWithIncident = ComponentGroup::factory()->create(['collapsed' => ComponentGroupVisibilityEnum::collapsed]);
+    Component::factory()->create(['component_group_id' => $expanded->id]);
+    Component::factory()->create(['component_group_id' => $collapsed->id]);
+    Incident::factory()->create()->components()->attach(
+        Component::factory()->create(['component_group_id' => $collapsedWithIncident->id]),
+        ['component_status' => ComponentStatusEnum::major_outage],
+    );
+
+    $page = $this->get(route('cachet.status-page'))->assertOk()->getContent();
+
+    expect($page)
+        ->toMatch(systemStatusGroupPattern($expanded, open: true))
+        ->toMatch(systemStatusGroupPattern($collapsed, open: false))
+        ->toMatch(systemStatusGroupPattern($collapsedWithIncident, open: false));
+});
+
+it('opens a collapsed unless incident system status group only for an unresolved incident on a displayed component', function (Closure $setup, bool $open) {
+    $settings = app(AppSettings::class);
+    $settings->display_system_status = true;
+    $settings->save();
+
+    $group = ComponentGroup::factory()->create(['collapsed' => ComponentGroupVisibilityEnum::collapsed_unless_incident]);
+    Component::factory()->create(['component_group_id' => $group->id]);
+    $setup($group);
+
+    $page = $this->get(route('cachet.status-page'))->assertOk()->getContent();
+
+    expect($page)->toMatch(systemStatusGroupPattern($group, $open));
+})->with([
+    'no incident' => [fn (ComponentGroup $group) => null, false],
+    'unresolved incident' => [fn (ComponentGroup $group) => Incident::factory()->create()->components()->attach(
+        $group->components()->first(),
+        ['component_status' => ComponentStatusEnum::major_outage],
+    ), true],
+    'resolved incident' => [fn (ComponentGroup $group) => Incident::factory()->create(['status' => IncidentStatusEnum::fixed])->components()->attach(
+        $group->components()->first(),
+        ['component_status' => ComponentStatusEnum::major_outage],
+    ), false],
+    'unresolved incident on a disabled component' => [fn (ComponentGroup $group) => Incident::factory()->create()->components()->attach(
+        Component::factory()->disabled()->create(['component_group_id' => $group->id]),
+        ['component_status' => ComponentStatusEnum::major_outage],
+    ), false],
+    'unpublished incident' => [fn (ComponentGroup $group) => Incident::factory()->scheduled()->create()->components()->attach(
+        $group->components()->first(),
+        ['component_status' => ComponentStatusEnum::major_outage],
+    ), false],
+]);
+
+it('opens a collapsed unless incident system status group for an authenticated-only incident only when signed in', function (bool $authenticated) {
+    $settings = app(AppSettings::class);
+    $settings->display_system_status = true;
+    $settings->save();
+
+    $group = ComponentGroup::factory()->create(['collapsed' => ComponentGroupVisibilityEnum::collapsed_unless_incident]);
+    Incident::factory()->create(['visible' => ResourceVisibilityEnum::authenticated])->components()->attach(
+        Component::factory()->create(['component_group_id' => $group->id]),
+        ['component_status' => ComponentStatusEnum::major_outage],
+    );
+
+    if ($authenticated) {
+        $this->actingAs(User::factory()->create());
+    }
+
+    $page = $this->get(route('cachet.status-page'))->assertOk()->getContent();
+
+    expect($page)->toMatch(systemStatusGroupPattern($group, open: $authenticated));
+})->with([
+    'guest' => [false],
+    'signed in' => [true],
+]);
+
+function systemStatusGroupPattern(ComponentGroup $group, bool $open): string
+{
+    $tag = '<div data-component="system-status-group" data-component-group-id="'.$group->id.'" x-data x-disclosure';
+
+    return '/'.preg_quote($tag, '/').($open ? '\s+default-open\s*>' : '\s*>').'/';
+}
+
 it('renders stable theme attributes on the status page', function () {
     $settings = app(AppSettings::class);
     $settings->about = 'Service status and uptime.';

From 605a11b16a8452bd3b95314beec8e990930eb194 Mon Sep 17 00:00:00 2001
From: Taylor Downs <taylor@openfn.org>
Date: Fri, 23 Oct 2026 07:38:30 +0200
Subject: [PATCH 3/3] checks

---
 src/View/Components/SystemStatus.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/View/Components/SystemStatus.php b/src/View/Components/SystemStatus.php
index 475abfb6..2a675025 100644
--- a/src/View/Components/SystemStatus.php
+++ b/src/View/Components/SystemStatus.php
@@ -106,7 +106,7 @@ private function componentsWithActiveIncidents(): Collection
         $authenticated = Cachet::user() !== null;
 
         return Component::query()
-            ->whereHas('incidents', fn ($query) => $query->unresolved()->viewableBy($authenticated))
+            ->whereHas('incidents', fn ($query) => $query->whereIn('incidents.id', Incident::query()->unresolved()->viewableBy($authenticated)->select('id')))
             ->pluck('id')
             ->toBase();
     }