fix(symbols): add c_type and decoded_value to all symbol read endpoints

- Extract c_type from sym_info in /symbols/read and /symbols/read/stream
- Decode scalar values on backend using c_type for proper display
- Render scalar variables using struct tree style for consistent UI
- Fix NameError: 'c_type' not defined in device read endpoint

Author: FASTSHIFT

Tools/WebServer/app/routes/symbols.py

@@ -182,11 +182,13 @@ def _decode_field_value(raw_bytes: bytes, type_name: str):
     is_int = any(kw in type_lower for kw in _INT_TYPE_KEYWORDS)
     if is_int and size <= 8:
         val = int.from_bytes(raw_bytes, "little")
-        # Signed check: type doesn't start with 'u'/'U' and doesn't contain 'uint'
+        # Signed check: type doesn't start with 'u'/'U', doesn't contain 'uint',
+        # and doesn't contain 'unsigned' (handles "long unsigned int" from DWARF)
         if (
             not type_name.startswith("u")
             and not type_name.startswith("U")
             and "uint" not in type_name
+            and "unsigned" not in type_name
         ):
             max_signed = 1 << (size * 8 - 1)
             if val >= max_signed:
@@ -796,6 +798,7 @@ def api_get_symbol_value():
         sym_info.get("type", "other") if isinstance(sym_info, dict) else "function"
     )
     section = sym_info.get("section", "") if isinstance(sym_info, dict) else ""
+    c_type = sym_info.get("c_type") if isinstance(sym_info, dict) else None
     is_pointer = (
         sym_info.get("is_pointer", False) if isinstance(sym_info, dict) else False
     )
@@ -845,6 +848,14 @@ def api_get_symbol_value():
 
     logger.info(f"[value] Total for '{sym_name}': {time.time() - t_start:.2f}s")
 
+    # Decode scalar value from hex_data using c_type when no struct layout
+    decoded_value = None
+    if hex_data and c_type and not struct_layout and not is_pointer:
+        raw = bytes.fromhex(hex_data)
+        decoded_value = _decode_field_value(raw, c_type)
+        if decoded_value is None:
+            decoded_value = _decode_field_value_fallback(raw, c_type)
+
     resp = {
         "success": True,
         "name": sym_name,
@@ -856,6 +867,10 @@ def api_get_symbol_value():
         "struct_layout": struct_layout,
         "gdb_values": gdb_values,
     }
+    if c_type:
+        resp["c_type"] = c_type
+    if decoded_value is not None:
+        resp["decoded_value"] = decoded_value
     if is_pointer:
         resp["is_pointer"] = True
         resp["pointer_target"] = pointer_target
@@ -897,6 +912,7 @@ def api_read_symbol_from_device():
     pointer_target = (
         sym_info.get("pointer_target") if isinstance(sym_info, dict) else None
     )
