update UI

README.md

@@ -3,7 +3,7 @@
 Công cụ desktop dùng để **scan, phát hiện và flash firmware hàng loạt** cho các thiết bị OpenWrt trong mạng LAN.  
 Hỗ trợ nạp **thủ công** (chọn thiết bị → flash) và **tự động hóa** (scan → phát hiện → flash → retry).
 
-> **Tech stack:** Python 3.9+ · PyQt6 · Paramiko/SCP · Scapy · Requests · PyInstaller  
+> **Tech stack:** Python 3.9+ · PyQt6 · Paramiko/SCP · Requests · PyInstaller  
 > **Phiên bản:** `1.2.0`
 
 ---
@@ -16,7 +16,7 @@ Mira_Firmware_Loader/
 ├── version.txt                # Số phiên bản ứng dụng
 ├── requirements.txt           # Danh sách dependencies
 ├── core/
-│   ├── scanner.py             # Quét thiết bị mạng đa lớp (Ping sweep + ARP + Scapy)
+│   ├── scanner.py             # Quét thiết bị mạng (Ping sweep đa luồng)
 │   ├── workers.py             # ScanThread — chạy scanner trong background thread
 │   ├── api_flash.py           # Flash firmware qua LuCI HTTP API
 │   ├── ssh_utils.py           # SSH/SCP transport helpers dùng chung
@@ -48,8 +48,7 @@ Mira_Firmware_Loader/
 ### Yêu cầu
 
 - Python **3.9+**
