test(bindings): drop redundant comments

Author: GrapeBaBa

bindings/napi/BeaconStateView.zig

@@ -66,10 +66,6 @@ pub const js_meta = js.class(.{ .properties = .{
 } });
 
 cached_state: ?*CachedBeaconState = null,
-/// Holds a strong reference to the shared `Node.Pool` for the lifetime of
-/// `cached_state`. Released in `deinit` AFTER `cached_state.deinit` finishes
-/// its `pool.unref` calls, so the pool stays alive even if the module's
-/// NAPI env cleanup hook fired before this view's JS finalizer.
 pool_rc: ?*pool.PoolRc = null,
 const BeaconStateView = @This();
 

bindings/napi/pool.zig

@@ -11,12 +11,6 @@ const default_pool_size: u32 = 0;
 
 pub const PoolRc = RefCount(Node.Pool);
 
-/// Pool is wrapped in `RefCount` so that any `BeaconStateView` (or other
-/// binding object) holding pool references at process exit keeps the pool
-/// alive until its own JS finalizer runs. Without this, NAPI env cleanup
-/// can run before module-level JS holders are finalized, freeing the pool
-/// before the holder's `pool.unref()` calls — causing "incorrect alignment"
-/// panics during teardown.
 pub const State = struct {
     pool_rc: ?*PoolRc = null,
 
@@ -27,9 +21,6 @@ pub const State = struct {
         self.pool_rc = try PoolRc.init(allocator, pool_value);
     }
 
-    /// Drops the module's reference. Pool is actually destroyed when the
-    /// last live reference is released (could be here or in a later
-    /// BeaconStateView finalizer).
     pub fn deinit(self: *State) void {
         if (self.pool_rc) |rc| {
             rc.unref();

bindings/test/teardown.test.ts

@@ -6,15 +6,10 @@ import {describe, expect, it} from "vitest";
 describe("BeaconStateView teardown", () => {
   it("creates view at module scope and exits cleanly", () => {
     const projectRoot = join(import.meta.dirname, "../..");
-    // Fixture must live under the project root so Node resolves its
-    // node_modules from there (workspace packages like @lodestar/config).
+    // Fixture must live under the project root so Node resolves
+    // workspace packages like @lodestar/config from local node_modules.
     const fixturePath = join(projectRoot, `bindings/test/.tmp-teardown-${process.pid}.mjs`);
 
-    // Module-scope `const seedState = ...` mirrors how perf bench files
-    // hold native objects. Without the Pool refcount fix, NAPI env cleanup
-    // frees the pool before this view's finalizer runs, and the chained
-    // pool.unref calls panic with "incorrect alignment" on process exit
-    // (exit code 134 / SIGABRT).
     writeFileSync(
       fixturePath,
       `