refactor: use bindgen-generated v8::HeapCodeStatistics instead of manual mirror

Use the real V8 HeapCodeStatistics type via bindgen, matching the
pattern used by HeapStatistics and HeapSpaceStatistics.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: bartlomieju

src/binding.cc

@@ -349,24 +349,9 @@ bool v8__Isolate__GetHeapSpaceStatistics(
   return isolate->GetHeapSpaceStatistics(space_statistics, index);
 }
 
-struct v8__HeapCodeStatistics {
-  size_t code_and_metadata_size_;
-  size_t bytecode_and_metadata_size_;
-  size_t external_script_source_size_;
-  size_t cpu_profiler_metadata_size_;
-};
-
-bool v8__Isolate__GetHeapCodeAndMetadataStatistics(v8::Isolate* isolate,
-                                                   v8__HeapCodeStatistics* s) {
-  v8::HeapCodeStatistics stats;
-  bool ok = isolate->GetHeapCodeAndMetadataStatistics(&stats);
-  if (ok) {
-    s->code_and_metadata_size_ = stats.code_and_metadata_size();
-    s->bytecode_and_metadata_size_ = stats.bytecode_and_metadata_size();
-    s->external_script_source_size_ = stats.external_script_source_size();
-    s->cpu_profiler_metadata_size_ = stats.cpu_profiler_metadata_size();
-  }
-  return ok;
+bool v8__Isolate__GetHeapCodeAndMetadataStatistics(
+    v8::Isolate* isolate, v8::HeapCodeStatistics* s) {
+  return isolate->GetHeapCodeAndMetadataStatistics(s);
 }
 
 void v8__Isolate__RemoveNearHeapLimitCallback(

src/binding.hpp

@@ -42,6 +42,7 @@ using v8__String__WriteFlags = v8::String::WriteFlags;
 using v8__ModuleImportPhase = v8::ModuleImportPhase;
 using v8__HeapStatistics = v8::HeapStatistics;
 using v8__HeapSpaceStatistics = v8::HeapSpaceStatistics;
+using v8__HeapCodeStatistics = v8::HeapCodeStatistics;
 using v8__GCType = v8::GCType;
 using v8__GCCallbackFlags = v8::GCCallbackFlags;
 using v8__Intercepted = v8::Intercepted;

src/isolate.rs

@@ -18,6 +18,7 @@ use crate::StartupData;
 use crate::String;
 use crate::V8::get_current_platform;
 use crate::Value;
+use crate::binding::v8__HeapCodeStatistics;
 use crate::binding::v8__HeapSpaceStatistics;
 use crate::binding::v8__HeapStatistics;
 use crate::binding::v8__Isolate__UseCounterFeature;
@@ -668,7 +669,7 @@ unsafe extern "C" {
   ) -> bool;
   fn v8__Isolate__GetHeapCodeAndMetadataStatistics(
     isolate: *mut RealIsolate,
-    code_statistics: *mut HeapCodeStatistics,
+    code_statistics: *mut v8__HeapCodeStatistics,
   ) -> bool;
   fn v8__Isolate__AddNearHeapLimitCallback(
     isolate: *mut RealIsolate,
@@ -1280,16 +1281,17 @@ impl Isolate {
   pub fn get_heap_code_and_metadata_statistics(
     &mut self,
   ) -> Option<HeapCodeStatistics> {
-    unsafe {
+    let inner = unsafe {
       let mut s = MaybeUninit::zeroed();
       if !v8__Isolate__GetHeapCodeAndMetadataStatistics(
         self.as_real_ptr(),
         s.as_mut_ptr(),
       ) {
         return None;
       }
-      Some(s.assume_init())
-    }
+      s.assume_init()
+    };
+    Some(HeapCodeStatistics(inner))
   }
 
   /// Tells V8 to capture current stack trace when uncaught exception occurs
@@ -2257,29 +2259,23 @@ impl HeapSpaceStatistics {
   }
 }
 
-#[repr(C)]
-pub struct HeapCodeStatistics {
-  code_and_metadata_size_: usize,
-  bytecode_and_metadata_size_: usize,
-  external_script_source_size_: usize,
-  cpu_profiler_metadata_size_: usize,
-}
+pub struct HeapCodeStatistics(v8__HeapCodeStatistics);
 
 impl HeapCodeStatistics {
   pub fn code_and_metadata_size(&self) -> usize {
-    self.code_and_metadata_size_
+    self.0.code_and_metadata_size_
   }
 
   pub fn bytecode_and_metadata_size(&self) -> usize {
-    self.bytecode_and_metadata_size_
+    self.0.bytecode_and_metadata_size_
   }
 
   pub fn external_script_source_size(&self) -> usize {
-    self.external_script_source_size_
+    self.0.external_script_source_size_
   }
 
   pub fn cpu_profiler_metadata_size(&self) -> usize {
-    self.cpu_profiler_metadata_size_
+    self.0.cpu_profiler_metadata_size_
   }
 }