refactor(libdd-common)!: preparatory cleanup for restructure

.claude/skills/release-proposal-pr-review-bumps/SKILL.md

@@ -25,8 +25,8 @@ Bump rules (per crate, based on the highest-severity change):
 > **`pub` is not the reachability test.** Classify a rename or removal by whether the item is
 > *externally reachable*, never by the presence of the `pub` token:
 > - A `#[macro_export] macro_rules!` item carries no `pub` yet **is** public API — e.g.
->   `libdd-common`'s `tag!` (`src/tag.rs`), `cstr!` and `cstr_u8!` (`src/cstr.rs`). Renaming or
->   removing one breaks every consumer: **major**.
+>   `libdd-common`'s `tag!` (`src/tag.rs`). Renaming or removing one breaks every consumer:
+>   **major**.
 > - Conversely, a rename confined to a genuinely private item — or to a `pub` item inside a
 >   non-`pub` module that nothing re-exports with `pub use` — has no public delta: **patch**, not
 >   minor.

.github/CODEOWNERS

@@ -121,7 +121,6 @@ flake.*                              @DataDog/nix-guild @DataDog/apm-common-comp
 examples/ffi/ffe.c                  @DataDog/feature-flagging-and-experimentation-sdk
 bin_tests/tests/test_the_tests.rs    @DataDog/apm-common-components-core @DataDog/libdatadog-guild-reviewers
 bin_tests/src/bin/test_the_tests.rs  @DataDog/apm-common-components-core @DataDog/libdatadog-guild-reviewers
-tools/cc_utils/                      @DataDog/libdatadog-php
 tools/sidecar_mockgen/               @DataDog/libdatadog-php
 libdd-data-pipeline/src/otlp/                           @DataDog/apm-sdk-capabilities @DataDog/apm-common-components-core
 libdd-data-pipeline/tests/test_trace_exporter_otlp_export.rs @DataDog/apm-sdk-capabilities @DataDog/apm-common-components-core
@@ -206,7 +205,6 @@ libdd-ipc-macros/CHANGELOG.md                @DataDog/libdatadog-php @DataDog/li
 libdd-ipc-macros/Cargo.toml                  @DataDog/libdatadog-php @DataDog/libdatadog-release
 libdd-ipc/CHANGELOG.md                       @DataDog/libdatadog-php @DataDog/libdatadog-release
 libdd-ipc/Cargo.toml                         @DataDog/libdatadog-php @DataDog/libdatadog-release
-tools/cc_utils/Cargo.toml                    @DataDog/libdatadog-php @DataDog/libdatadog-release
 tools/sidecar_mockgen/Cargo.toml             @DataDog/libdatadog-php @DataDog/libdatadog-release
 
 datadog-sidecar-ffi/Cargo.toml               @DataDog/libdatadog-php @DataDog/libdatadog-release @DataDog/apm-common-components-core-reviewers

Cargo.toml

