Skip to content

Commit 1f61a0c

Browse files
committed
feat: enhance logging visibility in System Messages
- Backend HTTP logs now prominently displayed in System Messages with [BACKEND] prefix - Client errors now routed to System Messages with [CLIENT] prefix - Improved visibility of all server and client logs in preview panel console - System Messages console now shows all terminal activity in addition to specific terminal views
1 parent 40029db commit 1f61a0c

File tree

2 files changed

+17
-6
lines changed

2 files changed

+17
-6
lines changed

src/ipc/handlers/app_handlers.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1170,15 +1170,15 @@ export function registerAppHandlers() {
11701170

11711171
// Handle client-side errors from frontend
11721172
handle("log-client-error", async (event, { appId, error, context }) => {
1173-
const errorMessage = `Frontend Error: ${error.message || error}\n${context || ''}`.trim();
1173+
const errorMessage = `Frontend Error: ${error.message || error}${context ? `\nContext: ${context}` : ''}`.trim();
11741174

11751175
// Log to system console
11761176
logger.error(`Client error from app ${appId}:`, error);
11771177

1178-
// Route to frontend terminal output
1178+
// Route to both frontend terminal and system messages for visibility
11791179
safeSend(event.sender, "app:output", {
11801180
type: "client-error",
1181-
message: errorMessage,
1181+
message: `[CLIENT] ${errorMessage}`,
11821182
appId,
11831183
});
11841184
});

src/ipc/handlers/terminal_handlers.ts

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -104,10 +104,21 @@ export function routeTerminalOutput(event: Electron.IpcMainInvokeEvent, appId: n
104104
addTerminalOutput(appId, terminal, message, terminalOutputType);
105105
}
106106

107-
// Also send to app:output for backward compatibility and UI display
107+
// Enhanced system message routing - ensure important server logs are visible in System Messages
108+
// For backend server logs (like HTTP requests), make them more prominent in system messages
109+
let systemMessageType = type;
110+
let systemMessage = message;
111+
112+
// For backend server logs, enhance visibility
113+
if (terminalType === "backend" && (message.includes("HTTP/") || message.includes("OPTIONS") || message.includes("GET") || message.includes("POST") || message.includes("PUT") || message.includes("DELETE"))) {
114+
systemMessageType = "info"; // Use info type to make server logs stand out
115+
systemMessage = `[${terminalType.toUpperCase()}] ${message}`;
116+
}
117+
118+
// Always send to system messages for visibility
108119
safeSend(event.sender, "app:output", {
109-
type,
110-
message,
120+
type: systemMessageType,
121+
message: systemMessage,
111122
appId,
112123
});
113124
}

0 commit comments

Comments
 (0)