From bdec163fb0843653f384a3c569a715243d9199df Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sebastian=20Kr=C3=BCger?= <valknar@pivoine.art>
Date: Sun, 23 Nov 2025 22:58:03 +0100
Subject: [PATCH] refactor: simplify readLog to use standard Supervisor XML-RPC
 API
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Removed SUPERVISOR_LOGFILE environment variable and simplified readLog()
to use the standard 2-parameter XML-RPC API that relies on supervisord's
configured logfile path.

Changes:
- Removed SUPERVISOR_LOGFILE from .env.example
- Simplified SupervisorClient.readLog() to accept only offset and length
- Removed logfile path parameter and all environment variable logic
- Fixed mobile nav container padding (removed px-4 py-6)

The readLog() method now uses the standard supervisor.readLog(offset, length)
XML-RPC call, which automatically reads from the logfile path configured
in supervisord.conf.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 .env.example                 | 4 ----
 components/layout/Navbar.tsx | 2 +-
 lib/supervisor/client.ts     | 7 ++-----
 3 files changed, 3 insertions(+), 10 deletions(-)

diff --git a/.env.example b/.env.example
index 41c8969..749f024 100644
--- a/.env.example
+++ b/.env.example
@@ -11,10 +11,6 @@ SUPERVISOR_PORT=9001
 # SUPERVISOR_USERNAME=user
 # SUPERVISOR_PASSWORD=pass
 
-# Optional: Path to supervisord main logfile (for reading via XML-RPC)
-# Default: /var/log/supervisor/supervisord.log
-# SUPERVISOR_LOGFILE=/var/log/supervisor/supervisord.log
-
 # Logging Configuration
 # Log level: debug, info, warn, error (default: info in prod, debug in dev)
 LOG_LEVEL=info
diff --git a/components/layout/Navbar.tsx b/components/layout/Navbar.tsx
index bb7cb7f..e93b4cb 100644
--- a/components/layout/Navbar.tsx
+++ b/components/layout/Navbar.tsx
@@ -117,7 +117,7 @@ export function Navbar() {
       {/* Mobile Menu Drawer */}
       {mobileMenuOpen && (
         <div className="md:hidden fixed inset-0 top-16 z-50">
-          <div className="container px-4 py-6">
+          <div className="container">
             <nav className="flex flex-col gap-2 bg-background/95 backdrop-blur-md rounded-lg p-4">
               {navItems.map((item) => (
                 <Link key={item.href} href={item.href}>
diff --git a/lib/supervisor/client.ts b/lib/supervisor/client.ts
index 6179aeb..6c737f2 100644
--- a/lib/supervisor/client.ts
+++ b/lib/supervisor/client.ts
@@ -252,11 +252,8 @@ export class SupervisorClient {
     return this.call<ProcessActionResult[]>('supervisor.clearAllProcessLogs');
   }
 
-  async readLog(offset: number, length: number, logfilePath?: string): Promise<string> {
-    // Use provided path, or fall back to env variable, or use default
-    const path = logfilePath || process.env.SUPERVISOR_LOGFILE || '/var/log/supervisor/supervisord.log';
-    this.logger.debug({ path, offset, length }, 'Reading supervisor main log');
-    return this.call<string>('supervisor.readLog', [offset, length, path]);
+  async readLog(offset: number, length: number): Promise<string> {
+    return this.call<string>('supervisor.readLog', [offset, length]);
   }
 
   async clearLog(): Promise<boolean> {