style(iwyu): set up include cleaner of clang-tidy

This PR enables the `misc-include-cleaner` of `clang-tidy`, which
enforces a version of include-what-you-use (IWYU), and applies all
suggested fixes.

Signed-off-by: Ingo Müller <ingomueller@google.com>

Author: ingomueller-net

.clang-tidy

@@ -50,6 +50,7 @@ Checks: >
   bugprone-*,
   google-*,
   modernize-*,
+  misc-include-cleaner,
   performance-*,
   portability-*,
   readability-*,
@@ -108,3 +109,5 @@ CheckOptions:
   - { key: readability-implicit-bool-conversion.AllowIntegerConditions,            value: 1         }
   - { key: readability-implicit-bool-conversion.AllowPointerConditions,            value: 1         }
   - { key: readability-function-cognitive-complexity.IgnoreMacros,                 value: 1         }
+  - { key: misc-include-cleaner.IgnoreHeaders
+      value: bits/getopt_.*;getopt.h }

export/planloader/planloader.cpp

@@ -2,7 +2,12 @@
 
 #include "planloader.h"
 
+#include <substrait/proto/plan.pb.h>
+
+#include <cstdint>
+#include <cstring>
 #include <limits>
+#include <string>
 
 #include "substrait/common/Io.h"
 

export/planloader/tests/PlanLoaderTest.cpp

@@ -6,7 +6,7 @@
 #include <gtest/gtest.h>
 #include <substrait/proto/plan.pb.h>
 
-#include <functional>
+#include "substrait/common/Io.h"
 
 namespace io::substrait::textplan {
 namespace {

src/substrait/common/Exceptions.cpp

@@ -2,7 +2,10 @@
 
 #include "substrait/common/Exceptions.h"
 
-#include <fmt/format.h>
+#include <fmt/core.h>
+
+#include <cstddef>
+#include <string>
 
 namespace io::substrait::common {
 

src/substrait/common/Io.cpp

@@ -2,6 +2,8 @@
 
 #include "substrait/common/Io.h"
 
+#include <absl/status/status.h>
+#include <absl/status/statusor.h>
 #include <substrait/proto/plan.pb.h>
 
 #include <regex>

src/substrait/common/PlanTransformerTool.cpp

@@ -1,6 +1,12 @@
 /* SPDX-License-Identifier: Apache-2.0 */
 
+#include <algorithm>
+#include <cctype>
+#include <cstdio>
+#include <cstdlib>
 #include <iostream>
+#include <string>
+#include <string_view>
 
 #include "substrait/common/Io.h"
 

src/substrait/common/tests/IoTest.cpp

@@ -5,12 +5,11 @@
 #include <gmock/gmock-matchers.h>
 #include <gtest/gtest.h>
 #include <protobuf-matchers/protocol-buffer-matchers.h>
+#include <substrait/proto/plan.pb.h>
 
 #include <filesystem>
-
-#ifndef _WIN32
-#include <unistd.h>
-#endif
+#include <string>
+#include <system_error>
 
 using ::protobuf_matchers::EqualsProto;
 using ::protobuf_matchers::Partially;

src/substrait/expression/DecimalLiteral.cpp

@@ -6,7 +6,11 @@
 #include <absl/strings/numbers.h>
 #include <substrait/proto/algebra.pb.h>
 
+#include <cstddef>
+#include <cstdint>
+#include <cstring>
 #include <sstream>
+#include <string>
 
 namespace io::substrait::expression {
 

src/substrait/expression/tests/DecimalTest.cpp

@@ -4,6 +4,11 @@
 #include <gtest/gtest.h>
 #include <substrait/proto/algebra.pb.h>
 
+#include <algorithm>
+#include <cstdint>
+#include <string>
+#include <vector>
+
 #include "substrait/expression/DecimalLiteral.h"
 
 namespace io::substrait::expression {

src/substrait/function/Extension.cpp

@@ -4,6 +4,16 @@
 
 #include <yaml-cpp/yaml.h>
 
+#include <memory>
+#include <optional>
+#include <sstream>
+#include <string>
+#include <utility>
+#include <vector>
+
+#include "substrait/function/Function.h"
+#include "substrait/type/Type.h"
+
 bool decodeFunctionImpl(
     const YAML::Node& node,
     io::substrait::FunctionImplementation& function) {