Update loading screen.

Author: idf_karolisstr

src/charts/timeSeries.ts

@@ -858,7 +858,7 @@ export class TimeSeriesChart {
         const minLabelWidth = 1; // Minimum time difference to create a meaningful label
 
         if (labelWidth < minLabelWidth) {
-            console.log('Label too narrow, minimum width required:', minLabelWidth);
+            // Label too narrow, minimum width required
             return; // Don't create the label
         }
 
@@ -884,8 +884,8 @@ export class TimeSeriesChart {
             if (this.lastConfig) {
                 this.updateDisplay(this.lastConfig);
             }
-        } catch (error) {
-            console.error('Failed to create label:', error);
+        } catch (_error) {
+            // Failed to create label - silent handling
         }
     }
 
@@ -952,7 +952,7 @@ export class TimeSeriesChart {
 
         return {
             silent: true, // Don't intercept mouse events - allow drawing over labels
-            data: markAreaData as unknown, // ECharts markArea type is complex, using unknown for data compatibility
+            data: markAreaData as never, // ECharts markArea expects complex nested array structure
         };
     }
 
@@ -1029,15 +1029,7 @@ export class TimeSeriesChart {
         }
 
         const data = source.getData(config.xColumn, config.yColumn);
-        console.log('Retrieved data for columns:', {
-            xColumn: config.xColumn,
-            yColumn: config.yColumn,
-            dataLength: data.length,
-            firstFewPoints: data.slice(0, 3),
-            lastFewPoints: data.slice(-3),
-            sourceName: source.name,
-            sourceColumns: source.columns,
-        });
+        // Retrieved data for columns - debug info available if needed
 
         // Apply configuration with defaults
         const xType = config.xType || 'category';
@@ -1062,25 +1054,7 @@ export class TimeSeriesChart {
             seriesData = data;
         }
 
