LLM/BrowserOS
1
0
Fork 0
mirror of https://github.com/browseros-ai/BrowserOS.git synced 2026-05-18 11:06:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: pino thread stream doesn't work with bun compile

Browse Source
This commit is contained in:
Nikhil Sonti
2026-01-05 17:36:52 -08:00
parent 79944037bb
commit 980c10a411
1 changed files with 14 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
apps/server/src/common/logger.ts
View File

@@ -95,8 +95,9 @@ function rotateLogIfNeeded(logPath: string): void {
/**
* Create pino transport configuration for console output.
* Returns null for production (use sync stdout to avoid thread-stream issues with Bun compile).
*/
function createConsoleTransport(): pino.TransportSingleOptions {
function createConsoleTransport(): pino.TransportSingleOptions | null {
if (isDev) {
return {
target: 'pino-pretty',
@@ -108,10 +109,9 @@ function createConsoleTransport(): pino.TransportSingleOptions {
}
}
return {
target: 'pino/file',
options: { destination: 1 }, // stdout
}
// Production: return null to use synchronous stdout logging.
// pino.transport() uses thread-stream which doesn't work with Bun compile.
return null
}
export class Logger implements LoggerInterface {
@@ -137,7 +137,15 @@ export class Logger implements LoggerInterface {
}
}
return pino(options, pino.transport(createConsoleTransport()))
const transport = createConsoleTransport()
if (transport) {
return pino(options, pino.transport(transport))
}
// Production: use pino.destination() for async writes without worker threads.
// pino.transport() uses thread-stream which fails with Bun compile.
// pino.destination() uses SonicBoom directly - no workers, bundling-safe.
return pino(options, pino.destination({ dest: 1, sync: false }))
}
/**