@smartnet360/svelte-components 0.0.103 → 0.0.104

package/dist/apps/site-check/SiteCheck.svelte

@@ -5,7 +5,6 @@
 	import { ChartComponent,  type Layout, type CellStylingConfig } from '../../core/Charts';
 	import { buildTreeNodes, filterChartData, transformChartData, type CellTrafficRecord, defaultCellStyling, type TreeGroupingConfig, type TreeGroupField, type ColorDimension, defaultTreeGrouping } from './index';
 	import { expandLayoutForCells } from './helper';
-	import { log } from '../../core/logger';
     import type {ChartMarker, Mode } from '../../index.js';
 	import { checkHealth } from '../../core/FeatureRegistry';
 	import SiteCheckControls from './SiteCheckControls.svelte';
@@ -52,24 +51,16 @@
 
 	// Rebuild tree whenever treeGrouping, singleRootSelect, or singleLevel1Select changes
 	$effect(() => {
-
-		log('🔄 Rebuilding tree with grouping', { treeGrouping, singleRootSelect, singleLevel1Select });
-
 		// Clear any existing localStorage data to prevent stale state
 		// This includes both tree state AND chart settings to avoid cell mismatches
 		if (typeof window !== 'undefined') {
 			localStorage.removeItem('site-check:treeState');
 			localStorage.removeItem('charts:globalControls');
-			log('🧹 Cleared localStorage: tree state and chart settings');
 		}
 
 		// Build tree nodes from raw data with custom grouping
 		const treeNodes = buildTreeNodes(rawData, treeGrouping);
-		log('🌲 Tree Nodes Built', {
-			nodeCount: treeNodes.length,
-			firstNode: treeNodes[0],
-			grouping: treeGrouping
-		});
+		
 		if(!isHealthy) {
 			return;
 		}
@@ -82,55 +73,24 @@
 			singleRootSelect, // Pass single root select mode
 			singleLevel1Select // Pass single Level 1 select mode
 		});
-		log('✅ Tree Store Created', { 
-			namespace: 'site-check',
-			grouping: treeGrouping,
-			singleRootSelect,
-			singleLevel1Select
-		});
 	});
 
 	// Derive chart data from tree selection
 	let filteredData = $derived.by(() => {
 		if (!treeStore) return [];
-		const filtered = filterChartData(rawData, treeStore.state.checkedPaths);
-		log('🔍 Filtered Data:', {
-			totalRaw: rawData.length,
-			checkedPaths: Array.from(treeStore.state.checkedPaths),
-			filteredCount: filtered.length,
-			cells: Array.from(new Set(filtered.map(r => r.cellName)))
-		});
-		return filtered;
+		return filterChartData(rawData, treeStore.state.checkedPaths);
 	});
 
 	// Transform data using base metrics from layout
 	let chartData = $derived.by(() => {
-		const transformed = transformChartData(filteredData, baseMetrics);
-		log('📊 Chart Data:', {
-			filteredRows: filteredData.length,
-			transformedRows: transformed.length,
-			baseMetrics,
-			sampleRow: transformed[0],
-			columns: transformed[0] ? Object.keys(transformed[0]) : []
-		});
-		return transformed;
+		return transformChartData(filteredData, baseMetrics);
 	});
 
 	// Expand layout based on selected cells and chosen base layout
 	let chartLayout = $derived.by(() => {
 		// Pass cellStyling, treeGrouping, colorDimension, and useSectorLineStyles - helper will decide per-section whether to use styling,
 		// and generate appropriate labels based on grouping, colors based on colorDimension, and line styles based on useSectorLineStyles
-		const expanded = expandLayoutForCells(selectedBaseLayout, filteredData, treeGrouping, colorDimension, useSectorLineStyles, cellStyling);
-		log('📐 Chart Layout:', {
-			layoutName: selectedBaseLayout.layoutName,
-			layoutDefaultColors: selectedBaseLayout.useDefaultChartColors ?? false,
-			sectionsCount: expanded.sections.length,
-			totalCharts: expanded.sections.reduce((sum, s) => sum + s.charts.length, 0),
-			firstSection: expanded.sections[0],
-			grouping: treeGrouping,
-			colorDimension
-		});
-		return expanded;
+		return expandLayoutForCells(selectedBaseLayout, filteredData, treeGrouping, colorDimension, useSectorLineStyles, cellStyling);
 	});
 
 	let totalRecords = $derived(rawData.length);
