fix: sort dictionary keys bytewise per BEP 3

lib/encode.js

@@ -74,13 +74,23 @@ encode.number = function (buffers, data) {
   }
 }
 
+function compareKeysByBytes (a, b) {
+  const ab = ArrayBuffer.isView(a) ? a : text2arr(String(a))
+  const bb = ArrayBuffer.isView(b) ? b : text2arr(String(b))
+  const length = Math.min(ab.length, bb.length)
+  for (let i = 0; i < length; i++) {
+    if (ab[i] !== bb[i]) return ab[i] - bb[i]
+  }
+  return ab.length - bb.length
+}
+
 encode.dict = function (buffers, data) {
   buffers.push(buffD)
 
   let j = 0
   let k
   // fix for issue #13 - sorted dicts
-  const keys = Object.keys(data).sort()
+  const keys = Object.keys(data).sort(compareKeysByBytes)
   const kl = keys.length
 
   for (; j < kl; j++) {
@@ -96,7 +106,7 @@ encode.dict = function (buffers, data) {
 encode.dictMap = function (buffers, data) {
   buffers.push(buffD)
 
-  const keys = Array.from(data.keys()).sort()
+  const keys = Array.from(data.keys()).sort(compareKeysByBytes)
 
   for (const key of keys) {
     if (data.get(key) == null) continue

test/encode.test.js

@@ -22,6 +22,14 @@ test('bencode#encode()', function (t) {
     t.equal(Buffer.from(bencode.encode(data)).toString(), 'd7:integeri12345e6:string11:Hello Worlde')
   })
 
+  t.test('should sort dictionary keys bytewise per BEP 3, including astral keys', function (t) {
+    t.plan(1)
+    // '\uE000' (UTF-8 ee 80 80) must sort before '\u{1F600}' (UTF-8 f0 9f 98 80),
+    // even though the astral key's leading UTF-16 surrogate (d83d) sorts first.
+    const data = { '\u{1F600}': 2, '\uE000': 1 }
+    t.equal(Buffer.from(bencode.encode(data)).toString('hex'), '64333aee8080693165343af09f988069326565')
+  })
+
   t.test('should force keys to be strings', function (t) {
     t.plan(1)
     const data = {