cborg 4.5.6 → 4.5.8

package/CHANGELOG.md

@@ -1,3 +1,12 @@
+## [4.5.8](https://github.com/rvagg/cborg/compare/v4.5.7...v4.5.8) (2026-01-21)
+
+### Bug Fixes
+
+* **cli:** don't omit array length bytes in diagnostic output ([b8dacb6](https://github.com/rvagg/cborg/commit/b8dacb652c02566c7b64c2638b7bcb36bb519b6b))
+* **test:** add CLI tests to `npm test` ([b19bc87](https://github.com/rvagg/cborg/commit/b19bc87ea04691f9173a5b13284e21192e0f1e3c))
+
+## [4.5.7](https://github.com/rvagg/cborg/compare/v4.5.6...v4.5.7) (2026-01-21)
+
 ## [4.5.6](https://github.com/rvagg/cborg/compare/v4.5.5...v4.5.6) (2026-01-21)
 
 ## [4.5.5](https://github.com/rvagg/cborg/compare/v4.5.4...v4.5.5) (2026-01-20)

package/lib/byte-utils.js

@@ -33,13 +33,19 @@ export function asU8A (buf) {
   return isBuffer(buf) ? new Uint8Array(buf.buffer, buf.byteOffset, buf.byteLength) : buf
 }
 
+// Threshold for manual UTF-8 encoding vs native methods.
+// Node.js Buffer.from: crossover ~24 chars
+// Browser TextEncoder: crossover ~200 chars
+const FROM_STRING_THRESHOLD_BUFFER = 24
+const FROM_STRING_THRESHOLD_TEXTENCODER = 200
+
 export const fromString = useBuffer
   ? // eslint-disable-line operator-linebreak
     /**
      * @param {string} string
      */
     (string) => {
-      return string.length > 64
+      return string.length >= FROM_STRING_THRESHOLD_BUFFER
         ? // eslint-disable-line operator-linebreak
       // @ts-ignore
         globalThis.Buffer.from(string)
@@ -51,7 +57,7 @@ export const fromString = useBuffer
      * @param {string} string
      */
     (string) => {
-      return string.length > 64 ? textEncoder.encode(string) : utf8ToBytes(string)
+      return string.length >= FROM_STRING_THRESHOLD_TEXTENCODER ? textEncoder.encode(string) : utf8ToBytes(string)
     }
 
 /**

package/lib/diagnostic.js

@@ -47,9 +47,16 @@ function * tokensToDiagnostic (inp, width = 100) {
       case 'bytes':
       case 'map':
       case 'array':
-        // for bytes and string, we want to print out the length part of the value prefix if it
-        // exists - it exists for short lengths (<24) but does for longer lengths
-        multilen = token.type.name === 'string' ? utf8Encoder.encode(token.value).length : token.value.length
+        // print the length bytes if they exist (length >= 24)
+        // for string/bytes, token.value is the content; for array/map, token.value IS the count
+        if (token.type.name === 'string') {
+          multilen = utf8Encoder.encode(token.value).length
+        } else if (token.type.name === 'bytes') {
+          multilen = token.value.length
+        } else {
+          // array or map - value is the count directly
+          multilen = token.value
+        }
         if (multilen >= uintBoundaries[0]) {
           if (multilen < uintBoundaries[1]) {
             outp += ` ${slc(1, 1)}`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cborg",
-  "version": "4.5.6",
+  "version": "4.5.8",
   "description": "Fast CBOR with a focus on strictness",
   "main": "cborg.js",
   "type": "module",
@@ -13,8 +13,9 @@
     "build:types": "tsc --build",
     "prepublishOnly": "npm run build",
     "test:node": "c8 --check-coverage --exclude=test/** mocha test/test-*.js",
+    "test:node-bin": "mocha test/node-test-bin.js",
     "test:browser": "polendina --cleanup test/test-*.js",
-    "test": "npm run lint && npm run build && npm run test:node && npm run test:browser",
+    "test": "npm run lint && npm run build && npm run test:node && npm run test:node-bin && npm run test:browser",
     "test:ci": "npm run test",
     "coverage": "c8 --reporter=html --reporter=text mocha test/test-*.js && npx st -d coverage -p 8888"
   },

package/test/node-test-bin.js

@@ -97,17 +97,17 @@ describe('Bin', () => {
       assert.strictEqual(e.stderr,
 `Usage: cborg <command> <args>
 Valid commands:
-\tbin2diag [binary input]
+\tbin2diag [--width <width>] [binary input]
 \tbin2hex [binary input]
 \tbin2json [--pretty] [binary input]
 \tdiag2bin [diagnostic input]
 \tdiag2hex [diagnostic input]
 \tdiag2json [--pretty] [diagnostic input]
 \thex2bin [hex input]
-\thex2diag [hex input]
+\thex2diag [--width <width>] [hex input]
 \thex2json [--pretty] [hex input]
 \tjson2bin '[json input]'
-\tjson2diag '[json input]'
+\tjson2diag [--width <width>] '[json input]'
 \tjson2hex '[json input]'
 Input may either be supplied as an argument or piped via stdin
 `)
@@ -124,17 +124,17 @@ Input may either be supplied as an argument or piped via stdin
 `Unknown command: 'blip'
 Usage: cborg <command> <args>
 Valid commands:
-\tbin2diag [binary input]
+\tbin2diag [--width <width>] [binary input]
 \tbin2hex [binary input]
 \tbin2json [--pretty] [binary input]
 \tdiag2bin [diagnostic input]
 \tdiag2hex [diagnostic input]
 \tdiag2json [--pretty] [diagnostic input]
 \thex2bin [hex input]
-\thex2diag [hex input]
+\thex2diag [--width <width>] [hex input]
 \thex2json [--pretty] [hex input]
 \tjson2bin '[json input]'
-\tjson2diag '[json input]'
+\tjson2diag [--width <width>] '[json input]'
 \tjson2hex '[json input]'
 Input may either be supplied as an argument or piped via stdin
 `)
@@ -147,17 +147,17 @@ Input may either be supplied as an argument or piped via stdin
     assert.strictEqual(stderr,
 `Usage: cborg <command> <args>
 Valid commands:
-\tbin2diag [binary input]
+\tbin2diag [--width <width>] [binary input]
 \tbin2hex [binary input]
 \tbin2json [--pretty] [binary input]
 \tdiag2bin [diagnostic input]
 \tdiag2hex [diagnostic input]
 \tdiag2json [--pretty] [diagnostic input]
 \thex2bin [hex input]
-\thex2diag [hex input]
+\thex2diag [--width <width>] [hex input]
 \thex2json [--pretty] [hex input]
 \tjson2bin '[json input]'
-\tjson2diag '[json input]'
+\tjson2diag [--width <width>] '[json input]'
 \tjson2hex '[json input]'
 Input may either be supplied as an argument or piped via stdin
 `)
@@ -341,6 +341,23 @@ Input may either be supplied as an argument or piped via stdin
   })
 
   describe('diag length bytes', () => {
+    // issue #137 - array and map length bytes were missing for lengths >= 24
+    it('array with 24 elements', async () => {
+      // 98 18 = array(24), then 24 uint(1) values
+      const hex = '9818' + '01'.repeat(24)
+      const { stdout, stderr } = await execBin(`hex2diag ${hex}`)
+      assert.strictEqual(stderr, '')
+      assert.ok(stdout.startsWith('98 18                                             # array(24)\n'))
+    })
+
+    it('map with 24 entries', async () => {
+      // b8 18 = map(24), then 24 pairs of string('a')=uint(1)
+      const hex = 'b818' + '616101'.repeat(24)
+      const { stdout, stderr } = await execBin(`hex2diag ${hex}`)
+      assert.strictEqual(stderr, '')
+      assert.ok(stdout.startsWith('b8 18                                             # map(24)\n'))
+    })
+
     it('compact', async () => {
       const { stdout, stderr } = await execBin('json2diag', '"aaaaaaaaaaaaaaaaaaaaaaa"')
       assert.strictEqual(stderr, '')

package/types/lib/byte-utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"byte-utils.d.ts","sourceRoot":"","sources":["../../lib/byte-utils.js"],"names":[],"mappings":"AAuBA;;;GAGG;AACH,2BAHW,UAAU,GAAC,MAAM,EAAE,GACjB,UAAU,CAQtB;AAoMD;;;;GAIG;AACH,4BAJW,UAAU,MACV,UAAU,GACR,MAAM,CAgBlB;AA6CD;;;GAGG;AACH,kDAHW,MAAM,EAAE,GACN,MAAM,CAkBlB;AArTD,gCAMkD;AAyBlD,mCAGe,MAAM,iDAYN,MAAM,yCAIhB;AAOE,+BAHI,MAAM,EAAE,GACN,UAAU,CAItB;AAIG;;;;GAIG;AAEH,6BALW,UAAU,SACV,MAAM,OACN,MAAM,2BAQhB;AAcD;;;;GAIG;AACH,+BAJW,UAAU,EAAE,UACZ,MAAM,GACJ,UAAU,CActB;AAwBD;;;GAGG;AACH,4BAHW,MAAM,GACJ,UAAU,CAMtB;AAaD;;;GAGG;AACH,yBAHW,UAAU,GACR,MAAM,CAQlB;AAiBH;;;GAGG;AACD,6BAHS,MAAM,GAAC,UAAU,GACf,UAAU,CAQpB"}
+{"version":3,"file":"byte-utils.d.ts","sourceRoot":"","sources":["../../lib/byte-utils.js"],"names":[],"mappings":"AAuBA;;;GAGG;AACH,2BAHW,UAAU,GAAC,MAAM,EAAE,GACjB,UAAU,CAQtB;AA0MD;;;;GAIG;AACH,4BAJW,UAAU,MACV,UAAU,GACR,MAAM,CAgBlB;AA6CD;;;GAGG;AACH,kDAHW,MAAM,EAAE,GACN,MAAM,CAkBlB;AA3TD,gCAMkD;AA+BlD,mCAGe,MAAM,iDAYN,MAAM,yCAIhB;AAOE,+BAHI,MAAM,EAAE,GACN,UAAU,CAItB;AAIG;;;;GAIG;AAEH,6BALW,UAAU,SACV,MAAM,OACN,MAAM,2BAQhB;AAcD;;;;GAIG;AACH,+BAJW,UAAU,EAAE,UACZ,MAAM,GACJ,UAAU,CActB;AAwBD;;;GAGG;AACH,4BAHW,MAAM,GACJ,UAAU,CAMtB;AAaD;;;GAGG;AACH,yBAHW,UAAU,GACR,MAAM,CAQlB;AAiBH;;;GAGG;AACD,6BAHS,MAAM,GAAC,UAAU,GACf,UAAU,CAQpB"}

package/types/lib/diagnostic.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"diagnostic.d.ts","sourceRoot":"","sources":["../../lib/diagnostic.js"],"names":[],"mappings":"AAOA;;;GAGG;AACH,wCAHW,UAAU,UACV,MAAM,oCA4HhB;AAED;;;;GAIG;AACH,gCAHW,MAAM,GACJ,UAAU,CAatB"}
+{"version":3,"file":"diagnostic.d.ts","sourceRoot":"","sources":["../../lib/diagnostic.js"],"names":[],"mappings":"AAOA;;;GAGG;AACH,wCAHW,UAAU,UACV,MAAM,oCAmIhB;AAED;;;;GAIG;AACH,gCAHW,MAAM,GACJ,UAAU,CAatB"}