From 9e7cff87172137f45802d24b52fd23e04f691c94 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 24 Dec 2025 12:44:08 +0000
Subject: [PATCH 1/6] feat: Add Modules tab with Bill of Materials

- Created modules.js variable file with product number mappings for all EBP files
- Added new "Modules" tab button to index.html
- Implemented displayModules function in ui.js to show:
  - Unique modules list with product numbers, variant numbers, and unit type IDs
  - Bill of Materials with quantities for each module
- Updated tab switching logic to handle modules tab
- Modules tab displays metadata and provides two separate views for module management

This enables users to view all available modules and generate a complete
Bill of Materials from the loaded EBP file.
---
 index.html |  18 +++++--
 js/ui.js   | 111 +++++++++++++++++++++++++++++++++++++++
 modules.js | 151 +++++++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 277 insertions(+), 3 deletions(-)
 create mode 100644 modules.js

diff --git a/index.html b/index.html
index b54a3dd..c3fa0b9 100644
--- a/index.html
+++ b/index.html
@@ -48,6 +48,7 @@ <h1>
         <button class="tab-button" data-tab="components">NMEA Components</button>
         <button class="tab-button" data-tab="alerts">Alerts</button>
         <button class="tab-button" data-tab="memory">Memory</button>
+        <button class="tab-button" data-tab="modules">Modules</button>
     </div>
 
     <!-- Results container -->
@@ -65,7 +66,7 @@ <h1>
     <!-- Import modules -->
     <script type="module">
         import { parseUnits, parseProjectMetadata, parseAlarms, parseComponents, parseMemory, validateEBP } from './js/parser.js';
-        import { displayUnits, displayComponents, displayAlertsDetailed, displayMemory, displayError } from './js/ui.js';
+        import { displayUnits, displayComponents, displayAlertsDetailed, displayMemory, displayModules, displayError } from './js/ui.js';
         import { downloadPDF, filterUnitsAndChannels, debounce } from './js/utils.js';
 
         // DOM elements
@@ -167,6 +168,9 @@ <h1>
                 case 'memory':
                     displayMemory(currentMemory, results, currentMetadata);
                     break;
+                case 'modules':
+                    displayModules(allUnits, results, currentMetadata);
+                    break;
             }
         }
 
@@ -188,10 +192,18 @@ <h1>
                 // Hide search for non-units tabs
                 if (activeTab !== 'units') {
                     searchBox.style.display = 'none';
-                    metadataCard.style.display = 'none';
                 } else {
                     searchBox.style.display = 'block';
-                    metadataCard.style.display = 'block';
+                }
+
+                // Show metadata card only for units tab
+                const metadataCard = document.querySelector('.metadata-card');
+                if (metadataCard) {
+                    if (activeTab === 'units') {
+                        metadataCard.style.display = 'block';
+                    } else {
+                        metadataCard.style.display = 'none';
+                    }
                 }
             });
         });
