feat: migrate logging system from custom transports to Winston

## Summary
Migrated Levante's custom logging system (420 lines) to Winston logger while maintaining 100% backward compatibility and zero overhead for disabled categories.

## Key Changes

### New Files
- `src/main/services/logging/winstonConfig.ts` - Winston transport configuration
- `src/main/services/logging/timezoneFormat.ts` - Timezone support (extracted from transports.ts)
- `docs/testing/winston-migration-testing.md` - Comprehensive testing guide
- `scripts/test-winston-logging.ts` - Automated testing script

### Modified Files
- `src/main/services/logging/logger.ts` - Replaced custom transports with Winston logger
- `src/main/services/logging/config.ts` - Added production/development defaults
- `src/main/services/logging/index.ts` - Updated exports
- `package.json` - Added winston-daily-rotate-file dependency

### Removed Files
- `src/main/services/logging/transports.ts` - Replaced by Winston transports (420 lines removed)

## Features

### Production vs Development
**Development** (NODE_ENV=development):
- Console output with colors
- Debug level logging
- Verbose categories enabled (ai-sdk, mcp, database, oauth)
- Single file: levante-YYYY-MM-DD-HHmmss.log
- Rotation: 10MB, 3 files, 7 days

**Production** (NODE_ENV=production):
- No console output (file only)
- Warn level logging
- Verbose categories disabled
- Two files: all logs + error-only log
- Rotation: 50MB, 10 files, 30 days
- Gzip compression enabled

### Preserved Features
✅ Zero overhead for disabled categories
✅ 9 categories (ai-sdk, mcp, database, ipc, preferences, models, core, analytics, oauth)
✅ Environment variable configuration
✅ Timezone support
✅ IPC bridge (renderer → main)
✅ Same public API (no breaking changes)

### New Capabilities
✨ Structured JSON logs (production)
✨ Separate error log file (production)
✨ Battle-tested Winston transports
✨ Automatic gzip compression
✨ Winston audit files for rotation tracking
✨ Foundation for future log viewer UI

## Code Impact
- Lines added: ~350
- Lines removed: ~420
- Net reduction: ~70 lines
- Breaking changes: 0

## Documentation
- Updated `docs/LOGGING.md` with "Arquitectura Winston" section
- Updated `CLAUDE.md` logging section
- Added comprehensive testing guide

## Testing
- ✅ Typecheck passes
- ✅ Zero overhead verified
- ✅ Backward compatible API
- Manual testing required per testing guide

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: 2 people

CLAUDE.md

@@ -285,50 +285,20 @@ Environment variables loaded from:
 
 ## Logging System
 
-Levante uses a centralized logging system for better development experience and debugging:
+Levante utiliza **Winston** para logging centralizado con sistema de categorías:
 
-### Usage
+- **Categorías**: ai-sdk, mcp, database, ipc, preferences, models, core, analytics, oauth
+- **Niveles**: debug, info, warn, error
+- **Producción**: Logs JSON estructurados con archivo de errores separado
+- **Desarrollo**: Salida coloreada legible en consola
+- **Zero Overhead**: Categorías deshabilitadas no ejecutan código de log
+- **Rotación**: Automática con winston-daily-rotate-file
 
-```typescript
-// Main Process
-import { getLogger } from './services/logging';
-const logger = getLogger();
-
-// Renderer Process  
-import { logger } from '@/services/logger';
-
-// Usage examples
-logger.aiSdk.debug('Model provider loaded', { provider: 'openai' });
-logger.mcp.info('Server started', { serverId: 'filesystem' });
-logger.database.error('Migration failed', { error: error.message });
-logger.core.info('Application initialized');
-```
-
-### Categories
-
-- **ai-sdk**: AI service operations, model interactions, streaming
-- **mcp**: MCP server management, tool execution, health monitoring  
-- **database**: Database operations and migrations
-- **ipc**: Inter-process communication
-- **preferences**: Settings and configuration management
-- **core**: General application lifecycle and errors
-
-### Configuration
-
-Control logging via `.env.local`:
-
-```bash
-DEBUG_ENABLED=true
-DEBUG_AI_SDK=true
-DEBUG_MCP=true
-DEBUG_DATABASE=false
-DEBUG_IPC=false
-DEBUG_PREFERENCES=false
-DEBUG_CORE=true
-LOG_LEVEL=debug
-```
+**Configuración por Entorno**:
+- `NODE_ENV=development`: Verbose, consola, rotación pequeña
+- `NODE_ENV=production`: Minimal, solo archivo, rotación grande
 
-See [docs/LOGGING.md](docs/LOGGING.md) for complete documentation.
+Ver [docs/LOGGING.md](docs/LOGGING.md) para documentación completa.
 - No utilices pnpm dev
 
 ## Developer Documentation

docs/LOGGING.md