@@ -138,14 +98,10 @@
 
 	// Compute simple stats
 	let totalCells = $derived.by(() => {
-		const count = new Set(filteredData.map((r) => r.cellName)).size;
-		log('📱 Total Cells:', count);
-		return count;
+		return new Set(filteredData.map((r) => r.cellName)).size;
 	});
 	let totalSites = $derived.by(() => {
-		const count = new Set(filteredData.map((r) => r.siteName)).size;
-		log('📡 Total Sites:', count);
-		return count;
+		return new Set(filteredData.map((r) => r.siteName)).size;
 	});
 
 	// Detect cell technology (LTE vs NR) for single-cell layout selection
@@ -155,45 +111,23 @@
 		const cell = filteredData[0];
 		const band = cell?.band?.toUpperCase() || '';
 		
-		if (band.startsWith('LTE')) {
-			log('📡 Detected Technology: LTE', { band });
-			return 'LTE';
-		}
-		if (band.startsWith('NR') || band.startsWith('5G')) {
-			log('📡 Detected Technology: NR', { band });
-			return 'NR';
-		}
-		
-		log('📡 Detected Technology: UNKNOWN', { band });
+		if (band.startsWith('LTE')) return 'LTE';
+		if (band.startsWith('NR') || band.startsWith('5G')) return 'NR';
 		return 'UNKNOWN';
 	});
 
 	// Select appropriate layout based on cell count and technology
 	let selectedBaseLayout = $derived.by(() => {
 		// Multiple cells → always use multi-cell layout
-		if (totalCells !== 1) {
-			log('📐 Layout Selection: Multi-cell (count=' + totalCells + ')');
-			return multiCellLayout;
-		}
+		if (totalCells !== 1) return multiCellLayout;
 		
 		// Single LTE cell → use LTE layout if available, otherwise fallback
-		if (cellTechnology === 'LTE' && singleLteLayout) {
-			log('📐 Layout Selection: Single LTE (optimized)');
-			return singleLteLayout;
-		}
+		if (cellTechnology === 'LTE' && singleLteLayout) return singleLteLayout;
 		
 		// Single NR cell → use NR layout if available, otherwise fallback
-		if (cellTechnology === 'NR' && singleNrLayout) {
-			log('📐 Layout Selection: Single NR (optimized)');
-			return singleNrLayout;
-		}
+		if (cellTechnology === 'NR' && singleNrLayout) return singleNrLayout;
 		
 		// Fallback to multi-cell layout for single cells (works fine)
-		log('📐 Layout Selection: Multi-cell (fallback for single cell)', { 
-			technology: cellTechnology,
-			lteLayout: !!singleLteLayout,
-			nrLayout: !!singleNrLayout
-		});
 		return multiCellLayout;
 	});
 </script>

package/dist/apps/site-check/SiteCheckControls.svelte

@@ -1,7 +1,6 @@
 <svelte:options runes={true} />
 
 <script lang="ts">
-	import { log } from '../../core/logger';
 	import type { TreeGroupingConfig, TreeGroupField, ColorDimension } from './index';
 
 	interface Props {
@@ -68,7 +67,6 @@
 	function handleSearch() {
 		if (onSearch) {
 			onSearch(searchTerm);
-			log('🔍 Search triggered:', searchTerm);
 		}
 	}
 
@@ -76,7 +74,6 @@
 		searchTerm = '';
 		if (onSearch) {
 			onSearch('');
-			log('🧹 Search cleared');
 		}
 	}
 