diff --git a/js/ui.js b/js/ui.js
index 62f2c3d..4bfeec5 100644
--- a/js/ui.js
+++ b/js/ui.js
@@ -437,4 +437,115 @@ export function displayMemory(memory, container, metadata = null) {
     html += '</tbody></table></div></div>';
 
     container.innerHTML = html;
+}
+
+/**
+ * Display modules with product numbers and variant numbers
+ * @param {Array} units - Array of unit objects
+ * @param {HTMLElement} container - Container element
+ * @param {Object} metadata - Optional project metadata
+ */
+export function displayModules(units, container, metadata = null) {
+    container.style.display = 'block';
+
+    let html = '';
+
+    if (metadata) {
+        html += renderMetadata(metadata);
+    }
+
+    // Generate Bill of Materials
+    const bom = generateBOMFromUnits(units);
+    const uniqueModules = getUniqueModulesFromUnits(units);
+
+    // Modules List Section
+    html += '<div style="padding: 20px; background: white; border-radius: 8px; box-shadow: 0 2px 4px rgba(0,0,0,0.1); margin-bottom: 20px;">';
+    html += '<h3>📦 Modules</h3>';
+    html += `<p style="margin-bottom: 15px; color: #666;">Found ${uniqueModules.length} unique module${uniqueModules.length !== 1 ? 's' : ''}</p>`;
+    html += '<div style="overflow-x: auto;"><table><thead><tr>';
+    html += '<th>Product Number</th><th>Variant Number</th><th>Unit Type ID</th>';
+    html += '</tr></thead><tbody>';
+
+    uniqueModules.forEach(module => {
+        html += '<tr>';
+        html += `<td>${escapeHtml(module.productNumber)}</td>`;
+        html += `<td>${escapeHtml(module.variantNumber)}</td>`;
+        html += `<td>${escapeHtml(module.unitTypeId)}</td>`;
+        html += '</tr>';
+    });
+
+    html += '</tbody></table></div></div>';
+
+    // Bill of Materials Section
+    html += '<div style="padding: 20px; background: white; border-radius: 8px; box-shadow: 0 2px 4px rgba(0,0,0,0.1);">';
+    html += '<h3>📋 Bill of Materials</h3>';
+    html += `<p style="margin-bottom: 15px; color: #666;">Total items: ${bom.reduce((sum, item) => sum + item.quantity, 0)}</p>`;
+    html += '<div style="overflow-x: auto;"><table><thead><tr>';
+    html += '<th>Quantity</th><th>Product Number</th><th>Variant Number</th><th>Unit Type ID</th>';
+    html += '</tr></thead><tbody>';
+
+    bom.forEach(item => {
+        html += '<tr>';
+        html += `<td style="text-align: center; font-weight: bold;">${item.quantity}</td>`;
+        html += `<td>${escapeHtml(item.productNumber)}</td>`;
+        html += `<td>${escapeHtml(item.variantNumber)}</td>`;
+        html += `<td>${escapeHtml(item.unitTypeId)}</td>`;
+        html += '</tr>';
+    });
+
+    html += '</tbody></table></div></div>';
+
+    container.innerHTML = html;
+}
+
+/**
+ * Get unique modules from units array
+ * @param {Array} units - Array of unit objects
+ * @returns {Array} Array of unique modules
+ */
+function getUniqueModulesFromUnits(units) {
+    const uniqueMap = new Map();
+
+    units.forEach(unit => {
+        const key = `${unit.name}|${unit.standardUnitVariantNumber || 'N/A'}`;
+        if (!uniqueMap.has(key)) {
+            uniqueMap.set(key, {
+                productNumber: unit.name,
+                variantNumber: unit.standardUnitVariantNumber || 'N/A',
+                unitTypeId: unit.unitTypeId || 'N/A'
+            });
+        }
+    });
+
+    return Array.from(uniqueMap.values()).sort((a, b) =>
+        a.productNumber.localeCompare(b.productNumber)
+    );
+}
+
+/**
+ * Generate Bill of Materials from units array
+ * @param {Array} units - Array of unit objects
+ * @returns {Array} BOM entries with product number, variant number, unit type, and quantity
+ */
+function generateBOMFromUnits(units) {
+    const bomMap = new Map();
+
+    units.forEach(unit => {
+        const key = `${unit.name}|${unit.standardUnitVariantNumber || 'N/A'}|${unit.unitTypeId || 'N/A'}`;
+
+        if (bomMap.has(key)) {
+            bomMap.get(key).quantity++;
+        } else {
+            bomMap.set(key, {
+                productNumber: unit.name,
+                variantNumber: unit.standardUnitVariantNumber || 'N/A',
+                unitTypeId: unit.unitTypeId || 'N/A',
+                quantity: 1
+            });
+        }
+    });
+
+    return Array.from(bomMap.values()).sort((a, b) =>
+        a.productNumber.localeCompare(b.productNumber)
+    );
 }
