revert: profile overrides for artifact dependencies (#597)

Cargo.toml

@@ -40,7 +40,6 @@ future_not_send = "allow"
 
 [workspace.dependencies]
 allocator-api2 = { version = "0.2", default-features = false }
-artifact_profile = { path = "crates/artifact_profile" }
 anstream = "1.0.0"
 anyhow = "1.0.103"
 assert2 = "0.4.0"
@@ -209,32 +208,3 @@ codegen-units = 1
 strip = "symbols" # set to `false` for debug information
 debug = false # set to `true` for debug information
 panic = "abort" # Let it crash and force ourselves to write safe Rust.
-
-# Cargo compiles artifact dependencies declared under `[build-dependencies]`
-# with the build-override profile — opt-level 0, no codegen-units setting —
-# even in release builds, and even with `target = "target"`. Their output
-# still lands under `target/<triple>/release/`, so the missing optimization
-# is easy to miss. (Artifact deps under `[dependencies]`/`[dev-dependencies]`
-# get the normal profile and need nothing here.) See
-# https://github.com/rust-lang/cargo/issues/16719.
-#
-# These are the workspace's build-dependency artifacts: the preload libraries
-# are injected into every traced process and run inside every intercepted
-# libc call, and the napi library backs the Node.js client.
-#
-# `strip` must be restated: a package override resets an inherited `strip`
-# to "debuginfo" when the field is omitted (Cargo quirk in profile merging).
-[profile.release.package.fspy_preload_unix]
-opt-level = 3
-codegen-units = 1
-strip = "symbols"
-
-[profile.release.package.fspy_preload_windows]
-opt-level = 3
-codegen-units = 1
-strip = "symbols"
-
-[profile.release.package.vt_client_napi]
-opt-level = 3
-codegen-units = 1
-strip = "symbols"

crates/fspy_preload_unix/Cargo.toml

@@ -17,8 +17,5 @@ nix = { workspace = true, features = ["signal", "fs", "socket", "mman", "time"]
 sigsafe = { workspace = true }
 sigsafe_alloc = { workspace = true }
 
-[build-dependencies]
-artifact_profile = { workspace = true }
-
 [lints]
 workspace = true

crates/fspy_preload_windows/Cargo.toml

@@ -29,8 +29,5 @@ winsafe = { workspace = true }
 [target.'cfg(target_os = "windows")'.dev-dependencies]
 tempfile = { workspace = true }
 
-[build-dependencies]
-artifact_profile = { workspace = true }
-
 [lints]
 workspace = true

crates/fspy_preload_windows/build.rs

@@ -1,7 +1,4 @@
 fn main() {
-    // Injected into every traced process; see the unix preload's build.rs.
-    artifact_profile::require_optimized_in_release();
-
     if std::env::var_os("CARGO_CFG_TARGET_OS").unwrap() == "windows" {
         println!("cargo:rustc-cdylib-link-arg=/EXPORT:DetourFinishHelperProcess,@1,NONAME");
     }

crates/vt_client_napi/Cargo.toml

@@ -18,7 +18,6 @@ vt_str = { workspace = true }
 vt_client = { workspace = true }
 
 [build-dependencies]
-artifact_profile = { workspace = true }
 napi-build = { workspace = true }
 
 [lints]

crates/vt_client_napi/build.rs

@@ -8,9 +8,6 @@ extern crate napi_build;
 use std::{env, fs, path::PathBuf};
 
 fn main() {
-    // Shipped inside the built product; see `artifact_profile`.
-    artifact_profile::require_optimized_in_release();
-
     napi_build::setup();
 
     // Keep this crate's napi-derive type-defs out of any consumer's generated