tigerbeetle-node 0.13.98 → 0.13.100

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tigerbeetle-node",
-  "version": "0.13.98",
+  "version": "0.13.100",
   "description": "TigerBeetle Node.js client",
   "main": "dist/index.js",
   "typings": "dist/index.d.ts",
@@ -28,7 +28,7 @@
     "test": "node dist/test",
     "build": "npm run build_tsc && npm run build_lib",
     "build_tsc": "./node_modules/typescript/bin/tsc",
-    "build_lib": "node scripts/windows_generate_node_lib.js && cd ../../.. && ./zig/zig build node_client -Drelease-safe",
+    "build_lib": "node scripts/windows_generate_node_lib.js && cd ../../.. && ./zig/zig build node_client -Doptimize=ReleaseSafe",
     "prepack": "npm run build && find dist/bin -name '*.o' -delete && for path in dist/bin/*; do mv ${path}/libtb_nodeclient.* ${path}/client.node; done",
     "clean": "rm -rf build dist node_modules src/zig-cache zig"
   },

package/src/node.zig

@@ -24,8 +24,10 @@ const vsr = @import("../../../vsr.zig");
 const Header = vsr.Header;
 const Client = vsr.Client(StateMachine, MessageBus);
 
-// Since this is running in application space, log only critical messages to reduce noise.
-pub const log_level: std.log.Level = .err;
+pub const std_options = struct {
+    // Since this is running in application space, log only critical messages to reduce noise.
+    pub const log_level: std.log.Level = .err;
+};
 
 /// N-API will call this constructor automatically to register the module.
 export fn napi_register_module_v1(env: c.napi_env, exports: c.napi_value) c.napi_value {
@@ -57,7 +59,7 @@ export fn napi_register_module_v1(env: c.napi_env, exports: c.napi_value) c.napi
     // state.
     translate.set_instance_data(
         env,
-        @ptrCast(*anyopaque, @alignCast(@alignOf(u8), global)),
+        @ptrCast(@alignCast(global)),
         Globals.destroy,
     ) catch {
         global.deinit();
@@ -115,7 +117,7 @@ const Globals = struct {
 };
 
 fn globalsCast(globals_raw: *anyopaque) *Globals {
-    return @ptrCast(*Globals, @alignCast(@alignOf(Globals), globals_raw));
+    return @ptrCast(@alignCast(globals_raw));
 }
 
 const Context = struct {
@@ -147,7 +149,7 @@ const Context = struct {
             allocator,
             client_id,
             cluster,
-            @intCast(u8, context.addresses.len),
+            @as(u8, @intCast(context.addresses.len)),
             &context.message_pool,
             .{
                 .configuration = context.addresses,
@@ -161,7 +163,7 @@ const Context = struct {
 };
 
 fn contextCast(context_raw: *anyopaque) !*Context {
-    return @ptrCast(*Context, @alignCast(@alignOf(Context), context_raw));
+    return @ptrCast(@alignCast(context_raw));
 }
 
 fn validate_timestamp(env: c.napi_env, object: c.napi_value) !u64 {
@@ -188,7 +190,7 @@ fn decode_from_object(comptime T: type, env: c.napi_env, object: c.napi_value) !
             .timeout = try translate.u64_from_object(env, object, "timeout"),
             .ledger = try translate.u32_from_object(env, object, "ledger"),
             .code = try translate.u16_from_object(env, object, "code"),
-            .flags = @bitCast(TransferFlags, try translate.u16_from_object(env, object, "flags")),
+            .flags = @as(TransferFlags, @bitCast(try translate.u16_from_object(env, object, "flags"))),
             .amount = try translate.u64_from_object(env, object, "amount"),
             .timestamp = try validate_timestamp(env, object),
         },
@@ -198,7 +200,7 @@ fn decode_from_object(comptime T: type, env: c.napi_env, object: c.napi_value) !
             .reserved = try translate.bytes_from_object(env, object, 48, "reserved"),
             .ledger = try translate.u32_from_object(env, object, "ledger"),
             .code = try translate.u16_from_object(env, object, "code"),
-            .flags = @bitCast(AccountFlags, try translate.u16_from_object(env, object, "flags")),
+            .flags = @as(AccountFlags, @bitCast(try translate.u16_from_object(env, object, "flags"))),
             .debits_pending = try translate.u64_from_object(env, object, "debits_pending"),
             .debits_posted = try translate.u64_from_object(env, object, "debits_posted"),
             .credits_pending = try translate.u64_from_object(env, object, "credits_pending"),
@@ -259,7 +261,7 @@ fn encode_napi_results_array(
     const results = std.mem.bytesAsSlice(Result, data);
     const napi_array = try translate.create_array(
         env,
-        @intCast(u32, results.len),
+        @as(u32, @intCast(results.len)),
         "Failed to allocate array for results.",
     );
 
@@ -285,7 +287,7 @@ fn encode_napi_results_array(
                     env,
                     napi_object,
                     "result",
-                    @enumToInt(result.result),
+                    @intFromEnum(result.result),
                     "Failed to set property \"result\" of result.",
                 );
 
@@ -335,7 +337,7 @@ fn encode_napi_results_array(
                     env,
                     napi_object,
                     "ledger",
-                    @intCast(u32, result.ledger),
+                    @as(u32, @intCast(result.ledger)),
                     "Failed to set property \"ledger\" of account lookup result.",
                 );
 
@@ -343,7 +345,7 @@ fn encode_napi_results_array(
                     env,
                     napi_object,
                     "code",
-                    @intCast(u16, result.code),
+                    @as(u16, @intCast(result.code)),
                     "Failed to set property \"code\" of account lookup result.",
                 );
 
@@ -351,7 +353,7 @@ fn encode_napi_results_array(
                     env,
                     napi_object,
                     "flags",
-                    @bitCast(u16, result.flags),
+                    @as(u16, @bitCast(result.flags)),
                     "Failed to set property \"flags\" of account lookup result.",
                 );
 
@@ -473,7 +475,7 @@ fn encode_napi_results_array(
                     env,
                     napi_object,
                     "ledger",
-                    @intCast(u32, result.ledger),
+                    @as(u32, @intCast(result.ledger)),
                     "Failed to set property \"ledger\" of transfer lookup result.",
                 );
 
@@ -481,7 +483,7 @@ fn encode_napi_results_array(
                     env,
                     napi_object,
                     "code",
-                    @intCast(u16, result.code),
+                    @as(u16, @intCast(result.code)),
                     "Failed to set property \"code\" of transfer lookup result.",
                 );
 
@@ -489,7 +491,7 @@ fn encode_napi_results_array(
                     env,
                     napi_object,
                     "flags",
-                    @bitCast(u16, result.flags),
+                    @as(u16, @bitCast(result.flags)),
                     "Failed to set property \"flags\" of transfer lookup result.",
                 );
 
@@ -578,7 +580,7 @@ fn request(env: c.napi_env, info: c.napi_callback_info) callconv(.C) c.napi_valu
     const context = contextCast(context_raw.?) catch return null;
     const operation_int = translate.u32_from_value(env, argv[1], "operation") catch return null;
 
-    if (!@intToEnum(vsr.Operation, operation_int).valid(StateMachine)) {
+    if (!@as(vsr.Operation, @enumFromInt(operation_int)).valid(StateMachine)) {
         translate.throw(env, "Unknown operation.") catch return null;
     }
 
@@ -591,7 +593,7 @@ fn request(env: c.napi_env, info: c.napi_callback_info) callconv(.C) c.napi_valu
     const message = context.client.get_message();
     defer context.client.unref(message);
 
-    const operation = @intToEnum(Operation, @intCast(u8, operation_int));
+    const operation = @as(Operation, @enumFromInt(@as(u8, @intCast(operation_int))));
     const body_length = decode_events(
         env,
         argv[2],
@@ -605,7 +607,7 @@ fn request(env: c.napi_env, info: c.napi_callback_info) callconv(.C) c.napi_valu
     // free in order to avoid a leak. We therefore do this last to ensure we cannot fail after
     // taking this reference.
     const user_data = translate.user_data_from_value(env, argv[3]) catch return null;
-    context.client.request(@bitCast(u128, user_data), on_result, operation, message, body_length);
+    context.client.request(@as(u128, @bitCast(user_data)), on_result, operation, message, body_length);
 
     return null;
 }
@@ -633,10 +635,10 @@ fn raw_request(env: c.napi_env, info: c.napi_callback_info) callconv(.C) c.napi_
     const context = contextCast(context_raw.?) catch return null;
     const operation_int = translate.u32_from_value(env, argv[1], "operation") catch return null;
 
-    if (!@intToEnum(vsr.Operation, operation_int).valid(StateMachine)) {
+    if (!@as(vsr.Operation, @enumFromInt(operation_int)).valid(StateMachine)) {
         translate.throw(env, "Unknown operation.") catch return null;
     }
-    const operation = @intToEnum(Operation, @intCast(u8, operation_int));
+    const operation = @as(Operation, @enumFromInt(@as(u8, @intCast(operation_int))));
 
     if (context.client.messages_available == 0) {
         translate.throw(
@@ -660,7 +662,7 @@ fn raw_request(env: c.napi_env, info: c.napi_callback_info) callconv(.C) c.napi_
     // free in order to avoid a leak. We therefore do this last to ensure we cannot fail after
     // taking this reference.
     const user_data = translate.user_data_from_value(env, argv[3]) catch return null;
-    context.client.request(@bitCast(u128, user_data), on_result, operation, message, body_length);
+    context.client.request(@as(u128, @bitCast(user_data)), on_result, operation, message, body_length);
 
     return null;
 }
@@ -668,8 +670,8 @@ fn raw_request(env: c.napi_env, info: c.napi_callback_info) callconv(.C) c.napi_
 fn on_result(user_data: u128, operation: Operation, results: []const u8) void {
     // A reference to the user's JS callback was made in `request` or `raw_request`. This MUST be
     // cleaned up regardless of the result of this function.
-    const env = @bitCast(translate.UserData, user_data).env;
-    const callback_reference = @bitCast(translate.UserData, user_data).callback_reference;
+    const env = @as(translate.UserData, @bitCast(user_data)).env;
+    const callback_reference = @as(translate.UserData, @bitCast(user_data)).callback_reference;
     defer translate.delete_reference(env, callback_reference) catch {
         std.log.warn("on_result: Failed to delete reference to user's JS callback.", .{});
     };

package/src/translate.zig

@@ -13,14 +13,14 @@ pub fn register_function(
         return throw(env, "Failed to create function " ++ name ++ "().");
     }
 
-    if (c.napi_set_named_property(env, exports, @ptrCast([*c]const u8, name), napi_function) != c.napi_ok) {
+    if (c.napi_set_named_property(env, exports, @as([*c]const u8, @ptrCast(name)), napi_function) != c.napi_ok) {
         return throw(env, "Failed to add " ++ name ++ "() to exports.");
     }
 }
 
 const TranslationError = error{ExceptionThrown};
 pub fn throw(env: c.napi_env, comptime message: [:0]const u8) TranslationError {
-    var result = c.napi_throw_error(env, null, @ptrCast([*c]const u8, message));
+    var result = c.napi_throw_error(env, null, @as([*c]const u8, @ptrCast(message)));
     switch (result) {
         c.napi_ok, c.napi_pending_exception => {},
         else => unreachable,
@@ -110,7 +110,7 @@ pub fn slice_from_object(
     comptime key: [:0]const u8,
 ) ![]const u8 {
     var property: c.napi_value = undefined;
-    if (c.napi_get_named_property(env, object, @ptrCast([*c]const u8, key), &property) != c.napi_ok) {
+    if (c.napi_get_named_property(env, object, @as([*c]const u8, @ptrCast(key)), &property) != c.napi_ok) {
         return throw(env, key ++ " must be defined");
     }
 
@@ -133,7 +133,7 @@ pub fn slice_from_value(
 
     if (data_length < 1) return throw(env, key ++ " must not be empty");
 
-    return @ptrCast([*]u8, data.?)[0..data_length];
+    return @as([*]u8, @ptrCast(data.?))[0..data_length];
 }
 
 pub fn bytes_from_object(
@@ -143,7 +143,7 @@ pub fn bytes_from_object(
     comptime key: [:0]const u8,
 ) ![length]u8 {
     var property: c.napi_value = undefined;
-    if (c.napi_get_named_property(env, object, @ptrCast([*c]const u8, key), &property) != c.napi_ok) {
+    if (c.napi_get_named_property(env, object, @as([*c]const u8, @ptrCast(key)), &property) != c.napi_ok) {
         return throw(env, key ++ " must be defined");
     }
 
@@ -181,7 +181,7 @@ pub fn bytes_from_buffer(
 
 pub fn u128_from_object(env: c.napi_env, object: c.napi_value, comptime key: [:0]const u8) !u128 {
     var property: c.napi_value = undefined;
-    if (c.napi_get_named_property(env, object, @ptrCast([*c]const u8, key), &property) != c.napi_ok) {
+    if (c.napi_get_named_property(env, object, @as([*c]const u8, @ptrCast(key)), &property) != c.napi_ok) {
         return throw(env, key ++ " must be defined");
     }
 
@@ -190,7 +190,7 @@ pub fn u128_from_object(env: c.napi_env, object: c.napi_value, comptime key: [:0
 
 pub fn u64_from_object(env: c.napi_env, object: c.napi_value, comptime key: [:0]const u8) !u64 {
     var property: c.napi_value = undefined;
-    if (c.napi_get_named_property(env, object, @ptrCast([*c]const u8, key), &property) != c.napi_ok) {
+    if (c.napi_get_named_property(env, object, @as([*c]const u8, @ptrCast(key)), &property) != c.napi_ok) {
         return throw(env, key ++ " must be defined");
     }
 
@@ -199,7 +199,7 @@ pub fn u64_from_object(env: c.napi_env, object: c.napi_value, comptime key: [:0]
 
 pub fn u32_from_object(env: c.napi_env, object: c.napi_value, comptime key: [:0]const u8) !u32 {
     var property: c.napi_value = undefined;
-    if (c.napi_get_named_property(env, object, @ptrCast([*c]const u8, key), &property) != c.napi_ok) {
+    if (c.napi_get_named_property(env, object, @as([*c]const u8, @ptrCast(key)), &property) != c.napi_ok) {
         return throw(env, key ++ " must be defined");
     }
 
@@ -212,7 +212,7 @@ pub fn u16_from_object(env: c.napi_env, object: c.napi_value, comptime key: [:0]
         return throw(env, key ++ " must be a u16.");
     }
 
-    return @intCast(u16, result);
+    return @as(u16, @intCast(result));
 }
 
 pub fn u128_from_value(env: c.napi_env, value: c.napi_value, comptime name: [:0]const u8) !u128 {
@@ -222,7 +222,7 @@ pub fn u128_from_value(env: c.napi_env, value: c.napi_value, comptime name: [:0]
     // we would need to convert, but big endian is not supported by tigerbeetle.
     var result: u128 = 0;
     var sign_bit: c_int = undefined;
-    const words = @ptrCast(*[2]u64, &result);
+    const words = @as(*[2]u64, @ptrCast(&result));
     var word_count: usize = 2;
     switch (c.napi_get_value_bigint_words(env, value, &sign_bit, &word_count, words)) {
         c.napi_ok => {},
@@ -274,7 +274,7 @@ pub fn byte_slice_into_object(
         return throw(env, error_message ++ " Failed to allocate Buffer in V8.");
     }
 
-    if (c.napi_set_named_property(env, object, @ptrCast([*c]const u8, key), result) != c.napi_ok) {
+    if (c.napi_set_named_property(env, object, @as([*c]const u8, @ptrCast(key)), result) != c.napi_ok) {
         return throw(env, error_message);
     }
 }
@@ -295,13 +295,13 @@ pub fn u128_into_object(
         env,
         0,
         2,
-        @ptrCast(*const [2]u64, &value),
+        @as(*const [2]u64, @ptrCast(&value)),
         &bigint,
     ) != c.napi_ok) {
         return throw(env, error_message);
     }
 
-    if (c.napi_set_named_property(env, object, @ptrCast([*c]const u8, key), bigint) != c.napi_ok) {
+    if (c.napi_set_named_property(env, object, @as([*c]const u8, @ptrCast(key)), bigint) != c.napi_ok) {
         return throw(env, error_message);
     }
 }
@@ -318,7 +318,7 @@ pub fn u64_into_object(
         return throw(env, error_message);
     }
 
-    if (c.napi_set_named_property(env, object, @ptrCast([*c]const u8, key), result) != c.napi_ok) {
+    if (c.napi_set_named_property(env, object, @as([*c]const u8, @ptrCast(key)), result) != c.napi_ok) {
         return throw(env, error_message);
     }
 }
@@ -335,7 +335,7 @@ pub fn u32_into_object(
         return throw(env, error_message);
     }
 
-    if (c.napi_set_named_property(env, object, @ptrCast([*c]const u8, key), result) != c.napi_ok) {
+    if (c.napi_set_named_property(env, object, @as([*c]const u8, @ptrCast(key)), result) != c.napi_ok) {
         return throw(env, error_message);
     }
 }
@@ -370,7 +370,7 @@ fn create_buffer(
         return throw(env, error_message);
     }
 
-    std.mem.copy(u8, @ptrCast([*]u8, data.?)[0..value.len], value[0..value.len]);
+    std.mem.copy(u8, @as([*]u8, @ptrCast(data.?))[0..value.len], value[0..value.len]);
 
     return result;
 }