Remove buffer-builder dependency (#416)

The buffer-builder dependency has not been updated in 11 years.
That might be fine as it doesn't do a whole lot, but we're
not using much of it anyway, and it does use a deprecated API
(although it does so safely).

Author: Trott

lib/src/packet-transformer.ts

@@ -4,7 +4,6 @@
 
 import {Observable, Subject} from 'rxjs';
 import {mergeMap} from 'rxjs/operators';
-import BufferBuilder = require('buffer-builder');
 
 /**
  * Decodes arbitrarily-chunked buffers, for example
@@ -57,18 +56,20 @@ export class PacketTransformer {
 
       // Write the length in varint format, 7 bits at a time from least to most
       // significant.
-      const header = new BufferBuilder(8);
+      const header = Buffer.alloc(8);
+      let offset = 0;
       while (length > 0) {
         // The highest-order bit indicates whether more bytes are necessary to
         // fully express the number. The lower 7 bits indicate the number's
         // value.
-        header.appendUInt8((length > 0x7f ? 0x80 : 0) | (length & 0x7f));
+        header.writeUInt8((length > 0x7f ? 0x80 : 0) | (length & 0x7f), offset);
+        offset++;
         length >>= 7;
       }
 
-      const packet = Buffer.alloc(header.length + protobuf.length);
-      header.copy(packet);
-      packet.set(protobuf, header.length);
+      const packet = Buffer.alloc(offset + protobuf.length);
+      header.copy(packet, 0, 0, offset);
+      packet.set(protobuf, offset);
       this.writeInboundBuffer(packet);
     } catch (error) {
       this.outboundProtobufsInternal$.error(error);

package.json

@@ -62,7 +62,6 @@
   },
   "dependencies": {
     "@bufbuild/protobuf": "^2.5.0",
-    "buffer-builder": "^0.2.0",
     "colorjs.io": "^0.5.0",
     "immutable": "^5.0.2",
     "rxjs": "^7.4.0",