From 9505303d1d7d0dbba8f7784b57b37892791b7718 Mon Sep 17 00:00:00 2001
From: Mikhail Chusavitin <mchusavitin@mchusmbp.local>
Date: Thu, 21 May 2026 14:15:28 +0300
Subject: [PATCH] fix(inspur): show microcode version for every CPU, not just
 the first

Dedup by version caused CPU1 Microcode to be omitted when both CPUs run
the same version, leaving the firmware column blank for the second socket.
Each CPU gets its own firmware entry keyed by index.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 internal/parser/vendors/inspur/asset.go     | 5 +----
 internal/parser/vendors/inspur/component.go | 4 +---
 2 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/internal/parser/vendors/inspur/asset.go b/internal/parser/vendors/inspur/asset.go
index 77969d2..06a019f 100644
--- a/internal/parser/vendors/inspur/asset.go
+++ b/internal/parser/vendors/inspur/asset.go
@@ -117,7 +117,6 @@ func ParseAssetJSON(content []byte, pcieSlotDeviceNames map[int]string, pcieSlot
 	}
 
 	// Parse CPU info
-	seenMicrocode := make(map[string]bool)
 	for i, cpu := range asset.CpuInfo {
 		config.CPUs = append(config.CPUs, models.CPU{
 			Socket:       i,
@@ -133,13 +132,11 @@ func ParseAssetJSON(content []byte, pcieSlotDeviceNames map[int]string, pcieSlot
 			PPIN:         cpu.PPIN,
 		})
 
-		// Add CPU microcode to firmware list (deduplicated)
-		if cpu.MicroCodeVer != "" && !seenMicrocode[cpu.MicroCodeVer] {
+		if cpu.MicroCodeVer != "" {
 			config.Firmware = append(config.Firmware, models.FirmwareInfo{
 				DeviceName: fmt.Sprintf("CPU%d Microcode", i),
 				Version:    cpu.MicroCodeVer,
 			})
-			seenMicrocode[cpu.MicroCodeVer] = true
 		}
 	}
 
diff --git a/internal/parser/vendors/inspur/component.go b/internal/parser/vendors/inspur/component.go
index dd4f40d..1330215 100644
--- a/internal/parser/vendors/inspur/component.go
+++ b/internal/parser/vendors/inspur/component.go
@@ -102,7 +102,6 @@ func parseCPUInfo(text string, hw *models.HardwareConfig) {
 		return
 	}
 
-	seenMicrocode := make(map[string]bool)
 	for _, proc := range cpuInfo.Processors {
 		if proc.ProcStatus != 1 && proc.ConfigStatus != 1 {
 			continue
@@ -120,12 +119,11 @@ func parseCPUInfo(text string, hw *models.HardwareConfig) {
 			TDP:          proc.TDP,
 			PPIN:         proc.PPIN,
 		})
-		if proc.MicroCode != "" && !seenMicrocode[proc.MicroCode] {
+		if proc.MicroCode != "" {
 			hw.Firmware = append(hw.Firmware, models.FirmwareInfo{
 				DeviceName: fmt.Sprintf("CPU%d Microcode", proc.ProcID),
 				Version:    proc.MicroCode,
 			})
-			seenMicrocode[proc.MicroCode] = true
 		}
 	}
 }