Added new check and fixed files with it

.clang-tidy

@@ -3,6 +3,7 @@ Checks: "
   readability-*,
   modernize-*,
   performance-*,
+  misc-include-cleaner,
   -google-readability-todo,
   -readability-convert-member-functions-to-static,
   -readability-function-cognitive-complexity,

providers/flagd/src/configuration.cpp

@@ -1,18 +1,25 @@
 #include "configuration.h"
 
-#include <grpcpp/grpcpp.h>
 #include <grpcpp/security/credentials.h>
+#include <grpcpp/support/status.h>
 
 #include <algorithm>
+#include <cctype>
 #include <cstdlib>
 #include <fstream>
 #include <map>
+#include <memory>
+#include <nlohmann/json_fwd.hpp>
+#include <optional>
 #include <sstream>
+#include <stdexcept>
 #include <string>
+#include <string_view>
 #include <utility>
 #include <vector>
 
 #include "absl/log/log.h"
+#include "absl/status/status.h"
 #include "absl/status/statusor.h"
 #include "absl/strings/ascii.h"
 #include "absl/strings/str_cat.h"

providers/flagd/src/evaluator/evaluator.cpp

@@ -1,14 +1,29 @@
 #include "evaluator.h"
 
+#include <any>
 #include <chrono>
+#include <cstdint>
+#include <map>
 #include <memory>
 #include <nlohmann/json.hpp>
+#include <nlohmann/json_fwd.hpp>
 #include <optional>
+#include <string>
+#include <string_view>
+#include <utility>
+#include <vector>
 
 #include "absl/log/log.h"
+#include "absl/status/statusor.h"
 #include "absl/strings/str_cat.h"
 #include "flagd/sync/sync.h"
 #include "flagd_ops.h"
