docs(bible-local): add backlog with sfp_modules implementation plan

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
docs: update hardware ingest contract to v2.11
2026-06-19 18:39:28 +03:00 · 2026-06-19 18:28:38 +03:00 · 2026-06-19 15:21:35 +03:00 · 2026-06-19 15:20:57 +03:00 · 2026-06-19 15:13:42 +03:00 · 2026-06-19 15:11:46 +03:00
47 changed files with 3869 additions and 1044 deletions
--- a/2
+++ b/2
--- a/bible-local/07-exporters.md
+++ b/bible-local/07-exporters.md
@@ -7,6 +7,7 @@
 | `GET /api/export/csv` | CSV | Serial-number export |
 | `GET /api/export/json` | raw-export ZIP bundle | Reopen and re-analyze later |
 | `GET /api/export/reanimator` | JSON | Reanimator hardware payload |
+| `GET /chart/current?print=true` | HTML (auto-print) | Print/PDF version of the report — opens in new tab, calls `window.print()` |
 | `POST /api/convert` | async ZIP artifact | Batch archive-to-Reanimator conversion |

 ## Raw export
--- a/bible-local/BACKLOG.md
+++ b/bible-local/BACKLOG.md
@@ -0,0 +1,21 @@
+# Backlog
+
+## [sfp_modules] Поддержка per-port SFP/QSFP модулей в экспорте Reanimator
+
+**Приоритет:** низкий (до выхода Reanimator v3.0, пока deprecated sfp_* скаляры ещё принимаются)
+
+**Контекст:**
+Reanimator Hardware Ingest Contract v2.11 вводит массив `pcie_devices[].sfp_modules[]` для передачи данных SFP/QSFP-модулей по портам. Старые скалярные поля (`sfp_temperature_c`, `sfp_tx_power_dbm`, `sfp_rx_power_dbm`, `sfp_voltage_v`, `sfp_bias_ma`) помечены deprecated и будут удалены в v3.0. Для многопортовых NIC (ConnectX-6 Dx, Intel X710 и подобных) текущая реализация теряет данные — коллектор берёт первое найденное значение и не знает о портах.
+
+**Текущее состояние:**
+- Коллектор (`internal/collector/redfish.go`, `redfishPCIeDetailsWithSupplementalDocs`) собирает SFP как 5 скалярных `float64` на устройство через `redfishFirstNumericAcrossDocs`
+- Внутренняя модель (`internal/models/models.go`, struct `PCIeDevice`) не имеет SFP-полей — всё хранится в `Details map[string]any`
+- Конвертер (`internal/exporter/reanimator_converter.go`, строки 864–868) читает скаляры из `Details` и кладёт в deprecated поля `ReanimatorPCIe`
+
+**Что нужно сделать:**
+1. **Исследование** — проверить, отдают ли реальные Redfish-источники SFP-данные per-port и в каком виде (прежде чем менять модель)
+2. **Коллектор** (`redfish.go`) — если Redfish отдаёт per-port данные, собирать их в массив с индексом порта
+3. **Внутренняя модель** (`models.go`) — добавить `SFPModules []SFPModule` в `PCIeDevice`
+4. **Экспорт** (`reanimator_models.go`, `reanimator_converter.go`) — добавить `ReanimatorSFPModule`, смапить `SFPModules` в `sfp_modules[]`; убрать deprecated скаляры
+
+**Триггер для реализации:** анонс Reanimator v3.0 с удалением deprecated sfp_* полей.
--- a/bible-local/docs/hardware-ingest-contract.md
+++ b/bible-local/docs/hardware-ingest-contract.md
@@ -1,7 +1,7 @@
 ---
 title: Hardware Ingest JSON Contract
-version: "2.7"
-updated: "2026-03-15"
+version: "2.11"
+updated: "2026-06-19"
 maintainer: Reanimator Core
 audience: external-integrators, ai-agents
 language: ru
@@ -9,7 +9,7 @@ language: ru

 # Интеграция с Reanimator: контракт JSON-импорта аппаратного обеспечения

-Версия: **2.7** · Дата: **2026-03-15**
+Версия: **2.11** · Дата: **2026-06-19**

 Документ описывает формат JSON для передачи данных об аппаратном обеспечении серверов в систему **Reanimator** (управление жизненным циклом аппаратного обеспечения).
 Предназначен для разработчиков смежных систем (Redfish-коллекторов, агентов мониторинга, CMDB-экспортёров) и может быть включён в документацию интегрируемых проектов.
@@ -22,6 +22,10 @@ language: ru

 | Версия | Дата | Изменения |
 |--------|------|-----------|
+| 2.11 | 2026-06-19 | В `pcie_devices[]` добавлен необязательный массив `sfp_modules[]` с идентификацией и DOM telemetry SFP/QSFP-модулей. Скалярные поля `sfp_temperature_c` / `sfp_tx_power_dbm` / `sfp_rx_power_dbm` / `sfp_voltage_v` / `sfp_bias_ma` помечены как deprecated (принимаются, но `sfp_modules[]` имеет приоритет) |
+| 2.10 | 2026-04-29 | Для `hardware.storage[]` добавлены необязательные числовые поля `logical_block_size_bytes`, `physical_block_size_bytes`, `metadata_bytes_per_block` для нормализованного описания формата блока накопителя |
+| 2.9 | 2026-03-19 | Добавлена необязательная секция `hardware.platform_config` — произвольный объект с настройками платформы (BIOS/Redfish); хранится как latest-snapshot per machine |
+| 2.8 | 2026-03-15 | Поле `location` удалено из всех `sensors.*`; сенсоры передаются только по `name` и измеренным значениям |
 | 2.7 | 2026-03-15 | Явно запрещён синтез данных в `event_logs`; интеграторы не должны придумывать серийные номера компонентов, если источник их не отдал |
 | 2.6 | 2026-03-15 | Добавлена необязательная секция `event_logs` для dedup/upsert логов `host` / `bmc` / `redfish` вне history timeline |
 | 2.5 | 2026-03-15 | Добавлено общее необязательное поле `manufactured_year_week` для компонентных секций (`YYYY-Www`) |
@@ -131,8 +135,9 @@ GET /ingest/hardware/jobs/{job_id}
    "storage":        [ ... ],
    "pcie_devices":   [ ... ],
    "power_supplies": [ ... ],
