fix: resolve build errors and remaining linting issues

Author: MeridianAlgo-Developer

src/backtest/engine.ts

@@ -169,14 +169,18 @@ export class TimeBasedEngine {
 
     // Win rate and profit factor
     const completedTrades = this.trades.filter(t => t.pnl !== undefined);
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     const winningTrades = completedTrades.filter(t => t.pnl! > 0);
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     const losingTrades = completedTrades.filter(t => t.pnl! < 0);
 
     const winRate = completedTrades.length > 0 
       ? winningTrades.length / completedTrades.length 
       : 0;
 
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     const grossProfit = winningTrades.reduce((sum, t) => sum + t.pnl!, 0);
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
     const grossLoss = Math.abs(losingTrades.reduce((sum, t) => sum + t.pnl!, 0));
     const profitFactor = grossLoss > 0 ? grossProfit / grossLoss : 0;
 

src/cli/cli.ts

@@ -20,7 +20,9 @@ program
   .option('--end <date>', 'End date', '2023-12-31')
   .option('--strategy <strategy>', 'Strategy to use', 'trend-following')
   .action(async (options) => {
+    // eslint-disable-next-line no-console
     console.log('Running backtest with options:', options);
+    // eslint-disable-next-line no-console
     console.log('Note: Full implementation requires running example code');
   });
 
@@ -30,7 +32,9 @@ program
   .argument('<name>', 'Project name')
   .option('-t, --template <template>', 'Template to use', 'basic')
   .action((name, options) => {
+    // eslint-disable-next-line no-console
     console.log(`Initializing project: ${name} with template: ${options.template}`);
+    // eslint-disable-next-line no-console
     console.log('Note: Template scaffolding not yet implemented');
   });
 

src/core/logger.ts

@@ -23,24 +23,28 @@ export class ConsoleLogger implements Logger {
 
   debug(message: string, meta?: any): void {
     if (this.shouldLog('debug')) {
+      // eslint-disable-next-line no-console
       console.debug(`[DEBUG] ${message}`, meta || '');
     }
   }
 
   info(message: string, meta?: any): void {
     if (this.shouldLog('info')) {
+      // eslint-disable-next-line no-console
       console.info(`[INFO] ${message}`, meta || '');
     }
   }
 
   warn(message: string, meta?: any): void {
     if (this.shouldLog('warn')) {
+      // eslint-disable-next-line no-console
       console.warn(`[WARN] ${message}`, meta || '');
     }
   }
 
   error(message: string, meta?: any): void {
     if (this.shouldLog('error')) {
+      // eslint-disable-next-line no-console
       console.error(`[ERROR] ${message}`, meta || '');
     }
   }

src/utils/time.ts

@@ -9,13 +9,13 @@ export class TimeUtils {
    * Check if market is open
    * Simplified - assumes US market hours (9:30 AM - 4:00 PM ET)
    */
-  static isMarketOpen(date: Date, exchange: string = 'NYSE'): boolean {
+  static isMarketOpen(date: Date): boolean {
     // Check if weekend
     const day = date.getDay();
     if (day === 0 || day === 6) return false;
 
-    // Check if holiday (simplified - major US holidays)
-    if (this.isHoliday(date, exchange)) return false;
+    // Check if holiday
+    if (this.isHoliday(date)) return false;
 
     // Check market hours (simplified - assumes ET timezone)
     const hours = date.getHours();