+#include "openfeature/error_code.h"
+#include "openfeature/evaluation_context.h"
+#include "openfeature/flag_metadata.h"
+#include "openfeature/reason.h"
+#include "openfeature/resolution_details.h"
+#include "openfeature/value.h"
 
 namespace flagd {
 

providers/flagd/src/evaluator/flagd_ops.cpp

@@ -1,17 +1,23 @@
 #include "flagd_ops.h"
 
 #include <algorithm>
+#include <cctype>
+#include <cstddef>
 #include <cstdint>
 #include <limits>
+#include <nlohmann/json_fwd.hpp>
 #include <string>
 #include <string_view>
+#include <utility>
 #include <vector>
 
 #include "absl/status/status.h"
+#include "absl/status/statusor.h"
 #include "absl/strings/match.h"
 #include "absl/strings/numbers.h"
 #include "absl/strings/str_cat.h"
 #include "absl/strings/str_split.h"
+#include "flagd/evaluator/json_logic/json_logic.h"
 #include "flagd/evaluator/murmur_hash/MurmurHash3.h"
 
 namespace flagd {

providers/flagd/src/evaluator/json_logic/array.cpp

@@ -1,6 +1,11 @@
 
 #include "array.h"
 
+#include <nlohmann/json_fwd.hpp>
+
+#include "absl/status/statusor.h"
+#include "providers/flagd/src/evaluator/json_logic/json_logic.h"
+
 namespace json_logic::ops {
 
 absl::StatusOr<nlohmann::json> Merge(const JsonLogic& eval,

providers/flagd/src/evaluator/json_logic/cli.cpp

@@ -1,8 +1,10 @@
 #include <exception>
 #include <iostream>
+#include <nlohmann/json_fwd.hpp>
 #include <string>
 
 #include "absl/status/status.h"
+#include "absl/status/statusor.h"
 #include "json_logic.h"
 #include "nlohmann/json.hpp"
 

providers/flagd/src/evaluator/json_logic/data.cpp

@@ -1,10 +1,13 @@
 #include "data.h"
 
 #include <algorithm>
+#include <cstdint>
+#include <nlohmann/json_fwd.hpp>
 #include <optional>
 #include <string>
 
 #include "absl/status/status.h"
+#include "absl/status/statusor.h"
 #include "absl/strings/str_cat.h"
 #include "json_logic.h"
 

providers/flagd/src/evaluator/json_logic/json_logic.cpp

@@ -1,9 +1,11 @@
 #include "json_logic.h"
 
+#include <nlohmann/json_fwd.hpp>
 #include <string>
 #include <utility>
 
 #include "absl/status/status.h"
+#include "absl/status/statusor.h"
 #include "array.h"
 #include "data.h"
 #include "logic.h"

providers/flagd/src/evaluator/json_logic/logic.cpp

@@ -1,6 +1,11 @@
 #include "logic.h"
 
+#include <cstddef>
+#include <nlohmann/json_fwd.hpp>
+
 #include "absl/status/status.h"
+#include "absl/status/statusor.h"
+#include "providers/flagd/src/evaluator/json_logic/json_logic.h"
 #include "providers/flagd/src/evaluator/json_logic/utils.h"
 
 namespace json_logic::ops {

providers/flagd/src/evaluator/json_logic/numeric.cpp

@@ -3,12 +3,18 @@
 #include <algorithm>
 #include <cmath>
 #include <cstdint>
+#include <limits>
+#include <nlohmann/json_fwd.hpp>
 #include <numeric>
+#include <string>
+#include <string_view>
 #include <type_traits>
 #include <variant>
 #include <vector>
 
 #include "absl/status/status.h"
+#include "absl/status/statusor.h"
+#include "providers/flagd/src/evaluator/json_logic/json_logic.h"
 
 namespace json_logic::ops {
 
@@ -258,8 +264,8 @@ absl::StatusOr<nlohmann::json> Add(const JsonLogic& eval,
   if (!nums_res.ok()) return nums_res.status();
   if (nums_res->empty()) return 0;
 
-  return std::accumulate(nums_res->begin() + 1, nums_res->end(), (*nums_res)[0],
-                         Number::Add)
+  return std::accumulate(nums_res->begin() + 1, nums_res->end(),
+                         nums_res.value()[0], Number::Add)
       .ToJson();
 }
 
@@ -271,10 +277,10 @@ absl::StatusOr<nlohmann::json> Subtract(const JsonLogic& eval,
   if (!nums_res.ok()) return nums_res.status();
 
   if (nums_res->size() == 1) {
-    return Number::Sub(Number(int64_t{0}), (*nums_res)[0]).ToJson();
+    return Number::Sub(Number(int64_t{0}), nums_res.value()[0]).ToJson();
   }
   if (nums_res->size() == 2) {
-    return Number::Sub((*nums_res)[0], (*nums_res)[1]).ToJson();
+    return Number::Sub(nums_res.value()[0], nums_res.value()[1]).ToJson();
   }
   return absl::InvalidArgumentError(
       "Subtract requires exactly one or two arguments");
@@ -288,8 +294,8 @@ absl::StatusOr<nlohmann::json> Multiply(const JsonLogic& eval,
   if (!nums_res.ok()) return nums_res.status();
   if (nums_res->empty()) return 1;
 
-  return std::accumulate(nums_res->begin() + 1, nums_res->end(), (*nums_res)[0],
-                         Number::Mul)
+  return std::accumulate(nums_res->begin() + 1, nums_res->end(),
+                         nums_res.value()[0], Number::Mul)
       .ToJson();
 }
 
@@ -304,7 +310,7 @@ absl::StatusOr<nlohmann::json> Divide(const JsonLogic& eval,
   }
 
   absl::StatusOr<Number> result_res =
-      Number::Div((*nums_res)[0], (*nums_res)[1]);
+      Number::Div(nums_res.value()[0], nums_res.value()[1]);
   if (!result_res.ok()) return result_res.status();
   return result_res->ToJson();
 }
@@ -320,7 +326,7 @@ absl::StatusOr<nlohmann::json> Modulo(const JsonLogic& eval,
   }
 
   absl::StatusOr<Number> result_res =
-      Number::Mod((*nums_res)[0], (*nums_res)[1]);
+      Number::Mod(nums_res.value()[0], nums_res.value()[1]);
   if (!result_res.ok()) return result_res.status();
   return result_res->ToJson();
 }
@@ -332,9 +338,10 @@ absl::StatusOr<nlohmann::json> LessThan(const JsonLogic& eval,
       GetNumbers(eval, values, data, "<");
   if (!nums_res.ok()) return nums_res.status();
 
-  if (nums_res->size() == 2) return (*nums_res)[0] < (*nums_res)[1];
+  if (nums_res->size() == 2) return nums_res.value()[0] < nums_res.value()[1];
   if (nums_res->size() == 3) {
-    return (*nums_res)[0] < (*nums_res)[1] && (*nums_res)[1] < (*nums_res)[2];
+    return nums_res.value()[0] < nums_res.value()[1] &&
+           nums_res.value()[1] < nums_res.value()[2];
   }
   return absl::InvalidArgumentError("Comparison requires 2 or 3 arguments");
 }
@@ -349,10 +356,12 @@ absl::StatusOr<nlohmann::json> LessThanOrEqual(const JsonLogic& eval,
   auto is_lte = [](const Number& first, const Number& second) {
     return first < second || first == second;
   };
-  if (nums_res->size() == 2) return is_lte((*nums_res)[0], (*nums_res)[1]);
+  if (nums_res->size() == 2) {
+    return is_lte(nums_res.value()[0], nums_res.value()[1]);
+  }
   if (nums_res->size() == 3) {
-    return is_lte((*nums_res)[0], (*nums_res)[1]) &&
-           is_lte((*nums_res)[1], (*nums_res)[2]);
+    return is_lte(nums_res.value()[0], nums_res.value()[1]) &&
+           is_lte(nums_res.value()[1], nums_res.value()[2]);
   }
   return absl::InvalidArgumentError("Comparison requires 2 or 3 arguments");
 }
@@ -364,7 +373,7 @@ absl::StatusOr<nlohmann::json> GreaterThan(const JsonLogic& eval,
       GetNumbers(eval, values, data, ">");
   if (!nums_res.ok()) return nums_res.status();
 
-  if (nums_res->size() == 2) return (*nums_res)[1] < (*nums_res)[0];
+  if (nums_res->size() == 2) return nums_res.value()[1] < nums_res.value()[0];
   return absl::InvalidArgumentError("GreaterThan requires 2 arguments");
 }
 
@@ -376,7 +385,8 @@ absl::StatusOr<nlohmann::json> GreaterThanOrEqual(const JsonLogic& eval,
   if (!nums_res.ok()) return nums_res.status();
 
   if (nums_res->size() == 2) {
-    return (*nums_res)[1] < (*nums_res)[0] || (*nums_res)[0] == (*nums_res)[1];
+    return nums_res.value()[1] < nums_res.value()[0] ||
+           nums_res.value()[0] == nums_res.value()[1];
   }
   return absl::InvalidArgumentError("GreaterThanOrEqual requires 2 arguments");
 }

providers/flagd/src/evaluator/json_logic/string_ops.cpp

@@ -2,12 +2,17 @@
 #include "string_ops.h"
 
 #include <algorithm>
+#include <cstddef>
 #include <cstdint>
+#include <nlohmann/json_fwd.hpp>
 #include <string>
+#include <string_view>
 #include <vector>
 
 #include "absl/status/status.h"
+#include "absl/status/statusor.h"
 #include "absl/strings/str_join.h"
+#include "providers/flagd/src/evaluator/json_logic/json_logic.h"
 
 namespace json_logic::ops {
 

providers/flagd/src/evaluator/json_logic/utils.cpp

@@ -1,5 +1,7 @@
 #include "utils.h"
 
+#include <cstdint>
+#include <nlohmann/json_fwd.hpp>
 #include <string>
 
 namespace json_logic::ops {

providers/flagd/src/provider.cpp

@@ -1,13 +1,24 @@
 #include "provider.h"
 
-#include <openfeature/provider.h>
-
+#include <cstdint>
+#include <memory>
 #include <optional>
+#include <string>
+#include <string_view>
+#include <utility>
 
 #include "absl/log/log.h"
 #include "absl/status/status.h"
 #include "flagd/configuration.h"
+#include "flagd/evaluator/evaluator.h"
 #include "flagd/sync/grpc/grpc_sync.h"
+#include "flagd/sync/sync.h"
+#include "openfeature/error_code.h"
+#include "openfeature/evaluation_context.h"
+#include "openfeature/metadata.h"
+#include "openfeature/reason.h"
+#include "openfeature/resolution_details.h"
+#include "openfeature/value.h"
 
 namespace flagd {
 

providers/flagd/src/sync/grpc/grpc_sync.cpp

@@ -1,19 +1,32 @@
 #include "grpc_sync.h"
 
+#include <grpc/grpc.h>
 #include <grpcpp/grpcpp.h>
+#include <grpcpp/security/credentials.h>
+#include <grpcpp/support/channel_arguments.h>
+#include <grpcpp/support/status.h>
 
 #include <algorithm>
 #include <chrono>
+#include <cstdint>
+#include <exception>
 #include <memory>
 #include <mutex>
 #include <nlohmann/json.hpp>
+#include <nlohmann/json_fwd.hpp>
+#include <string>
 #include <thread>
+#include <utility>
+#include <vector>
 
 #include "absl/log/log.h"
 #include "absl/status/status.h"
+#include "absl/status/statusor.h"
 #include "absl/strings/str_cat.h"
 #include "flagd/configuration.h"
 #include "flagd/sync/v1/sync.grpc.pb.h"
+#include "flagd/sync/v1/sync.pb.h"
+#include "openfeature/evaluation_context.h"
 
 using ::flagd::sync::v1::FlagSyncService;
 using ::flagd::sync::v1::SyncFlagsRequest;

providers/flagd/src/sync/sync.cpp

@@ -1,11 +1,16 @@
 #include "sync.h"
 
+#include <exception>
 #include <memory>
 #include <mutex>
 #include <nlohmann/json-schema.hpp>
-#include <nlohmann/json.hpp>
+#include <nlohmann/json_fwd.hpp>
+#include <shared_mutex>
+#include <string>
+#include <utility>
 
 #include "absl/log/log.h"
+#include "absl/status/status.h"
 #include "absl/strings/str_cat.h"
 #include "embedded_schemas.h"
 

providers/flagd/tests/evaluator/flagd_ops_test.cpp

@@ -3,6 +3,7 @@
 #include <gtest/gtest.h>
 
 #include <nlohmann/json.hpp>
+#include <nlohmann/json_fwd.hpp>
 
 #include "providers/flagd/src/evaluator/json_logic/json_logic.h"
 

providers/flagd/tests/evaluator/json_logic/json_logic_annotated_test.cpp

@@ -3,8 +3,11 @@
 #include <fstream>
 #include <iostream>
 #include <nlohmann/json.hpp>
+#include <nlohmann/json_fwd.hpp>
+#include <string>
 #include <vector>
 
+#include "absl/status/statusor.h"
 #include "providers/flagd/src/evaluator/json_logic/json_logic.h"
 
 using json_logic::JsonLogic;