refactor: return non-nullable from InitFunction

Author: typesafe

src/root.zig

@@ -61,15 +61,15 @@ pub fn @"export"(comptime value: anytype) void {
 /// ```
 pub fn init(comptime f: InitFunction) void {
     const module = opaque {
-        pub fn napi_register_module_v1(env: c.napi_env, exp: c.napi_value) callconv(.c) c.napi_value {
+        pub fn napi_register_module_v1(env: c.napi_env, _: c.napi_value) callconv(.c) c.napi_value {
             const node = NodeContext.init(env);
 
             const exports = f(node) catch |err| {
                 node.handleError(err);
                 return null;
             };
 
-            return if (exports) |v| v.napi_value else exp;
+            return exports.napi_value;
         }
     };
 
@@ -79,7 +79,7 @@ pub fn init(comptime f: InitFunction) void {
 /// The InitFunction to pass to the `register` method. The `ctx` parameter
 /// represents the Node context. The returned value becomes the `exports` value
 /// of the JS module.
-pub const InitFunction = fn (ctx: NodeContext) anyerror!?NodeValue;
+pub const InitFunction = fn (ctx: NodeContext) anyerror!NodeValue;
 
 inline fn registerModule(comptime ptr: *const anyopaque) void {
     @export(ptr, .{ .name = "napi_register_module_v1" });

tests/zig_modules/test-module/src/root.zig

@@ -11,7 +11,7 @@ comptime {
     node_api.init(init);
 }
 
-fn init(node: node_api.NodeContext) !?node_api.NodeValue {
+fn init(node: node_api.NodeContext) !node_api.NodeValue {
     const ptr = try std.heap.c_allocator.create(WrapTarget);
     ptr.* = .{ .foo = 123, .bar = "hopla" };