fix: RangeError [ERR_BUFFER_OUT_OF_BOUNDS] "length" is outside of buffer bounds since nodejs 22.7

call buf.utf8Write() conditionally,  it will be called in buf.write()

ref: #2025

Author: waitingsong

.github/workflows/test.yml

@@ -23,7 +23,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node_version: ["12", "14", "16", "18"]
+        node_version: ["12", "14", "16", "18", "20", "22"]
     steps:
     - uses: actions/checkout@v1
     - uses: actions/setup-node@v1

src/writer_buffer.js

@@ -55,10 +55,11 @@ BufferWriter.prototype.bytes = function write_bytes_buffer(value) {
 };
 
 function writeStringBuffer(val, buf, pos) {
+    const targetVersion = '22.7.0';
     if (val.length < 40) // plain js is faster for short strings (probably due to redundant assertions)
         util.utf8.write(val, buf, pos);
-    else if (buf.utf8Write)
-        buf.utf8Write(val, pos);
+    else if (buf.utf8Write && compareVersions(process.version.slice(1), targetVersion) < 0)
+        buf.utf8Write(val, pos); // node less than 22.7.0
     else
         buf.write(val, pos);
 }
@@ -83,3 +84,14 @@ BufferWriter.prototype.string = function write_string_buffer(value) {
  */
 
 BufferWriter._configure();
+
+function compareVersions(version1, version2) {
+  const v1Parts = version1.split('.').map(Number);
+  const v2Parts = version2.split('.').map(Number);
+
+  for (let i = 0; i < Math.max(v1Parts.length, v2Parts.length); i+=1) {
+    if ((v1Parts[i] ?? 0) < (v2Parts[i] ?? 0)) { return -1 }
+    if ((v1Parts[i] ?? 0) > (v2Parts[i] ?? 0)) { return 1 }
+  }
+  return 0;
+}