\ No newline at end of file
diff --git a/modules.js b/modules.js
new file mode 100644
index 0000000..14e63c2
--- /dev/null
+++ b/modules.js
@@ -0,0 +1,151 @@
+/**
+ * Modules Variable File
+ * Maps product numbers (unit names) to their standard unit variant numbers
+ *
+ * This file serves as a lookup table for generating Bill of Materials (BOM)
+ * and enriching unit data with product information.
+ */
+
+/**
+ * Module definition
+ * @typedef {Object} Module
+ * @property {string} productNumber - The product number (unit name)
+ * @property {string} standardUnitVariantNumber - The standard unit variant number
+ * @property {string} description - Optional description of the module
+ */
+
+/**
+ * All available modules from EBP files
+ * @type {Module[]}
+ */
+export const MODULES = [
+    // From connect50_v-1+.ebp
+    { productNumber: "010-02575-10_mcuv2", standardUnitVariantNumber: "100257510", description: "MCU v2" },
+    { productNumber: "010-02223-11_mcu100", standardUnitVariantNumber: "2051011", description: "MCU 100" },
+    { productNumber: "010-02225-10", standardUnitVariantNumber: "2110110", description: "" },
+    { productNumber: "010-02225-16", standardUnitVariantNumber: "2110115", description: "" },
+    { productNumber: "010-02225-17", standardUnitVariantNumber: "2110116", description: "" },
+    { productNumber: "010-02225-18", standardUnitVariantNumber: "2110117", description: "" },
+    { productNumber: "010-02275-01", standardUnitVariantNumber: "0152", description: "" },
+    { productNumber: "010-02275-02", standardUnitVariantNumber: "0151", description: "" },
+    { productNumber: "010-02225-19", standardUnitVariantNumber: "2110118", description: "" },
+    { productNumber: "010-02275-03", standardUnitVariantNumber: "0152", description: "" },
+    { productNumber: "010-02225-05", standardUnitVariantNumber: "2110103", description: "" },
+    { productNumber: "010-02225-06", standardUnitVariantNumber: "2110104", description: "" },
+    { productNumber: "010-02225-07", standardUnitVariantNumber: "2110105", description: "" },
+    { productNumber: "010-02279-02", standardUnitVariantNumber: "2210102", description: "" },
+    { productNumber: "010-02279-03", standardUnitVariantNumber: "2210103", description: "" },
+
+    // From connect50 v2_v-1+.ebp
+    { productNumber: "010-02575-10", standardUnitVariantNumber: "100257510", description: "MCU v2" },
+    { productNumber: "MFD / WDU", standardUnitVariantNumber: "88888888", description: "MFD / WDU" },
+    { productNumber: "010-02225-30", standardUnitVariantNumber: "100222530", description: "" },
+    { productNumber: "010-02225-31", standardUnitVariantNumber: "100222531", description: "" },
+    { productNumber: "010-02278-21", standardUnitVariantNumber: "100227821", description: "" },
+    { productNumber: "010-02279-20", standardUnitVariantNumber: "100227920", description: "" },
+    { productNumber: "010-02279-21", standardUnitVariantNumber: "100227921", description: "" },
+
+    // From DCM.ebp
+    { productNumber: "MCU v2", standardUnitVariantNumber: "100257510", description: "MCU v2" },
+    { productNumber: "010-02219-01", standardUnitVariantNumber: "11", description: "" },
+    { productNumber: "010-02219-02", standardUnitVariantNumber: "12", description: "" },
+    { productNumber: "010-02219-03", standardUnitVariantNumber: "13", description: "" },
+    { productNumber: "010-02219-04", standardUnitVariantNumber: "14", description: "" },
+    { productNumber: "010-02219-05", standardUnitVariantNumber: "15", description: "" },
+    { productNumber: "010-02219-55", standardUnitVariantNumber: "55", description: "" },
+    { productNumber: "010-02220-06", standardUnitVariantNumber: "16", description: "" },
+    { productNumber: "010-02220-07", standardUnitVariantNumber: "17", description: "" },
+    { productNumber: "010-02220-08", standardUnitVariantNumber: "18", description: "" },
+    { productNumber: "010-02220-09", standardUnitVariantNumber: "19", description: "" },
+    { productNumber: "010-02220-10", standardUnitVariantNumber: "20", description: "" },
+    { productNumber: "010-02221-08", standardUnitVariantNumber: "28", description: "" },
+    { productNumber: "010-02222-10", standardUnitVariantNumber: "30", description: "" }
+];
+
+/**
+ * Create a lookup map for quick access by product number
+ * @type {Map<string, Module>}
+ */
+export const MODULE_MAP = new Map(
+    MODULES.map(module => [module.productNumber, module])
+);
+
+/**
+ * Create a reverse lookup map by standard unit variant number
+ * @type {Map<string, Module[]>}
+ */
+export const VARIANT_MAP = MODULES.reduce((map, module) => {
+    const variant = module.standardUnitVariantNumber;
+    if (!map.has(variant)) {
+        map.set(variant, []);
+    }
+    map.get(variant).push(module);
+    return map;
+}, new Map());
+
+/**
+ * Get module information by product number
+ * @param {string} productNumber - The product number to look up
+ * @returns {Module|undefined} The module information or undefined if not found
+ */
+export function getModuleByProductNumber(productNumber) {
+    return MODULE_MAP.get(productNumber);
+}
+
+/**
+ * Get module(s) by standard unit variant number
+ * @param {string} variantNumber - The variant number to look up
+ * @returns {Module[]} Array of modules with this variant number
+ */
+export function getModulesByVariantNumber(variantNumber) {
+    return VARIANT_MAP.get(variantNumber) || [];
+}
+
+/**
+ * Generate a Bill of Materials from an array of units
+ * @param {Array} units - Array of unit objects with name and standardUnitVariantNumber
+ * @returns {Array} BOM entries with product number, variant number, and quantity
+ */
+export function generateBOM(units) {
+    const bomMap = new Map();
+
+    units.forEach(unit => {
+        const key = `${unit.name}|${unit.standardUnitVariantNumber || ''}`;
+
+        if (bomMap.has(key)) {
+            bomMap.get(key).quantity++;
+        } else {
+            bomMap.set(key, {
+                productNumber: unit.name,
+                standardUnitVariantNumber: unit.standardUnitVariantNumber || 'N/A',
+                quantity: 1,
+                serial: unit.serial || '0',
+                unitTypeId: unit.unitTypeId || 'N/A'
+            });
+        }
+    });
+
+    return Array.from(bomMap.values()).sort((a, b) =>
+        a.productNumber.localeCompare(b.productNumber)
+    );
+}
+
+/**
+ * Get unique modules from the MODULES array
+ * Removes duplicates based on product number and variant number combination
+ * @returns {Module[]} Array of unique modules
+ */
+export function getUniqueModules() {
+    const uniqueMap = new Map();
+
+    MODULES.forEach(module => {
+        const key = `${module.productNumber}|${module.standardUnitVariantNumber}`;
+        if (!uniqueMap.has(key)) {
+            uniqueMap.set(key, module);
+        }
+    });
+
+    return Array.from(uniqueMap.values()).sort((a, b) =>
+        a.productNumber.localeCompare(b.productNumber)
+    );
+}

