refactor check tools install

Author: Jason2866

platform.py

@@ -115,7 +115,7 @@ def install_tool(TOOL):
             if mcu == "esp32c2":
                 self.packages["framework-arduino-c2-skeleton-lib"]["optional"] = False
 
-        mcu_toolchain_mapping = {
+        MCU_TOOLCHAIN_MAPPING = {
             # Xtensa based and FSM toolchain
             ("esp32", "esp32s2", "esp32s3"): {
                 "toolchains": ["toolchain-xtensa-esp-elf"],
@@ -129,9 +129,8 @@ def install_tool(TOOL):
                 "debug_tools": ["tool-riscv32-esp-elf-gdb"]
             }
         }
-
         # Iterate through MCU mappings
-        for supported_mcus, toolchain_data in mcu_toolchain_mapping.items():
+        for supported_mcus, toolchain_data in MCU_TOOLCHAIN_MAPPING.items():
             if mcu in supported_mcus:
                 # Set mandatory toolchains
                 for toolchain in toolchain_data["toolchains"]:
@@ -160,10 +159,17 @@ def install_tool(TOOL):
             for package in COMMON_IDF_PACKAGES:
                 self.packages[package]["optional"] = False
 
-        # Enable check tools only when "check_tool" is active
-        for p in self.packages:
-            if p in ("tool-cppcheck", "tool-clangtidy", "tool-pvs-studio"):
-                self.packages[p]["optional"] = False if str(variables.get("check_tool")).strip("['']") in p else True
+        CHECK_PACKAGES = [
+            "tool-cppcheck",
+            "tool-clangtidy",
+            "tool-pvs-studio"
+        ]
+        # Install check tool listed in pio entry "check_tool"
+        if variables.get("check_tool") is not None:
+            for package in CHECK_PACKAGES:
+                for check_tool in variables.get("check_tool", ""):
+                    if check_tool in package:
+                        self.packages[package]["optional"] = False
 
         if "buildfs" in targets:
             filesystem = variables.get("board_build.filesystem", "littlefs")
@@ -226,7 +232,6 @@ def _add_dynamic_options(self, board):
         # A special case for the Kaluga board that has a separate interface config
         if board.id == "esp32-s2-kaluga-1":
             supported_debug_tools.append("ftdi")
-
         if board.get("build.mcu", "") in ("esp32c3", "esp32c5", "esp32c6", "esp32s3", "esp32h2", "esp32p4"):
             supported_debug_tools.append("esp-builtin")