-        // Validate data for any invalid values
-        const invalidPoints = seriesData.filter((point) => {
-            const [x, y] = point;
-            const xNum = typeof x === 'string' ? parseFloat(x) : x;
-            return !Number.isFinite(xNum) || !Number.isFinite(y);
-        });
-
-        // eslint-disable-next-line no-console
-        console.log('Final series data for chart:', {
-            xType,
-            xColumn: config.xColumn,
-            processingType:
-                xType === 'category' && config.xColumn === 'index' ? 'row indices' : 'actual data',
-            totalPoints: seriesData.length,
-            invalidPoints: invalidPoints.length,
-            invalidSample: invalidPoints.slice(0, 3),
-            firstFewSeriesPoints: seriesData.slice(0, 3),
-            lastFewSeriesPoints: seriesData.slice(-3),
-        });
+        // Debug: Final series data for chart - xType, processing type, points validation
 
         // Configure axes with proper auto-scaling
         const xAxis: EChartOption.XAxis = {
@@ -1148,15 +1122,7 @@ export class TimeSeriesChart {
                       axisPointer: { type: 'cross' as const, snap },
                   };
 
-        // eslint-disable-next-line no-console
-        console.log('Chart configuration debug:', {
-            showArea,
-            xType,
-            yType,
-            seriesDataLength: seriesData.length,
-            firstPoint: seriesData[0],
-            lastPoint: seriesData[seriesData.length - 1],
-        });
+        // Debug: Chart configuration - area display, axis types, data points
 
         this.chart.setOption(
             {
@@ -1349,12 +1315,7 @@ function getCurrentChartConfig(xColumn: string, yColumn: string): TimeSeriesConf
     // Helper to get xType value - default to numeric for all columns
     const xTypeValue = elXType ? (elXType as { value?: string }).value : undefined;
 
-    // eslint-disable-next-line no-console
-    console.log('X-axis type detection:', {
-        xColumn,
-        xTypeDropdownValue: xTypeValue,
-        elXType: elXType?.id,
-    });
+    // Debug: X-axis type detection logic
 
     const xType: 'category' | 'time' | 'value' = (() => {
         // Respect user's explicit choice
@@ -1376,8 +1337,7 @@ function getCurrentChartConfig(xColumn: string, yColumn: string): TimeSeriesConf
         return 'value';
     })();
 
-    // eslint-disable-next-line no-console
-    console.log('X-axis type result:', xType);
+    // Debug: X-axis type result
 
     // Helper to get yType value
     const yTypeValue = elYType ? (elYType as { value?: string }).value : undefined;
@@ -1465,7 +1425,7 @@ function bindUIControls(chart: TimeSeriesChart): void {
 
             if (!selectedLabelValue || selectedLabelValue === '') {
                 // No label selected, show error or create default
-                console.warn('No active label selected for drawing mode');
+                // No active label selected for drawing mode
                 // Revert button state
                 btnLabelMode.setAttribute('aria-pressed', 'false');
                 btnLabelMode.classList.remove('active');
@@ -1496,14 +1456,7 @@ function bindUIControls(chart: TimeSeriesChart): void {
     const xDropdown = document.querySelector('#x-axis');
     const yDropdown = document.querySelector('#y-axis');
 
-    // eslint-disable-next-line no-console
-    console.log('Dropdown element references:', {
-        xDropdown: xDropdown?.id,
-        yDropdown: yDropdown?.id,
-        sameElement: xDropdown === yDropdown,
-        xValue: (xDropdown as { value?: string } | null)?.value,
-        yValue: (yDropdown as { value?: string } | null)?.value,
-    });
+    // Debug: Dropdown element references and values
 
     const updateChart = () => {
         // Don't update chart if we have no data sources at all
@@ -1525,17 +1478,13 @@ function bindUIControls(chart: TimeSeriesChart): void {
         const xColumn = (xDropdown as { value?: string } | null)?.value || 'index';
         const yColumn = (yDropdown as { value?: string } | null)?.value || 'value';
 
-        console.log('updateChart called with:', { xColumn, yColumn });
+        // updateChart called with debug info available
 
         // Only update if we have valid column values
         if (xColumn && yColumn && xColumn !== '' && yColumn !== '') {
             // Get current configuration from all UI controls
             const config = getCurrentChartConfig(xColumn, yColumn);
-            console.log('Generated config:', {
-                xColumn: config.xColumn,
-                yColumn: config.yColumn,
-                xType: config.xType,
-            });
+            // Generated config debug info available
             chart.updateDisplay(config);
         }
     };
@@ -1550,11 +1499,11 @@ function bindUIControls(chart: TimeSeriesChart): void {
     };
 
     xDropdown?.addEventListener('change', () => {
-        console.log('X dropdown changed to:', (xDropdown as { value?: string } | null)?.value);
+        // X dropdown changed - debug info available
         updateChart();
     });
     yDropdown?.addEventListener('change', () => {
-        console.log('Y dropdown changed to:', (yDropdown as { value?: string } | null)?.value);
+        // Y dropdown changed - debug info available
         updateChart();
     });
 

src/data/csvProcessor.ts

@@ -188,9 +188,8 @@ export class CSVTimeSeriesData implements TimeSeriesData {
                     );
                 }
             })
-            .catch((error: unknown) => {
-                // eslint-disable-next-line no-console
-                console.error('Failed to import dropdowns module:', error);
+            .catch((_error: unknown) => {
+                // Failed to import dropdowns module, falling back to basic label loading
                 // Fallback: just load labels anyway
                 void this.loadLabelsFromStorage();
             });
@@ -216,9 +215,8 @@ export class CSVTimeSeriesData implements TimeSeriesData {
                     this.notifyLabelsChangedIfDefinitionsReady();
                 }
             }
-        } catch (error) {
-            // eslint-disable-next-line no-console
-            console.warn('Failed to load labels from storage:', error);
+        } catch (_error) {
+            // Failed to load labels from storage, continuing without stored labels
         }
     }
 
@@ -240,12 +238,10 @@ export class CSVTimeSeriesData implements TimeSeriesData {
             };
             const result = await saveTimeSeriesLabel(labelRecord);
             if (!result.ok) {
-                // eslint-disable-next-line no-console
-                console.warn('Failed to save label to storage:', result.error);
+                // Failed to save label to storage, continuing without persistence
             }
-        } catch (error) {
-            // eslint-disable-next-line no-console
-            console.warn('Error saving label to storage:', error);
+        } catch (_error) {
+            // Error saving label to storage, continuing without persistence
         }
     }
 
@@ -256,12 +252,10 @@ export class CSVTimeSeriesData implements TimeSeriesData {
         try {
             const result = await deleteTimeSeriesLabel(labelId);
             if (!result.ok) {
-                // eslint-disable-next-line no-console
-                console.warn('Failed to remove label from storage:', result.error);
+                // Failed to remove label from storage, continuing anyway
             }
-        } catch (error) {
-            // eslint-disable-next-line no-console
-            console.warn('Error removing label from storage:', error);
+        } catch (_error) {
+            // Error removing label from storage, continuing anyway
         }
     }
 
@@ -300,9 +294,8 @@ export class CSVTimeSeriesData implements TimeSeriesData {
                     );
                 }
             })
