feat: track benchmark in CI

Author: twoeths

.github/workflows/test.yml

@@ -73,3 +73,9 @@ jobs:
       - name: Bun - Unit Tests
         run: bun test:unit
         working-directory: ./test/bun
+
+      - name: Benchmark
+        run: bun benchmark
+        env:
+          # To write to PRs and commits
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

src/pubkey_index_map.zig

@@ -10,7 +10,7 @@ const AutoHashMap = std.AutoHashMap(Key, Val);
 
 /// a generic implementation for both zig application and Bun ffi
 pub const PubkeyIndexMap = struct {
-    // TODO: seems this HashMap allocates items using its own allocator
+    // this HashMap copies key/value using its own allocator
     // this duplicates all items at Bun side
     map: AutoHashMap,
 

test/bun/src/pubkeyIndexMap.ts

@@ -8,8 +8,8 @@ const NOT_FOUND_INDEX = binding.getNotFoundIndex();
  * This has the same interface to the napi-rs implementation in https://github.com/ChainSafe/pubkey-index-map/blob/main/index.d.ts
  */
 export class PubkeyIndexMap {
-	// even through zig returns u64, it's safe to use number at Bun side
-	// see https://bun.sh/docs/api/ffi#pointers
+  // even through zig returns u64, it's safe to use number at Bun side
+  // see https://bun.sh/docs/api/ffi#pointers
 	private native_ptr: number;
 	constructor() {
 		const pointer = binding.createPubkeyIndexMap();