Greptile Overview

Greptile Summary

Refactored logger initialization from global configuration to per-app singleton pattern with lazy initialization. Each app now configures only the logger categories it needs, improving performance and reducing overhead.

• Replaced configureLogger()/configureLoggerAsync() with new buildLogger() function that configures specific categories per app
• Introduced singleton pattern for logger instances to prevent re-initialization across module imports
• Moved from eager global configuration to lazy initialization on first logger use
• Added LoggerCategory.EMAIL and removed unused LoggerCategory.SECURITY
• Fixed hardcoded category string in workflows package to use LoggerCategory.INNGEST constant
• Apps now configure async logging only where needed (API server and app server-side)
• Client-side apps (desktop, extension, mobile, app client) use synchronous logging for immediate console output

Confidence Score: 5/5

• This PR is safe to merge with no concerns
• Clean refactoring that improves architecture by making logger configuration explicit per-app. All changes follow consistent patterns, unused code removed, and hardcoded strings replaced with constants. No logic errors or breaking changes detected.
• No files require special attention

Important Files Changed

Sequence Diagram

sequenceDiagram
 participant App as Application Entry Point
 participant Logger as app/shared/logger.ts
 participant Singleton as @init/utils/singleton
 participant Builder as @init/observability/logger
 participant LogTape as @logtape/logtape
 Note over App,LogTape: Before: Eager Global Initialization
 App->>Builder: configureLogger() / configureLoggerAsync()
 Builder->>LogTape: configure() / configureSync() with all categories
 Note over LogTape: All categories configured globally
 Note over App,LogTape: After: Lazy Per-App Initialization
 App->>Logger: import { logger }
 Logger->>Singleton: singleton("logger:app", buildLogger)
 alt First access
 Singleton->>Builder: buildLogger(categories, options)
 Builder->>Builder: Filter LOGGER_CONFIGS by categories
 Builder->>LogTape: configure() / configureSync() with filtered config
 LogTape-->>Builder: configured
 Builder->>LogTape: getLogger(defaultCategory)
 LogTape-->>Builder: logger instance
 Builder-->>Singleton: logger instance
 Singleton-->>Logger: cached logger
 else Subsequent access
 Singleton-->>Logger: cached logger (from globalThis.__remember_init)
 end
 Logger-->>App: logger instance