@@ -110,21 +110,147 @@ Always include relevant context for better debugging:
 
 ```typescript
 // Good
-logger.aiSdk.debug('Model provider loaded', { 
-  provider: 'openai', 
-  models: 15, 
-  loadTime: '45ms' 
+logger.aiSdk.debug('Model provider loaded', {
+  provider: 'openai',
+  models: 15,
+  loadTime: '45ms'
 });
 
 // Better
-logger.mcp.error('Server connection failed', { 
+logger.mcp.error('Server connection failed', {
   serverId: 'filesystem',
   error: error.message,
   attempt: 3,
   timestamp: Date.now()
 });
 ```
 
+## Arquitectura Winston
+
+Levante usa Winston como sistema de logging con las siguientes características:
+
+### Producción vs Desarrollo
+
+El comportamiento del logger cambia según `NODE_ENV`:
+
+**Desarrollo** (`NODE_ENV=development` o no configurado):
+- Salida a consola con colores
+- Log level: `debug` (muy verboso)
+- Categorías verbose habilitadas por defecto: ai-sdk, mcp, database, oauth
+- Archivo único: `levante-YYYY-MM-DD-HHmmss.log` (datePattern configurable)
+- Rotación: 10MB, 3 archivos, 7 días
+
+**Producción** (`NODE_ENV=production`):
+- Sin salida a consola (solo archivo)
+- Log level: `warn` (solo warnings y errores)
+- Categorías verbose deshabilitadas
+- Dos archivos:
+  - Todos: `levante-YYYY-MM-DD-HHmmss.log`
+  - Errores: `levante-error-YYYY-MM-DD-HHmmss.log`
+- Rotación: 50MB, 10 archivos, 30 días (errores: 90 días)
+- Compresión gzip habilitada por defecto
+
+### Archivos de Log
+
+**Ubicación**: `~/levante/`
+
+**Desarrollo**:
+```
+~/levante/
+├── levante-2025-01-28-143025.log    # Con datePattern YYYY-MM-DD-HHmmss
+├── levante-2025-01-28-120000.log    # Rotado anteriormente
+└── .winston-audit.json               # Tracking de Winston
+```
+
+**Producción**:
+```
+~/levante/
+├── levante-2025-01-28-143025.log        # Todos los logs
+├── levante-error-2025-01-28-143025.log  # Solo errores
+├── levante-2025-01-27-120000.log.gz     # Comprimidos (del día anterior)
+├── .winston-audit.json
+└── .winston-error-audit.json
+```
+
+### Sistema de Rotación
+
+Winston's `winston-daily-rotate-file` maneja:
+- **Rotación temporal**: Según datePattern configurado (soporta precisión de segundos con moment.js)
+- **Límite de tamaño**: Máximo por archivo antes de comprimir
+- **Compresión automática**: Gzip en producción
+- **Limpieza por edad**: Elimina logs antiguos según maxAge
+- **Tracking**: `.winston-audit.json` para gestión de archivos
+
+### Formato de Logs
+
+**Desarrollo** (legible, con colores):
+```
+[2025-01-28 14:30:25] [AI-SDK] [DEBUG] Model provider loaded
+{
+  "provider": "openai",
+  "models": 15
+}
+```
+
+**Producción** (JSON estructurado):
+```json
+{
+  "timestamp": "2025-01-28T14:30:25.123Z",
+  "level": "error",
+  "category": "ai-sdk",
+  "message": "Model provider failed",
+  "provider": "openai",
+  "error": "API key invalid"
+}
+```
+
+### Winston Transports
+
+Levante usa los siguientes transports de Winston:
+
+1. **Console Transport** (solo desarrollo):
+   - Formato legible con colores ANSI
+   - Errores a stderr, resto a stdout
+
+2. **DailyRotateFile Transport** (siempre):
+   - Todos los logs (nivel configurado)
+   - Rotación automática
+
+3. **DailyRotateFile Transport** (solo producción):
+   - Solo errores (level: error)
+   - Retención más larga (3x maxAge)
+
+### Variables de Entorno
+
+Las variables de entorno pueden override los defaults de producción/desarrollo:
+
+```bash
+# Forzar level específico (override defaults)
+LOG_LEVEL=debug
+
+# Forzar console en producción
+DEBUG_ENABLED=true
+
+# Rotación personalizada
+LOG_MAX_SIZE=20971520     # 20MB
+LOG_MAX_FILES=5
+LOG_MAX_AGE=14            # 14 días
+LOG_COMPRESS=true
+```
+
+### Migración desde Sistema Custom
+
+El sistema Winston mantiene compatibilidad 100% con el API anterior:
+- Mismo API de logger: `logger.aiSdk.debug(...)`
+- Mismas variables de entorno
+- Mismo formato de LogRotationConfig
+- Mismo comportamiento de timezone
+
+**Cambios internos** (no afectan uso):
+- Transports custom reemplazados por winston transports
+- Rotación manejada por winston-daily-rotate-file (usa moment.js, soporta todos los formatos de fecha incluyendo segundos)
+- Mismo date pattern que el sistema custom (`YYYY-MM-DD-HHmmss`)
+
 ## Log Format
 
 Logs are formatted with timestamps, colored categories, and structured context: