Fix return type of Structure::destroy (#539)

CHANGELOG.md

@@ -1,6 +1,10 @@
 Unreleased
 ==========
 
+### Breaking:
+
+- Change return type of `Structure::destroy` from `i8` to `Result<(), ErrorCode>`.
+
 0.22.0 (2024-08-27)
 ===================
 

src/objects/impls/structure.rs

@@ -36,11 +36,8 @@ extern "C" {
     #[wasm_bindgen(method, getter = structureType)]
     pub fn structure_type(this: &Structure) -> StructureType;
 
-    /// Destroy the structure, if possible.
-    ///
-    /// [Screeps documentation](https://docs.screeps.com/api/#Structure.destroy)
-    #[wasm_bindgen(method)]
-    pub fn destroy(this: &Structure) -> i8;
+    #[wasm_bindgen(method, js_name = destroy)]
+    fn destroy_internal(this: &Structure) -> i8;
 
     /// Determine if the structure is active and can be used at the current RCL.
     ///
@@ -74,6 +71,13 @@ impl Structure {
     pub fn hits_max(&self) -> u32 {
         self.hits_max_internal().unwrap_or(0)
     }
+
+    /// Destroy the structure, if possible.
+    ///
+    /// [Screeps documentation](https://docs.screeps.com/api/#Structure.destroy)
+    pub fn destroy(&self) -> Result<(), ErrorCode> {
+        ErrorCode::result_from_i8(self.destroy_internal())
+    }
 }
 
 impl<T> HasId for T
@@ -107,7 +111,7 @@ where
     }
 
     fn destroy(&self) -> Result<(), ErrorCode> {
-        ErrorCode::result_from_i8(Structure::destroy(self.as_ref()))
+        Structure::destroy(self.as_ref())
     }
 
     fn is_active(&self) -> bool {