src: fix extra includes of "env.h" and "env-inl.h"

Due to how the Environment class is used throughout the codebase, there
are a log of includes referencing eitehr env.h or env-inl.h.

This commit cleans up the remaining extra includes of 'env.h' or
'env-inl.h' and adds forward declarations of the Environment class.

PR-URL: #32293
Refs: #27531

src/node_crypto_common.h

@@ -3,7 +3,6 @@
 
 #if defined(NODE_WANT_INTERNALS) && NODE_WANT_INTERNALS
 
-#include "env.h"
 #include "node_crypto.h"
 #include "v8.h"
 #include <openssl/ssl.h>
@@ -13,6 +12,9 @@
 #include <unordered_map>
 
 namespace node {
+
+class Environment;
+
 namespace crypto {
 
 // OPENSSL_free is a macro, so we need a wrapper function.

src/node_http_common-inl.h

@@ -4,7 +4,6 @@
 #include "node_http_common.h"
 #include "node.h"
 #include "node_mem-inl.h"
-#include "env-inl.h"
 #include "v8.h"
 
 #include <algorithm>

src/node_sockaddr.h

@@ -3,7 +3,6 @@
 
 #if defined(NODE_WANT_INTERNALS) && NODE_WANT_INTERNALS
 
-#include "env.h"
 #include "memory_tracker.h"
 #include "node.h"
 #include "uv.h"
@@ -14,6 +13,8 @@
 
 namespace node {
 
+class Environment;
+
 class SocketAddress : public MemoryRetainer {
  public:
   struct Hash {

src/stream_base-inl.h

@@ -6,7 +6,6 @@
 #include "stream_base.h"
 
 #include "node.h"
-#include "env-inl.h"
 #include "v8.h"
 
 namespace node {