@@ -100,12 +97,10 @@
 
 	function handleColorDimensionChange(dimension: ColorDimension) {
 		onColorDimensionChange?.(dimension);
-		log('🎨 Color dimension changed:', dimension);
 	}
 
 	function handleSingleRootSelectChange(enabled: boolean) {
 		onSingleRootSelectChange?.(enabled);
-		log('🔘 Single root select mode:', enabled);
 
 		// When enabling single root mode, uncheck all roots except the first one
 		if (enabled && treeStore) {
@@ -115,7 +110,6 @@
 					store.state.checkedPaths.has(path)
 				);
 				if (checkedRoots.length > 1) {
-					log('🔘 Multiple roots selected, keeping only first one:', checkedRoots[0]);
 					for (let i = 1; i < checkedRoots.length; i++) {
 						store.toggle(checkedRoots[i]);
 					}
@@ -126,7 +120,6 @@
 
 	function handleSingleLevel1SelectChange(enabled: boolean) {
 		onSingleLevel1SelectChange?.(enabled);
-		log('🔘 Single Level 1 select mode:', enabled);
 	}
 </script>
 

package/dist/apps/site-check/helper.js

@@ -103,36 +103,3 @@ export function extractBaseMetrics(layout) {
     });
     return Array.from(metrics);
 }
-/**
- * Get a distinct color for each cell line
- * Uses a predefined color palette with good contrast
- */
-function getColorForIndex(index) {
-    const colors = [
-        '#0d6efd', // Blue
-        '#198754', // Green
-        '#dc3545', // Red
-        '#ffc107', // Yellow
-        '#0dcaf0', // Cyan
-        '#6f42c1', // Purple
-        '#fd7e14', // Orange
-        '#20c997', // Teal
-        '#d63384', // Pink
-        '#6610f2', // Indigo
-        '#17a2b8', // Info
-        '#28a745', // Success
-        '#e83e8c', // Magenta
-        '#6c757d', // Gray
-        '#007bff', // Primary
-        '#28a745', // Green variant
-        '#17a2b8', // Cyan variant
-        '#ffc107', // Amber
-        '#dc3545', // Danger
-        '#343a40', // Dark
-        '#6c757d', // Secondary
-        '#fd7e14', // Orange variant
-        '#20c997', // Teal variant
-        '#6f42c1' // Violet
-    ];
-    return colors[index % colors.length];
-}

package/dist/apps/site-check/transforms.js

@@ -2,7 +2,6 @@
  * Data Transforms for Site Check Component
  * Converts raw CSV data to TreeView nodes and Chart configurations
  */
-import { log } from '../../core/logger';
 /**
  * Generate a deterministic, production-safe stackgroup ID
  * This function ensures stackgroups are:
@@ -37,16 +36,17 @@ export function createStackGroupId(value, mode) {
 export function assignStackGroups(kpis, cells, mode) {
     // Create a mapping of cellName → metadata for quick lookup
     const cellMetadata = new Map(cells);
-    return kpis.map(kpi => {
+    // Track missing cells for aggregated warning
+    const missingCells = [];
+    const result = kpis.map(kpi => {
         // Extract cellName from the rawName (format: "METRIC_CELLNAME")
         const cellName = kpi.rawName.split('_').slice(1).join('_');
         const record = cellMetadata.get(cellName);
         if (!record) {
-            // Fallback if cell not found - use default stackgroup
-            log('⚠️ Cell not found for KPI, using default stackgroup', {
-                kpiRawName: kpi.rawName,
-                cellName
-            });
+            // Track missing cell for aggregated warning
+            if (!missingCells.includes(cellName)) {
+                missingCells.push(cellName);
+            }
             return {
                 ...kpi,
                 stackGroup: createStackGroupId(null, mode)
@@ -77,18 +77,16 @@ export function assignStackGroups(kpis, cells, mode) {
         }
         // Create deterministic stackgroup ID
         const stackGroup = createStackGroupId(groupValue, mode);
-        log('📊 Assigned stackgroup', {
-            kpiName: kpi.name,
-            cellName,
-            mode,
-            groupValue,
-            stackGroup
-        });
         return {
             ...kpi,
             stackGroup
         };
     });
+    // Log aggregated warning if any cells were missing
+    if (missingCells.length > 0) {
+        console.warn(`[SiteCheck] ${missingCells.length} cells not found for KPIs, using default stackgroup. Cells: ${missingCells.slice(0, 5).join(', ')}${missingCells.length > 5 ? ` (+${missingCells.length - 5} more)` : ''}`);
+    }
+    return result;
 }
 /**
  * Band frequency mapping for consistent ordering
@@ -155,11 +153,6 @@ export function sortCellsByBandFrequency(items) {
  * @param grouping - Tree grouping configuration (defaults to Site → Azimuth → Cell)
  */
 export function buildTreeNodes(data, grouping = { level0: 'site', level1: 'azimuth' }) {
-    log('🔄 Building tree nodes', {
-        recordCount: data.length,
-        grouping,
-        treeDepth: grouping.level1 === null ? 2 : 3
-    });
     // Check if this is a 2-level tree (no level1)
     if (grouping.level1 === null) {
         return build2LevelTree(data, grouping);
@@ -237,11 +230,6 @@ export function buildTreeNodes(data, grouping = { level0: 'site', level1: 'azimu
         });
         treeNodes.push(level0Node);
     });