From 38e1a17b618a31003a32e618ea243ce3a69b7dd1 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 24 Dec 2025 12:55:16 +0000
Subject: [PATCH 2/6] fix: Import modules.js and fix BOM aggregation

- Import MODULES array from modules.js in index.html
- Pass MODULES to displayModules function for reference data
- Update displayModules to show two sections:
  1. "Available Modules" - all modules from modules.js database
  2. "Bill of Materials" - actual items from loaded file with proper counting
- Fix BOM aggregation to count by product number only (not variant/unitTypeId)
- Now BOM properly aggregates quantities instead of showing separate entries

This ensures the BOM displays accurate counts for each product and provides
a complete reference list of all available modules.
---
 index.html |  3 ++-
 js/ui.js   | 37 +++++++++++++++++++++++++------------
 2 files changed, 27 insertions(+), 13 deletions(-)

diff --git a/index.html b/index.html
index c3fa0b9..655a956 100644
--- a/index.html
+++ b/index.html
@@ -68,6 +68,7 @@ <h1>
         import { parseUnits, parseProjectMetadata, parseAlarms, parseComponents, parseMemory, validateEBP } from './js/parser.js';
         import { displayUnits, displayComponents, displayAlertsDetailed, displayMemory, displayModules, displayError } from './js/ui.js';
         import { downloadPDF, filterUnitsAndChannels, debounce } from './js/utils.js';
+        import { MODULES, getModuleByProductNumber } from './modules.js';
 
         // DOM elements
         const dropZone = document.getElementById('dropZone');
@@ -169,7 +170,7 @@ <h1>
                     displayMemory(currentMemory, results, currentMetadata);
                     break;
                 case 'modules':
-                    displayModules(allUnits, results, currentMetadata);
+                    displayModules(allUnits, results, currentMetadata, MODULES);
                     break;
             }
         }
