Enable misc-include-cleaner
Change-Id: I22099a0f95b2eb884dec7d95cee0bdd0c159fd1f
Signed-off-by: Ed Tanous <ed@tanous.net>
diff --git a/.clang-tidy b/.clang-tidy
index f2d0cc7..7a92985 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -219,6 +219,7 @@
cppcoreguidelines-rvalue-reference-param-not-moved,
cppcoreguidelines-slicing,
google-explicit-constructor,
+misc-include-cleaner,
misc-misplaced-const,
misc-redundant-expression,
misc-static-assert,
@@ -304,7 +305,7 @@
readability-uppercase-literal-suffix'
WarningsAsErrors: '*'
-HeaderFilterRegex: '(?!^subprojects).*'
+HeaderFilterRegex: '.*'
CheckOptions:
- { key: readability-identifier-naming.ClassCase, value: CamelCase }
- { key: readability-identifier-naming.VariableCase, value: camelBack }
@@ -312,3 +313,4 @@
- { key: readability-identifier-naming.ParameterCase, value: camelBack }
- { key: readability-identifier-naming.NamespaceCase, value: lower_case }
- { key: readability-identifier-naming.StructCase, value: CamelCase }
+ - { key: misc-include-cleaner.IgnoreHeaders, value: ((stdlib.h)|(nlohmann/json_fwd.hpp)|(boost/.*/src.hpp)|(boost/.*/detail/.*)|(nlohmann/detail/.*)|(stdio.h)|(ranges)|(bits/chrono.h)|(boost/system/error_code.hpp)) }
diff --git a/src/ADCSensor.cpp b/src/ADCSensor.cpp
index d6dcdca..a025419 100644
--- a/src/ADCSensor.cpp
+++ b/src/ADCSensor.cpp
@@ -16,20 +16,29 @@
#include "ADCSensor.hpp"
-#include <unistd.h>
+#include "SensorPaths.hpp"
+#include "Thresholds.hpp"
+#include "Utils.hpp"
+#include "sensor.hpp"
+#include <fcntl.h>
+
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
#include <boost/asio/read_until.hpp>
+#include <boost/asio/streambuf.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <chrono>
#include <cmath>
-#include <filesystem>
-#include <fstream>
+#include <cstddef>
#include <iostream>
-#include <limits>
#include <memory>
#include <optional>
+#include <stdexcept>
#include <string>
+#include <utility>
#include <vector>
// scaling factor from hwmon
diff --git a/src/ADCSensorMain.cpp b/src/ADCSensorMain.cpp
index 21711a9..723ff01 100644
--- a/src/ADCSensorMain.cpp
+++ b/src/ADCSensorMain.cpp
@@ -15,23 +15,38 @@
*/
#include "ADCSensor.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
#include "VariantVisitors.hpp"
#include <boost/algorithm/string/case_conv.hpp>
-#include <boost/algorithm/string/replace.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/post.hpp>
+#include <boost/asio/steady_timer.hpp>
+#include <boost/container/flat_map.hpp>
#include <boost/container/flat_set.hpp>
+#include <gpiod.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <sdbusplus/bus.hpp>
#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/message.hpp>
+#include <sdbusplus/message/native_types.hpp>
+#include <array>
+#include <chrono>
+#include <cstddef>
#include <filesystem>
#include <fstream>
#include <functional>
+#include <iostream>
#include <memory>
#include <optional>
#include <regex>
+#include <stdexcept>
#include <string>
+#include <utility>
#include <variant>
#include <vector>
diff --git a/src/ChassisIntrusionSensor.cpp b/src/ChassisIntrusionSensor.cpp
index ee49e6e..52c254a 100644
--- a/src/ChassisIntrusionSensor.cpp
+++ b/src/ChassisIntrusionSensor.cpp
@@ -17,22 +17,30 @@
#include "ChassisIntrusionSensor.hpp"
#include <fcntl.h>
+#include <linux/i2c.h>
#include <sys/ioctl.h>
+#include <sys/syslog.h>
#include <systemd/sd-journal.h>
#include <unistd.h>
#include <Utils.hpp>
+#include <boost/asio/error.hpp>
#include <boost/asio/io_context.hpp>
+#include <boost/asio/posix/stream_descriptor.hpp>
+#include <gpiod.hpp>
#include <sdbusplus/asio/object_server.hpp>
-#include <cerrno>
#include <chrono>
+#include <cstddef>
+#include <cstdint>
+#include <filesystem>
#include <fstream>
#include <iostream>
#include <memory>
+#include <stdexcept>
#include <string>
-#include <thread>
#include <utility>
+#include <vector>
extern "C"
{
diff --git a/src/DeviceMgmt.cpp b/src/DeviceMgmt.cpp
index e8eb875..1987c73 100644
--- a/src/DeviceMgmt.cpp
+++ b/src/DeviceMgmt.cpp
@@ -1,7 +1,19 @@
#include "DeviceMgmt.hpp"
+#include "Utils.hpp"
+
+#include <cstdint>
#include <filesystem>
#include <fstream>
+#include <iomanip>
+#include <ios>
+#include <iostream>
+#include <optional>
+#include <sstream>
+#include <stdexcept>
+#include <string>
+#include <system_error>
+#include <variant>
namespace fs = std::filesystem;
diff --git a/src/ExitAirTempSensor.cpp b/src/ExitAirTempSensor.cpp
index f0c5c59..533c94d 100644
--- a/src/ExitAirTempSensor.cpp
+++ b/src/ExitAirTempSensor.cpp
@@ -16,24 +16,36 @@
#include "ExitAirTempSensor.hpp"
+#include "SensorPaths.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
#include "VariantVisitors.hpp"
+#include "sensor.hpp"
#include <boost/algorithm/string/replace.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/post.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <sdbusplus/bus.hpp>
#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/message.hpp>
+#include <algorithm>
#include <array>
#include <chrono>
#include <cmath>
+#include <cstddef>
+#include <cstdint>
#include <functional>
#include <iostream>
#include <limits>
#include <memory>
-#include <numeric>
#include <stdexcept>
+#include <string>
#include <utility>
#include <variant>
#include <vector>
diff --git a/src/ExternalSensor.cpp b/src/ExternalSensor.cpp
index 9a7a690..bce0a31 100644
--- a/src/ExternalSensor.cpp
+++ b/src/ExternalSensor.cpp
@@ -1,18 +1,22 @@
#include "ExternalSensor.hpp"
#include "SensorPaths.hpp"
-
-#include <unistd.h>
+#include "Thresholds.hpp"
+#include "Utils.hpp"
+#include "sensor.hpp"
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
#include <chrono>
+#include <cstddef>
+#include <functional>
#include <iostream>
-#include <istream>
#include <limits>
#include <memory>
+#include <stdexcept>
#include <string>
+#include <utility>
#include <vector>
static constexpr bool debug = false;
diff --git a/src/ExternalSensorMain.cpp b/src/ExternalSensorMain.cpp
index adaa8e3..3beb5b9 100644
--- a/src/ExternalSensorMain.cpp
+++ b/src/ExternalSensorMain.cpp
@@ -1,21 +1,27 @@
#include "ExternalSensor.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
#include "VariantVisitors.hpp"
-#include <boost/algorithm/string/replace.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/post.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
#include <boost/container/flat_set.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/message.hpp>
+#include <sdbusplus/message/native_types.hpp>
+#include <algorithm>
#include <array>
-#include <filesystem>
-#include <fstream>
+#include <chrono>
+#include <cmath>
#include <functional>
+#include <iostream>
#include <memory>
-#include <regex>
-#include <stdexcept>
#include <string>
#include <utility>
#include <variant>
diff --git a/src/FanMain.cpp b/src/FanMain.cpp
index 7ea8806..a7b8b72 100644
--- a/src/FanMain.cpp
+++ b/src/FanMain.cpp
@@ -16,24 +16,38 @@
#include "PwmSensor.hpp"
#include "TachSensor.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
#include "VariantVisitors.hpp"
#include <boost/algorithm/string/replace.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/post.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
#include <boost/container/flat_set.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <sdbusplus/bus.hpp>
#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/message.hpp>
#include <array>
+#include <chrono>
+#include <cstddef>
+#include <cstdint>
#include <filesystem>
#include <fstream>
#include <functional>
+#include <ios>
+#include <iostream>
+#include <map>
#include <memory>
#include <optional>
#include <regex>
#include <string>
+#include <system_error>
#include <utility>
#include <variant>
#include <vector>
diff --git a/src/FileHandle.cpp b/src/FileHandle.cpp
index d36911c..cb33e90 100644
--- a/src/FileHandle.cpp
+++ b/src/FileHandle.cpp
@@ -3,8 +3,10 @@
#include <fcntl.h>
#include <unistd.h>
+#include <filesystem>
#include <iostream>
#include <stdexcept>
+#include <string>
FileHandle::FileHandle(const std::filesystem::path& name,
std::ios_base::openmode mode) :
diff --git a/src/HwmonTempMain.cpp b/src/HwmonTempMain.cpp
index e76ef18..943443d 100644
--- a/src/HwmonTempMain.cpp
+++ b/src/HwmonTempMain.cpp
@@ -16,24 +16,37 @@
#include "DeviceMgmt.hpp"
#include "HwmonTempSensor.hpp"
+#include "SensorPaths.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
-#include <boost/algorithm/string/replace.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/post.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
#include <boost/container/flat_set.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <sdbusplus/bus.hpp>
#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/message.hpp>
+#include <sdbusplus/message/native_types.hpp>
+#include <algorithm>
#include <array>
-#include <charconv>
+#include <chrono>
+#include <cstddef>
+#include <cstdint>
#include <filesystem>
-#include <fstream>
#include <functional>
+#include <ios>
+#include <iostream>
#include <memory>
+#include <optional>
#include <regex>
-#include <stdexcept>
#include <string>
+#include <system_error>
#include <utility>
#include <variant>
#include <vector>
diff --git a/src/HwmonTempSensor.cpp b/src/HwmonTempSensor.cpp
index dd95c3b..0afacf8 100644
--- a/src/HwmonTempSensor.cpp
+++ b/src/HwmonTempSensor.cpp
@@ -16,18 +16,27 @@
#include "HwmonTempSensor.hpp"
-#include <unistd.h>
+#include "DeviceMgmt.hpp"
+#include "Thresholds.hpp"
+#include "Utils.hpp"
+#include "sensor.hpp"
-#include <boost/asio/read_until.hpp>
+#include <boost/algorithm/string/replace.hpp>
+#include <boost/asio/buffer.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/random_access_file.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
#include <charconv>
+#include <chrono>
+#include <cstddef>
#include <iostream>
-#include <istream>
#include <limits>
#include <memory>
#include <string>
+#include <system_error>
#include <utility>
#include <vector>
diff --git a/src/IntelCPUSensor.cpp b/src/IntelCPUSensor.cpp
index 5bf0fb6..e21f4b0 100644
--- a/src/IntelCPUSensor.cpp
+++ b/src/IntelCPUSensor.cpp
@@ -16,22 +16,34 @@
#include "IntelCPUSensor.hpp"
+#include "SensorPaths.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
+#include "sensor.hpp"
+#include <fcntl.h>
#include <unistd.h>
#include <boost/algorithm/string/replace.hpp>
-#include <boost/asio/read_until.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/posix/descriptor_base.hpp>
+#include <boost/container/flat_map.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <algorithm>
+#include <chrono>
#include <cstddef>
+#include <cstdint>
+#include <functional>
#include <iostream>
-#include <istream>
#include <limits>
#include <memory>
#include <stdexcept>
#include <string>
+#include <tuple>
+#include <utility>
#include <vector>
IntelCPUSensor::IntelCPUSensor(
diff --git a/src/IntelCPUSensorMain.cpp b/src/IntelCPUSensorMain.cpp
index 2976ade..643f49d 100644
--- a/src/IntelCPUSensorMain.cpp
+++ b/src/IntelCPUSensorMain.cpp
@@ -15,28 +15,41 @@
*/
#include "IntelCPUSensor.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
#include "VariantVisitors.hpp"
-#include <fcntl.h>
#include <peci.h>
#include <boost/algorithm/string/replace.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
#include <boost/container/flat_set.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/message.hpp>
+#include <algorithm>
#include <array>
+#include <cctype>
#include <cerrno>
+#include <chrono>
+#include <cstddef>
+#include <cstdint>
+#include <cstring>
#include <filesystem>
#include <fstream>
#include <functional>
+#include <ios>
+#include <iostream>
+#include <iterator>
#include <memory>
+#include <optional>
#include <regex>
#include <sstream>
-#include <stdexcept>
#include <string>
#include <utility>
#include <variant>
diff --git a/src/IntrusionSensorMain.cpp b/src/IntrusionSensorMain.cpp
index d59b2e6..a0cf1b7 100644
--- a/src/IntrusionSensorMain.cpp
+++ b/src/IntrusionSensorMain.cpp
@@ -17,28 +17,33 @@
#include "ChassisIntrusionSensor.hpp"
#include "Utils.hpp"
+#include <boost/asio/error.hpp>
#include <boost/asio/io_context.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
#include <phosphor-logging/lg2.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
#include <sdbusplus/bus.hpp>
#include <sdbusplus/bus/match.hpp>
-#include <sdbusplus/exception.hpp>
-#include <sdbusplus/server.hpp>
-#include <sdbusplus/timer.hpp>
+#include <sdbusplus/message.hpp>
#include <array>
#include <charconv>
#include <chrono>
+#include <cstdint>
#include <ctime>
+#include <exception>
+#include <filesystem>
#include <fstream>
#include <functional>
#include <iostream>
+#include <map>
#include <memory>
-#include <stdexcept>
#include <string>
+#include <system_error>
#include <utility>
+#include <variant>
#include <vector>
static constexpr bool debug = false;
diff --git a/src/IpmbSDRSensor.cpp b/src/IpmbSDRSensor.cpp
index 72a2cdb..299ff80 100644
--- a/src/IpmbSDRSensor.cpp
+++ b/src/IpmbSDRSensor.cpp
@@ -1,5 +1,17 @@
#include "IpmbSDRSensor.hpp"
+#include <sdbusplus/asio/connection.hpp>
+
+#include <cmath>
+#include <cstddef>
+#include <cstdint>
+#include <functional>
+#include <iostream>
+#include <memory>
+#include <string>
+#include <utility>
+#include <vector>
+
const constexpr char* ipmbService = "xyz.openbmc_project.Ipmi.Channel.Ipmb";
const constexpr char* ipmbDbusPath = "/xyz/openbmc_project/Ipmi/Channel/Ipmb";
const constexpr char* ipmbInterface = "org.openbmc.Ipmb";
diff --git a/src/IpmbSensor.cpp b/src/IpmbSensor.cpp
index 519a05b..5b4649b 100644
--- a/src/IpmbSensor.cpp
+++ b/src/IpmbSensor.cpp
@@ -17,23 +17,37 @@
#include "IpmbSensor.hpp"
#include "IpmbSDRSensor.hpp"
+#include "SensorPaths.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
#include "VariantVisitors.hpp"
+#include "sensor.hpp"
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/post.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <sdbusplus/bus.hpp>
#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/message.hpp>
+#include <sdbusplus/message/native_types.hpp>
+#include <algorithm>
+#include <array>
#include <chrono>
-#include <cmath>
+#include <cstddef>
+#include <cstdint>
#include <functional>
#include <iostream>
#include <limits>
#include <memory>
-#include <numeric>
+#include <stdexcept>
#include <string>
#include <tuple>
+#include <utility>
#include <variant>
#include <vector>
diff --git a/src/MCUTempSensor.cpp b/src/MCUTempSensor.cpp
index fef23e9..48d2300 100644
--- a/src/MCUTempSensor.cpp
+++ b/src/MCUTempSensor.cpp
@@ -16,22 +16,35 @@
#include "MCUTempSensor.hpp"
+#include "SensorPaths.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
-#include "VariantVisitors.hpp"
+#include "sensor.hpp"
+#include <fcntl.h>
+#include <linux/i2c.h>
+#include <sys/ioctl.h>
+#include <unistd.h>
+
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/post.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/message.hpp>
+#include <array>
#include <chrono>
-#include <cmath>
+#include <cstddef>
+#include <cstdint>
#include <functional>
#include <iostream>
-#include <limits>
#include <memory>
-#include <numeric>
#include <string>
+#include <utility>
#include <vector>
extern "C"
diff --git a/src/NVMeBasicContext.cpp b/src/NVMeBasicContext.cpp
index 527605c..eb5ecd9 100644
--- a/src/NVMeBasicContext.cpp
+++ b/src/NVMeBasicContext.cpp
@@ -1,20 +1,39 @@
#include "NVMeBasicContext.hpp"
+#include "NVMeContext.hpp"
+#include "NVMeSensor.hpp"
+
#include <endian.h>
#include <sys/ioctl.h>
#include <unistd.h>
#include <FileHandle.hpp>
+#include <boost/asio/buffer.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
#include <boost/asio/read.hpp>
#include <boost/asio/streambuf.hpp>
#include <boost/asio/write.hpp>
-#include <cassert>
+#include <array>
#include <cerrno>
-#include <cinttypes>
+#include <chrono>
+#include <cmath>
+#include <cstdint>
#include <cstdio>
#include <cstring>
+#include <filesystem>
+#include <ios>
+#include <iostream>
+#include <iterator>
+#include <limits>
+#include <memory>
+#include <stdexcept>
+#include <string>
#include <system_error>
+#include <thread>
+#include <utility>
+#include <vector>
extern "C"
{
diff --git a/src/NVMeSensor.cpp b/src/NVMeSensor.cpp
index 9c1b631..4c372d6 100644
--- a/src/NVMeSensor.cpp
+++ b/src/NVMeSensor.cpp
@@ -16,7 +16,22 @@
#include "NVMeSensor.hpp"
-#include <iostream>
+#include "SensorPaths.hpp"
+#include "Thresholds.hpp"
+#include "Utils.hpp"
+#include "sensor.hpp"
+
+#include <boost/asio/io_context.hpp>
+#include <sdbusplus/asio/connection.hpp>
+#include <sdbusplus/asio/object_server.hpp>
+
+#include <cstddef>
+#include <cstdint>
+#include <memory>
+#include <stdexcept>
+#include <string>
+#include <utility>
+#include <vector>
static constexpr double maxReading = 127;
static constexpr double minReading = 0;
diff --git a/src/NVMeSensorMain.cpp b/src/NVMeSensorMain.cpp
index 81be2eb..5f0222d 100644
--- a/src/NVMeSensorMain.cpp
+++ b/src/NVMeSensorMain.cpp
@@ -17,11 +17,36 @@
#include "NVMeBasicContext.hpp"
#include "NVMeContext.hpp"
#include "NVMeSensor.hpp"
+#include "Thresholds.hpp"
+#include "Utils.hpp"
+#include "VariantVisitors.hpp"
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/post.hpp>
#include <boost/asio/steady_timer.hpp>
+#include <sdbusplus/asio/connection.hpp>
+#include <sdbusplus/asio/object_server.hpp>
+#include <sdbusplus/bus.hpp>
+#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/message.hpp>
+#include <sdbusplus/message/native_types.hpp>
+#include <algorithm>
+#include <array>
+#include <chrono>
+#include <cstddef>
+#include <cstdint>
+#include <filesystem>
+#include <functional>
+#include <iostream>
+#include <memory>
#include <optional>
-#include <regex>
+#include <stdexcept>
+#include <string>
+#include <utility>
+#include <variant>
+#include <vector>
static constexpr uint8_t nvmeMiDefaultSlaveAddr = 0x6A;
diff --git a/src/PSUEvent.cpp b/src/PSUEvent.cpp
index 1201a19..8a83f37 100644
--- a/src/PSUEvent.cpp
+++ b/src/PSUEvent.cpp
@@ -17,20 +17,26 @@
#include "PSUEvent.hpp"
#include "SensorPaths.hpp"
+#include "Utils.hpp"
+#include <boost/asio/buffer.hpp>
+#include <boost/asio/error.hpp>
#include <boost/asio/io_context.hpp>
-#include <boost/asio/read_until.hpp>
+#include <boost/asio/random_access_file.hpp>
#include <boost/container/flat_map.hpp>
#include <phosphor-logging/lg2.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <array>
+#include <chrono>
+#include <cstddef>
#include <iostream>
#include <memory>
+#include <set>
#include <stdexcept>
#include <string>
#include <utility>
-#include <variant>
#include <vector>
PSUCombineEvent::PSUCombineEvent(
diff --git a/src/PSUSensor.cpp b/src/PSUSensor.cpp
index 96f849a..db632b6 100644
--- a/src/PSUSensor.cpp
+++ b/src/PSUSensor.cpp
@@ -16,19 +16,28 @@
#include "PSUSensor.hpp"
-#include <unistd.h>
+#include "DeviceMgmt.hpp"
+#include "SensorPaths.hpp"
+#include "Thresholds.hpp"
+#include "Utils.hpp"
+#include "sensor.hpp"
+#include <boost/asio/buffer.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
#include <boost/asio/random_access_file.hpp>
-#include <boost/asio/read_until.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <array>
+#include <chrono>
+#include <cstddef>
#include <iostream>
-#include <istream>
#include <limits>
#include <memory>
+#include <stdexcept>
#include <string>
-#include <system_error>
+#include <utility>
#include <vector>
static constexpr const char* sensorPathPrefix = "/xyz/openbmc_project/sensors/";
diff --git a/src/PSUSensorMain.cpp b/src/PSUSensorMain.cpp
index 93d280f..2c8d7c0 100644
--- a/src/PSUSensorMain.cpp
+++ b/src/PSUSensorMain.cpp
@@ -17,23 +17,43 @@
#include "DeviceMgmt.hpp"
#include "PSUEvent.hpp"
#include "PSUSensor.hpp"
+#include "PwmSensor.hpp"
+#include "SensorPaths.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
+#include "VariantVisitors.hpp"
#include <boost/algorithm/string/case_conv.hpp>
#include <boost/algorithm/string/replace.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/post.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
#include <boost/container/flat_set.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <sdbusplus/bus.hpp>
#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/exception.hpp>
+#include <sdbusplus/message.hpp>
+#include <sdbusplus/message/native_types.hpp>
+#include <algorithm>
#include <array>
+#include <cctype>
+#include <chrono>
#include <cmath>
+#include <cstddef>
+#include <cstdint>
+#include <exception>
#include <filesystem>
#include <fstream>
#include <functional>
#include <iostream>
+#include <memory>
#include <regex>
+#include <stdexcept>
#include <string>
#include <string_view>
#include <utility>
diff --git a/src/PwmSensor.cpp b/src/PwmSensor.cpp
index 796e51d..2a431eb 100644
--- a/src/PwmSensor.cpp
+++ b/src/PwmSensor.cpp
@@ -16,12 +16,18 @@
#include "PwmSensor.hpp"
+#include "SensorPaths.hpp"
#include "Utils.hpp"
+#include "sensor.hpp"
+#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <cmath>
+#include <cstdint>
#include <fstream>
#include <iostream>
+#include <memory>
#include <stdexcept>
#include <string>
diff --git a/src/SensorPaths.cpp b/src/SensorPaths.cpp
index bd8f173..e5ed60f 100644
--- a/src/SensorPaths.cpp
+++ b/src/SensorPaths.cpp
@@ -1,6 +1,5 @@
#include "SensorPaths.hpp"
-#include <cstring>
#include <regex>
#include <string>
diff --git a/src/TachSensor.cpp b/src/TachSensor.cpp
index 0a7048a..5019fd5 100644
--- a/src/TachSensor.cpp
+++ b/src/TachSensor.cpp
@@ -16,24 +16,29 @@
#include "TachSensor.hpp"
+#include "SensorPaths.hpp"
+#include "Thresholds.hpp"
#include "Utils.hpp"
+#include "sensor.hpp"
-#include <unistd.h>
-
-#include <boost/asio/read_until.hpp>
+#include <boost/asio/buffer.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/io_context.hpp>
+#include <boost/asio/posix/stream_descriptor.hpp>
+#include <boost/asio/random_access_file.hpp>
#include <gpiod.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
#include <charconv>
-#include <fstream>
+#include <chrono>
+#include <cstddef>
+#include <cstdint>
#include <iostream>
-#include <istream>
-#include <limits>
#include <memory>
#include <optional>
-#include <stdexcept>
#include <string>
+#include <system_error>
#include <utility>
#include <vector>
diff --git a/src/Thresholds.cpp b/src/Thresholds.cpp
index 7a64d1e..42c634f 100644
--- a/src/Thresholds.cpp
+++ b/src/Thresholds.cpp
@@ -1,17 +1,25 @@
#include "Thresholds.hpp"
+#include "Utils.hpp"
#include "VariantVisitors.hpp"
#include "sensor.hpp"
#include <boost/algorithm/string/replace.hpp>
+#include <boost/asio/error.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
+#include <sdbusplus/asio/connection.hpp>
+#include <sdbusplus/asio/object_server.hpp>
+#include <sdbusplus/exception.hpp>
+#include <sdbusplus/message.hpp>
#include <array>
-#include <cmath>
-#include <fstream>
+#include <chrono>
+#include <cstddef>
+#include <cstdint>
#include <iostream>
+#include <limits>
#include <memory>
-#include <stdexcept>
#include <string>
#include <tuple>
#include <utility>
diff --git a/src/Utils.cpp b/src/Utils.cpp
index 213f790..e3cbfeb 100644
--- a/src/Utils.cpp
+++ b/src/Utils.cpp
@@ -19,19 +19,39 @@
#include "dbus-sensor_config.h"
#include "DeviceMgmt.hpp"
+#include "VariantVisitors.hpp"
+#include <boost/asio/error.hpp>
+#include <boost/asio/steady_timer.hpp>
#include <boost/container/flat_map.hpp>
#include <sdbusplus/asio/connection.hpp>
#include <sdbusplus/asio/object_server.hpp>
+#include <sdbusplus/bus.hpp>
#include <sdbusplus/bus/match.hpp>
+#include <sdbusplus/exception.hpp>
+#include <sdbusplus/message.hpp>
+#include <sdbusplus/message/native_types.hpp>
-#include <charconv>
+#include <algorithm>
+#include <array>
+#include <chrono>
+#include <cstddef>
+#include <cstring>
#include <filesystem>
#include <fstream>
+#include <functional>
+#include <iostream>
+#include <iterator>
#include <memory>
+#include <optional>
#include <regex>
+#include <set>
+#include <span>
#include <stdexcept>
#include <string>
+#include <string_view>
+#include <system_error>
+#include <tuple>
#include <utility>
#include <variant>
#include <vector>
diff --git a/tests/test_Utils.cpp b/tests/test_Utils.cpp
index 59a4e13..b46db7b 100644
--- a/tests/test_Utils.cpp
+++ b/tests/test_Utils.cpp
@@ -1,8 +1,15 @@
#include "Utils.hpp"
#include <array>
+#include <cstddef>
+#include <cstdint>
+#include <cstdlib>
#include <filesystem>
#include <fstream>
+#include <iostream>
+#include <new>
+#include <string>
+#include <vector>
#include <gtest/gtest.h>