fix RangeError in encoding BLOB (resolve #65)

Author: gfx

src/Encoder.ts

@@ -79,7 +79,8 @@ export class Encoder {
           this.writeU8(object);
         } else if (object < 0x100) {
           // uint 8
-          this.writeU8v(0xcc, object);
+          this.writeU8(0xcc);
+          this.writeU8(object);
         } else if (object < 0x10000) {
           // uint 16
           this.writeU8(0xcd);
@@ -199,7 +200,7 @@ export class Encoder {
       throw new Error(`Too large binary: ${size}`);
     }
     const bytes = ensureUint8Array(object);
-    this.writeU8v(...bytes);
+    this.writeU8a(bytes);
   }
 
   encodeArray(object: Array<unknown>, depth: number) {
@@ -290,7 +291,7 @@ export class Encoder {
       throw new Error(`Too large extension object: ${size}`);
     }
     this.writeI8(ext.type);
-    this.writeU8v(...ext.data);
+    this.writeU8a(ext.data);
   }
 
   writeU8(value: number) {
@@ -300,6 +301,9 @@ export class Encoder {
     this.pos++;
   }
 
+  /**
+   * @deprecated No longer used. Will be removed in a semver-major
+   */
   writeU8v(...values: Array<number>) {
     const size = values.length;
     this.ensureBufferSizeToWrite(size);
@@ -311,6 +315,14 @@ export class Encoder {
     this.pos += size;
   }
 
+  writeU8a(values: ArrayLike<number>) {
+    const size = values.length;
+    this.ensureBufferSizeToWrite(size);
+
+    this.bytes.set(values, this.pos);
+    this.pos += size;
+  }
+
   writeI8(value: number) {
     this.ensureBufferSizeToWrite(1);
 

test/msgpack-test-suite.test.ts

@@ -96,7 +96,8 @@ describe("msgpack-test-suite", () => {
       STR_INCLUDING_NUL: "foo\0bar\0",
       STR_BROKEN_FF: "\xff",
       BIN16: new Uint8Array(0x100).fill(0xff),
-      BIN32: new Uint8Array(0x10000).fill(0xff),
+      BIN32: new Uint8Array(0x10_000).fill(0xff),
+      BIN32LARGE: new Uint8Array(0x100_000).fill(0xff), // regression: caused "RangeError: Maximum call stack size exceeded"
       ARRAY16: new Array<boolean>(0x100).fill(true),
       ARRAY32: new Array<boolean>(0x10000).fill(true),
       MAP16: new Array<null>(0x100).fill(null).reduce<Record<string, number>>((acc, _val, i) => {