-            .catch((error: unknown) => {
-                // eslint-disable-next-line no-console
-                console.error('Failed to import dropdowns module:', error);
+            .catch((_error: unknown) => {
+                // Failed to import dropdowns module, falling back to basic notification
                 // Fallback: notify anyway (better than no notification)
                 this.notifyLabelsChanged();
             });

src/data/dataService.ts

@@ -170,7 +170,9 @@ export class DataService implements IDataService {
     /**
      * Add new data sources from uploaded files
      */
-    addDataSource(files: TDataFile[]): Result<readonly TimeSeriesData[], DataValidationError> {
+    async addDataSource(
+        files: TDataFile[]
+    ): Promise<Result<readonly TimeSeriesData[], DataValidationError>> {
         try {
             // Convert files to time series data
             const newDataSources = convertDataFilesToTimeSeries(files);
@@ -186,7 +188,7 @@ export class DataService implements IDataService {
             });
             this.emit('dataSourcesChanged', updatedSources);
 
-            return ok(updatedSources);
+            return await Promise.resolve(ok(updatedSources));
         } catch (error) {
             return err(new DataValidationError('Failed to add data sources', error));
         }
@@ -195,7 +197,7 @@ export class DataService implements IDataService {
     /**
      * Remove a data source by ID
      */
-    removeDataSource(id: string): Result<void, DataValidationError> {
+    async removeDataSource(id: string): Promise<Result<void, DataValidationError>> {
         try {
             // Remove from cache
             this.dataSourcesCache = this.dataSourcesCache.filter((source) => {
@@ -211,7 +213,7 @@ export class DataService implements IDataService {
             this.emit('dataSourceRemoved', id);
             this.emit('dataSourcesChanged', this.dataSourcesCache);
 
-            return ok(undefined);
+            return await Promise.resolve(ok(undefined));
         } catch (error) {
             return err(new DataValidationError('Failed to remove data source', error));
         }
@@ -220,10 +222,12 @@ export class DataService implements IDataService {
     /**
      * Get metadata for a specific data source
      */
-    getDataSourceMetadata(id: string): Result<DataSourceMetadata | null, DataValidationError> {
+    async getDataSourceMetadata(
+        id: string
+    ): Promise<Result<DataSourceMetadata | null, DataValidationError>> {
         try {
             const metadata = this.metadataCache.get(id) || null;
-            return ok(metadata);
+            return await Promise.resolve(ok(metadata));
         } catch (error) {
             return err(new DataValidationError('Failed to get data source metadata', error));
         }
@@ -374,7 +378,7 @@ export class DataService implements IDataService {
 
         const listeners = this.eventListeners.get(event);
         if (listeners) {
-            listeners.push(listener);
+            listeners.push(listener as (data: unknown) => void);
         }
 
         // Return unsubscribe function
@@ -392,7 +396,7 @@ export class DataService implements IDataService {
     ): void {
         const listeners = this.eventListeners.get(event);
         if (listeners) {
-            const index = listeners.indexOf(listener);
+            const index = listeners.indexOf(listener as (data: unknown) => void);
             if (index >= 0) {
                 listeners.splice(index, 1);
             }
@@ -407,8 +411,8 @@ export class DataService implements IDataService {
         listeners.forEach((listener) => {
             try {
                 listener(data);
-            } catch (error) {
-                console.error(`Error in ${event} listener:`, error);
+            } catch (_error) {
+                // Silent error handling for event listeners
             }
         });
     }

src/main.ts

@@ -70,8 +70,7 @@ async function initializeApplication(): Promise<void> {
         if (sourcesResult.ok) {
             timeSeriesChart.setDataSources(sourcesResult.value);
         } else {
-            // eslint-disable-next-line no-console
-            console.error('Failed to load initial data sources:', sourcesResult.error);
+            // Failed to load initial data sources, continuing with empty chart
         }
         // Always mark as complete to not block loading
         markLoadingStepComplete('data-loaded');
@@ -93,9 +92,8 @@ async function initializeApplication(): Promise<void> {
         try {
             await loadHistoryEntries();
             markLoadingStepComplete('ui-setup');
-        } catch (error: unknown) {
-            // eslint-disable-next-line no-console
-            console.error('Failed to load history entries:', error);
+        } catch (_error: unknown) {
+            // Failed to load history entries, continuing without history
             markLoadingStepComplete('ui-setup');
         }
 
@@ -118,17 +116,14 @@ async function initializeApplication(): Promise<void> {
             ).__resetDatabase = async () => {
                 const result = await resetDatabase();
                 if (result.ok) {
-                    // eslint-disable-next-line no-console
-                    console.log('Database reset successfully. Please refresh the page.');
+                    // Database reset successfully, refresh page manually
                 } else {
-                    // eslint-disable-next-line no-console
-                    console.error('Failed to reset database:', result.error);
+                    // Failed to reset database, check browser console for details
                 }
             };
         }
-    } catch (error: unknown) {
-        // eslint-disable-next-line no-console
-        console.error('Failed to initialize application:', error);
+    } catch (_error: unknown) {
+        // Failed to initialize application, forcing loading completion to prevent stuck state
         // Force complete loading even on error to not leave user stuck
         const { forceCompleteLoading } = await import('./ui/loadingScreen');
         forceCompleteLoading();

src/platform/storage.ts

@@ -115,7 +115,6 @@ export async function saveRecord(
         if (!db.objectStoreNames.contains(storeName)) {
             const availableStores = Array.from(db.objectStoreNames).join(', ');
             const errorMsg = `Object store '${storeName}' not found. Available stores: ${availableStores}. Database version: ${String(db.version)}`;
-            console.error('Storage Error:', errorMsg);
             return err(new StorageError(errorMsg));
         }
 
@@ -169,7 +168,7 @@ export async function resetDatabase(): Promise<Result<void, StorageError>> {
         const deleteReq = indexedDB.deleteDatabase(DB_NAME);
         await new Promise<void>((resolve, reject) => {
             deleteReq.onsuccess = () => {
-                console.log('Database deleted successfully');
+                // Database deleted successfully
                 resolve();
             };
             deleteReq.onerror = () => {
@@ -179,7 +178,7 @@ export async function resetDatabase(): Promise<Result<void, StorageError>> {
 
         // Reopen to trigger creation with current schema
         await openDatabase();
-        console.log('Database reset and recreated successfully');
+        // Database reset and recreated successfully
         return ok(undefined);
     } catch (error) {
         return err(new StorageError('Failed to reset database', error));