@@ -45,7 +45,6 @@ members = [
   "libdd-telemetry",
   "libdd-telemetry-ffi",
   "tools",
-  "tools/cc_utils",
   "tools/sidecar_mockgen",
   "spawn_worker",
   "tests/spawn_from_lib",
@@ -77,7 +76,6 @@ ignored = [
   "bindgen",
   "build_common",
   "cc",
-  "cc_utils",
   "cxx-build",
   "glibc_version",
   "prost-build",

libdd-common/Cargo.toml

@@ -19,12 +19,9 @@ bench = false
 anyhow = { workspace = true, optional = true }
 arc-swap = { workspace = true, optional = true }
 futures = { workspace = true, features = ["alloc", "executor"], optional = true }
-futures-util = { workspace = true, optional = true }
 hex = { workspace = true, default-features = true, optional = true }
 httparse = { workspace = true, features = ["std"], optional = true }
 http = { workspace = true, features = ["std"], optional = true }
-mime = { workspace = true, optional = true }
-multer = { workspace = true, optional = true }
 bytes = { workspace = true, features = ["std"], optional = true }
 pin-project = { workspace = true, optional = true }
 rand = { workspace = true, default-features = true, optional = true }
@@ -36,7 +33,6 @@ regex-lite = { workspace = true, default-features = true, optional = true }
 # Use rustls-no-provider instead of rustls to avoid reqwest forcing aws-lc-rs as the crypto
 # backend. We install the ring provider explicitly in connector/mod.rs instead.
 reqwest = { workspace = true, features = ["rustls-no-provider", "hickory-dns"], optional = true }
-criterion = { workspace = true, optional = true }
 rustls-platform-verifier = { workspace = true, optional = true }
 thiserror = { workspace = true, default-features = true, optional = true }
 serde = { workspace = true, features = ["derive"] }
@@ -59,7 +55,6 @@ hyper-util = { workspace = true, features = ["http1", "client", "client-legacy"]
 http-body = { workspace = true, optional = true }
 http-body-util = {workspace = true, optional = true }
 tower-service = { workspace = true, optional = true }
-cc = { workspace = true, optional = true }
 libc = { workspace = true, optional = true }
 tokio = { workspace = true, features = ["rt", "rt-multi-thread", "macros", "net", "io-util", "fs", "time"], optional = true }
 
@@ -68,8 +63,6 @@ workspace = true
 optional = true
 features = [
     "Win32_Foundation",
-    "Win32_System_Diagnostics_ToolHelp",
-    "Win32_System_Performance",
     "Win32_System_Registry",
     "Win32_System_Threading",
 ]
@@ -79,17 +72,15 @@ nix = { workspace = true, features = ["process"], optional = true }
 
 [dev-dependencies]
 bytes = { workspace = true, features = ["std"] }
-httparse = { workspace = true, features = ["std"] }
 indexmap = { workspace = true, default-features = true }
 maplit.workspace = true
-mime.workspace = true
-multer.workspace = true
 proptest = { workspace = true, default-features = true }
 rand = { workspace = true, default-features = true }
 rcgen.workspace = true
 tempfile.workspace = true
 tokio = { workspace = true, features = ["rt", "macros", "time", "net", "io-util"] }
 tokio-rustls.workspace = true
+tools = { path = "../tools", default-features = false, features = ["test-utils"] }
 
 [features]
 default = ["std", "https"]
@@ -104,9 +95,7 @@ std = [
     "anyhow/std",
     "dep:arc-swap",
     "dep:bytes",
-    "dep:cc",
     "dep:futures",
-    "dep:futures-util",
     "dep:hex",
     "dep:http",
     "dep:libc",
@@ -147,11 +136,7 @@ fips = ["tls-core", "hyper-rustls/fips"]
 # Honor HTTP(S)_PROXY/NO_PROXY environment variables for the hyper connector,
 hyper-proxy = ["http-client", "hyper-util/client-proxy"]
 # Enable reqwest client builder support with file dump debugging
-reqwest = ["http-client", "dep:reqwest", "test-utils"]
-# Enable test utilities for use in other crates
-test-utils = ["std", "dep:httparse", "dep:rand", "dep:mime", "dep:multer"]
-# Enable benchmark utilities (ReportingAllocator, Criterion allocation measurement)
-bench-utils = ["std", "dep:criterion"]
+reqwest = ["http-client", "dep:reqwest", "dep:httparse", "dep:rand"]
 
 [lints.rust]
 # We run coverage checks in our github actions. These checks are run with

libdd-common/README.md

@@ -12,7 +12,6 @@ Common utilities and foundational components shared across Datadog telemetry lib
 - **Entity ID Detection**: Automatic container ID and entity ID detection from cgroups, Docker, Kubernetes, and Azure App Services
 - **Tag Management**: Compile-time validated tags with the `tag!` macro
 - **URI Parsing**: Special handling for Unix sockets, Windows named pipes, and file URIs  
-- **Rate Limiting**: Token bucket rate limiter implementation
 - **Platform Utilities**: Unix-specific utilities for fork, exec, and process management
 - **Datadog Headers**: Standard Datadog HTTP headers and constants
 - **Timeout Support**: Configurable timeouts for HTTP requests
@@ -23,12 +22,10 @@ Common utilities and foundational components shared across Datadog telemetry lib
 - `azure_app_services`: Azure App Services integration
 - `config`: Configuration utilities
 - `connector`: HTTP/HTTPS connector implementations
-- `cstr`: C string utilities and macros
 - `entity_id`: Container and entity ID detection
 - `error`: Common error types
 - `header`: Datadog HTTP headers
 - `http_common`: Hyper version migration helpers
-- `rate_limiter`: Rate limiting implementation
 - `tag`: Tag creation and validation
 - `timeout`: Timeout utilities
 - `unix_utils`: Unix-specific process utilities