revert features

Author: Bromeon

.github/workflows/full-ci.yml

@@ -326,7 +326,7 @@ jobs:
             artifact-name: linux-nightly
             godot-binary: godot.linuxbsd.editor.dev.x86_64
             # Important to keep both experimental-threads and codegen-full. Some itests (native_st_audio) require both.
-            rust-extra-args: --features itest/experimental-threads,itest/codegen-full-experimental,godot/api-custom,godot/serde,itest/register-docs
+            rust-extra-args: --features itest/experimental-threads,itest/codegen-full-experimental,godot/api-custom,godot/serde,itest/register-docs,godot/safeguards-dev-balanced
 
           # Compiles godot-rust with `api-custom-json` feature against the JSON file generated via `--dump-extension-api`.
           # Uses latest 4.x headers, while `extension_api.json` comes from the latest Godot binary.
@@ -350,7 +350,7 @@ jobs:
             os: ubuntu-22.04
             artifact-name: linux-release-nightly
             godot-binary: godot.linuxbsd.template_release.x86_64
-            rust-extra-args: --release --features itest/codegen-full-experimental,godot/safeguards-disengaged
+            rust-extra-args: --release --features itest/codegen-full-experimental,godot/safeguards-release-disengaged
             rust-cache-key: release-disengaged
 
           # Linux compat:

godot-bindings/Cargo.toml

@@ -34,8 +34,8 @@ api-custom-json = ["dep:nanoserde", "dep:bindgen", "dep:regex", "dep:which"]
 api-custom-extheader = []
 
 # Safeguard levels (see godot/lib.rs for detailed documentation).
-safeguards-strict = []
-safeguards-disengaged = []
+safeguards-dev-balanced = []
+safeguards-release-disengaged = []
 
 [dependencies]
 gdextension-api = { workspace = true }

godot-bindings/src/lib.rs

@@ -276,10 +276,12 @@ pub fn emit_safeguard_levels() {
     let mut safeguards_level = if cfg!(debug_assertions) { 2 } else { 1 };
 
     // Features can override the default level
-    if cfg!(feature = "safeguards-strict") {
-        safeguards_level = 2;
+    #[cfg(debug_assertions)]
+    if cfg!(feature = "safeguards-dev-balanced") {
+        safeguards_level = 1;
     }
-    if cfg!(feature = "safeguards-disengaged") {
+    #[cfg(not(debug_assertions))]
+    if cfg!(feature = "safeguards-release-disengaged") {
         safeguards_level = 0;
     }
 

godot-core/Cargo.toml

@@ -39,8 +39,8 @@ api-4-5 = ["godot-ffi/api-4-5"]
 # ]]
 
 # Safeguard levels (see godot/lib.rs for detailed documentation).
-safeguards-strict = ["godot-ffi/safeguards-strict"]
-safeguards-disengaged = ["godot-ffi/safeguards-disengaged"]
+safeguards-dev-balanced = ["godot-ffi/safeguards-dev-balanced"]
+safeguards-release-disengaged = ["godot-ffi/safeguards-release-disengaged"]
 
 [dependencies]
 godot-ffi = { path = "../godot-ffi", version = "=0.4.1" }

godot-ffi/Cargo.toml

@@ -31,8 +31,8 @@ api-4-5 = ["godot-bindings/api-4-5"]
 # ]]
 
 # Safeguard levels (see godot/lib.rs for detailed documentation).
-safeguards-strict = ["godot-bindings/safeguards-strict"]
-safeguards-disengaged = ["godot-bindings/safeguards-disengaged"]
+safeguards-dev-balanced = ["godot-bindings/safeguards-dev-balanced"]
+safeguards-release-disengaged = ["godot-bindings/safeguards-release-disengaged"]
 
 [dependencies]
 

godot/Cargo.toml

@@ -40,8 +40,8 @@ api-4-5 = ["godot-core/api-4-5"]
 # ]]
 
 # Safeguard levels (see godot/lib.rs for detailed documentation).
-safeguards-strict = ["godot-core/safeguards-strict"]
-safeguards-disengaged = ["godot-core/safeguards-disengaged"]
+safeguards-dev-balanced = ["godot-core/safeguards-dev-balanced"]
+safeguards-release-disengaged = ["godot-core/safeguards-release-disengaged"]
 
 default = ["__codegen-full"]
 

godot/src/lib.rs

@@ -181,13 +181,13 @@
 //!
 //! See [Safeguard levels](#safeguard-levels).
 //!
-//! * **`safeguards-strict`**
+//! * **`safeguards-dev-balanced`**
 //!
-//!   Use the **strict** safeguard level. This is the default for debug builds, but can be enabled for release builds too.<br><br>
+//!   For the `dev` Cargo profile, use the **balanced** safeguard level instead of the default strict level.<br><br>
 //!
-//! * **`safeguards-disengaged`**
+//! * **`safeguards-release-disengaged`**
 //!
-//!   Use the **disengaged** safeguard level. This disables most runtime checks for maximum performance.
+//!   For the `release` Cargo profile, use the **disengaged** safeguard level instead of the default balanced level.
 //!
 //! _Third-party integrations:_
 //!