-    "sensors":        { ... },
-    "event_logs":     [ ... ]
+    "sensors":         { ... },
+    "event_logs":      [ ... ],
+    "platform_config": { ... }
  }
 }
 ```
@@ -343,6 +348,9 @@ GET /ingest/hardware/jobs/{job_id}
 | `type` | string | нет | Тип: `NVMe`, `SSD`, `HDD` |
 | `interface` | string | нет | Интерфейс: `NVMe`, `SATA`, `SAS` |
 | `size_gb` | int | нет | Размер в ГБ |
+| `logical_block_size_bytes` | int64 | нет | Логический размер пользовательского блока данных, например `512` или `4096` |
+| `physical_block_size_bytes` | int64 | нет | Физический размер блока, если известен, например `4096` |
+| `metadata_bytes_per_block` | int64 | нет | Metadata / protection bytes на логический блок, например `0` или `8` |
 | `temperature_c` | float | нет | Температура накопителя, °C (telemetry) |
 | `power_on_hours` | int64 | нет | Время работы, часы |
 | `power_cycles` | int64 | нет | Количество циклов питания |
@@ -363,6 +371,11 @@ GET /ingest/hardware/jobs/{job_id}

 Диск без `serial_number` игнорируется. Изменение `firmware` создаёт событие `FIRMWARE_CHANGED`.

+Формат вида `512+8` в контракт не добавляется отдельным строковым полем. Если источник знает такую форму, он должен передавать её как:
+- `logical_block_size_bytes = 512`
+- `metadata_bytes_per_block = 8`
+- `physical_block_size_bytes = 512` или `4096`, если известен физический размер блока
+
 ```json
 "storage": [
  {
@@ -370,6 +383,9 @@ GET /ingest/hardware/jobs/{job_id}
    "type": "NVMe",
    "model": "INTEL SSDPF2KX076T1",
    "size_gb": 7680,
+    "logical_block_size_bytes": 512,
+    "physical_block_size_bytes": 4096,
+    "metadata_bytes_per_block": 8,
    "temperature_c": 38.5,
    "power_on_hours": 12450,
    "unsafe_shutdowns": 3,
@@ -407,11 +423,12 @@ GET /ingest/hardware/jobs/{job_id}
 | `battery_temperature_c` | float | нет | Температура батареи / supercap, °C |
 | `battery_voltage_v` | float | нет | Напряжение батареи / supercap, В |
 | `battery_replace_required` | bool | нет | Требуется замена батареи / supercap |
-| `sfp_temperature_c` | float | нет | Температура SFP/optic, °C |
-| `sfp_tx_power_dbm` | float | нет | TX optical power, dBm |
-| `sfp_rx_power_dbm` | float | нет | RX optical power, dBm |
-| `sfp_voltage_v` | float | нет | Напряжение SFP, В |
-| `sfp_bias_ma` | float | нет | Bias current SFP, мА |
+| `sfp_temperature_c` | float | нет | Температура SFP/optic, °C *(deprecated since 2.11)* |
+| `sfp_tx_power_dbm` | float | нет | TX optical power, dBm *(deprecated since 2.11)* |
+| `sfp_rx_power_dbm` | float | нет | RX optical power, dBm *(deprecated since 2.11)* |
+| `sfp_voltage_v` | float | нет | Напряжение SFP, В *(deprecated since 2.11)* |
+| `sfp_bias_ma` | float | нет | Bias current SFP, мА *(deprecated since 2.11)* |
+| `sfp_modules` | array | нет | Установленные SFP/QSFP-модули по портам (см. sfp_modules[]) |
 | `bdf` | string | нет | Deprecated alias для `slot`; при наличии ingest нормализует его в `slot` |
 | `device_class` | string | нет | Класс устройства (см. список ниже) |
 | `manufacturer` | string | нет | Производитель |
@@ -429,10 +446,43 @@ GET /ingest/hardware/jobs/{job_id}
 `numa_node` передавайте для NIC / InfiniBand / RAID / GPU, когда источник знает CPU/NUMA affinity. Поле сохраняется в snapshot-атрибутах PCIe-компонента и дублируется в telemetry для topology use cases.
 Поля `temperature_c` и `power_w` используйте для device-level telemetry GPU / accelerator / smart PCIe devices. Они не влияют на идентификацию компонента.

+**Deprecated поля sfp_\*:** Скалярные поля `sfp_temperature_c`, `sfp_tx_power_dbm`, `sfp_rx_power_dbm`, `sfp_voltage_v`, `sfp_bias_ma` продолжают приниматься, но помечены как deprecated since 2.11. Если в payload одновременно присутствуют `sfp_modules[]` и deprecated sfp_-скаляры — приоритет у `sfp_modules[]`, скаляры игнорируются. Deprecated поля будут удалены в версии 3.0.
+
 **Генерация serial_number при отсутствии или `"N/A"`:** `{board_serial}-PCIE-{slot}`, где `slot` для PCIe равен BDF.

 `slot` — единственный канонический адрес компонента. Для PCIe в `slot` передавайте BDF. Поле `bdf` сохраняется только как переходный alias на входе и не должно использоваться как отдельная координата рядом со `slot`.

+#### pcie_devices[].sfp_modules[]
+
+Необязательный массив установленных SFP/QSFP-модулей для данного PCIe-устройства. Один элемент — один порт. Используйте для многопортовых NIC (ConnectX-6 Dx, Intel X710, Mellanox HDR и др.).
+
+| Поле | Тип | Обязательно | Описание |
+|------|-----|-------------|----------|
+| `port` | int | **да** | Номер порта на NIC (0-based). Ключ дедупликации внутри устройства |
+| `identifier` | string | нет | Тип модуля: `SFP`, `SFP+`, `SFP28`, `QSFP+`, `QSFP28`, `QSFP-DD`, `DAC` |
+| `connector` | string | нет | Тип разъёма: `LC`, `MPO`, `RJ45`, `DAC`, `AOC`, `No separable connector` |
+| `vendor` | string | нет | Производитель модуля из EEPROM |
+| `part_number` | string | нет | Партномер из EEPROM |
+| `serial_number` | string | нет | Серийный номер из EEPROM |
+| `revision` | string | нет | Ревизия из EEPROM |
+| `wavelength_nm` | int | нет | Длина волны, нм (0 для DAC/медных кабелей) |
+| `transceiver_type` | string | нет | `10GBase-SR`, `10GBase-LR`, `25GBase-SR`, `100GBase-SR4`, `DAC`, … |
+| `temperature_c` | float | нет | Температура модуля, °C (DOM telemetry) |
+| `voltage_v` | float | нет | Напряжение питания, В (DOM telemetry) |
+| `tx_power_dbm` | float | нет | TX оптическая мощность, dBm (DOM telemetry) |
+| `rx_power_dbm` | float | нет | RX оптическая мощность, dBm (DOM telemetry) |
+| `bias_ma` | float | нет | Bias current, мА (DOM telemetry) |
+
+**Ключ дедупликации:** `(pcie_devices[].slot, sfp_modules[].port)`.
+
+**Правила ingest:**
+- При каждом импорте — полная замена `sfp_modules[]` для данного `pcie_devices[].slot` (upsert всего массива целиком).
+- Если `sfp_modules` отсутствует или `null` — существующие данные SFP не трогать.
+- Если `sfp_modules: []` (пустой массив) — трактовать как «модули не обнаружены», очистить сохранённые данные.
+- Дубли по `port` внутри одного `pcie_devices[]` — невалидны, endpoint возвращает `400` с описанием поля.
+- Модули без `serial_number` допустимы (многие DAC-кабели не имеют SN); сохраняются по ключу `(slot, port)`.
+- Изменение `serial_number` или `part_number` модуля на порту создаёт событие `COMPONENT_CHANGED` для PCIe-устройства с описанием «SFP module replaced on port N».
+
 **Значения `device_class`:**

 | Значение | Назначение |
@@ -457,16 +507,47 @@ GET /ingest/hardware/jobs/{job_id}
    "numa_node": 0,
    "temperature_c": 48.5,
    "power_w": 18.2,
-    "sfp_temperature_c": 36.2,
-    "sfp_tx_power_dbm": -1.8,
-    "sfp_rx_power_dbm": -2.1,
    "device_class": "EthernetController",
-    "manufacturer": "Intel",
-    "model": "X710 10GbE",
-    "serial_number": "K65472-003",
-    "firmware": "9.20 0x8000d4ae",
+    "manufacturer": "Mellanox",
+    "model": "ConnectX-6 Dx",
+    "serial_number": "MT2012X12345",
+    "firmware": "22.35.2010",
    "mac_addresses": ["3c:fd:fe:aa:bb:cc", "3c:fd:fe:aa:bb:cd"],
-    "status": "OK"
+    "status": "OK",
+    "sfp_modules": [
+      {
+        "port": 0,
+        "identifier": "QSFP28",
+        "connector": "LC",
+        "vendor": "Mellanox",
+        "part_number": "MFA1A00-C003",
+        "serial_number": "MT2124VS09999",
+        "revision": "A",
+        "wavelength_nm": 850,
+        "transceiver_type": "100GBase-SR4",
+        "temperature_c": 36.4,
+        "voltage_v": 3.29,
+        "tx_power_dbm": -1.8,
+        "rx_power_dbm": -2.1,
+        "bias_ma": 7.2
+      },
+      {
+        "port": 1,
+        "identifier": "QSFP28",
+        "connector": "LC",
+        "vendor": "Mellanox",
+        "part_number": "MFA1A00-C003",
+        "serial_number": "MT2124VS09998",
+        "revision": "A",
+        "wavelength_nm": 850,
+        "transceiver_type": "100GBase-SR4",
+        "temperature_c": 35.9,
+        "voltage_v": 3.28,
+        "tx_power_dbm": -1.9,
+        "rx_power_dbm": -2.3,
+        "bias_ma": 7.1
+      }
+    ]
  }
 ]
 ```
@@ -592,7 +673,6 @@ PSU без `serial_number` игнорируется.
 | Поле | Тип | Обязательно | Описание |
 |------|-----|-------------|----------|
 | `name` | string | **да** | Уникальное имя сенсора в рамках секции |
-| `location` | string | нет | Физическое расположение |
 | `rpm` | int | нет | Обороты, RPM |
 | `status` | string | нет | Статус: `OK`, `Warning`, `Critical`, `Unknown` |

@@ -601,7 +681,6 @@ PSU без `serial_number` игнорируется.
 | Поле | Тип | Обязательно | Описание |
 |------|-----|-------------|----------|
 | `name` | string | **да** | Уникальное имя сенсора |
-| `location` | string | нет | Физическое расположение |
 | `voltage_v` | float | нет | Напряжение, В |
 | `current_a` | float | нет | Ток, А |
 | `power_w` | float | нет | Мощность, Вт |
@@ -612,7 +691,6 @@ PSU без `serial_number` игнорируется.
 | Поле | Тип | Обязательно | Описание |
 |------|-----|-------------|----------|
 | `name` | string | **да** | Уникальное имя сенсора |
-| `location` | string | нет | Физическое расположение |
 | `celsius` | float | нет | Температура, °C |
 | `threshold_warning_celsius` | float | нет | Порог Warning, °C |
 | `threshold_critical_celsius` | float | нет | Порог Critical, °C |
@@ -623,38 +701,63 @@ PSU без `serial_number` игнорируется.
 | Поле | Тип | Обязательно | Описание |
 |------|-----|-------------|----------|
 | `name` | string | **да** | Уникальное имя сенсора |
-| `location` | string | нет | Физическое расположение |
 | `value` | float | нет | Значение |
 | `unit` | string | нет | Единица измерения |
 | `status` | string | нет | Статус |

 **Правила sensors:**
 - Идентификатор сенсора: пара `(sensor_type, name)`. Дубли в одном payload — берётся первое вхождение.
+- `location` для сенсоров передавать не нужно и не следует: в Reanimator location/slot используется только для проверки перемещения и установки компонентов, а не для last-known-value sensor ingest.
 - Сенсоры без `name` игнорируются.
 - При каждом импорте значения перезаписываются (upsert по ключу).

 ```json
 "sensors": {
  "fans": [
-    { "name": "FAN1", "location": "Front", "rpm": 4200, "status": "OK" },
-    { "name": "FAN_CPU0", "location": "CPU0", "rpm": 5600, "status": "OK" }
+    { "name": "FAN1", "rpm": 4200, "status": "OK" },
+    { "name": "FAN_CPU0", "rpm": 5600, "status": "OK" }
  ],
  "power": [
-    { "name": "12V Rail", "location": "Mainboard", "voltage_v": 12.06, "status": "OK" },
-    { "name": "PSU0 Input", "location": "PSU0", "voltage_v": 215.25, "current_a": 0.64, "power_w": 137.0, "status": "OK" }
+    { "name": "12V Rail", "voltage_v": 12.06, "status": "OK" },
+    { "name": "PSU0 Input", "voltage_v": 215.25, "current_a": 0.64, "power_w": 137.0, "status": "OK" }
  ],
  "temperatures": [
-    { "name": "CPU0 Temp", "location": "CPU0", "celsius": 46.0, "threshold_warning_celsius": 80.0, "threshold_critical_celsius": 95.0, "status": "OK" },
-    { "name": "Inlet Temp", "location": "Front", "celsius": 22.0, "threshold_warning_celsius": 40.0, "threshold_critical_celsius": 50.0, "status": "OK" }
+    { "name": "CPU0 Temp", "celsius": 46.0, "threshold_warning_celsius": 80.0, "threshold_critical_celsius": 95.0, "status": "OK" },
+    { "name": "Inlet Temp", "celsius": 22.0, "threshold_warning_celsius": 40.0, "threshold_critical_celsius": 50.0, "status": "OK" }
  ],
  "other": [
-    { "name": "System Humidity", "value": 38.5, "unit": "%" , "status": "OK" }
+    { "name": "System Humidity", "value": 38.5, "unit": "%", "status": "OK" }
  ]
 }
 ```

 ---

+## Секция platform_config
+
+Необязательный объект с произвольными ключами — настройки платформы как есть из источника (BIOS, Redfish, IPMI).
+
+| Поле | Тип | Обязательно | Описание |
+|------|-----|-------------|----------|
+| `platform_config` | object | нет | Произвольный объект: ключи — строки, значения — строки, числа, булевы |
+
+**Правила platform_config:**
+- Содержимое объекта не валидируется: передавайте параметры как есть.
+- При каждом импорте хранится latest-snapshot per machine; история изменений по каждому ключу накапливается отдельно.
+- Если секция отсутствует или равна `null` — данные платформы не обновляются.
+
+```json
+"platform_config": {
+  "SecureBoot": "Enabled",
+  "BiosVersion": "06.08.05",
+  "TpmEnabled": true,
+  "NumaEnabled": false,
+  "HyperThreading": "Enabled"
+}
+```
+
+---
+
 ## Обработка статусов компонентов

 | Статус | Поведение |
@@ -756,7 +859,24 @@ PSU без `serial_number` игнорируется.
        "model": "X710 10GbE",
        "serial_number": "K65472-003",
        "mac_addresses": ["3c:fd:fe:aa:bb:cc", "3c:fd:fe:aa:bb:cd"],
-        "status": "OK"
+        "status": "OK",
+        "sfp_modules": [
+          {
+            "port": 0,
+            "identifier": "SFP+",
+            "connector": "LC",
+            "vendor": "Intel",
+            "part_number": "FTLX8574D3BCV-IT",
+            "serial_number": "FNS123456789",
+            "wavelength_nm": 850,
+            "transceiver_type": "10GBase-SR",
+            "temperature_c": 34.1,
+            "voltage_v": 3.30,
+            "tx_power_dbm": -2.5,
+            "rx_power_dbm": -3.0,
+            "bias_ma": 6.8
+          }
+        ]
      }
    ],
    "power_supplies": [
@@ -787,6 +907,12 @@ PSU без `serial_number` игнорируется.
      "other": [
        { "name": "System Humidity", "value": 38.5, "unit": "%" }
      ]
+    },
+    "platform_config": {
+      "SecureBoot": "Enabled",
+      "BiosVersion": "06.08.05",
+      "TpmEnabled": true,
+      "HyperThreading": "Enabled"
    }
  }
 }
--- a/cmd/logpile/main.go
+++ b/cmd/logpile/main.go
@@ -4,7 +4,7 @@ import (
 	"bufio"
 	"flag"
 	"fmt"
-	"log"
+	"log/slog"
 	"os"
 	"os/exec"
 	"runtime"
@@ -49,8 +49,8 @@ func main() {
 	srv := server.New(cfg)

 	url := fmt.Sprintf("http://localhost:%d", *port)
-	log.Printf("LOGPile starting on %s", url)
-	log.Printf("Registered parsers: %v", parser.ListParsers())
+	slog.Info("LOGPile starting", "url", url)
+	slog.Info("registered parsers", "parsers", parser.ListParsers())

 	// Open browser automatically
 	if !*noBrowser {
@@ -61,7 +61,7 @@ func main() {
 	}

 	if err := runServer(srv); err != nil {
-		log.Printf("FATAL: %v", err)
+		slog.Error("fatal error", "err", err)
 		maybeWaitForCrashInput(*holdOnCrash)
 		os.Exit(1)
 	}
@@ -90,7 +90,7 @@ func openBrowser(url string) {
 	}

 	if err := cmd.Start(); err != nil {
-		log.Printf("Failed to open browser: %v", err)
+		slog.Warn("failed to open browser", "err", err)
 	}
 }

--- a/internal/chart
+++ b/internal/chart
--- a/internal/collector/context_sleep.go
+++ b/internal/collector/context_sleep.go
--- a/internal/collector/ipmi_mock.go
+++ b/internal/collector/ipmi_mock.go
@@ -19,9 +19,9 @@ func (c *IPMIMockConnector) Protocol() string {

 func (c *IPMIMockConnector) Collect(ctx context.Context, req Request, emit ProgressFn) (*models.AnalysisResult, error) {
 	steps := []Progress{
-		{Status: "running", Progress: 20, Message: "IPMI: подключение к BMC..."},
-		{Status: "running", Progress: 55, Message: "IPMI: чтение инвентаря..."},
-		{Status: "running", Progress: 85, Message: "IPMI: нормализация данных..."},
+		{Status: "running", Progress: 20, Message: "IPMI: connecting to BMC..."},
+		{Status: "running", Progress: 55, Message: "IPMI: reading inventory..."},
+		{Status: "running", Progress: 85, Message: "IPMI: normalizing data..."},
 	}

 	for _, step := range steps {
--- a/internal/collector/redfish.go
+++ b/internal/collector/redfish.go
@@ -6,7 +6,7 @@ import (
 	"encoding/json"
 	"fmt"
 	"io"
-	"log"
+	"log/slog"
 	"net/http"
 	"net/url"
 	"os"
@@ -124,14 +124,14 @@ func (c *RedfishConnector) debugf(format string, args ...interface{}) {
 	if !c.debug {
 		return
 	}
-	log.Printf("redfish-debug: "+format, args...)
+	slog.Debug("redfish-debug: " + fmt.Sprintf(format, args...))
 }

 func (c *RedfishConnector) debugSnapshotf(format string, args ...interface{}) {
 	if !c.debugSnapshot {
 		return
 	}
-	log.Printf("redfish-snapshot-debug: "+format, args...)
+	slog.Debug("redfish-snapshot-debug: " + fmt.Sprintf(format, args...))
 }

 func (c *RedfishConnector) Collect(ctx context.Context, req Request, emit ProgressFn) (*models.AnalysisResult, error) {
@@ -149,7 +149,7 @@ func (c *RedfishConnector) Collect(ctx context.Context, req Request, emit Progre
 	hintClient := c.httpClientWithTimeout(req, 4*time.Second)

 	if emit != nil {
-		emit(Progress{Status: "running", Progress: 10, Message: "Redfish: подключение к BMC..."})
+		emit(Progress{Status: "running", Progress: 10, Message: "Redfish: connecting to BMC..."})
 	}
 	discoveryCtx := withRedfishTelemetryPhase(ctx, "discovery")
 	serviceRootDoc, err := c.getJSON(discoveryCtx, snapshotClient, req, baseURL, "/redfish/v1")
@@ -192,7 +192,7 @@ func (c *RedfishConnector) Collect(ctx context.Context, req Request, emit Progre
 		emit(Progress{
 			Status:        "running",
 			Progress:      25,
-			Message:       fmt.Sprintf("Redfish: профили mode=%s active=%s", acquisitionPlan.Mode, formatActiveModuleLog(activeModules)),
+			Message:       fmt.Sprintf("Redfish: profiles mode=%s active=%s", acquisitionPlan.Mode, formatActiveModuleLog(activeModules)),
 			ActiveModules: activeModules,
 			ModuleScores:  moduleScores,
 			DebugInfo: &CollectDebugInfo{
@@ -229,33 +229,32 @@ func (c *RedfishConnector) Collect(ctx context.Context, req Request, emit Progre
 	seedPaths := resolvedPlan.SeedPaths
 	criticalPaths := resolvedPlan.CriticalPaths
 	if len(acquisitionPlan.Profiles) > 0 {
-		log.Printf(
-			"redfish-profile-plan: mode=%s profiles=%s notes=%s scores=%s req=%d err=%d p95=%dms avg=%dms throttled=%t",
-			acquisitionPlan.Mode,
-			strings.Join(acquisitionPlan.Profiles, ","),
-			strings.Join(acquisitionPlan.Notes, "; "),
-			formatModuleScoreLog(moduleScores),
-			telemetrySummary.Requests,
-			telemetrySummary.Errors,
-			telemetrySummary.P95.Milliseconds(),
-			telemetrySummary.Avg.Milliseconds(),
-			throttled,
+		slog.Info("redfish-profile-plan",
+			"mode", acquisitionPlan.Mode,
+			"profiles", strings.Join(acquisitionPlan.Profiles, ","),
+			"notes", strings.Join(acquisitionPlan.Notes, "; "),
+			"scores", formatModuleScoreLog(moduleScores),
+			"req", telemetrySummary.Requests,
+			"err", telemetrySummary.Errors,
+			"p95_ms", telemetrySummary.P95.Milliseconds(),
+			"avg_ms", telemetrySummary.Avg.Milliseconds(),
+			"throttled", throttled,
 		)
 	}
 	if emit != nil {
 		emit(Progress{
 			Status:       "running",
 			Progress:     30,
-			Message:      "Redfish: чтение структуры Redfish...",
+			Message:      "Redfish: reading Redfish structure...",
 			CurrentPhase: "snapshot",
 			ETASeconds:   acquisitionPlan.Tuning.ETABaseline.SnapshotSeconds,
 		})
 	}

 	if emit != nil {
-		emit(Progress{Status: "running", Progress: 55, Message: "Redfish: подготовка snapshot...", CurrentPhase: "snapshot", ETASeconds: acquisitionPlan.Tuning.ETABaseline.SnapshotSeconds})
-		emit(Progress{Status: "running", Progress: 80, Message: "Redfish: подготовка расширенного snapshot...", CurrentPhase: "snapshot", ETASeconds: acquisitionPlan.Tuning.ETABaseline.SnapshotSeconds})
-		emit(Progress{Status: "running", Progress: 90, Message: "Redfish: сбор расширенного snapshot...", CurrentPhase: "snapshot", ETASeconds: acquisitionPlan.Tuning.ETABaseline.SnapshotSeconds})
+		emit(Progress{Status: "running", Progress: 55, Message: "Redfish: preparing snapshot...", CurrentPhase: "snapshot", ETASeconds: acquisitionPlan.Tuning.ETABaseline.SnapshotSeconds})
+		emit(Progress{Status: "running", Progress: 80, Message: "Redfish: preparing extended snapshot...", CurrentPhase: "snapshot", ETASeconds: acquisitionPlan.Tuning.ETABaseline.SnapshotSeconds})
+		emit(Progress{Status: "running", Progress: 90, Message: "Redfish: collecting extended snapshot...", CurrentPhase: "snapshot", ETASeconds: acquisitionPlan.Tuning.ETABaseline.SnapshotSeconds})
 	}
 	// collectCtx covers all data-fetching phases (snapshot, prefetch, plan-B).
 	// Cancelling it via the skip signal aborts only the collection phases while
@@ -270,10 +269,10 @@ func (c *RedfishConnector) Collect(ctx context.Context, req Request, emit Progre
 					emit(Progress{
 						Status:   "running",
 						Progress: 97,
-						Message:  "Redfish: пропуск зависших запросов, анализ уже собранных данных...",
+						Message:  "Redfish: skipping stalled requests, analyzing collected data...",
 					})
 				}
-				log.Printf("redfish: skip-hung triggered, cancelling collection phases")
+				slog.Info("redfish: skip-hung triggered, cancelling collection phases")
 				cancelCollect()
 			case <-ctx.Done():
 			}
@@ -296,15 +295,14 @@ func (c *RedfishConnector) Collect(ctx context.Context, req Request, emit Progre
 	for p := range prefetchedCritical {
 		delete(fetchErrMap, p)
 	}
-	log.Printf(
-		"redfish-prefetch-metrics: enabled=%t candidates=%d targets=%d docs=%d added=%d dur=%s skip=%s",
-		prefetchMetrics.Enabled,
-		prefetchMetrics.Candidates,
-		prefetchMetrics.Targets,
-		prefetchMetrics.Docs,
-		prefetchMetrics.Added,
-		prefetchMetrics.Duration.Round(time.Millisecond),
-		firstNonEmpty(prefetchMetrics.SkipReason, "-"),
+	slog.Info("redfish-prefetch-metrics",
+		"enabled", prefetchMetrics.Enabled,
+		"candidates", prefetchMetrics.Candidates,
+		"targets", prefetchMetrics.Targets,
+		"docs", prefetchMetrics.Docs,
+		"added", prefetchMetrics.Added,
+		"dur", prefetchMetrics.Duration.Round(time.Millisecond),
+		"skip", firstNonEmpty(prefetchMetrics.SkipReason, "-"),
 	)
 	if recoveredN := c.recoverCriticalRedfishDocsPlanB(withRedfishTelemetryPhase(collectCtx, "critical_plan_b"), criticalClient, req, baseURL, criticalPaths, rawTree, fetchErrMap, acquisitionPlan.Tuning, emit); recoveredN > 0 {
 		c.debugSnapshotf("critical plan-b recovered docs=%d", recoveredN)
@@ -319,7 +317,7 @@ func (c *RedfishConnector) Collect(ctx context.Context, req Request, emit Progre
 		}
 	}
 	if emit != nil {
-		emit(Progress{Status: "running", Progress: 99, Message: "Redfish: анализ raw snapshot..."})
+		emit(Progress{Status: "running", Progress: 99, Message: "Redfish: analyzing raw snapshot..."})
 	}
 	// Collect hardware event logs separately (not part of tree-walk to avoid bloat).
 	rawLogEntries := c.collectRedfishLogEntries(withRedfishTelemetryPhase(ctx, "log_entries"), snapshotClient, req, baseURL, systemPaths, managerPaths)
@@ -443,38 +441,36 @@ func (c *RedfishConnector) Collect(ctx context.Context, req Request, emit Progre
 	}
 	totalElapsed := time.Since(collectStart).Round(time.Second)
 	if !result.InventoryLastModifiedAt.IsZero() {
-		log.Printf("redfish-collect: inventory last modified at %s (age: %s)",
-			result.InventoryLastModifiedAt.Format(time.RFC3339),
-			time.Since(result.InventoryLastModifiedAt).Round(time.Minute),
+		slog.Info("redfish-collect: inventory last modified",
+			"at", result.InventoryLastModifiedAt.Format(time.RFC3339),
+			"age", time.Since(result.InventoryLastModifiedAt).Round(time.Minute),
 		)
 	}
-	log.Printf(
-		"redfish-postprobe-metrics: nvme_candidates=%d nvme_selected=%d nvme_added=%d candidates=%d selected=%d skipped_explicit=%d added=%d dur=%s",
-		postProbeMetrics.NVMECandidates,
-		postProbeMetrics.NVMESelected,
-		postProbeMetrics.NVMEAdded,
-		postProbeMetrics.CollectionCandidates,
-		postProbeMetrics.CollectionSelected,
-		postProbeMetrics.SkippedExplicit,
-		postProbeMetrics.Added,
-		postProbeMetrics.Duration.Round(time.Millisecond),
+	slog.Info("redfish-postprobe-metrics",
+		"nvme_candidates", postProbeMetrics.NVMECandidates,
+		"nvme_selected", postProbeMetrics.NVMESelected,
+		"nvme_added", postProbeMetrics.NVMEAdded,
+		"candidates", postProbeMetrics.CollectionCandidates,
+		"selected", postProbeMetrics.CollectionSelected,
+		"skipped_explicit", postProbeMetrics.SkippedExplicit,
+		"added", postProbeMetrics.Added,
+		"dur", postProbeMetrics.Duration.Round(time.Millisecond),
 	)
-	log.Printf(
-		"redfish-telemetry: req=%d err=%d err_rate=%.2f avg=%dms p95=%dms throttled=%t snapshot_workers=%d prefetch_workers=%d timing_top=%s",
-		telemetrySummary.Requests,
-		telemetrySummary.Errors,
-		telemetrySummary.ErrorRate,
-		telemetrySummary.Avg.Milliseconds(),
-		telemetrySummary.P95.Milliseconds(),
-		throttled,
-		acquisitionPlan.Tuning.SnapshotWorkers,
-		acquisitionPlan.Tuning.PrefetchWorkers,
-		firstNonEmpty(snapshotTimingSummary, "-"),
+	slog.Info("redfish-telemetry",
+		"req", telemetrySummary.Requests,
+		"err", telemetrySummary.Errors,
+		"err_rate", telemetrySummary.ErrorRate,
+		"avg_ms", telemetrySummary.Avg.Milliseconds(),
+		"p95_ms", telemetrySummary.P95.Milliseconds(),
+		"throttled", throttled,
+		"snapshot_workers", acquisitionPlan.Tuning.SnapshotWorkers,
+		"prefetch_workers", acquisitionPlan.Tuning.PrefetchWorkers,
+		"timing_top", firstNonEmpty(snapshotTimingSummary, "-"),
 	)
 	for _, line := range redfishPhaseTelemetryLogLines(phaseTelemetry) {
-		log.Printf("redfish-telemetry-phase: %s", line)
+		slog.Info("redfish-telemetry-phase", "line", line)
 	}
-	log.Printf("redfish-collect: completed in %s (docs=%d, fetch_errors=%d)", totalElapsed, len(rawTree), len(fetchErrMap))
+	slog.Info("redfish-collect: completed", "elapsed", totalElapsed, "docs", len(rawTree), "fetch_errors", len(fetchErrMap))
 	if emit != nil {
 		emit(Progress{
 			Status:   "running",
@@ -491,7 +487,7 @@ func (c *RedfishConnector) Collect(ctx context.Context, req Request, emit Progre
 		emit(Progress{
 			Status:   "running",
 			Progress: 100,
-			Message:  fmt.Sprintf("Redfish: сбор завершен за %s", totalElapsed),
+			Message:  fmt.Sprintf("Redfish: collection completed in %s", totalElapsed),
 		})
 	}
 	return result, nil
@@ -611,7 +607,7 @@ func (c *RedfishConnector) prefetchCriticalRedfishDocs(
 			emit(Progress{
 				Status:   "running",
 				Progress: 96,
-				Message:  fmt.Sprintf("Redfish: prefetch пропущен (адаптивно, кандидатов=%d)", metrics.Candidates),
+				Message:  fmt.Sprintf("Redfish: prefetch skipped (adaptive, candidates=%d)", metrics.Candidates),
 			})
 		}
 		return nil, metrics
@@ -620,7 +616,7 @@ func (c *RedfishConnector) prefetchCriticalRedfishDocs(
 		emit(Progress{
 			Status:       "running",
 			Progress:     96,
-			Message:      fmt.Sprintf("Redfish: prefetch критичных endpoint (адаптивно %d/%d)...", len(targets), len(candidates)),
+			Message:      fmt.Sprintf("Redfish: prefetch critical endpoints (adaptive %d/%d)...", len(targets), len(candidates)),
 			CurrentPhase: "prefetch",
 			ETASeconds:   int(estimateProgressETA(time.Now(), 0, len(targets), 2*time.Second).Seconds()),
 		})
@@ -706,7 +702,7 @@ func (c *RedfishConnector) prefetchCriticalRedfishDocs(
 		emit(Progress{
 			Status:       "running",
 			Progress:     96,
-			Message:      fmt.Sprintf("Redfish: prefetch завершен (адаптивно targets=%d, docs=%d)", len(targets), len(out)),
+			Message:      fmt.Sprintf("Redfish: prefetch completed (adaptive targets=%d, docs=%d)", len(targets), len(out)),
 			CurrentPhase: "prefetch",
 		})
 	}
@@ -1397,7 +1393,7 @@ func (c *RedfishConnector) collectRawRedfishTree(ctx context.Context, client *ht
 					emit(Progress{
 						Status:       "running",
 						Progress:     92 + int(minInt32(n/200, 6)),
-						Message:      fmt.Sprintf("Redfish snapshot: heartbeat документов=%d (ok=%d, seen=%d), ETA≈%s, корни=%s, последний=%s", n, outN, seenN, eta, strings.Join(roots, ", "), compactProgressPath(last)),
+						Message:      fmt.Sprintf("Redfish snapshot: heartbeat docs=%d (ok=%d, seen=%d), ETA≈%s, roots=%s, last=%s", n, outN, seenN, eta, strings.Join(roots, ", "), compactProgressPath(last)),
 						CurrentPhase: "snapshot",
 						ETASeconds:   int(estimateSnapshotETA(crawlStart, int(n), seenN, len(jobs), workers, client.Timeout).Seconds()),
 					})
@@ -1434,7 +1430,7 @@ func (c *RedfishConnector) collectRawRedfishTree(ctx context.Context, client *ht
 							emit(Progress{
 								Status:   "running",
 								Progress: 92 + int(minInt32(n/200, 6)),
-								Message:  fmt.Sprintf("Redfish snapshot: ошибка на %s", compactProgressPath(current)),
+								Message:  fmt.Sprintf("Redfish snapshot: error on %s", compactProgressPath(current)),
 							})
 						}
 						wg.Done()
@@ -1489,7 +1485,7 @@ func (c *RedfishConnector) collectRawRedfishTree(ctx context.Context, client *ht
 						emit(Progress{
 							Status:   "running",
 							Progress: 92 + int(minInt32(n/200, 6)),
-							Message:  fmt.Sprintf("Redfish snapshot: ошибка на %s", compactProgressPath(current)),
+							Message:  fmt.Sprintf("Redfish snapshot: error on %s", compactProgressPath(current)),
 						})
 					}
 				}
@@ -1512,7 +1508,7 @@ func (c *RedfishConnector) collectRawRedfishTree(ctx context.Context, client *ht
 					emit(Progress{
 						Status:       "running",
 						Progress:     92 + int(minInt32(n/200, 6)),
-						Message:      fmt.Sprintf("Redfish snapshot: документов=%d, ETA≈%s, корни=%s, последний=%s", n, eta, strings.Join(roots, ", "), compactProgressPath(last)),
+						Message:      fmt.Sprintf("Redfish snapshot: docs=%d, ETA≈%s, roots=%s, last=%s", n, eta, strings.Join(roots, ", "), compactProgressPath(last)),
 						CurrentPhase: "snapshot",
 						ETASeconds:   int(estimateSnapshotETA(crawlStart, int(n), seenN, len(jobs), workers, client.Timeout).Seconds()),
 					})
@@ -1574,7 +1570,7 @@ func (c *RedfishConnector) collectRawRedfishTree(ctx context.Context, client *ht
 			emit(Progress{
 				Status:       "running",
 				Progress:     97,
-				Message:      fmt.Sprintf("Redfish snapshot: post-probe NVMe (%d/%d, ETA≈%s), коллекция=%s", i+1, len(driveCollections), formatETA(estimateProgressETA(nvmeProbeStart, i, len(driveCollections), 2*time.Second)), compactProgressPath(path)),
+				Message:      fmt.Sprintf("Redfish snapshot: post-probe NVMe (%d/%d, ETA≈%s), collection=%s", i+1, len(driveCollections), formatETA(estimateProgressETA(nvmeProbeStart, i, len(driveCollections), 2*time.Second)), compactProgressPath(path)),
 				CurrentPhase: "snapshot_postprobe_nvme",
 				ETASeconds:   int(estimateProgressETA(nvmeProbeStart, i, len(driveCollections), 2*time.Second).Seconds()),
 			})
@@ -1622,7 +1618,7 @@ func (c *RedfishConnector) collectRawRedfishTree(ctx context.Context, client *ht
 			emit(Progress{
 				Status:       "running",
 				Progress:     98,
-				Message:      fmt.Sprintf("Redfish snapshot: post-probe коллекций (%d/%d, ETA≈%s), текущая=%s", i+1, len(postProbeCollections), formatETA(estimateProgressETA(postProbeStart, i, len(postProbeCollections), 3*time.Second)), compactProgressPath(path)),
+				Message:      fmt.Sprintf("Redfish snapshot: post-probe collections (%d/%d, ETA≈%s), current=%s", i+1, len(postProbeCollections), formatETA(estimateProgressETA(postProbeStart, i, len(postProbeCollections), 3*time.Second)), compactProgressPath(path)),
 				CurrentPhase: "snapshot_postprobe_collections",
 				ETASeconds:   int(estimateProgressETA(postProbeStart, i, len(postProbeCollections), 3*time.Second).Seconds()),
 			})
@@ -1641,14 +1637,14 @@ func (c *RedfishConnector) collectRawRedfishTree(ctx context.Context, client *ht
 		emit(Progress{
 			Status:   "running",
 			Progress: 98,
-			Message:  fmt.Sprintf("Redfish snapshot: post-probe добавлено %d документов", addedPostProbe),
+			Message:  fmt.Sprintf("Redfish snapshot: post-probe added %d docs", addedPostProbe),
 		})
 	}
 	if emit != nil {
 		emit(Progress{
 			Status:   "running",
 			Progress: 98,
-			Message:  fmt.Sprintf("Redfish snapshot: post-probe метрики candidates=%d selected=%d skipped_explicit=%d added=%d", postProbeMetrics.CollectionCandidates, postProbeMetrics.CollectionSelected, postProbeMetrics.SkippedExplicit, postProbeMetrics.Added),
+			Message:  fmt.Sprintf("Redfish snapshot: post-probe metrics candidates=%d selected=%d skipped_explicit=%d added=%d", postProbeMetrics.CollectionCandidates, postProbeMetrics.CollectionSelected, postProbeMetrics.SkippedExplicit, postProbeMetrics.Added),
 		})
 	}

@@ -1656,7 +1652,7 @@ func (c *RedfishConnector) collectRawRedfishTree(ctx context.Context, client *ht
 		emit(Progress{
 			Status:   "running",
 			Progress: 98,
-			Message:  fmt.Sprintf("Redfish snapshot: собрано %d документов", len(out)),
+			Message:  fmt.Sprintf("Redfish snapshot: collected %d docs", len(out)),
 		})
 	}

@@ -1671,14 +1667,14 @@ func (c *RedfishConnector) collectRawRedfishTree(ctx context.Context, client *ht
 		return asString(errorList[i]["path"]) < asString(errorList[j]["path"])
 	})
 	if summary := timings.Summary(12); summary != "" {
-		log.Printf("redfish-snapshot-timing: %s", summary)
+		slog.Info("redfish-snapshot-timing", "summary", summary)
 	}
 	if emit != nil {
 		if summary := timings.Summary(3); summary != "" {
 			emit(Progress{
 				Status:   "running",
 				Progress: 98,
-				Message:  fmt.Sprintf("Redfish snapshot: топ веток по времени: %s", summary),
+				Message:  fmt.Sprintf("Redfish snapshot: top branches by time: %s", summary),
 			})
 		}
 	}
@@ -2979,14 +2975,14 @@ func (c *RedfishConnector) recoverCriticalRedfishDocsPlanB(ctx context.Context,
 			emit(Progress{
 				Status:   "running",
 				Progress: 97,
-				Message:  fmt.Sprintf("Redfish: расширенная диагностика выключена, пропущено %d тяжелых diagnostic endpoint", skippedDiagnosticTargets),
+				Message:  fmt.Sprintf("Redfish: extended diagnostics disabled, skipped %d heavy diagnostic endpoints", skippedDiagnosticTargets),
 			})
 		}
 		totalETA := redfishCriticalCooldown() + estimatePlanBETA(len(targets))
 		emit(Progress{
 			Status:       "running",
 			Progress:     97,
-			Message:      fmt.Sprintf("Redfish: cooldown перед повторным добором критичных endpoint... ETA≈%s", formatETA(totalETA)),
+			Message:      fmt.Sprintf("Redfish: cooldown before retrying critical endpoints... ETA≈%s", formatETA(totalETA)),
 			CurrentPhase: "critical_plan_b",
 			ETASeconds:   int(totalETA.Seconds()),
 		})
@@ -3072,17 +3068,17 @@ func (c *RedfishConnector) recoverCriticalRedfishDocsPlanB(ctx context.Context,
 			emit(Progress{
 				Status:   "running",
 				Progress: 97,
-				Message:  fmt.Sprintf("Redfish: plan-B топ веток по времени: %s", summary),
+				Message:  fmt.Sprintf("Redfish: plan-B top branches by time: %s", summary),
 			})
 		}
 		emit(Progress{
 			Status:   "running",
 			Progress: 97,
-			Message:  fmt.Sprintf("Redfish: plan-B завершен за %s (targets=%d, recovered=%d)", time.Since(planBStart).Round(time.Second), len(targets), recovered),
+			Message:  fmt.Sprintf("Redfish: plan-B completed in %s (targets=%d, recovered=%d)", time.Since(planBStart).Round(time.Second), len(targets), recovered),
 		})
 	}
 	if summary := timings.Summary(12); summary != "" {
-		log.Printf("redfish-planb-timing: %s", summary)
+		slog.Info("redfish-planb-timing", "summary", summary)
 	}
 	return recovered
 }
@@ -3143,7 +3139,7 @@ func (c *RedfishConnector) recoverProfilePlanBDocs(ctx context.Context, client *
 		emit(Progress{
 			Status:       "running",
 			Progress:     98,
-			Message:      fmt.Sprintf("Redfish: profile plan-B добирает %d endpoint...", len(targets)),
+			Message:      fmt.Sprintf("Redfish: profile plan-B fetching %d endpoints...", len(targets)),
 			CurrentPhase: "profile_plan_b",
 			ETASeconds:   int(estimateProgressETA(planBStart, 0, len(targets), 2*time.Second).Seconds()),
 		})
@@ -3167,19 +3163,18 @@ func (c *RedfishConnector) recoverProfilePlanBDocs(ctx context.Context, client *
 		recovered++
 	}
 	if recovered > 0 {
-		log.Printf(
-			"redfish-profile-planb: mode=%s profiles=%s targets=%d recovered=%d",
-			plan.Mode,
-			strings.Join(plan.Profiles, ","),
-			len(targets),
-			recovered,
+		slog.Info("redfish-profile-planb",
+			"mode", plan.Mode,
+			"profiles", strings.Join(plan.Profiles, ","),
+			"targets", len(targets),
+			"recovered", recovered,
 		)
 	}
 	if emit != nil {
 		emit(Progress{
 			Status:       "running",
 			Progress:     98,
-			Message:      fmt.Sprintf("Redfish: profile plan-B завершен за %s (targets=%d, recovered=%d)", time.Since(planBStart).Round(time.Second), len(targets), recovered),
+			Message:      fmt.Sprintf("Redfish: profile plan-B completed in %s (targets=%d, recovered=%d)", time.Since(planBStart).Round(time.Second), len(targets), recovered),
 			CurrentPhase: "profile_plan_b",
 		})
 	}
--- a/internal/collector/redfish_logentries.go
+++ b/internal/collector/redfish_logentries.go
@@ -2,7 +2,7 @@ package collector

 import (
 	"context"
-	"log"
+	"log/slog"
 	"net/http"
 	"strings"
 	"time"
@@ -62,7 +62,7 @@ func (c *RedfishConnector) collectRedfishLogEntries(ctx context.Context, client
 	}

 	if len(out) > 0 {
-		log.Printf("redfish: collected %d hardware log entries (Systems+Managers SEL, window=7d)", len(out))
+		slog.Info("redfish: collected hardware log entries", "count", len(out), "window", "7d")
 	}
 	return out
 }
--- a/internal/collector/redfish_replay.go
+++ b/internal/collector/redfish_replay.go
@@ -3,7 +3,7 @@ package collector
 import (
 	"encoding/json"
 	"fmt"
-	"log"
+	"log/slog"
 	"sort"
 	"strings"
 	"time"
@@ -32,7 +32,7 @@ func ReplayRedfishFromRawPayloads(rawPayloads map[string]any, emit ProgressFn) (
 		emit(Progress{Status: "running", Progress: 10, Message: "Redfish snapshot: replay service root..."})
 	}
 	if _, err := r.getJSON("/redfish/v1"); err != nil {
-		log.Printf("redfish replay: service root /redfish/v1 missing from snapshot, continuing with defaults: %v", err)
+		slog.Warn("redfish replay: service root /redfish/v1 missing from snapshot, continuing with defaults", "err", err)
 	}

 	systemPaths := r.discoverMemberPaths("/redfish/v1/Systems", "/redfish/v1/Systems/1")
@@ -219,7 +219,7 @@ func inferInventoryLastModifiedTime(snapshot map[string]interface{}) time.Time {
 	for _, layout := range []string{time.RFC3339, time.RFC3339Nano} {
 		if ts, err := time.Parse(layout, raw); err == nil {
 			t := ts.UTC()
-			log.Printf("redfish replay: inventory last modified at %s (InventoryData/Status.LastModifiedTime)", t.Format(time.RFC3339))
+			slog.Info("redfish replay: inventory last modified", "at", t.Format(time.RFC3339), "source", "InventoryData/Status.LastModifiedTime")
 			return t
 		}
 	}
--- a/internal/collector/redfish_replay_inventory.go
+++ b/internal/collector/redfish_replay_inventory.go
@@ -244,7 +244,7 @@ func isReplayStorageServiceEndpoint(doc map[string]interface{}, dev models.PCIeD
 	if strings.Contains(name, "pcie switch management endpoint") {
 		return true
 	}
-	if strings.Contains(name, "volume management device nvme raid controller") {
+	if strings.Contains(name, "volume management device") {
 		return true
 	}
 	return false
--- a/internal/collector/redfishprofile/profile_inspur_group_oem_platforms.go
+++ b/internal/collector/redfishprofile/profile_inspur_group_oem_platforms.go
@@ -29,6 +29,7 @@ func inspurGroupOEMPlatformsProfile() Profile {
 		matchFn: func(s MatchSignals) int {
 			topologyScore := 0
 			boardScore := 0
+			manufacturerScore := 0
 			chassisOutboard := matchedPathTokens(s.ResourceHints, "/redfish/v1/Chassis/", outboardCardHintRe)
 			systemOutboard := matchedPathTokens(s.ResourceHints, "/redfish/v1/Systems/", outboardCardHintRe)
 			obDrives := matchedPathTokens(s.ResourceHints, "", obDriveHintRe)
@@ -62,10 +63,17 @@ func inspurGroupOEMPlatformsProfile() Profile {
 			if anySignalContains(s, "GetServerAllUSBStatus") {
 				boardScore += 8
 			}
-			if topologyScore == 0 || boardScore == 0 {
+			// Manufacturer alone is sufficient for standard Inspur servers (e.g. NF-series
+			// storage servers) that lack GPU/outboard-PCIe topology signals. Score 60 is
+			// the minimum to enter matched mode; topology+board can push it higher.
+			if containsFold(s.SystemManufacturer, "inspur") || containsFold(s.ChassisManufacturer, "inspur") {
+				manufacturerScore = 60
+			}
+			total := manufacturerScore + topologyScore + boardScore
+			if total < 60 {
 				return 0
 			}
-			return min(topologyScore+boardScore, 100)
+			return min(total, 100)
 		},
 		extendAcquisition: func(plan *AcquisitionPlan, _ MatchSignals) {
 			addPlanNote(plan, "Inspur Group OEM platform fingerprint matched")
--- a/internal/collector/redfishprofile/profile_inspur_group_oem_platforms_test.go
+++ b/internal/collector/redfishprofile/profile_inspur_group_oem_platforms_test.go
@@ -118,6 +118,52 @@ func TestCollectSignalsFromTree_InspurGroupOEMPlatformsSelectsMatchedMode(t *tes
 	assertProfileSelected(t, match, "inspur-group-oem-platforms")
 }

+// TestCollectSignalsFromTree_InspurGroupOEMPlatformsMatchesViaManufacturer covers standard
+// Inspur storage servers (e.g. NF5280M6) that have no outboard PCIe / GPU topology but
+// do expose Manufacturer="Inspur" in their System document.
+func TestCollectSignalsFromTree_InspurGroupOEMPlatformsMatchesViaManufacturer(t *testing.T) {
+	// Minimal tree: no GPU cards, no OEM firmware hints — only System Manufacturer.
+	tree := map[string]interface{}{
+		"/redfish/v1": map[string]interface{}{
+			"@odata.id": "/redfish/v1",
+		},
+		"/redfish/v1/Systems": map[string]interface{}{
+			"Members": []interface{}{
+				map[string]interface{}{"@odata.id": "/redfish/v1/Systems/1"},
+			},
+		},
+		"/redfish/v1/Systems/1": map[string]interface{}{
+			"@odata.id":    "/redfish/v1/Systems/1",
+			"Manufacturer": "Inspur",
+			"Model":        "NF5280M6",
+		},
+		"/redfish/v1/Chassis": map[string]interface{}{
+			"Members": []interface{}{
+				map[string]interface{}{"@odata.id": "/redfish/v1/Chassis/1"},
+			},
+		},
+		"/redfish/v1/Chassis/1": map[string]interface{}{
+			"@odata.id": "/redfish/v1/Chassis/1",
+		},
+		"/redfish/v1/Managers": map[string]interface{}{
+			"Members": []interface{}{
+				map[string]interface{}{"@odata.id": "/redfish/v1/Managers/1"},
+			},
+		},
+		"/redfish/v1/Managers/1": map[string]interface{}{
+			"@odata.id": "/redfish/v1/Managers/1",
+		},
+	}
+
+	signals := CollectSignalsFromTree(tree)
+	match := MatchProfiles(signals)
+
+	if match.Mode != ModeMatched {
+		t.Fatalf("expected matched mode for Inspur NF-series, got %q (scores: %v)", match.Mode, match.Scores)
+	}
+	assertProfileSelected(t, match, "inspur-group-oem-platforms")
+}
+
 func TestCollectSignalsFromTree_InspurGroupOEMPlatformsDoesNotFalsePositiveOnExampleRawExports(t *testing.T) {
 	examples := []string{
 		"2026-03-18 (G5500 V7) - 210619KUGGXGS2000015.zip",
--- a/internal/exporter/exporter.go
+++ b/internal/exporter/exporter.go
@@ -21,7 +21,11 @@ func New(result *models.AnalysisResult) *Exporter {

 // ExportCSV exports serial numbers to CSV format
 func (e *Exporter) ExportCSV(w io.Writer) error {
+	if _, err := w.Write([]byte{0xEF, 0xBB, 0xBF}); err != nil {
+		return err
+	}
 	writer := csv.NewWriter(w)
+	writer.Comma = ';'
 	defer writer.Flush()

 	// Header
@@ -170,3 +174,42 @@ func firstNonEmptyString(values ...string) string {
 	}
 	return ""
 }
+
+// ExportLogsCSV writes all recognized events as a semicolon-delimited UTF-8 CSV readable in Excel.
+func ExportLogsCSV(w io.Writer, result *models.AnalysisResult) error {
+	if _, err := w.Write([]byte{0xEF, 0xBB, 0xBF}); err != nil {
+		return err
+	}
+	writer := csv.NewWriter(w)
+	writer.Comma = ';'
+	defer writer.Flush()
+
+	if err := writer.Write([]string{"timestamp", "source", "severity", "sensor_type", "sensor_name", "event_type", "id", "description", "raw_data"}); err != nil {
+		return err
+	}
+
+	if result == nil {
+		return nil
+	}
+
+	for _, e := range result.Events {
+		ts := ""
+		if !e.Timestamp.IsZero() {
+			ts = e.Timestamp.UTC().Format("2006-01-02T15:04:05Z")
+		}
+		if err := writer.Write([]string{
+			ts,
+			e.Source,
+			string(e.Severity),
+			e.SensorType,
+			e.SensorName,
+			e.EventType,
+			e.ID,
+			e.Description,
+			e.RawData,
+		}); err != nil {
+			return err
+		}
+	}
+	return nil
+}
--- a/internal/exporter/exporter_csv_test.go
+++ b/internal/exporter/exporter_csv_test.go
@@ -52,7 +52,13 @@ func TestExportCSV_IncludesAllComponentTypesWithUsableSerials(t *testing.T) {
 		t.Fatalf("ExportCSV failed: %v", err)
 	}

-	rows, err := csv.NewReader(bytes.NewReader(buf.Bytes())).ReadAll()
+	b := buf.Bytes()
+	if len(b) >= 3 && b[0] == 0xEF && b[1] == 0xBB && b[2] == 0xBF {
+		b = b[3:] // strip UTF-8 BOM
+	}
+	r := csv.NewReader(bytes.NewReader(b))
+	r.Comma = ';'
+	rows, err := r.ReadAll()
 	if err != nil {
 		t.Fatalf("read csv: %v", err)
 	}
--- a/internal/exporter/reanimator_converter.go
+++ b/internal/exporter/reanimator_converter.go
@@ -49,9 +49,10 @@ func ConvertToReanimator(result *models.AnalysisResult) (*ReanimatorExport, erro
 			Memory:        dedupeMemory(convertMemoryFromDevices(devices, collectedAt)),
 			Storage:       dedupeStorage(convertStorageFromDevices(devices, collectedAt)),
 			PCIeDevices:   dedupePCIe(convertPCIeFromDevices(devices, collectedAt)),
-			PowerSupplies: dedupePSUs(convertPSUsFromDevices(devices, collectedAt)),
-			Sensors:       convertSensors(result.Sensors),
-			EventLogs:     convertEventLogs(result.Events, collectedAt),
+			PowerSupplies:   dedupePSUs(convertPSUsFromDevices(devices, collectedAt)),
+			Sensors:         convertSensors(result.Sensors),
+			BMCEventSummary: buildBMCEventSummary(result.Events, collectedAt),
+			EventLogs:       convertEventLogs(result.Events, collectedAt),
 		},
 	}

@@ -218,6 +219,9 @@ func buildDevicesFromLegacy(hw *models.HardwareConfig) []models.HardwareDevice {
 		if pcie.HWSlowdown != nil {
 			details = mergeDetailMaps(details, map[string]any{"hw_slowdown": *pcie.HWSlowdown})
 		}
+		if pcie.IOMMUGroup != nil {
+			details = mergeDetailMaps(details, map[string]any{"iommu_group": *pcie.IOMMUGroup})
+		}
 		present := pcie.Present
 		appendDevice(models.HardwareDevice{
 			Kind:             models.DeviceKindPCIe,
@@ -844,6 +848,7 @@ func convertPCIeFromDevices(devices []models.HardwareDevice, collectedAt string)
 			VendorID:               d.VendorID,
 			DeviceID:               d.DeviceID,
 			NUMANode:               d.NUMANode,
+			IOMMUGroup:             intPtrFromDetailMap(d.Details, "iommu_group"),
 			TemperatureC:           temperatureC,
 			PowerW:                 powerW,
 			LifeRemainingPct:       floatFromDetailMap(d.Details, "life_remaining_pct"),
@@ -1230,7 +1235,7 @@ func normalizeEventLogSource(source string) string {
 	switch strings.ToLower(strings.TrimSpace(source)) {
 	case "redfish":
 		return "redfish"
-	case "sel", "bmc", "ipmi", "idrac", "lifecycle controller", "lifecyclecontroller":
+	case "sel", "bmc", "ipmi", "idrac", "lifecycle controller", "lifecyclecontroller", "hpe ilo", "ilo":
 		return "bmc"
 	case "system", "syslog", "smart", "zfs", "file", "gpu", "dmi", "nvidia driver", "gpu field diagnostics", "fan", "memory", "host":
 		return "host"
@@ -2119,6 +2124,17 @@ func parseSocketFromSlot(slot string) int {
 	return v
 }

+func intPtrFromDetailMap(details map[string]any, key string) *int {
+	if details == nil {
+		return nil
+	}
+	if _, ok := details[key]; !ok {
+		return nil
+	}
+	v := intFromDetailMap(details, key)
+	return &v
+}
+
 func intFromDetailMap(details map[string]any, key string) int {
 	if details == nil {
 		return 0
@@ -2442,3 +2458,76 @@ func inferTargetHost(targetHost, filename string) string {

 	return ""
 }
+
+// buildBMCEventSummary produces a summary table of Critical/Warning BMC events
+// with their resolution status derived from Assert/Deassert pairs.
+func buildBMCEventSummary(events []models.Event, collectedAt string) []ReanimatorBMCEventRow {
+	type assertKey struct {
+		id   string
+		desc string
+	}
+	type eventPair struct {
+		assertEvent   *models.Event
+		deassertEvent *models.Event
+	}
+
+	pairs := make(map[assertKey]*eventPair)
+	order := make([]assertKey, 0)
+
+	for i := range events {
+		e := &events[i]
+		if e.Severity != models.SeverityCritical && e.Severity != models.SeverityWarning {
+			continue
+		}
+		key := assertKey{id: e.ID, desc: e.Description}
+		p, exists := pairs[key]
+		if !exists {
+			p = &eventPair{}
+			pairs[key] = p
+			order = append(order, key)
+		}
+		switch strings.ToLower(e.EventType) {
+		case "deassert":
+			if p.deassertEvent == nil || e.Timestamp.After(p.deassertEvent.Timestamp) {
+				p.deassertEvent = e
+			}
+		default:
+			if p.assertEvent == nil || e.Timestamp.Before(p.assertEvent.Timestamp) {
+				p.assertEvent = e
+			}
+		}
+	}
+
+	rows := make([]ReanimatorBMCEventRow, 0, len(order))
+	for _, key := range order {
+		p := pairs[key]
+		ref := p.assertEvent
+		if ref == nil {
+			ref = p.deassertEvent
+		}
+		if ref == nil {
+			continue
+		}
+
+		status := "Active"
+		resolvedAt := ""
+		if p.deassertEvent != nil {
+			status = "Resolved"
+			resolvedAt = formatEventLogTime(p.deassertEvent.Timestamp, collectedAt)
+		}
+
+		rows = append(rows, ReanimatorBMCEventRow{
+			Severity:    normalizeEventLogSeverity(ref.Severity),
+			Component:   strings.ToUpper(strings.TrimSpace(ref.SensorType)),
+			MessageID:   strings.TrimSpace(ref.ID),
+			Timestamp:   formatEventLogTime(ref.Timestamp, collectedAt),
+			Description: strings.TrimSpace(ref.Description),
+			Status:      status,
+			ResolvedAt:  resolvedAt,
+		})
+	}
+	if len(rows) == 0 {
+		return nil
+	}
+	return rows
+}
--- a/internal/exporter/reanimator_models.go
+++ b/internal/exporter/reanimator_models.go
@@ -12,16 +12,28 @@ type ReanimatorExport struct {

 // ReanimatorHardware contains all hardware components
 type ReanimatorHardware struct {
-	Board          ReanimatorBoard      `json:"board"`
-	Firmware       []ReanimatorFirmware `json:"firmware,omitempty"`
-	CPUs           []ReanimatorCPU      `json:"cpus,omitempty"`
-	Memory         []ReanimatorMemory   `json:"memory,omitempty"`
-	Storage        []ReanimatorStorage  `json:"storage,omitempty"`
-	PCIeDevices    []ReanimatorPCIe     `json:"pcie_devices,omitempty"`
-	PowerSupplies  []ReanimatorPSU      `json:"power_supplies,omitempty"`
-	Sensors        *ReanimatorSensors   `json:"sensors,omitempty"`
-	EventLogs      []ReanimatorEventLog `json:"event_logs,omitempty"`
-	PlatformConfig map[string]any       `json:"platform_config,omitempty"`
+	Board            ReanimatorBoard            `json:"board"`
+	Firmware         []ReanimatorFirmware       `json:"firmware,omitempty"`
+	CPUs             []ReanimatorCPU            `json:"cpus,omitempty"`
+	Memory           []ReanimatorMemory         `json:"memory,omitempty"`
+	Storage          []ReanimatorStorage        `json:"storage,omitempty"`
+	PCIeDevices      []ReanimatorPCIe           `json:"pcie_devices,omitempty"`
+	PowerSupplies    []ReanimatorPSU            `json:"power_supplies,omitempty"`
+	Sensors          *ReanimatorSensors         `json:"sensors,omitempty"`
+	BMCEventSummary  []ReanimatorBMCEventRow    `json:"bmc_event_summary,omitempty"`
+	EventLogs        []ReanimatorEventLog       `json:"event_logs,omitempty"`
+	PlatformConfig   map[string]any             `json:"platform_config,omitempty"`
+}
+
+// ReanimatorBMCEventRow is one row in the BMC critical/warning event summary table.
+type ReanimatorBMCEventRow struct {
+	Severity   string `json:"severity"`
+	Component  string `json:"component"`
+	MessageID  string `json:"message_id"`
+	Timestamp  string `json:"timestamp"`
+	Description string `json:"description"`
+	Status     string `json:"status"`
+	ResolvedAt string `json:"resolved_at,omitempty"`
 }

 // ReanimatorBoard represents motherboard/server information
@@ -143,6 +155,7 @@ type ReanimatorPCIe struct {
 	VendorID               int                            `json:"vendor_id,omitempty"`
 	DeviceID               int                            `json:"device_id,omitempty"`
 	NUMANode               int                            `json:"numa_node,omitempty"`
+	IOMMUGroup             *int                           `json:"iommu_group,omitempty"`
 	TemperatureC           float64                        `json:"temperature_c,omitempty"`
 	PowerW                 float64                        `json:"power_w,omitempty"`
 	LifeRemainingPct       float64                        `json:"life_remaining_pct,omitempty"`
--- a/internal/models/models.go
+++ b/internal/models/models.go
@@ -16,11 +16,21 @@ type AnalysisResult struct {
 	SourceTimezone string          `json:"source_timezone,omitempty"` // Source timezone/offset used during collection (e.g. +08:00)
 	CollectedAt              time.Time       `json:"collected_at,omitempty"`               // Collection/upload timestamp
 	InventoryLastModifiedAt  time.Time       `json:"inventory_last_modified_at,omitempty"` // Redfish inventory last modified (InventoryData/Status)
-	RawPayloads    map[string]any  `json:"raw_payloads,omitempty"`    // Additional source payloads (e.g. Redfish tree)
-	Events         []Event         `json:"events"`
-	FRU            []FRUInfo       `json:"fru"`
-	Sensors        []SensorReading `json:"sensors"`
-	Hardware       *HardwareConfig `json:"hardware"`
+	RawPayloads      map[string]any   `json:"raw_payloads,omitempty"`      // Additional source payloads (e.g. Redfish tree)
+	CollectionErrors []CollectionError `json:"collection_errors,omitempty"` // BMC-reported failures to collect specific sections
+	Events           []Event           `json:"events"`
+	FRU              []FRUInfo         `json:"fru"`
+	Sensors          []SensorReading   `json:"sensors"`
+	Hardware         *HardwareConfig   `json:"hardware"`
+}
+
+// CollectionError represents a BMC-reported failure to collect a specific data section.
+// Populated by vendor parsers when the source explicitly returns an error response
+// instead of structured data (e.g. {"error":"...","code":1458} in Inspur component.log).
+type CollectionError struct {
+	Section string `json:"section"`
+	Message string `json:"message"`
+	Code    int    `json:"code,omitempty"`
 }

 // Event represents a single log event
--- a/internal/parser/archive.go
+++ b/internal/parser/archive.go
@@ -15,9 +15,11 @@ import (
 )

 const maxSingleFileSize = 10 * 1024 * 1024
+const maxSingleFileSizeLarge = 1024 * 1024 * 1024
 const maxZipArchiveSize = 50 * 1024 * 1024
 const maxGzipDecompressedSize = 50 * 1024 * 1024

+
 var supportedArchiveExt = map[string]struct{}{
 	".ahs": {},
 	".gz":  {},
@@ -47,7 +49,7 @@ func ExtractArchive(archivePath string) ([]ExtractedFile, error) {

 	switch ext {
 	case ".ahs":
-		return extractSingleFile(archivePath)
+		return extractSingleFileWithLimit(archivePath, maxSingleFileSizeLarge)
 	case ".gz", ".tgz":
 		return extractTarGz(archivePath)
 	case ".tar", ".sds":
@@ -55,7 +57,7 @@ func ExtractArchive(archivePath string) ([]ExtractedFile, error) {
 	case ".zip":
 		return extractZip(archivePath)
 	case ".txt", ".log":
-		return extractSingleFile(archivePath)
+		return extractSingleFileWithLimit(archivePath, maxSingleFileSize)
 	default:
 		return nil, fmt.Errorf("unsupported archive format: %s", ext)
 	}
@@ -70,7 +72,7 @@ func ExtractArchiveFromReader(r io.Reader, filename string) ([]ExtractedFile, er

 	switch ext {
 	case ".ahs":
-		return extractSingleFileFromReader(r, filename)
+		return extractSingleFileFromReaderWithLimit(r, filename, maxSingleFileSizeLarge)
 	case ".gz", ".tgz":
 		return extractTarGzFromReader(r, filename)
 	case ".tar", ".sds":
@@ -78,7 +80,7 @@ func ExtractArchiveFromReader(r io.Reader, filename string) ([]ExtractedFile, er
 	case ".zip":
 		return extractZipFromReader(r)
 	case ".txt", ".log":
-		return extractSingleFileFromReader(r, filename)
+		return extractSingleFileFromReaderWithLimit(r, filename, maxSingleFileSize)
 	default:
 		return nil, fmt.Errorf("unsupported archive format: %s", ext)
 	}
@@ -337,7 +339,7 @@ func extractZipFromReader(r io.Reader) ([]ExtractedFile, error) {
 	return files, nil
 }

-func extractSingleFile(path string) ([]ExtractedFile, error) {
+func extractSingleFileWithLimit(path string, limit int64) ([]ExtractedFile, error) {
 	info, err := os.Stat(path)
 	if err != nil {
 		return nil, fmt.Errorf("stat file: %w", err)
@@ -348,7 +350,7 @@ func extractSingleFile(path string) ([]ExtractedFile, error) {
 	}
 	defer f.Close()

-	files, err := extractSingleFileFromReader(f, filepath.Base(path))
+	files, err := extractSingleFileFromReaderWithLimit(f, filepath.Base(path), limit)
 	if err != nil {
 		return nil, err
 	}
@@ -358,14 +360,14 @@ func extractSingleFile(path string) ([]ExtractedFile, error) {
 	return files, nil
 }

-func extractSingleFileFromReader(r io.Reader, filename string) ([]ExtractedFile, error) {
-	content, err := io.ReadAll(io.LimitReader(r, maxSingleFileSize+1))
+func extractSingleFileFromReaderWithLimit(r io.Reader, filename string, limit int64) ([]ExtractedFile, error) {
+	content, err := io.ReadAll(io.LimitReader(r, limit+1))
 	if err != nil {
 		return nil, fmt.Errorf("read file content: %w", err)
 	}
-	truncated := len(content) > maxSingleFileSize
+	truncated := int64(len(content)) > limit
 	if truncated {
-		content = content[:maxSingleFileSize]
+		content = content[:limit]
 	}

 	file := ExtractedFile{
@@ -376,7 +378,7 @@ func extractSingleFileFromReader(r io.Reader, filename string) ([]ExtractedFile,
 		file.Truncated = true
 		file.TruncatedMessage = fmt.Sprintf(
 			"file exceeded %d bytes and was truncated",
-			maxSingleFileSize,
+			limit,
 		)
 	}

--- a/internal/parser/vendors/h3c/parser.go
+++ b/internal/parser/vendors/h3c/parser.go
@@ -2867,9 +2867,9 @@ func parseKeyValueBlocks(content string) []map[string]string {

 func findCPUIndex(items []models.CPU, target models.CPU) int {
 	targetSocket := target.Socket
-	targetPPIN := strings.ToLower(strings.TrimSpace(target.PPIN))
-	targetSerial := strings.ToLower(strings.TrimSpace(target.SerialNumber))
-	targetModel := strings.ToLower(strings.TrimSpace(target.Model))
+	targetPPIN := strings.TrimSpace(target.PPIN)
+	targetSerial := strings.TrimSpace(target.SerialNumber)
+	targetModel := strings.TrimSpace(target.Model)

 	for i := range items {
 		cpu := items[i]
@@ -2880,18 +2880,18 @@ func findCPUIndex(items []models.CPU, target models.CPU) int {
 			continue
 		}

-		ppin := strings.ToLower(strings.TrimSpace(cpu.PPIN))
-		if targetPPIN != "" && ppin != "" && targetPPIN == ppin {
+		ppin := strings.TrimSpace(cpu.PPIN)
+		if targetPPIN != "" && ppin != "" && strings.EqualFold(targetPPIN, ppin) {
 			return i
 		}

-		serial := strings.ToLower(strings.TrimSpace(cpu.SerialNumber))
-		if targetSerial != "" && serial != "" && targetSerial == serial {
+		serial := strings.TrimSpace(cpu.SerialNumber)
+		if targetSerial != "" && serial != "" && strings.EqualFold(targetSerial, serial) {
 			return i
 		}

-		model := strings.ToLower(strings.TrimSpace(cpu.Model))
-		if targetSocket == 0 && cpu.Socket == 0 && targetModel != "" && model == targetModel {
+		model := strings.TrimSpace(cpu.Model)
+		if targetSocket == 0 && cpu.Socket == 0 && targetModel != "" && strings.EqualFold(model, targetModel) {
 			return i
 		}
 	}
@@ -2931,15 +2931,15 @@ func mergeCPU(dst *models.CPU, src models.CPU) {
 }

 func findMemoryIndex(items []models.MemoryDIMM, target models.MemoryDIMM) int {
-	targetSerial := strings.ToLower(strings.TrimSpace(target.SerialNumber))
-	targetSlot := strings.ToLower(strings.TrimSpace(target.Slot))
+	targetSerial := strings.TrimSpace(target.SerialNumber)
+	targetSlot := strings.TrimSpace(target.Slot)
 	for i := range items {
-		serial := strings.ToLower(strings.TrimSpace(items[i].SerialNumber))
-		slot := strings.ToLower(strings.TrimSpace(items[i].Slot))
-		if targetSerial != "" && serial != "" && targetSerial == serial {
+		serial := strings.TrimSpace(items[i].SerialNumber)
+		slot := strings.TrimSpace(items[i].Slot)
+		if targetSerial != "" && serial != "" && strings.EqualFold(targetSerial, serial) {
 			return i
 		}
-		if targetSerial == "" && targetSlot != "" && slot != "" && targetSlot == slot {
+		if targetSerial == "" && targetSlot != "" && slot != "" && strings.EqualFold(targetSlot, slot) {
 			return i
 		}
 	}
@@ -2993,15 +2993,15 @@ func dedupeStorage(items []models.Storage) []models.Storage {
 }

 func findStorageIndex(items []models.Storage, target models.Storage) int {
-	targetSerial := strings.ToLower(strings.TrimSpace(target.SerialNumber))
-	targetSlot := strings.ToLower(strings.TrimSpace(target.Slot))
+	targetSerial := strings.TrimSpace(target.SerialNumber)
+	targetSlot := strings.TrimSpace(target.Slot)
 	for i := range items {
-		serial := strings.ToLower(strings.TrimSpace(items[i].SerialNumber))
-		slot := strings.ToLower(strings.TrimSpace(items[i].Slot))
-		if targetSerial != "" && serial != "" && targetSerial == serial {
+		serial := strings.TrimSpace(items[i].SerialNumber)
+		slot := strings.TrimSpace(items[i].Slot)
+		if targetSerial != "" && serial != "" && strings.EqualFold(targetSerial, serial) {
 			return i
 		}
-		if targetSerial == "" && targetSlot != "" && slot != "" && targetSlot == slot {
+		if targetSerial == "" && targetSlot != "" && slot != "" && strings.EqualFold(targetSlot, slot) {
 			return i
 		}
 	}
@@ -3248,15 +3248,15 @@ func isPSUEmpty(p models.PSU) bool {
 }

 func findPSUIndex(items []models.PSU, target models.PSU) int {
-	targetSerial := strings.ToLower(strings.TrimSpace(target.SerialNumber))
-	targetSlot := strings.ToLower(strings.TrimSpace(target.Slot))
+	targetSerial := strings.TrimSpace(target.SerialNumber)
+	targetSlot := strings.TrimSpace(target.Slot)
 	for i := range items {
-		serial := strings.ToLower(strings.TrimSpace(items[i].SerialNumber))
-		slot := strings.ToLower(strings.TrimSpace(items[i].Slot))
-		if targetSerial != "" && serial != "" && targetSerial == serial {
+		serial := strings.TrimSpace(items[i].SerialNumber)
+		slot := strings.TrimSpace(items[i].Slot)
+		if targetSerial != "" && serial != "" && strings.EqualFold(targetSerial, serial) {
 			return i
 		}
-		if targetSerial == "" && targetSlot != "" && slot != "" && targetSlot == slot {
+		if targetSerial == "" && targetSlot != "" && slot != "" && strings.EqualFold(targetSlot, slot) {
 			return i
 		}
 	}
--- a/internal/parser/vendors/hpe_ilo_ahs/parser.go
+++ b/internal/parser/vendors/hpe_ilo_ahs/parser.go
@@ -214,8 +214,10 @@ func parseAHSContainer(data []byte) ([]ahsEntry, error) {
 		name := strings.TrimRight(string(data[offset+20:offset+52]), "\x00")
 		start := offset + ahsHeaderSize
 		end := start + size
+		truncated := false
 		if size < 0 || end > len(data) {
-			return nil, fmt.Errorf("invalid payload size for %q", name)
+			end = len(data)
+			truncated = true
 		}

 		payload := append([]byte(nil), data[start:end]...)
@@ -235,6 +237,9 @@ func parseAHSContainer(data []byte) ([]ahsEntry, error) {
 			Content:    content,
 			Compressed: compressed,
 		})
+		if truncated {
+			break
+		}
 		offset = end
 	}

@@ -992,7 +997,7 @@ func parseEvents(tokens []string) []models.Event {
 				break
 			}
 			if looksLikeEventMessage(tokens[j]) {
-				message = tokens[j]
+				message = trimEventJunk(tokens[j])
 				break
 			}
 		}
@@ -1173,7 +1178,7 @@ func looksLikeServerModel(v string) bool {
 		return false
 	}
 	lower := strings.ToLower(v)
-	return strings.Contains(lower, "proliant") || strings.Contains(lower, "apollo") || strings.Contains(lower, "synergy") || strings.Contains(lower, "edgeline")
+	return strings.Contains(lower, "proliant") || strings.Contains(lower, "apollo") || strings.Contains(lower, "synergy") || strings.Contains(lower, "edgeline") || strings.Contains(lower, "alletra")
 }

 func looksLikeCPUVendor(v string) bool {
@@ -1464,7 +1469,19 @@ func fabricIDFromPath(path string) string {
 func inferSeverity(message string) models.Severity {
 	lower := strings.ToLower(message)
 	switch {
-	case strings.Contains(lower, " down"), strings.Contains(lower, "warning"), strings.Contains(lower, "fail"), strings.Contains(lower, "error"):
+	case strings.Contains(lower, "critical"):
+		return models.SeverityCritical
+	case strings.Contains(lower, " down"),
+		strings.Contains(lower, "warning"),
+		strings.Contains(lower, "fail"),
+		strings.Contains(lower, "error"),
+		strings.Contains(lower, "server reset"),
+		strings.Contains(lower, "server power"),
+		strings.Contains(lower, "power restored"),
+		strings.Contains(lower, "ilo reset"),
+		strings.Contains(lower, "ilo restarted"),
+		strings.Contains(lower, "pcr measurements"),
+		strings.Contains(lower, "hardware data received from uefi"):
 		return models.SeverityWarning
 	default:
 		return models.SeverityInfo
@@ -1478,21 +1495,73 @@ func inferEventType(message string) string {
 		return "Login"
 	case strings.Contains(lower, "logout"):
 		return "Logout"
-	case strings.Contains(lower, "network"):
+	case strings.Contains(lower, "network"), strings.Contains(lower, "link"):
 		return "Network"
 	case strings.Contains(lower, "license"):
 		return "License"
+	case strings.Contains(lower, "backup operation"), strings.Contains(lower, "remote console"):
+		return "Management"
+	case strings.Contains(lower, "server power"), strings.Contains(lower, "power restored"), strings.Contains(lower, "power off"), strings.Contains(lower, "server reset"), strings.Contains(lower, "ilo reset"), strings.Contains(lower, "ilo restarted"):
+		return "Power"
+	case strings.Contains(lower, "storage"), strings.Contains(lower, "volume"), strings.Contains(lower, "drive"), strings.Contains(lower, "firmware"):
+		return "Hardware"
+	case strings.Contains(lower, "certificate"), strings.Contains(lower, "pcr measurements"), strings.Contains(lower, "hardware data"), strings.Contains(lower, "security"):
+		return "Security"
 	default:
 		return "Event"
 	}
 }

+// trimEventJunk strips trailing single-byte frame markers written by iLO into
+// binary .zbb log records. These markers are printable ASCII (letters, *, +, ')
+// that appear immediately after the sentence-ending punctuation or a digit.
+func trimEventJunk(s string) string {
+	if len(s) < 3 {
+		return s
+	}
+	last := s[len(s)-1]
+	prev := s[len(s)-2]
+	isJunk := (last >= 'A' && last <= 'Z') || (last >= 'a' && last <= 'z') ||
+		last == '*' || last == '+' || last == '\''
+	prevIsBoundary := prev == '.' || prev == '!' || prev == '"' || prev == ')' ||
+		(prev >= '0' && prev <= '9')
+	if isJunk && prevIsBoundary {
+		return s[:len(s)-1]
+	}
+	return s
+}
+
 func looksLikeEventMessage(v string) bool {
 	if len(v) < 8 || strings.HasPrefix(v, "src/") || strings.HasPrefix(v, "PciRoot(") {
 		return false
 	}
+	// JSON document accidentally extracted — skip
+	if strings.HasPrefix(v, "{") || strings.HasPrefix(v, "[") {
+		return false
+	}
+	// Numbered list items (e.g. "2.Perform the iLO reset.") are instructions, not events
+	if len(v) > 2 && v[0] >= '1' && v[0] <= '9' && v[1] == '.' {
+		return false
+	}
 	lower := strings.ToLower(v)
-	return strings.Contains(lower, "login") || strings.Contains(lower, "logout") || strings.Contains(lower, "link") || strings.Contains(lower, "license") || strings.Contains(lower, "security state")
+	return strings.Contains(lower, "login") ||
+		strings.Contains(lower, "logout") ||
+		strings.Contains(lower, "link") ||
+		strings.Contains(lower, "license") ||
+		strings.Contains(lower, "security state") ||
+		strings.Contains(lower, "server power") ||
+		strings.Contains(lower, "server reset") ||
+		strings.Contains(lower, "power restored") ||
+		strings.Contains(lower, "power off") ||
+		strings.Contains(lower, "storage") ||
+		strings.Contains(lower, "firmware") ||
+		strings.Contains(lower, "certificate") ||
+		strings.Contains(lower, "backup operation") ||
+		strings.Contains(lower, "pcr measurements") ||
+		strings.Contains(lower, "hardware data") ||
+		strings.Contains(lower, "ilo reset") ||
+		strings.Contains(lower, "ilo restarted") ||
+		strings.Contains(lower, "remote console")
 }

 func sanitizeModel(v string) string {
--- a/internal/parser/vendors/hpe_ilo_ahs/parser_test.go
+++ b/internal/parser/vendors/hpe_ilo_ahs/parser_test.go
@@ -153,6 +153,29 @@ func TestParseAHSInventory(t *testing.T) {
 	}
 }

+func TestParseAHSTruncatedEntry(t *testing.T) {
+	p := &Parser{}
+	// Build archive where the last entry's declared size exceeds available data.
+	archive := makeAHSArchive(t, []ahsTestEntry{
+		{Name: "CUST_INFO.DAT", Payload: []byte("HPE\x00ProLiant DL380 Gen11\x00CZ2D1X0GS3\x00P52560-421")},
+		{Name: "0000150-2025-11-27.zbb", Payload: []byte("some content")},
+	})
+	// Corrupt the size field of the second entry to exceed len(archive).
+	secondHeaderOffset := ahsHeaderSize + len([]byte("HPE\x00ProLiant DL380 Gen11\x00CZ2D1X0GS3\x00P52560-421"))
+	binary.LittleEndian.PutUint32(archive[secondHeaderOffset+8:secondHeaderOffset+12], 0xFFFFFFFF)
+
+	result, err := p.Parse([]parser.ExtractedFile{{
+		Path:    "HPE_CZ2D1X0GS3_20251127.ahs",
+		Content: archive,
+	}})
+	if err != nil {
+		t.Fatalf("expected graceful handling of truncated entry, got error: %v", err)
+	}
+	if result == nil {
+		t.Fatal("expected non-nil result")
+	}
+}
+
 func TestParseExampleAHS(t *testing.T) {
 	path := filepath.Join("..", "..", "..", "..", "example", "HPE_CZ2D1X0GS3_20260330.ahs")
 	content, err := os.ReadFile(path)
--- 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
 		}
 	}

--- a/internal/parser/vendors/inspur/component.go
+++ b/internal/parser/vendors/inspur/component.go
@@ -19,6 +19,11 @@ func ParseComponentLog(content []byte, hw *models.HardwareConfig) {

 	text := string(content)

+	// Parse RESTful CPU info — fallback when asset.json is absent
+	if len(hw.CPUs) == 0 {
+		parseCPUInfo(text, hw)
+	}
+
 	// Parse RESTful Memory info (detailed memory data)
 	parseMemoryInfo(text, hw)

@@ -51,6 +56,52 @@ func ParseComponentLogEvents(content []byte) []models.Event {
 	return events
 }

+// ParseComponentLogCollectionErrors detects BMC-reported collection failures in component.log.
+// When a RESTful section returns {"error":"...","code":N} instead of structured data,
+// the BMC itself failed to collect that subsystem — the parser emits a CollectionError
+// so the UI can surface it explicitly rather than showing an empty section.
+func ParseComponentLogCollectionErrors(content []byte) []models.CollectionError {
+	type bmcErrorResponse struct {
+		Error string `json:"error"`
+		Code  int    `json:"code"`
+	}
+
+	// Map of section name (for display) → regex that captures its JSON payload.
+	// Sections that return arrays use \[ ... \]; object sections use \{ ... \}.
+	// We only probe sections that are expected to have structured hardware data.
+	sections := []struct {
+		name string
+		re   *regexp.Regexp
+	}{
+		{"HDD", regexp.MustCompile(`RESTful HDD info:\s*(\{[^\n]*\})`)},
+		{"PCIe Devices", regexp.MustCompile(`RESTful PCIE Device info:\s*(\{[^\n]*\})`)},
+		{"Network Adapters", regexp.MustCompile(`RESTful Network Adapter info:\s*(\{[^\n]*\})`)},
+		{"Disk Backplane", regexp.MustCompile(`RESTful diskbackplane info:\s*(\{[^\n]*\})`)},
+	}
+
+	text := string(content)
+	var out []models.CollectionError
+	for _, s := range sections {
+		m := s.re.FindStringSubmatch(text)
+		if m == nil {
+			continue
+		}
+		var errResp bmcErrorResponse
+		if err := json.Unmarshal([]byte(m[1]), &errResp); err != nil {
+			continue
+		}
+		if strings.TrimSpace(errResp.Error) == "" {
+			continue
+		}
+		out = append(out, models.CollectionError{
+			Section: s.name,
+			Message: errResp.Error,
+			Code:    errResp.Code,
+		})
+	}
+	return out
+}
+
 // ParseComponentLogSensors extracts sensor readings from component.log JSON sections.
 func ParseComponentLogSensors(content []byte) []models.SensorReading {
 	text := string(content)
@@ -61,6 +112,68 @@ func ParseComponentLogSensors(content []byte) []models.SensorReading {
 	return out
 }

+// CPURESTInfo represents the RESTful CPU info structure in component.log
+type CPURESTInfo struct {
+	Processors []struct {
+		ProcID       int    `json:"proc_id"`
+		CPUID        string `json:"PROC_ID"` // uppercase key — prevents case-insensitive collision with proc_id
+		Manufacturer string `json:"Manufacturer"`
+		MaxSpeedMHz  int    `json:"MaxSpeedMHz"`
+		ConfigStatus int    `json:"configStatus"`
+		ProcName     string `json:"proc_name"`
+		ProcStatus   int    `json:"proc_status"`
+		ProcSpeed    int    `json:"proc_speed"`
+		CoreCount    int    `json:"proc_core_count"`
+		ThreadCount  int    `json:"proc_thread_count"`
+		TDP          int    `json:"proc_tdp"`
+		L1Cache      int    `json:"proc_l1cache_size"`
+		L2Cache      int    `json:"proc_l2cache_size"`
+		L3Cache      int    `json:"proc_l3cache_size"`
+		MicroCode    string `json:"micro_code"`
+		PPIN         string `json:"ppin"`
+		Status       string `json:"status"`
+	} `json:"processors"`
+}
+
+func parseCPUInfo(text string, hw *models.HardwareConfig) {
+	re := regexp.MustCompile(`RESTful CPU info:\s*(\{[\s\S]*?\})\s*RESTful Memory`)
+	match := re.FindStringSubmatch(text)
+	if match == nil {
+		return
+	}
+
+	jsonStr := strings.ReplaceAll(match[1], "\n", "")
+	var cpuInfo CPURESTInfo
+	if err := json.Unmarshal([]byte(jsonStr), &cpuInfo); err != nil {
+		return
+	}
+
+	for _, proc := range cpuInfo.Processors {
+		if proc.ProcStatus != 1 && proc.ConfigStatus != 1 {
+			continue
+		}
+		hw.CPUs = append(hw.CPUs, models.CPU{
+			Socket:       proc.ProcID,
+			Model:        strings.TrimSpace(proc.ProcName),
+			Cores:        proc.CoreCount,
+			Threads:      proc.ThreadCount,
+			FrequencyMHz: proc.ProcSpeed,
+			MaxFreqMHz:   proc.MaxSpeedMHz,
+			L1CacheKB:    proc.L1Cache,
+			L2CacheKB:    proc.L2Cache,
+			L3CacheKB:    proc.L3Cache,
+			TDP:          proc.TDP,
+			PPIN:         proc.PPIN,
+		})
+		if proc.MicroCode != "" {
+			hw.Firmware = append(hw.Firmware, models.FirmwareInfo{
+				DeviceName: fmt.Sprintf("CPU%d Microcode", proc.ProcID),
+				Version:    proc.MicroCode,
+			})
+		}
+	}
+}
+
 // MemoryRESTInfo represents the RESTful Memory info structure
 type MemoryRESTInfo struct {
 	MemModules []struct {
@@ -112,9 +225,10 @@ func parseMemoryInfo(text string, hw *models.HardwareConfig) {
 	}
 	for _, mem := range memInfo.MemModules {
 		item := models.MemoryDIMM{
-			Slot:            mem.MemModSlot,
-			Location:        mem.MemModSlot,
-			Present:         mem.MemModStatus == 1 && mem.MemModSize > 0,
+			Slot:     mem.MemModSlot,
+			Location: mem.MemModSlot,
+			// status=1 with a known serial/part is definitely present even if BMC reports size=0
+			Present: mem.MemModStatus == 1 && (mem.MemModSize > 0 || strings.TrimSpace(mem.MemModSerial) != "" || strings.TrimSpace(mem.MemModPartNum) != ""),
 			SizeMB:          mem.MemModSize * 1024, // Convert GB to MB
 			Type:            mem.MemModType,
 			Technology:      strings.TrimSpace(mem.MemModTechnology),
@@ -136,6 +250,25 @@ func parseMemoryInfo(text string, hw *models.HardwareConfig) {
 		}
 		merged = append(merged, item)
 	}
+
+	// If a present DIMM has size=0 (BMC firmware glitch), infer size from
+	// another present DIMM with the same part number in the same batch.
+	partSize := make(map[string]int)
+	for _, m := range merged {
+		if m.Present && m.SizeMB > 0 && strings.TrimSpace(m.PartNumber) != "" {
+			partSize[strings.TrimSpace(m.PartNumber)] = m.SizeMB
+		}
+	}
+	for i := range merged {
+		if merged[i].Present && merged[i].SizeMB == 0 {
+			if pn := strings.TrimSpace(merged[i].PartNumber); pn != "" {
+				if sz, ok := partSize[pn]; ok {
+					merged[i].SizeMB = sz
+				}
+			}
+		}
+	}
+
 	hw.Memory = merged
 }

@@ -163,7 +296,7 @@ type PSURESTInfo struct {

 func parsePSUInfo(text string, hw *models.HardwareConfig) {
 	// Find RESTful PSU info section
-	re := regexp.MustCompile(`RESTful PSU info:\s*(\{[\s\S]*?\})\s*RESTful Network`)
+	re := regexp.MustCompile(`RESTful PSU info:\s*(\{[\s\S]*?\})\s*RESTful (?:PCIE|Network)`)
 	match := re.FindStringSubmatch(text)
 	if match == nil {
 		return
@@ -793,7 +926,7 @@ func parseDiskBackplaneSensors(text string) []models.SensorReading {
 }

 func parsePSUSummarySensors(text string) []models.SensorReading {
-	re := regexp.MustCompile(`RESTful PSU info:\s*(\{[\s\S]*?\})\s*RESTful Network`)
+	re := regexp.MustCompile(`RESTful PSU info:\s*(\{[\s\S]*?\})\s*RESTful (?:PCIE|Network)`)
 	match := re.FindStringSubmatch(text)
 	if match == nil {
 		return nil
@@ -941,7 +1074,7 @@ func extractComponentFirmware(text string, hw *models.HardwareConfig) {
 	// Skip extracting from component.log to avoid duplicates

 	// Extract PSU firmware from RESTful PSU info
-	rePSU := regexp.MustCompile(`RESTful PSU info:\s*(\{[\s\S]*?\})\s*RESTful Network`)
+	rePSU := regexp.MustCompile(`RESTful PSU info:\s*(\{[\s\S]*?\})\s*RESTful (?:PCIE|Network)`)
 	if match := rePSU.FindStringSubmatch(text); match != nil {
 		jsonStr := strings.ReplaceAll(match[1], "\n", "")
 		var psuInfo PSURESTInfo
--- a/internal/parser/vendors/inspur/cpu_mem_fix_test.go
+++ b/internal/parser/vendors/inspur/cpu_mem_fix_test.go
@@ -0,0 +1,83 @@
+package inspur
+
+import (
+	"strings"
+	"testing"
+
+	"git.mchus.pro/mchus/logpile/internal/models"
+)
+
+const cpuMemComponentLog = `RESTful version info:
+[]
+RESTful CPU info:
+{ "processors": [ { "proc_id": 0, "PROC_ID": "A6-06-06-00-FF-FB-EB-BF", "InstructionSet": "x86-64", "Manufacturer": "Intel(R) Corporation", "MaxSpeedMHz": 3100, "configStatus": 1, "proc_name": "Intel(R) Xeon(R) Gold 6330 CPU @ 2.00GHz", "proc_status": 1, "proc_speed": 2000, "proc_core_count": 28, "proc_used_core_count": 28, "proc_thread_count": 56, "proc_tdp": 205, "proc_l1cache_size": 80, "proc_l2cache_size": 1280, "proc_l3cache_size": 43008, "micro_code": "0x0D000410", "ppin": "47149E2253E81688", "status": "OK" }, { "proc_id": 1, "PROC_ID": "A6-06-06-00-FF-FB-EB-BF", "InstructionSet": "x86-64", "Manufacturer": "Intel(R) Corporation", "MaxSpeedMHz": 3100, "configStatus": 1, "proc_name": "Intel(R) Xeon(R) Gold 6330 CPU @ 2.00GHz", "proc_status": 1, "proc_speed": 2000, "proc_core_count": 28, "proc_thread_count": 56, "proc_tdp": 205, "proc_l1cache_size": 80, "proc_l2cache_size": 1280, "proc_l3cache_size": 43008, "micro_code": "0x0D000410", "ppin": "475AC1221D41F557", "status": "OK" } ] }
+RESTful Memory info:
+{ "mem_modules": [ { "mem_mod_id": 0, "config_status": 1, "mem_mod_slot": "CPU0_C0D0", "mem_mod_status": 1, "mem_mod_size": 32, "mem_mod_type": "DDR4", "mem_mod_technology": "Synchronous", "mem_mod_frequency": 3200, "mem_mod_current_frequency": 2933, "mem_mod_vendor": "Samsung", "mem_mod_part_num": "M393A4K40EB3-CWE", "mem_mod_serial_num": "S1440202433526FC12", "mem_mod_ranks": 2, "status": "OK" }, { "mem_mod_id": 16, "config_status": 1, "mem_mod_slot": "CPU1_C0D0", "mem_mod_status": 1, "mem_mod_size": 0, "mem_mod_type": "DDR4", "mem_mod_technology": "Synchronous", "mem_mod_frequency": 3200, "mem_mod_current_frequency": 2933, "mem_mod_vendor": "Samsung", "mem_mod_part_num": "M393A4K40EB3-CWE", "mem_mod_serial_num": "K0UX000401205D2037", "mem_mod_ranks": 2, "status": "OK" } ], "total_memory_count": 2, "present_memory_count": 2, "mem_total_mem_size": 32 }
+RESTful HDD info:
+[]
+RESTful PSU info:
+{ "power_supplies": [] }
+RESTful Network Adapter info:
+{ "sys_adapters": [] }
+RESTful fan info:
+{ "fans": [] }
+RESTful diskbackplane info:
+[]
+BMC done
+`
+
+func TestParseCPUInfo_FromComponentLog(t *testing.T) {
+	hw := &models.HardwareConfig{}
+	ParseComponentLog([]byte(cpuMemComponentLog), hw)
+
+	if len(hw.CPUs) != 2 {
+		t.Fatalf("expected 2 CPUs, got %d", len(hw.CPUs))
+	}
+	if !strings.Contains(hw.CPUs[0].Model, "Gold 6330") {
+		t.Errorf("unexpected CPU model: %s", hw.CPUs[0].Model)
+	}
+	if hw.CPUs[0].Cores != 28 {
+		t.Errorf("expected 28 cores, got %d", hw.CPUs[0].Cores)
+	}
+	if hw.CPUs[0].PPIN != "47149E2253E81688" {
+		t.Errorf("unexpected PPIN: %s", hw.CPUs[0].PPIN)
+	}
+	if hw.CPUs[1].PPIN != "475AC1221D41F557" {
+		t.Errorf("unexpected CPU1 PPIN: %s", hw.CPUs[1].PPIN)
+	}
+}
+
+func TestParseMemoryInfo_PresentWithZeroSize(t *testing.T) {
+	hw := &models.HardwareConfig{}
+	ParseComponentLog([]byte(cpuMemComponentLog), hw)
+
+	presentCount := 0
+	for _, m := range hw.Memory {
+		if m.Present {
+			presentCount++
+		}
+	}
+	if presentCount != 2 {
+		t.Errorf("expected 2 present DIMMs, got %d", presentCount)
+	}
+
+	// Find CPU1_C0D0 (size=0 but serial present — size should be inferred from same part number)
+	found := false
+	for _, m := range hw.Memory {
+		if m.Slot == "CPU1_C0D0" {
+			found = true
+			if !m.Present {
+				t.Error("CPU1_C0D0 should be Present=true despite size=0")
+			}
+			if m.SerialNumber != "K0UX000401205D2037" {
+				t.Errorf("wrong serial: %s", m.SerialNumber)
+			}
+			if m.SizeMB != 32768 {
+				t.Errorf("expected SizeMB=32768 inferred from part number, got %d", m.SizeMB)
+			}
+		}
+	}
+	if !found {
+		t.Error("CPU1_C0D0 not found in memory list")
+	}
+}
--- a/internal/parser/vendors/inspur/gpu_status.go
+++ b/internal/parser/vendors/inspur/gpu_status.go
@@ -56,10 +56,12 @@ func applyGPUStatusFromEvents(hw *models.HardwareConfig, events []models.Event)
 	}

 	for _, e := range relevantEvents {
+		// Deassert means the alarm was cleared: all GPUs return to OK.
+		isDeassert := strings.EqualFold(strings.TrimSpace(e.EventType), "Deassert")
 		faultySet := extractFaultyGPUSet(e.Description)
 		for idx, gpu := range gpuByIndex {
 			newStatus := "OK"
-			if faultySet[idx] {
+			if !isDeassert && faultySet[idx] {
 				newStatus = "Critical"
 				lastCriticalDetails[idx] = strings.TrimSpace(e.Description)
 			}
--- a/internal/parser/vendors/inspur/hgx_gpu_status_test.go
+++ b/internal/parser/vendors/inspur/hgx_gpu_status_test.go
@@ -155,6 +155,40 @@ func TestApplyGPUStatusFromEvents_UsesLatestEventAsCurrentStatusAndKeepsHistory(
 	}
 }

+func TestApplyGPUStatusFromEvents_DeassertClearsAllGPUs(t *testing.T) {
+	hw := &models.HardwareConfig{
+		GPUs: []models.GPU{
+			{Slot: "#GPU1"},
+			{Slot: "#GPU3"},
+			{Slot: "#GPU5"},
+			{Slot: "#GPU6"},
+		},
+	}
+
+	events := []models.Event{
+		{
+			ID:          "17FFB002",
+			EventType:   "Assert",
+			Timestamp:   time.Date(2026, 5, 27, 13, 6, 56, 0, time.FixedZone("UTC+8", 8*3600)),
+			Description: "PCIe Present mismatch BIOS Scan, BIOS miss F_GPU1 F_GPU3 F_GPU5 F_GPU6",
+		},
+		{
+			ID:          "17FFB002",
+			EventType:   "Deassert",
+			Timestamp:   time.Date(2026, 5, 27, 13, 15, 56, 0, time.FixedZone("UTC+8", 8*3600)),
+			Description: "PCIe Present mismatch BIOS Scan, BIOS miss F_GPU1 F_GPU3 F_GPU5 F_GPU6",
+		},
+	}
+
+	applyGPUStatusFromEvents(hw, events)
+
+	for _, gpu := range hw.GPUs {
+		if gpu.Status != "OK" {
+			t.Fatalf("expected %s to recover to OK after Deassert, got %q", gpu.Slot, gpu.Status)
+		}
+	}
+}
+
 func TestParseIDLLog_ParsesStructuredJSONLine(t *testing.T) {
 	content := []byte(`{ "MESSAGE": "|2026-01-12T23:05:18+08:00|PCIE|Assert|Critical|17FFB002|PCIe Present mismatch BIOS miss F_GPU6 - Assert|" }`)

--- a/internal/parser/vendors/inspur/idl.go
+++ b/internal/parser/vendors/inspur/idl.go
@@ -48,7 +48,7 @@ func ParseIDLLog(content []byte) []models.Event {
 		description = cleanDescription(description)

 		// Create unique key for deduplication
-		eventKey := eventID + "|" + description
+		eventKey := eventID + "|" + eventType + "|" + description
 		if seenEvents[eventKey] {
 			continue
 		}
--- a/internal/parser/vendors/inspur/parser.go
+++ b/internal/parser/vendors/inspur/parser.go
@@ -16,7 +16,7 @@ import (

 // parserVersion - version of this parser module
 // IMPORTANT: Increment this version when making changes to parser logic!
-const parserVersion = "1.8"
+const parserVersion = "2.1"

 func init() {
 	parser.Register(&Parser{})
@@ -163,6 +163,26 @@ func (p *Parser) Parse(files []parser.ExtractedFile) (*models.AnalysisResult, er
 		// (fan RPM, backplane temperature, PSU summary power, etc.).
 		componentSensors := ParseComponentLogSensors(f.Content)
 		result.Sensors = mergeSensorReadings(result.Sensors, componentSensors)
+
+		// Record sections where BMC itself returned an error instead of data,
+		// and mirror each one into the Events stream so they appear in the log viewer.
+		// Source is set to "BMC/<section>" so the viewer can show the specific module.
+		for _, ce := range ParseComponentLogCollectionErrors(f.Content) {
+			result.CollectionErrors = append(result.CollectionErrors, ce)
+			desc := ce.Message
+			if ce.Code != 0 {
+				desc = fmt.Sprintf("%s (code %d)", ce.Message, ce.Code)
+			}
+			result.Events = append(result.Events, models.Event{
+				ID:          fmt.Sprintf("bmc_collection_error_%s", strings.ToLower(strings.ReplaceAll(ce.Section, " ", "_"))),
+				Timestamp:   time.Time{}, // no timestamp available
+				Source:      fmt.Sprintf("BMC/%s", ce.Section),
+				SensorType:  "bmc_collection_error",
+				EventType:   "Collection Error",
+				Severity:    models.SeverityWarning,
+				Description: desc,
+			})
+		}
 	}

 	// Enrich runtime component data from Redis snapshot (serials, FW, telemetry),
@@ -214,6 +234,9 @@ func (p *Parser) Parse(files []parser.ExtractedFile) (*models.AnalysisResult, er
 	if result.Hardware != nil {
 		applyGPUStatusFromEvents(result.Hardware, result.Events)
 		enrichStorageFromSerialFallbackFiles(files, result.Hardware)
+		// Enrich storage serial numbers from smartd output in SOLHostCapture.log.
+		// Fills in serial, model, firmware for backplane slots that the BMC HDD API left empty.
+		enrichStorageFromSOLSmartd(files, result.Hardware)
 		// Apply RAID disk serials from audit.log (authoritative: last non-NULL SN change).
 		// These override redis/component.log serials which may be stale after disk replacement.
 		applyRAIDSlotSerials(result.Hardware, raidSlotSerials)
--- a/internal/parser/vendors/inspur/sol_smartd.go
+++ b/internal/parser/vendors/inspur/sol_smartd.go
@@ -0,0 +1,247 @@
+package inspur
+
+import (
+	"regexp"
+	"sort"
+	"strconv"
+	"strings"
+
+	"git.mchus.pro/mchus/logpile/internal/models"
+	"git.mchus.pro/mchus/logpile/internal/parser"
+)
+
+// solSmartdDeviceRe matches smartd device info lines from SOLHostCapture.log.
+// Example:
+//
+//	Device: /dev/sda [SAT], Micron_5400_MTFDDAK480TGA, S/N:2310400DC7E3, WWN:..., FW:D4CM003, 480 GB
+var solSmartdDeviceRe = regexp.MustCompile(
+	`Device: /dev/\S+ \[SAT\], (.+?), S/N:(\S+),.*?FW:(\S+), ([\d.]+) (GB|TB)`,
+)
+
+type solSmartdDevice struct {
+	Model    string
+	Serial   string
+	Firmware string
+	SizeGB   int
+}
+
+// parseSOLSmartdDevices extracts unique disk entries from SOLHostCapture.log content.
+// Deduplicates by serial number (case-insensitive); preserves first-seen order.
+func parseSOLSmartdDevices(content []byte) []solSmartdDevice {
+	seen := make(map[string]struct{})
+	var out []solSmartdDevice
+
+	for _, line := range strings.Split(string(content), "\n") {
+		m := solSmartdDeviceRe.FindStringSubmatch(line)
+		if m == nil {
+			continue
+		}
+		serial := strings.TrimSpace(m[2])
+		if serial == "" {
+			continue
+		}
+		key := strings.ToLower(serial)
+		if _, ok := seen[key]; ok {
+			continue
+		}
+		seen[key] = struct{}{}
+
+		sizeGB := parseSolSizeGB(m[4], m[5])
+		out = append(out, solSmartdDevice{
+			Model:    strings.TrimSpace(m[1]),
+			Serial:   serial,
+			Firmware: strings.TrimSpace(m[3]),
+			SizeGB:   sizeGB,
+		})
+	}
+	return out
+}
+
+// parseSolSizeGB converts smartd size string ("480", "3.84") + unit ("GB", "TB") to integer GB.
+// Uses decimal TB (1 TB = 1000 GB) matching disk manufacturer conventions.
+func parseSolSizeGB(value, unit string) int {
+	f, err := strconv.ParseFloat(value, 64)
+	if err != nil || f <= 0 {
+		return 0
+	}
+	if strings.EqualFold(unit, "TB") {
+		f *= 1000
+	}
+	return int(f + 0.5)
+}
+
+// enrichStorageFromSOLSmartd enriches the storage inventory using disk info from
+// SOLHostCapture.log (smartd startup messages). Both the log/ and runningdata/ copies
+// are processed; serials are deduplicated across both files.
+//
+// Enrichment priority:
+//  1. Exact model match to existing entries that are missing a serial.
+//  2. Positional assignment to present placeholder slots (no model, no serial).
+//  3. New entries added for any remaining devices.
+func enrichStorageFromSOLSmartd(files []parser.ExtractedFile, hw *models.HardwareConfig) {
+	if hw == nil {
+		return
+	}
+
+	solFiles := parser.FindFileByPattern(files, "SOLHostCapture.log")
+	if len(solFiles) == 0 {
+		return
+	}
+
+	// Collect unique devices from all SOL log copies.
+	seenSerial := make(map[string]struct{})
+	var devices []solSmartdDevice
+	for _, f := range solFiles {
+		for _, d := range parseSOLSmartdDevices(f.Content) {
+			key := strings.ToLower(d.Serial)
+			if _, ok := seenSerial[key]; ok {
+				continue
+			}
+			seenSerial[key] = struct{}{}
+			devices = append(devices, d)
+		}
+	}
+	if len(devices) == 0 {
+		return
+	}
+
+	// Skip devices whose serial already appears in the storage inventory.
+	existingSerials := make(map[string]struct{}, len(hw.Storage))
+	for _, dev := range hw.Storage {
+		sn := strings.ToLower(strings.TrimSpace(dev.SerialNumber))
+		if sn != "" {
+			existingSerials[sn] = struct{}{}
+		}
+	}
+	var newDevices []solSmartdDevice
+	for _, d := range devices {
+		if _, ok := existingSerials[strings.ToLower(d.Serial)]; !ok {
+			newDevices = append(newDevices, d)
+		}
+	}
+	if len(newDevices) == 0 {
+		return
+	}
+
+	// Pass 1: enrich existing entries that match by model (first-match wins per device).
+	remaining := solEnrichByModel(hw, newDevices)
+	if len(remaining) == 0 {
+		return
+	}
+
+	// Pass 2: assign to present placeholder slots (present=true, no model, no serial).
+	remaining = solEnrichByPlaceholder(hw, remaining)
+	if len(remaining) == 0 {
+		return
+	}
+
+	// Pass 3: add as new storage entries without a slot assignment.
+	for _, d := range remaining {
+		hw.Storage = append(hw.Storage, solMakeStorage(d))
+	}
+}
+
+// solEnrichByModel fills SerialNumber (and optionally Firmware/SizeGB) on existing storage
+// entries whose model matches the smartd model exactly. Returns unmatched devices.
+func solEnrichByModel(hw *models.HardwareConfig, devices []solSmartdDevice) []solSmartdDevice {
+	var unmatched []solSmartdDevice
+	for _, d := range devices {
+		matched := false
+		for i := range hw.Storage {
+			if strings.TrimSpace(hw.Storage[i].SerialNumber) != "" {
+				continue
+			}
+			if !strings.EqualFold(strings.TrimSpace(hw.Storage[i].Model), d.Model) {
+				continue
+			}
+			hw.Storage[i].SerialNumber = d.Serial
+			if strings.TrimSpace(hw.Storage[i].Firmware) == "" {
+				hw.Storage[i].Firmware = d.Firmware
+			}
+			if hw.Storage[i].SizeGB == 0 {
+				hw.Storage[i].SizeGB = d.SizeGB
+			}
+			matched = true
+			break
+		}
+		if !matched {
+			unmatched = append(unmatched, d)
+		}
+	}
+	return unmatched
+}
+
+// solEnrichByPlaceholder assigns smartd devices to present storage entries that have
+// neither a model nor a serial number, sorted by slot name. Returns unmatched devices.
+func solEnrichByPlaceholder(hw *models.HardwareConfig, devices []solSmartdDevice) []solSmartdDevice {
+	type slot struct {
+		index int
+		name  string
+	}
+	var placeholders []slot
+	for i := range hw.Storage {
+		if !hw.Storage[i].Present {
+			continue
+		}
+		if strings.TrimSpace(hw.Storage[i].SerialNumber) != "" {
+			continue
+		}
+		if strings.TrimSpace(hw.Storage[i].Model) != "" {
+			continue
+		}
+		placeholders = append(placeholders, slot{index: i, name: hw.Storage[i].Slot})
+	}
+	sort.Slice(placeholders, func(i, j int) bool {
+		return placeholders[i].name < placeholders[j].name
+	})
+
+	pi := 0
+	var unmatched []solSmartdDevice
+	for _, d := range devices {
+		if pi >= len(placeholders) {
+			unmatched = append(unmatched, d)
+			continue
+		}
+		idx := placeholders[pi].index
+		pi++
+		hw.Storage[idx].SerialNumber = d.Serial
+		hw.Storage[idx].Model = d.Model
+		hw.Storage[idx].Firmware = d.Firmware
+		if hw.Storage[idx].SizeGB == 0 {
+			hw.Storage[idx].SizeGB = d.SizeGB
+		}
+		hw.Storage[idx].Type = solStorageType(d.Model)
+		if hw.Storage[idx].Manufacturer == "" {
+			hw.Storage[idx].Manufacturer = extractStorageManufacturer(d.Model)
+		}
+		if hw.Storage[idx].Interface == "" {
+			hw.Storage[idx].Interface = "SATA"
+		}
+	}
+	return unmatched
+}
+
+func solMakeStorage(d solSmartdDevice) models.Storage {
+	return models.Storage{
+		Model:        d.Model,
+		SerialNumber: d.Serial,
+		Firmware:     d.Firmware,
+		SizeGB:       d.SizeGB,
+		Type:         solStorageType(d.Model),
+		Manufacturer: extractStorageManufacturer(d.Model),
+		Interface:    "SATA",
+		Present:      true,
+	}
+}
+
+// solStorageType infers SSD vs HDD from the model string.
+// Micron SSD models start with "MTFDD"; Intel SSDs contain "SSD".
+func solStorageType(model string) string {
+	upper := strings.ToUpper(model)
+	if strings.Contains(upper, "SSD") ||
+		strings.HasPrefix(upper, "MTFDD") ||
+		strings.HasPrefix(upper, "MICRON_5") {
+		return "SSD"
+	}
+	return "HDD"
+}
--- a/internal/parser/vendors/inspur/sol_smartd_test.go
+++ b/internal/parser/vendors/inspur/sol_smartd_test.go
@@ -0,0 +1,191 @@
+package inspur
+
+import (
+	"strings"
+	"testing"
+
+	"git.mchus.pro/mchus/logpile/internal/models"
+	"git.mchus.pro/mchus/logpile/internal/parser"
+)
+
+const solSmartdSample = `
+[   17.219818] smartd[3321]: Device: /dev/sda [SAT], Micron_5400_MTFDDAK480TGA, S/N:2310400DC7E3, WWN:5-00a075-1400dc7e3, FW:D4CM003, 480 GB
+[   17.553024] smartd[3321]: Device: /dev/sdc [SAT], MTFDDAK3T8TGA-1BC1ZABDA, S/N:25134F172DB3, WWN:5-00a075-14f172db3, FW:D4DK403, 3.84 TB
+[   17.553331] smartd[3321]: Device: /dev/sde [SAT], Micron_5400_MTFDDAK480TGA, S/N:2310400DC80F, WWN:5-00a075-1400dc80f, FW:D4CM003, 480 GB
+[   17.553709] smartd[3321]: Device: /dev/sdh [SAT], MTFDDAK3T8TGA-1BC1ZABDA, S/N:25134F57DAB8, WWN:5-00a075-14f57dab8, FW:D4DK403, 3.84 TB
+[   17.886180] smartd[3321]: Device: /dev/sda [SAT], state written to /var/lib/smartmontools/smartd.Micron-2310400DC7E3.ata.state
+`
+
+func TestParseSOLSmartdDevices_Dedup(t *testing.T) {
+	devices := parseSOLSmartdDevices([]byte(solSmartdSample))
+	if len(devices) != 4 {
+		t.Fatalf("expected 4 unique devices, got %d: %v", len(devices), devices)
+	}
+	// order matches first-seen
+	if devices[0].Serial != "2310400DC7E3" {
+		t.Errorf("first device serial: got %q, want 2310400DC7E3", devices[0].Serial)
+	}
+	if devices[0].SizeGB != 480 {
+		t.Errorf("first device size: got %d, want 480", devices[0].SizeGB)
+	}
+	if devices[1].SizeGB != 3840 {
+		t.Errorf("TB device size: got %d, want 3840", devices[1].SizeGB)
+	}
+	if devices[1].Firmware != "D4DK403" {
+		t.Errorf("firmware: got %q, want D4DK403", devices[1].Firmware)
+	}
+}
+
+func TestParseSOLSmartdDevices_SkipsNonInfoLines(t *testing.T) {
+	content := `
+[   17.886177] smartd[3321]: Device: /dev/sda [SAT], state written to /var/lib/smartmontools/smartd.foo.ata.state
+[   17.040843] smartd[3321]: Device: /dev/sda [SAT], not found in smartd database 7.3/5319.
+[   17.040865] smartd[3321]: Device: /dev/sda [SAT], is SMART capable. Adding to "monitor" list.
+`
+	devices := parseSOLSmartdDevices([]byte(content))
+	if len(devices) != 0 {
+		t.Errorf("expected 0 devices, got %d", len(devices))
+	}
+}
+
+func TestParseSolSizeGB(t *testing.T) {
+	cases := []struct {
+		value, unit string
+		want        int
+	}{
+		{"480", "GB", 480},
+		{"1.92", "TB", 1920},
+		{"3.84", "TB", 3840},
+		{"1", "TB", 1000},
+		{"0", "GB", 0},
+	}
+	for _, c := range cases {
+		got := parseSolSizeGB(c.value, c.unit)
+		if got != c.want {
+			t.Errorf("parseSolSizeGB(%q, %q) = %d, want %d", c.value, c.unit, got, c.want)
+		}
+	}
+}
+
+func TestSolStorageType(t *testing.T) {
+	cases := []struct {
+		model string
+		want  string
+	}{
+		{"MTFDDAK3T8TGA-1BC1ZABDA", "SSD"},
+		{"Micron_5400_MTFDDAK480TGA", "SSD"},
+		{"INTEL SSDSC2KB019TZ", "SSD"},
+		{"SEAGATE ST4000NM0115", "HDD"},
+	}
+	for _, c := range cases {
+		got := solStorageType(c.model)
+		if got != c.want {
+			t.Errorf("solStorageType(%q) = %q, want %q", c.model, got, c.want)
+		}
+	}
+}
+
+func TestEnrichStorageFromSOLSmartd_ModelMatch(t *testing.T) {
+	files := []parser.ExtractedFile{
+		{
+			Path:    "onekeylog/log/sollog/SOLHostCapture.log",
+			Content: []byte(solSmartdSample),
+		},
+	}
+	hw := &models.HardwareConfig{
+		Storage: []models.Storage{
+			{Slot: "BP0:0", Model: "MTFDDAK3T8TGA-1BC1ZABDA", SizeGB: 3576, Present: true},
+			{Slot: "BP0:1", Model: "MTFDDAK3T8TGA-1BC1ZABDA", SizeGB: 3576, Present: true},
+		},
+	}
+
+	enrichStorageFromSOLSmartd(files, hw)
+
+	// The two existing slots must have received serials via model match.
+	for _, s := range hw.Storage[:2] {
+		if s.SerialNumber == "" {
+			t.Errorf("slot %q: expected serial to be assigned via model match", s.Slot)
+		}
+		if s.SizeGB != 3576 {
+			t.Errorf("slot %q: size should be preserved, got %d", s.Slot, s.SizeGB)
+		}
+	}
+	// The two unmatched Micron entries should be added as new storage entries.
+	if len(hw.Storage) != 4 {
+		t.Errorf("expected 4 total storage entries (2 existing + 2 new Micron), got %d", len(hw.Storage))
+	}
+}
+
+func TestEnrichStorageFromSOLSmartd_PlaceholderSlots(t *testing.T) {
+	files := []parser.ExtractedFile{
+		{
+			Path:    "onekeylog/log/sollog/SOLHostCapture.log",
+			Content: []byte(solSmartdSample),
+		},
+	}
+	hw := &models.HardwareConfig{
+		Storage: []models.Storage{
+			{Slot: "BP0:0", Present: true},
+			{Slot: "BP0:1", Present: true},
+		},
+	}
+
+	enrichStorageFromSOLSmartd(files, hw)
+
+	for _, s := range hw.Storage {
+		if s.SerialNumber == "" {
+			t.Errorf("slot %q: expected serial to be assigned", s.Slot)
+		}
+		if s.Model == "" {
+			t.Errorf("slot %q: expected model to be assigned", s.Slot)
+		}
+	}
+}
+
+func TestEnrichStorageFromSOLSmartd_SkipsExistingSerial(t *testing.T) {
+	files := []parser.ExtractedFile{
+		{
+			Path:    "onekeylog/log/sollog/SOLHostCapture.log",
+			Content: []byte(solSmartdSample),
+		},
+	}
+	hw := &models.HardwareConfig{
+		Storage: []models.Storage{
+			{Slot: "BP0:0", SerialNumber: "2310400DC7E3", Present: true},
+		},
+	}
+	before := len(hw.Storage)
+
+	enrichStorageFromSOLSmartd(files, hw)
+
+	// BP0:0 should still have original serial unchanged
+	if hw.Storage[0].SerialNumber != "2310400DC7E3" {
+		t.Errorf("existing serial was changed: got %q", hw.Storage[0].SerialNumber)
+	}
+	// Remaining 3 devices should be added as new entries
+	if len(hw.Storage) <= before {
+		t.Errorf("expected new entries to be added, got %d (same as before)", len(hw.Storage))
+	}
+}
+
+func TestEnrichStorageFromSOLSmartd_MergesTwoFiles(t *testing.T) {
+	// Two SOL files with partial overlap; combined unique serials = 3
+	file1 := `[   17.0] smartd[1]: Device: /dev/sda [SAT], ModelA, S/N:SN001, WWN:w, FW:fw1, 480 GB`
+	file2 := strings.Join([]string{
+		`[   17.0] smartd[2]: Device: /dev/sda [SAT], ModelA, S/N:SN001, WWN:w, FW:fw1, 480 GB`,
+		`[   17.1] smartd[2]: Device: /dev/sdb [SAT], ModelB, S/N:SN002, WWN:w, FW:fw2, 480 GB`,
+		`[   17.2] smartd[2]: Device: /dev/sdc [SAT], ModelC, S/N:SN003, WWN:w, FW:fw3, 480 GB`,
+	}, "\n")
+
+	files := []parser.ExtractedFile{
+		{Path: "log/sollog/SOLHostCapture.log", Content: []byte(file1)},
+		{Path: "runningdata/var/sollog/SOLHostCapture.log", Content: []byte(file2)},
+	}
+	hw := &models.HardwareConfig{}
+
+	enrichStorageFromSOLSmartd(files, hw)
+
+	if len(hw.Storage) != 3 {
+		t.Fatalf("expected 3 unique storage entries, got %d", len(hw.Storage))
+	}
+}
--- a/internal/parser/vendors/pciids/pci.ids
+++ b/internal/parser/vendors/pciids/pci.ids
--- a/internal/server/chart_view_test.go
+++ b/internal/server/chart_view_test.go
@@ -44,7 +44,10 @@ func TestHandleChartCurrent_RendersCurrentReanimatorSnapshot(t *testing.T) {
 		t.Fatalf("expected chart title in body, got %q", body)
 	}
 	if !strings.Contains(body, `/chart/static/view.css`) {
-		t.Fatalf("expected rewritten chart static path, got %q", body)
+		t.Fatalf("expected rewritten chart css path, got %q", body)
+	}
+	if !strings.Contains(body, `/chart/static/view.js`) {
+		t.Fatalf("expected rewritten chart js path, got %q", body)
 	}
 	if !strings.Contains(body, "Snapshot Metadata") {
 		t.Fatalf("expected rendered chart output, got %q", body)
--- a/internal/server/collect_types.go
+++ b/internal/server/collect_types.go
@@ -38,18 +38,21 @@ type CollectJobResponse struct {
 }

 type CollectJobStatusResponse struct {
-	JobID         string                `json:"job_id"`
-	Status        string                `json:"status"`
-	Progress      *int                  `json:"progress,omitempty"`
-	CurrentPhase  string                `json:"current_phase,omitempty"`
-	ETASeconds    *int                  `json:"eta_seconds,omitempty"`
-	Logs          []string              `json:"logs,omitempty"`
-	Error         string                `json:"error,omitempty"`
-	ActiveModules []CollectModuleStatus `json:"active_modules,omitempty"`
-	ModuleScores  []CollectModuleStatus `json:"module_scores,omitempty"`
-	DebugInfo     *CollectDebugInfo     `json:"debug_info,omitempty"`
-	CreatedAt     time.Time             `json:"created_at,omitempty"`
-	UpdatedAt     time.Time             `json:"updated_at"`
+	JobID         string                 `json:"job_id"`
+	Type          string                 `json:"type,omitempty"`
+	Status        string                 `json:"status"`
+	Progress      *int                   `json:"progress,omitempty"`
+	Message       string                 `json:"message,omitempty"`
+	CurrentPhase  string                 `json:"current_phase,omitempty"`
+	ETASeconds    *int                   `json:"eta_seconds,omitempty"`
+	Logs          []string               `json:"logs,omitempty"`
+	Error         string                 `json:"error,omitempty"`
+	Result        map[string]interface{} `json:"result,omitempty"`
+	ActiveModules []CollectModuleStatus  `json:"active_modules,omitempty"`
+	ModuleScores  []CollectModuleStatus  `json:"module_scores,omitempty"`
+	DebugInfo     *CollectDebugInfo      `json:"debug_info,omitempty"`
+	CreatedAt     time.Time              `json:"created_at,omitempty"`
+	UpdatedAt     time.Time              `json:"updated_at"`
 }

 type CollectRequestMeta struct {
@@ -63,12 +66,15 @@ type CollectRequestMeta struct {

 type Job struct {
 	ID            string
+	Type          string
 	Status        string
 	Progress      int
+	Message       string
 	CurrentPhase  string
 	ETASeconds    int
 	Logs          []string
 	Error         string
+	Result        map[string]interface{}
 	ActiveModules []CollectModuleStatus
 	ModuleScores  []CollectModuleStatus
 	DebugInfo     *CollectDebugInfo
@@ -107,11 +113,14 @@ func (j *Job) toStatusResponse() CollectJobStatusResponse {
 	progress := j.Progress
 	resp := CollectJobStatusResponse{
 		JobID:         j.ID,
+		Type:          j.Type,
 		Status:        j.Status,
 		Progress:      &progress,
+		Message:       j.Message,
 		CurrentPhase:  j.CurrentPhase,
 		Logs:          append([]string(nil), j.Logs...),
 		Error:         j.Error,
+		Result:        j.Result,
 		ActiveModules: append([]CollectModuleStatus(nil), j.ActiveModules...),
 		ModuleScores:  append([]CollectModuleStatus(nil), j.ModuleScores...),
 		DebugInfo:     cloneCollectDebugInfo(j.DebugInfo),
--- a/internal/server/device_repository_test.go
+++ b/internal/server/device_repository_test.go
@@ -174,7 +174,7 @@ func TestBuildSpecification_ZeroSizeMemoryWithInventoryIsShown(t *testing.T) {

 	spec := buildSpecification(hw)
 	for _, line := range spec {
-		if line.Category == "Память" && line.Name == "Hynix HMCG88AEBRA115N (size unknown)" && line.Quantity == 1 {
+		if line.Category == "Memory" && line.Name == "Hynix HMCG88AEBRA115N (size unknown)" && line.Quantity == 1 {
 			return
 		}
 	}
--- a/internal/server/handlers.go
+++ b/internal/server/handlers.go
@@ -38,13 +38,13 @@ func (s *Server) handleIndex(w http.ResponseWriter, r *http.Request) {

 	tmplContent, err := WebFS.ReadFile("templates/index.html")
 	if err != nil {
-		http.Error(w, "Template not found", http.StatusInternalServerError)
+		s.htmlError(w, "Template not found", http.StatusInternalServerError)
 		return
 	}

 	tmpl, err := template.New("index").Parse(string(tmplContent))
 	if err != nil {
-		http.Error(w, "Template parse error", http.StatusInternalServerError)
+		s.htmlError(w, "Template parse error", http.StatusInternalServerError)
 		return
 	}

@@ -70,7 +70,7 @@ func (s *Server) handleChartCurrent(w http.ResponseWriter, r *http.Request) {
 	if result == nil || result.Hardware == nil {
 		html, err := chartviewer.RenderHTML(nil, title)
 		if err != nil {
-			http.Error(w, "failed to render viewer", http.StatusInternalServerError)
+			s.htmlError(w, "failed to render viewer", http.StatusInternalServerError)
 			return
 		}
 		w.Header().Set("Content-Type", "text/html; charset=utf-8")
@@ -80,13 +80,13 @@ func (s *Server) handleChartCurrent(w http.ResponseWriter, r *http.Request) {

 	snapshotBytes, err := currentReanimatorSnapshotBytes(result)
 	if err != nil {
-		http.Error(w, "failed to build reanimator snapshot: "+err.Error(), http.StatusInternalServerError)
+		s.htmlError(w, "failed to build reanimator snapshot: "+err.Error(), http.StatusInternalServerError)
 		return
 	}

-	html, err := chartviewer.RenderHTML(snapshotBytes, title)
+	html, err := chartviewer.RenderHTMLWithOptions(snapshotBytes, title, chartviewer.RenderOptions{})
 	if err != nil {
-		http.Error(w, "failed to render chart: "+err.Error(), http.StatusInternalServerError)
+		s.htmlError(w, "failed to render chart: "+err.Error(), http.StatusInternalServerError)
 		return
 	}

@@ -137,7 +137,9 @@ func chartTitle(result *models.AnalysisResult) string {
 }

 func rewriteChartStaticPaths(html []byte) []byte {
-	return bytes.ReplaceAll(html, []byte(`href="/static/view.css"`), []byte(`href="/chart/static/view.css"`))
+	html = bytes.ReplaceAll(html, []byte(`href="/static/view.css"`), []byte(`href="/chart/static/view.css"`))
+	html = bytes.ReplaceAll(html, []byte(`src="/static/view.js"`), []byte(`src="/chart/static/view.js"`))
+	return html
 }

 func (s *Server) handleUpload(w http.ResponseWriter, r *http.Request) {
@@ -392,7 +394,7 @@ func uniqueSortedExtensions(exts []string) []string {
 func (s *Server) handleGetEvents(w http.ResponseWriter, r *http.Request) {
 	result := s.GetResult()
 	if result == nil {
-		jsonResponse(w, []interface{}{})
+		jsonList(w, []interface{}{}, 0)
 		return
 	}

@@ -405,18 +407,18 @@ func (s *Server) handleGetEvents(w http.ResponseWriter, r *http.Request) {
 		return events[i].Timestamp.After(events[j].Timestamp)
 	})

-	jsonResponse(w, events)
+	jsonList(w, events, len(events))
 }

 func (s *Server) handleGetSensors(w http.ResponseWriter, r *http.Request) {
 	result := s.GetResult()
 	if result == nil {
-		jsonResponse(w, []interface{}{})
+		jsonList(w, []interface{}{}, 0)
 		return
 	}
 	sensors := append([]models.SensorReading{}, result.Sensors...)
 	sensors = append(sensors, synthesizePSUVoltageSensors(result.Hardware)...)
-	jsonResponse(w, sensors)
+	jsonList(w, sensors, len(sensors))
 }

 func synthesizePSUVoltageSensors(hw *models.HardwareConfig) []models.SensorReading {
@@ -530,7 +532,7 @@ func buildSpecification(hw *models.HardwareConfig) []SpecLine {
 			float64(cpu.FrequencyMHz)/1000,
 			cpu.Cores,
 			intFromDetails(cpu.Details, "tdp_w"))
-		spec = append(spec, SpecLine{Category: "Процессор", Name: name, Quantity: count})
+		spec = append(spec, SpecLine{Category: "CPU", Name: name, Quantity: count})
 	}

 	// Memory - group by size, type and frequency (only installed modules)
@@ -565,7 +567,7 @@ func buildSpecification(hw *models.HardwareConfig) []SpecLine {
 		memGroups[key]++
 	}
 	for key, count := range memGroups {
-		spec = append(spec, SpecLine{Category: "Память", Name: key, Quantity: count})
+		spec = append(spec, SpecLine{Category: "Memory", Name: key, Quantity: count})
 	}

 	// Storage - group by type and capacity
@@ -583,7 +585,7 @@ func buildSpecification(hw *models.HardwareConfig) []SpecLine {
 		storGroups[key]++
 	}
 	for key, count := range storGroups {
-		spec = append(spec, SpecLine{Category: "Накопитель", Name: key, Quantity: count})
+		spec = append(spec, SpecLine{Category: "Storage", Name: key, Quantity: count})
 	}

 	// PCIe devices - group by device class/name and manufacturer
@@ -606,7 +608,7 @@ func buildSpecification(hw *models.HardwareConfig) []SpecLine {
 	}
 	for key, count := range pcieGroups {
 		pcie := pcieDetails[key]
-		category := "PCIe устройство"
+		category := "PCIe Device"
 		name := key

 		// Determine category based on device class or known GPU names
@@ -615,11 +617,11 @@ func buildSpecification(hw *models.HardwareConfig) []SpecLine {
 		isNetwork := deviceClass == "Network" || strings.Contains(deviceClass, "ConnectX")

 		if isGPU {
-			category = "Графический процессор"
+			category = "GPU"
 		} else if isNetwork {
-			category = "Сетевой адаптер"
+			category = "Network Adapter"
 		} else if deviceClass == "NVMe" || deviceClass == "RAID" || deviceClass == "SAS" || deviceClass == "SATA" || deviceClass == "Storage" {
-			category = "Контроллер"
+			category = "Controller"
 		}

 		spec = append(spec, SpecLine{Category: category, Name: name, Quantity: count})
@@ -640,7 +642,7 @@ func buildSpecification(hw *models.HardwareConfig) []SpecLine {
 		}
 	}
 	for key, count := range psuGroups {
-		spec = append(spec, SpecLine{Category: "Блок питания", Name: key, Quantity: count})
+		spec = append(spec, SpecLine{Category: "Power Supply", Name: key, Quantity: count})
 	}

 	return spec
@@ -661,7 +663,7 @@ func nonEmptyStrings(values ...string) []string {
 func (s *Server) handleGetSerials(w http.ResponseWriter, r *http.Request) {
 	result := s.GetResult()
 	if result == nil {
-		jsonResponse(w, []interface{}{})
+		jsonList(w, []interface{}{}, 0)
 		return
 	}

@@ -711,7 +713,7 @@ func (s *Server) handleGetSerials(w http.ResponseWriter, r *http.Request) {
 		}
 	}

-	jsonResponse(w, serials)
+	jsonList(w, serials, len(serials))
 }

 func normalizePCIeSerialComponentName(p models.PCIeDevice) string {
@@ -765,11 +767,12 @@ func hasUsableFirmwareVersion(version string) bool {
 func (s *Server) handleGetFirmware(w http.ResponseWriter, r *http.Request) {
 	result := s.GetResult()
 	if result == nil || result.Hardware == nil {
-		jsonResponse(w, []interface{}{})
+		jsonList(w, []interface{}{}, 0)
 		return
 	}

-	jsonResponse(w, buildFirmwareEntries(result.Hardware))
+	entries := buildFirmwareEntries(result.Hardware)
+	jsonList(w, entries, len(entries))
 }

 type parseErrorEntry struct {
@@ -854,6 +857,28 @@ func (s *Server) handleGetParseErrors(w http.ResponseWriter, r *http.Request) {
 		}
 	}

+	// BMC-reported collection failures surfaced by vendor parsers.
+	if result != nil {
+		for _, ce := range result.CollectionErrors {
+			msg := strings.TrimSpace(ce.Message)
+			if msg == "" {
+				continue
+			}
+			detail := ""
+			if ce.Code != 0 {
+				detail = fmt.Sprintf("code %d", ce.Code)
+			}
+			add(parseErrorEntry{
+				Source:   "bmc",
+				Category: "bmc_collection_error",
+				Severity: "warning",
+				Path:     ce.Section,
+				Message:  msg,
+				Detail:   detail,
+			})
+		}
+	}
+
 	sort.Slice(items, func(i, j int) bool {
 		if items[i].Severity != items[j].Severity {
 			// error > warning > info
@@ -916,8 +941,7 @@ func looksLikeErrorLogLine(line string) bool {
 	if s == "" {
 		return false
 	}
-	return strings.Contains(s, "ошибка") ||
-		strings.Contains(s, "error") ||
+	return strings.Contains(s, "error") ||
 		strings.Contains(s, "failed") ||
 		strings.Contains(s, "timeout") ||
 		strings.Contains(s, "deadline exceeded")
@@ -952,7 +976,7 @@ func parseErrorSeverityFromMessage(msg string) string {
 	if strings.HasPrefix(s, "status 404 ") || strings.HasPrefix(s, "status 405 ") || strings.HasPrefix(s, "status 410 ") || strings.HasPrefix(s, "status 501 ") {
 		return "info"
 	}
-	if strings.Contains(s, "ошибка") || strings.Contains(s, "error") || strings.Contains(s, "failed") {
+	if strings.Contains(s, "error") || strings.Contains(s, "failed") {
 		return "warning"
 	}
 	return "info"
@@ -1210,6 +1234,13 @@ func (s *Server) handleExportCSV(w http.ResponseWriter, r *http.Request) {
 	exp.ExportCSV(w)
 }

+func (s *Server) handleExportLogsCSV(w http.ResponseWriter, r *http.Request) {
+	result := s.GetResult()
+	w.Header().Set("Content-Type", "text/csv; charset=utf-8")
+	w.Header().Set("Content-Disposition", fmt.Sprintf("attachment; filename=%q", exportFilename(result, "logs.csv")))
+	exporter.ExportLogsCSV(w, result)
+}
+
 func (s *Server) handleExportJSON(w http.ResponseWriter, r *http.Request) {
 	result := s.GetResult()

@@ -1291,7 +1322,7 @@ func (s *Server) handleConvertReanimatorBatch(w http.ResponseWriter, r *http.Req

 	tempDir, err := os.MkdirTemp("", "logpile-convert-input-*")
 	if err != nil {
-		jsonError(w, "Не удалось создать временную директорию", http.StatusInternalServerError)
+		jsonError(w, "Failed to create temp directory", http.StatusInternalServerError)
 		return
 	}

@@ -1338,7 +1369,7 @@ func (s *Server) handleConvertReanimatorBatch(w http.ResponseWriter, r *http.Req

 	if len(inputFiles) == 0 {
 		_ = os.RemoveAll(tempDir)
-		jsonError(w, "Нет файлов поддерживаемого типа для конвертации", http.StatusBadRequest)
+		jsonError(w, "No supported files to convert", http.StatusBadRequest)
 		return
 	}

@@ -1351,9 +1382,9 @@ func (s *Server) handleConvertReanimatorBatch(w http.ResponseWriter, r *http.Req
 		TLSMode:  "insecure",
 	})
 	s.markConvertJob(job.ID)
-	s.jobManager.AppendJobLog(job.ID, fmt.Sprintf("Запущена пакетная конвертация: %d файлов", len(inputFiles)))
+	s.jobManager.AppendJobLog(job.ID, fmt.Sprintf("Batch conversion started: %d files", len(inputFiles)))
 	if skipped > 0 {
-		s.jobManager.AppendJobLog(job.ID, fmt.Sprintf("Пропущено неподдерживаемых файлов: %d", skipped))
+		s.jobManager.AppendJobLog(job.ID, fmt.Sprintf("Skipped unsupported files: %d", skipped))
 	}
 	s.jobManager.UpdateJobStatus(job.ID, CollectStatusRunning, 0, "")

@@ -1381,7 +1412,7 @@ func (s *Server) runConvertJob(jobID, tempDir string, inputFiles []convertInputF

 	resultFile, err := os.CreateTemp("", "logpile-convert-result-*.zip")
 	if err != nil {
-		s.jobManager.UpdateJobStatus(jobID, CollectStatusFailed, 100, "не удалось создать zip")
+		s.jobManager.UpdateJobStatus(jobID, CollectStatusFailed, 100, "failed to create zip")
 		return
 	}
 	resultPath := resultFile.Name()
@@ -1393,7 +1424,7 @@ func (s *Server) runConvertJob(jobID, tempDir string, inputFiles []convertInputF
 	totalProcess := len(inputFiles)

 	for i, in := range inputFiles {
-		s.jobManager.AppendJobLog(jobID, fmt.Sprintf("Обработка %s", in.Name))
+		s.jobManager.AppendJobLog(jobID, fmt.Sprintf("Processing %s", in.Name))
 		payload, err := os.ReadFile(in.Path)
 		if err != nil {
 			failures = append(failures, fmt.Sprintf("%s: %v", in.Name, err))
@@ -1446,13 +1477,13 @@ func (s *Server) runConvertJob(jobID, tempDir string, inputFiles []convertInputF
 	if success == 0 {
 		_ = zw.Close()
 		_ = os.Remove(resultPath)
-		s.jobManager.UpdateJobStatus(jobID, CollectStatusFailed, 100, "Не удалось конвертировать ни один файл")
+		s.jobManager.UpdateJobStatus(jobID, CollectStatusFailed, 100, "Failed to convert any file")
 		return
 	}

-	summaryLines := []string{fmt.Sprintf("Конвертировано %d из %d файлов", success, total)}
+	summaryLines := []string{fmt.Sprintf("Converted %d of %d files", success, total)}
 	if skipped > 0 {
-		summaryLines = append(summaryLines, fmt.Sprintf("Пропущено неподдерживаемых: %d", skipped))
+		summaryLines = append(summaryLines, fmt.Sprintf("Skipped unsupported: %d", skipped))
 	}
 	summaryLines = append(summaryLines, failures...)
 	if entry, err := zw.Create("convert-summary.txt"); err == nil {
@@ -1460,7 +1491,7 @@ func (s *Server) runConvertJob(jobID, tempDir string, inputFiles []convertInputF
 	}
 	if err := zw.Close(); err != nil {
 		_ = os.Remove(resultPath)
-		s.jobManager.UpdateJobStatus(jobID, CollectStatusFailed, 100, "Не удалось упаковать результаты")
+		s.jobManager.UpdateJobStatus(jobID, CollectStatusFailed, 100, "Failed to pack results")
 		return
 	}

@@ -1613,7 +1644,7 @@ func (s *Server) handleCollectStart(w http.ResponseWriter, r *http.Request) {
 	}

 	job := s.jobManager.CreateJob(req)
-	s.jobManager.AppendJobLog(job.ID, "Клиент: "+s.ClientVersionString())
+	s.jobManager.AppendJobLog(job.ID, "Client: "+s.ClientVersionString())
 	s.startCollectionJob(job.ID, req)

 	w.Header().Set("Content-Type", "application/json")
@@ -1642,7 +1673,7 @@ func pingHost(host string, port int, total, need int) (bool, string) {
 	}
 	n := int(successes.Load())
 	if n < need {
-		return false, fmt.Sprintf("Хост недоступен: только %d из %d попыток подключения к %s прошли успешно (требуется минимум %d)", n, total, addr, need)
+		return false, fmt.Sprintf("Host unreachable: only %d of %d connection attempts to %s succeeded (minimum required: %d)", n, total, addr, need)
 	}
 	return true, ""
 }
@@ -1659,12 +1690,12 @@ func (s *Server) handleCollectProbe(w http.ResponseWriter, r *http.Request) {
 	}
 	connector, ok := s.getCollector(req.Protocol)
 	if !ok {
-		jsonError(w, "Коннектор для протокола не зарегистрирован", http.StatusBadRequest)
+		jsonError(w, "Protocol connector not registered", http.StatusBadRequest)
 		return
 	}
 	prober, ok := connector.(collector.Prober)
 	if !ok {
-		jsonError(w, "Проверка подключения для протокола не поддерживается", http.StatusBadRequest)
+		jsonError(w, "Connection probe not supported for this protocol", http.StatusBadRequest)
 		return
 	}

@@ -1678,16 +1709,16 @@ func (s *Server) handleCollectProbe(w http.ResponseWriter, r *http.Request) {

 	result, err := prober.Probe(ctx, toCollectorRequest(req))
 	if err != nil {
-		jsonError(w, "Проверка подключения не удалась: "+err.Error(), http.StatusBadRequest)
+		jsonError(w, "Connection probe failed: "+err.Error(), http.StatusBadRequest)
 		return
 	}

-	message := "Связь с BMC установлена"
+	message := "BMC connection established"
 	if result != nil {
 		if result.HostPoweredOn {
-			message = "Связь с BMC установлена, host включён."
+			message = "BMC connection established, host is powered on."
 		} else {
-			message = "Связь с BMC установлена, host выключен. Данные инвентаря могут быть неполными."
+			message = "BMC connection established, host is powered off. Inventory data may be incomplete."
 		}
 	}

@@ -1772,8 +1803,8 @@ func (s *Server) startCollectionJob(jobID string, req CollectRequest) {
 	go func() {
 		connector, ok := s.getCollector(req.Protocol)
 		if !ok {
-			s.jobManager.UpdateJobStatus(jobID, CollectStatusFailed, 100, "Коннектор для протокола не зарегистрирован")
-			s.jobManager.AppendJobLog(jobID, "Сбор завершен с ошибкой")
+			s.jobManager.UpdateJobStatus(jobID, CollectStatusFailed, 100, "Protocol connector not registered")
+			s.jobManager.AppendJobLog(jobID, "Collection completed with error")
 			return
 		}

@@ -1847,7 +1878,7 @@ func (s *Server) startCollectionJob(jobID string, req CollectRequest) {
 				return
 			}
 			s.jobManager.UpdateJobStatus(jobID, CollectStatusFailed, 100, err.Error())
-			s.jobManager.AppendJobLog(jobID, "Сбор завершен с ошибкой")
+			s.jobManager.AppendJobLog(jobID, "Collection completed with error")
 			return
 		}

@@ -1857,7 +1888,7 @@ func (s *Server) startCollectionJob(jobID string, req CollectRequest) {

 		applyCollectSourceMetadata(result, req)
 		s.jobManager.UpdateJobStatus(jobID, CollectStatusSuccess, 100, "")
-		s.jobManager.AppendJobLog(jobID, "Сбор завершен")
+		s.jobManager.AppendJobLog(jobID, "Collection completed")
 		s.SetResult(result)
 		s.SetDetectedVendor(req.Protocol)
 		if job, ok := s.jobManager.GetJob(jobID); ok {
@@ -2117,6 +2148,27 @@ func jsonError(w http.ResponseWriter, message string, code int) {
 	json.NewEncoder(w).Encode(map[string]string{"error": message})
 }

+func (s *Server) htmlError(w http.ResponseWriter, message string, code int) {
+	w.Header().Set("Content-Type", "text/html; charset=utf-8")
+	w.WriteHeader(code)
+	version := normalizeDisplayVersion(s.config.AppVersion)
+	fmt.Fprintf(w, `<!DOCTYPE html><html><head><meta charset="utf-8"><title>Error %d</title></head>`+
+		`<body><h1>Error %d</h1><p>%s</p>`+
+		`<footer style="margin-top:2em;color:#999;font-size:12px">LOGPile %s</footer></body></html>`,
+		code, code, template.HTMLEscapeString(message), template.HTMLEscapeString(version))
+}
+
+func jsonList(w http.ResponseWriter, items interface{}, totalCount int) {
+	w.Header().Set("Content-Type", "application/json")
+	json.NewEncoder(w).Encode(map[string]interface{}{
+		"items":       items,
+		"total_count": totalCount,
+		"page":        1,
+		"per_page":    totalCount,
+		"total_pages": 1,
+	})
+}
+
 // isGPUDevice checks if device class indicates a GPU
 func isGPUDevice(deviceClass string) bool {
 	// Standard PCI class names
--- a/internal/server/handlers_gpu_test.go
+++ b/internal/server/handlers_gpu_test.go
@@ -51,17 +51,20 @@ func TestHandleGetSerials_WithGPUs(t *testing.T) {
 	}

 	// Parse response
-	var serials []struct {
-		Component    string `json:"component"`
-		Location     string `json:"location,omitempty"`
-		SerialNumber string `json:"serial_number"`
-		Manufacturer string `json:"manufacturer,omitempty"`
-		Category     string `json:"category"`
+	var resp struct {
+		Items []struct {
+			Component    string `json:"component"`
+			Location     string `json:"location,omitempty"`
+			SerialNumber string `json:"serial_number"`
+			Manufacturer string `json:"manufacturer,omitempty"`
+			Category     string `json:"category"`
+		} `json:"items"`
 	}

-	if err := json.NewDecoder(w.Body).Decode(&serials); err != nil {
+	if err := json.NewDecoder(w.Body).Decode(&resp); err != nil {
 		t.Fatalf("Failed to decode response: %v", err)
 	}
+	serials := resp.Items

 	// Check that we have GPU entries
 	gpuCount := 0
@@ -115,13 +118,16 @@ func TestHandleGetSerials_WithoutGPUSerials(t *testing.T) {
 	srv.handleGetSerials(w, req)

 	// Parse response
-	var serials []struct {
-		Category string `json:"category"`
+	var resp struct {
+		Items []struct {
+			Category string `json:"category"`
+		} `json:"items"`
 	}

-	if err := json.NewDecoder(w.Body).Decode(&serials); err != nil {
+	if err := json.NewDecoder(w.Body).Decode(&resp); err != nil {
 		t.Fatalf("Failed to decode response: %v", err)
 	}
+	serials := resp.Items

 	// Check that GPUs without serial numbers are not included
 	for _, s := range serials {
--- a/internal/server/job_manager.go
+++ b/internal/server/job_manager.go
@@ -3,6 +3,7 @@ package server
 import (
 	"context"
 	"fmt"
+	"maps"
 	"sync"
 	"time"
 )
@@ -22,9 +23,11 @@ func (m *JobManager) CreateJob(req CollectRequest) *Job {
 	now := time.Now().UTC()
 	job := &Job{
 		ID:        generateJobID(),
+		Type:      req.Protocol,
 		Status:    CollectStatusQueued,
 		Progress:  0,
-		Logs:      []string{formatCollectLogLine(now, "Задача поставлена в очередь")},
+		Message:   "Job queued",
+		Logs:      []string{formatCollectLogLine(now, "Job queued")},
 		CreatedAt: now,
 		UpdatedAt: now,
 		RequestMeta: CollectRequestMeta{
@@ -66,7 +69,7 @@ func (m *JobManager) CancelJob(id string) (*Job, bool) {
 		job.Status = CollectStatusCanceled
 		job.Error = ""
 		job.UpdatedAt = time.Now().UTC()
-		job.Logs = append(job.Logs, formatCollectLogLine(job.UpdatedAt, "Сбор отменен пользователем"))
+		job.Logs = append(job.Logs, formatCollectLogLine(job.UpdatedAt, "Collection canceled by user"))
 	}

 	cancelFn := job.cancel
@@ -122,6 +125,7 @@ func (m *JobManager) AppendJobLog(id, message string) (*Job, bool) {
 	job.Logs = append(job.Logs, message)
 	job.UpdatedAt = time.Now().UTC()
 	job.Logs[len(job.Logs)-1] = formatCollectLogLine(job.UpdatedAt, message)
+	job.Message = message

 	cloned := cloneJob(job)
 	m.mu.Unlock()
@@ -202,7 +206,7 @@ func (m *JobManager) SkipJob(id string) (*Job, bool) {
 	skipFn := job.skipFn
 	job.skipFn = nil
 	job.UpdatedAt = time.Now().UTC()
-	job.Logs = append(job.Logs, formatCollectLogLine(job.UpdatedAt, "Пропуск зависших запросов по команде пользователя"))
+	job.Logs = append(job.Logs, formatCollectLogLine(job.UpdatedAt, "Skipping stalled requests on user request"))
 	cloned := cloneJob(job)
 	m.mu.Unlock()

@@ -212,6 +216,18 @@ func (m *JobManager) SkipJob(id string) (*Job, bool) {
 	return cloned, true
 }

+func (m *JobManager) SetJobResult(id string, result map[string]interface{}) bool {
+	m.mu.Lock()
+	defer m.mu.Unlock()
+	job, ok := m.jobs[id]
+	if !ok || job == nil {
+		return false
+	}
+	job.Result = result
+	job.UpdatedAt = time.Now().UTC()
+	return true
+}
+
 func (m *JobManager) AttachJobCancel(id string, cancelFn context.CancelFunc) bool {
 	m.mu.Lock()
 	defer m.mu.Unlock()
@@ -265,6 +281,9 @@ func cloneJob(job *Job) *Job {
 	cloned.DebugInfo = cloneCollectDebugInfo(job.DebugInfo)
 	cloned.CurrentPhase = job.CurrentPhase
 	cloned.ETASeconds = job.ETASeconds
+	if job.Result != nil {
+		cloned.Result = maps.Clone(job.Result)
+	}
 	cloned.cancel = nil
 	cloned.skipFn = nil
 	return &cloned
--- a/internal/server/server.go
+++ b/internal/server/server.go
@@ -91,6 +91,7 @@ func (s *Server) setupRoutes() {
 	s.mux.HandleFunc("GET /api/export/csv", s.handleExportCSV)
 	s.mux.HandleFunc("GET /api/export/json", s.handleExportJSON)
 	s.mux.HandleFunc("GET /api/export/reanimator", s.handleExportReanimator)
+	s.mux.HandleFunc("GET /api/export/logs-csv", s.handleExportLogsCSV)
 	s.mux.HandleFunc("POST /api/convert", s.handleConvertReanimatorBatch)
 	s.mux.HandleFunc("GET /api/convert/{id}", s.handleConvertStatus)
 	s.mux.HandleFunc("GET /api/convert/{id}/download", s.handleConvertDownload)
--- a/releases/v1.21/RELEASE_NOTES.md
+++ b/releases/v1.21/RELEASE_NOTES.md
@@ -0,0 +1,62 @@
+# logpile v1.21
+
+Дата релиза: 2026-06-15
+Тег: `v1.21`
+
+## Что нового
+
+### Inspur/Kaytus (onekeylog) — серийные номера дисков из SOLHostCapture.log
+
+Когда RAID-контроллер (например, Microchip PM8204-2GB) подключён напрямую через PCIe,
+BMC возвращает пустой массив в секции `RESTful HDD info`. Серийники дисков теперь
+восстанавливаются из вывода smartd в `SOLHostCapture.log`:
+
+- Обрабатываются оба экземпляра файла (`log/sollog/` и `runningdata/var/sollog/`),
+  серийники дедуплицируются по обоим источникам.
+- Три прохода обогащения: совпадение по модели → позиционное заполнение пустых
+  backplane-слотов → добавление новых записей.
+- Определяется тип (SSD/HDD), производитель, прошивка и ёмкость.
+
+### Inspur/Kaytus — корректное определение live-сбора на NF-серверах
+
+NF-серверы хранения (например, NF5280M6) не имеют GPU-топологии, из-за чего
+Redfish-коллектор раньше не мог идентифицировать их как Inspur и переходил в
+режим fallback с AMI-профилем, пробуя несуществующие пути `/Oem/Ami`.
+
+Добавлено определение по `SystemManufacturer` / `ChassisManufacturer`: значение
+`"Inspur"` теперь даёт 60 очков — достаточно для входа в matched-режим без
+GPU-сигналов.
+
+### Inspur/Kaytus — исправление IDL-событий GPU (Assert/Deassert)
+
+- Deassert-события больше не отбрасываются как дубликаты Assert — в ключ дедупликации
+  добавлен `EventType`.
+- Deassert корректно снимает критический статус GPU: раньше GPUы оставались в Critical
+  даже после сброса аварии.
+- В экспорт Reanimator добавлена секция `bmc_event_summary` — дедуплицированная таблица
+  критических и предупреждающих событий со статусом Active/Resolved на основе пар
+  Assert/Deassert.
+
+### UI — кнопка PDF
+
+Добавлена кнопка «PDF» в шапку отчёта. При нажатии отчёт открывается в новой
+вкладке, откуда можно сохранить в PDF через системный диалог печати браузера.
+
+### Внутренние изменения (bible-контракты)
+
+- Идентификаторы нормализованы через `strings.EqualFold` (H3C-парсер).
+- CSV-экспорт: UTF-8 BOM + разделитель `;`.
+- Все русскоязычные строки в исходниках переведены на английский (ADL-007).
+- `Job` расширен полями `Type`, `Message`, `Result`.
+- List-эндпоинты обёрнуты в конверт `{items, total_count, page, per_page, total_pages}`.
+- Страницы ошибок рендерят footer с версией.
+- Логирование переведено на `log/slog` со структурированными атрибутами.
+
+### pci.ids обновлён
+
+База идентификаторов PCI-устройств обновлена до актуальной версии от 2026-06-15.
+
+## Запуск на macOS
+
+Снимите карантинный атрибут через терминал: `xattr -d com.apple.quarantine /path/to/logpile-darwin-arm64`
+После этого бинарник запустится без предупреждения Gatekeeper.
--- a/releases/v1.22/RELEASE_NOTES.md
+++ b/releases/v1.22/RELEASE_NOTES.md
@@ -0,0 +1,60 @@
+# logpile v1.22
+
+Дата релиза: 2026-06-19
+Тег: `v1.22`
+
+## Что нового
+
+### HPE iLO AHS — новый парсер
+
+Добавлена поддержка файлов `*.ahs` (Active Health System), экспортируемых
+из веб-интерфейса iLO. Парсер извлекает:
+
+- **Инвентарь оборудования**: плата, процессоры, память, диски, сетевые
+  адаптеры, блоки питания, backplane, RAID-контроллеры.
+- **Прошивки**: iLO, System ROM, SPS, TPM, SPLD, контроллеры, NIC, backplane —
+  из основного бинарного контейнера и XML-сертификата `bcert.pkg`.
+- **События**: разбор `.zbb`-файлов с журналом iLO; определение типа и
+  серьёзности по тексту сообщения; очистка однобайтовых frame-сепараторов
+  из концов строк.
+- **Устойчивость к битым файлам**: если последняя запись в AHS-контейнере
+  обрезана (объявленный размер выходит за границу файла), парсер обрабатывает
+  данные частично вместо возврата ошибки.
+- Добавлено распознавание модельного ряда **Alletra Storage Server** (ранее
+  `ProductName` оставался пустым).
+
+### Экспорт логов в CSV («Logs Export»)
+
+Новая кнопка «**Logs Export**» в шапке интерфейса выгружает все
+распознанные события (без какой-либо фильтрации) в CSV-файл:
+
+- Разделитель — точка с запятой (`;`), кодировка — UTF-8 с BOM.
+- Файл открывается в Excel без дополнительных настроек импорта.
+- Колонки: `timestamp`, `source`, `severity`, `sensor_type`, `sensor_name`,
+  `event_type`, `id`, `description`, `raw_data`.
+
+Кнопка «PDF» удалена.
+
+### Исправления в Reanimator-экспорте
+
+- `event_logs` в JSON-экспорте Reanimator больше не оказывается пустым для
+  HPE iLO AHS: источник `"HPE iLO"` теперь корректно нормализуется в `"bmc"`.
+
+### Исправления chart viewer
+
+- JavaScript `view.js` не загружался в LOGPile из-за отсутствия перезаписи
+  пути `/static/view.js` → `/chart/static/view.js`. Исправлено; фильтры
+  по колонкам в таблицах теперь работают.
+- Субмодуль chart обновлён до **v2.7**: фильтры вынесены в отдельную строку
+  под заголовком, исправлена минимальная ширина колонок.
+
+### Обновления зависимостей
+
+- **pci.ids** (база PCI-устройств) обновлена. Коллектор скорректирован под
+  переименование `0x8086:0x28c0`: `"Volume Management Device NVMe RAID
+  Controller"` → `"Volume Management Device (VMD)"`.
+
+## Запуск на macOS
+
+Снимите карантинный атрибут через терминал: `xattr -d com.apple.quarantine /path/to/logpile-darwin-arm64`
+После этого бинарник запустится без предупреждения Gatekeeper.
--- a/releases/v1.23/RELEASE_NOTES.md
+++ b/releases/v1.23/RELEASE_NOTES.md
@@ -0,0 +1,23 @@
+# logpile v1.23
+
+Дата релиза: 2026-06-19
+Тег: `v1.23`
+
+## Что нового
+
+### Исправление: HPE iLO AHS файлы больше 10 МБ не обрезаются
+
+AHS-файлы могут весить сотни мегабайт (типичный пример — 104 МБ). Универсальный
+лимит в 10 МБ молча обрезал их, из-за чего парсер видел лишь начало файла и
+извлекал неполный список событий.
+
+Теперь лимит зависит от расширения: `.ahs` — до **1 ГБ**, прочие
+одиночные файлы (`.txt`, `.log`) — прежние 10 МБ.
+
+Для AHS-файла размером 104 МБ количество распознанных событий увеличивается
+с ~529 до ~12 600.
+
+## Запуск на macOS
+
+Снимите карантинный атрибут через терминал: `xattr -d com.apple.quarantine /path/to/logpile-darwin-arm64`
+После этого бинарник запустится без предупреждения Gatekeeper.
--- a/third_party/pciids
+++ b/third_party/pciids
--- a/web/static/css/style.css
+++ b/web/static/css/style.css
@@ -933,3 +933,71 @@ code {
        grid-template-columns: 1fr;
    }
 }