+    c_type = sym_info.get("c_type") if isinstance(sym_info, dict) else None
     if size <= 0:
         return jsonify(
             {"success": False, "error": f"Symbol '{sym_name}' has unknown size"}
@@ -934,6 +950,14 @@ def api_read_symbol_from_device():
                 if struct_layout:
                     gdb_values = _decode_struct_values(struct_layout, hex_data)
 
+        # Decode scalar value for non-struct, non-pointer types
+        decoded_value = None
+        if hex_data and c_type and not struct_layout and not is_pointer:
+            raw = bytes.fromhex(hex_data)
+            decoded_value = _decode_field_value(raw, c_type)
+            if decoded_value is None:
+                decoded_value = _decode_field_value_fallback(raw, c_type)
+
         resp = {
             "success": True,
             "name": sym_name,
@@ -944,6 +968,10 @@ def api_read_symbol_from_device():
             "gdb_values": gdb_values,
             "source": "device",
         }
+        if c_type:
+            resp["c_type"] = c_type
+        if decoded_value is not None:
+            resp["decoded_value"] = decoded_value
         if is_pointer:
             resp["is_pointer"] = True
             resp["pointer_target"] = pointer_target
@@ -1038,6 +1066,7 @@ def api_read_symbol_stream():
     pointer_target = (
         sym_info.get("pointer_target") if isinstance(sym_info, dict) else None
     )
+    c_type = sym_info.get("c_type") if isinstance(sym_info, dict) else None
     if size <= 0:
         return jsonify(
             {"success": False, "error": f"Symbol '{sym_name}' has unknown size"}
@@ -1141,6 +1170,14 @@ def do_read_wrapper():
                     if struct_layout:
                         gdb_values = _decode_struct_values(struct_layout, hex_data)
 
+            # Decode scalar value for non-struct, non-pointer types
+            decoded_value = None
+            if hex_data and c_type and not struct_layout and not is_pointer:
+                raw = bytes.fromhex(hex_data)
+                decoded_value = _decode_field_value(raw, c_type)
+                if decoded_value is None:
+                    decoded_value = _decode_field_value_fallback(raw, c_type)
+
             resp = {
                 "type": "result",
                 "success": True,
@@ -1152,6 +1189,10 @@ def do_read_wrapper():
                 "gdb_values": gdb_values,
                 "source": "device",
             }
+            if c_type:
+                resp["c_type"] = c_type
+            if decoded_value is not None:
+                resp["decoded_value"] = decoded_value
             if is_pointer:
                 resp["is_pointer"] = True
                 resp["pointer_target"] = pointer_target

Tools/WebServer/core/gdb_session.py

@@ -540,15 +540,17 @@ def _lookup_symbol_impl(self, sym_name: str) -> Optional[dict]:
                 if ptype_out and re.match(r"type\s*=\s*const\b", ptype_out):
                     sym_type = "const"
 
-        # Detect pointer types via whatis (e.g. "type = lv_disp_t *")
+        # Detect pointer types and C type name via whatis
         is_pointer = False
         pointer_target = None
+        c_type = None
         if sym_type in ("variable", "const"):
             whatis_out = self._execute_cli(f"whatis {query_name}")
             if whatis_out:
                 wm = re.match(r"type\s*=\s*(.+)", whatis_out.strip())
                 if wm:
                     raw_type = wm.group(1).strip()
+                    c_type = raw_type
                     # Pointer if type ends with '*' (but not function pointer)
                     if raw_type.endswith("*") and "(" not in raw_type:
                         is_pointer = True
@@ -557,7 +559,7 @@ def _lookup_symbol_impl(self, sym_name: str) -> Optional[dict]:
         elapsed = time.time() - t_start
         logger.info(
             f"[GDB] lookup_symbol done: '{sym_name}' -> "
-            f"0x{addr:08X} size={size} type={sym_type}"
+            f"0x{addr:08X} size={size} type={sym_type} c_type={c_type}"
             f"{' ptr->' + pointer_target if is_pointer else ''}"
             f" ({elapsed:.3f}s)"
         )
@@ -568,6 +570,8 @@ def _lookup_symbol_impl(self, sym_name: str) -> Optional[dict]:
             "type": sym_type,
             "section": section,
         }
+        if c_type:
+            result["c_type"] = c_type
         if is_pointer:
             result["is_pointer"] = True
             result["pointer_target"] = pointer_target

Tools/WebServer/static/js/features/symbols.js

@@ -171,10 +171,12 @@ function _renderSymbolValueContent(data, isConst) {
     : t('symbols.read_write', 'Read-Write');
   const isBss = data.section && data.section.startsWith('.bss');
 
-  // For pointer types, show the pointer type in the header
+  // Show C type in the header (from GDB whatis)
   const typeDisplay = data.is_pointer
     ? `${_escapeHtml(data.pointer_target || '?')} *`
-    : '';
+    : data.c_type
+      ? _escapeHtml(data.c_type)
+      : '';
 
   let headerHtml = `
     <div class="sym-viewer-header">
@@ -245,6 +247,32 @@ function _renderSymbolValueContent(data, isConst) {
     </div>`;
   }
 
+  // Scalar value decoded by backend (non-struct, non-pointer)
+  // Render as a single tree node to reuse the struct tree style
+  if (
+    data.decoded_value !== undefined &&
+    data.decoded_value !== null &&
+    !data.is_pointer &&
+    !(data.struct_layout && data.struct_layout.length > 0)
+  ) {
+    const scalarLayout = [
+      {
+        name: data.name,
+        offset: 0,
+        size: data.size,
+        type_name: data.c_type || '',
+      },
+    ];
+    const scalarGdbValues = { [data.name]: data.decoded_value };
+    bodyHtml += _renderStructTree(
+      scalarLayout,
+      data.hex_data,
+      isBss,
+      scalarGdbValues,
+      data.name,
+    );
+  }
+
   // Struct tree view (for non-pointer structs, or for deref data)
   if (data.struct_layout && data.struct_layout.length > 0) {
     bodyHtml += _renderStructTree(
@@ -667,11 +695,12 @@ function _decodeFieldValue(hexData, offset, size, typeName) {
     for (let i = 0; i < bytes.length; i++) {
       val += bytes[i] * 256 ** i;
     }
-    // Check for signed types
+    // Check for signed types (handle "unsigned long", "long unsigned int", etc.)
     if (
       !typeName.startsWith('u') &&
       !typeName.startsWith('U') &&
-      !typeName.includes('uint')
+      !typeName.includes('uint') &&
+      !typeName.includes('unsigned')
     ) {
       const maxSigned = 2 ** (size * 8 - 1);
       if (val >= maxSigned) val -= 2 ** (size * 8);