diff --git a/js/ui.js b/js/ui.js
index 4bfeec5..2f7bcae 100644
--- a/js/ui.js
+++ b/js/ui.js
@@ -444,8 +444,9 @@ export function displayMemory(memory, container, metadata = null) {
  * @param {Array} units - Array of unit objects
  * @param {HTMLElement} container - Container element
  * @param {Object} metadata - Optional project metadata
+ * @param {Array} modulesList - Array of module definitions from modules.js
  */
-export function displayModules(units, container, metadata = null) {
+export function displayModules(units, container, metadata = null, modulesList = []) {
     container.style.display = 'block';
 
     let html = '';
@@ -454,32 +455,43 @@ export function displayModules(units, container, metadata = null) {
         html += renderMetadata(metadata);
     }
 
+    // Create a lookup map from the modules list
+    const moduleLookup = new Map();
+    modulesList.forEach(module => {
+        moduleLookup.set(module.productNumber, module);
+    });
+
     // Generate Bill of Materials
     const bom = generateBOMFromUnits(units);
     const uniqueModules = getUniqueModulesFromUnits(units);
 
-    // Modules List Section
+    // Modules List Section - showing all available modules from modules.js
     html += '<div style="padding: 20px; background: white; border-radius: 8px; box-shadow: 0 2px 4px rgba(0,0,0,0.1); margin-bottom: 20px;">';
-    html += '<h3>📦 Modules</h3>';
-    html += `<p style="margin-bottom: 15px; color: #666;">Found ${uniqueModules.length} unique module${uniqueModules.length !== 1 ? 's' : ''}</p>`;
+    html += '<h3>📦 Available Modules</h3>';
+    html += `<p style="margin-bottom: 15px; color: #666;">Total available modules in database: ${modulesList.length}</p>`;
     html += '<div style="overflow-x: auto;"><table><thead><tr>';
-    html += '<th>Product Number</th><th>Variant Number</th><th>Unit Type ID</th>';
+    html += '<th>Product Number</th><th>Variant Number</th><th>Description</th>';
     html += '</tr></thead><tbody>';
 
-    uniqueModules.forEach(module => {
+    // Sort modules by product number
+    const sortedModules = [...modulesList].sort((a, b) =>
+        a.productNumber.localeCompare(b.productNumber)
+    );
+
+    sortedModules.forEach(module => {
         html += '<tr>';
         html += `<td>${escapeHtml(module.productNumber)}</td>`;
-        html += `<td>${escapeHtml(module.variantNumber)}</td>`;
-        html += `<td>${escapeHtml(module.unitTypeId)}</td>`;
+        html += `<td>${escapeHtml(module.standardUnitVariantNumber)}</td>`;
+        html += `<td>${escapeHtml(module.description || '-')}</td>`;
         html += '</tr>';
     });
 
     html += '</tbody></table></div></div>';
 
-    // Bill of Materials Section
+    // Bill of Materials Section - showing what's actually used in the loaded file
     html += '<div style="padding: 20px; background: white; border-radius: 8px; box-shadow: 0 2px 4px rgba(0,0,0,0.1);">';
-    html += '<h3>📋 Bill of Materials</h3>';
-    html += `<p style="margin-bottom: 15px; color: #666;">Total items: ${bom.reduce((sum, item) => sum + item.quantity, 0)}</p>`;
+    html += '<h3>📋 Bill of Materials (from loaded file)</h3>';
+    html += `<p style="margin-bottom: 15px; color: #666;">Total items: ${bom.reduce((sum, item) => sum + item.quantity, 0)} | Unique products: ${bom.length}</p>`;
     html += '<div style="overflow-x: auto;"><table><thead><tr>';
     html += '<th>Quantity</th><th>Product Number</th><th>Variant Number</th><th>Unit Type ID</th>';
     html += '</tr></thead><tbody>';
@@ -531,7 +543,8 @@ function generateBOMFromUnits(units) {
     const bomMap = new Map();
 
     units.forEach(unit => {
-        const key = `${unit.name}|${unit.standardUnitVariantNumber || 'N/A'}|${unit.unitTypeId || 'N/A'}`;
+        // Use only product number as the key for proper aggregation
+        const key = unit.name;
 
         if (bomMap.has(key)) {
             bomMap.get(key).quantity++;

From f03bccab08ce8c0fdf9c6ff6a9e1bdb08c905ab5 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 24 Dec 2025 14:01:34 +0000
Subject: [PATCH 3/6] fix: Match modules by variant number instead of product
 number

- Updated module lookup to use standardUnitVariantNumber as the key
- Enriches units with product numbers from modules.js based on variant number matching
- BOM now aggregates by variant number (proper grouping)
- Added "Unit Name" column to BOM for clarity
- Product number now comes from modules.js lookup when variant matches

How it works now:
1. Scan modules.js and create lookup map by variant number
2. For each unit in EBP file, lookup product number using variant number
3. BOM groups by variant number and shows matched product number
4. Displays: Quantity, Product Number (from modules.js), Variant Number, Unit Name, Unit Type ID
---
 js/ui.js | 33 +++++++++++++++++++++++----------
 1 file changed, 23 insertions(+), 10 deletions(-)

diff --git a/js/ui.js b/js/ui.js
index 2f7bcae..87768db 100644
--- a/js/ui.js
+++ b/js/ui.js
@@ -455,15 +455,26 @@ export function displayModules(units, container, metadata = null, modulesList =
         html += renderMetadata(metadata);
     }
 
-    // Create a lookup map from the modules list
+    // Create a lookup map from the modules list - MATCH ON VARIANT NUMBER
     const moduleLookup = new Map();
     modulesList.forEach(module => {
-        moduleLookup.set(module.productNumber, module);
+        moduleLookup.set(module.standardUnitVariantNumber, module);
     });
 
-    // Generate Bill of Materials
-    const bom = generateBOMFromUnits(units);
-    const uniqueModules = getUniqueModulesFromUnits(units);
+    // Enrich units with product numbers from modules.js based on variant number
+    const enrichedUnits = units.map(unit => {
+        const variantNumber = unit.standardUnitVariantNumber;
+        const moduleInfo = moduleLookup.get(variantNumber);
+
+        return {
+            ...unit,
+            productNumber: moduleInfo ? moduleInfo.productNumber : unit.name,
+            moduleDescription: moduleInfo ? moduleInfo.description : ''
+        };
+    });
+
+    // Generate Bill of Materials from enriched units
+    const bom = generateBOMFromUnits(enrichedUnits);
 
     // Modules List Section - showing all available modules from modules.js
     html += '<div style="padding: 20px; background: white; border-radius: 8px; box-shadow: 0 2px 4px rgba(0,0,0,0.1); margin-bottom: 20px;">';
@@ -493,7 +504,7 @@ export function displayModules(units, container, metadata = null, modulesList =
     html += '<h3>📋 Bill of Materials (from loaded file)</h3>';
     html += `<p style="margin-bottom: 15px; color: #666;">Total items: ${bom.reduce((sum, item) => sum + item.quantity, 0)} | Unique products: ${bom.length}</p>`;
     html += '<div style="overflow-x: auto;"><table><thead><tr>';
-    html += '<th>Quantity</th><th>Product Number</th><th>Variant Number</th><th>Unit Type ID</th>';
+    html += '<th>Quantity</th><th>Product Number</th><th>Variant Number</th><th>Unit Name</th><th>Unit Type ID</th>';
     html += '</tr></thead><tbody>';
 
     bom.forEach(item => {
@@ -501,6 +512,7 @@ export function displayModules(units, container, metadata = null, modulesList =
         html += `<td style="text-align: center; font-weight: bold;">${item.quantity}</td>`;
         html += `<td>${escapeHtml(item.productNumber)}</td>`;
         html += `<td>${escapeHtml(item.variantNumber)}</td>`;
+        html += `<td>${escapeHtml(item.unitName)}</td>`;
         html += `<td>${escapeHtml(item.unitTypeId)}</td>`;
         html += '</tr>';
     });
@@ -536,22 +548,23 @@ function getUniqueModulesFromUnits(units) {
 
 /**
  * Generate Bill of Materials from units array
- * @param {Array} units - Array of unit objects
+ * @param {Array} units - Array of unit objects (can be enriched with productNumber)
  * @returns {Array} BOM entries with product number, variant number, unit type, and quantity
  */
 function generateBOMFromUnits(units) {
     const bomMap = new Map();
 
     units.forEach(unit => {
-        // Use only product number as the key for proper aggregation
-        const key = unit.name;
+        // Use variant number as the key for proper aggregation
+        const key = unit.standardUnitVariantNumber || unit.name;
 
         if (bomMap.has(key)) {
             bomMap.get(key).quantity++;
         } else {
             bomMap.set(key, {
-                productNumber: unit.name,
+                productNumber: unit.productNumber || unit.name,
                 variantNumber: unit.standardUnitVariantNumber || 'N/A',
+                unitName: unit.name,
                 unitTypeId: unit.unitTypeId || 'N/A',
                 quantity: 1
             });

From 3bf5bdc6c67b92323d92e61d0227a52f84e55c8f Mon Sep 17 00:00:00 2001
From: Remco Halman <remco.halman@gmail.com>
Date: Wed, 24 Dec 2025 20:33:39 +0100
Subject: [PATCH 4/6] Chore: Update ui.js to reclect only the product numbers
 in the bom or other layout

---
 js/ui.js | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/js/ui.js b/js/ui.js
index 87768db..01269fe 100644
--- a/js/ui.js
+++ b/js/ui.js
@@ -468,7 +468,7 @@ export function displayModules(units, container, metadata = null, modulesList =
 
         return {
             ...unit,
-            productNumber: moduleInfo ? moduleInfo.productNumber : unit.name,
+            productNumber: moduleInfo.productNumber,
             moduleDescription: moduleInfo ? moduleInfo.description : ''
         };
     });
@@ -481,7 +481,7 @@ export function displayModules(units, container, metadata = null, modulesList =
     html += '<h3>📦 Available Modules</h3>';
     html += `<p style="margin-bottom: 15px; color: #666;">Total available modules in database: ${modulesList.length}</p>`;
     html += '<div style="overflow-x: auto;"><table><thead><tr>';
-    html += '<th>Product Number</th><th>Variant Number</th><th>Description</th>';
+    html += '<th>Product Number</th><th>Description</th>';
     html += '</tr></thead><tbody>';
 
     // Sort modules by product number
@@ -492,7 +492,6 @@ export function displayModules(units, container, metadata = null, modulesList =
     sortedModules.forEach(module => {
         html += '<tr>';
         html += `<td>${escapeHtml(module.productNumber)}</td>`;
-        html += `<td>${escapeHtml(module.standardUnitVariantNumber)}</td>`;
         html += `<td>${escapeHtml(module.description || '-')}</td>`;
         html += '</tr>';
     });
@@ -511,8 +510,6 @@ export function displayModules(units, container, metadata = null, modulesList =
         html += '<tr>';
         html += `<td style="text-align: center; font-weight: bold;">${item.quantity}</td>`;
         html += `<td>${escapeHtml(item.productNumber)}</td>`;
-        html += `<td>${escapeHtml(item.variantNumber)}</td>`;
-        html += `<td>${escapeHtml(item.unitName)}</td>`;
         html += `<td>${escapeHtml(item.unitTypeId)}</td>`;
         html += '</tr>';
     });
@@ -562,7 +559,7 @@ function generateBOMFromUnits(units) {
             bomMap.get(key).quantity++;
         } else {
             bomMap.set(key, {
-                productNumber: unit.productNumber || unit.name,
+                productNumber: unit.productNumber,
                 variantNumber: unit.standardUnitVariantNumber || 'N/A',
                 unitName: unit.name,
                 unitTypeId: unit.unitTypeId || 'N/A',

From 91677a62104047400a4afd997667982e529a99f5 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 24 Dec 2025 19:42:32 +0000
Subject: [PATCH 5/6] feat: Simplify Modules tab and show product numbers in
 Units

Changes:
- Simplified Modules tab to only show Quantity and Product Number in BOM
- Removed "Available Modules" section (was redundant)
- Updated Units page to show "Product Number" instead of "Variant Number"
- Enrich all units with product numbers from modules.js on file load
- Product numbers are looked up by matching variant numbers in modules.js

How it works:
1. When EBP file is loaded, units are enriched with product numbers
2. Units tab displays product numbers (from modules.js lookup)
3. Modules tab shows clean BOM with just Quantity and Product Number
4. All lookups are done by matching standardUnitVariantNumber
---
 index.html | 15 +++++++++++++++
 js/ui.js   | 31 ++++---------------------------
 2 files changed, 19 insertions(+), 27 deletions(-)

diff --git a/index.html b/index.html
index 655a956..3a50973 100644
--- a/index.html
+++ b/index.html
@@ -131,6 +131,21 @@ <h1>
 
                     // Parse all data
                     allUnits = await parseUnits(content);
+
+                    // Enrich units with product numbers from modules.js
+                    const moduleLookup = new Map();
+                    MODULES.forEach(module => {
+                        moduleLookup.set(module.standardUnitVariantNumber, module);
+                    });
+
+                    allUnits = allUnits.map(unit => {
+                        const moduleInfo = moduleLookup.get(unit.standardUnitVariantNumber);
+                        return {
+                            ...unit,
+                            productNumber: moduleInfo ? moduleInfo.productNumber : unit.name
+                        };
+                    });
+
                     currentUnits = allUnits;
                     currentMetadata = parseProjectMetadata(content);
                     currentAlarms = parseAlarms(content);
diff --git a/js/ui.js b/js/ui.js
index 01269fe..5449b0a 100644
--- a/js/ui.js
+++ b/js/ui.js
@@ -59,8 +59,8 @@ function renderUnitCard(unit, hasSearch = false) {
                     <div class="detail-value">${escapeHtml(unit.unitTypeId)}</div>
                 </div>
                 <div class="detail-item">
-                    <div class="detail-label">Variant Number</div>
-                    <div class="detail-value">${escapeHtml(unit.standardUnitVariantNumber)}</div>
+                    <div class="detail-label">Product Number</div>
+                    <div class="detail-value">${escapeHtml(unit.productNumber || unit.standardUnitVariantNumber)}</div>
                 </div>
             </div>
 
@@ -476,41 +476,18 @@ export function displayModules(units, container, metadata = null, modulesList =
     // Generate Bill of Materials from enriched units
     const bom = generateBOMFromUnits(enrichedUnits);
 
-    // Modules List Section - showing all available modules from modules.js
-    html += '<div style="padding: 20px; background: white; border-radius: 8px; box-shadow: 0 2px 4px rgba(0,0,0,0.1); margin-bottom: 20px;">';
-    html += '<h3>📦 Available Modules</h3>';
-    html += `<p style="margin-bottom: 15px; color: #666;">Total available modules in database: ${modulesList.length}</p>`;
-    html += '<div style="overflow-x: auto;"><table><thead><tr>';
-    html += '<th>Product Number</th><th>Description</th>';
-    html += '</tr></thead><tbody>';
-
-    // Sort modules by product number
-    const sortedModules = [...modulesList].sort((a, b) =>
-        a.productNumber.localeCompare(b.productNumber)
-    );
-
-    sortedModules.forEach(module => {
-        html += '<tr>';
-        html += `<td>${escapeHtml(module.productNumber)}</td>`;
-        html += `<td>${escapeHtml(module.description || '-')}</td>`;
-        html += '</tr>';
-    });
-
-    html += '</tbody></table></div></div>';
-
     // Bill of Materials Section - showing what's actually used in the loaded file
     html += '<div style="padding: 20px; background: white; border-radius: 8px; box-shadow: 0 2px 4px rgba(0,0,0,0.1);">';
-    html += '<h3>📋 Bill of Materials (from loaded file)</h3>';
+    html += '<h3>📋 Bill of Materials</h3>';
     html += `<p style="margin-bottom: 15px; color: #666;">Total items: ${bom.reduce((sum, item) => sum + item.quantity, 0)} | Unique products: ${bom.length}</p>`;
     html += '<div style="overflow-x: auto;"><table><thead><tr>';
-    html += '<th>Quantity</th><th>Product Number</th><th>Variant Number</th><th>Unit Name</th><th>Unit Type ID</th>';
+    html += '<th>Quantity</th><th>Product Number</th>';
     html += '</tr></thead><tbody>';
 
     bom.forEach(item => {
         html += '<tr>';
         html += `<td style="text-align: center; font-weight: bold;">${item.quantity}</td>`;
         html += `<td>${escapeHtml(item.productNumber)}</td>`;
-        html += `<td>${escapeHtml(item.unitTypeId)}</td>`;
         html += '</tr>';
     });
 

From af2491331cda70d4f02b72ffa807acba01b5dc19 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 24 Dec 2025 19:50:55 +0000
Subject: [PATCH 6/6] fix: Show 'Unknown' for missing product numbers and fix
 Modules tab

Critical fixes:
1. Units page now shows "Unknown" instead of unit name when no product match
2. Fixed Modules tab crash caused by undefined moduleInfo access
3. Added null checks throughout to prevent JavaScript errors

The Modules tab was failing because line 471 tried to access
moduleInfo.productNumber without checking if moduleInfo exists first.
This caused a runtime error and prevented the tab from displaying.

All fallback values now consistently use "Unknown" instead of unit.name.
---
 index.html | 2 +-
 js/ui.js   | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/index.html b/index.html
index 3a50973..33e51be 100644
--- a/index.html
+++ b/index.html
@@ -142,7 +142,7 @@ <h1>
                         const moduleInfo = moduleLookup.get(unit.standardUnitVariantNumber);
                         return {
                             ...unit,
-                            productNumber: moduleInfo ? moduleInfo.productNumber : unit.name
+                            productNumber: moduleInfo ? moduleInfo.productNumber : 'Unknown'
                         };
                     });
 
diff --git a/js/ui.js b/js/ui.js
index 5449b0a..1f88d6d 100644
--- a/js/ui.js
+++ b/js/ui.js
@@ -60,7 +60,7 @@ function renderUnitCard(unit, hasSearch = false) {
                 </div>
                 <div class="detail-item">
                     <div class="detail-label">Product Number</div>
-                    <div class="detail-value">${escapeHtml(unit.productNumber || unit.standardUnitVariantNumber)}</div>
+                    <div class="detail-value">${escapeHtml(unit.productNumber || 'Unknown')}</div>
                 </div>
             </div>
 
@@ -468,7 +468,7 @@ export function displayModules(units, container, metadata = null, modulesList =
 
         return {
             ...unit,
-            productNumber: moduleInfo.productNumber,
+            productNumber: moduleInfo ? moduleInfo.productNumber : 'Unknown',
             moduleDescription: moduleInfo ? moduleInfo.description : ''
         };
     });
@@ -536,7 +536,7 @@ function generateBOMFromUnits(units) {
             bomMap.get(key).quantity++;
         } else {
             bomMap.set(key, {
-                productNumber: unit.productNumber,
+                productNumber: unit.productNumber || 'Unknown',
                 variantNumber: unit.standardUnitVariantNumber || 'N/A',
                 unitName: unit.name,
                 unitTypeId: unit.unitTypeId || 'N/A',
@@ -546,6 +546,6 @@ function generateBOMFromUnits(units) {
     });
 
     return Array.from(bomMap.values()).sort((a, b) =>
-        a.productNumber.localeCompare(b.productNumber)
+        (a.productNumber || 'Unknown').localeCompare(b.productNumber || 'Unknown')
     );
 }
\ No newline at end of file