+
+/* ── Parse / collection errors panel ───────────────────────────────────── */
+.parse-errors-section {
+    overflow: hidden;
+}
+
+.parse-errors-header {
+    display: flex;
+    align-items: center;
+    justify-content: space-between;
+    padding: .55rem .75rem;
+    background: var(--warn-bg);
+    border: 1px solid #f0e0c0;
+    border-radius: 6px 6px 0 0;
+    cursor: pointer;
+    user-select: none;
+    font-weight: 600;
+    font-size: .85rem;
+    color: var(--warn-fg);
+}
+
+.parse-errors-toggle {
+    font-size: .75rem;
+    opacity: .7;
+}
+
+.parse-errors-body {
+    border: 1px solid #f0e0c0;
+    border-top: none;
+    border-radius: 0 0 6px 6px;
+    overflow-x: auto;
+}
+
+.parse-errors-table {
+    width: 100%;
+    border-collapse: collapse;
+    font-size: .82rem;
+}
+
+.parse-errors-table th {
+    background: var(--surface-3);
+    padding: .4rem .65rem;
+    text-align: left;
+    font-weight: 600;
+    color: var(--muted);
+    border-bottom: 1px solid var(--border);
+    white-space: nowrap;
+}
+
+.parse-errors-table td {
+    padding: .38rem .65rem;
+    border-bottom: 1px solid var(--border-lite);
+    vertical-align: top;
+}
+
+.parse-errors-table tr:last-child td {
+    border-bottom: none;
+}
+
+.parse-error-row.parse-error-error td:first-child {
+    color: var(--crit-fg);
+    font-weight: 600;
+}
+
+.parse-error-row.parse-error-warning td:first-child {
+    color: #7a5200;
+    font-weight: 600;
+}
--- a/web/static/js/app.js
+++ b/web/static/js/app.js
@@ -1410,9 +1410,66 @@ async function loadData(vendor, filename) {
    document.getElementById('clear-btn').classList.remove('hidden');
    document.getElementById('header-raw-btn').classList.remove('hidden');
    document.getElementById('header-reanimator-btn').classList.remove('hidden');
+    document.getElementById('header-logs-csv-btn').classList.remove('hidden');
    document.getElementById('header-log-meta').classList.remove('hidden');

    loadAuditViewer();
+    loadParseErrors();
+}
+
+async function loadParseErrors() {
+    const section = document.getElementById('parse-errors-section');
+    const rows = document.getElementById('parse-errors-rows');
+    const title = document.getElementById('parse-errors-title');
+    if (!section || !rows) return;
+
+    let data;
+    try {
+        const resp = await fetch('/api/parse-errors');
+        if (!resp.ok) return;
+        data = await resp.json();
+    } catch (e) {
+        return;
+    }
+
+    const items = (data && data.items) ? data.items : [];
+    if (items.length === 0) {
+        section.classList.add('hidden');
+        return;
+    }
+
+    const errorCount = items.filter(i => i.severity === 'error').length;
+    const warnCount  = items.filter(i => i.severity === 'warning').length;
+    const parts = [];
+    if (errorCount > 0) parts.push(`${errorCount} error${errorCount > 1 ? 's' : ''}`);
+    if (warnCount  > 0) parts.push(`${warnCount} warning${warnCount > 1 ? 's' : ''}`);
+    const otherCount = items.length - errorCount - warnCount;
+    if (otherCount > 0) parts.push(`${otherCount} notice${otherCount > 1 ? 's' : ''}`);
+    title.textContent = `Collection diagnostics — ${parts.join(', ')}`;
+
+    rows.innerHTML = '';
+    for (const item of items) {
+        const tr = document.createElement('tr');
+        tr.className = `parse-error-row parse-error-${item.severity || 'info'}`;
+        tr.innerHTML =
+            `<td>${escapeHtml(item.source || '')}</td>` +
+            `<td>${escapeHtml(item.path || item.category || '')}</td>` +
+            `<td>${escapeHtml(item.message || '')}</td>` +
+            `<td>${escapeHtml(item.detail || '')}</td>`;
+        rows.appendChild(tr);
+    }
+
+    section.classList.remove('hidden');
+}
+
+let parseErrorsCollapsed = false;
+function toggleParseErrors() {
+    const body   = document.getElementById('parse-errors-body');
+    const toggle = document.getElementById('parse-errors-toggle');
+    if (!body) return;
+    parseErrorsCollapsed = !parseErrorsCollapsed;
+    body.style.display   = parseErrorsCollapsed ? 'none' : '';
+    toggle.textContent   = parseErrorsCollapsed ? '▼' : '▲';
 }

 function loadAuditViewer() {
@@ -1462,12 +1519,22 @@ async function clearData() {
        document.getElementById('clear-btn').classList.add('hidden');
        document.getElementById('header-raw-btn').classList.add('hidden');
        document.getElementById('header-reanimator-btn').classList.add('hidden');
+        document.getElementById('header-logs-csv-btn').classList.add('hidden');
        document.getElementById('header-log-meta').classList.add('hidden');
        document.getElementById('upload-status').textContent = '';
        const frame = document.getElementById('audit-viewer-frame');
        if (frame) {
            frame.src = 'about:blank';
        }
+        const parseErrSection = document.getElementById('parse-errors-section');
+        if (parseErrSection) parseErrSection.classList.add('hidden');
+        const parseErrRows = document.getElementById('parse-errors-rows');
+        if (parseErrRows) parseErrRows.innerHTML = '';
+        parseErrorsCollapsed = false;
+        const parseErrBody = document.getElementById('parse-errors-body');
+        if (parseErrBody) parseErrBody.style.display = '';
+        const parseErrToggle = document.getElementById('parse-errors-toggle');
+        if (parseErrToggle) parseErrToggle.textContent = '▲';
    } catch (err) {
        console.error('Failed to clear data:', err);
    }
--- a/web/templates/index.html
+++ b/web/templates/index.html
@@ -18,6 +18,7 @@
                <button id="clear-btn" class="header-action hidden" onclick="clearData()">Clear Data</button>
                <button id="header-raw-btn" class="header-action hidden" onclick="exportData('json')">Raw Data</button>
                <button id="header-reanimator-btn" class="header-action hidden" onclick="exportData('reanimator')">Reanimator</button>
+                <button id="header-logs-csv-btn" class="header-action hidden" onclick="exportData('logs-csv')">Logs Export</button>
                <button id="restart-btn" class="header-action" onclick="restartApp()">Restart</button>
                <button id="exit-btn" class="header-action" onclick="exitApp()">Exit</button>
            </div>
@@ -170,6 +171,25 @@
                    </iframe>
                </div>
            </section>
+            <section id="parse-errors-section" class="parse-errors-section hidden">
+                <div class="parse-errors-header" onclick="toggleParseErrors()">
+                    <span id="parse-errors-title">Collection warnings</span>
+                    <span id="parse-errors-toggle" class="parse-errors-toggle">▲</span>
+                </div>
+                <div id="parse-errors-body" class="parse-errors-body">
+                    <table class="parse-errors-table">
+                        <thead>
+                            <tr>
+                                <th>Source</th>
+                                <th>Section</th>
+                                <th>Message</th>
+                                <th>Detail</th>
+                            </tr>
+                        </thead>
+                        <tbody id="parse-errors-rows"></tbody>
+                    </table>
+                </div>
+            </section>
        </section>
    </main>
Author	SHA1	Message	Date
Mikhail Chusavitin	4ce0251ce4	docs(bible-local): add backlog with sfp_modules implementation plan Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-19 18:39:28 +03:00
Mikhail Chusavitin	994d46f3b3	docs: update hardware ingest contract to v2.11 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-19 18:28:38 +03:00
Mikhail Chusavitin	ee3e8a6e33	docs: release notes for v1.23 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-19 15:21:35 +03:00
Mikhail Chusavitin	e2c81758b5	fix(parser): raise file size limit for .ahs to 1 GB AHS files can exceed 100 MB; the previous 10 MB universal cap silently truncated them and caused incomplete event parsing. Per-extension limits are now used: .ahs gets 1 GB, all other single-file types keep 10 MB. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-19 15:20:57 +03:00
Mikhail Chusavitin	6b52a1876f	docs: release notes for v1.22 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-19 15:13:42 +03:00
Mikhail Chusavitin	3e3c48bc08	feat(hpe-ilo): parse AHS files, fix event logs export, add logs CSV export - HPE iLO AHS parser: handle truncated last entry gracefully, recognize Alletra product line, expand event type/severity inference, trim iLO frame separators from event messages - Fix event_logs always 0 in Reanimator export: normalizeEventLogSource now maps "HPE iLO" → "bmc" - Fix chart JS not loading in LOGPile: rewriteChartStaticPaths now also rewrites src="/static/view.js" → /chart/static/view.js - Add "Logs Export" button (CSV, semicolon-delimited, UTF-8 BOM) and remove PDF button - Fix collector test broken by pciids rename of Intel VMD device - Update submodules: chart v2.7, pciids, bible Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-19 15:11:46 +03:00
Mikhail Chusavitin	cd864c3d6c	docs: release notes for v1.21 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-15 17:56:17 +03:00
Mikhail Chusavitin	5128ac5303	fix(server): remove PrintMode from RenderOptions after chart submodule update Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-15 17:53:55 +03:00
Mikhail Chusavitin	53cda82c79	chore: update submodules (bible, chart, pciids) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-15 17:52:51 +03:00
Mikhail Chusavitin	a18d8fe648	feat(inspur): enrich storage serials from SOLHostCapture.log smartd output When the BMC HDD API returns an empty array (RAID controller attached via PCIe, e.g. PM8204-2GB), disk serial numbers are now recovered from smartd startup messages in SOLHostCapture.log. Enrichment runs in three passes: model-match on existing slots, positional fill of empty backplane placeholders, then new entries for any remainder. Both log/ and runningdata/var/ copies are merged with serial deduplication. Parser version bumped to 2.1. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-15 17:52:07 +03:00
Michael Chus	6ab0f4eb20	chore: update bible submodule Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-13 14:36:53 +03:00
Michael Chus	57de3ba6eb	chore: align codebase with bible engineering contracts - identifier-normalization: use strings.EqualFold in h3c/parser.go - import-export: CSV now uses UTF-8 BOM and semicolon delimiter - go-code-style: translate all Russian source strings to English (ADL-007) - go-background-tasks: add Type, Message, Result fields to Job struct - go-api: wrap list endpoints in {items, total_count, page, per_page, total_pages} - module-structure: rename helpers.go → context_sleep.go - build-version-display: htmlError renders version footer on error pages - go-logging: migrate all log.Printf calls to log/slog with structured attrs Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-13 14:35:39 +03:00
Michael Chus	47ff1c3796	fix(inspur): detect standard Inspur servers via SystemManufacturer NF-series storage servers (e.g. NF5280M6) have no GPU/outboard-PCIe topology, so the previous score gate (topologyScore==0 \|\| boardScore==0 → return 0) always produced score=0 despite SystemManufacturer="Inspur" being available. These servers fell into mode=fallback, activating the AMI profile and probing /Oem/Ami paths that don't exist on the BMC. Add manufacturer-based detection: SystemManufacturer or ChassisManufacturer containing "inspur" contributes 60 points — enough to enter matched mode on its own. GPU servers with full topology+board signals still score higher as before. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-12 03:58:55 +03:00
Michael Chus	1c4a3b0c09	feat(ui): add PDF export via browser print Adds a PDF button to the report header. Clicking it opens /chart/current?print=true in a new tab, which auto-triggers window.print() so the user can save to PDF via the browser dialog. - chart submodule bumped: PrintMode support (no filter JS, auto-print, @media print CSS) - handlers.go: passes PrintMode=true when ?print=true query param is set - index.html: PDF button alongside Raw Data / Reanimator - app.js: printReport() helper; button shown/hidden with other exports Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-05 12:24:01 +03:00
Michael Chus	10c381c8ec	chore: update bible and pci.ids submodules Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-28 03:40:15 +03:00
Michael Chus	440959483e	fix(inspur): correctly handle PCIe Assert/Deassert GPU fault events Three related fixes for IDL event processing: 1. idl.go: include EventType in dedup key so Deassert events are no longer silently dropped as duplicates of their Assert counterparts. 2. gpu_status.go: treat Deassert events as clearing all GPU faults — previously the code re-applied the same faulty GPU set from the description, leaving GPUs stuck in Critical even after alarm cleared. 3. reanimator_models/converter: add bmc_event_summary section to the Reanimator export — a deduplicated Critical/Warning event table with Active/Resolved status derived from Assert/Deassert pairs. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-28 03:38:04 +03:00
Mikhail Chusavitin	f3836a34cc	chore: update chart submodule to v2.0 and refresh pci.ids (2026-05-21) chart: feat(viewer): replace severity dropdown with per-column header filters pci.ids: 2026-02-17 → 2026-05-21 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-21 14:37:26 +03:00
Mikhail Chusavitin	ba9a52a61a	fix(ui): parse-errors panel full width Removed max-width/padding constraints — panel now stretches to grid column width like the viewer-panel above it. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-21 14:32:47 +03:00
Mikhail Chusavitin	27373aa104	feat: surface BMC collection errors in parse-errors panel and event log When Inspur component.log sections return {"error":"...","code":N} instead of hardware data, the parser now: - stores them in AnalysisResult.CollectionErrors (new model field) - mirrors each one into result.Events with Source="BMC/<section>" so the chart viewer event table shows the specific BMC module - feeds them into /api/parse-errors as bmc_collection_error entries UI adds a collapsible "Collection diagnostics" panel below the chart iframe (outside /chart) that appears when /api/parse-errors returns any items; resets on data clear. Affected sections in this dump: HDD (1458), PCIe Devices (1458), Network Adapters (1458), Disk Backplane. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-21 14:30:01 +03:00
Mikhail Chusavitin	4f7b5b826a	fix(inspur): fix PSU section regex when PCIE section precedes Network The PSU regex used "RESTful Network" as its end anchor, but in standard Inspur component.log layout the PCIE Device section sits between PSU and Network Adapter. The lazy [\s\S]*? captured across the PCIE error block, producing invalid JSON and silently dropping all PSU data. Changed anchor to RESTful (?:PCIE\|Network) — matches whichever section immediately follows PSU in a given archive. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-21 14:21:13 +03:00
Mikhail Chusavitin	dfd64550cf	fix(inspur): infer DIMM size from part number when BMC reports size=0 When BMC firmware fails to read capacity for a present DIMM, size_mb stays 0. If another DIMM with the same part number in the same batch has a known size, use it to fill the gap. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-21 14:18:15 +03:00
Mikhail Chusavitin	9505303d1d	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>	2026-05-21 14:15:28 +03:00
Mikhail Chusavitin	f2c04cf0e8	fix(inspur): parse CPUs from component.log and fix DIMM present detection Two bugs in onekeylog archives that lack asset.json: - CPU count was always 0: ParseComponentLog never parsed the "RESTful CPU info" section. Added parseCPUInfo as a fallback when hw.CPUs is empty (asset.json remains the primary source when present). Also worked around a Go JSON case-insensitive collision between "proc_id" (int) and "PROC_ID" (string CPUID) by adding an explicit PROC_ID field with an exact-case tag. - Only 1 of 2 DIMMs shown: Present condition required mem_mod_size > 0, but some BMC firmware reports size=0 for a physically installed module while still providing serial and part number. Now treats a DIMM as present when status=1 and any of size/serial/partnum is non-empty. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-21 14:13:55 +03:00
Mikhail Chusavitin	ca457ac72b	fix(exporter): propagate iommu_group through PCIe export pipeline IOMMUGroup was added to models.PCIeDevice but never wired into the converter — missing from Details in buildDevicesFromLegacy, no field in ReanimatorPCIe, and convertPCIeFromDevices never read it. Add IOMMUGroup *int to ReanimatorPCIe, propagate through Details, add intPtrFromDetailMap helper. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-04-30 16:05:30 +03:00