chore: bump napi-rs to v3 (#485)

This commit migrates the code base to napi.rs v3, drops `linux-riscv64-gnu` and `darwin-universal` binaries and removes node 16 from the testing matrix.

BREAKING CHANGE: dropped `linux-riscv64-gnu` and `darwin-universal` binaries

Author: lsndr

.github/workflows/build-and-test.yml

@@ -12,14 +12,13 @@ chrono = "0.4.41"
 chrono-tz = "0.10.4"
 indexmap = "2.10.0"
 itertools = "0.14.0"
-# Default enable napi5 feature, see https://nodejs.org/api/n-api.html#node-api-version-matrix
-napi = { version = "2.16.17", default-features = false, features = ["napi5"] }
-napi-derive = "2.16.13"
+napi = "3.2.3"
+napi-derive = "3.2.3"
 replace_with = "0.1.8"
 rrule = { version = "0.14.0", features = ["exrule"] }
 
 [build-dependencies]
-napi-build = "2.2.2"
+napi-build = "2.2.3"
 
 [profile.release]
 lto = true

Cargo.lock

@@ -64,6 +64,6 @@ function suite(tzid: string) {
 }
 
 Promise.all([
-  b.suite('UTC TZ', ...suite('UTC')).then(console.log),
+  b.suite('UTC TZ', ...suite('UTC')),
   b.suite('Other TZ', ...suite('Pacific/Kiritimati')),
 ]).catch(console.error);

Cargo.toml

@@ -171,6 +171,7 @@ impl RRuleSet {
       return Ok(iter.take(limit as usize).collect());
     }
 
+    // TODO: use array instead
     Ok(iter.collect())
   }
 
@@ -199,13 +200,13 @@ impl RRuleSet {
   }
 
   #[napi(ts_return_type = "number[]")]
-  pub fn between(
-    &self,
-    env: Env,
+  pub fn between<'a>(
+    &'a self,
+    env: &'a Env,
     after_datetime: i64,
     before_datetime: i64,
     inclusive: Option<bool>,
-  ) -> napi::Result<Array> {
+  ) -> napi::Result<Array<'a>> {
     let mut arr = env.create_array(0).unwrap();
     let timezone = self.rrule_set.dtstart().timezone();
     let after_timestamp = DateTime::from(after_datetime)

benchmark/index.ts

@@ -1,3 +1,3 @@
-# `rrule-rust-android-arm-eabi`
+# `@rrule-rust/lib-android-arm-eabi`
 
-This is the **armv7-linux-androideabi** binary for `rrule-rust`
+This is the **armv7-linux-androideabi** binary for `@rrule-rust/lib`

lib/js/rrule_set.rs

@@ -1,20 +1,33 @@
 {
   "name": "@rrule-rust/lib-android-arm-eabi",
   "version": "0.0.0",
-  "os": [
-    "android"
-  ],
   "cpu": [
     "arm"
   ],
   "main": "rrule-rust.android-arm-eabi.node",
   "files": [
     "rrule-rust.android-arm-eabi.node"
   ],
+  "description": "RRule implementation for Node.js written in Rust",
+  "keywords": [
+    "rrule",
+    "icalendar",
+    "rust",
+    "rs"
+  ],
+  "author": {
+    "name": "lsndr",
+    "email": "[email protected]",
+    "url": "https://lsndr.me"
+  },
   "license": "MIT",
   "engines": {
     "node": ">= 10"
   },
+  "repository": "lsndr/rrule-rust",
+  "os": [
+    "android"
+  ],
   "publishConfig": {
     "access": "public"
   }

npm/android-arm-eabi/README.md

@@ -1,3 +1,3 @@
-# `rrule-rust-android-arm64`
+# `@rrule-rust/lib-android-arm64`
 
-This is the **aarch64-linux-android** binary for `rrule-rust`
+This is the **aarch64-linux-android** binary for `@rrule-rust/lib`

npm/android-arm-eabi/package.json

@@ -1,20 +1,33 @@
 {
   "name": "@rrule-rust/lib-android-arm64",
   "version": "0.0.0",
-  "os": [
-    "android"
-  ],
   "cpu": [
     "arm64"
   ],
   "main": "rrule-rust.android-arm64.node",
   "files": [
     "rrule-rust.android-arm64.node"
   ],
+  "description": "RRule implementation for Node.js written in Rust",
+  "keywords": [
+    "rrule",
+    "icalendar",
+    "rust",
+    "rs"
+  ],
+  "author": {
+    "name": "lsndr",
+    "email": "[email protected]",
+    "url": "https://lsndr.me"
+  },
   "license": "MIT",
   "engines": {
     "node": ">= 10"
   },
+  "repository": "lsndr/rrule-rust",
+  "os": [
+    "android"
+  ],
   "publishConfig": {
     "access": "public"
   }

npm/android-arm64/README.md

@@ -1,3 +1,3 @@
-# `rrule-rust-darwin-arm64`
+# `@rrule-rust/lib-darwin-arm64`
 
-This is the **aarch64-apple-darwin** binary for `rrule-rust`
+This is the **aarch64-apple-darwin** binary for `@rrule-rust/lib`