-    log('✅ Tree nodes built', {
-        totalNodes: treeNodes.length,
-        grouping,
-        sampleNode: treeNodes[0]?.label
-    });
     return treeNodes;
 }
 /**
@@ -299,11 +287,6 @@ function build2LevelTree(data, grouping) {
         });
         treeNodes.push(level0Node);
     });
-    log('✅ 2-level tree nodes built', {
-        totalNodes: treeNodes.length,
-        grouping: `${grouping.level0} → cell`,
-        sampleNode: treeNodes[0]?.label
-    });
     return treeNodes;
 }
 /**
@@ -351,14 +334,10 @@ function compareValues(a, b) {
 }
 /**
  * Get icon emoji based on band technology
+ * Currently disabled - returns empty string to keep tree labels clean
  */
-function getBandIcon(band) {
+function getBandIcon(_band) {
     return '';
-    if (band.startsWith('NR'))
-        return '📶'; // 5G
-    if (band.startsWith('LTE'))
-        return '📱'; // 4G
-    return '📡'; // Fallback
 }
 /**
  * Filter chart data based on selected tree paths
@@ -366,11 +345,6 @@ function getBandIcon(band) {
  * Handles both 2-level (level0:cellName) and 3-level (level0:level1:cellName) paths
  */
 export function filterChartData(data, checkedPaths) {
-    log('🔄 Filtering chart data', {
-        totalRecords: data.length,
-        checkedPathsCount: checkedPaths.size,
-        paths: Array.from(checkedPaths)
-    });
     // Extract cell names from checked leaf paths
     const selectedCells = new Set();
     checkedPaths.forEach((path) => {
@@ -385,12 +359,7 @@ export function filterChartData(data, checkedPaths) {
         }
     });
     // Filter data to only include selected cells
-    const filtered = data.filter((record) => selectedCells.has(record.cellName));
-    log('✅ Filtered chart data', {
-        selectedCells: Array.from(selectedCells),
-        filteredCount: filtered.length
-    });
-    return filtered;
+    return data.filter((record) => selectedCells.has(record.cellName));
 }
 /**
  * Transform data for chart component consumption
@@ -401,10 +370,6 @@ export function filterChartData(data, checkedPaths) {
  * @param baseMetrics - Array of metric names to pivot (e.g., ['DL_GBYTES', 'UL_GBYTES'])
  */
 export function transformChartData(data, baseMetrics) {
-    log('🔄 Transforming chart data', {
-        rowCount: data.length,
-        baseMetrics
-    });
     // Group data by date
     const dateMap = new Map();
     data.forEach((record) => {
@@ -435,10 +400,6 @@ export function transformChartData(data, baseMetrics) {
     });
     // Sort by date
     pivotedData.sort((a, b) => a.TIMESTAMP.localeCompare(b.TIMESTAMP));
-    log('✅ Chart data transformed', {
-        outputRows: pivotedData.length,
-        sampleColumns: pivotedData[0] ? Object.keys(pivotedData[0]) : []
-    });
     return pivotedData;
 }
 /**
@@ -508,17 +469,6 @@ export function createStyledKPI(metricName, cellRecord, unit, grouping, colorDim
         ...(color && { color }),
         ...(lineStyle && { lineStyle })
     };
-    log('🎨 Styled KPI created', {
-        metricName,
-        cellName,
-        displayName,
-        band,
-        sector,
-        azimuth,
-        colorDimension,
-        color,
-        lineStyle
-    });
     return kpi;
 }
 /**