-- Thư viện: `PyQt6`, `scapy`, `requests`, `paramiko`, `scp`, `pyinstaller` (để build)
-- _Windows:_ Cài [Npcap](https://npcap.com/) để Scapy có thể gửi ARP broadcast (không bắt buộc — có fallback)
+- Thư viện: `PyQt6`, `requests`, `paramiko`, `scp`, `pyinstaller` (để build)
 
 ### Khởi chạy nhanh
 
@@ -101,11 +100,11 @@ Output: `dist\Mira_Firmware_Loader.exe` — không cần cài Python trên máy
 ┌──────▼──────┐  ┌───────▼──────────┐  ┌───▼──────────────────┐
 │  scanner.py │  │  Flash Workers   │  │  AutoFlashWorker     │
 │             │  │  (thủ công)      │  │  (tự động hóa)       │
-│  1. Ping    │  │                  │  │                      │
-│     Sweep   │  │  NewFlashThread  │  │  Phase 1: Scan LAN   │
-│  2. arp -a  │  │  ├─ api_flash    │  │    (tối đa 15 lần)   │
-│  3. Scapy   │  │  └─ ssh_new_flash│  │  Phase 2: Flash      │
-│     ARP     │  │                  │  │    (auto-retry x3)   │
+│  Ping Sweep │  │                  │  │                      │
+│ (Multithread│  │  NewFlashThread  │  │  Phase 1: Scan LAN   │
+│  gọi  OS    │  │  ├─ api_flash    │  │    (tối đa 15 lần)   │
+│  ping cmd)  │  │  └─ ssh_new_flash│  │  Phase 2: Flash      │
+│             │  │                  │  │    (auto-retry x3)   │
 └─────────────┘  │  UpdateFlash     │  │  ├─ api_flash        │
                  │  Thread          │  │  └─ ssh_new_flash    │
                  │  └─ ssh_update   │  └──────────────────────┘
@@ -127,15 +126,14 @@ Output: `dist\Mira_Firmware_Loader.exe` — không cần cài Python trên máy
 
 ### 1. Quét mạng (Network Scan)
 
-`scanner.py` dùng chiến lược 3 lớp, đảm bảo phát hiện đầy đủ mà không cần quyền Root:
+`scanner.py` sử dụng chiến lược **Ping trước → ARP sau**, đảm bảo vừa lấy được MAC vừa loại bỏ hoàn toàn lỗi thiết bị ảo do ARP cache cũ:
 
 | Giai đoạn      | Mô tả                                                                                                                                                  |
 | -------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------ |
-| **Ping Sweep** | Gửi ping đồng thời tới toàn bộ host trong dải `/24` (tất cả cùng lúc, không batching) để đánh thức thiết bị và điền ARP cache                          |
-| **ARP Table**  | Đọc `arp -a` bằng regex, hỗ trợ cả định dạng Windows (`cc-2d-...`) và macOS/Linux (`aa:bb:...`)                                                        |
-| **Scapy ARP**  | Chạy **song song** với ARP Table — gửi ARP broadcast để lấp khoảng trống. Yêu cầu Npcap (Windows) hoặc root (Linux); tự động bỏ qua nếu không khả dụng |
+| **Ping Sweep** | Gửi ping đồng thời tới toàn bộ host trong dải mạng bằng tiến trình con (đa luồng). Nhằm xác định chính xác thiết bị nào đang thực sự cấp nguồn online. |
+| **ARP Lookup** | Ngay sau khi ping, query ARP table hệ điều hành (song song) _chỉ cho các IP vừa ping thành công_, lấy MAC Address chuẩn xác trước khi cache cũ hết hạn.  |
 
-Kết quả được merge theo IP và sort tăng dần trước khi trả về UI.
+Kết quả (IP và MAC) được sort tăng dần theo IP trước khi trả về UI.
 
 ### 2. Bảng thiết bị (Device Table)
 
@@ -281,7 +279,7 @@ Kết quả nạp được lưu ở 2 cấp:
 
 ## 🔒 Lưu ý bảo mật & Quyền
 
-- **Scapy (chế độ sâu):** Cần Npcap (Windows) hoặc `sudo` (macOS/Linux). App vẫn hoạt động mà không cần quyền Admin nhờ fallback Ping Sweep + `arp -a`.
-- **CREATE_NO_WINDOW:** Khi gọi subprocess (`ping`, `arp`), ứng dụng dùng flag `CREATE_NO_WINDOW` trên Windows để ngăn cửa sổ console hiện ra.
+- **Quyền hệ thống:** Quá trình quét của `scanner.py` chỉ chạy lệnh ping từ hệ điều hành, vì vậy ứng dụng có thể chạy hoàn toàn **không cần** quyền Admin (trên Windows) / Root (trên macOS/Linux).
+- **CREATE_NO_WINDOW:** Khi gọi subprocess lệnh `ping`, ứng dụng dùng flag `CREATE_NO_WINDOW` trên Windows để ngăn cửa sổ console command prompt (cmd) liên tục hiện lên màn hình.
 - **HTTP thuần:** Firmware được upload qua HTTP (không HTTPS). Chỉ dùng trong mạng LAN nội bộ, không nên dùng trên mạng mở.
 - **Credentials cố định:** SSH credentials (`root`/`admin123a`) được hardcode trong `flash_update_worker.py` và truyền từ `main.py`, không hiển thị trên UI.

core/auto_flash_worker.py

@@ -18,7 +18,7 @@ from core.api_flash import flash_device_api
 from core.ssh_new_flash import flash_device_new_ssh
 
 MAX_FLASH_RETRIES = 3     # Số lần retry nạp FW khi thất bại
-MAX_SCAN_ROUNDS = 15       # Số lần scan tối đa trước khi báo không đủ thiết bị
+MAX_SCAN_ROUNDS = 10       # Số lần scan tối đa trước khi báo không đủ thiết bị
 
 
 class AutoFlashWorker(QThread):

core/scanner.py

@@ -1,155 +1,172 @@
-import subprocess
 import re
+import subprocess
 import sys
-import time
 import ipaddress
+import threading
 from concurrent.futures import ThreadPoolExecutor, as_completed
 
 # Windows: prevent subprocess from opening visible console windows
 _NO_WINDOW = subprocess.CREATE_NO_WINDOW if sys.platform == "win32" else 0
 
+# Concurrent ping workers per platform
+_MAX_WORKERS_WIN = 50
+_MAX_WORKERS_OTHER = 64
+
 
 def _ping_one(ip, is_win):
-    """Ping a single IP to populate ARP table."""
+    """Ping a single IP. Returns True if host responds."""
     try:
         if is_win:
-            subprocess.run(
-                ["ping", "-n", "1", "-w", "300", str(ip)],  # 300ms (was 600ms)
-                stdout=subprocess.DEVNULL,
+            # Capture stdout to check for TTL= — more reliable than returncode
+            # on Windows (returncode can be 0 even for "Destination unreachable")
+            r = subprocess.run(
+                ["ping", "-n", "1", "-w", "500", str(ip)],
+                stdout=subprocess.PIPE,
                 stderr=subprocess.DEVNULL,
-                timeout=2,
+                timeout=3,
                 creationflags=_NO_WINDOW
             )
+            return r.returncode == 0 and b"TTL=" in r.stdout
         elif sys.platform == "darwin":
-            subprocess.run(
-                ["ping", "-c", "1", "-W", "500", str(ip)],  # 500ms — macOS: -W unit là ms
+            r = subprocess.run(
+                ["ping", "-c", "1", "-W", "300", str(ip)],
                 stdout=subprocess.DEVNULL,
                 stderr=subprocess.DEVNULL,
                 timeout=2
             )
         else:
-            subprocess.run(
-                ["ping", "-c", "1", "-W", "1", str(ip)],   # 1s — Linux: -W unit là giây
+            r = subprocess.run(
+                ["ping", "-c", "1", "-W", "1", str(ip)],
                 stdout=subprocess.DEVNULL,
                 stderr=subprocess.DEVNULL,
                 timeout=2
             )
+        return r.returncode == 0
+    except Exception:
+        return False
+
+
+def _get_mac_from_arp(ip):
+    """
+    Lấy MAC address của một IP qua ARP table.
+
+    Chỉ được gọi SAU KHI IP đã phản hồi ping thành công — đảm bảo ARP
+    cache vừa được OS cập nhật với thông tin mới nhất (không bị stale).
+
+    Trả về chuỗi MAC dạng 'AA:BB:CC:DD:EE:FF' hoặc 'N/A' nếu không tra được.
+    """
+    try:
+        if sys.platform == "win32":
+            # arp -a <ip>  → "  192.168.4.5         aa-bb-cc-dd-ee-ff     dynamic"
+            r = subprocess.run(
+                ["arp", "-a", str(ip)],
+                stdout=subprocess.PIPE,
+                stderr=subprocess.DEVNULL,
+                timeout=3,
+                creationflags=_NO_WINDOW
+            )
+            output = r.stdout.decode(errors="ignore")
+            # Dạng Windows: aa-bb-cc-dd-ee-ff
+            match = re.search(
+                r"([0-9a-fA-F]{2}[-:][0-9a-fA-F]{2}[-:][0-9a-fA-F]{2}"
+                r"[-:][0-9a-fA-F]{2}[-:][0-9a-fA-F]{2}[-:][0-9a-fA-F]{2})",
+                output
+            )
+        else:
+            # macOS / Linux: arp -n <ip>
+            # macOS output: "? (192.168.4.5) at aa:bb:cc:dd:ee:ff on en0 ..."
+            # Linux output:  "192.168.4.5 ether aa:bb:cc:dd:ee:ff C eth0"
+            r = subprocess.run(
+                ["arp", "-n", str(ip)],
+                stdout=subprocess.PIPE,
+                stderr=subprocess.DEVNULL,
+                timeout=3
+            )
+            output = r.stdout.decode(errors="ignore")
+            match = re.search(
+                r"([0-9a-fA-F]{2}[:-][0-9a-fA-F]{2}[:-][0-9a-fA-F]{2}"
+                r"[:-][0-9a-fA-F]{2}[:-][0-9a-fA-F]{2}[:-][0-9a-fA-F]{2})",
+                output
+            )
+
+        if match:
+            # Chuẩn hoá sang dấu ':' và chữ hoa
+            mac = match.group(1).replace("-", ":").upper()
+            return mac
     except Exception:
         pass
+    return "N/A"
 
 
 def _ping_sweep(network, progress_cb=None):
-    """Ping tất cả host trong network đồng thời để điền ARP cache.
-    Gọi progress_cb(done, total) sau mỗi ping nếu được cung cấp.
+    """Ping tất cả host trong network đồng thời.
+    Trả về list IP đã phản hồi. Gọi progress_cb(done, total) sau mỗi ping.
     """
     net = ipaddress.ip_network(network, strict=False)
 
-    # Chỉ ping sweep cho /24 hoặc nhỏ hơn
     if net.num_addresses > 256:
-        return
+        return []
 
     is_win = sys.platform == "win32"
     hosts = list(net.hosts())
     total = len(hosts)
+    workers = min(_MAX_WORKERS_WIN if is_win else _MAX_WORKERS_OTHER, len(hosts))
+
     done_count = [0]
+    lock = threading.Lock()
+    alive = []
 
     def _ping_and_track(ip):
-        _ping_one(ip, is_win)
+        ok = _ping_one(ip, is_win)
+        with lock:
             done_count[0] += 1
+            current = done_count[0]
         if progress_cb:
-            progress_cb(done_count[0], total)
+            progress_cb(current, total)
+        return (str(ip), ok)
 
-    # Tất cả host cùng lúc — loại bỏ overhead batching (was max_workers=100)
-    with ThreadPoolExecutor(max_workers=len(hosts)) as executor:
+    with ThreadPoolExecutor(max_workers=workers) as executor:
         futures = [executor.submit(_ping_and_track, ip) for ip in hosts]
         for f in as_completed(futures):
-            pass
+            ip_str, ok = f.result()
+            if ok:
+                alive.append(ip_str)
+
+    return alive
 
 
 def scan_network(network, progress_cb=None, stage_cb=None):
-    """Scan network: ping sweep → ARP table + Scapy song song."""
-    # Phase 1: Ping sweep
+    """Scan network: ping → lấy MAC từ ARP (chỉ cho IP đang online).
+
+    Flow:
+      1. Ping sweep — xác định thiết bị online
+      2. MAC lookup — query ARP cho từng IP vừa phản hồi ping (song song)
+         => ARP cache vừa được OS cập nhật sau ping, không bị stale.
+    """
+    # ── Stage 1: Ping ──
     if stage_cb:
         stage_cb("ping")
-    _ping_sweep(network, progress_cb)
-    time.sleep(0.3)  # Giảm từ 1s xuống 0.3s
+    alive_ips = _ping_sweep(network, progress_cb)
 
-    # Phase 2 + 3: ARP table và Scapy chạy song song
+    # ── Stage 2: MAC lookup ──
     if stage_cb:
-        stage_cb("arp")
+        stage_cb("mac")
 
-    def _collect_arp():
-        result = {}
+    results = []
+    if alive_ips:
+        # Chạy song song để tra MAC nhanh hơn
+        mac_workers = min(32, len(alive_ips))
+        macs = {}
+        with ThreadPoolExecutor(max_workers=mac_workers) as executor:
+            future_to_ip = {executor.submit(_get_mac_from_arp, ip): ip for ip in alive_ips}
+            for future in as_completed(future_to_ip):
+                ip = future_to_ip[future]
                 try:
-            output = subprocess.check_output(
-                ["arp", "-a"], text=True, creationflags=_NO_WINDOW
-            )
-            net = ipaddress.ip_network(network, strict=False)
-            if sys.platform == "win32":
-                pattern = re.compile(
-                    r"(\d+\.\d+\.\d+\.\d+)\s+"
-                    r"([0-9a-fA-F]{2}-[0-9a-fA-F]{2}-[0-9a-fA-F]{2}-"
-                    r"[0-9a-fA-F]{2}-[0-9a-fA-F]{2}-[0-9a-fA-F]{2})\s+"
-                    r"(dynamic|static)",
-                    re.IGNORECASE
-                )
-                for line in output.splitlines():
-                    m = pattern.search(line)
-                    if m:
-                        ip_str = m.group(1)
-                        mac = m.group(2).replace("-", ":")
-                        if mac.upper() != "FF:FF:FF:FF:FF:FF":
-                            try:
-                                if ipaddress.ip_address(ip_str) in net:
-                                    result[ip_str] = {"ip": ip_str, "mac": mac}
-                            except ValueError:
-                                pass
-            else:
-                pattern = re.compile(
-                    r"\((\d+\.\d+\.\d+\.\d+)\)\s+at\s+([0-9a-fA-F:]+)"
-                )
-                for line in output.splitlines():
-                    m = pattern.search(line)
-                    if m:
-                        ip_str, mac = m.group(1), m.group(2)
-                        if mac.lower() not in ("(incomplete)", "ff:ff:ff:ff:ff:ff"):
-                            try:
-                                if ipaddress.ip_address(ip_str) in net:
-                                    result[ip_str] = {"ip": ip_str, "mac": mac}
-                            except ValueError:
-                                pass
+                    macs[ip] = future.result()
                 except Exception:
-            pass
-        return result
+                    macs[ip] = "N/A"
 
-    def _collect_scapy():
-        result = {}
-        try:
-            import io
-            _stderr = sys.stderr
-            sys.stderr = io.StringIO()
-            try:
-                from scapy.all import ARP, Ether, srp
-            finally:
-                sys.stderr = _stderr
+        for ip_str in alive_ips:
+            results.append({"ip": ip_str, "mac": macs.get(ip_str, "N/A")})
 
-            arp = ARP(pdst=str(network))
-            ether = Ether(dst="ff:ff:ff:ff:ff:ff")
-            raw = srp(ether / arp, timeout=1, verbose=0)[0]  # Giảm từ 2s xuống 1s
-            for _, received in raw:
-                result[received.psrc] = {"ip": received.psrc, "mac": received.hwsrc}
-        except Exception:
-            pass
-        return result
-
-    # Chạy ARP read và Scapy đồng thời
-    with ThreadPoolExecutor(max_workers=2) as executor:
-        f_arp = executor.submit(_collect_arp)
-        f_scapy = executor.submit(_collect_scapy)
-        seen = f_arp.result()
-        # Scapy bổ sung những IP chưa có trong ARP table
-        for ip, dev in f_scapy.result().items():
-            if ip not in seen:
-                seen[ip] = dev
-
-    return sorted(seen.values(), key=lambda d: ipaddress.ip_address(d["ip"]))
+    return sorted(results, key=lambda d: ipaddress.ip_address(d["ip"]))

docs/scanner_docs.md

@@ -2,37 +2,50 @@
 
 ## 1. Tổng quan
 
-Thành phần quét IP trong `scanner.py` dò tìm và liệt kê tất cả thiết bị đang hoạt động trên một dải mạng (ví dụ: `192.168.1.0/24`), trả về danh sách chứa **IP** và **MAC Address** của từng thiết bị.
+Thành phần quét IP trong `scanner.py` dò tìm và liệt kê tất cả thiết bị **đang thực sự online** trên một dải mạng (ví dụ: `192.168.1.0/24`), trả về danh sách chứa **IP** và **MAC Address** của từng thiết bị.
 
-Để đảm bảo tỷ lệ phát hiện cao trên mọi hệ điều hành (Windows, macOS, Linux), scanner kết hợp 3 giai đoạn:
+Scanner hoạt động theo cơ chế 2 bước (Ping trước → ARP sau):
+1. **ICMP Ping Sweep:** Chỉ những IP có phản hồi ping mới được ghi nhận là online.
+2. **ARP Lookup:** Ngay sau khi IP phản hồi ping, OS tự động cập nhật ARP cache cho IP đó. Scanner lập tức query ARP table để lấy MAC chính xác nhất.
 
-1. **Ping Sweep** — đánh thức thiết bị, điền ARP cache
-2. **Đọc bảng ARP hệ điều hành** (`arp -a`) — không cần quyền Admin
-3. **Scapy ARP broadcast** — bổ sung thiết bị chặn ICMP
-
-Bước 2 và 3 chạy **song song** để giảm tổng thời gian scan.
+> **Giải quyết vấn đề ARP cache cũ (stale entries):** Khác với phương pháp ARP scan truyền thống (`arp -a` liệt kê toàn mạng) thường dính thiết bị đã ngắt kết nối do bị cache, cách làm này **chỉ query ARP cho những IP vừa pass qua bài test ping**. Thiết bị offline sẽ rớt ở bước ping và không bao giờ được đưa vào danh sách.
 
 ---
 
 ## 2. Luồng hoạt động chính (Hàm `scan_network`)
 
+```
+scan_network(network)
+    │
+    ├── stage_cb("ping")          ← Thông báo UI bắt đầu quét
+    │
+    ├── _ping_sweep(network)      ← Ping đồng thời toàn bộ host
+    │       │
+    │       ├── _ping_one(ip) × N ← Mỗi host 1 thread
+    │       │
+    │       └── return [alive IPs]
+    │
+    ├── stage_cb("mac")           ← Thông báo UI bắt đầu lấy MAC
+    │
+    ├── _get_mac_from_arp(ip) × K ← Tra MAC song song cho K IP alive
+    │
+    └── return [{"ip": ..., "mac": "AA:BB:CC:..."}, ...]   ← Sorted by IP
+```
+
 **Bước 1 — Ping Sweep**
 
 - Gọi `_ping_sweep(network)`: gửi ICMP Echo Request đồng thời tới **toàn bộ host** trong dải mạng.
-- Mỗi thiết bị phản hồi sẽ khiến hệ điều hành **tự ghi MAC vào ARP Cache**.
-- Sau khi sweep xong, chờ `0.3s` để OS kịp finalize ARP cache (giảm từ 1s trước đây).
+- Chỉ các IP có `returncode == 0` (phản hồi thành công) mới được đưa vào danh sách `alive_ips`.
 
-**Bước 2 + 3 — ARP Table & Scapy (song song)**
+**Bước 2 — Địa chỉ MAC (ARP Lookup)**
 
-- Hai hàm `_collect_arp()` và `_collect_scapy()` được submit vào `ThreadPoolExecutor(max_workers=2)` và chạy đồng thời:
-  - `_collect_arp()`: đọc `arp -a`, parse regex lấy IP + MAC.
-  - `_collect_scapy()`: gửi ARP broadcast, nhận phản hồi trực tiếp từ thiết bị.
-- Kết quả merge theo IP: ARP table làm nền, Scapy bổ sung IP còn thiếu.
+- Từ danh sách `alive_ips`, tạo ThreadPoolExecutor (max_workers=32) để gọi `_get_mac_from_arp(ip)` đồng thời.
+- Trích xuất MAC address bằng pattern matching chéo nền tảng.
 
-**Bước 4 — Trả về kết quả**
+**Bước 3 — Trả về kết quả**
 
 - Danh sách sort tăng dần theo IP:
-  `[{"ip": "192.168.1.2", "mac": "aa:bb:cc:dd:ee:ff"}, ...]`
+  `[{"ip": "192.168.1.2", "mac": "AA:BB:CC:DD:EE:FF"}, ...]`
 
 ---
 
@@ -40,48 +53,53 @@ Bước 2 và 3 chạy **song song** để giảm tổng thời gian scan.
 
 ### `_ping_one(ip, is_win)`
 
-Ping một IP đơn lẻ với timeout tối ưu theo nền tảng:
+Ping một IP đơn lẻ, trả về `True` nếu host phản hồi, `False` nếu không.
+
+Timeout tối ưu theo nền tảng:
 
 | OS      | Lệnh               | Timeout wait | Timeout process |
-| ------- | ------------------ | ----------------- | --------------- |
+| ------- | ------------------ | ------------ | --------------- |
 | Windows | `ping -n 1 -w 300` | 300ms        | 2s              |
-| macOS   | `ping -c 1 -W 500` | 500ms (đơn vị ms) | 2s              |
-| Linux   | `ping -c 1 -W 1`   | 1s (đơn vị giây)  | 2s              |
+| macOS   | `ping -c 1 -W 300` | 300ms        | 2s              |
+| Linux   | `ping -c 1 -W 1`   | 1s           | 2s              |
 
-> macOS và Linux dùng cùng flag `-W` nhưng đơn vị khác nhau — được xử lý tách biệt theo `sys.platform`.
+> macOS và Linux dùng cùng flag `-W` nhưng đơn vị khác nhau (macOS: ms, Linux: giây) — được xử lý tách biệt theo `sys.platform`.
+
+Windows sử dụng `CREATE_NO_WINDOW` flag để tránh mở cửa sổ console cho mỗi subprocess.
 
 ### `_ping_sweep(network, progress_cb)`
 
 - Tạo `ThreadPoolExecutor(max_workers=len(hosts))` — **toàn bộ host ping đồng thời**, không batching.
 - Giới hạn an toàn: chỉ chạy với subnet `/24` trở xuống (`num_addresses <= 256`).
+- Trả về danh sách IP (string) đã phản hồi thành công.
 - Gọi `progress_cb(done, total)` sau mỗi ping để UI cập nhật thanh tiến độ.
 
-### `_collect_arp()` (nội bộ trong `scan_network`)
+### `_get_mac_from_arp(ip)`
 
-Đọc và parse output `arp -a`, hỗ trợ đa nền tảng:
+- Gọi lệnh hệ điều hành để đọc ARP cache cho IP cụ thể:
+  - **Windows**: `arp -a <ip>`
+  - **macOS / Linux**: `arp -n <ip>`
+- Sử dụng Regex để parse MAC address và trả về dưới format chuẩn `AA:BB:CC:DD:EE:FF`.
+- Nếu không tìm thấy, fallback thành `"N/A"`.
 
-- **Windows:** Regex nhận dạng dạng `cc-2d-21-a5-85-b0 dynamic`, chuẩn hóa MAC sang `cc:2d:21:a5:85:b0`.
-- **macOS/Linux:** Regex nhận dạng dạng `(192.168.1.1) at aa:bb:cc:dd:ee:ff`, bỏ qua entry `(incomplete)`.
+### `scan_network(network, progress_cb, stage_cb)`
 
-### `_collect_scapy()` (nội bộ trong `scan_network`)
-
-- Gửi ARP `Who-has` broadcast (`Ether/ARP` qua `srp`) với **timeout 1s** (giảm từ 2s).
-- Stderr bị redirect tạm thời khi import scapy để tránh spam log ra console.
-- Tự động bỏ qua nếu scapy không khả dụng (không có Npcap / không có quyền root).
+- Entry point chính.
+- `stage_cb("ping")` và `stage_cb("mac")` thông báo UI giai đoạn hiện tại.
+- Trả về `list[dict]` với format `{"ip": str, "mac": str}`, sorted theo IP tăng dần.
 
 ---
 
-## 4. So sánh hiệu năng (trước và sau tối ưu)
+## 4. Hiệu năng
 
-| Thay đổi                    | Trước                 | Sau                                  |
-| --------------------------- | --------------------- | ------------------------------------ |
-| Ping workers                | 100 (batching ~3 đợt) | `len(hosts)` (~254, tất cả cùng lúc) |
-| Ping timeout — Windows      | 600ms                 | 300ms                                |
-| Ping timeout — macOS        | 1ms (sai đơn vị)      | 500ms                                |
-| Sleep sau ping sweep        | 1.0s                  | 0.3s                                 |
-| ARP + Scapy                 | Tuần tự               | **Song song**                        |
-| Scapy timeout               | 2s                    | 1s                                   |
-| **Tổng thời gian scan /24** | ~5–7s                 | **~1.5–2s**                          |
+| Thông số                    | Giá trị                              |
+| --------------------------- | ------------------------------------ |
+| Ping workers                | `len(hosts)` (~254, tất cả cùng lúc) |
+| Ping timeout — Windows      | 300ms                                |
+| Ping timeout — macOS        | 300ms                                |
+| Ping timeout — Linux        | 1s                                   |
+| Process timeout             | 2s                                   |
+| **Tổng thời gian scan /24** | **~1–2s**                            |
 
 ---
 
@@ -89,13 +107,14 @@ Ping một IP đơn lẻ với timeout tối ưu theo nền tảng:
 
 **Ưu điểm:**
 
-- Tỷ lệ phát hiện thiết bị cao nhờ kết hợp 3 lớp.
-- Không cần quyền Admin/Root — ping sweep + ARP table vẫn tìm được ~90% thiết bị.
-- Tương thích đa nền tảng (Windows/macOS/Linux) qua xử lý riêng từng OS.
-- ARP table và Scapy chạy song song → không cộng dồn thời gian chờ.
+- **Có đầy đủ IP và MAC**, rất hữu ích cho log và tracking lịch sử thiết bị nạp FW.
+- **Không bị dính ARP cache cũ**: Do chỉ lấy MAC của các IP vừa ping thành công.
+- Không cần quyền Admin/Root.
+- Không phụ thuộc thư viện bên ngoài (không cần Npcap / Scapy phức tạp).
+- Tương thích đa nền tảng (Windows/macOS/Linux).
+- Cực nhanh nhờ cơ chế full-parallel.
 
 **Nhược điểm:**
 
-- Vẫn cần `sleep(0.3s)` để OS kịp ghi ARP cache sau ping sweep.
-- Thiết bị tắt hoàn toàn cả ICMP lẫn ARP sẽ không bị phát hiện.
-- Spawn ~254 process `ping` đồng thời trên Windows có overhead cao hơn Unix do Windows tạo process chậm hơn.
+- Thiết bị cố tình chặn gói tin ICMP (tắt ping) sẽ không bị phát hiện.
+- Ping 254 thiết bị cùng lúc bằng tiến trình con (`subprocess`) trên Windows có overhead hệ thống cao hơn Unix.

main.py

@@ -16,7 +16,7 @@ from PyQt6.QtWidgets import (
 )
 from PyQt6.QtCore import (
     Qt, QThread, pyqtSignal, QPropertyAnimation, QAbstractAnimation,
-    QParallelAnimationGroup, QRect
+    QParallelAnimationGroup, QRect, QTimer
 )
 from PyQt6.QtGui import QFont, QColor, QIcon, QAction
 
@@ -83,7 +83,7 @@ class App(QWidget):
         title_row.addStretch()
         copyright_label = QLabel(f"© {datetime.datetime.now().year} Smatec — R&D Software Team. v{get_version()}")
         copyright_label.setStyleSheet(
-            "color: #9399b2; font-size: 15px; font-weight: 500;"
+            "color: #9399b2; font-size: 13px; font-weight: 500;"
         )
         title_row.addWidget(copyright_label)
         layout.addLayout(title_row)
@@ -138,7 +138,7 @@ class App(QWidget):
         self.fw_label = QLabel("No firmware selected")
         self.fw_label.setStyleSheet("font-size: 12px; color: #94a3b8; border: none;")
         fw_card_layout.addWidget(self.fw_label, 1)
-        btn_fw = QPushButton("📁 Chọn FW")
+        btn_fw = QPushButton("📁 Browse FW")
         btn_fw.setFixedHeight(28)
         btn_fw.setStyleSheet("""
             QPushButton {
@@ -239,7 +239,7 @@ class App(QWidget):
         filter_row.addWidget(self.device_count_label)
         filter_row.addStretch()
 
-        btn_history = QPushButton("📋 Lịch sử nạp")
+        btn_history = QPushButton("📋 Flash History")
         btn_history.setFixedHeight(24)
         btn_history.clicked.connect(self._show_history)
         btn_history.setStyleSheet("background-color: #313244; color: #cdd6f4; font-size: 11px; padding: 2px 8px;")
@@ -315,8 +315,8 @@ class App(QWidget):
         mc_layout.addWidget(meth_lbl)
 
         self.method_combo = QComboBox()
-        self.method_combo.addItem("🌐 API (LuCI)", "api")
         self.method_combo.addItem("💻 SSH", "ssh")
+        self.method_combo.addItem("🌐 API (LuCI)", "api")
         self.method_combo.setMinimumWidth(140)
         self.method_combo.setFixedHeight(28)
         self.method_combo.setStyleSheet("""
@@ -379,7 +379,7 @@ class App(QWidget):
         action_row.setSpacing(10)
 
         par_lbl = QLabel("Concurrent:")
-        par_lbl.setStyleSheet("font-size: 12px; font-weight: bold;")
+        par_lbl.setStyleSheet("font-size: 13px; font-weight: bold; color: #e2e8f0;")
         action_row.addWidget(par_lbl)
 
         self.parallel_spin = QSpinBox()
@@ -387,9 +387,13 @@ class App(QWidget):
         self.parallel_spin.setValue(10)
         self.parallel_spin.setSpecialValueText("∞")
         self.parallel_spin.setToolTip("0 = unlimited (all at once)")
-        self.parallel_spin.setFixedWidth(65)
-        self.parallel_spin.setFixedHeight(28)
-        self.parallel_spin.setStyleSheet("QSpinBox { font-size: 13px; font-weight: bold; }")
+        self.parallel_spin.setMinimumWidth(80)
+        self.parallel_spin.setFixedHeight(34)
+        self.parallel_spin.setStyleSheet(
+            "QSpinBox { font-size: 14px; font-weight: bold; color: #e2e8f0; "
+            "background-color: #2d3352; border: 1px solid #3d4a6b; border-radius: 6px; padding: 2px 6px; } "
+            "QSpinBox::up-button { width: 18px; } QSpinBox::down-button { width: 18px; }"
+        )
         action_row.addWidget(self.parallel_spin)
 
         self.btn_flash = QPushButton("⚡ FLASH SELECTED DEVICES")
@@ -404,7 +408,7 @@ class App(QWidget):
         layout.addWidget(flash_group)
 
         # ── Automation Button ──
-        self.btn_auto = QPushButton("🤖  Tự động hóa nạp FW")
+        self.btn_auto = QPushButton("🤖  Auto Firmware Flash")
         self.btn_auto.setObjectName("auto")
         self.btn_auto.setFixedHeight(32)
         self.btn_auto.setStyleSheet("""
@@ -453,11 +457,14 @@ class App(QWidget):
         lbl.setStyleSheet("color: #cdd6f4; font-weight: bold;")
         return lbl
 
+    # IPs permanently hidden from the device list (never shown in UI)
+    _HIDDEN_IPS = {"192.168.11.102"}
+
     def _get_filtered_devices(self):
         """Return devices filtered based on show_all checkbox."""
         if self.show_all_cb.isChecked():
-            return list(self.all_devices)
-        excluded = {self.local_ip, self.gateway_ip}
+            return [d for d in self.all_devices if d["ip"] not in self._HIDDEN_IPS]
+        excluded = {self.local_ip, self.gateway_ip} | self._HIDDEN_IPS
         return [d for d in self.all_devices if d["ip"] not in excluded]
 
     def _refresh_table(self):
@@ -528,17 +535,77 @@ class App(QWidget):
                 item.setCheckState(Qt.CheckState.Unchecked)
                 
     def _show_history(self):
-        """Hiển thị lịch sử thiết bị đã nạp trong phiên."""
+        """Show history of flashed devices in this session using a table dialog."""
         if not self.flashed_macs:
-            QMessageBox.information(self, "Lịch sử nạp", "Chưa có thiết bị nào được nạp trong phiên này.")
-        else:
-            lines = []
-            for mac in sorted(self.flashed_macs.keys()):
+            QMessageBox.information(self, "Flash History", "No devices have been flashed in this session.")
+            return
+
+        from PyQt6.QtWidgets import QDialog
+        dialog = QDialog(self)
+        dialog.setWindowTitle("Flash History")
+        dialog.resize(600, 400)
+        dialog.setStyleSheet("QDialog { background-color: #1a1b2e; } QTableWidget { font-size: 12px; }")
+
+        layout = QVBoxLayout(dialog)
+        layout.setContentsMargins(10, 10, 10, 10)
+
+        table = QTableWidget()
+        table.setColumnCount(4)
+        table.setHorizontalHeaderLabels(["Time", "IP", "MAC", "Result"])
+        table.setAlternatingRowColors(True)
+        table.setEditTriggers(QTableWidget.EditTrigger.NoEditTriggers)
+        table.setSelectionBehavior(QTableWidget.SelectionBehavior.SelectRows)
+        
+        header = table.horizontalHeader()
+        header.setSectionResizeMode(0, QHeaderView.ResizeMode.Fixed)
+        header.resizeSection(0, 80)
+        header.setSectionResizeMode(1, QHeaderView.ResizeMode.Fixed)
+        header.resizeSection(1, 120)
+        header.setSectionResizeMode(2, QHeaderView.ResizeMode.Fixed)
+        header.resizeSection(2, 140)
+        header.setSectionResizeMode(3, QHeaderView.ResizeMode.Stretch)
+        table.verticalHeader().setVisible(False)
+
+        table.setRowCount(len(self.flashed_macs))
+        
+        success_count = 0
+        fail_count = 0
+
+        for row, mac in enumerate(sorted(self.flashed_macs.keys())):
             ip, _, result, ts = self.flashed_macs[mac]
-                icon = "✅" if result.startswith("DONE") else "❌"
-                lines.append(f"[{ts}] {icon} {ip}  ({mac})  — {result}")
-            msg = f"Lịch sử nạp FW ({len(self.flashed_macs)} thiết bị):\n\n" + "\n".join(lines)
-            QMessageBox.information(self, "Lịch sử nạp", msg)
+            ok = result.startswith("DONE")
+            if ok:
+                success_count += 1
+            else:
+                fail_count += 1
+
+            time_item = QTableWidgetItem(ts)
+            time_item.setTextAlignment(Qt.AlignmentFlag.AlignCenter)
+            table.setItem(row, 0, time_item)
+            table.setItem(row, 1, QTableWidgetItem(ip))
+            table.setItem(row, 2, QTableWidgetItem(mac))
+            
+            res_item = QTableWidgetItem(f"✅ {result}" if ok else f"❌ {result}")
+            res_item.setForeground(QColor("#a6e3a1") if ok else QColor("#f38ba8"))
+            table.setItem(row, 3, res_item)
+
+        layout.addWidget(table)
+
+        summary_lbl = QLabel(f"Total: {len(self.flashed_macs)}  |  ✅ {success_count}  |  ❌ {fail_count}")
+        summary_lbl.setStyleSheet("color: #cdd6f4; font-size: 13px; font-weight: bold;")
+        layout.addWidget(summary_lbl)
+
+        btn_close = QPushButton("Close")
+        btn_close.setFixedHeight(30)
+        btn_close.setFixedWidth(100)
+        btn_close.clicked.connect(dialog.accept)
+        
+        btn_layout = QHBoxLayout()
+        btn_layout.addStretch()
+        btn_layout.addWidget(btn_close)
+        layout.addLayout(btn_layout)
+
+        dialog.exec()
 
     # ── Actions ──
 
@@ -607,6 +674,11 @@ class App(QWidget):
             self.scan_progress_bar.setFormat("⚡  Pinging hosts... %v / %m")
             self.scan_status.setText("⏳  Pinging all hosts...")
             self.scan_status.setStyleSheet("color: #f9e2af;")
+        elif stage == "mac":
+            self.scan_progress_bar.setRange(0, 0)
+            self.scan_progress_bar.setFormat("  Reading MAC addresses...")
+            self.scan_status.setText("⏳  Reading MAC addresses from ARP...")
+            self.scan_status.setStyleSheet("color: #f9e2af;")
         elif stage == "arp":
             self.scan_progress_bar.setRange(0, 0)
             self.scan_progress_bar.setFormat("  Reading ARP cache...")
@@ -694,9 +766,9 @@ class App(QWidget):
             blocked = [dev["ip"] for _, dev in selected if dev["ip"] == "192.168.11.102"]
             if blocked:
                 QMessageBox.warning(
-                    self, "Không được phép",
-                    "⚠️ Thiết bị 192.168.11.102 chỉ được nạp ở chế độ Update FW.\n"
-                    "Vui lòng bỏ chọn thiết bị này hoặc chuyển sang chế độ Update FW."
+                    self, "Not Allowed",
+                    "⚠️ Device 192.168.11.102 can only be flashed in Update FW mode.\n"
+                    "Please deselect this device or switch to Update FW mode."
                 )
                 return
 
@@ -754,10 +826,32 @@ class App(QWidget):
         self.flash_thread.device_done.connect(self._on_flash_done)
         self.flash_thread.all_done.connect(self._on_flash_all_done)
 
-        # Disable flash button trong khi đang flash
+        # Disable flash button và lock UI trong khi đang flash
         self.btn_flash.setEnabled(False)
+        self._set_main_ui_locked(True)
         self.flash_thread.start()
 
+    def _set_main_ui_locked(self, locked):
+        """Khóa toàn bộ UI nhưng vẫn cho cuộn dọc bảng thiết bị."""
+        enabled = not locked
+        self.btn_scan.setEnabled(enabled)
+        self.net_input.setEnabled(enabled)
+        self.mode_combo.setEnabled(enabled)
+        self.method_combo.setEnabled(enabled)
+        self.parallel_spin.setEnabled(enabled)
+        self.show_all_cb.setEnabled(enabled)
+        
+        # Bảng thiết bị: không setEnabled(False) để giữ thanh cuộn
+        if locked:
+            self.table.setSelectionMode(QTableWidget.SelectionMode.NoSelection)
+            self.table.setStyleSheet(
+                "QTableWidget { color: #6b7280; font-size: 13px; }"
+                "QTableWidget::item { selection-background-color: transparent; }"
+            )
+        else:
+            self.table.setSelectionMode(QTableWidget.SelectionMode.ExtendedSelection)
+            self.table.setStyleSheet("QTableWidget { font-size: 13px; }")
+
     def _on_flash_status(self, index, msg):
         """Update status column while flashing."""
         row = self._flash_row_map.get(index, index)
@@ -768,7 +862,6 @@ class App(QWidget):
         row = self._flash_row_map.get(index, index)
         mac_item = self.table.item(row, 2)
         ip_item = self.table.item(row, 1)
-        import datetime
         now_str = datetime.datetime.now().strftime("%H:%M:%S")
         mac_str = mac_item.text().strip() if mac_item else ""
         ip_str = ip_item.text().strip() if ip_item else ""
@@ -795,13 +888,14 @@ class App(QWidget):
     def _on_flash_all_done(self):
         """All flashing complete."""
         self.btn_flash.setEnabled(True)
+        self._set_main_ui_locked(False)
         QMessageBox.information(self, "Flash Complete", "All devices have been processed.")
 
     def _open_auto_flash(self):
-        """Chuyển sang giao diện Tự động hóa nạp FW."""
-        # Sync firmware & network từ main sang auto
+        """Switch to Auto Flash UI."""
+        # Sync firmware & network from main to auto
         self.auto_fw_label.setText(
-            os.path.basename(self.firmware) if self.firmware else "Chưa chọn"
+            os.path.basename(self.firmware) if self.firmware else "Not selected"
         )
         self.auto_fw_label.setStyleSheet(
             "color: #a6e3a1; font-weight: bold; font-size: 12px;" if self.firmware
@@ -814,14 +908,14 @@ class App(QWidget):
         self.auto_container.setVisible(True)
 
     def _back_to_main(self):
-        """Quay lại giao diện chính."""
+        """Return to main view."""
         self.auto_container.setVisible(False)
         self.main_container.setVisible(True)
 
     # ── Auto Flash UI Builder ──
 
     def _build_auto_ui(self):
-        """Xây dựng giao diện tự động nạp FW bên trong auto_container."""
+        """Build the Auto Firmware Flash UI inside auto_container."""
         self.auto_firmware = self.firmware
         self._auto_worker = None
         self._auto_device_rows = {}
@@ -837,7 +931,7 @@ class App(QWidget):
         # ── Back button + Title row ──
         top_row = QHBoxLayout()
         top_row.setSpacing(8)
-        self.btn_back = QPushButton("⬅  Quay lại")
+        self.btn_back = QPushButton("⬅  Back")
         self.btn_back.setFixedHeight(32)
         self.btn_back.setStyleSheet("""
             QPushButton {
@@ -855,14 +949,14 @@ class App(QWidget):
         self.btn_back.clicked.connect(self._back_to_main)
         top_row.addWidget(self.btn_back)
 
-        auto_title = QLabel("🤖 Tự động hóa nạp FW")
+        auto_title = QLabel("🤖 Auto Firmware Flash")
         auto_title.setStyleSheet("font-size: 16px; font-weight: bold; color: #c4b5fd; letter-spacing: 1px;")
         auto_title.setAlignment(Qt.AlignmentFlag.AlignCenter)
         top_row.addWidget(auto_title, 1)
         auto_layout.addLayout(top_row)
 
         # ── Config group ──
-        config_group = CollapsibleGroupBox("⚙️  Cấu hình nạp")
+        config_group = CollapsibleGroupBox("⚙️  Flash Configuration")
         config_layout = QVBoxLayout()
         config_layout.setSpacing(6)
         config_layout.setContentsMargins(8, 4, 8, 4)
@@ -873,7 +967,7 @@ class App(QWidget):
         fw_lbl = QLabel("FW:")
         fw_lbl.setStyleSheet("font-weight: bold; font-size: 12px;")
         row1.addWidget(fw_lbl)
-        self.auto_fw_label = QLabel(os.path.basename(self.firmware) if self.firmware else "Chưa chọn")
+        self.auto_fw_label = QLabel(os.path.basename(self.firmware) if self.firmware else "Not selected")
         self.auto_fw_label.setStyleSheet(
             "color: #a6e3a1; font-weight: bold; font-size: 12px;" if self.firmware
             else "color: #f38ba8; font-size: 12px;"
@@ -882,7 +976,7 @@ class App(QWidget):
         btn_fw = QPushButton("📁")
         btn_fw.setFixedSize(40, 30)
         btn_fw.setStyleSheet("font-size: 16px;")
-        btn_fw.setToolTip("Chọn firmware")
+        btn_fw.setToolTip("Select firmware")
         btn_fw.clicked.connect(self._auto_select_firmware)
         row1.addWidget(btn_fw)
 
@@ -890,7 +984,7 @@ class App(QWidget):
         sep1.setStyleSheet("color: #3d4a6b; font-size: 14px;")
         row1.addWidget(sep1)
 
-        net_lbl = QLabel("Mạng:")
+        net_lbl = QLabel("Network:")
         net_lbl.setStyleSheet("font-weight: bold; font-size: 12px;")
         row1.addWidget(net_lbl)
         self.auto_net_input = QLineEdit(get_default_network(self.local_ip))
@@ -906,46 +1000,66 @@ class App(QWidget):
         row2 = QHBoxLayout()
         row2.setSpacing(12)
 
-        cnt_lbl = QLabel("Số lượng:")
-        cnt_lbl.setStyleSheet("font-weight: bold; font-size: 12px;")
+        cnt_lbl = QLabel("Target Count:")
+        cnt_lbl.setStyleSheet("font-weight: bold; font-size: 13px; color: #e2e8f0;")
         row2.addWidget(cnt_lbl)
         self.auto_target_spin = QSpinBox()
         self.auto_target_spin.setRange(1, 500)
         self.auto_target_spin.setValue(5)
-        self.auto_target_spin.setFixedWidth(70)
-        self.auto_target_spin.setFixedHeight(28)
-        self.auto_target_spin.setStyleSheet("QSpinBox { font-size: 13px; font-weight: bold; }")
+        self.auto_target_spin.setMinimumWidth(80)
+        self.auto_target_spin.setFixedHeight(34)
+        self.auto_target_spin.setStyleSheet(
+            "QSpinBox { font-size: 14px; font-weight: bold; color: #e2e8f0; "
+            "background-color: #2d3352; border: 1px solid #3d4a6b; border-radius: 6px; padding: 2px 6px; } "
+            "QSpinBox::up-button { width: 18px; } QSpinBox::down-button { width: 18px; }"
+        )
         row2.addWidget(self.auto_target_spin)
 
         sep2 = QLabel("│")
         sep2.setStyleSheet("color: #3d4a6b; font-size: 14px;")
         row2.addWidget(sep2)
 
-        meth_lbl = QLabel("Phương thức:")
+        meth_lbl = QLabel("Method:")
         meth_lbl.setStyleSheet("font-weight: bold; font-size: 12px;")
         row2.addWidget(meth_lbl)
         self.auto_method_combo = QComboBox()
-        self.auto_method_combo.addItem("🌐 API (LuCI)", "api")
         self.auto_method_combo.addItem("💻 SSH", "ssh")
-        self.auto_method_combo.setFixedHeight(28)
+        self.auto_method_combo.addItem("🌐 API (LuCI)", "api")
+        self.auto_method_combo.setFixedHeight(34)
         self.auto_method_combo.setMinimumWidth(140)
+        self.auto_method_combo.setStyleSheet("""
+            QComboBox {
+                background-color: #2d3352; border: 1px solid #3d4a6b; border-radius: 6px;
+                padding: 2px 8px; color: #e2e8f0; font-size: 13px; font-weight: bold;
+            }
+            QComboBox:hover { border-color: #7eb8f7; }
+            QComboBox::drop-down { border: none; width: 20px; }
+            QComboBox QAbstractItemView {
+                background-color: #2d3352; color: #e2e8f0;
+                border: 1px solid #3d4a6b; selection-background-color: #3d4a6b;
+            }
+        """)
         row2.addWidget(self.auto_method_combo)
 
         sep3 = QLabel("│")
         sep3.setStyleSheet("color: #3d4a6b; font-size: 14px;")
         row2.addWidget(sep3)
 
-        par_lbl = QLabel("Song song:")
-        par_lbl.setStyleSheet("font-weight: bold; font-size: 12px;")
+        par_lbl = QLabel("Concurrent:")
+        par_lbl.setStyleSheet("font-weight: bold; font-size: 13px; color: #e2e8f0;")
         row2.addWidget(par_lbl)
         self.auto_parallel_spin = QSpinBox()
         self.auto_parallel_spin.setRange(0, 100)
         self.auto_parallel_spin.setValue(10)
         self.auto_parallel_spin.setSpecialValueText("∞")
-        self.auto_parallel_spin.setToolTip("0 = không giới hạn")
-        self.auto_parallel_spin.setFixedWidth(65)
-        self.auto_parallel_spin.setFixedHeight(28)
-        self.auto_parallel_spin.setStyleSheet("QSpinBox { font-size: 13px; font-weight: bold; }")
+        self.auto_parallel_spin.setToolTip("0 = unlimited")
+        self.auto_parallel_spin.setMinimumWidth(80)
+        self.auto_parallel_spin.setFixedHeight(34)
+        self.auto_parallel_spin.setStyleSheet(
+            "QSpinBox { font-size: 14px; font-weight: bold; color: #e2e8f0; "
+            "background-color: #2d3352; border: 1px solid #3d4a6b; border-radius: 6px; padding: 2px 6px; } "
+            "QSpinBox::up-button { width: 18px; } QSpinBox::down-button { width: 18px; }"
+        )
         row2.addWidget(self.auto_parallel_spin)
         row2.addStretch()
         config_layout.addLayout(row2)
@@ -956,7 +1070,7 @@ class App(QWidget):
         # ── Control Buttons ──
         btn_row = QHBoxLayout()
         btn_row.setSpacing(8)
-        self.auto_btn_start = QPushButton("▶  XÁC NHẬN & BẮT ĐẦU")
+        self.auto_btn_start = QPushButton("▶  CONFIRM & START")
         self.auto_btn_start.setObjectName("start_btn")
         self.auto_btn_start.setFixedHeight(36)
         self.auto_btn_start.setStyleSheet("""
@@ -975,7 +1089,7 @@ class App(QWidget):
         self.auto_btn_start.clicked.connect(self._auto_on_start)
         btn_row.addWidget(self.auto_btn_start)
 
-        self.auto_btn_stop = QPushButton("⏹  DỪNG")
+        self.auto_btn_stop = QPushButton("⏹  STOP")
         self.auto_btn_stop.setObjectName("stop_btn")
         self.auto_btn_stop.setFixedHeight(36)
         self.auto_btn_stop.setEnabled(False)
@@ -999,7 +1113,7 @@ class App(QWidget):
         # ── Status + Progress ──
         status_row = QHBoxLayout()
         status_row.setSpacing(8)
-        self.auto_status_label = QLabel("⏸  Chờ bắt đầu...")
+        self.auto_status_label = QLabel("⏸  Waiting to start...")
         self.auto_status_label.setStyleSheet("font-size: 12px; font-weight: bold; color: #94a3b8;")
         status_row.addWidget(self.auto_status_label, 1)
 
@@ -1012,14 +1126,14 @@ class App(QWidget):
         auto_layout.addLayout(status_row)
 
         # ── Device Table ──
-        dev_group = QGroupBox("📋  Danh sách thiết bị")
+        dev_group = QGroupBox("📋  Device List")
         dev_layout = QVBoxLayout()
         dev_layout.setSpacing(2)
         dev_layout.setContentsMargins(4, 12, 4, 4)
 
         self.auto_result_table = QTableWidget()
         self.auto_result_table.setColumnCount(4)
-        self.auto_result_table.setHorizontalHeaderLabels(["#", "IP", "MAC", "Kết quả"])
+        self.auto_result_table.setHorizontalHeaderLabels(["#", "IP", "MAC", "Result"])
         self.auto_result_table.setAlternatingRowColors(True)
         header = self.auto_result_table.horizontalHeader()
         header.setSectionResizeMode(0, QHeaderView.ResizeMode.Fixed)
@@ -1040,7 +1154,7 @@ class App(QWidget):
         self.auto_summary_label.setStyleSheet("color: #94a3b8; font-size: 11px; padding: 2px;")
         summary_row.addWidget(self.auto_summary_label, 1)
 
-        btn_auto_history = QPushButton("📋 Lịch sử nạp")
+        btn_auto_history = QPushButton("📋 Flash History")
         btn_auto_history.setFixedHeight(24)
         btn_auto_history.setStyleSheet("background-color: #313244; color: #cdd6f4; font-size: 11px; padding: 2px 8px;")
         btn_auto_history.clicked.connect(self._show_auto_history)
@@ -1060,14 +1174,13 @@ class App(QWidget):
         self.auto_log_content.setWordWrap(True)
         self.auto_log_content.setAlignment(Qt.AlignmentFlag.AlignTop)
         self.auto_log_content.setStyleSheet(
-            "color: #cdd6f4; font-size: 10px; font-family: 'SF Mono', 'Menlo', monospace;"
-            "padding: 4px; background-color: #11121d; border-radius: 4px;"
+            "color: #cdd6f4; font-size: 12px; font-family: 'Consolas', 'Courier New', monospace;"
+            "padding: 6px; background-color: #11121d; border-radius: 4px;"
         )
         self.auto_log_content.setTextFormat(Qt.TextFormat.PlainText)
         self.auto_log_area.setWidget(self.auto_log_content)
         self.auto_log_area.setWidgetResizable(True)
-        self.auto_log_area.setMinimumHeight(120)
-        self.auto_log_area.setMaximumHeight(280)
+        self.auto_log_area.setMinimumHeight(150)
         self.auto_log_area.setStyleSheet("QScrollArea { border: 1px solid #2d3748; border-radius: 4px; background-color: #11121d; }")
         log_layout.addWidget(self.auto_log_area)
 
@@ -1077,19 +1190,80 @@ class App(QWidget):
     # ── Auto Flash Actions ──
 
     def _show_auto_history(self):
+        """Show history of auto-flashed devices using a table dialog."""
         if not self._auto_history:
-            QMessageBox.information(self, "Lịch sử nạp", "Chưa có thiết bị nào được nạp trong phiên này.")
+            QMessageBox.information(self, "Flash History", "No devices have been flashed in this session.")
             return
-        lines = []
-        for ip, mac, result, ts in self._auto_history:
-            icon = "✅" if result.startswith("DONE") else "❌"
-            lines.append(f"[{ts}] {icon} {ip}  ({mac})  — {result}")
-        msg = f"Lịch sử nạp FW ({len(self._auto_history)} thiết bị):\n\n" + "\n".join(lines)
-        QMessageBox.information(self, "Lịch sử nạp", msg)
+
+        from PyQt6.QtWidgets import QDialog
+        dialog = QDialog(self)
+        dialog.setWindowTitle("Auto Flash History")
+        dialog.resize(600, 400)
+        dialog.setStyleSheet("QDialog { background-color: #1a1b2e; } QTableWidget { font-size: 12px; }")
+
+        layout = QVBoxLayout(dialog)
+        layout.setContentsMargins(10, 10, 10, 10)
+
+        table = QTableWidget()
+        table.setColumnCount(4)
+        table.setHorizontalHeaderLabels(["Time", "IP", "MAC", "Result"])
+        table.setAlternatingRowColors(True)
+        table.setEditTriggers(QTableWidget.EditTrigger.NoEditTriggers)
+        table.setSelectionBehavior(QTableWidget.SelectionBehavior.SelectRows)
+        
+        header = table.horizontalHeader()
+        header.setSectionResizeMode(0, QHeaderView.ResizeMode.Fixed)
+        header.resizeSection(0, 80)
+        header.setSectionResizeMode(1, QHeaderView.ResizeMode.Fixed)
+        header.resizeSection(1, 120)
+        header.setSectionResizeMode(2, QHeaderView.ResizeMode.Fixed)
+        header.resizeSection(2, 140)
+        header.setSectionResizeMode(3, QHeaderView.ResizeMode.Stretch)
+        table.verticalHeader().setVisible(False)
+
+        table.setRowCount(len(self._auto_history))
+        
+        success_count = 0
+        fail_count = 0
+
+        for row, (ip, mac, result, ts) in enumerate(self._auto_history):
+            ok = result.startswith("DONE")
+            if ok:
+                success_count += 1
+            else:
+                fail_count += 1
+
+            time_item = QTableWidgetItem(ts)
+            time_item.setTextAlignment(Qt.AlignmentFlag.AlignCenter)
+            table.setItem(row, 0, time_item)
+            table.setItem(row, 1, QTableWidgetItem(ip))
+            table.setItem(row, 2, QTableWidgetItem(mac))
+            
+            res_item = QTableWidgetItem(f"✅ {result}" if ok else f"❌ {result}")
+            res_item.setForeground(QColor("#a6e3a1") if ok else QColor("#f38ba8"))
+            table.setItem(row, 3, res_item)
+
+        layout.addWidget(table)
+
+        summary_lbl = QLabel(f"Total: {len(self._auto_history)}  |  ✅ {success_count}  |  ❌ {fail_count}")
+        summary_lbl.setStyleSheet("color: #cdd6f4; font-size: 13px; font-weight: bold;")
+        layout.addWidget(summary_lbl)
+
+        btn_close = QPushButton("Close")
+        btn_close.setFixedHeight(30)
+        btn_close.setFixedWidth(100)
+        btn_close.clicked.connect(dialog.accept)
+        
+        btn_layout = QHBoxLayout()
+        btn_layout.addStretch()
+        btn_layout.addWidget(btn_close)
+        layout.addLayout(btn_layout)
+
+        dialog.exec()
 
     def _auto_select_firmware(self):
         file, _ = QFileDialog.getOpenFileName(
-            self, "Chọn Firmware", "",
+            self, "Select Firmware", "",
             "Firmware Files (*.bin *.hex *.uf2);;All Files (*)"
         )
         if file:
@@ -1107,19 +1281,20 @@ class App(QWidget):
         if len(self._auto_log_lines) > 500:
             self._auto_log_lines = self._auto_log_lines[-500:]
         self.auto_log_content.setText("\n".join(self._auto_log_lines))
-        sb = self.auto_log_area.verticalScrollBar()
-        sb.setValue(sb.maximum())
+        QTimer.singleShot(0, lambda: self.auto_log_area.verticalScrollBar().setValue(
+            self.auto_log_area.verticalScrollBar().maximum()
+        ))
 
     def _auto_on_start(self):
         if not self.auto_firmware:
-            QMessageBox.warning(self, "Chưa chọn FW", "Vui lòng chọn file firmware trước.")
+            QMessageBox.warning(self, "No Firmware Selected", "Please select a firmware file first.")
             return
 
         network_str = self.auto_net_input.text().strip()
         try:
             ipaddress.ip_network(network_str, strict=False)
         except ValueError:
-            QMessageBox.warning(self, "Mạng không hợp lệ", f"'{network_str}' không phải network hợp lệ.\nVí dụ: 192.168.4.0/24")
+            QMessageBox.warning(self, "Invalid Network", f"'{network_str}' is not a valid network.\nExample: 192.168.4.0/24")
             return
 
         target = self.auto_target_spin.value()
@@ -1127,13 +1302,13 @@ class App(QWidget):
         max_workers = self.auto_parallel_spin.value()
 
         reply = QMessageBox.question(
-            self, "Xác nhận",
-            f"Bắt đầu tự động nạp FW?\n\n"
+            self, "Confirm",
+            f"Start auto firmware flash?\n\n"
             f"  Firmware: {os.path.basename(self.auto_firmware)}\n"
-            f"  Mạng: {network_str}\n"
-            f"  Số lượng: {target} thiết bị\n"
-            f"  Phương thức: {method.upper()}\n"
-            f"  Song song: {max_workers if max_workers > 0 else 'Không giới hạn'}\n",
+            f"  Network: {network_str}\n"
+            f"  Target count: {target} device(s)\n"
+            f"  Method: {method.upper()}\n"
+            f"  Concurrent: {max_workers if max_workers > 0 else 'Unlimited'}\n",
             QMessageBox.StandardButton.Yes | QMessageBox.StandardButton.No,
             QMessageBox.StandardButton.Yes,
         )
@@ -1153,13 +1328,9 @@ class App(QWidget):
 
         self.auto_btn_start.setEnabled(False)
         self.auto_btn_stop.setEnabled(True)
-        self.auto_net_input.setEnabled(False)
-        self.auto_target_spin.setEnabled(False)
-        self.auto_method_combo.setEnabled(False)
-        self.auto_parallel_spin.setEnabled(False)
-        self.btn_back.setEnabled(False)
+        self._set_auto_ui_locked(True)
 
-        self.auto_status_label.setText("🔍  Đang scan mạng LAN...")
+        self.auto_status_label.setText("🔍  Scanning LAN...")
         self.auto_status_label.setStyleSheet("font-size: 12px; font-weight: bold; color: #f9e2af;")
 
         self._auto_worker = AutoFlashWorker(
@@ -1186,13 +1357,13 @@ class App(QWidget):
         if self._auto_worker:
             self._auto_worker.stop()
         self.auto_btn_stop.setEnabled(False)
-        self.auto_status_label.setText("⏳  Đang dừng...")
+        self.auto_status_label.setText("⏳  Stopping...")
 
     def _auto_on_scan_found(self, count):
         target = self.auto_target_spin.value()
-        self.auto_status_label.setText(f"🔍  Scan: tìm thấy {count}/{target} thiết bị...")
+        self.auto_status_label.setText(f"🔍  Scan: found {count}/{target} device(s)...")
         if count >= target:
-            self.auto_status_label.setText(f"⚡  Đủ {target} thiết bị — đang nạp FW...")
+            self.auto_status_label.setText(f"⚡  {target} device(s) found — flashing firmware...")
             self.auto_status_label.setStyleSheet("font-size: 12px; font-weight: bold; color: #a6e3a1;")
 
     def _auto_on_devices_ready(self, devices):
@@ -1205,11 +1376,11 @@ class App(QWidget):
             self.auto_result_table.setItem(i, 0, num_item)
             self.auto_result_table.setItem(i, 1, QTableWidgetItem(dev["ip"]))
             self.auto_result_table.setItem(i, 2, QTableWidgetItem(dev.get("mac", "N/A").upper()))
-            waiting_item = QTableWidgetItem("⏳ Đang chờ...")
+            waiting_item = QTableWidgetItem("⏳ Waiting...")
             waiting_item.setForeground(QColor("#94a3b8"))
             self.auto_result_table.setItem(i, 3, waiting_item)
             self._auto_device_rows[dev["ip"]] = i
-        self.auto_summary_label.setText(f"Tổng: {len(devices)} thiết bị")
+        self.auto_summary_label.setText(f"Total: {len(devices)} device(s)")
 
     def _auto_on_device_status(self, ip, msg):
         row = self._auto_device_rows.get(ip)
@@ -1242,57 +1413,73 @@ class App(QWidget):
             self._auto_fail_count += 1
         self.auto_result_table.setItem(row, 3, item)
         self._auto_history.append((ip, mac.upper(), result, now_str))
+        
         total = len(self._auto_device_rows)
         done = self._auto_success_count + self._auto_fail_count
         self.auto_summary_label.setText(
-            f"Tổng: {total}  |  Xong: {done}  |  ✅ {self._auto_success_count}  |  ❌ {self._auto_fail_count}"
+            f"Total: {total}  |  Done: {done}  |  ✅ {self._auto_success_count}  |  ❌ {self._auto_fail_count}"
         )
 
     def _auto_on_flash_progress(self, done, total):
         self.auto_progress_bar.setVisible(True)
         self.auto_progress_bar.setMaximum(total)
         self.auto_progress_bar.setValue(done)
-        self.auto_status_label.setText(f"⚡  Nạp FW: {done}/{total} thiết bị...")
+        self.auto_status_label.setText(f"⚡  Flashing: {done}/{total} device(s)...")
 
     def _auto_on_all_done(self, success, fail):
         self._auto_reset_controls()
-        self.auto_status_label.setText(f"🏁  Hoàn thành! ✅ {success}  |  ❌ {fail}")
+        self.auto_status_label.setText(f"🏁  Complete! ✅ {success}  |  ❌ {fail}")
         self.auto_status_label.setStyleSheet("font-size: 12px; font-weight: bold; color: #a6e3a1;")
         QMessageBox.information(
-            self, "Hoàn thành",
-            f"Tự động nạp FW hoàn thành!\n\n"
-            f"✅  Thành công: {success}\n"
-            f"❌  Thất bại: {fail}",
+            self, "Complete",
+            f"Auto firmware flash complete!\n\n"
+            f"✅  Success: {success}\n"
+            f"❌  Failed: {fail}",
         )
 
     def _auto_on_stopped(self):
         self._auto_reset_controls()
-        self.auto_status_label.setText("⛔  Đã dừng bởi người dùng")
+        self.auto_status_label.setText("⛔  Stopped by user")
         self.auto_status_label.setStyleSheet("font-size: 12px; font-weight: bold; color: #f38ba8;")
 
     def _auto_on_scan_timeout(self, found, target):
         self._auto_reset_controls()
-        self.auto_status_label.setText(f"⚠️  Scan hết lần: chỉ tìm thấy {found}/{target} thiết bị")
+        self.auto_status_label.setText(f"⚠️  Scan timed out: only found {found}/{target} device(s)")
         self.auto_status_label.setStyleSheet("font-size: 12px; font-weight: bold; color: #fab387;")
         QMessageBox.warning(
-            self, "Không đủ thiết bị",
-            f"Đã scan tối đa nhưng chỉ tìm thấy {found}/{target} thiết bị.\n\n"
-            f"Vui lòng kiểm tra:\n"
-            f"  • Thiết bị đã bật và kết nối mạng chưa\n"
-            f"  • Dải mạng ({self.auto_net_input.text()}) có đúng không\n"
-            f"  • Thử lại sau khi kiểm tra",
+            self, "Not Enough Devices",
+            f"Scan reached maximum attempts but only found {found}/{target} device(s).\n\n"
+            f"Please check:\n"
+            f"  • Devices are powered on and connected to the network\n"
+            f"  • Network range ({self.auto_net_input.text()}) is correct\n"
+            f"  • Try again after verifying",
         )
 
     def _auto_reset_controls(self):
         self.auto_btn_start.setEnabled(True)
         self.auto_btn_stop.setEnabled(False)
-        self.auto_net_input.setEnabled(True)
-        self.auto_target_spin.setEnabled(True)
-        self.auto_method_combo.setEnabled(True)
-        self.auto_parallel_spin.setEnabled(True)
-        self.btn_back.setEnabled(True)
+        self._set_auto_ui_locked(False)
         self._auto_worker = None
 
+    def _set_auto_ui_locked(self, locked):
+        """Khóa cấu hình Auto Flash nhưng vẫn cho cuộn xem list thiết bị."""
+        enabled = not locked
+        self.auto_net_input.setEnabled(enabled)
+        self.auto_target_spin.setEnabled(enabled)
+        self.auto_method_combo.setEnabled(enabled)
+        self.auto_parallel_spin.setEnabled(enabled)
+        self.btn_back.setEnabled(enabled)
+        
+        # Bảng thiết bị Auto: không setEnabled(False) để giữ thanh cuộn
+        if locked:
+            self.auto_result_table.setSelectionMode(QTableWidget.SelectionMode.NoSelection)
+            self.auto_result_table.setStyleSheet(
+                "QTableWidget { color: #6b7280; font-size: 12px; }"
+                "QTableWidget::item { selection-background-color: transparent; }"
+            )
+        else:
+            self.auto_result_table.setSelectionMode(QTableWidget.SelectionMode.ExtendedSelection)
+            self.auto_result_table.setStyleSheet("QTableWidget { font-size: 12px; }")
 
 if __name__ == "__main__":
     app = QApplication(sys.argv)

version.txt

@@ -1 +1 @@
-1.2.1
+1.2.3