wip: clean up

framework/core/src/Admin/Middleware/GatherDebugInformation.php

@@ -23,29 +23,44 @@ public function __construct(protected SettingsRepositoryInterface $settings)
 
     public function process(Request $request, Handler $handler): Response
     {
-        // Read current web user, so we can compare that against CLI executions,
-        // these often cause file permission issues.
+        $this->upsertWebUser();
+
+        $this->upsertOpCacheStatus();
+
+        return $handler->handle($request);
+    }
+
+    /**
+     * Read current web user, so we can compare that against CLI executions,
+     * these often cause file permission issues.
+     */
+    public function upsertWebUser(): void
+    {
         $user = $this->settings->get('core.debug.web_user');
         $currentUser = get_current_user();
+
         if ($user !== $currentUser) {
             $this->settings->set(
                 'core.debug.web_user',
                 $currentUser
             );
         }
+    }
 
-        // Read the opcache situation, this is only visible in web.
-        // Cli has opcache disabled by default.
+    /**
+     * Read the opcache situation, this is only visible in web.
+     * Cli has opcache disabled by default.
+     */
+    public function upsertOpCacheStatus(): void
+    {
         $opcache = $this->settings->get('core.debug.opcache');
-        $opcacheStatus = function_exists('opcache_get_configuration')
-            && opcache_get_configuration() !== false ? 'on' : 'off';
+        $opcacheStatus = function_exists('opcache_get_configuration') && opcache_get_configuration() !== false ? 'on' : 'off';
+
         if ($opcache !== $opcacheStatus) {
             $this->settings->set(
                 'core.debug.opcache',
                 $opcacheStatus
             );
         }
-
-        return $handler->handle($request);
     }
 }

framework/core/src/Foundation/Info/Renderer/CliRenderer.php

@@ -22,7 +22,7 @@ public function __construct(protected OutputInterface $output)
 
     public function heading(string $title): void
     {
-        $this->output->writeln("<bg=gray,options=bold>$title</>");
+        $this->output->writeln("<bg=gray>$title</>");
     }
 
     public function keyValue(string $key, mixed $value): void

framework/core/src/Foundation/Info/Report.php

@@ -19,8 +19,8 @@ class Report
         Section\Database::class,
         Section\EnabledExtensions::class,
         Section\Features::class,
-        Section\Webserver::class,
         Section\Debug::class,
+        Section\ServiceUsers::class,
     ];
 
     public function __construct(

framework/core/src/Foundation/Info/Section/Webserver.php → framework/core/src/Foundation/Info/Section/ServiceUsers.php

@@ -13,17 +13,28 @@
 use Flarum\Foundation\Info\SectionInterface;
 use Flarum\Settings\SettingsRepositoryInterface;
 
-class Webserver implements SectionInterface
+class ServiceUsers implements SectionInterface
 {
     public function __construct(protected SettingsRepositoryInterface $settings)
     {
     }
 
     public function __invoke(RendererInterface $renderer): void
     {
-        $renderer->keyValue(
-            'Web user',
-            $this->settings->get('core.debug.web_user') ?? 'visit admin to identify'
-        );
+        $rows = [
+            ['Web user', $this->settings->get('core.debug.web_user') ?? 'visit admin to identify']
+        ];
+
+        if (php_sapi_name() === 'cli') {
+            $rows[] = [
+                'Current user',
+                posix_getpwuid(posix_geteuid())['name']
+            ];
+        }
+
+        $renderer->table([
+            ['Service/process users'],
+            ['Type', 'User']
+        ], $rows);
     }
 }